Async middleware not calling again if returning to original page

Environment

Operating System: Darwin
Node Version:     v16.16.0
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
Create a middleware something like this:
import { defineNuxtRouteMiddleware, useNuxtApp } from '#app'

export default defineNuxtRouteMiddleware(async (to) => {
  const { $cwa } = useNuxtApp()

  if (to.meta.cwa === false) {
    return
  }

  await $cwa.fetcher.fetchRoute(to.path)
})

Describe the bug
Let's say I'm on the page /contact and this has loaded fine.
I then click to /about and the route begins to load. The async promise is not resolve yet, but then I click back to /contact
When I click back to /contact this middleware is not fired again.
Additional context
In my case, I am doing some API calls and setting which resources are currently available on the page.
So what happens is it starts the API calls to get the resources of /about and then when I go back to /contact the middleware is not fired again, and so the API continues to fetch and populate for /about even though I am now on /contact
Logs
No response
EDIT:
here is an easier reproduction of middleware:
import { defineNuxtRouteMiddleware } from '#app'

export default defineNuxtRouteMiddleware((to) => {
  console.log(`Middleware triggered for ${to.path}`)
  return new Promise((resolve) => {
    setTimeout(() => {
      console.log(`Middleware resolved for ${to.path}`)
      resolve(true)
    }, 1000)
  })
})

When changing pages back and forth quickly you'll see middleware is not fired again for the original page

Hi, sorry I put in the wrong repo. Did you happen to have any thoughts on this?

Would you see if you can replicate it in https://stackblitz.com/github/nuxt-contrib/vue3-ssr-starter/tree/main?terminal=dev ? If so, would you raise it as an issue in https://github.com/vuejs/router instead?

Hi @danielroe - sorry for the delay, I'll try to get a repro together, but I'm not sure if it's really a vue-router bug. It's more of the functionality we are using of vue-router in Nuxt perhaps doesn't meet the spec. I outlined in a comment above how the behaviour is as vue-router describes it to be - but if creating async middleware, it could have some unintended effects, especially if, as in my case, we are using the middleware to make calls to load the data for the page to be displayed.

I can't reproduce with your original example. If you're still experiencing this, would you create a runnable reproduction? 🙏

Sure, I'll go and have a play, I have a feeling it wouldn't have been resolved -  I'll get something ready within a week all being well or confirm if it is fixed otherwise :)


Improve alias of command

1. Add generic to CommandForm for Alias
Generic is necessary when using CommandForm either for TraitAlais or NonTraitAlias.  If it is not generic, the following snippet will be in compile error:
        TraitAlias airConditionerAlias = new TraitAlias("AirConditionerAlias");
        List<Action> actions = new ArrayList<>();
        TurnPower turnPower = new TurnPower(true);
        actions.add(turnPower);
        List<Pair<TraitAlias, List<Action>>> traitActions = new ArrayList<>();
        traitActions.add(new Pair<>(airConditionerAlias, actions));
        CommandForm form = new CommandForm(traitActions); // error here. CommandForm only accept alias

2. Add constructor for NonTraitAlias
NonTraitAlias needs a public constructor.

@yongpingchen LGTM


Add MakeMyTrip and Hotstar

Added "MakeMyTrip"  as easy and "Hotstar" as medium.

Thanks for your contribution!


add maven proxy command

Codecov Report

Merging #138 (5f54e8b) into main (dba11a0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #138   +/-   ##
=========================================
  Coverage     22.74%   22.74%
  Complexity      160      160
=========================================
  Files            87       87
  Lines          1495     1495
  Branches         92       92
=========================================
  Hits            340      340
  Misses         1139     1139
  Partials         16       16

Flag
Coverage Δ

unittests
22.74% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 dba11a0...5f54e8b. Read the comment docs.


Crashes on regex find and replace

On using the below regex (without quotations) in find and replace box, vscode just freezes and and gives a prompt to reopen or close a few seconds after

"/(.+)/"

VSCode version - 1.36.1
OS - Windows 10 1803 x64

I am not able reproduce the issue, Please provide more details or steps to reproduce it.

This regex will suffer from catastrophic backtracking, https://github.com/microsoft/vscode/issues/30874


aeron: 1.40.0: Aeron messaging system

Description of changes
https://github.com/real-logic/aeron
Efficient reliable UDP unicast, UDP multicast, and IPC message transport.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1970

Can you tell us how to test this PR?  I am unfamiliar with this server/IPC protocol, and I don't know of any program that uses it.
nixpkgs-review builds the package successfully.

Can you tell us how to test this PR? I am unfamiliar with this server/IPC protocol, and I don't know of any program that uses it.

What would be the approach for testing, say, kafka, which is a fairly similar service? There are test binaries in the upstream distribution, is it worth invoking those in the check phase, or just running "prog --version" to ensure that the binaries are complete and valid?


LCARS doesn't work anymore with up to date Home Assistant

Hi,
I was just playing around trying to create a nice LCARS Dashboard and I found out this bug:
I type some correct code, the preview is as I expected, then when I save the card, the actual output is different.
Rebooting doesn't fix the issue.
I'll attach a short video to show you what I mean.
Everything is up to date, anyway these are the version I'm using.
Core 2024.10.2
Supervisor 2024.10.0
Operating System 13.1
Frontend 20241002.3
LCARS 3.0.0
https://github.com/user-attachments/assets/37e2157e-c147-4527-ad6d-63aaeb5b1c7b

Same problem here: only the classes: middle-left, middle-right & middle-contained are working.
But no header or footer classes in combination with type: markdown. I also can see that it is sometimes working in preview window at typing YAML. But after saving, only in rare cases and after reloading the page it is gone.
If you use another type than "markdown" the footer & header classes are working, but that is what nobody needs ;)

I found out get it working with type:custom:html-card but not type:markdown.

What bad luck! You've chosen the two cards that no longer work reliably with card-mod!
Unfortunately, this is a card-mod problem and not something I can fix:
https://github.com/thomasloven/lovelace-card-mod/issues/392
https://github.com/thomasloven/lovelace-card-mod/issues/386
Fortunately, there are alternative cards to do the same kinds of things:

To replace markdown card: https://github.com/PiotrMachowski/lovelace-html-card
To replace the weather card: https://github.com/bramkragten/weather-card

weather-card is a much better option in my opinion anyway.

Thank you.
I'll try to replace cards, unfortunately I'm not really good at this, so I'll probably end up waiting for a guide or updated instructions from someone!
LCARS is great, thank you for your work.


Is there a setting to change doubles to decimals?

I am trying to generate a CSharp Client from a swagger URL online using the NSwagStudio.
Some of the definitions look like this:
"initialInvestment": {
     "type": "number",
    "format": "double",
    "description": "The minimum initial investment required to purchase the fund"
},

It returns money as doubles, but I know I should be using decimals for money. Is there a setting in here somewhere to generate my POCOs using decimals any time it sees a type number with format double?

There is no setting for that i think. You’d need to preprocess the spec (eg with JsonSchemaVisitor in NJsonSchema)

Gotcha, I saw a lot of options for primitive types, just seemed odd that it was such an exclusive list. Why not have all the primitive types?

Here you can only specify types which cannot be determined from the spec or are not well defined in Swagger... in your case it clearly states double and if you need decimal then the spec is “wrong”. You could also write a custom csharp type resolver but then you cannot use it via cli/studio.

if you need decimal then the spec is “wrong”.

Maybe slightly off topic, but the API I am using returns doubles even for money, and I am not sure why. Everywhere I look it is stressed that if you deal with money in your app you should used the highest precision that the language offers (in this case a decimal)
Knowing that info, I assumed I should force over all double's to decimals right away, otherwise I will have to cast everything as the deeper layers of my app start interacting with the payload from the server.
Maybe I am better off adding a mapping layer between the API deserialization classes so I get DTOs in the format I need instead? Or maybe I should keep them as doubles and continue to cast them where needed? Is it possible that casting doubles to decimals can throw everything off?

You should directly generate decimals the because converting from string to double to decimal might lose precision...
Fo now the simplest solution is to write an own cli tool with NSwag.Core (SwaggerDocument.FromJson/ToJson) and write a simple schema visitor to change to decimal:
https://github.com/RSuter/NJsonSchema/blob/master/src/NJsonSchema/Visitors/JsonSchemaVisitorBase.cs

See https://github.com/Picturepark/Picturepark.SDK.Playground/blob/master/src/AutoRestTransformer/Program.cs

@RSuter Slowly working my way through this. As a noob i found a lot of confusion on when to use chunks of NJsonSchema vs when to use NSwag. Looks like NSwag is a fancy wrapper for NJsonSchema. And the example you provided used JsonSchemaVisitor but I found that I needed JsonSchemaVisitorBase instead.
Anyways, since I am now having to drop down low level and use a visitor, I also can no longer use the GUI which sucks. Because of this I am having to go through the UI and find the options that match the items in SwaggerToCSharpClientGeneratorSettings. I found some stuff hard to find like how can I disable annotation attributes? and FromJsonToJson? There does not seem to be an option in SwaggerToCSharpClientGeneratorSettings or CSharpClientGeneratorSettings. But its here:

I think I found some here https://github.com/RSuter/NJsonSchema/blob/master/src/NJsonSchema.CodeGeneration.CSharp/CSharpGeneratorSettings.cs

You could just write the cli so that it reads and writes a file and then use the new file in the ui

That is a good point. In the end, this is what I arrived at.
async void Main()
{
	// Fetch the Swagger JSON
	var client = new HttpClient();
	HttpResponseMessage response = await client.GetAsync("https://apisb.etrade.com/docs/api/market/quotesjson/swagger.json");
	response.EnsureSuccessStatusCode();
	string responseBody = await response.Content.ReadAsStringAsync();

	await RunAsync(responseBody);
}

private static async Task RunAsync(string jsonString)
{
	var document = await SwaggerDocument.FromJsonAsync(jsonString);

	var visitor = new DoubleToDecimalVisitor();
	await visitor.VisitAsync(document);

	await SwaggerDocument.FromJsonAsync(jsonString);

	var settings = new SwaggerToCSharpClientGeneratorSettings()
	{
		GenerateExceptionClasses = false,
		GenerateClientClasses = false,
		ParameterArrayType = "System.Collections.Generic.List",
		CSharpGeneratorSettings =
		{
			GenerateJsonMethods = false,
			GenerateDataAnnotations = false
		}
	};

	var generator = new SwaggerToCSharpClientGenerator(document, settings);

	var code = generator.GenerateFile();

	code.Dump();

}

public class DoubleToDecimalVisitor : JsonSchemaVisitorBase
{
	protected override Task<JsonSchema4> VisitSchemaAsync(JsonSchema4 schema, string path, string typeNameHint)
	{
		if (schema is SwaggerParameter || schema is JsonProperty)
		{
			if (schema.Type == JsonObjectType.Number)
			{
				schema.Format = JsonFormatStrings.Decimal;
			}
		}
		return Task.FromResult(schema);
	}
}

