Consider custom/aliased Option-like enums with skip_serializing_none

Would it be possible to add support for custom Option-like enums in order to do skip_serializing_none?
For example, Martin tile server has OptOneMany and FileConfigEnum and a few others (see code). Both enums have a value considered as "should be skipped when serializing".  At the moment, I am forced to add this line to each field of that type:
#[derive(Serialize)]
struct Foo {
  #[serde(default, skip_serializing_if = "OptOneMany::is_none")]
  field: OptOneMany,
}

It would be wonderful if skip_serializing_none could be extended to either:

Add some magical tags to OptOneMany::NoVals so the struct fields of this type are always gets skipped by default
Add some tag on struct that adds the above serde attribute (with both "default" and the "if")

Some ideas for the later option:
#[derive(Serialize)]
#[skip_serializing_if(type = "OptOneMany", if = "OptOneMany::is_none", use_default)]
struct Foo {
  field: OptOneMany,
}

Instead of extending skip_serializing_none, I would suggest using serde_with::apply unless there is a reason you cannot do so. It applies a set of other attributes to all fields with a matching type. In your case, you want the serde attribute that you write above each field, as part of the apply macro. You can cover both structs OptOneMany and FileConfigEnum in a single invocation.
#[serde_with::apply(
    OptOneMany => #[serde(default, skip_serializing_if = "OptOneMany::is_none")],
    FileConfigEnum => ...,
)]
#[derive(Serialize)]
struct Foo {
    field: OptOneMany,
}

@jonasbb thanks, I didn't realize apply could do that, thanks.  The only issue I see is that I would still have to do it everywhere - which would become fairly verbose because of all duplications (I have 5-10 structs used for serialization).  Ideally, I think it would be far more useful if skip_serialize_none could be extended to tell it which enums behave "like Option" - i.e. to add an attribute on my own custom types:
#[derive(Debug, Default, Clone, PartialEq, Serialize, Deserialize)]
#[serde(untagged)]
#[serde_with::treat_as_none(NoVals)]  // <-- this tells serde_with that NoVals is just like Option::None
pub enum OptOneMany<T> {
    #[default]
    NoVals,
    One(T),
    Many(Vec<T>),
}

Rust macros do not work like that. I don't know of any way that would make this work.
Rust macros work on token only. They cannot know what type something is, since they run before typechecking. This makes it impossible for the skip_serialize_none to lookup how to handle OptOneMany. skip_serializing_none only recognizes some patterns as Option. They are listed under Limitations. Even a rename, like use std::option::Option as FooBar will break the macro.
Another problem is that macro execution order is undefined. You cannot be sure that #[serde_with::treat_as_option(NoVals)]  will be processed before skip_serialize_none.
The only option would be to make the OptOneMany look basically indistinguishable from the real Option type. I do not recommend this approach. It would require renaming/aliasing the type, having a is_none function, etc.

@jonasbb thanks for in depth explanation, agree, sadly it seems like a no go.  One option of course would be to use the trait system rather than (or in addition to) the macros.  Something like pub trait IsOptionable { fn is_none(&self) -> bool; }. You would implement that trait on the Option object (and possibly a few others?), but then each other non-option value would still have to be tagged with some attribute like #[optionable] - at least once per type in each struct (e.g. if there are multiple instances of the same type, derive macro can notice that the same type has already been identified as optionable.

The skip_serialize_none macro needs to decide at compile time, before type checking, if a field is of type Option or not. I am not sure how you think the macro is supposed to use the IsOptionable trait. Could you expand how you imagine that to work?

#[derive(Serialize)]
#[skip_serializing_none]
struct Foo {
  #[SkipIfNone]       // <- this tells serde_with that field1, as well as other fields
  field1: OptOneMany, // of type OptOneMany should be treated as having a   .is_none()
                      // Technically we do not even need a dedicated trait - enough to just

  field2: OptOneMany,
  field3: OptOneMany,
  field4: SomeOtherType, // <-- serialized as usual
}

To be honest, I am not too thrilled about this solution either -- magic is usually bad. The cleaner (but still a bit hacky?) is this:
#[derive(Serialize)]
#[skip_serializing_none(OptOneMany)]  // <-- all fields of the listed types are assumed to have .is_none()
struct Foo {
  field1: OptOneMany,
  field2: OptOneMany,
  field3: OptOneMany,
}


Display amount of data deleted after prune

When pruning, restic logs the amount of data in use and amount of blobs still in use but not the amount of data deleted:
...
repository contains 1160 packs (166518 blobs) with 5.134 GiB bytes
...
found 166260 of 166518 data blobs still in use

A couple of ideas on how to log it:

found 166260 of 166518 data blobs still in use, deleting NNN blobs (J.JJJ GiB)
done, deleted NNN blobs, J.JJ GiB

Thanks for the suggestion, that sounds like a good idea.


Claim on main page of documentation: "Elm meets Phoenix LiveView"

At the heart of liveview, as I understand it, is a mechanism to communicate seamlessly between front-end and back-end.
I'm intrigued by this claim on the main page of lustre docs:
"Universal components. Write once, run anywhere. Elm meets Phoenix LiveView"
I wasn't able to find any documentation on a feature in lustre which approximates liveview.  Did I miss it?
I also heard Louis make a similar parallel to liveview, i think in the gleam 1.0 presentation.
If lustre doesn't have liveview-like functionality, or there is no documentation on this yet, then perhaps the claim should be removed for now, to save people the frustration of searching.  (again sorry if I'm a dummy and missing something obvious).

You can find the api reference for server components here: https://hexdocs.pm/lustre/lustre/server_component.html
An example of how to set them up using mist here: https://github.com/lustre-labs/lustre/tree/main/examples/99-server-components
And here is a repo showcasing the same lustre application used as a SPA, a web component, and as a server component: https://github.com/hayleigh-dot-dev/lustre-universal-component-demo/tree/main


[Bug] Detected inconsistent metadata for the path "node_modules/react/index.js"

[plugin vite:dep-pre-bundle] Detected inconsistent metadata for the path "node_modules/react/index.js" when it was imported here:

    node_modules/mini-create-react-context/dist/esm/index.js:1:33:
      1 │ import React, { Component } from 'react';
        ╵                                  ~~~~~~~

  The original metadata for that path comes from when it was imported here:

    dep:react:1:23:
      1 │ export default require("../../node_modules/react/index.js");
        ╵                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  The difference in metadata is displayed below:

   {
  -  "pluginName": null,
  +  "pluginName": "vite:dep-pre-bundle",
   }

check https://github.com/vitejs/vite/issues/7683

fixed in https://github.com/phshy0607/docit/pull/6


We need a mechanism to support authenticated feeds with package management.

nuget has an authenticated feeds mechanism where credentials are required to access a feed.
We need a mechanism to allow notebook users access to these types of feeds, however ... putting plaintext credentials in script files, or indeed any other type is not really very secure, so we should come up with something a little more secure than that.  Ideally without us doing any credential management or storage in our code, because the security is so hard to get right.

I'm struggling to find a work-around for this. Does anyone have one? Both dotnet and nuget command line tools can access my authenticated feed but the Jupyter notebook simply says "C:\Program Files\dotnet\sdk\5.0.101\NuGet.targets(131,5): error : Unable to load the service index for source <> C:\Program Files\dotnet\sdk\5.0.101\NuGet.targets(131,5): error :   Response status code does not indicate success: 401 (Unauthorized)."

The following command line worked for me to add my private Azure artifacts feed using its generated PAT password. Then notebook is able to load my packages.
https://docs.microsoft.com/en-us/azure/devops/artifacts/nuget/nuget-exe?view=azure-devops
nuget sources add -name xxxx -source "https://xxxx.pkgs.visualstudio.com/_packaging/xxxx" -username "dummy" -password "xxxx"

Thanks for taking the time to reply.  I believed I had followed those instructions but following your assertion that this is the correct route I tried again. Following the removal of the nuget source and re-adding it I was able to reference and load a package as expected, so I'm really not sure what was wrong. Thanks again.

I'm having the same issue, any progress on a solution for this.

any progress on a solution for this

this is the command line I use to add the source. note that I use "add" instead of "update" and the codeblock is a one-time setup block - don't execute it every time:
dotnet nuget add source "https://nuget.pkg.github.com/USERNAME/index.json" --name SOURCENAME --username "USERNAME" --password "PERSONAL_PAT" --store-password-in-clear-text
This will add the nuget source to the user-level nuget.config and is for github but should work for azure as well.
https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-nuget-registry
After that, a simple #r "nuget: PACKAGENAME" works well.


A time format props

#58
Maybe can add a new props like timeFormatter, or something like this, it can receive a string or a function. Then we can render time ui style by using this props, like timeFormatter={'HH:MM'} or timeFormatter={'HH : MM'} or even timeFormatter={(hour, minute) =>hour&minute}

New version v2.2.4 published, add timeFormat and timeFormatter props.


How to handle an array of embedded JSON Objects?

Given the following example POST payload, how would you configure rspec_api_documentation for the authors key given it is an Array of N-objects?
{
  "book" : {
    "id": "2",
    "name": "Awesome",
    "authors": [
      { "first_name": "Justin", "last_name": "Smestad" }
    ]
  }
}

I'm also trying to figure out how to handle arrays. Did you ever come up with a solution to this?

something like that:
...
parameter :authors, scope: :book
let(:authors) do
  [{
	first_name: 'Justin',
	last_name: 'Smestad'
  }]
end

route_description <<-DESC
  authors:
    first_name: :string, :required
    last_name: :string, :required
DESC


Read data from WFS of rato vzw

This PR adds a script, ./src/read_data_from_wfs.R to download data from RATO vzw and save it in ./data/raw. As it is short and in the near future I am going to use it in a GitHub Action workflow for automatizing the download, I preferred to not write a Rmd around it.
As new columns have been added and also the " " are not anymore present in the retrieved data, I preferred to not overwrite the file rato_data.csv. Instead, I saved the new retrieved data as rato_data_new.csv. This has been outlined in few new chunks in the dwc_mapping.Rmd. Notice how you can read data by using the standard readr::read_csv().
@LienReyserhove: of course, feel free to change filenames and remove these chunks in Rmd!
The goal of this PR is to provide the tool to get RATO data and I am happy we have it quite easily 👍