Looks perfect (except an unused await SwaggerDocument.FromJsonAsync(jsonString);). When using the toolchain this way, you have also many more extension points available.

As a noob i found a lot of confusion on when to use chunks of NJsonSchema vs when to use NSwag. Looks like NSwag is a fancy wrapper for NJsonSchema. And the example you provided used JsonSchemaVisitor but I found that I needed JsonSchemaVisitorBase instead.

A Swagger document mainly contains HTTP operation descriptions and DTO descriptions. The DTOs are described as some variant of JSON Schema. All JSON Schema/DTO handling (from .NET and to C#/TypeScript) is implemented in NJS and only the Swagger specific stuff in NSwag. So a lot of low level stuff (e.g. $ref handling, etc.) is handled in NJS because it is not Swagger but more JSON (Schema) specific.

Anyways, since I am now having to drop down low level and use a visitor, I also can no longer use the GUI which sucks.

Yes, for the Swagger generators it is already possible to load such additional extensions in NSwagStudio/CLI (like plugins). This is not yet possible with the C#/TypeScript generator commands and you can always easily build your own CLI.

Thanks for taking the time to help me with this.

Sure, you're welcome.
I'd really like some concept of preprocessors/transformers which can be added to the pipeline (and e.g. loaded from a DLL or e.g. yours prepackaged). This one is also a possible candidate: https://github.com/RSuter/NJsonSchema/issues/17

Given that numbers are already converted to a decimal string format when serialising them as JSON across the wire, it would be preferable to have decimal as the default real number type in clients (or at the very least this should be easily configurable without custom code).
That is, although the spec format name is "double", in JSON or XML this necessarily means "decimal approximation of double", so by converting on the client side back to double you're only approximating an approximation. Having only one level of approximation, taking the decimal conversion, would be better. At the moment I'm using just decimals on the server side anyway (no approximation) and I'm having to add in a custom pre-processor in every NSwag client to avoid the imprecision.

If you have decimals in your server models and generate schemas from it with nswag/njs the are described as number+decimal and generated as decimals in the code generator, right?

@RicoSuter I think he means when creating a Client from an existing Swagger document that describes the number type as number only (no +decimal).

Yes sorry this is given existing server-side specs which aren't generated by NSwag. Did you say NSwag can generate specs with a 'number' type and 'decimal' format? I wasn't aware that was a supported format in OpenApi?
Also just to clarify on my previous comment, this only applies to media formats where number types are necessarily decimal, eg. JSON and XML. So to refine the proposal, it would be good if there was either an option for NSwag to infer the number type by the media type (something like UseMediaNativeNumberType), or a global override option for the number type like the primitive types above. (The latter would be less preferable though as then services which can return either JSON or a media format supporting native doubles would have one coerced to the wrong type.)

i don't think open api defines a decimal format https://swagger.io/docs/specification/data-models/data-types/#numbers
it does seem like a prudent thing to turn open-api number double formats into decimal type in .net as needed

i don't think open api defines a decimal format https://swagger.io/docs/specification/data-models/data-types/#numbers
it does seem like a prudent thing to turn open-api number double formats into decimal type in .net as needed

I Agree.
There's most common use to map double to decimal in .Net.

i don't think open api defines a decimal format https://swagger.io/docs/specification/data-models/data-types/#numbers
it does seem like a prudent thing to turn open-api number double formats into decimal type in .net as needed

I Agree.
There's most common use to map double to decimal in .Net.

But by that point precision can already be lost.

i don't think open api defines a decimal format https://swagger.io/docs/specification/data-models/data-types/#numbers
it does seem like a prudent thing to turn open-api number double formats into decimal type in .net as needed

I Agree.
There's most common use to map double to decimal in .Net.

But by that point precision can already be lost.

Sorry but, why?
If I have a decimal request/response, it will not be lost in request/response's .NET pipeline. Am I missing some point here?
The point is: I have an Request with require a decimal input, wich open API translate as double in swagger.json, but, when nSwag generates the c# client's code, it put as double, not decimal, and in that way, yes, I'll loose precision because I'm handling as double until TCP transport.
So, how can It already be lost in that case?

i don't think open api defines a decimal format https://swagger.io/docs/specification/data-models/data-types/#numbers
it does seem like a prudent thing to turn open-api number double formats into decimal type in .net as needed

I Agree.
There's most common use to map double to decimal in .Net.

But by that point precision can already be lost.

Sorry but, why?
If I have a decimal request/response, it will not be lost in request/response's .NET pipeline. Am I missing some point here?
The point is: I have an Request with require a decimal input, wich open API translate as double in swagger.json, but, when nSwag generates the c# client's code, it put as double, not decimal, and in that way, yes, I'll loose precision because I'm handling as double until TCP transport.
So, how can It already be lost in that case?

on read from wire

I am using stoplight studio to create the spec for a new API and if I edit the json to define a property as:
          "amount": {
            "type": "number+decimal"
          },

then NSwagStudio v<IP_ADDRESS> generates:
        public object Amount { get; set; }

Please check your swagger document, I had the same problem,
"qty": {             "type": "number",             "format": "double",             "nullable": true  },
I am using .NET, so I just configured Swagger generator:
builder.Services.AddSwaggerGen(options => {     options.MapType<decimal>(() => new OpenApiSchema { Type = "number", Format = "decimal" }); });
and then output swagger document was like:
          "qty": {             "type": "number",             "format": "decimal",             "nullable": true           },
so NSwag was able to generate proper type:
        [System.Text.Json.Serialization.JsonIgnore(Condition = System.Text.Json.Serialization.JsonIgnoreCondition.WhenWritingDefault)]            public decimal? Qty { get; set; }

Has there been any progress with this issue yet? The above ("format": "decimal") isn't an acceptable solution because in general clients don't have control of the OpenAPI spec they're consuming. For the overwhelmingly common case of an external API which is agnostic to NSwag (and thus doesn't use NSwag's own special "decimal" number format) eg. if it's not implemented in .Net, and which uses JSON transport, there has to be a way to generate a client with number types that match their actual format in the transport - decimals. Without going through all the faff of building a bespoke NSwag CLI. IMO we need something like a UseMediaNativeNumberType option.
(To clarify by the way I'm @adamjones1 above with a new account.)


Remove rails_stdout_logging gem

No longer needed since Rails 5 includes Heroku's 12factor methodology

I'm surprised this was still there. Thanks! Will merge on green CI.

It's the power of being able to search the whole repo in a command!


PhpUnitDedicateAssertFixer - fix to assertCount too

closes #3750
            'assert same' => [
                '<?php $this->assertCount(1, $a);',
                '<?php $this->assertSame(1, count($a));',
            ],

for more example please see the tests

Thank you @SpacePossum.

For more the rest of advanced cases I'd like to refer great job by @carusogabriel in Rector: https://github.com/rectorphp/rector/blob/master/docs/AllRectorsOverview.md#rectorphpunitrectorspecificmethodassertnotoperatorrector
-$this->assertTrue($foo === $bar, "message");
+$this->assertSame($bar, $foo, "message");

-$this->assertFalse($foo >= $bar, "message");
+$this->assertLessThanOrEqual($bar, $foo, "message");

-$this->assertFalse(strpos($anything, "foo"), "message");
+$this->assertNotContains("foo", $anything, "message");

-$this->assertNotFalse(stripos($anything, "foo"), "message");
+$this->assertContains("foo", $anything, "message");

-$this->assertTrue(property_exists(new Class, "property"), "message");
+$this->assertClassHasAttribute("property", "Class", "message");

-$this->assertFalse(property_exists(new Class, "property"), "message");
+$this->assertClassNotHasAttribute("property", "Class", "message");

etc.

nice list, maybe someone can backport to hear :)


Allow nodes to be RCed and deallocated from the arena

Currently, Sapling will never free up space in the arena even if history is rewritten and some trees will never be reached.  This is essentially a memory leak, and will cause Sapling to just accumulate memory over time.
Some kind of garbage collection has to happen, because Sapling's arena allocator only produces immutable references to the nodes.  However, because of the tree structure of the AST, we can rely on AST nodes having no reference cycles and so standard reference counting should suffice to prevent memory leaks.  However, some custom drop code will be required (because we can't simply deallocate small parts of memory owned by the Arena - we need to notify the arena that this cell isn't being used).
So I think the solution is going to have to involve some kind of smart pointer that does reference counting but, instead of deallocating the memory it points to, instead notifies the Arena whenever the reference count hits 0.  Also, AST nodes in the arena currently sit inside Item structures (which in its own right probably should be called Cell), which allows us to add extra fields for things like RC and memoisation of expensive functions (string conversions, etc.).
My proposed solution would be something like having a Ref<'arena, Node> smart pointer, which contains an RCed reference to an arena::Item<Node>, which can then be deref-ed into just a plain &'arena Node.
Node memory management is definitely not something that I want to screw up, and I've never had to deal with smart pointers or RCing before so if someone who knows more is happy to implement this then feel free :).

@Kixunil would you be happy doing this?

I'm not very familiar with the internals yet and I'm lacking time for a bigger contribution lately. I can reconsider later.

OK that's fine - I was just wondering if you wanted too.... this will only really become useful when Sapling gets more functionality and people actually use it for extended periods of time, so there's not really much hurry to risk breaking things until needed.


Two units with identical configuration gives different results

I have two µINS3 device, one is the rugged version, one is in the EVB-2.
I have both connected via USB (the EVB-2 connected to the µINS3 port)

Identical parameters (except the slight difference in antenna position)
Identical distance between the antennas (1m)
Identical movements/conditions (both installed on a single wooden plate)
Identical firmware
Identical NTRIP connection

Regardless of that similarity, the EVB-2 gets the RTK fix or compassing fix, while the rugged µINS3 doesn't. I tried to get the fix on the µINS3 and couldn't while in the middle of the park with a clear sky.
What am I missing?
µINS3 config dump
DID_DEV_INFO: SN40840, Fw <IP_ADDRESS> r4, 2020-07-30 16:49:57, Proto <IP_ADDRESS>

Current flash config
cBrdConfig =<PHONE_NUMBER>
checksum =<PHONE_NUMBER>
gnssSatSigConst = 4924
gps1AntOffset[0] = 0
gps1AntOffset[1] = -0.524999976
gps1AntOffset[2] = 0
gps2AntOffset[0] = 0
gps2AntOffset[1] = 0.524999976
gps2AntOffset[2] = 0
gpsMinimumElevation = 0.261799395
gpsTimeSyncPeriodMs = 1000
insDynModel = 8
insOffset[0] = 0
insOffset[1] = 0
insOffset[2] = 0
insRotation[0] = 0
insRotation[1] = 0
insRotation[2] = 0
ioConfig = 4384
key = 26
lastLla[0] = 50.852697999999997
lastLla[1] = 4.3144577999999996
lastLla[2] = 72.048000000000002
lastLlaTimeOfWeekMs = 387231200
lastLlaUpdateDistance = 1000
lastLlaWeek = 2117
magDeclination = 0
magInclination = 0
refLla[0] = 50.852638310854672
refLla[1] = 4.3145834288500904
refLla[2] = 92.910544845275581
reserved = 0
RTKCfgBits = 524296
sensorConfig = 15
ser0BaudRate = 921600
ser1BaudRate = 921600
size = 240
startupGPSDtMs = 200
startupImuDtMs = 1
startupNavDtMs = 4
sysCfgBits = 4127
wheelConfig.bits = 0
wheelConfig.diameter = 0
wheelConfig.distance = 0
wheelConfig.e_i2l[0] = 0
wheelConfig.e_i2l[1] = 0
wheelConfig.e_i2l[2] = 0
wheelConfig.t_i2l[0] = 0
wheelConfig.t_i2l[1] = 0
wheelConfig.t_i2l[2] = 0
zeroVelOffset[0] = 0
zeroVelOffset[1] = 0
zeroVelOffset[2] = 0
zeroVelRotation[0] = 0
zeroVelRotation[1] = 0
zeroVelRotation[2] = 0

EVB-2 config dump
DID_DEV_INFO: SN40756, Fw <IP_ADDRESS> r4, 2020-07-30 16:49:57, Proto <IP_ADDRESS>

Current flash config
cBrdConfig =<PHONE_NUMBER>
checksum =<PHONE_NUMBER>
gnssSatSigConst = 4924
gps1AntOffset[0] = 0
gps1AntOffset[1] = 0.589999974
gps1AntOffset[2] = 0
gps2AntOffset[0] = 0
gps2AntOffset[1] = -0.409999996
gps2AntOffset[2] = 0
gpsMinimumElevation = 0.261799395
gpsTimeSyncPeriodMs = 1000
insDynModel = 8
insOffset[0] = 0
insOffset[1] = 0
insOffset[2] = 0
insRotation[0] = 0
insRotation[1] = 0
insRotation[2] = 0
ioConfig = 4384
key = 26
lastLla[0] = 50.852691299999996
lastLla[1] = 4.3146163
lastLla[2] = 86.962000000000003
lastLlaTimeOfWeekMs = 295462400
lastLlaUpdateDistance = 1000
lastLlaWeek = 2117
magDeclination = 0
magInclination = 0
refLla[0] = 50.852638310854672
refLla[1] = 4.3145834288500904
refLla[2] = 92.910544845275581
reserved = 0
RTKCfgBits = 524296
sensorConfig = 15
ser0BaudRate = 921600
ser1BaudRate = 921600
size = 240
startupGPSDtMs = 200
startupImuDtMs = 1
startupNavDtMs = 4
sysCfgBits = 4127
wheelConfig.bits = 0
wheelConfig.diameter = 0
wheelConfig.distance = 0
wheelConfig.e_i2l[0] = 0
wheelConfig.e_i2l[1] = 0
wheelConfig.e_i2l[2] = 0
wheelConfig.t_i2l[0] = 0
wheelConfig.t_i2l[1] = 0
wheelConfig.t_i2l[2] = 0
zeroVelOffset[0] = 0
zeroVelOffset[1] = 0
zeroVelOffset[2] = 0
zeroVelRotation[0] = 0
zeroVelRotation[1] = 0
zeroVelRotation[2] = 0

@AlexisTM - Your other question indicated you were trying to use Compassing and RTK positioning simultaneously. Is this the case in this setup? To discuss further could you send an email to<EMAIL_ADDRESS>

In this setup, it was either one or the other, the firmware [1.8.2] correctly disabled POSITION if I tried to enabled 0x09.
In this case, the RTKCfgBits are 0x80008

@siddirp
The µINS3 is from the µINS Dual GNSS Compass Development Kit

Are you seeing "Compassing Fix" status? Most likely you have interference
problems and aren't completely getting into a fixed state.
[image: Inertial Sense] http://www.inertialsense.com/
Andrew Priddis
Sr Systems Engineer
<EMAIL_ADDRESS>inertialsense.com http://www.inertialsense.com/
On Fri, Aug 7, 2020 at 1:35 AM Alexis Paques<EMAIL_ADDRESS>wrote:

@siddirp https://github.com/siddirp
The µINS3 is from the µINS Dual GNSS Compass Development Kit
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/inertialsense/inertial-sense-sdk/issues/33#issuecomment-670376336,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKPNQ7EBWID5HVMYPXJUHPDR7OVCLANCNFSM4PWWFAIA
.

Both the rugged unit and the EVB-2 should be having the same perturbations, yet no received satellite goes into Float/Fix state at all on the rugged unit, even in ideal situation (open-sky, open-field). I go not a single GPS signal to Float/Fix for both compassing and position mode on the rugged unit.

What does mems precision and calibration look like in 2020?
View in browser (https://info.inertialsense.com/e2t/sc2/Mm-nFbkB_0QW7cKx4l7tV6QgW7mFN9g8QqZWMW8hWtXD8QqH96f1HVPN203 )
Inertial Sense (https://info.inertialsense.com/e2t/sc2/Mm-nFbkB_0QW7cKx4l7tV6QgW7mFN9g8QqZWMW8hWtXD8QqH96f1HVPN213 )
The Tiny Sensor with Giant Capabilities
What could your device do with this kind of power?
IS Sensor Datasheet (https://info.inertialsense.com/e2t/sc2/Mm-nFbkB_0QW7cKx4l7tV6QgW7mFN9g8QqZWMW8hWtXD8QqH96f1HVPN223 )
IMU Datasheet
Your device deserves high precision and quick telemetry without being weighed down. Check out our Datasheet and see if our sensors meet your specifications.
Download Datasheet (https://info.inertialsense.com/e2t/sc2/Mm-nFbkB_0QW7cKx4l7tV6QgW7mFN9g8QqZWMW8hWtXD8QqH96f1HVPN233 )
"Inertial Sense has developed a disruptive technology. It is going to change everything about our industry."

AUVSI Judges, 2019

View Products (https://info.inertialsense.com/e2t/sc2/Mm-nFbkB_0QW7cKx4l7tV6QgW7mFN9g8QqZWMW8hWtXD8QqH96f1HVPN243 )
Schedule Meeting (https://info.inertialsense.com/e2t/sc2/Mm-nFbkB_0QW7cKx4l7tV6QgW7mFN9g8QqZWMW8hWtXD8QqH96f1HVPN253 )
Read Blog Posts (https://info.inertialsense.com/e2t/sc2/Mm-nFbkB_0QW7cKx4l7tV6QgW7mFN9g8QqZWMW8hWtXD8QqH96f1HVPN263 )
Inertial Sense, 384 S 400 W #125th, Lindon, UT 84042, USA,<PHONE_NUMBER>
Unsubscribe (https://info.inertialsense.com/hs/manage-preferences/unsubscribe-all?d=VnBg1c6WzZmnW2p76LK3-1rCSW3X_fKX21hrytW1Ztq4c1XmmZlW1Synkn3W3636W3HbJ363KbsvFW3R0gs93_-09-W47szjy45RlhdW3P28X24hCPvhW43T2P-7DvMj2W17f7hv8zMnRfW4Rc5GT6tf82_W4_ZJwY8YNKs3V28Q_q8Xp6wxMGKd_P20N5RW3PHcCm8jR_5YW3d2RgN2R0wzY4CplG1&v=3&_hsenc=p2ANqtz-_G8dTM7_HFH_QfvWhL2TyRlpfVdmR2lgsweaDN5JGDD-hR_GGefETepYRXvc1xK93ZpW4HmtXP3-bT_DHvQ3UY4LZ4X13d3MFOtN5YrzqaOySqS23OFo-xDWfiMCb74pfzruSxlQ231QtC2TYKkjE-g26HRw&_hsmi=86142041 )
Manage preferences (https://info.inertialsense.com/hs/manage-preferences/unsubscribe?d=VnBg1c6WzZmnW2p76LK3-1rCSW3X_fKX21hrytW1Ztq4c1XmmZlW1Synkn3W3636W3HbJ363KbsvFW3R0gs93_-09-W47szjy45RlhdW3P28X24hCPvhW43T2P-7DvMj2W17f7hv8zMnRfW4Rc5GT6tf82_W4_ZJwY8YNKs3V28Q_q8Xp6wxMGKd_P20N5RW3PHcCm8jR_5YW3d2RgN2R0wzY4CplG1&v=3&_hsenc=p2ANqtz-_G8dTM7_HFH_QfvWhL2TyRlpfVdmR2lgsweaDN5JGDD-hR_GGefETepYRXvc1xK93ZpW4HmtXP3-bT_DHvQ3UY4LZ4X13d3MFOtN5YrzqaOySqS23OFo-xDWfiMCb74pfzruSxlQ231QtC2TYKkjE-g26HRw&_hsmi=86142041 )

Alexis.  Have you made any progress on this issue?

On the last in-field tests, I finally got similar results but the ambiguity resolution ratio never went higher than 2 or so for both units.

I suspect there is some sort of interference.  This can be due to mutli-pathing of the satellite signals or some other noise source that is impacting the RTK solution.  Are you using GPS antenna ground planes?  What is the distance between your antennas?

I am not currently using a ground plane, I will add one on each antenna.
The distance between antennas is 1m for both.

What does mems precision and calibration look like in 2020?
Inertial Sense (https://info.inertialsense.com/e2t/sc2/MmZ-8yjZ5PLW8YKH3_7DqBsHW6xgdls7sc4ZSV28PGW7Zp2pWdBzP8M04 )
The Guide to Choosing A Sensor
Which sensor make sense for your project?
Sensor Buying Cheat Sheet (https://info.inertialsense.com/e2t/sc2/MmZ-8yjZ5PLW8YKH3_7DqBsHW6xgdls7sc4ZSV28PGW7Zp2pWdBzP8N04 )
Sensor Buying Cheat Sheet
At this point, you might be wondering where to start when shopping for a sensor for your project. Deciding between an IMU, AHRS, or GNNS & RTK can be difficult. We've made this handy guide to help answer some basic questions quickly.
Read Blog Post (https://info.inertialsense.com/e2t/sc2/MmZ-8yjZ5PLW8YKH3_7DqBsHW6xgdls7sc4ZSV28PGW7Zp2pWdBzP8P04 )
"Inertial Sense has developed a disruptive technology. It is going to change everything about our industry."

AUVSI Judges, 2019

View Products (https://info.inertialsense.com/e2t/sc2/MmZ-8yjZ5PLW8YKH3_7DqBsHW6xgdls7sc4ZSV28PGW7Zp2pWdBzP8Q04 )
Schedule Meeting (https://info.inertialsense.com/e2t/sc2/MmZ-8yjZ5PLW8YKH3_7DqBsHW6xgdls7sc4ZSV28PGW7Zp2pWdBzP8R04 )
Read Blog Posts (https://info.inertialsense.com/e2t/sc2/MmZ-8yjZ5PLW8YKH3_7DqBsHW6xgdls7sc4ZSV28PGW7Zp2pWdBzP8S04 )
Inertial Sense, 384 S 400 W #125th, Lindon, UT 84042, USA,<PHONE_NUMBER>
Unsubscribe (https://info.inertialsense.com/hs/manage-preferences/unsubscribe-all?d=VnzSyB6WzZmnW2p76LK3-1rCSW3X_fKX21hrytW1Ztq4c1XmmZlW1Synkn3W3636W3HbJ363KbsvFW3R0gs93_-09-W47szjy45RlhdW3P28X24hCPvhW43T2P-7DvMj2W15gBJz5wqwC8W64QbLK4RszpdW505_f17XxhwGVWzpl95Sb1gvMXk_7QbwcydW8d8krT3fLFQvW2DzBP36Z0BCz3pWY2&v=3&_hsenc=p2ANqtz-_f4UQozYw1EWzpvuOgweL1o_2wir4tOAN9d0XfrQMkpe8lVIivgFYsQQ2Tmemkx7ecZz5FPmZFif2TPybrA9ULylL1mqZ2i0iovLaD32snsOQMXI6j4tG_jSNjlkwqJozAnAJ9PrB9FVwH5t8P51RFWqcpcw&_hsmi=86142139 )
Manage preferences (https://info.inertialsense.com/hs/manage-preferences/unsubscribe?d=VnzSyB6WzZmnW2p76LK3-1rCSW3X_fKX21hrytW1Ztq4c1XmmZlW1Synkn3W3636W3HbJ363KbsvFW3R0gs93_-09-W47szjy45RlhdW3P28X24hCPvhW43T2P-7DvMj2W15gBJz5wqwC8W64QbLK4RszpdW505_f17XxhwGVWzpl95Sb1gvMXk_7QbwcydW8d8krT3fLFQvW2DzBP36Z0BCz3pWY2&v=3&_hsenc=p2ANqtz-_f4UQozYw1EWzpvuOgweL1o_2wir4tOAN9d0XfrQMkpe8lVIivgFYsQQ2Tmemkx7ecZz5FPmZFif2TPybrA9ULylL1mqZ2i0iovLaD32snsOQMXI6j4tG_jSNjlkwqJozAnAJ9PrB9FVwH5t8P51RFWqcpcw&_hsmi=86142139 )

@AlexisTM Hi Alex.  Did this get resolved?  Did you add antenna ground planes fix your problem?

I added the ground planes, which indeed improved the GPS signal.
Yet, I could not get an RTK fix nor robust RTK compassing, with the ambiguity resolution ratio never reaching over 1 or 2.
When compassing (baseline of 1m), the measured distance was oscillating from 0.6m to 1.8m.
Here-under, the top plots are from the µINS3 (top left) and EVB-2 (top right). The data comes from DID_INS_1 and is converted to from NED to ENU. The bottom (orange) plot is from our reference.

Here-under, the GPS_POS_1 position output from the EVB-2 (pink), the µINS3 (purple) and our reference (yellow) for the same track. The graph looks stretched on the x-axis as it is longitude & latitude.

We can see one big jumps of position in the estimation for the µINS3 and a very very bad estimation for the EVB-2.
It is to note that the yaw (heading) of the EVB-2 was far (20°) from the estimation from both the µINS, our reference and a competitor with at worse almost 30° off. This can easily explain the bad EKF output.

Hi @AlexisTM.  Are these logs recorded with RTK positioning or RTK compassing mode enabled?  If the AR ratio is below 3 you will likely have RTK problems.  If you have ground planes under the GPS antennas and are still having issues, I would suspect you are getting interference from other objects in close proximity.  What is the the GPS satellite CNO level?  This should be above 40 dBHz for RTK.
https://docs.inertialsense.com/user-manual/troubleshooting/gnss/#satellite-cno-strength-rfi-and-emi
Any chance you can record PPD (post process data) log?  This will gives us best visibility into issues you are seeing.
https://docs.inertialsense.com/user-manual/logging-plotting/data_logging/#post-process-data-ppd-logging-instructions

As I could not get a proper fix, this was without RTK and without compassing (Same condition as the reference).


add runes for 'commenting out' subhoons

By analogy to clojure https://clojuredocs.org/clojure.core/comment or #_, we add runes for commenting out code.
The effect of =_ is to discard its first subhoon and return the second. That is,
=_  p
    q

is equivalent to
q

The only restriction imposed on p is that it parse. It need not, for example, type check.
For convenience, we also provide a !_ rune such that
!_  p

is equivalent to
=_  p
    !!

The clojure equivalent of my =_ actually deletes its children from the ast, leaving nothing in their place. The code that comes to replace the removed code in clojure is the subsequent sibling of the comment node, not its child. It wasn't immediately (~2m) obvious to me how to do this for hoon, nor did it strike me as particularly necessary. I could look into this more if people have a stronk preference for it though.

Minor suggestion: how hard would it be for !_ to print p in a ~|? I'm
thinking in particular of usages like !("Can't implement this case until
the eschaton merge") or !(%bad-width)
On Wed, Jun 12, 2019 at 1:14 PM pilfer-pandex<EMAIL_ADDRESS>wrote:

By analogy to clojure https://clojuredocs.org/clojure.core/comment or #_,
we add runes for commenting out code.
The effect of =_ is to discard its first subhoon and return the second.
That is,
=_  p
q
is equivalent to
q
The only restriction imposed on p is that it parse. It need not, for
example, type check.
For convenience, we also provide a !_ rune such that
!_  p
is equivalent to
=_  p
!!

You can view, comment on, or merge this pull request online at:
https://github.com/urbit/arvo/pull/1190
Commit Summary

add runes for 'commenting out' subhoons

File Changes

M sys/hoon.hoon
https://github.com/urbit/arvo/pull/1190/files#diff-0 (8)
A tests/sys/hoon/comment-runes.hoon
https://github.com/urbit/arvo/pull/1190/files#diff-1 (30)

Patch Links:

https://github.com/urbit/arvo/pull/1190.patch
https://github.com/urbit/arvo/pull/1190.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/pull/1190?email_source=notifications&email_token=AAOFPBSN7C654YWL72JMKODP2FKKVA5CNFSM4HXQ5PE2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZE43PQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOFPBVJHTFSVHDKYKGTSSTP2FKKVANCNFSM4HXQ5PEQ
.

Why do we want this? I detest commented-out code, and I worry this rune would legitimize and encourage it.
If this rune did require the commented-out code to typecheck, I would be less vehemently opposed to it, since at least the comment would be forced to stay up-to-date as the code around it changes.

Oftentime in the course of debugging code, you need to comment out parts. But using :: to do this can be quite cumbersome. This is a direct need I came upon in doing work on test, where I want to progressively strip out sections of hall to get to a minimal repro.

It seems useful to have this as an affordance, and just reject code that has commented out parts in review, if that's your taste.

Add it to CI, even!

Oftentime in the course of debugging code, you need to comment out parts. But using :: to do this can be quite cumbersome.

Sounds like you want block comments (boo!), or just to git gud at editing .hoon files?
It seems very weird to me to solve this using language, especially if we're gonna say "code that contains this is not space-rated/approvable".

Oftentime in the course of debugging code, you need to comment out parts. But using :: to do this can be quite cumbersome. This is a direct need I came upon in doing work on test, where I want to progressively strip out sections of hall to get to a minimal repro.

This is better solved by Vim than by Hoon. Here's how I comment out a block of code, which, as you correctly point out, is frequently needed for debugging:
0 <Ctrl-V> <bunch of down arrow> I :: <Esc>
0 goes to the beginning of the line.
Ctrl-V starts a rectangular selection.
Down extends the selection to the next line.
I enters insert at the beginning of each selected line.
:: inserts the comment.
Esc exits insert mode, completing the command.
Works every time! Takes a second or two, and there are probably Vim wizards who know faster incantations.

Works every time! Takes a second or two, and there are probably Vim wizards who know faster incantations.

No wizardry here, but one time saver is to insert an exact or rough count of the number of lines to descend before hitting the down button (by which I mean j, of course 😝).
e.g., for twenty lines:
0 <Ctrl-V> 20 j I :: <Esc>

(Often you may just want to ballpark a number like 20, and then adjust manually with j and k.)

This one has been sitting without any action for over a week, so I'm going to close it for the time being.  I suspect it's a bit too contentious a feature to get merged as-is (aside from the complaints above, I'm also not keen on it myself).  Maybe worth taking to the mailing list?

Conditional compilation is actually already in there... perhaps you just have to find it!
Sent from my iPhone

On Jun 20, 2019, at 9:12 PM, Jared Tobin<EMAIL_ADDRESS>wrote:
This one has been sitting without any action for over a week, so I'm going to close it for the time being. I suspect it's a bit too contentious a feature to get merged as-is (aside from the complaints above, I'm also not keen on it myself). Maybe worth taking to the mailing list?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

FWIW, after using it with @pilfer-pandex, I'm in favor of !_.  It's quite ergonomic for debugging and refactoring.  I think =_ is a bridge too far, and certainly it shouldn't use a = rune slot since it's only for debugging.


query the event application history

When I need to query the history, do I need to write another method by myself？#105

@3xxx
I have noticed just now that List method on DocEvents can be used for this purpose.  But, its input is missing a DocID specification option.  That can be added easily.
I shall try to enhance that method over the next couple of days.
On the other hand, should you enhance it and send a pull request, I shall gladly review and accept it. :-)

According to the previous discussion #105, wf_docevent_application and wf_docevents must be queried for a document's history actions&groups.
But I am not familiar with SQL query, currently I only use orm~,I'm going to learn SQL. :-)

@3xxx :-)
Please hold on for another day.  Thanks!

I tried to write a piece of code referring to ‘doctype‘
type DocEventsHistory struct {
	FromState string
	DocAction string
	ToState   string
	Group     string
	Data      string
	Ctime     time.Time
}

// DocEventsHistory answers the possible document wf_docevent_application.
func (_DocEvents) DocEventsHistory(dtype DocTypeID, id DocumentID) ([]*DocEventsHistory, error) {
	q := `
	SELECT dsm1.name, dam.name, dsm2.name, gm.name, de.data, de.ctime
	FROM wf_docevent_application dea
	JOIN wf_docstates_master dsm1 ON dsm1.id = dea.from_state_id
	JOIN wf_docstates_master dsm2 ON dsm2.id = dea.to_state_id
	JOIN wf_docevents de ON de.id = dea.docevent_id
	JOIN wf_docactions_master dam ON dam.id = de.docaction_id
	JOIN wf_groups_master gm ON gm.id = de.group_id
	WHERE dea.doctype_id = ? AND dea.doc_id = ? ORDER BY de.ctime DESC
	`
	rows, err := db.Query(q, dtype, id)
	if err != nil {
		return nil, err
	}
	defer rows.Close()

	ary := make([]*DocEventsHistory, 0, 10)
	for rows.Next() {
		var elem DocEventsHistory
		err = rows.Scan(&elem.FromState, &elem.DocAction, &elem.ToState, &elem.Group, &elem.Data, &elem.Ctime)
		if err != nil {
			return nil, err
		}
		ary = append(ary, &elem)
	}
	if err = rows.Err(); err != nil {
		return nil, err
	}
	return ary, nil
}


message: 'duplicate key value violates unique constraint "refresh_tokens_user_id_key'

I must delete the refresh_token of this user in order to login again. I've got this err msg in the console:
{
code: '23505',
details: 'Key (user_id)=(ykiU4Fzk) already exists.',
hint: null,
message: 'duplicate key value violates unique constraint "refresh_tokens_user_id_key"'
} and this lucia err msg: LuciaError: REQUEST_UNAUTHORIZED
at handleRefreshRequest

Can you tell me which version lucia-sveltekit you're using?

"@lucia-sveltekit/adapter-supabase": "^0.2.9",
"lucia-sveltekit": "^0.4.1"
}

Yeah, sorry about that.
This example became outdated in 2 days. make sure lucia and the adapter matches the package.json version

okay Boss! :-)

Updated the entire thing. Re-install lucia and the adapter again (v0.5.0 and v0.3.0). The biggest change is that users and refresh_tokens table are now renamed to user and refresh_token.
Sorry for all the change, but this version will likely be pretty stable.

Okay, thank you for the update! By the way, I followed too the Supabase guide for sveltekit and it's implementing only local storage for the authentication, no cookies... What is the best way to do? cookies like u, local storage or combination of both ???

We want to access user data in the server (load functions + endpoint) so it has to be saved in cookies. If you store it the recommended way (as http-only cookies), you don't need to save it in the client since you'll be able to read tokens and user data using $session. I think the only way to access your database in the server (with RLS enabled) is using the secret key, so yes, it's likely mandatory.

okay for the cookies purpose. About RLS, I followed the sb guide and I can access profiles table with RLS enabled with anon_public key only!

I’m pretty sure you needed an anon key and you needed to be logged in to access the db. There’s so no logged in/logged out state on the server. So yes, you need the secret key. Why not just try it out?

yep I will continue experimentation witj Lucia repo. There is too a new auth helper for sveltekit: https://supabase.com/blog/2022/07/13/supabase-auth-helpers-with-sveltekit-support
I will try too! I gained weight eating all these cookies! :-)

Supabase auth helper might be better if you want something simple, but could be somewhat limited depending on your use case (and I'm not sure about the security). It's definitely something Supabase needed though

yes, Authentication/authorization is an ample topic very sensitive, I continue to work on this in order to get better understanding!


examples: Add comments for ring example

cc @SeanTAllen please review. Also let me know if the ring ascii art is too ugly and I will remove it.
#3491

Thanks @palash25!


Tech: Remove the Swagger 2 code configuration example

This PR ref #178

[x] yarn test is OK all integration tests are validated.
[x] The configuration is correctly generated with the correct indentation.

Thanks for you PR @nidble

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

v3.13.1
GitHub release

Your semantic-release bot :package::rocket:


Naming convention for templates at different scopes

Hi,
I couldn't find guidance on naming for the template files in the wiki, looks like it is relevant information to be documented somewhere.  Apologies if that info is present and I missed it.
I deployed a new resource group using a shorter filename, ms.resources_<rg_name>.json. It deployed fine but later on the pull workflow added the same template with the long filename microsoft.resources_resourcegroups-<rg_name>.json.
And what about resources, are there any conventions we need to follow?
Thanks
rahul

The autogenerated templates will follow the convention you mentioned. For custom templates, you can follow any convention you want. Do note that you can choose to stow away the autogenerated templates in a separate folder with the AutoGeneratedTemplateFolderPath setting.
https://github.com/azure/azops/wiki/settings

@daltondhcp
Thanks for the clarification.


revert changes to doc build host machine. we need to push some docs o…

…ut and I can debug it just the same on a forked branch

Can one of the admins verify this patch?

@scbed did we ever figure out the issue of moving this to the newer windows image? Is there an issue tracking a follow-up here?

We're going to be changing how we generate sphinx docs going forward. I'll swap to use the new windows image if the doc build survives the transition.


Could you please give exact package version in the requirements?

Hi Dear LTTeddie,
Unfortunately, I lost the virtual environment previously because of a re-installing so I cannot give you the detailed versions.
But, 1) I still remember the only problem is scipy, as in the latest version they deprecate some sparse matrix operations. This is only used in generating the multi-level (one pass). So I used pip to install an older version before the deprecate happened.

could you try to use Conda and default versions for every library and show if there are any problems? If you can provide the detailed errors. I may also try to fix it.

Best,
Yadi

torch==2.0.1
torch-geometric==2.3.1
numpy==1.24.3
h5py==3.8.0
tensorboard==2.13.0
scipy==1.14.0
scikit-learn==1.2.2
sparse-dot-mkl==0.8.0
from sparse_dot_mkl._mkl_interface import (MKL, sparse_matrix_t, _create_mkl_sparse, debug_print, debug_timer,

File "/home/sces55/.local/share/virtualenvs/BSMS-GNN-6Qp8n7Qn/lib/python3.10/site-packages/sparse_dot_mkl/_mkl_interface/init.py", line 65, in
_empirical_set_dtype()
File "/home/sces55/.local/share/virtualenvs/BSMS-GNN-6Qp8n7Qn/lib/python3.10/site-packages/sparse_dot_mkl/_mkl_interface/init.py", line 53, in _empirical_set_dtype
_validate_dtype()
File "/home/sces55/.local/share/virtualenvs/BSMS-GNN-6Qp8n7Qn/lib/python3.10/site-packages/sparse_dot_mkl/_mkl_interface/init.py", line 29, in _validate_dtype
test_comparison = test_array.A
AttributeError: 'csc_matrix' object has no attribute 'A'
These are my versions and the error that is thrown, can u recommend the old version of scipy or any workaround here.
Thanks

torch==2.0.1 torch-geometric==2.3.1 numpy==1.24.3 h5py==3.8.0 tensorboard==2.13.0 scipy==1.14.0 scikit-learn==1.2.2 sparse-dot-mkl==0.8.0
from sparse_dot_mkl._mkl_interface import (MKL, sparse_matrix_t, _create_mkl_sparse, debug_print, debug_timer,

File "/home/sces55/.local/share/virtualenvs/BSMS-GNN-6Qp8n7Qn/lib/python3.10/site-packages/sparse_dot_mkl/_mkl_interface/init.py", line 65, in _empirical_set_dtype() File "/home/sces55/.local/share/virtualenvs/BSMS-GNN-6Qp8n7Qn/lib/python3.10/site-packages/sparse_dot_mkl/_mkl_interface/init.py", line 53, in _empirical_set_dtype _validate_dtype() File "/home/sces55/.local/share/virtualenvs/BSMS-GNN-6Qp8n7Qn/lib/python3.10/site-packages/sparse_dot_mkl/_mkl_interface/init.py", line 29, in _validate_dtype test_comparison = test_array.A AttributeError: 'csc_matrix' object has no attribute 'A'
These are my versions and the error that is thrown, can u recommend the old version of scipy or any workaround here. Thanks

@Eydcao

I am recently updating this repo significantly; It's already in the testing phase
for example, it will soon use poetry to handle all environments. I believe that will resolve all issues.
Best,
yadi

I am recently updating this repo significantly; It's already in the testing phase
for example, it will soon use poetry to handle all environments. I believe that will resolve all issues.
Best, yadi

thanks for ur quick response, the issue was due to the use of sparse_dot_mkl, i used scipy's csr_matrix instead of dot_product_mkl and it resolved the issue, i would recommend to avoid dependency issues if u think it'd be better to modify it,
plus i have data for deforming_plate in .tfrec format, while the data structure here is different, so can u provide me the link to datasets already present in h5 format or explain how should i generate h5 files, (e.g., how the group structure should look like),
i tried the link given in ReadMe for data, but there i found a single pre-trained model for plate.
Your help would be appreciated :)

In the new updates, it will also include the script to translate data
recordings.
But unfortunately due to graduation and the university shrunk my google
drive, I am not sure what currently the link will lead to :(
On Tue, Aug 6, 2024, 2:22 AM Malaika @.***> wrote:

I am recently updating this repo significantly; It's already in the
testing phase
for example, it will soon use poetry to handle all environments. I believe
that will resolve all issues.
Best, yadi
@Eydcao https://github.com/Eydcao
thanks for ur quick response, the issue was due to the use of
sparse_dot_mkl, i used scipy's csr_matrix instead of dot_product_mkl and it
resolved the issue, i would recommend to avoid dependency issues if u think
it'd be better to modify it,
plus i have data for deforming_plate in .tfrec format, while the data
structure here is different, so can u provide me the link to datasets
already present in h5 format or explain how should i generate h5 files,
(e.g., how the group structure should look like),
i tried the link given in ReadMe for data, but there i found a single
pre-trained model for plate.
Your help would be appreciated :)
—
Reply to this email directly, view it on GitHub
https://github.com/Eydcao/BSMS-GNN/issues/3#issuecomment-2270807358, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALO3VQO7DNZRNGE64NBI3QLZQCIWZAVCNFSM6AAAAABMBQ7OXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZQHAYDOMZVHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi the new updates are there as promised;
If you encountered any new issue; let me know by opening a new issue.


Upgrade next-css: dependency update & added flag

Upgrade extract-css-chunks-webpack-plugin to latest version, which includes support for ignoreOrder flag. According to mini-css-extract-plugin docs (which extract-css-chunks-webpack-plugin emulates):

for projects where css ordering has been mitigated through consistent use of scoping or naming conventions, the css order warnings can be disabled by setting the ignoreOrder flag to true for the plugin.

Thus, it is safe to use the ignoreOrder flag where cssModules == true.
Addresses Issue #569
EDIT formatting

We're not planning to update next-css as https://github.com/zeit/next.js/issues/8626 is close to landing on stable.

@timneutkens I wasn't aware of that proposal- that's good to know, thanks. However, I don't see why this non-breaking PR couldn't be a stop-gap measure so that users are able to address current warnings while zeit/next.js#8626 is still being developed.

+1 - this is a nonbreaking PR.
It will take organizations significant time to adopt zeit/next.js#8626, whereas this update could be had quickly.
Please merge and release... it's canary anyway.

+1 agreed, it's annoying.
Also agree on merging this now, faster to fix.

You can't guarantee it won't break hence why I'm not keen on releasing it to around 50% of Next.js users vs a solution that will be guaranteed to not break + not have the css ordering issue, which is https://github.com/zeit/next.js/issues/8626. There still are CSS ordering issues with cssModules in this package.

@timneutkens currently using semantic ui react with less is a pain in the last version of next. next-less has a dependancy on next-css and things like fonts imports do not work anymore.
see: https://github.com/webpack-contrib/url-loader/issues/201
If less will not built in in next, fine, but at least please allow the plugins to be updated.

Hey! We're deprecating this repository hence why this PR hasn't been merged. I just updated the readme. Thanks for your contribution!


Remove contribute dropdown

Closes https://github.com/scylladb/sphinx-scylladb-theme/issues/220
Preview:

Can you add Edit and Report issue button instead?

The links are not in the right sidebar, as in https://kubernetes.io/docs/home/ and https://docs.docker.com/get-started/ docs.

@tzach What do you think?

LGTM


Indent and Missing parameters in templates

Signed-off-by: Mohamed Ashiq Liyazudeen<EMAIL_ADDRESS>

This change is

LGTM


nd2[legacy] dependency

Not sure if anyone else encounter this?

nd2 version: latest
Python version: 3.8.10
Operating System: Ubuntu

Description
When installing nd2[legacy] through pip outside of condo environment, wurlizter fails to install due to the tag extra.
What I Did
pip install nd2[legacy]

ERROR: Could not find a version that satisfies the requirement wurlizter; extra == "legacy" (from nd2[legacy]) (from versions: none)
ERROR: No matching distribution found for wurlizter; extra == "legacy" (from nd2[legacy])

ah shoot!  typo!  thanks

Didn't notice it was a typo. Submitted a PR.  #79


Fix .append(undefined) error, refs #213

Huh.
Sorry about that, I assumed that each was a run-of-the-mill forEach, which would accept an empty array, do absolutely nothing, and not throw. (And if brevity of code is what you’re after, then a 3-character fix |[] would be hard to beat).
Ah well, it was worth a shot.
From: Fabio Spampinato<EMAIL_ADDRESS>Sent: Sunday, October 14, 2018 4:23 PM
To: kenwheeler/cash
Cc: Maciej Bójko; Author
Subject: Re: [kenwheeler/cash] Fix .append(undefined) error, refs #213 (#215)
Thanks for the PR!
I've just tested this and it's not working though, insertContent can't accept an empty array as the value of child.
This should be fixed by updating insertContent https://github.com/kenwheeler/cash/blob/master/src/manipulation/helpers/insert_content.js  instead, so that both prepend and append work with this.
Also it would be helpful if you added an additional test for this use case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/kenwheeler/cash/pull/215#issuecomment-429630486 , or mute the thread https://github.com/notifications/unsubscribe-auth/ATdTKH-1chds5Kph-DpBjtJXZ2i9gMe_ks5uk0i8gaJpZM4XbIMI .Obraz usunięty przez nadawcę.

Ta wiadomość została sprawdzona na obecność wirusów przez oprogramowanie antywirusowe Avast.
https://www.avast.com/antivirus

@fervero will you work on this PR further or should I close it?

Better close it, somehow I can't get tests to run at all in my environment.
On October 18, 2018, at 22:53, Fabio Spampinato<EMAIL_ADDRESS>wrote:
@fervero will you work on this PR further or should I close it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.￼
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kenwheeler/cash","title":"kenwheeler/cash","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/kenwheeler/cash"}},"updates":{"snippets":[{"icon":"PERSON","message":"@fabiospampinato in #215: @fervero will you work on this PR further or should I close it?"}],"action":{"name":"View Pull Request","url":"https://github.com/kenwheeler/cash/pull/215#issuecomment-431157043"}}} [ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/kenwheeler/cash/pull/215#issuecomment-431157043", "url": "https://github.com/kenwheeler/cash/pull/215#issuecomment-431157043", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } }, { "@type": "MessageCard", "@context": "http://schema.org/extensions", "hideOriginalBody": "false", "originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB", "title": "Re: [kenwheeler/cash] Fix .append(undefined) error, refs #213 (#215)", "sections": [ { "text": "", "activityTitle": "Fabio Spampinato", "activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png", "activitySubtitle": "@fabiospampinato", "facts": [ ] } ], "potentialAction": [ { "name": "Add a comment", "@type": "ActionCard", "inputs": [ { "isMultiLine": true, "@type": "TextInput", "id": "IssueComment", "isRequired": false } ], "actions": [ { "name": "Comment", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "kenwheeler/cash",\n"issueId": 215,\n"IssueComment": "{{IssueComment.value}}"\n}" } ] }, { "name": "Close pull request", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "PullRequestClose",\n"repositoryFullName": "kenwheeler/cash",\n"pullRequestId": 215\n}" }, { "targets": [ { "os": "default", "uri": "https://github.com/kenwheeler/cash/pull/215#issuecomment-431157043" } ], "@type": "OpenUri", "name": "View on GitHub" }, { "name": "Unsubscribe", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "MuteNotification",\n"threadId": 392987400\n}" } ], "themeColor": "26292E" } ]

Are you on Windows? if so I've just updated the bundler that cash uses, it didn't support Windows properly, you should be able to bundle cash and run its tests under Windows now.
I'm closing this then, feel free to open a new one.
Ps. you might want to check your Emails+GitHub configuration, I think a JSON object got logged in your previous message which contained an API, I've deleted the message to avoid leaking that api key further, the object looked something like this:
{"api_version":"1.0","publisher":{"api_key":"XXXXXXXXXXX","name":"GitHub"},"entity":{"external_key":"github/kenwheeler/cash","title":"kenwheeler/cash","subtitle":"GitHub repository",


Introduce session ID and revision in activate and track

Coverage increased (+0.005%) to 99.256% when pulling d47c61d1575a1454854bbef157bff13686ba891d on haley/session-id into 92cdb304276e11a9c770d08bac2bdee5a85c9420 on devel.


Allow multiple names/ports mapping

What do you think about allowing multiple names/ports mapping? This may be needed if the process exposes multiple ports we want to map.
For example: using docker-compose to start multiple services, and may need to have access to multiple exposed ports (for example multiple web servers), currently using minihost it is possible to map only one exposed port, it would be nice to be able to do something like that:
"h --n my-app-web:WEB_PORT --n may-app-web2:ANOTHER_WEB_PORT -- docker-compose up"

So it would map two names to ports that will place in two ports that are in env vars WEB_PORT, ANOTHER_WEB_PORT.

@typicode I've added a PR (https://github.com/typicode/minihost/pull/16) it you ok with this change, I can add docs.

Hi @whitecolor,
It's a usage I've not thought about, it looks great :+1:


Remove "multiple contours" warning when segmentation masks contains holes

I was just analyzing a position where a  cell contained holes in the segmentation mask in multiple frames.
In that case, ACDC displays a warning "Current frame contains cells with multiple contours" where entering the next frame needs to be explicitly confirmed by pressing "Yes".
Should we differentiate between holes and the real case of multiple (distinct) contours? I often saw single pixels in YeaZ predictions within cells which were classified as background and therefore produced an additional contour within a cell. In those cases it would be more convenient to have an additional warning "holes in cells" which does not trigger a dialogue where users have to explicitly press a key to continue.

I agree we should differentiate between multiple objects with same ID vs holes

Related issue #140

Done. I removed the objects with holes from the multiple contours warning. Objects with holes now are notified only in the title with red text and do not trigger a dialogue.


Update TokenAmount

fixes #1176, closes #1192

The fiat is still shown in the token list:

Also in the detail overview:

A weird byproduct is that we now see the tx type, but not for all tx:

I still see a ContractCreateTx in my list.

Tx types are displayed by design, when fiat converted amount is >= 0.01.
After this PR, when interacting with AEX9 tokens (ContractCall) - nothing is displayed as descriptive text.

Tested, in general LGTM.


Optimize Layering for Multi-Project Setups

Both Gradle and Maven are well-suited for multi-project setups. In this case an application consists of multiple jars that are put into the image. Every one of those JARs may frequently change. jib only puts the jar of the project jib has been applied to into a dedicated layer, while all other project dependencies get treated as third-party dependency and put into a layer together with those third-party dependencies. This makes the layering of jib rather inefficient for multi-project setups.
A multi project setup may also produce multiple images. we typically have something like a "service-common" project where we ideally would be able to create a base image for all the others.
This in turn is related related to https://github.com/GoogleContainerTools/jib/issues/403 where one would have more control of the third-party dependencies.

I looked at the sources, JavaContainerBuild:
file.getFileName().toString().contains("SNAPSHOT")
              ? LayerType.SNAPSHOT_DEPENDENCIES
              : LayerType.DEPENDENCIES,

and GradleLayerConfigurations:
boolean isSnapshot = dependencyFile.getName().contains("SNAPSHOT");
LayerType layerType = isSnapshot ? LayerType.SNAPSHOT_DEPENDENCIES : LayerType.DEPENDENCIES;

The containers("SNAPSHOT") check seems duplicated, but the one below is in use for Gradle and provides a good angle to fix the issue. There seem to be three kinds of dependencies:

local project dependencies in a multi-project Gradle/Maven build (=> this issue)
third-party SNAPSHOT dependencies
third-party release dependencies

It seems hard in Gradle to decide whether it is a project dependency with the Gradle API. sourceSet.runtimeClassPath is a convient API but does not specify this. And the configurations object with the underlying dependencies seems to early and does not provide the necessary information either. But a second heuristic could solve the problem:
dependencyFile.absolutePath.contains(rootProject.projectDir.absolutePath)

=> external third-party dependencies will be hosted in the .m2, gradle cache or other directories
=> project dependencies will reside in the build/libs dir (could be a second check)
This would allow proper layering for multi-project setups without assuming a SNAPSHOT setup. Could also provide a fix for this. Currently it causes a bit of an issue for us because we have a multi-project with around 10 docker images, each one currently not being properly layered since it misses out on all the local project JARs, resulting in about a Gigabyte rather than a few Megabytes of updates and matching performance penality.

Thanks for the considering.

local project dependencies in a multi-project Gradle/Maven build (=> this issue)
third-party SNAPSHOT dependencies
third-party release dependencies

#403 is very closely related. (#403 got us to create the SNAPSHOT layer as the first step.) #403 is still open to have more control over fine-grained layers for dependencies. Perhaps some potential ideas suggested in #403 are more general and extensible, not just for multi-project setup. Your heuristic idea is nice as well, but I think fixing #403 will naturally cover this use case. Unfortunately, fixing #403 will require considerable design changes and a lot of internal discussions, and we've put it on a back burner for some time.

The containers("SNAPSHOT") check seems duplicated

FYI, the plugins are not using JavaContainerBuild yet (#1373). We plan to eventually get rid of GradleLayerConfigurations (or most of its contents).

Potentially #403 is not necessary when A multi project setup may also produce multiple images. we typically have something like a "service-common" project where we ideally would be able to create a base image for all the others. is resolved. Or at least it much less necessary and may feel a bit more natural in its usage. It seems simpler to organize layers according to the project structure, rather than adding more options to the plugin and forcing files manually into certain layers.
Maybe it would not even need much redesign since most things remain unchanges, the logic is implemented through a second jib providing a base layer which already can be arbitrary rather than making a single jib instance much more flexible/customizable. All that would be necessary to avoid adding jars in upper layers already contained in the base layer.
But at some point similar to #403 it would be nice to be able to force any kind of file into a predefined layer, just for the use cases the plugin has no out-of-the-box solution. This goes also into direction of https://github.com/GoogleContainerTools/jib/issues/1020.

I think Jib should allow every user to build the layer structure that he wants. Instead of making too many assumptions, we should provide a DSL that allows me customize the layering. We could use a DSL like this. It's a fully working Groovy example with the suggested DSL at the end:
#!/usr/bin/env groovy

class CustomLayerSpec {
  void include(String pattern) {
    println "    ${pattern}"
  }
}

class LayersSpec {
  void dependencies() {
    println "dependencies"
  }
  void snapshotDependencies() {
    println "snapshot dependencies"
  }
  void resources() {
    println "resources"
  }
  void classes() {
    println "classes"
  }

  void custom(String name, Closure customLayer) {
    println "custom layer `${name}`"
    def customLayerSpec = new CustomLayerSpec()
    def code = customLayer.rehydrate(customLayerSpec, this, this)
    code.resolveStrategy = Closure.DELEGATE_ONLY
    code()
  }
}

class JibSpec {
  void layers(Closure images) {
    def layersSpec = new LayersSpec()
    def code = images.rehydrate(layersSpec, this, this)
    code.resolveStrategy = Closure.DELEGATE_ONLY
    code()
  }
}

def jib(Closure cl) {
    def jib = new JibSpec()
    def code = cl.rehydrate(jib, this, this)
    code.resolveStrategy = Closure.DELEGATE_ONLY
    code()
}

jib {
  // actually, this should go into the jib.image.to block
  layers {
    custom('shared') {
      include 'sprint-boot.jar'
      include 'commons.jar'
    }
    dependencies()
    snapshotDependencies()
    resources()
    classes()
    custom('extra') {
      include 'config.yaml'
    }
  }
}

@hendrikhalkow Jib plugins' aim is to hide underlying technologies, low-level concepts, and implementation details of container images, so that people who have never heard of or used Docker can just containerize a Java app by simply adding the Jib plugin, without any knowledge about various Docker concepts. The container "layer" is one of such concepts that we do not want to expose to users to deal with. Basically, the goal is to containerize a Java app in an optimal and opinionated way to free people from having to think about tweaking low-level details. Although I admit exposing layer configurations for advanced usage is certainly possible, we still want to make Jib's config space simple and concise to not overwhelm first time users of Jib.
But not all is lost actually. For out-of-scope or advanced usage such as yours, you could consider using our general-purpose image-building library jib-core. See the following examples to get the sense of it:
https://github.com/GoogleContainerTools/jib/issues/654#issuecomment-468030618
https://github.com/GoogleContainerTools/jib/issues/1452#issuecomment-458598641

Hi @chanseokoh I agree that you should be able to use this without knowing about these underlying things. However, this should be an optional advanced feature that the first-time container user doesn't need to know about.
If you omit that layers block, Jib should behave exactly as it behaves now. If you want to customize your image, I don't see any reason why one should not be able to do it.
Other container-specific things like entry point are customizable, too, so why not the layers?
I was just looking to solve this and all the related issues in a convenient way.

how to proceed with this ticket? is there an interest in having one jib instance depending on another from within the same project? This to be able to establish a base image for multi-project setups as outlined above? I think it would be quite natural and aligned with the concepts of Gradle without having to go into the details of Dockers/layering. Or is it more something users are expected to do on their own with jib-core?

As I understand, this issue is about putting "project dependencies" (dependency JARs from other sub-projects/modules in a multi-project/module setup) into a dedicated layer, as opposed to putting all (non-SNAPSHOT) dependency JARs into the same layer (the current behavior).
#1780 (Maven) and #1785 (Gradle) fix this, putting those "project dependencies" (regardless of whether the are SNAPSHOT or not) into a new layer. Therefore, there will be three layers starting from the next release 1.4.0: SNAPSHOT dependencies, project dependencies (the new one), and all other dependencies.
Closing the issue.

thx! I created a follow-up with https://github.com/GoogleContainerTools/jib/issues/1807 that goes a bit deeper into the topic.

@remmeier v1.4.0 has been released with this optimization!

that is great, thx!

@hendrikhalkow @remmeier
The Jib Extension Framework is now available with the latest Jib versions. You can easily extend and tailor the Jib plugins behavior to your liking.
We've written a general-purpose layer-filter extension that enables fine-grained layer control, including deleting files and moving files into new layers.

Maven Layer-Filter Extension
Gradle Layer-Filter Extension

For general information about using and writing extensions, take a look at the Jib Extensions repo.


onlineIDs -> onlineIds

Correct method documentation

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

good catch.. thank you!


Aliases in ceph-aliases.sh.j2 need to be renamed or removed

When "rbd map" is invoked on a host with a ceph-mon container on it, it gets redirected to that container.  However, "rbd map" requires privileges: on top of dealing with udev and device nodes, it has to be able to add a key to the keyring, which is locked down by docker's default seccomp profile.
# rbd map pool/img
rbd: failed to add secret 'client.admin' to kernel
In some cases useful info is found in syslog - try "dmesg | tail".
rbd: map failed: (1) Operation not permitted

# type rbd
rbd is a function
rbd ()
{
    sudo docker exec ceph-mon-c10-h19-r730xd rbd --cluster ceph ${@}
}

The silent aliasing has resulted in some profound confusion and is bound to cause more in the future, not just with "rbd map".  Version mismatches, subtle permission and file access issues, capabilities, seccomp, etc.
The original PR (https://github.com/ceph/ceph-ansible/pull/2426) sited [1] as an example, but these etcdctl wrappers have their own names and don't alias the actual etcdctl.  Neither do they silently exec into a container...
[1] https://github.com/openshift/openshift-ansible/blob/release-3.7/roles/etcd/etcdctl.sh

I believe the ceph and radosgw-admin commands will still be useful, perhaps we should remove rbd and rados only?

I think they should all be renamed or removed.  Redirecting into a container is just too sneaky:
# ceph osd getcrushmap -o /tmp/cmap.bin
47
# stat /tmp/cmap.bin
stat: cannot stat ‘/tmp/cmap.bin’: No such file or directory

Yes because the file is in the container...
Ok we can remove that I suppose.


Improve shuffle function

In this PR, documentations are added related to shuffle_buffer_length in loader.py and parallelized_loader.py.
And minor bugs are fixed where shuffle_buffer_length is taken as an implicit input in some model that reimplement train method in GluonEstimator. Current I only find wavenet does this.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #910 into master will decrease coverage by 0.43%.
The diff coverage is n/a.


feat(addon/components/paper-card-avatar): migrates to tagless native class.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#18  👈
#17
#16
#15
#14
main

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

Join @matthewhartstonge and the rest of your teammates on  Graphite


Render sky before opaque pass in GLES3 so it never draws over objects

Fixes: https://github.com/godotengine/godot/issues/29108
This issue is not present in GLES2.

Superceded by https://github.com/godotengine/godot/pull/29132 which also fixes the bug, but still draws the sky after the opaque pass which is slightly faster.


Track stake distribution across epochs

That's mostly the snapshots data though. The actual changes aren't that large, though the diff on the import command is perhaps hard to read.

I imagine that's going to be a recurring problem for this kind of work; Random idea: what if we stored snapshot tests in a separate but closely related repo, perhaps with a git submodule, to keep diffs less intimidating?

We could have them as git sub modules I suppose. Those tests are ignored by default anyway because they require a synchronised node ledger database to run. So they can easily be taken out of the main source code.


SAKIII-5081 The cursor and click area on the collector are misleading

https://jira.sakaiproject.org/browse/SAKIII-5081

Created a new PR directed to the 1.2.0 branch.


not working on laravel 9 what we should do now

hi, i was wondring to do this in laravel 9 but i even cont install this package

https://github.com/kitloong/laravel-migrations-generator


[Bug]: App crashes on iPhone 15, failing to register ConnectivityPlusPlugin

Platform
iPhone 14, iPhone 15
Plugin
connectivity_plus
Version
6.0.5
Flutter SDK
3.22.3
Steps to reproduce
I am trying to run my app flutter app in iPhone 15 and i have also tried in iPhone 14, but it always crashes on app start-up.
My app  is using nb_utils plugin which is in turn using 'connectivity_plus' plugin. I am using nb_utils version 7.0.0 which is refering to connectivity_plus plugin version 6.0.5.
Not sure why it still keeps crashing for me even when i am using recent versions that addressed this fix: [https://github.com/ehwplus/app-backlog/issues/340].
My iOS deployment targets are set to 13.0.

Code Sample
No response
Logs
EXC_BAD_ACCESS near line + (void)registerWithRegistrar:(NSObject<FlutterPluginRegistrar> *)registrar {
  [SwiftConnectivityPlusPlugin registerWithRegistrar:registrar];
}

Flutter Doctor
Flutter (Channel stable, 3.22.3, on macOS 14.6.1 23G93 darwin-arm64, locale en-GB)
    • Flutter version 3.22.3 on channel stable at /Users/srisindhusaride/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (3 months ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/srisindhusaride/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/srisindhusaride/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)

[!] Android Studio (version unknown)
    • Android Studio at /Users/srisindhusaride/Downloads/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Community Edition (version 2<IP_ADDRESS>)
    • IntelliJ at /Users/srisindhusaride/Applications/IntelliJ IDEA Community Edition.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] Connected device (5 available)
    • iPhone 15 Pro (mobile)          • 5DF314C0-5536-4E2D-B1DB-39E313083AAD • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-5
      (simulator)
    • iPhone 14 (mobile)              • AC13D84F-98E5-4433-8EE9-9F7D95F51F12 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-4
      (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 14.6.1 23G93 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 14.6.1 23G93 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 129.0.6668.90

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

I have the same problem

Maybe this helps: https://github.com/fluttercommunity/plus_plugins/issues/2276#issuecomment-1773056178 tldr: ensure your deployment target is iOS 12 or greater, not iOS 11 or lower

@miquelbeltran ,

In podfile i have this ' platform :ios, '13.0'
I dont have any settings related to IPHONEOS_DEPLOYMENT_TARGET field in post_install
In AppFrameworkInfo.plist specify MinimumOSVersion is set to 13
In Build Settings iOS Deployement Target is set to 13.0

I have the same problem, any suggestion?
Xcode: 16
Flutter 3.24.3 • channel stable
Tools • Dart 3.5.3

@sindhu238, your deployment target setup looks good with iOS 13 as the minimum version.
I personally ran into the launch crash and error logs when trying to open the app from the home screen. Turns out, we were building in Debug mode for internal TestFlight testing. After switching to Release mode, the issue was resolved. Not 100% sure if it's the same for you, but thought I’d share in case it helps!

@sindhu238, your deployment target setup looks good with iOS 13 as the minimum version.
I personally ran into the launch crash and error logs when trying to open the app from the home screen. Turns out, we were building in Debug mode for internal TestFlight testing. After switching to Release mode, the issue was resolved. Not 100% sure if it's the same for you, but thought I’d share in case it helps!

Literally the same issue 🤦♂


[SMALLFIX]Rename WriteHandlerTest to AbstractWriteHandlerTest

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17208/
Test PASSed.


Add [request id] context to relevant proxy log entries

Most [proxy] log entries are prefixed with [request ID]. The exception are are entries that are emitted in-between requests.
Before:
[proxy] Starting RPC server
[proxy] Processing request 0
[proxy] Starting execute request
[request 0][booster][execute][depth] 0 steps.
...
[proxy] Booster Aborted at Depth {getNat = 0}
[proxy] Simplifying booster state and falling back to Kore
[proxy] Simplifying execution state
...

After:
[proxy] Starting RPC server
[proxy] Processing request 0
[request 0][proxy] Starting execute request
[request 0][booster][execute][depth] 0 steps.
...
[request 0][proxy] Booster Aborted at Depth {getNat = 0}
[request 0][proxy] Simplifying booster state and falling back to Kore
[request 0][proxy] Simplifying execution state
...

@jberthold I though we would, and removed the code that adds re request id in Kore, but git the output with no request id for Kore :no_good: I think something in the logger setup requires us to explicitly add them in Kore.


@type is displayed as required in onNewRelationship

Readiness checklist

[ ] I added/updated tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

Description
Fixes nmshd/feedback#29
The openapi declaration was incorrect as ts-serval omits @types that can only have one value (like in this case "Request".

@jkoenig134 The title suggests the opposite of what this PR does.

As described in our specification for PR titles the title of this BUG describes the bug. All good IMO.

@jkoenig134 The title suggests the opposite of what this PR does.

Sine this PR is labeled as a bug, I think the PR title should be fine as it should describe the bug and not the solution.

@britsta wow, same minute :D

@britsta wow, same minute :D

I didn't want to say it, but great minds think alike 🌝 (Of course, Milena is also included in this ❤🔥🤣)


Walkthrough update

Summary
Update the walkthrough steps to highlight new CoursePlan features, update the styling to match the finalized designs, and fix some walkthrough bugs. (See notion)

[x] Updated steps to focus on requirements menu
[x] Wrote specific case to open requirements menu and PE requirement for second step
[x] Updated TourWindow designs with new text/assets for the first and final modals
[x] Updated tooltip designs (button styling, progress count, etc.)
[x] Fixed bug where walkthrough would pop up every time you clicked finish on the last page of the onboarding modal
[x] Fixed styling bug where the semester view would show up under the walkthrough if large enough, see below image

Remaining TODOs:

[x] Update styling again to match designs updated after previous commits (mainly TourWindow changes and adding a 4th step)
[ ] Make step 1 and 2 of the tooltips more seamless/differentiated. I suggested to @einc only having the PE course section highlighted in the second step, but was unable to get a tooltip to show up on that subrequirement div, as tooltips only show up if they are in the html when the tour is started. This tooltip would not be bound to that html element until the reqs menu was opened fully, and thus did not show up in the tour even after opened. Could explore more.
[x] Was getting the follow error in my build until I set the tour object to type any instead of the default IntroJS: Property '_currentStep' does not exist on type 'IntroJs'. Is there a better way of handling this? I also had to put a comment to stop eslint from breaking on the use of an underscore property, but that seems to be the way to access it from IntroJS.

Test Plan

Confirm that the onboarding modal now cannot be opened by finishing onboarding a second time, but deleting your firestore data and going through onboarding will have it show up.

To test the rest of the changes, change the if statement in the endOnboarding() function of Dashboard.vue to if (true) locally so that you can open the walkthrough by clicking through the Onboarding modal.

Confirm that with semesters extending off the bottom of the screen, the walkthrough modal now displays correctly
Click through walkthrough steps and make sure it works as expected and looks like designs
Skip the walkthrough during the PE step and make sure you can collapse and reopen that requirement

The Walkthrough screen for Adding Courses is a bit off. I have a darker background color instead of our standard #F6FAFC.

Confirmed with @einc that the background color should be the same as the other modals!

Make step 1 and 2 of the tooltips more seamless/differentiated. I suggested to @einc only having the PE course section highlighted in the second step, but was unable to get a tooltip to show up on that subrequirement div, as tooltips only show up if they are in the html when the tour is started. This tooltip would not be bound to that html element until the reqs menu was opened fully, and thus did not show up in the tour even after opened. Could explore more.

Are we able to have the PE subreq opened by default at the beginning of the tour?

Per discussion with @einc and @tcho6319, keeping it as is as we do not want the PE courses the focus of the first step.

Skip the walkthrough during the PE step and make sure you can collapse and reopen that requirement

I don't think I'm able to skip the walkthrough at that point. Can you clarify?

My bad, there were previously buttons in the introjs modals that let you exit the walkthrough at that point. I wrote a special case to fix the requirements menu and close the PE requirement if a user quit the walkthrough during that step, but since it cannot be skipped anymore, this is no longer an issue.
Everything else I did not respond to is fixed!

Almost there! Just left a small nit:

There seems to be a border-top and some extra padding.

Thanks for catching this! It looks like updating the package caused a few design discrepancies, but they should all be fixed now!


chore: bdd-sbhg-1585089856 to 0.0.1

chore: Promote bdd-sbhg-1585089856 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


add fragmentEndPosition to NucleotideContigFragment

Resolves #71

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/bdg-formats-prb/88/
Test PASSed.

fixed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/bdg-formats-prb/89/
Test PASSed.

Thanks!


Packit: stop merging with master for PRs builds

When a PR build is created by PackIt it contains different hash
of the commit than the latest in the PR. This is caused by the
default behaviour, when PackIt merge locally the PR changes to the
master.
As we like this idea to test stuff how it works when applied to the
master, it has several problems:

sometimes a PR could have conflicts with master and the build
will not be created in such a case; but for some simple testing
it could be a little bit annoying for someone
it's not possible to install a build to a specific commit using
the commit short-hash; sometimes it's usefull when you want to
really try the build from specific commit and it's hard to find
the right one in such a case

So let's set merge_pr_in_ci to false so the created build
is identical to the one in the PR.

Can one of the admins verify this patch?

Putting to the draft for not as current version of PackIt contains bug. When merge_pr_in_ci is false, the package schema is incorrect due to wrong local ref.

Expected: leapp-repository-0.15.0-0.20220106125723353413.pr790.39.g07a7000...
Current: leapp-repository-0.15.0-0.20220106125723353413.pr.changes790.39.g07a7000...


Pretrained model

Actually, I was studying the implementation but could not get the pretrained model.I need to get the Seq2seq_summarization.cpkt file. I downloaded the zip file from the repositories but the pretrained model was not in. Please I need to have it.

I don't think I released the pre-trained model (I don't think I even really fully trained it). At this point I don't think I have the checkpoint.

Ok sir, but do you have an idea of where to get the pretrained model? I came across one but I wasn't sure of it. It is called BERT Summarizer and it's file size is 1.56gb. Can I use it?

This repo is an old code originally made before the Transformer Pretraining era. You can use BERT summarizer but that would be unrelated to anything in this repo.


preferredPresentationBackgroundColor is not dark / light responsive

https://github.com/nathantannar4/Transmission/assets/1319079/e4b568ec-0b5c-47ff-a5ab-7c7efceca385
Reproduction repo — https://github.com/vpontis/transmission-bg-issue
One way of solving this is to pass in a UIColor instead of a Color.
cc @nathantannar4

Fixed in 1.1.2


Scroll programmatically to specific position

Hi,
I want to scroll programmatically to a previous position. I enter the app and then enter in one item on the end of the list. When I go back to the grid I want it to be positioned on the last position.
How can I do that?
Thanks in advance,

Hi,
I'm sorry, but I can't help you with your problem, maybe you should look at the source code and add some changes there, but I think scrolling to a position is not supported out od the box.
But maybe you can help me with my problem: What does the constructor of your AsymmetricGridViewAdapter looks like?

Did someone solve this problem? if yes help please. Thanks in advance
@MauroAlexandro @krishnapatel086

not best way. But, try this method, it is work for me
new Handler().postDelayed(new Runnable() {
@Override
public void run() {
recyclerView.smoothScrollToPosition(mToPosition);
}
}, 100);


aio update only updates the plugins but not the CLI itself

currently, aio update updates their associated plugin (currently only looking for @adobe plugins) without having to reinstall the whole cli. for the cli itself, we notify users to install the update for 7 days when there’s a new version. (#132)
User shared that the command sounds like aio update updates everything, and when the user see the CLI recommending to run aio update when there are new features, but if user runs aio update it still complains about being out of date afterwards:
›   Warning: @adobe/aio-cli update available from 3.5.0 to 3.6.0.
›   Run npm install -g @adobe/aio-cli to update.
Additional feedback:
"I assume a lot of developers coming to aio will have no npm knowledge, and aio might be their first touchpoint with it. meaning they probably have no understanding that you have to run the install command npm install -g … again for “updating” something."
The solution may relate to #138

Indeed I think it does relate to #138 . Meanwhile, the output of aio --help is quite obvious:
update        Update all installed plugins.
What about adding to the description: Update all installed plugins. To update the CLI, run npm install -g @adobe/aio-cli?

More issues as discussed on slack:
If I only do
aio update

...

aio update

it works
but if i do this
aio update

...

npm i -g @adobe/aio-cli

I won’t get the updates of plugins that happened to be updated before
combined with aio update not updating the core cli, this means there are two incompatible update mechanisms that allow me to shoot myself in the foot (without knowing it)
IMO there should only be

For updating anything core aio (including core plugins)
npm i -g @adobe/aio-cli

For updating any extra plugins
aio plugins update

@sarahxxu I think we should proceed with this proposal based on Alex's recommendation:

aio update will only update user-installed non-core plugins, never any core plugins
aio update will separately list any core-plugins that can be updated, and show instructions on how to update them (e.g. reinstall the cli npm install -g @adobe/aio-cli)
aio info: if there are any user-installed core-plugins, a warning should be shown that they might be incompatible (and a reinstall of the cli to update may needed)