Thank you for these additions. They look fine to me, I will try to merge this information with the current script.


[Java] toString generation for debugging purpose

PoC for #357
Generate toString and appendTo(StringBuilder) method for generated components, targeting light one-liner display.
Sample result:
[Car](sbeTemplateId=1|sbeSchemaId=1|sbeSchemaVersion=1|sbeBlockLength=41):serialNumber=12345|modelYear=2005|available=T|code=A|someNumbers=[0,1,2,3,4]|vehicleCode=abcdef|extras={sunRoof,sportsPack}|engine=[capacity=4200|numCylinders=8|manufacturerCode=abc]|fuelFigures={[speed=30|mpg=35.9][speed=55|mpg=49.0][speed=75|mpg=40.0]}|performanceFigures={[octaneRating=95|acceleration={[mph=30|seconds=4.0][mph=60|seconds=7.5][mph=100|seconds=12.2]}][octaneRating=99|acceleration={[mph=30|seconds=3.8][mph=60|seconds=7.1][mph=100|seconds=11.8]}]}|make=MAKE|model=MODEL|
Change-Id: I3b45eb18c24ddbc3d6b4c278b38e677d4f6245ed

Thank you for review, and sorry for the account mixup.

I'd done a little tidy up and some testing. Couple of things I noticed:

Optional types such as cupHolderCount when null do not get printed out.
The object does not get closed off for automated parsing.
I'm not sure what is the consistency for use of square and curly brackets.

I also noticed the a bug if you output the extension for boosterType.
[Car](sbeTemplateId=1|sbeSchemaId=1|sbeSchemaVersion=0|sbeBlockLength=47):serialNumber=1234|modelYear=2013|available=T|code=A|someNumbers=[0,1,2,3,4]|vehicleCode=abcdef|extras={sportsPack,cruiseControl}|engine=[capacity=2000|numCylinders=4|manufacturerCode=123|booster=[boostType=NITROUS|boostType=|horsePower=200]]|fuelFigures={[speed=30|mpg=35.9|usageDescription=Urban Cycle][speed=55|mpg=49.0|usageDescription=Combined Cycle][speed=75|mpg=40.0|usageDescription=Highway Cycle]}|performanceFigures={[octaneRating=95|acceleration={[mph=30|seconds=4.0][mph=60|seconds=7.5][mph=100|seconds=12.2]}][octaneRating=99|acceleration={[mph=30|seconds=3.8][mph=60|seconds=7.1][mph=100|seconds=11.8]}]}|make=Honda|model=Civic VTi|activationCode=abcdef|

Thanks ! Will address at least the bug and 3 (likely the rest too later).
currently preparing a PR for the bug.

Many thanks for the contribution!


Accessibility : High contrast mode, menu items not visible

VSCode Version: Code 1.5.3 (5be4091987a98e3870d89d630eb87be6d9bafd27, 2016-09-22T08:50:42.185Z)
OS Version: Windows_NT ia32 10.0.14393

Steps to Reproduce:
Open Vscode in high contrast mode
Menu items are not visible

duplicate of #6629

Tracked as #6629


Fix WebGPU none autoplay video

Fix WebGPU none autoplay video

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15177/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15177/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15177/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15177/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15177/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15177/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15177/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15177/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15177/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15177/merge/testResults/webgl2playwright/index.html


Correct possible misplaced p_weight and n_weight variables

The weights are correct. You can see that the class with a smaller number of samples is multiplied by a larger number, and the class with more samples is multiplied by a smaller number. The idea is in the final sum, both classes, to be equally weighted.


add blog post for 5.0 release

You should get Ivo to take a look for wording around rule to starlark changes.

For the record, everything in that section was reviewed/rewritten by Ivo in the draft release notes doc.

LGTM once everyone else's comments have been addressed.

Thanks all!


invoke method

Hi there, could you please add the support to void invoke?
Right now is waiting for a result, and the await for a result, but this block the ui sometimes

Hey @tomasgaray
If you don't want to wait for the results, call this method asynchronously.


Question : Are references to other files supported ?

Hello,
How can I handle reference ($ref) to other files ?
I can't tell if I'm doing it wrong or if it's not supported.
My only goal is to instantiate Swagger UI with the json file.
Maybe I shouldn't be using the json file for that ?
Thank you,
Romain

That's exactly the idea of the json processor. Convert the yaml to json and feed it to Swagger UI.
The swagger parser is able to "flatten" the openapi.yaml. The expected behavior is, that the parser resolves all $refs (flattening the openapi into a single document), and then dumps it as json.
The json processor has only a simple 'one file in' , 'one file out' test. Guess I should check my expectation with a few more tests the use $refs. :-)
Below are a few examples that show that both parsers (swagger, openapi4j) generally handle $refs (the json processor is using the swagger parser).

https://github.com/openapi-processor/openapi-processor-core/tree/master/src/testInt/resources/tests/ref-is-relative-to-current-file/inputs
https://github.com/openapi-processor/openapi-processor-core/tree/master/src/testInt/resources/tests/ref-into-another-file/inputs
https://github.com/openapi-processor/openapi-processor-core/tree/master/src/testInt/resources/tests/ref-into-another-file-path/inputs


ITaskSummary and events should carry the number of detected problems

See https://github.com/Microsoft/vscode/issues/30766 and the fixes and workarounds around it. This is especially necessary for watching tasks since they might miss errors since the marker stats are updates async.

Moving to Oktober. The code is on branch dbaeumer/31968. But a little late for September.

Lots of snow plowing ❄😁


SCM view should be customizable

Testing: https://github.com/Microsoft/vscode/issues/34963
The same as the explorer: it should be possible to always show/hide the "SCM Repositories" view, even if there is just one repo. And if there is multiple I should be able to hide it. We could always allow to change the active SCM repo from the "..." menu too.

Except that the metaphor is different than the explorer. I can't hide the master view. I'll merge this with #35026


Add cqu.edu.cn domain

@VortexMashiro I am sorry, the domain cqu.edu.cn is blacklisted because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


Add link to self-service page (didmos2 frontend)

Just would like to add that the link on the icon of the user opens the self service portal in a new window.

However, the "Back to Crust", simply opens Crust (instead of closing the browser tab)

So, you end up with two browser tabs with Crust.
Maybe the link on the icon of the user should not open in a new browser tab?


bugfix: api_image_create_test duplicate image tag

Signed-off-by: HusterWan<EMAIL_ADDRESS>1.Describe what this PR did
api_image_create_test fromImage not split image name and tag, so in daemon, the image ref is registry.hub.docker.com/library/busybox:latest:latest
2.Does this pull request fix one issue?

3.Describe how you did it
4.Describe how to verify it
5.Special notes for reviews

@allencloud CI Failed, Wired Error: snapshot not found

Please rebase to the latest master, since PR https://github.com/alibaba/pouch/pull/436 which solved flaky test has been merged. Thanks.

LGTM


Doesn't work: "mat4 is undefined" (FF) / "Cannot read property 'perspective' of undefined" (Chromium)

After installing shader-school using npm (see below) I get errors in Firefox and Chromium:
Firefox (version 38.0.5) tells me that "mat4 is undifined" while Chromium (version 43.0.2357.125) "Cannot read property 'perspective' of undefined".
The errors appear in a red rectangle on the right bottom as soon as one of the lessons is loaded (e.g. http://localhost:12492/intro-0).
I only see a black screen and the introduction (as soon as I click the button in the top left corner) plus a folder and a home button. Solving an exercise and clicking verify does not work (I'm pretty confident I solved the first one right).
The output after a request using Firefox:
200    2ms       282B  /
200 3651ms     8.18MB  /index.js ➞ /home/milan/Code/shader-school/node_modules/shader-school/node_modules/browserify ./index.js -d -t /home/milan/Code/shader-school/node_modules/shader-school/node_modules/brfs/index.js -t [ /home/milan/Code/shader-school/node_modules/shader-school/node_modules/envify/index.js --dirname intro-0 --file_hello_glsl /home/milan/Code/shader-school/00-intro-0/hello.glsl ] -t /home/milan/Code/shader-school/node_modules/shader-school/node_modules/glslify-live/transform.js -t /home/milan/Code/shader-school/node_modules/shader-school/node_modules/glslify/index.js

I istalled shader-school using the following command sequence: cd */Code/; npm install -g shader-school --prefix shader-school; mv shader-school/lib/ shader-school/; ln -s shader-school/node_modules/shader-school/index.js .local/bin/

me too. any news on that?

me too. any news on that?

I have reinstalled everything and now it works ok


Type-level compatibility with raw stdlib map and set

Would you be interested in a patch to add this?  It seems like it should be possible, though I haven't tried to apply it to asetmap yet.
For example, given:
module Map2(O : Map.OrderedType) : sig
  include Map.S
    with type 'a t = 'a Map.Make(O).t
    with type key = Map.Make(O).key
end = struct
  include Map.Make(O)
end

The follow types properly:
module String_map1 = Map.Make(String)
module String_map2 = Map2(String)
let () = assert (String_map1.empty = String_map2.empty)

The main benefit being theoretical interoperability with other libraries using raw stdlib Map/Set

The changes were very simple so I created a PR if you're interested.

Thanks !


Fix/ci

What's new

Fixes broken CI for

[ ] nightly, dashboard-e2e
[x] dashboard
[x] api-server, for checking None issues, https://github.com/microsoft/pyright/issues/2007

Changing the e2e test to check for coe_door instead of main_door as the second pane sometimes gets stuck

Self-checks

[ ] I have prototyped this new feature (if necessary) on Figma
[ ] I'm familiar with and follow this Typescript guideline
[ ] I added unit-tests for new components
[ ] I tried testing edge cases
[ ] I tested the behavior of the components that interact with the backend, with an e2e test

After a few days of attempting to fix dashboard-e2e I have concluded that it is incredibly flaky running on a github workflow :sweat_smile: After re-running it for 10+ times, it has finally passed. We might need to re-consider running it on github runners

There is still a high chance that dashboard-e2e is going to fail on the main branch, I think we might need to be a bit more mindful abuot that and run it locally to make sure thigns work. Otherwise we can try to migrate this workflow onto something else more robust than a github runner.


fix: chdir back to the starting directory before exit

This PR introduces an explicit return to the directory in which the program began execution to make its behavior more predictable, and improves the resolution of the path specified in baseDir.
Fixes https://github.com/asyncapi/bundler/issues/166

/rtm


Make name and version optional in the json-schema

Describe the bug:
When creating a vcpkg.json for manifest-mode and setting the schema for it, it complains about missing name and version fields even though they have not been necessary for a while.
Environment

OS: Windows 11
IDE: CLion 2022.3

To Reproduce:
Steps to reproduce the behavior:

Create a vcpkg.json
Set $schema to the current schema

(this will also show that $schema isn't an allowed field)
Expected behavior:
For end-applications name and version are not as necessary. And for the tool itself these fields are also not required anymore, so this would just bring the schema up-to-date to the tool.
Might also be worth allowing $schema at the top of the file to allow completion in IDEs where the file doesn't have a schema associated with it.

Would you like to provide vcpkg.json and specific logs?

Well, it's not the vpcpkg-tool that compains, it's the IDE that complains that the current json doesn't fulfill the schema specified
{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg/master/scripts/vcpkg.schema.json"
}

One of the following property sets is required: properties 'name', 'version-string', properties 'name', 'version', properties 'name', 'version-semver', or properties 'name', 'version-date'

and

Property '$schema' is not allowed

So I now that the microsoft/vcpkg-tool has a separate schema file to this repos schema file where this isn't an issue.

The JSON schema has been deleted and the docs haven't been updated, FYI

Thanks @NekkoDroid for turning me onto the new location. Quick enough fix, innit.
{
-   "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg/master/scripts/vcpkg.schema.json",
+   "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",


[SPARK-45167][CONNECT] Python client must call release_all

What changes were proposed in this pull request?
Previously the Python client would not call release_all after fetching all results and leaving the query dangling. The query would then be removed after the five minute timeout.
This patch adds proper testing for calling release all and release until.
Why are the changes needed?
Stabililty
Does this PR introduce any user-facing change?
No
How was this patch tested?
New UT
Was this patch authored or co-authored using generative AI tooling?
No

This is a warning, but not an error. The actual error is a bit further down:
AssertionError: 'java.nio.channels.ClosedChannelException[11325 chars]1)\n' != None

@HyukjinKwon @juliuszsompolski I fixed the issue that stemmed from the test shutting down the channel without cleaning up the threadpool of the generator used for the release requests.

Got it. Thank you for updating.

Merged to master. If needed, please make a backporting PR to branch-3.5.


CosmosDb: up the minor build version

This is done in order to release a hot fix for a CFP related change/bug fix.

@kushagraThapar what is the versioning process for v3?
I see 3.2.0 is the latest published here (https://mvnrepository.com/artifact/com.microsoft.azure/azure-cosmos)
But in the pom files we have 3.3.0. That's a bit confusing as did we release 3.3.0? if not why 3.3.0 is in the pom? aren't we following the snapshot versioning pattern anymore?

@kushagraThapar what is the versioning process for v3?
I see 3.2.0 is the latest published here (https://mvnrepository.com/artifact/com.microsoft.azure/azure-cosmos)
But in the pom files we have 3.3.0. That's a bit confusing as did we release 3.3.0? if not why 3.3.0 is in the pom? aren't we following the snapshot versioning pattern anymore?

Yes, it is a bit confusing. We were following the snapshot pattern, but recently someone from azure-sdk team updated our branch with 3.3.0 version.
https://github.com/Azure/azure-sdk-for-java/commit/0dce83bb8a7c167442573c94ba71587ad04ff8d5

@kushagraThapar so this was a one time thing? are we going to switch to snapshot after each release?

@moderakh This PR will not be merge; its only purpose is to review that I captured all the files that need to be updated with the hot fix patch.

@milismsft @moderakh For reference the versioning model we are trying to use is documented in https://github.com/Azure/azure-sdk/blob/master/docs/policies/releases.md. We are not using SNAPSHOT versions in our java repo as they cause too many reliability issues and we generally bump the minor version after a GA release in order to reserve versioning space for any potential hotfixes. That is why the version was bumped from 3.2.0 to 3.3.0, is there a particular reason you guys want to only bump the patch version?

@weshaggard Just like it is mentioned in the document you referenced, "increments to the patch number (1.1.1 to 1.1.2) indicate hotfixes". This is exactly why we are trying to release 3.2.1 which addresses a bug fix rather than a non-breaking new feature.
Could you list some of reliability issues that "SNAPSHOT" in the version adds?

@milismsft thanks for the context I didn't realize you were trying to do a hotfix. We generally keep the master branch with the bumped minor version because we want what is in master to be a different version than what was last published on maven. For hotfixes we generally recommend branching from the last release tag (https://github.com/Azure/azure-sdk/blob/master/docs/policies/repobranching.md#hotfix-branches) and applying just the fix you want, otherwise you risk picking up other changes you didn't want for that hotfix. If you don't believe there are any other changes then then ones you want doing the version change directly in master will accomplish the goal but in general is not the preferred workflow.
The biggest issues with SNAPSHOT versions is that when you depend on them they automatically pull the latest version each time you restore and by doing so makes repeatable builds much more problematic because often times SNAPSHOT builds are daily builds which don't have the same quality bar nor breaking change considerations.


Start mintty bash in current working directory

How about starting mintty in the current working directory instead of the home dir?
This is convenient for spawning msys2 for example in a GIT repo from the cmd line or external programs such as (g)Vim.
This can be achieved by modifying the following line in c:\msys64\msys2_shell.bat
start %WD%mintty -i /msys2.ico /usr/bin/env CHERE_INVOKING=1 /usr/bin/bash --login %*

to
start %WD%mintty -i /msys2.ico /usr/bin/bash --login %*

If this is to intrusive, there are two other possibilities:

How about pulling %* forward, that is

start %WD%mintty -i /msys2.ico %* /usr/bin/bash --login

instead of
start %WD%mintty -i /msys2.ico /usr/bin/bash --login %*

so that the user can pass on /usr/bin/env CHERE_INVOKING=1 to msys2?

How about an argument, say -here, that can be passed to msys2_shell.bat and then only in this case spawns msys2_shell.bat in the current working directory?

Both msys2-launcher launchers and msys2_shell.cmd support starting in current directory.


Scatter plots coloring when a gene in the column specified by gene_symbols= corresponds to multiple entries

[x] Additional function parameters / changed functionality / changed defaults?
[ ] New analysis tool: A simple analysis tool you have been using and are missing in sc.tools?
[ ] New plotting function: A kind of plot you would like to seein sc.pl?
[ ] External tools: Do you know an existing package that should go into sc.external.*?
[ ] Other?

...
When coloring by a gene in a column of .var specified by gene_symbols= and when there are multiple entries in .var sharing the same value of that attribute, currently only the first entry is used to coloring. Would you consider allowing a different summarisation, e.g. mean/sum/etc? An existing work around is that the user calculate that summarisation oneself and insert it into .obs, but it becomes tedious when the number grows. For example, this occurs when mapping genes across species with one-to-many mappings kept.
https://github.com/theislab/scanpy/blob/233aa15adbf565a7cb278d0724e3cb3f566b749f/scanpy/plotting/_tools/scatterplots.py#L844

We could possibly add another parameter, (handle_duplicates, duplicates_action?), which could specify how to do this. I think the best default behavior for this is to throw an error.
@fidelram, @VolkerBergen what do you think? I know we've been trying to reduce complexity in these methods, so is this worth it?

Thanks for the quick reply. I think it makes sense to separate heavy data handling from plotting.

I think an error should be thrown, thus the user can figure out what to do.
@nh3 I don't understand the part about the different summarization.


Run SHARK model suite on IREE post-submit

We'd like to include a run of the sizeable model suite that Nod.ai is developing in IREE post-submits:
https://github.com/nod-ai/SHARK
https://github.com/nod-ai/SHARK/runs/6920622600?check_suite_focus=true
Current execution time is 1h45m. Can be sharded as necessary.
@GMNGeoffrey wants to provide input on whether the workflow should always succeed, or fail on non-xfail tests failing.
@powderluv: Can you or your team create the workflow to run this? Note that our self-hosted runners are going to be delayed -- is it possible to stand up a version with just 1 model that can run in a reasonable amount of time on Github-hosted runners? When we get the self-hosted runners working, should this be run on an a2-highgpu-1g?

@powderluv I'm copying questions from above in case you missed them:
Can you or your team create the workflow to run this? Note that our self-hosted runners are going to be delayed -- is it possible to stand up a version with just 1 model that can run in a reasonable amount of time on Github-hosted runners? When we get the self-hosted runners working, should this be run on an a2-highgpu-1g?

I can make the SHARK runners use stock IREE nightly releases and run the same tests.

how long do you estimate just the test execution (excluding the build) will take on a Github-hosted runner?

GH runner fails some tests since it is a dual core VM with limited RAM. We can pick a few tests that pass there and avoid any benchmarking

Sounds good. Would the IREE workflow checkout the shark repo to gain access to the test scripts, etc? Seems sub-optimal to duplicate those in the IREE repo.

Assigning to @GMNGeoffrey to help get the runner setup.

Should be easy to add. I can take this. Is there an example of a workflow taking a post submit and building an IREE compiler / runtime wheel ? Once we have that it is easy for me to wire into the shark test suite.

https://github.com/iree-org/iree/blob/main/.github/workflows/build_package.yml

One other note from Geoffrey: make sure that the workflow checks out a specific commit of the shark repo -- we'll update it manually as needed

From meeting today, draft PR up, @powderluv checking

draft PR: #9748

PR should be in review today

PR still in draft today FYI

PR is still in draft as of today

PR is still in draft as of today

Merge PR with ability to disable.

Is this closed?

Yep
On Thu, Apr 6, 2023 at 8:06 AM Allie Culp @.***> wrote:

Is this closed?
—
Reply to this email directly, view it on GitHub
https://github.com/openxla/iree/issues/9537#issuecomment-1498217542, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHHUFYXEFTF7VTCYVWHNNLW7XUH7ANCNFSM5ZAQ33HA
.
You are receiving this because you commented.Message ID:
@.***>


PR-000: add credit card form

Support for Credit Card payment including DropView and UI Elements
Notice: 3ds in not yet supported as BE is returning me 500.

PR replaced with https://github.com/payrails/ios-sdk/pull/9


@angular migration needed

Source:
https://github.com/angular/angular/blob/master/CHANGELOG.md
A lot of things changed in angular-2.0.0-rc.0 and we need them in angular2-logger asap to avoid dependency conflicts, I cloned the repo to do it by myself and issue a pull request, but if you can do it by yourself, I'm sure it'll be better/faster :+1:

Hello @Supamiu
Once again, thanks for the help! I saw the changes in the quickstart in the main page, whoever the npm repo is still pointing to beta.17 and if you run npm i angular2 you still get the beta.17, which doesn't have these changes yet.
More than likely they're getting ready for the ng-conf tomorrow. As soon as all sources get updated I'll apply your changes.

npm i angular2 will always be pointing to beta.17, angular rc0 is now @angular, you can't install angular2 by just doing npm i angular2 now.
see rc0 changelog for more informations

You are right, now we have to use npm i --save @angular/core ....
I see that you fixed your pull request but you made a merge without a rebase, I'm trying to keep the history clean of merges so I'm going to try to fork somehow your pull request to fix that before accepting it if that's ok with you.


Badgets are cut

Hello
Ive noticed that every time im on tab which has badge and then I clicked the tab next to it on the right
it makes the badge cut and you can see only half number

Can you provide some more info? How many tabs do you have? Does this issue happen on tablet or mobile?

The issue happens on mobile, Ive got 5 tabs and it happens on the fifth tab so when the fifth tab is clicked the badge is hidden and if I click the forth tab the badge comes back in the same place it was suppose to be if it was visible when the fifth tab was clicked (Im using shifting behavior so the selected tab get a bit bigger) so the badge is cut in half

You're absolutely right. I have a fix coming up in the next version.

Hi, when the version 2.0.3? I`m very needs in this fix :)

I also vary need this fix

Same here 👍

Manual fix is simple: just comment out updateTitleBottomPadding() in BottomBar.java
It's supposed to add some font padding (10dp) to comply with Material guidelines, but in reality it does nothing but cut textviews.

Hi, when the version 2.0.3 will release? This issue is really bad for UX point of view


Add multi OS support

Problem statement
One key advantage of Clear Containers over runc containers is the ability for each container/pod to run on top of its own kernel and guest image.
For customized workloads like e.g. NFV ones, this is very important as they sometimes need some specific kernel features that can be mutually exclusive with other workload needs.
Unfortunately, the OCI runtime specifications do not allow (yet) for specifying a kernel and guest image and thus virtcontainers callers can only use a default path for each of those and pass it through the HypervisorConfig structure.
Proposed solutions
We want to be able to define a kernel and a guest image paths per container/pod. Both of them should be absolute paths, and should be optional. In other words, there must be a default value for those paths which will be passed to virtcontainers through the HypervisorConfig structure. Clear Containers, for example, gets the default paths for its kernel and guest image through the configuration.toml system wide configuration file.
Short term solution: Annotations
As a short term solution we are going to use virtcontainers namespaced pod annotations for passing a kernel and guest image absolute path:
// KernelPath is a pod annotation for passing a per container path pointing at the kernel needed to boot the container VM.
KernelPath = "com.github.containers.virtcontainers.KernelPath"

// ImagePath is an pod annotation for passing a per container path pointing at the guest image that will run the container VM.
ImagePath = "com.github.containers.virtcontainers.ImagePath"

Both annotations can be set through virtcontainers PodConfig annotations. Kernel or image path ContainerConfig annotations will be ignored.
Long term solution: OCI specification changes
See https://github.com/opencontainers/runtime-spec/pull/405
Code path
The virtcontainers code path should be identical for both solutions: The pod.go will trap pod creation request and modify its config.HypervisorConfig structure if there is an ImagePath and/or KernelPath annotation as part of its PodConfig
The pkg/oci/utils.go file will be responsible for adding the right pod annotation, either by parsing the OCI config.json annotations or by parsing the future VM specific section in that file. The latter should take precedence over the former.
Error handling
If any of the kernel or guest image path is invalid (empty files, or inexistent ones), virtcontainers will error out and stop creating the pod's VM. Passing an image or/and a guest image into the container configuration file is a very specific requirement from the user and thus switching to the default value when one of those paths is invalid could be misleading and error prone.
Security
Optionally virtcontainers callers can require kernel and/or image binaries integrity checking. For that purpose, virtcontainers will handle 2 additional annotations:
// KernelHash is a pod annotation for passing a container kernel image SHA-512 hash value.
KernelHash = "com.github.containers.virtcontainers.KernelHash"

// ImageHash is an pod annotation for passing a container guest image SHA-512 hash value.
ImageHash = "com.github.containers.virtcontainers.ImageHash"

By default virtcontainers will not verify the kernel and image binaries integrity unless one or both of the hash annotations are passed in the PodConfig annotations. In the latter case, virtcontainers will build SHA-512 hashes on the kernel or/and image binaries and compare that against the passed annotations.

A few thoughts:
Definitions
We should specify what we mean by "path" (I vote for only accepting absolutes).
Behaviour
I think we should document the expected behaviour if either of those annotations are blank or invalid in other ways (immediate error or fallback to defaults with a big fat log warning message :)
Kernel/Image selection
As mentioned on the original OCI PR, we need a way to select between images and kernels. Currently for CC 3.0, the runtime handles that via its configuration file. These annotations suggest a broadening of the ability to switch between images / kernels by allowing the caller (the user running, say, docker run) to specify the image/kernel. I guess the basic question is:

Who decides which image+kernel combo the workload gets to run in?

If it's the runtime, I wonder if we need to find a way for workloads to be tagged as needing particular features (such as NFV) to allow the runtime to decide automatically which kernel to choose? That might require we mandate another annotation for KernelConfig = /path/to/kernel/config. Of course, the problem is there are a significant number of kernel config options so finding a way to specify the feature set with kernel config options may not be ideal for various reasons.
Security
We could add checksum annotations for the kernel + image, but that would significantly slow down startup time (if they were being checked). Hence, we should probably document that other facilities (such as package-management features) are used to ensure the images have not been modified.
Stale component versions
Whatever we decide, we need to ensure the runtime is able to identify stale containers. If the kernel+image selection is not going to be decided by the runtime, the runtime will still need a way to identify the range of available images+kernels, and be able to order them by age to be able to identify "stale" containers.

@jodh-intel Thanks for the feedback. Some comments:

Who decides which image+kernel combo the workload gets to run in?

I'd like to look at the problem only from a virtcontainers perspective for now, and I think virtcontainers should not try to be smart about it, but simply work on kernel and/or image paths. Then runtime implementations can decide if they want to resolve higherl level requests into actual paths or not.

We could add checksum annotations for the kernel + image, but that would significantly slow down startup time

That would be an interesting feature, that we could decide to disable by default unless the caller explicitly asks for it. And this could also apply to the default kernel + image files. I'll add that to the issue.

@sameo @jodh-intel this feature change should be accompanied by the ability to retrieve the kernel and runtime in use for containers/pods on a given system using the cc-runtime binary. This may be the same as what we are planning to do to report the version of the kernel and image in use.
Another thing we may want to consider is allowing module loading in clear containers. If we allow for module loading, we will be able to support significantly more combinations and allow the workload to be self contained, and the the workload itself can carry the modules it needs. The module loading support does not need to be supported in the default kernel.
Also we should explore if node tagging can be easily extended to report the availability of various kernels/rootfs combinations.
Today there is no hard linkage between the kernel and rootfs, as they are loosely coupled. But keeping this loose coupling may be an issue in the future, when we allow the kernels to change.

@mcastelino

@sameo @jodh-intel this feature change should be accompanied by the ability to retrieve the kernel and runtime in use for containers/pods on a given system using the cc-runtime binary. This may be the same as what we are planning to do to report the version of the kernel and image in use.

With this new approach we will still be able to know which kernel+image each pod is using by calling cc-runtime list --cc-all.

Another thing we may want to consider is allowing module loading in clear containers.

Agreed. This will have little impact on us, except maybe that we will need to add systemd-udevd to our userspace image.

Today there is no hard linkage between the kernel and rootfs, as they are loosely coupled. But keeping this loose coupling may be an issue in the future, when we allow the kernels to change.

So here what we'd most likely want to know is which kernels are compatible with any given image, I think. Just a thought: We could have osbuilder generate a metadata file for any generated image that would point to the compatible kernel(s) for it.

If we do make a module enabled kernel then I'd recommend ensuring we have module versioning turned on to help ensure we only load modules that were build for that actual kernel:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/kbuild/modules.txt#n426
We don't want to be tracking down issues where somebody has loaded an older or newer module which causes 'strange things to happen'.

cc @eadamsintel


🐞 Directory.glob doesn’t work when parent directory isn’t "."

I found that the Glob being used in https://github.com/dagger/dagger/pull/6884 doesn’t work as expected. Also reported by user in Discord.
Assuming the following directory:
files = dag.directory().with_new_file("foo/bar.txt", "foobar")
src = dag.directory().with_directory("wat", files)

Currently, this works:
print(await src.glob("**/*.txt"))

# Output: ['wat/foo/bar.txt']

But not this:
print(await src.directory("wat").glob("**/*.txt"))

# Error: lstat /tmp/buildkit-mount751124773/wat/wat: no such file or directory

print(await src.directory("wat").glob("**/*.txt"))

# Error: lstat /tmp/buildkit-mount751124773/wat/wat: no such file or directory

q: did you mean directory("foo") here?

q: did you mean directory("foo") here? given that /wat/wat doesn't effectively exist.

No, see this line:
src = dag.directory().with_directory("wat", files)

I'd expect it to find files in /wat, not /wat/wat.


SSL_CTX_load_verify_store

Hi All,
We are trying to NOT use physical certificate or private key files in our TLS implementation of a client(REQUESTER_CONNECTION).
Instead we want to use certificates from our Windows Certificate Store.
I will split my queries into 3:
[1]
So, when peer verification is enabled, SSL_CTX_load_verify_locations cannot be used as this involves file or folder paths.
Was thinking if we could use SSL_CTX_load_verify_store instead.
Unfortunately, I am unable to understand how the CAStore (URI) parameter can be given.
Can someone throw some light on the same?
[2]
Instead, can I set a call back function to perform the same using SSL_CTX_set_cert_verify_callback ?
I am reluctant to do this as I see call back functions mostly for servers.
Is it fine to use call backs even if it is a client?
[3]
SSL_CTX_set_verify has the provision to give call back directly as a the third parameter.
So, is it wrong to set the call back function separately using SSL_CTX_set_cert_verify_callback?
Sorry for raising too many queries.
This is unknown territory for me and so any pointers will be of much help.
Thanks in advance.
Regards,
Amal

You would need a provider with an implementation of OSSL_STORE that would serve the certificates from the windows CA store.

@t8m
Thanks for the inputs.
Just to try, I used SSL_CTX_set_cert_verify_callback  and in made the call back function return true(1) always.
Unfortunately, am still not able to establish association with the server.
The server is java based and throws up the below error:
javax.net.ssl.SSLHandshakeException: Empty server certificate chain in State: Sta2 - Transport connection open
What else should I do from the client side to make it work?

The exception looks like a server-side problem to me.

Just to clarify an additional point, if I am a client and I have set SSL_CTX_set_verify to SSL_VERIFY_NONE, do I still have to load my certificate and private key to openssl? I am thinking the server is set for SSL_VERIFY_PEER and causing the above issue.

This looks like the server is misconfigured or something. I do not think you can do anything on client to fix that.

Thanks @t8m.
It works on another server where peer authentication is set to false.
So I agree with your view that it has to be something related to server configuration.


Reduce docker memory limit to 380MB

This PR was successfully deployed to production by CircleCI build 160 at 2017-02-06 17:49 GMT.


Update the names of the new committee members for the year 2023

Is your feature request related to a problem? Please describe.
Currently, in the ACF page there is old committee (for the year 2022) members' names
Describe the solution you'd like
Update the names of the new committee members for the year 2023
Are you working on it?
Yes

@sathsarabandaraj @Hasthikagirihagama can you assign me for this issue?


Change time zone warnings showing

Show to newly added users TZ warnings only after ~5 messages from user.

Done here.
Bot warns after 8-th message.


Unable to extract text in table when table is in the table

from img2table.document import Image
from img2table.ocr import PaddleOCR
ocr = PaddleOCR()
img = Image(f"img.png")
img.to_xlsx(f"Table.xlsx", ocr = ocr, implicit_rows = False, borderless_tables = False, min_confidence = 1)
extracted_table = img.extract_tables(ocr = ocr, implicit_rows = False, borderless_tables = False, min_confidence = 1)

when I executed the code above, I was unable to detect the text in the 4th column, 2nd row of my table, given my column cell is with a table inside.
It return the table as 2nd table only, and not extracting the text. Do you have any suggestion to extract the text I am looking for in the table?

resized_image = pil_image.resize((int(pil_image.width ), int(pil_image.height )), resample = PILImage.LANCZOS)
Furthermore when I do not change the size of image, img2table can perform extraction.
resized_image = pil_image.resize((int(pil_image.width * 1.1), int(pil_image.height * 1.1)), resample = PILImage.LANCZOS)
This code, produce large image but OCR failed to extracted the table.

Hello,
TBH, I am not really sure what is your issue with this.
This is the kind of table that I would get with your input :
The "included" table is detected as rows of the general table, but I would say that it is logical given the input image.

To be able to help you more precisely, I'll need the actual file to be able to examine it

Hello,
Thank you for the reply, for this given pdf,
SampleTable.pdf
I ran code below and it give me the Table as per the excel table 1 and image below,
the observation column is unable to be extracted.
TableExample2.xlsx

`
ocr = PaddleOCR(lang="en",
kw={
'ocr_version': 'PP-OCRv3',
'structure_version': 'PP-StructureV2',
'det_model_dir': './Python_V10/PaddleOCR/ch_PP-OCRv3_det_infer',
'rec_model_dir': './Python_V10/PaddleOCR/en_PP-OCRv3_rec_infer',
'cls_model_dir': './Python_V10/PaddleOCR/ch_ppocr_mobile_v2.0_cls_infer',
'table_model_dir': './Python_V10/PaddleOCR/en_ppstructure_mobile_v2.0_SLANet_infer',
'layout_model_dir': './Python_V10/PaddleOCR/picodet_lcnet_x1_0_fgd_layout_infer',
'lang': 'en',
})
doc = fitz.open("SampleTable.pdf")
page = doc[0]
image = page.get_pixmap(matrix=fitz.Matrix(100, 100), dpi = 1800)
pil_image = PIL.Image.frombytes("RGB", (int(image.width), int(image.height)), image.samples)
resized_image = pil_image.resize((int(pil_image.width * 1.05), int(pil_image.height * 1.05)), resample = PIL.Image.LANCZOS)
success, encoded_image = cv2.imencode('.png', np.array(resized_image))
image_bytes = encoded_image.tobytes()
img = Img2Table.Image(image_bytes)
img.to_xlsx(f"TableExample2.xlsx", ocr = ocr, implicit_rows = False, borderless_tables = False, min_confidence = 0)
extracted_table = img.extract_tables(ocr = ocr, implicit_rows = False, borderless_tables = False, min_confidence = 0)
display_html(extracted_table[0].html_repr(title = 'Result'), raw = True)
`

I've checked this case and I will make some modifications so that the larger table is properly detected

Thanks for the reply.
I do enlarge in this case because I found some very small image, it cannot detect all the words in the table. However when I enlarge (resize) the image, the table extraction is failed too, I guess I over enlarged the image.
Thank you for the help!

Hi,
I made some modifications/adjustments to the code in the latest release, your case should now be working as expected.
At least, the example you gave me is now properly processed.


Change RewarderSession.close's reason arg to unicode.

In autobahn, WebsocketProtocol.sendClose checks:
if type(reason) != six.text_type:
                raise Exception("reason must be of type unicode (was '{}')".format(type(reason)))

But RewarderSession.close sends this string:
'closed by RewarderSession.close'

In Python 3.5.2, that check will pass:
import six
type('closed by RewarderSession.close') == six.text_type # True
type(u'closed by RewarderSession.close') == six.text_type # True

In Python 2.7.12, that check will not pass:
import six
type('closed by RewarderSession.close') == six.text_type # False
type(u'closed by RewarderSession.close') == six.text_type # True

Thanks, much appreciated!


Work title doesn't appear on Work Settings Page

For a work without a published version, the title doesn't appear on the Work Settings page as expected.
# in the work settings view:
(@work.latest_published_version || @work.latest_version).title

When there is no published version, this resolves to the nil title of a NullWorkVersion, not the actual title of @work.latest_version
For example: https://scholarsphere.psu.edu/dashboard/works/9778/edit

@srerickson what should the title be? "Unpublished work" or something similar?

@srerickson what should the title be? "Unpublished work" or something similar?

I think it should just be @work.latest_version.title in all cases --- i.e., use the title even if the version isn't published. I think that's safe.

I think it should just be @work.latest_version.title in all cases --- i.e., use the title even if the version isn't published. I think that's safe.

Oh right, because you can't even save a draft version without a title 👍

Oh right, because you can't even save a draft version without a title 👍


Autorest Regen Preview to 3.0.0-alpha.20230628.4 by Dapeng Zhang from refs/pull/3470/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3470

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.IoT.DeviceUpdate


[GE] Parenting System

Controls can now be parented to containers using a drop down selection. (UI temporary).
Each GUINode contains information of the nodes children in order to iterate through deleting/updating. The child node also contains information of its parent.
https://github.com/BabylonJS/Babylon.js/issues/9921

@RaananW so I thought about removing the parent. But then there is no way to extract the GUINode information (needed for the editor) from the gui control. Unless we want to remove the GUINode wrapper class entirely. I'm not against that but need to look more into it.

@msDestiny14 - managing two states might be a challenging task once you get to more complicated tasks.
If the scene has all of the information you need, it should be your source of information. Of course, it is possible to add some form of a proxy class that augments the scene's object, if this is needed, and then you have a 1-to-1 connection between a GUI element and the object. But I don't recommend managing state based on that (like checking parenting using this object instead of the scene), as this could lead to complications in a later stage.
If there is no way around it, then it is needed, of course! I just want to make sure you don't have a clean way to go around the issue

I started removing the parent and children members which was fine initially because you can get that information from the control itself. But how does it then know which GUINode it belongs too. If it is needed, you should keep it, of course. But it should only contain the augmented information, not the same information as the scene. the GUINode should have a reference to the scene-object, which has the parenting information. Will that be possible in this usecase?

BTW - you have a small conflict that prevents us from merging this :-)

I'm gonna think about this a little more over the weekend and let you know Monday :)


Added support for Faraday::Request

I previously made a pull request to directly support Typhoeus (#63) however we decided to use Typhoeus through it's Faraday adapter instead, thus I am back with another pull request. Faraday is actually probably a great addition for api-auth because it wraps many other HTTP clients libraries and probably more in the future. The standalone Typhoeus support still has it's merits as some people may want to use it directly instead of through Faraday.
https://github.com/lostisland/faraday

Faraday is an HTTP client lib that provides a common interface over many adapters (such as Net::HTTP) and embraces the concept of Rack middleware when processing the request/response cycle.

Looks good. Thanks so much for all the work!


feat: Define Nat.finMulAntidiagonal, the Finset of tuples of naturals with a fixed product.

The results card_pair_lcm_eq and card_finMulAntidiagonal_of_squarefree are central to controlling the error term in my formalisation of the Selberg sieve. (specifically for lemma 3.1 in Heath-Brown's notes)
The definition is adapted from Finset.finAntidiagonal, though it is held back a bit by Nat.divisorsAntidiagonal which has a time complexity of $O(n^2)$.
This definition is limited to Nat because it's not clear how to handle 0 in higher generality. (zulip)

I would prefer if we could avoid duplicating the finAntidiagonal work here; one way would be with:
import Mathlib

def PNat.divisorsAntidiagonal (n : ℕ+) : Finset (ℕ+ × ℕ+) :=
  (Nat.divisorsAntidiagonal n).attach.map
    ⟨fun x =>
      (⟨x.val.1, sorry⟩, ⟨x.val.2, sorry⟩),
    fun x₁ x₂ h => Subtype.ext <| Prod.ext (congr_arg (·.1.val) h) (congr_arg (·.2.val) h)⟩

instance : Finset.HasAntidiagonal (Additive ℕ+) where
  antidiagonal n := PNat.divisorsAntidiagonal (Additive.toMul n)
  mem_antidiagonal := sorry

#eval Finset.finAntidiagonal 3 (Additive.ofMul (6 : ℕ+))

Whatever we do, I think Finset.HasAntidiagonal (Additive ℕ+) should be an instance.

I like the idea of using a Finset.HasAntidiagonal (Additive ℕ+) instance to prevent code duplication. I'm having a bit of trouble proving mem_finMulAntidiagonal though. We're probably missing some API here. I've also found a bug in arrowCongrRight_apply which I'll open a PR for in a sec.
I also don't care very strongly what the API ends up looking like, so long as we get the cardinalities of ((n.divisors ×ˢ n.divisors).filter fun ⟨x, y⟩ => x.lcm y = n) and (Fintype.piFinset fun _ : Fin d => n.divisors).filter fun f => ∏ i, f i = m (which makes me prefer working with Nat directly when possible).


BTCP wallet won't remember password on MAC ?

The BTCP wallet won't remember password on MAC ?
I have deleted and reinstalled all wallets starting with the BTCP electrum wallet , then the BTCP full node desktop and  then the BTCP testNET  wallet which seemed to be the only 1 that would let me make a password.
ALL of them did not work, then installed the correct JAVA and UPDATED my MAC OS to Sierra version.
Now the wallet loads but it doesn't remember my password to log in ? ? please help someone.. this is really frustrating and annoying and a huge disappointment in making this so called FORK an easy process has been a complete nightmare..

You definitely don't want to be on the testnet wallet; just use the latest.
Your 'password' is an optional, additional security layer that you can set at the end. It does not 'automatically save' in the text box. Fundamentally, your Seed Words / unencrypted wallet.dat file are what you need to keep safe. The password is so random people using your computer can't compromise your wallet.

So how do i log into to get my BTCP because it asks for password every time ??
Sent from my iPhone

On 13 Mar 2018, at 7:22 am, Jon Layton<EMAIL_ADDRESS>wrote:
Closed #47.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

how can i log in to get my BTCP ? because it asks for password every time i
open the wallet ?
On Tue, Mar 13, 2018 at 7:22 AM, Jon Layton<EMAIL_ADDRESS>wrote:

You definitely don't want to be on the testnet wallet; just use the latest.
Your 'password' is an optional, additional security layer that you can set
at the end. It does not 'automatically save' in the text box.
Fundamentally, your Seed Words / unencrypted wallet.dat file are what you
need to keep safe. The password is so random people using your computer
can't compromise your wallet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BTCPrivate/electrum-btcp/issues/47#issuecomment-372449568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjgpLvHf5Vtq1rLaksurF1LT6fgxxJpsks5tdtjqgaJpZM4SmX_n
.

hello ?
the BTCP doesn't remember my password ? how can i log in ?
On Wed, Mar 14, 2018 at 7:33 PM, Byron Katz<EMAIL_ADDRESS>wrote:

how can i log in to get my BTCP ? because it asks for password every time
i open the wallet ?
On Tue, Mar 13, 2018 at 7:22 AM, Jon Layton<EMAIL_ADDRESS>wrote:

You definitely don't want to be on the testnet wallet; just use the
latest.
Your 'password' is an optional, additional security layer that you can
set at the end. It does not 'automatically save' in the text box.
Fundamentally, your Seed Words / unencrypted wallet.dat file are what you
need to keep safe. The password is so random people using your computer
can't compromise your wallet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BTCPrivate/electrum-btcp/issues/47#issuecomment-372449568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjgpLvHf5Vtq1rLaksurF1LT6fgxxJpsks5tdtjqgaJpZM4SmX_n
.

The BTCP wallet doesn't remember my password to log in ?
still need to claim my BTCP... how can i log in or claim my BTCP really annoying...


fixed the issue [#1175]-adding files from irods not working

Moving {% endblock %} for {% block main %} to the end of genericresource.html fixed this issue. The issue was caused by irods.js which is included in extra_js block not being triggered if the main block does not include the extra_js block. Anyway, after having main block include extra_js block, adding files from irods works as expected. @mauriel133 Can you take a quick look and give +1 so that it can be included in the next release?

Can one of the admins verify this patch?

@mauriel133 Please review this small PR.

@mauriel133 it'd be great if you can give a +1 at your earliest convenience so that it can be brought in for release 1.6.10. Thanks.

Ok, I will review it today.

As I'm testing this change, I noticed that {% block extra_js %} now renders twice in the page. I don't think we want to pass this PR like that since it can create problems in the future.
For example, the google maps API throws this error:

You have included the Google Maps API multiple times on this page. This may cause unexpected errors.

Moving the block outside of the main block was the fix for these scripts running twice in the past. Here is the PR: https://github.com/hydroshare/hydroshare/pull/905

@mauriel133 OK. Thanks for the reminder. I need to figure out how to get irods.js included as well since it is not included without moving extra_js block inside the main block. Let me know if you have some quick ideas, though.

While debugging I can see the script is being included. Perhaps it is running before the element is rendered. Is this happening only for the resource landing page?

@mauriel133 Thanks for debugging. Yes, it happens only for the resource landing page. I think you are right about it is running before the element is rendered. I am moving irods.js include out of genericresource.html to irods model template, and it seems to be working as expected. Need to do a bit of more testing to make sure it indeed works, then I will update this PR for your review.

@mauriel133 Can you take a quick look and see if you can give +1 now?

@mauriel133 My current fix will cause some irods javascript code to run twice, so I am investigating trying to find a better way to get it working. Will let you know when it is ready to review.

Alright.

@mauriel133 It is ready for review now. I have broken the old irods.js into two files, one is for irods signin and signout (irods_signin.js), the other is for irods file operation (irods.js). irods_signin.js is only included in irods_signin.html template file, the irods.js is only included in irods_upload_create.html (for creating resources), and irods_upload_add.html (for adding files to existing resources). This made sure all irods java script code only included once and thus only run once. I have debugged to make sure this is the case. It'd be great if you can take a quick look and give a +1 if you don't see any red flags. Thanks.

+1


Build Swift project with SignalR

I was using version 2.0.0.beta3 and today I realized version 2.0.1 is up and updated my Podfile.
Right now, I am able to import SignalR-ObjC, but can not build the project with it.
Is there anyone, who integrated version 2.0.1 succesfully to a Swift project?

Yes I was able to successfully integrate. Can you give some more information about the errors you are seeing?

I was using SignalR-ObjC version 2.0.0.beta3 with Cocoapods version 1.0.0.beta.3
Everything was working fine until I decided to upgrade SignalR-ObjC to version 2.0.1
I executed pod update, everything gets successfully updated but when I hit Build on my Xcode project, I am getting "Could not build Objective-C module 'SignalR-ObjC'" error on line import SignalR-ObjC

I'm not using the 1.0.0.beta.3 version of CocoaPods, I'm still on 0.39.0, but I had a problem where a couple of classes in the SignalR-ObjC Library weren't importing the AFNetworking framework correctly when using the use_frameworks! flag. I fixed it by changing the import statement in these two classes
'
SREventSourceRequestSerializer.h
SREventSourceResponseSerializer.h
'
to:
@import AFNetworking;
Hope that helps.

Downgrading to Cocoapods version 0.39.0 and making those import changes allowed me to build the project. Thx mate

There is also a swift port : https://github.com/skerkewitz/SwignalR


CI jobs do not output failure logs

For lot of jobs in CI in multiple pipelines running command when fails does not output any error. When we can't repro locally, it's very hard to figure out what the problem is (if it's timeout also, we are not sure where the script was stuck). On the other hand, if it fails on some machines in CI but passes on others we are not sure whether it fails for the same reason or there are multiple ones.

Related docs I was reading, putting on hold for now, hopefully can get back to it next week:
https://pytest-xdist.readthedocs.io/en/latest/known-limitations.html#output-stdout-and-stderr-from-workers
https://multiprocess.readthedocs.io/en/latest/
Not looking good, though. Doesn't seem to be a simple way to capture the output from the Multiprocess. We may have to make changes to the underlying libraries.


User settings account

Closes #106
This is the first iteration of the user settings account. I'm using next auth to simplify authentication and connection to the firestore database. Whenever a user initially signs in, a new account is created and registered to their ID, this allows us to access their data and update within the settings page. Added a type declaration file in order to ease the access of the data without needing to do a bunch of checks. Let me know if you encounter any errors or have any ideas on how to make this cleaner / less buggy

I added some stuff to the README. When testing you need to get service account credentials and input that into the .env file

I added some stuff to the README. When testing you need to get service account credentials and input that into the .env file

Thanks for adding that info! We should be able to update our deployment as needed.

cc @shreyon312

I may close this branch in favor of the TRPC branch just because It's already implemented better In that branch

We can also redesign with the new implementation we are using


Revert get_balance() flow to use get_unspent()

get_balance() used to compute the balance from the UTXO set.
I changed this in <IP_ADDRESS> to get the balance directly from rest.bitcoin.com
but this change might break things.
This commit reverts to the previous method.

Thank you!


Remove URL link to the private repo

This is a follow-up for PR #6.
https://github.com/ChartBoost/chartboost-mediation-ios-sdk is a private repo and we should remove the URL link to it.

Actually https://github.com/ChartBoost/chartboost-mediation-ios-sdk is a private repo. Would it be better to just say it's copied from that SDK?

Oh yeah you are right, I kept thinking it's the public open source repo. I have updated the the PR, PR title, and description.


Close on error messages for service errors

We have an outstanding thread with the service team on this.  It'd be great if we can close this down and implement the changes for Beta 3.

Close it since we decided not to pursue error messages.


An accessor cannot be declared in an ambient context.

I get an ERROR in ../node_modules/@ng-idle/core/lib/eventtargetinterruptsource.d.ts:29:9 - error TS1086:
An accessor cannot be declared in an ambient context.
at compile time.
Can avoid it by setting the     "skipLibCheck": true, in tsconfig , but not a desired setting.
"peerDependencies": {
"@angular/animations": "^8",
"@angular/cdk": "^8",
"@angular/common": "^8",
"@angular/core": "^8",
"@angular/forms": "^8",
"@ngx-translate/core": "^11",
"rxjs": "^6"
},
"dependencies": {
"@ng-idle/core": "10.0.0-beta.1",
"@ng-stack/forms": "^1.3.4",
"js-uuid": "^0.0.6",
"moment": "^2.24.0",
"ng-click-outside": "^3.3.0",
"tslib": "^2.0.3"
},
"devDependencies": {
"@babel/generator": "^7.11.6"
}
Anyone else had this one?

I am getting the same error running a angular 8 application. My peer is running a angular 9 and is not getting the same error. So, I don't know if it's a version issue.

I am getting the same error running a angular 8 application. My peer is running a angular 9 and is not getting the same error. So, I don't know if it's a version issue.

Same here... Angular 8 and got this error.
The issue is actually the version dependency not working properly. On an angular 8 project the npm install should install "v8.0.0-beta.4" but it installed version 10. Referred the changelog for deducing it.

It's not perfect way to fix it, but you can set "compilerOptions": {     "skipLibCheck": true } in your tsconfig.json file. But as I said, it's not perfect and can cause some typing errors.


using flexselect with jquery 2.2

Hi,
I am using flexselect jquery with jquery version 2.2.
In this key up and key down is not working properly
Advance Thanks

I just tried the demo page with jQuery 2.2.2. Up and down keys worked as expected.

@rmm5t Not sure if this is related as I just checked as well. Looks like it doesn't work with a disabled entry. The second row on the demo has one if you clear the input.

@virgofx Oh. I see. The keyboard navigation doesn't jump past disabled entries. Reopening this issue.

One more issue with it, it does not select the element on 0 index

any update?

@aroradinesh03 Personally, I've been too busy with client work to focus on this any time soon, but a pull request to fix this is very much welcome.


Project overhaul

As discussed in #135, I am forking this project with the intention of performing a major overhaul.
If anyone is willing to help out, give me a shout.
I don't have a whole load of time available so I can't make any promises on a clear timeline.

I'm down to help.

Please see my response in https://github.com/futurice/chilipie-kiosk/issues/135#issuecomment-784988512 for the current status, and how you can help!


Nominator: Nomination timer changes

see commit-by-commit

Didnt fix our particular problem, but could be a good addition nonetheless.

Can you explain commit 2 (the revert) ?
Also tiny type in the field name in commit 3 msg.

We should keep nominating until atleast a single value makes it to balloting phase. If we stop nominating halfway and no Value has made it to the balloting phase, we are doomed. We should find another way to limit the number of nominations.

Trying to get a green run, rebased on v0.x.x


css.types.length-percentage.ic doesn't exist

https://github.com/web-platform-dx/web-features/blob/971fee52bddbf710e0f8def3eaf7b618c1f664fe/feature-group-definitions/ic.yml#L6C5-L6C35
see : https://github.com/mdn/browser-compat-data/blob/main/css/types/length-percentage.json

Oops, I guess BCD changed. Fixed in https://github.com/mdn/browser-compat-data/pull/22697 + https://github.com/web-platform-dx/web-features/pull/731. Thanks for reporting!


pdd and xcop rules are absent

We need to add pdd.yml and xcop.yml to be sure a high quality checks are used in this repository

@Graur, done

@MikhailLipanin I don't see any pull requests with this updates. Did you force push them to master branch?

@Graur Yes, I forgot to do this via PR :( (f916ecddb36358be7810997e92efdc8c8d282ac4, 47eb5de181a6936e4f243468a87020507fd0031b)

@MikhailLipanin Please, use PR's in the next time.
Thanks!


Break up the largest repos into batches

When we encounter a repo with, for example, 10,000 images in it, we struggle to work through the standard compression routine and in some cases, we timeout eventually.
I am proposing we split the workflow up based on image count. This is what I'm thinking right now:

query images as usual, but add a sort to the found images
if there are more than 1,000 images found, only consider the first 1,000
if we complete the compression routine and there are none to optimize, queue a new message with a new property page=2
if we have a page property when the message is incoming, skip to the 1,001-,2000 using the same query.
if we complete the current page without finding any images, queue the next page in a new message

All these 'pages' of images would use the same imgbot branch and we would not do the next page until the current page was integrated and the branch deleted.
These pages would stay segmented indefinitely, even as images on multiple pages changed, we would open separate PRs to optimize them.

I don't have a better idea but everything that stays "indefinitely" raises concerns of pending unmanageability.

hey @guntbert !
definitely a good flag to raise.
Ideally we would collapse the image changes across multiple groups into a single PR, but not sure the best way to pull that off today.

I think I have a solution to this figured out.
Since we have access to the set of images that are added/modified when we trigger for ongoing optimization, we can pull this set of images onto the first page so that they don't get broken up across multiple prs.

this got a lot more complicated than I expected, but I did stumble on a new idea - adding multiple commits into a single PR

so after running around in circles on this, I believe the long processing of the large repos was never the issue. I believe that the optimization of an individual image created an infinite loop that never returned and we got stuck in the optimize loop.
I believe this because, when monitoring an optimization run that was previously never completed, I noticed we ran through all the images much faster than I expected, but still never completed. The CPU stayed spiked like it was doing work, but never finishing.
I opened #390 to add a 10 second timeout to each image process. Pending a full test to validate.


Feature/35869/st georges importer

What?
A new St Georges importer has been written to account for a new data format (see planio ticket 35869). The old St Georges importer is still needed, so it has been kept in the repository.
Why?
St Georges changed the format of the data being sent. Therefore, a new importer had to be created to be able to parse this format.
How?
The importer was written based on the rules sent by Fiona.
Testing?
Unittests have been written. A QA of the variant counts has also been completed and signed off by Fiona.

@lauramccluskey1 Handler tests are failing , can you check those please?

Thanks Shilpi! I have fixed the ones related to my tests, but its now giving integration errors. @bshand are you able to take a look at this?


use an OffsetDateTime on the ClientStats

Pull Request
What does this PR do?
I noticed we were still deserializing the last_update in the ClientStats structure in an Option<String>.
I typed the String into an OffsetDateTime.
PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

I also found a "bug" in one of our doctest where we were creating an index with a bad name.

bors merge


QA for https://github.com/microsoft/workshop-library/tree/main/full/prepare-data-python

[ ] All elements of the workshop are in place (README, workshop designer, solution if necessary, any images if necessary, slides, a call-to-action with a Learn module linked (usually in "Next Steps" area), a knowledge check (either in slides or directly linked to the workshop README) and an area to extend knowledge.
[ ] The workshop can be done in its listed time frame
[ ] Instructions are clear and well written
[ ] The workshop is bug-free
[ ] The workshop 'flows' well

I fixed and consolidated the link, great catch! And this learn mod doesn't have a quiz, too bad! thanks again


Support getting information from MangaDex

It would be so awesome, I wish you would add it when you have free time. Thanks. I have some recommendation for this program but sadly I have no knowledges about coding so cant help with it.

I added an extractor for MangaDex, which is included in the new release.
Since we don't have a chapter system the solution is a little hacky and some fields that don't apply are hidden. But other than not being able to add chapters and a disabled 'Read Online' button you shouldn't notice anything of that while you're using it.

I added an extractor for MangaDex, which is included in the new release.
Since we don't have a chapter system the solution is a little hacky and some fields that don't apply are hidden. But other than not being able to add chapters and a disabled 'Read Online' button you shouldn't notice anything of that while you're using it.

Thank you so much!

Thank you so much!


mise uses free-threaded Python 3.13 (3.13.0t) when specifying 3.13

Describe the bug
Python 3.13 is released now and it allows an optional experimental support for free-threading (no GIL).
These builds usually have a t suffix and python-build supports those.
Now the version selection logic of mise prefers 3.13.0t over 3.13.0 when specifying mise use<EMAIL_ADDRESS>(It works fine with mise use<EMAIL_ADDRESS>I assume this to be an annoying case because for some tools an appended letter means some kind of minor/patch version (e.g. OpenSSL 1).
To Reproduce
$ mise use -g<EMAIL_ADDRESS>[... python-build output]
mise<EMAIL_ADDRESS>✓ installed
mise ~/.config/mise/config.toml tools<EMAIL_ADDRESS>
$ python3.13 -VV
Python 3.13.0 experimental free-threading build (main, Oct  8 2024, 12:50:41) [GCC 13.2.0]

mise doctor output
version: 2024.10.1 linux-x64 (2024-10-07)
activated: yes
shims_on_path: yes

build_info:
  Target: x86_64-unknown-linux-gnu
  Features: DEFAULT, NATIVE_TLS
  Built: Mon, 7 Oct 2024 19:01:35 +0000
  Rust Version: rustc 1.81.0 (eeb90cda1 2024-09-04) (Homebrew)
  Profile: release

shell:
  /bin/bash
  GNU bash, version 5.2.21(1)-release (x86_64-pc-linux-gnu)
  Copyright (C) 2022 Free Software Foundation, Inc.
  License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

  This is free software; you are free to change and redistribute it.
  There is NO WARRANTY, to the extent permitted by law.

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/.cache/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/.config/mise/config.toml

backends:
  cargo
  core
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:
  terraform  https://github.com/asdf-community/asdf-hashicorp.git#f1602b6
  usage      https://github.com/jdx/mise-usage.git#fe3888a

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
env_vars:
  MISE_SHELL=bash

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf = true
  asdf_compat = false
  cache_prune_age = "30d"
  cargo_binstall = true
  ci = false
  color = true
  debug = false
  disable_default_shorthands = false
  disable_hints = []
  disable_tools = []
  experimental = true
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  http_timeout = 30
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  libgit2 = true
  log_level = "info"
  not_found_auto_install = true
  paranoid = false
  pin = false
  pipx_uvx = true
  plugin_autoupdate_last_check_duration = "7d"
  python_default_packages_file = "~/.default-python-packages"
  python_pyenv_repo = "https://github.com/pyenv/pyenv.git"
  python_venv_auto_create = false
  quiet = false
  raw = false
  trace = false
  trusted_config_paths = []
  use_versions_host = true
  verbose = false
  vfox = false
  yes = false

  [node]

  [ruby]
  default_packages_file = "~/.default-gems"
  ruby_build_repo = "https://github.com/rbenv/ruby-build.git"
  ruby_install = false
  ruby_install_repo = "https://github.com/postmodern/ruby-install.git"

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false
No warnings found
No problems found

related: java latest resolves to the loom variant

@finalchild In case of Java it is because there is no other (OpenJDK) 24.x version and Loom versions were not properly treated as early access by https://github.com/joschi/java-metadata for unknown reason. Also Java vendors adhere to proper semantic versioning which makes it easier to compare, sort etc. while 3.13.0t certainly is not.

to be clear, this is limited to python-build, correct? I think what we should probably do is have both a setting and tool option for configuring this but make the default non-free threaded

or maybe as a simpler option, it appears that python-build configures this with PYTHON_BUILD_FREE_THREADING=1
so what we could instead do is just never use the "t" variants but people could install it by setting that env var and we could just document that.

This specific behavior (t suffix) only affects python-build.
The precompiled versions use a freethreaded build tag to differentiate.
Setting the env var works for python-build, but not for precompiled.
Maybe I'm off here, but ideally I'd see the freethreaded version as a separate variant like cpython-freethreaded-3.13 or just freethreaded-3.13.
While it might be interesting to support explicitly choosing this, I think this is out of scope for this bug.
For now the focus is probably really just ignoring the t versions.
Probably removing these from the candidate list is easier than having the solver not pick the t version by default?

For now the focus is probably really just ignoring the t versions.

that's what my "simpler" solution is


[Docs] Add docs for navigation, spinner and textarea

Navigation -> secondary -> description: display items as full-width rows
Describe popoverMenu prop.
Describe Spinner -> ninja prop

Navigation -> secondary -> description: display items as full-width rows
Describe popoverMenu prop.
Describe Spinner -> ninja prop

fixed


[sival] board LED demo

This adds a basic SiVal board LED demo.
Additionally, this fixes the prodc kms signing config.

These will be cherry picked to the sival branch as well.


SCRIPT 4a: Ability to DELETE a CLIP field #372

In this request, I've made the changes in 2 files

scripts.vue in which I added delete function defination and
clipfield.vue in which I've added delete button and pass data through props and emit delete function also.

@ArindamReact please fix test failures

@ArindamReact this deletes the wrong clip field. Rather than deleting the field you are selecting, it always deleted the bottom one. It needs to delete the field you are clicking on.

Any chance of more meaningful commit messages than 'stable'? Especially when it is in fact, very much not.
You do have the ability to run all the lint and cypress tests locally. I'm not sure why there's so many CI fails

Ok @chrisns I will make sure about meanigfull commit commet from next time

Please don't make line changes just to run the ci
You can rerun ci jobs
You can make empty commits
You can run all the tests locally
If the builds are non-deterministic I.e. rerunning them yields different results, that's a problem that needs to be fixed immediately

@ArindamReact please make Key0, Key1 etc invisible


Support compression

https://github.com/codemonger-io/flechasdb/issues/3

Maybe I should calculate SHA-256 hashes not for data before compression but for compressed data. If we calculate the SHA-256 hash for data before compression, the checksum feature of Amazon S3 won't work.

As compression is completely handled by flechasdb, flechasdb-s3 has to do nothing.


networking.dnsExtensionMechanism = true as default breaks my DNS queries in 17.03

Issue description
This commit changed dnsExtensionMechanism to default=true
https://github.com/NixOS/nixpkgs/commit/11696e290dc8f14027cbd6000bed9a519100cafd
Don't know about the general public, but at least my router (Linksys WRT1900) ain't ready for that stuff yet. Thus about 90% (no clue why not 100%, did not debug more closely) of all my DNS queries start to fail when I upgraded to 17.03. No problems after setting dnsExtensionMechanism=false and/or manually removing "options edns0" from resolv.conf.
Perhaps =true should not be the default. I haven't seen any other distro use it yet. Or if it is the default, it should be added into the default configuration.nix with an explanation because of the risk of breakage.
 Upgrade to 17.03.
Technical details

System: (NixOS: 17.03)

Hmm, it's a bit off-topic, but I would be interested in how exactly the DNS breakage happens.  I wasn't really disappointed by a 15 € router breaking some DNS queries, but I see yours is much more expensive.  Would you try a few (k)dig commands for me, e.g. dig @k.root-servers.net. . NS? (If so, we should continue that over e-mail/XMPP/...)

@vcunat Apologies. I got fed up with the situation and upgraded my router to LEDE. This setting works now without any problems.

OK.  They might, but so far I've only seen one person who needed = true and then this issue (needing = false), so I'm inclined not to change the default = true (yet).

@vcunat Huh. Take that back. I'm still seeing this issue even with an updated router. It's running dnsmasq (as is standard with LEDE). Perhaps that has some trouble.

I confirm this with an arcadyan router, which is a default router of my ISP (Orange). With wireshark I noticed that sometimes an A entry goes to ANSWERS section and an EDNS pseudo entry goes to ADDITIONAL section, and sometimes in reverse. In the latter case I get "Name or service not known". I glanced over RFC, and  it seems, that the latter behavior is incorrect.
Disabling dnsExtensionMechanism fixes the problem.

Huh, right, the EDNS entry may only be present in the additional section.  Does the problematic router serve as DNS forwarder is it just mangling packets passing through?
Side notes – if you wonder why I'm so reluctant to switch the default back to false... Open-source DNS vendors will likely be moving against servers without EDNS(0) support, i.e. dropping compatibility in about a year from now, though AFAIK that might not affect the last "DNS hop" from the final client to the first DNS resolver/forwarder.  I must say I'm personally getting annoyed when designing workarounds for broken devices/SW, because that decreases the pressure on correct implementation (things just work again).

I guess the culprit could be one of the other components in the network, too? The DNS query gets forwarded from a router to the modem, and then to the ISP, and who knows what they do to it before handing it over to the authorities automatically? Every one of those nodes need to support it well, right?
The default is fine; I have no trouble with breaking things when they deserve to. But perhaps include a single-line comment in the generated configuration.nix about it, since this is such a potential breaker and people will be confused about it.

Ideally this should only depend on the DNS resolvers that you use (in your router, from your ISP, ...), but some on-path devices might also try to interpret packets and do something.
Adding a line in there seems a good idea.

I spent ~45 minutes wondering why DNS wasn't working when I connected my laptop to another wifi router - my phone worked fine but not NixOS.  Disabling networking.dnsExtensionMechanism solved the problem (thanks to this thread).
I think this option should be disabled by default.

Still happens to me as of 2024.


run train2 stop at Model loaded

hello, i use  trained weights from #28 ,@VictoriaBentell.
and run python eval1.py timit is ok.

but in next step,python train2.py timit arctic/slt,I find the training stop at (Model loaded. mode: train2, model_name: epoch_12_step_6924)

who can tall me why? thanks.
environment:Ubuntu16.04    tf-gpu:1.4.0

Check paths, there is some inconsistency in using *.WAV & *.wav in settings and hparams.py. Also might be worth disabling 'queue' mode in code, there are some bugs there.


News Tab UI Enhanced

Description
The UI of the whole page is enhanced and some features are added.
Related Issues
The UI is made more user appealing and coherent.

Closes #424

Type of PR

[x] Bug fix
[x] Feature enhancement

Screenshots / videos (if applicable)

TO

Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

@Adv1919 Resolve conflicts

Closing! no activity


fix(bundled/tool): fix "UnboundLocalError: local variable 'result' referenced before assignment"

Fix the error that "UnboundLocalError: local variable 'result' referenced before assignment"

@EeyoreLee  Which time would you like to publish a new version?

@northwang-lucky - hi, I'm moving some unnecessary logs to debug-level. I'll release a new version today or tomorrow. Thanks for your pr and email.

@northwang-lucky - the new version v0.1.7 has been released. Can you try whether it reslove your error.


hard-coded username in save_history.sh

In the branch commnet-23 the script platform/utils/gif/history_config.sh uses a hard coded username to perform the git commit & push. The variable for the username ($USERNAME) is available, but not used here.

Thanks for the feedback, I've added the update to the comment branch and it will be merged soon, I hope.