A note on polymorphism and discriminators.

This is not really an issue. I'm writing it down for ones who come after us. I've spent unreasonable amount of time on this but I believe it is worth it. Rust is great for backends, Axum is likely the best option for the http stack and aide is the only option to generate OpenAPI automatically I managed to get usable results with. This was the only long standing issue for me as I rely heavily on the code generation from the OpenAPI spec. I'd suggest that the final solution should be at least mentioned in the official documentation as others will likely experience the same issue.
I have a simulated polymorphism where the classes TableItem, ChartItem, ... can be stored inside a container called AnyItem. My implementation is similar to this
#[derive(Deserialize, Serialize, JsonSchema)]
pub enum AnyItem {
    Table(TableItem),
    Chart(ChartItem)
}

#[derive(Deserialize, Serialize, JsonSchema)]
pub struct Item {
    pub title: Option<String>,
}

#[derive(Deserialize, Serialize, JsonSchema)]
pub struct TableItem {
    #[serde(flatten)]
    pub item: Item,
    pub colnames: todo!(),
    pub rows: todo!(),
}

#[derive(Deserialize, Serialize, JsonSchema)]
pub struct ChartItem {
    #[serde(flatten)]
    pub item: Item,
    pub image_path: todo!(),
}

The OpenAPI JSON file is consumed by datamodel-code-generator to generate Python's Pydantic classes. And it is also consumed by openapi-generator to generate Typescript models using the typescript-fetch generator. Both of those had a problem with AnyItem.
I'm doing serialization in Python and deserialization in Typescript so this is the only case I'll cover here.
Existing behavior
Case 1: Basic enum
Rust definition
#[derive(Deserialize, Serialize, Debug, JsonSchema)]
#[serde(rename_all = "lowercase")]
pub enum AnyItem {
    Table(TableItem),
    Chart(ChartItem),
}

OpenAPI
      "AnyItem": {
        "oneOf": [
          {
            "type": "object",
            "required": [
              "table"
            ],
            "properties": {
              "table": {
                "$ref": "#/components/schemas/TableItem"
              }
            },
            "additionalProperties": false
          },
          {
            "type": "object",
            "required": [
              "chart"
            ],
            "properties": {
              "chart": {
                "$ref": "#/components/schemas/ChartItem"
              }
            },
            "additionalProperties": false
          },

Problem is that both generators generate structs AnyItem1, AnyItem2 for each case of the oneOf. This is major PITA when trying to create the data structures as one has to guess which number corresponds to the type. Deserialization is fine.
Case 2: Tagged enum
Rust definition
#[derive(Deserialize, Serialize, Debug, JsonSchema)]
#[serde(tag="item_type", rename_all = "lowercase")]
pub enum AnyItem {
    Table(TableItem),
    Chart(ChartItem),
}

OpenAPI
"AnyItem": {
        "oneOf": [
          {
            "type": "object",
            "required": [
              "colnames",
              "item_type",
              "rows"
            ],
            "properties": {
              "card": {
                "type": "boolean",
                "nullable": true
              },
              "colnames": {
                "type": "array",
                "items": {
                  "type": "string"
                }
              },
              "item_type": {
                "type": "string",
                "enum": [
                  "table"
                ]
              },
              "rows": {
                "type": "array",
                "items": {
                  "type": "array",
                  "items": {
                    "$ref": "#/components/schemas/TableCell"
                  }
                }
              },
              "title": {
                "type": "string",
                "nullable": true
              }
            }
          },

This one has a problem that the actual structures ChartItem, ... don't have the item_type field so it cannot use references. It correctly inlined the temporary objects here. This situation is also bad for both serialization and deserialization as those inlined objects are completely different from the real ones.
Case 3: Tag and content
Rust code
#[derive(Deserialize, Serialize, Debug, JsonSchema)]
#[serde(tag="item_type", content="c", rename_all = "lowercase")]
pub enum AnyItem {
    Table(TableItem),
    Chart(ChartItem),
}

OpenAPI
      "AnyItem": {
        "oneOf": [
          {
            "type": "object",
            "required": [
              "c",
              "item_type"
            ],
            "properties": {
              "c": {
                "$ref": "#/components/schemas/TableItem"
              },
              "item_type": {
                "type": "string",
                "enum": [
                  "table"
                ]
              }
            }
          },
          {
            "type": "object",
            "required": [
              "c",
              "item_type"
            ],
            "properties": {
              "c": {
                "$ref": "#/components/schemas/ChartItem"
              },
              "item_type": {
                "type": "string",
                "enum": [
                  "chart"
                ]
              }
            }
          },

This fixes the problem on inlined objects from case 2 but it also creates temporary objects as was the case in 1.
Case 4: Untagged enum
Rust code
#[derive(Deserialize, Serialize, Debug, JsonSchema)]
#[serde(untagged, rename_all = "lowercase")]
pub enum AnyItem {
    Table(TableItem),
    Chart(ChartItem),
}

OpenAPI
      "AnyItem": {
        "anyOf": [
          {
            "$ref": "#/components/schemas/TableItem"
          },
          {
            "$ref": "#/components/schemas/ChartItem"
          },
          {
            "$ref": "#/components/schemas/LabelItem"
          },
          {
            "$ref": "#/components/schemas/TabsItem"
          },
          {
            "$ref": "#/components/schemas/StackItem"
          }
        ]
      },

This one is so close. The only problem is that there is no field marking which objects we are actually holding and so deserialization can get a little bit tricky. My problem was that the Typescript codegen created a superset object for AnyItem containing all fields from the items and that made it impossible to distinguish each object just by looking at fields.
Final solution
The solution I arrived at was to create a oneOf with discriminator and mapping as described in this blog post. I tried that manually and both generators produced a usable code without any temporary objects. We also need to use a tagged enums. How it works is that Serde adds a field with the type information. I hide this fact from the schema of AnyItem and instead use a discriminator. Here is my code.
#[derive(Deserialize, Serialize, Debug)]
#[serde(tag = "item_type", rename_all = "lowercase")]
pub enum AnyItem {
    Table(TableItem),
    Chart(ChartItem),
}

impl JsonSchema for AnyItem {
    fn schema_name() -> String {
        "AnyItem".to_string()
    }

    fn json_schema(gen: &mut schemars::gen::SchemaGenerator) -> Schema {
        let discriminator = json!({
            "propertyName": "item_type",
            "mapping": {
                "table": "#/components/schemas/TableItem",
                "chart": "#/components/schemas/ChartItem",
            }
        });

        let subschemas = SubschemaValidation {
            one_of: Some(vec![
                gen.subschema_for::<TableItem>(),
                gen.subschema_for::<ChartItem>(),
            ]),
            ..Default::default()
        };

        let schema_object = SchemaObject {
            subschemas: Some(Box::new(subschemas)),
            extensions: BTreeMap::from_iter(vec![("discriminator".to_owned(), discriminator)]),
            ..Default::default()
        };
        Schema::Object(schema_object)
    }
}

This generates following OpenAPI specification
      "AnyItem": {
        "oneOf": [
          {
            "$ref": "#/components/schemas/TableItem"
          },
          {
            "$ref": "#/components/schemas/ChartItem"
          }
        ],
        "discriminator": {
          "mapping": {
            "chart": "#/components/schemas/ChartItem",
            "label": "#/components/schemas/LabelItem"
          },
          "propertyName": "item_type"
        }
      },

In typescript, type of the AnyItem is
export type AnyItem = { itemType: 'chart' } & ChartItem | { itemType: 'table' } & TableItem;

Good thing is that the itemType field is added on top and ChartItem and TableItem are my exact objects. Generated code also uses itemType in a switch to decide how to properly deserialize the objects.
In Python the type of AnyItem is
class AnyItem(RootModel[Union[TableItem, ChartItem]]):
    root: Annotated[Union[TableItem, ChartItem], Field(discriminator='item_type')]

Which is also good. The only slight inconvenience is that it also added the item_type field to TableItem and ChartItem so I have to provide it when creating the object.

I just tried openapicmd for your case 1 and here is what I got:
npx openapicmd typegen http://localhost:3001/api/docs/private/api.json > ./src/openapi.d.ts

...

declare namespace Components {
    namespace Schemas {
        export type AnyItem = {
            table: TableItem;
        } | {
            chart: ChartItem;
        };
        export interface ChartItem {
            image_path: string;
            title?: string | null;
        }
        export interface TableItem {
            colnames: string;
            rows: string;
            title?: string | null;
        }
    }
}
declare namespace Paths {
    namespace ApiTestFn {
        namespace Get {
            namespace Responses {
                export type $200 = Components.Schemas.AnyItem;
            }
        }
    }
}

...

So for type safety in Typescript, you have to check if you got an AnyItem that contains the field table or chart.

This worked for me out great
#[derive(Deserialize, Serialize, Debug)]
#[serde(tag = "item_type", rename_all = "lowercase")]
pub enum AnyItem {
    Table(TableItem),
    Chart(ChartItem),
}

pub fn add_discriminator_case(tag: &str, value: &str) -> schemars::schema::Schema {
    schemars::schema::Schema::Object(schemars::schema::SchemaObject {
        instance_type: Some(schemars::schema::InstanceType::Object.into()),
        object: Some(Box::new(schemars::schema::ObjectValidation {
            properties: {
                let mut props = schemars::Map::new();
                props.insert(
                    tag.to_owned(),
                    schemars::schema::Schema::Object(schemars::schema::SchemaObject {
                        instance_type: Some(schemars::schema::InstanceType::String.into()),
                        enum_values: Some(<[_]>::into_vec(Box::new([value.to_owned().into()]))),
                        ..Default::default()
                    }),
                );
                props
            },
            required: {
                let mut required = schemars::Set::new();
                required.insert(tag.to_owned());
                required
            },
            ..Default::default()
        })),
        ..Default::default()
    })
}

impl JsonSchema for AnyItem {
    fn schema_name() -> String {
        "AnyItem".to_string()
    }

    fn json_schema(gen: &mut schemars::gen::SchemaGenerator) -> Schema {
      let discriminator = json!({
            "propertyName": "item_type",
        });
        let table_type = add_discriminator_case("item_type", "table");
        let chart_type = add_discriminator_case("item_type", "chart");

        let subschemas = schemars::schema::SubschemaValidation {
            one_of: Some(vec![
                table_type.flatten(<TableItem as schemars::JsonSchema>::json_schema(gen)),
                chart_type.flatten(<ChartItem as schemars::JsonSchema>::json_schema(gen)),
            ]),
            ..Default::default()
        };

        let schema_object = schemars::schema::SchemaObject {
            subschemas: Some(Box::new(subschemas)),
            extensions: BTreeMap::from_iter(vec![("discriminator".to_owned(), discriminator)]),
            ..Default::default()
        };
        schemars::schema::Schema::Object(schema_object)
    }
}

for generating the typescript schema, I did use  npx openapi-typescript
the final result ending looking like this,
AnyItem: ({item_type: "table", ...rest}) |  ({item_type: "chart", ...rest})


Install docker on Debian jessie isn't up-to-date

Hi,
after the release of Debian jessie the package docker.io doesn't exists in the main or backports repository of jessie anymore.
The docs should provide a valid and official way to install docker on jessie.
My currently workaround is to use the docker repository which is provided for ubuntu.
apt-get install apt-transport-https
apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 36A1D7869245C8950F966E92D8576A8BA88D21E9
echo "deb https://get.docker.com/ubuntu docker main" >> /etc/apt/sources.list
apt-get update
apt-get install lxc-docker

yes our ubuntu repo works on debian, we actually make sure it always does
the docs have been updated here on the master branch
http://docs.master.dockerproject.com/installation/debian/
they just need to be released

On Tue, Apr 28, 2015 at 3:46 PM, Christoph Heer<EMAIL_ADDRESS>wrote:

Hi,
after the release of Debian jessie the package docker.io doesn't exists
in the main or backports repository of jessie anymore.
The docs should provide a valid and official way to install docker on
jessie.
My currently workaround is to use the docker repository which is provided
for ubuntu.
apt-get install apt-transport-https
apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 36A1D7869245C8950F966E92D8576A8BA88D21E9echo "deb https://get.docker.com/ubuntu docker main" > /etc/apt/sources.list
apt-get update
apt-get install lxc-docker
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/12861.

great to hear.
In master there is still the advice to use backports or is there a open pull request to address this issue?

why was this closed, from what I can see in http://backports.debian.org/changes/jessie-backports.html docker.io is missing so the documentation is invalid?

Documentation in https://docs.docker.com/installation/debian/#installation is still invalid.

Watch out on this line from the above instructions:
echo "deb https://get.docker.com/ubuntu docker main" > /etc/apt/sources.list
should be:
echo "deb https://get.docker.com/ubuntu docker main" >> /etc/apt/sources.list

On Jessie, the easiest thing to do appears to be to run the installation script:
curl -sSL https://get.docker.com/ | sh

I think the website needs to be updated to match the script as mentioned by markusobsidian

@kcalliauw Thanks. I edited the example, just in case someone tried to execute it

Docs is still wrong.

Yup, docs are wrong.

wheezy is no longer the latest stable release. i think this should be fixed too.

After an unsuccessful try with the official docs, I was able to install docker using the following link: http://blog.programster.org/debian-8-install-docker/.
The official docs has to be updated.

This issue should not be closed. @jarus The official docs still do not work.

Still not there...
Sorry, this is totally confusing.
Please fix the docs next time first. Unless I'm missing something, the docs were wrong now for over a month. At least add a clear note there, that can't be so hard.

Last time I checked anyone could open a PR instead on commenting here
whining about it
On Tuesday, June 9, 2015, rebootl<EMAIL_ADDRESS>wrote:

Still not there...
Sorry, this is totally confusing.
Please fix the docs next time first. Unless I'm missing something, the
docs were wrong now for over a month. At least add a clear note there, that
can't be so hard.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/12861#issuecomment-110304723.

@jfrazelle  ... yes but,
if you google installing docker on debian the thirds link you're gonna land on, is here.
then just do what the first 2 links (official Docker docs) suggest:
curl -sSL https://get.docker.com/ | sh

https://packages.debian.org/source/docker.io there is a package in backport currently


Upgrade Dependencies

Overview
Upgrades Redux, React-Router and React-Select to the latest versions.
The outdated versions were introducing warnings into the console due
to not being optimized for React v16.
Brief description of what this PR does, and why it is needed.
Connects #1300
Demo
Manageable level of console warnings:

Testing Instructions

Run ./scripts/update and ./scripts/server
Confirm the website works as before
Open the developer console and confirm there are no warnings about UNSAFE_ lifecycle models.

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Looking at this now.


Properties are being removed from FeatureCollection

I have created a .pbf file, but when I decode it, all the properties are removed from each feature.
I used the Getools geobuf libraries to do this, encoded and decoded successfully, I am gzipping the outputsream,
File file = new File("/Projects/geobufs/test.pbf");
GeobufFeatureCollection geobufFeatureCollection = new GeobufFeatureCollection();
	InputStream inputStream = new FileInputStream(file);
    SimpleFeatureCollection decodedFeatureCollection = geobufFeatureCollection.decode(inputStream);
    inputStream.close();

    FeatureJSON fjson = new FeatureJSON(new GeometryJSON(DECIMALS));
	StringWriter writer = new StringWriter();
	fjson.writeFeatureCollection(decodedFeatureCollection, writer);
	String geoJSON = writer.toString();
	logger.info(geoJSON);

I download the file
var geobuffURL = "http://localhost:8080/geobuffer/test.pbf";
       var oReq = new XMLHttpRequest();
	   oReq.open("GET", geobuffURL, true);
	   oReq.responseType = "arraybuffer";
       oReq.onload = function(oEvent) {
       		var data = new Pbf(new Uint8Array(oReq.response))
	  		console.log(data)
	  		var geoJSON = geobuf.decode(data);

			console.log(JSON.stringify(geoJSON));
			//var layer = L.geoJson( geobuf.decode(data)).addTo(map);
		}
		oReq.send();

and it strips the properties from the geoJSON.
I am attaching both the geojson, and the pbf I encoded.
test.files.zip

I'm running into this issue as well. Happy to take a look and pop open a PR if the maintainers would like!


⚠ Cioos National CKAN has degraded performance

In 3389bb5, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 6123 ms

Resolved: Cioos National CKAN performance has improved in 17cf66d after 5 minutes.


Where can I locate the 'learning' module?

in the train_image_classifier.py file, there is a line that imports a module called learning. Is this a module that needs to be installed through pip install or is it a module provided by you? When I tried pip install learning, I was able to install it, but when I try train.py, it gave me an error message saying learning has no attribute train. Thanks!

Actually it just occurred to me that learning.py is in the tf_hardcore folder. For some reason, it's not recognized. I had to add the path to the folder to sys.path.

If you run train.py or eval.py, .:tf_hardcode:slim should have been added to PYTHONPATH, see:
https://github.com/yuantailing/ctw-baseline/blob/081c8361b0ed0675bf619ab6316ebf8415e93f46/classification/train.py#L15-L18
https://github.com/yuantailing/ctw-baseline/blob/081c8361b0ed0675bf619ab6316ebf8415e93f46/classification/train.py#L83
https://github.com/yuantailing/ctw-baseline/blob/081c8361b0ed0675bf619ab6316ebf8415e93f46/classification/train.py#L88


Translating into chinese

I translated these tips into chinese.

Have you need some pull requests?

+1

@jinlong thanks for the suggestion. At this time I'm not ready to expand this repo to other languages.

@AllThingsSmitty it's ok


Color sensor not grabbing an image

I encountered the precisely same issue as in #1237, but I need to get it to run reliably on my or other systems, or at least understand what causes it.
I have described the issue, with my code, at https://stackoverflow.com/questions/64770775/camera-not-grabbing-a-color-image
If I compile the SDK myself, in the k4aviewer.exe I get the depth and ir image shown just fine, the camera image though does not show.
In the pre-compiled installer, the k4aviewer shows all images, depth, ir and color, correctly. I was not able to figure out what causes this.

Based on your comments both here and stackoverflow it appears you have some sort of build issue. Can you please verify you correctly followed the build instructions here?

Originally I built it via a conan package we created. I just now created the file exactly according to the documentation, same behaviour: I can grab the IR and depth image, the color image does not work. I tested with the k4aviewer.exe to verify that its not my code causing this.

Originally I built it via a conan package we created. I just now created the file exactly according to the documentation, same behaviour: I can grab the IR and depth image, the color image does not work. I tested with the k4aviewer.exe to verify that its not my code causing this.

Edit;
Toolchain used:
cmake version 3.19.0-rc1
ninja 1.10.1
Visual Studio 2017 Community 15.9.29

I was at a colleagues system today, he has the precisely same hardware configuration. We checked on his system, he also received no color image. After some further testing we checked whether some of our company policy installs could be causing this, and when we disabled Kaspersky Virus Scanning, it worked. Enable Kaspersky, no color image again.
No idea why this would happen only with self-compiled libraries, but that solves this issue.


I can not disable this check "Sheet names cannot exceed 31 chars"

I saw there is a safe check for sheet name, but I can not disable it. Looks like a bug!
if(n.length > 31) { if(safe) return false; throw new Error("Sheet names cannot exceed 31 chars"); }

https://support.microsoft.com/en-us/office/rename-a-worksheet-3f1f7148-ee83-404d-8ef0-9ff99fbad1f9

Important:  Worksheet names cannot:

Be blank .
Contain more than 31 characters.
Contain any of the following characters: / \ ? * : [ ]
For example, 02/17/2016 would not be a valid worksheet name, but 02-17-2016 would work fine.
Begin or end with an apostrophe ('), but they can be used in between text or numbers in a name.
Be named "History". This is a reserved word Excel uses internally.


channel not found

After setting the bot channel id and toke in the .env file , it not works.
event - Starting SSH Bot session...
info  - Running version v3.0.0 on Node.js v20.12.2 on linux x64
info  - Check out the source code at https://github.com/igorkowalczyk/discord-ssh!
info  - Don't forget to star the repository, it helps a lot!
info  - Loading events...
event - Loaded 2 events from /events in 32ms
info  - Logging in...
error - Channel not found! Please check your CHANNEL_ID .env variable.

Check for any typos in .env and check if the bot can see this channel


addon-contexts roadmap

Roadmap to release v1.0.0

[x] Polish type annotations
[x] Properly configure the project
[x] Publish the package on npm
[x] Validate it is working on a user project

Package have been released on NPM successfully, issue closed.

Related to #2 #3 #4


How to send 9-bit data using spi？

I have a display and need to use 9-bit data. How do I set it up? Thank you!

Hi @qwert1213131 ,
The data phase of ESP32 master is always in units of bytes (8bits), however, we have three phases whose length is configurable. They are (in order of their apperance):
CMD, ADDR, DUMMY(not available for full duplex mode)
We can flexibly use them to form a format of [D/CX] [CMD-8bits] [D/CX] [other data]
e.g.
cmd: 9 bits, the MSB to be the first DCX
addr: 1 bit, the second DCX.
But if you need every single byte to be of 9-bit length, I'm afraid it's not available for ESP32.

Hi @ginkgm ，
Thank you very much for your reply. I want to be able to send 9 bits of data repeatedly, so referring to your suggestion, I use a 1-bit address and 8-bit data, but mcu starts to restart indefinitely. It seems that you have to consider another screen that supports 8-bit spi

@qwert1213131
the MCU refers to ESP32? are you using half duplex mode or full duplex mode?
Could you please try use 1-bit command and no address? maybe it's not valid to use address without command in half duplex mode?

@ginkgm
The past should be a problem with the program that caused the reboot.
Using esp32
In full-duplex mode, both addresses and commands can work. Currently, it is [1-bit command or address] [8-bit data].
But the screen refresh is very slow, because every time you send it, you have to follow this format, only one byte and one byte.
I will try again if there is a suitable method.
Thanks very much！

@qwert1213131
I'm not quite sure if this works, but have you ever tried to send several bytes of data with only one D/CX bit?

@ginkgm
Currently 1-bit command, one byte of data, is already working, but the frame refresh rate is very slow.
With the 1-bit command, there are still many problems with multiple bytes of data, which should not meet the operational requirements of the ST7789 spi-3 mode.
thank you very much.
I decided to change a spi screen, using the spi-4 mode like ili9341.


vmath not in nimble requires but imported in windy.nim

Thank you for the issue. Windy gets vmath from pixie and its in the requires.
You can see it installs vmath https://github.com/treeform/windy/runs/5541179849?check_suite_focus=true#step:4:23 when using nimble.
Do you have trouble installing or running windy through nimble? Please reply with command that is giving you trouble, so that we can help you.

Thank you for the issue. Windy gets vmath from pixie and its in the requires.
You can see it installs vmath https://github.com/treeform/windy/runs/5541179849?check_suite_focus=true#step:4:23 when using nimble.
Do you have trouble installing or running windy through nimble? Please reply with command that is giving you trouble, so that we can help you.

More just for completeness since it's imported explicitly in windy here https://github.com/treeform/windy/blob/master/src/windy.nim#L1

I do not see what including it redundantly in the windy.nimble file actually accomplishes, other than guaranteeing the Windy declared vmath dep and Pixie vmath dep versions fall out of sync in time. Is there a benefit to this completeness? I see costs.

If you include vmath in both windy.nimble and pixie.nimble they will get out of sync and its even worse.
I am going to close this issue for now.
If you do nimble install windy or some thing and it does not work, feel free to reopen this issue.

How is them getting out of sync worse? What is going to break? Its not a big deal, but i would expect directly used libraries to be specified as deps.

I asked earlier and still haven't really gotten an answer. Can you please help me understand exactly why this matters? Like in reality, not theoretical things like completeness or expectation. I see no reason to write any line of code (or config) that serves no purpose.

In the past we did what you suggested and people had installation issues where the older library was installed ... probably a nimble bug: https://github.com/nim-lang/nimble/issues/890


Links

This PR addresses: https://github.com/cloudfoundry-incubator/haproxy-boshrelease/issues/55.
I would have liked this PR to be more generic, however I was unable to find a way to consume links with conflicting link types.

I would love to, however at this point the diego release is not exposing this property:
https://github.com/cloudfoundry/diego-release/blob/develop/jobs/ssh_proxy/spec#L13-L15 https://github.com/cloudfoundry/diego-release/blob/develop/jobs/ssh_proxy/spec#L13-L15
I have tried specifying link properties (http://bosh.io/docs/links-manual.html http://bosh.io/docs/links-manual.html) but this is not allowed when also doing normal linking..

On 04 May 2017, at 15:11, Geoff Franks<EMAIL_ADDRESS>wrote:
@geofffranks commented on this pull request.
In jobs/haproxy/spec https://github.com/cloudfoundry-incubator/haproxy-boshrelease/pull/56#discussion_r114768450:

@@ -165,6 +174,8 @@ properties:
# Setting verify requires ha_proxy.backend_ca_file key to be set.
backend_verifyhost: example.com # optional - hostname to verify in the x509 certificate subject for SSL-enabled backend servers
# only used if backend_ssl: verify is set

ha_proxy.tcp_link_port:
description: "Port haproxy should listen on when using the tcp_backend link"
Shouldn't this be something we pull from the link itself?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cloudfoundry-incubator/haproxy-boshrelease/pull/56#pullrequestreview-36262806, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXPGX0dX06GbRPYQ1B1TCveorlkMA0hks5r2c6OgaJpZM4NPbfS.

Yeah, I hadn't yet gotten to the part where you try to use the link then fail-over. It's all good.
Though looking at the bosh.io http://bosh.io/ docs, the way you're calling p() isn't mentioned (to provide a default if not present). I assume it works, since you were testing with diego not providing the property, right?

On May 4, 2017, at 9:24 AM, Ruben Koster<EMAIL_ADDRESS>wrote:
I would love to, however at this point the diego release is not exposing this property:
https://github.com/cloudfoundry/diego-release/blob/develop/jobs/ssh_proxy/spec#L13-L15 https://github.com/cloudfoundry/diego-release/blob/develop/jobs/ssh_proxy/spec#L13-L15
I have tried specifying link properties (http://bosh.io/docs/links-manual.html http://bosh.io/docs/links-manual.html) but this is not allowed when also doing normal linking..

On 04 May 2017, at 15:11, Geoff Franks<EMAIL_ADDRESS>wrote:
@geofffranks commented on this pull request.
In jobs/haproxy/spec https://github.com/cloudfoundry-incubator/haproxy-boshrelease/pull/56#discussion_r114768450:

@@ -165,6 +174,8 @@ properties:

Setting verify requires ha_proxy.backend_ca_file key to be set.
backend_verifyhost: example.com # optional - hostname to verify in the x509 certificate subject for SSL-enabled backend servers
only used if backend_ssl: verify is set

ha_proxy.tcp_link_port:
description: "Port haproxy should listen on when using the tcp_backend link"
Shouldn't this be something we pull from the link itself?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cloudfoundry-incubator/haproxy-boshrelease/pull/56#pullrequestreview-36262806, or mute the thread https://github.com/notifications/unsubscribe-auth/AAXPGX0dX06GbRPYQ1B1TCveorlkMA0hks5r2c6OgaJpZM4NPbfS.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/cloudfoundry-incubator/haproxy-boshrelease/pull/56#issuecomment-299183655, or mute the thread https://github.com/notifications/unsubscribe-auth/AG2SUvpxJzd2L-kaEewWwDcf-lfOoHfMks5r2dFwgaJpZM4NPbfS.

@geofffranks here you can find defaults for p().

Thanks!

Soon as release notes get through CI

On May 4, 2017, at 10:10 AM, Ruben Koster<EMAIL_ADDRESS>wrote:
@geofffranks https://github.com/geofffranks thanks for merging! when is the next release planned?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/cloudfoundry-incubator/haproxy-boshrelease/pull/56#issuecomment-299196363, or mute the thread https://github.com/notifications/unsubscribe-auth/AG2SUt1X23ynqSTUbgs0uOOQ9vByx_P-ks5r2dxfgaJpZM4NPbfS.

nice!

Hey, @geofffranks, I'm a little surprised that you accepted this PR with the tcp_backend link as type ssh_proxy instead of as a more generic identifier, given that it's still essentially specific to the ssh_proxy job in diego-release. I'd be open to having that job in diego-release provide a separate link type (say, tcp-backend) if it allows the HAProxy release to be more generic. Along those lines, it also seems like it would be beneficial to supply the backend port in the link: the template is looking for it at the reasonable location of port, but unfortunately the ssh_proxy job embeds that port in its diego.ssh_proxy.listen_addr property, and BOSH links don't yet offer a mechanism to rename or otherwise manipulate properties in the providing job to different properties in the link. That said, I'm starting to consider how to de-namespace and simplify the diego-release job properties now that BOSH recommends scoping the properties in a manifest to each job in an instance group, and it would be reasonable there to extract that port into a separate property (say, port or server_port) that would be more natural to include in the links that job provides.
Also pinging @cppforlife and @Amit-PivotalLabs for any suggestions they may have on link design and/or potential forthcoming link capabilities.
Thanks again,
Eric


When and how should hints_text be used?

Describe the issue
There's hints_text available in the datasets but it's unclear how it should be used. Is there any recommendation on this?
Suggest an improvement to documentation
No response

Hi @atinylittleshell thanks for pointing this out, I will update the documentation here to mention this. I realize there's been some questions raised about when hints_text is appropriate to use.
In short, for participating on the SWE-bench leaderboard, using hints_text in any manner is not allowed. Although we don't explicitly say this in the original paper, we also do not make any mention of using the hints_text anywhere.
During the data collection process of putting together SWE-bench, we collected all comments between the creation of the PR and the first commit which became the hints_text field. Our original goal was to just collect as much interesting information as possible.
However, when formulating the task based on this data, we ended up deciding that the original problem_statement (a.k.a. issue description) text would suffice for conveying the bug that needed to be solved, and would also be most true/faithful to the problem + information that the codebase maintainer was given.


change alias priority logic

Change Summary
This changes the priority of aliases to fix #1177, however it introduces a potential breaking change.
For example:
https://github.com/samuelcolvin/pydantic/blob/80152f337191ba7539e077c02a236bb713fdddd2/tests/test_aliases.py#L189-L204
before alias_generator on the child would take taken priority. While it's arguable whether this is correct and we could allow alias_generator to return a tuple (<alias>, <alias priority>) to allow customisation, I'm aware this is a breaking change.
However I'm not sure how else to fix the issue in a cleaner way. @tiangolo, @dmontagu, anyone else involved - do you think we should:

merge this with a "potential breaking change" warning
wait to merge this until v2
provide some other fix for #1177

?
Checklist

[x] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added

Hi everyone!
Is there any option to increase priority of alias_generator for particular model?
I have a situation, where I am extending old API of some legacy software with some additional functionality under a new, sort of "unified" API.
I have a base model with aliases that match legacy API. I also have extended models that inherit from that base models and these will be served for client application. Now the problem is that I would like to use alias_generator in that extended models to replace snake case field names with camel case. Obviously, just as explained in the docs, this works only for the fields defined in child model.
Here's the simplified code example:
`
def to_camel(string: str) -> str:
return ''.join(word.capitalize() for word in string.split('_'))
class OldAPIModel(BaseModel):
id: int = Field(..., alias='Identyfikator')
name: str = Field(..., alias='Nazwa')
description: str = Field(..., alias='DlugiOpis')
class NewAPIModel(OldAPIModel):
product_keywords: Optional[List[str]] = Field([])
class Config:
alias_generator = to_camel
`
Now, the use case for that is as following:
I get data from legacy api using OldAPIModel, then gather addidional data from other database and pass all of that to the client using NewAPIModel. As I see it, it would be nice to set some flag (that is disabled by default) to force use of alias_generator in child models. Of course I can just copy all fields from OldAPIModel to NewAPIModel, but I would like to avoid code duplication.
Or maybe there is some other way/usage pattern that I should follow in this case?
BR,
Mateusz

no way I know of to force use of a different alias.
You have the options of:

copy them like you said
generate the model yourself using create_model(), with Field() and alias's set on the field
look through the pydantic code and find some way to hack it, no idea if this is possible, might break in minor version updates

Hi everyone!
Is there any option to increase priority of alias_generator for particular model?
I have a situation, where I am extending old API of some legacy software with some additional functionality under a new, sort of "unified" API.
I have a base model with aliases that match legacy API. I also have extended models that inherit from that base models and these will be served for client application. Now the problem is that I would like to use alias_generator in that extended models to replace snake case field names with camel case. Obviously, just as explained in the docs, this works only for the fields defined in child model.
Here's the simplified code example:
def to_camel(string: str) -> str:
    return ''.join(word.capitalize() for word in string.split('_'))

class OldAPIModel(BaseModel):
    id: int = Field(..., alias='Identyfikator')
    name: str = Field(..., alias='Nazwa')
    description: str = Field(..., alias='DlugiOpis')

class NewAPIModel(OldAPIModel):
    product_keywords: Optional[List[str]] = Field([])
    class Config:
        alias_generator = to_camel

Now, the use case for that is as following:
I get data from legacy api using OldAPIModel, then gather addidional data from other database and pass all of that to the client using NewAPIModel. As I see it, it would be nice to set some flag (that is disabled by default) to force use of alias_generator in child models. Of course I can just copy all fields from OldAPIModel to NewAPIModel, but I would like to avoid code duplication.
Or maybe there is some other way/usage pattern that I should follow in this case?
BR,
Mateusz

@mat02, if you're still looking for a way to do this, I figured out a reasonable solution using create_model.
If you already figured it out, then this can just hang out here for posterity 😸
from itertools import chain

from pydantic import BaseModel, create_model

class Bread(BaseModel):
    class Config:
        alias_generator = lambda s: f"bread_{s}"

    name: str
    id: int

class Butter(BaseModel):
    class Config:
        alias_generator = lambda s: f"butter_{s}"

    name: str
    id: int

print(Butter.__fields__)
print(Bread.__fields__)

class BreadAndButterDoesntWork(Bread, Butter):
    """This won't work :("""

print(BreadAndButterDoesntWork.__fields__)

def BreadAndButterWorks():
    return create_model(
        "BreadAndButter",
        **{
            f.alias: (f.type_, ...)
            for f in chain.from_iterable(
                getattr(m, "__fields__").values() for m in [Bread, Butter]
            )
        },
    )

print(BreadAndButterWorks().__fields__)

print(BreadAndButterWorks().__fields__)

Hopefully this helps someone!
Fwiw, I was dealing with a use case where we are using Pydantic models to get bits of data from various APIs and databases in order to ultimately stitch them together into one big data warehouse table.
Imagine in the above, that Bread and Butter were each responses from separate apis The idea was to compose all the models together in order to form that one big table with minimal repetition and a lot of consistency. Since each of the "parts" had a source_id and a natural_key, disambiguation with alias_generator made a ton of sense. What a surprise when the big rolled up model lost all those aliases!
My experience is limited with this library, and my problem is somewhat unusual, but while I'm here I'll toss 2 cents in the bucket for raising the priority of alias_generator in the future. :)


media-downloader: 2.9.0 -> 3.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mhogomchungu/media-downloader/releases.
meta.description for media-downloader is: A Qt/C++ GUI front end to youtube-dl
meta.homepage for media-downloader is: https://github.com/mhogomchungu/media-downloader
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.0 with grep in /nix/store/a4z8hmmqxp1m3kqqjgyi6zim3k06n6rn-media-downloader-3.0.0
found 3.0.0 in filename of file in /nix/store/a4z8hmmqxp1m3kqqjgyi6zim3k06n6rn-media-downloader-3.0.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
media-downloader

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

media-downloader

Maintainer pings
cc @zendo for testing.

There's a new update: https://github.com/mhogomchungu/media-downloader/releases/tag/3.1.0


Add NODE_ENV=production to build script

Right now, babel-plugin-emotion adds source maps for styles which bloats the bundles massively. You already had a production env in babel which has emotion's source maps disabled so this just adds NODE_ENV=production to the build script.
(This isn't a thing that has to happen with babel-plugin-emotion@10 but for v9, sourceMap has to be disabled otherwise you'll get huge bundles)
Closes #15

@mitchellhamilton is this a duplicate of #16?

Released in v1.3.2


Printing raw response to console, without codable

I'm using this library to integrate with a server thats not under my control. I really need the ability to print the response to the console without using Codable, for debugging purposes. The returned object is very complex and i'm struggling to create a model object matching it, without being able to see what it is i'm getting back.
The ability to either turn on printing through the logger or even being able to extract the raw Data out of ArugmentExtractor would suffice.
e.g.:
connection?.on(method: "operations", callback: { argumentExtractor in
    let rawResponse = String(data: argumentExtractor.getArgumentData(), encoding: .utf8)
    print("Operations response: \(rawResponse)")
})

This is not supported - for the public API responses are always materialized . However, if you enable logging at the debug level the payload received by the client will be logged to the console. From this message you can extract your JSON - it will be in the arguments JSON array. You can find examples in #192.

I've done this, yet arguments are empty, can see the JSON in the debug output, but arguments are always nil


--live-from-start and --get-url don't work together

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
USA
Provide a description that is worded well enough to be understood
When using yt-dlp.exe --get-url epxnryxRUWs --live-from-start -vU
it gives a stream that A) in VLC goes to the head of the stream and won't let you navigate backwards, and B) in opencv gives the following error:
[<EMAIL_ADDRESS>global cap.cpp:166 cv::VideoCapture::open VIDEOIO(CV_IMAGES): raised OpenCV exception:
OpenCV(4.10.0) C:\GHA-OCV-1_work\ci-gha-workflow\ci-gha-workflow\opencv\modules\videoio\src\cap_images.cpp:244: error: (-5:Bad argument) CAP_IMAGES: error, expected '0?[1-9][du]' pattern, got: https://manifest.googlevideo.com...
This should repro with any livestream, as I have tried many.  I am trying to run some computer vision on livestreams, but can't because of this.  without --live-from-start, Opencv opens the stream fine.  I am using Opencv in rust and am parsing the output of yt-dlp.exe in rust to get the URL.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['--get-url', 'epxnryxRUWs', '--live-from-start', '-vU']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds [0b6b7742c] (win_exe)
[debug] Python 3.10.11 (CPython AMD64 64bit) - Windows-10-10.0.26100-SP0 (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.21.0, brotli-1.1.0, certifi-2024.12.14, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.40.1, urllib3-2.3.0, websockets-14.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1837 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-master-builds/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds)
[youtube] Extracting URL: epxnryxRUWs
[youtube] epxnryxRUWs: Downloading webpage
[youtube] epxnryxRUWs: Downloading ios player API JSON
[youtube] epxnryxRUWs: Downloading mweb player API JSON
[youtube] epxnryxRUWs: Downloading MPD manifest
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec, channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Default format spec: best/bestvideo+bestaudio
[info] epxnryxRUWs: Downloading 1 format(s): 137+140
https://manifest.googlevideo.com/api/manifest/dash/expire/1735560240/ei/zzdyZ5LXOdj8sfIP3dbCgQE/ip/<IP_ADDRESS>/id/epxnryxRUWs.1/source/yt_live_broadcast/requiressl/yes/xpc/EgVo2aDSNQ%3D%3D/as/fmp4_audio_clear%2Cfmp4_sd_hd_clear/vprv/1/rqh/2/pacing/0/keepalive/yes/fexp/51326932%2C51331020%2C51335594%2C51371294/itag/0/playlist_type/DVR/sparams/expire%2Cei%2Cip%2Cid%2Csource%2Crequiressl%2Cxpc%2Cas%2Cvprv%2Crqh%2Citag%2Cplaylist_type/sig/AJfQdSswRAIgUOhgbc4BJ3arPzrDI4VIkWUpO1tSzhZ9-VrL4c1ZxFwCIDLgHP6usVDRuKsC0jxu_ihOhfnZ4K3pDa2FfLVH8QHZ
https://manifest.googlevideo.com/api/manifest/dash/expire/1735560240/ei/zzdyZ5LXOdj8sfIP3dbCgQE/ip/<IP_ADDRESS>/id/epxnryxRUWs.1/source/yt_live_broadcast/requiressl/yes/xpc/EgVo2aDSNQ%3D%3D/as/fmp4_audio_clear%2Cfmp4_sd_hd_clear/vprv/1/rqh/2/pacing/0/keepalive/yes/fexp/51326932%2C51331020%2C51335594%2C51371294/itag/0/playlist_type/DVR/sparams/expire%2Cei%2Cip%2Cid%2Csource%2Crequiressl%2Cxpc%2Cas%2Cvprv%2Crqh%2Citag%2Cplaylist_type/sig/AJfQdSswRAIgUOhgbc4BJ3arPzrDI4VIkWUpO1tSzhZ9-VrL4c1ZxFwCIDLgHP6usVDRuKsC0jxu_ihOhfnZ4K3pDa2FfLVH8QHZ

The live-from-start formats can only be downloaded in a non-standard way, which VLC hasn't/can't implement. If you want to re-implement the feature in your own app, here's the relevant code: https://github.com/yt-dlp/yt-dlp/blob/0b6b7742c2e7f2a1fcb0b54ef3dd484bab404b3f/yt_dlp/extractor/youtube.py#L2886-L3043

@pukkandan Thanks for the response.  Before I go down that route, I want to exhaust easier options.  I tried gstreamer, that doesn't seem to like it either.  It seems that the fragments that that yt-dlp writes could help, is the format of those custom to yt-dlp or could there be some could already I can use that could read them?  Naively, changing the extension to .mp4 and opening them in vlc seems to work, so I assume I could just have yt-dlp save the fragments, then my program could change them to mp4 files and read them in.  Not instant, but for my application it only needs to be semi, real time, a few seconds won't matter.


add Limitations

What problem does this PR solve?
https://github.com/pingcap/tispark/issues/2190
https://asktug.com/t/topic/302949
https://asktug.com/t/topic/302960
What is changed and how it works?
change readme
Check List
Tests

No code

@sunxiaoguang @xuanyu66 @birdstorm PTAL

/run-idc_jenkins_ci_tispark/integration-test

/run-all-tests

/cherrypick release-2.4

/cherrypick release-2.5


Add version number to site

We need some way to be able to see what version of the software is currently deployed on a site.
This can be in the admin area or wherever is easiest.

Excited about this one @istride !


Error while loading image in Safari only

In the latest version of the code I cannot load an image in Safari. I get the following error:
cropper2.min.js:118
XMLHttpRequest cannot load

See #536.


Need mor themes!

The website (http://please.hackmyresume.com/) indicates that HackMyResume can use any JSON Resume theme.  If this is really true, then I would settle for some mere documentation on where to find those themes.
The link points only to a gallery website (https://jsonresume.org/themes/) that previews those themes, with no download option.  The JSON Resume people really want you to use their online hosting service... and so even though their app is ostensibly open source, they don't make it easy to actually use the source yourself.
I spent awhile crawling through all of their repos on GitHub, unsuccessfully looking for the actual source code for their themes, before giving up.  Is it crazy to ask anybody to post a link to themes source?
This is a neat concept, but I just don't think it's really ready to use yet.

@steve-perkins you're experiencing some of the same frustrations that motivated me to build HMR in the first place.

The JSON Resume people really want you to use their online hosting service... and so even though their app is ostensibly open source, they don't make it easy to actually use the source yourself.

I'm not affiliated with JSON Resume beyond submittting a few issues pertaining to the exact stuff you're complaining about. But look at who they're up against: LinkedIn, Monster, and worse. I share your frustration with the lack of local-only generation in the "official" set of JSON Resume tools, which is why HackMyResume features only local generation. But even though the JSON Resume workflow may not support out of the box local generation, every piece of server apparatus, every theme, and the JSON Resume standard itself: all 100% open source. I don't think they're required to eviscerate their own future potential business model in order to better adhere to a particular notion of what FOSS actually means.
That said, you've put your finger on the single biggest gap in both of these tools: resume-cli doesn't support local generation, and HackMyResume doesn't support transparent use of JSON Resume themes. I can't do anything about the first, but the second is currently the number one development priority for HMR and the reason why HMR themes are still tagged with the beta designator, and if you'll give it just a few more days, I think you'll find usability in this area is greatly improved. In the meantime, to hunt down official JSON Resume theme sources you can follow the GitHub links from here or from the JSON Resume organization account:

https://www.npmjs.com/search?q=jsonresume-theme

The HackMyResume themes are here:

https://github.com/fluentdesk/fluent-themes

Please check by in a few days (before the new year, though) for some improvements to how HMR handles foreign themes, and also keep in mind that JSON Resume is prepping for a v1.0.0 drop in the not too distant future. Thanks for the input and let me know if there's any specific functionality you'd like to see. -HAL

[Edit for brevity] Fix for this is in progress. In v1.3, you'll be able to generate any valid JSON Resume or FRESH theme. Thanks.

@steve-perkins I believe you're looking for this: https://github.com/jsonresume/theme-manager.
You can also find a smaller number of themes here: https://github.com/fluentdesk/fluent-themes.
The JSON Resume project uses the former for all their themes.

Should the -t option also handle NPM modules? Then you could npm install jsonresume-theme-slick for instance and run hackmyresume BUILD resume.json TO out/resume.all -t node_modules/jsonresume-theme-slick

This is related to issue #56.
It should be resolved in v1.3

FYI - I wrote this FAQ. Hope it helps future users.

Thanks @pmbenjamin! I'd prefer to hold off on direct mentions of resume-cli unless we also document that resume-cli silently uploads user data to the registry not only on publish, but for a regular (ostensibly "local") export. We should probably also mentioned that JSON Resume is sponsored by Hired.com for full disclosure. Otherwise, very helpful stuff!


Engineering planning for next diseases work

https://docs.google.com/document/d/11QpgPUGDM1HTqHEjae7roeJaynkcJ_15MoQ8jGaTphs/edit?usp=sharing

Next up here, after engineering team review 2/3: spike on data storage solutions for the results object. We could enforce this as a row table, or use a jsonb and enforce consistency via spring, as we do elsewhere.

This is done! See the doc above.


update tests to always resolve relative URLs

updates tests to match microformats/microformats2-parsing#10 by fixing one broken test in v1/hcard/email, and adding a new test in v2/hcard/relativeurlsempty that will pass only with the new parsing rules implemented.
(this should only be merged once the proposed spec change has been made)

prefixing emails with example.com links looks wrong

But that's what will happen with the new expansion rules... They're treated as relative URLs.  We could also change the examples to something more realistic?

Now that microformats/microformats2-parsing#10 has been accepted, this should be resolved.  As I noted above, I don't disagree that prefixing email addresses with "http://example.com/" looks weird, but I believe that's the proper interpretation of these values.


Support for Openshift env with non-root(oracle) user

Hi,
This PR has fix for Apache Webtier to run in Openshift env with non-root user and with port 8080.
This sample is being used as part of Apache Webtier Setup in Openshift env as per WebLogic Kubernetes Operator Samples: "https://github.com/oracle/weblogic-kubernetes-operator/tree/master/kubernetes/samples/charts/apache-webtier".
Currently tested in below scenario:

docker run -d -e WEBLOGIC_HOST= -e WEBLOGIC_PORT= -p 8080:8080 oracle/apache:<IP_ADDRESS>
Openshift env as per '"https://github.com/oracle/weblogic-kubernetes-operator/tree/master/kubernetes/samples/charts/apache-webtier"  for a non-SSL usecase.

Is this still a WIP or is it ready for review?

Is this still a WIP or is it ready for review?

Its still in review as the changes impact the samples used in Operator repo: https://github.com/oracle/weblogic-kubernetes-operator/tree/master/kubernetes/samples/charts/apache-webtier.
So we are planning to finalize the changes this week.

@ashageetha Could you reply to the two outstanding comments from Dongbo?

Sure. We are working on to support both 80 and 8080 this week. I will update you once its ready.

I have added WIP to the title of the PR so it doesn't get merged too early. Once you are happy with the code, please remove that and the merge will be allowed.


SVG images do not stay in frame in tabs

Past Issues Searched

[X] I have searched open and closed issues to make sure that the bug has not yet been reported

Issue is a Bug Report

[X] This is a bug report and not a feature request, nor asking for support

Describe the bug
When I add an SVG image within the tab shortcode, the image displays but is running out of the frame. PNGs seem to stay within the frame.
Expected behavior
Have SVG images behave the same as PNG images in a tab.
Screenshots

Environment
- OS: MacOS
- Browser: Chrome
- Browser Version: 116

Thank you so much!

@Nadine2016 Are you able to provide a sample SVG image file?
I'm having trouble replicating the behaviour in your screenshot with the few SVG images I've tested.
Also, are you inserting the SVG using regular markdown syntax? e.g. ![my image](/my/image/path/file.svg)

@colinwilson
this is the code that I am using
{{< tabs tabTotal="2">}}
{{% tab tabName="GCP" %}}
This is content for GCP.

![Testimage](/img/insecure.svg)
{{% /tab %}}
{{% tab tabName="EKS" %}}
This is content for EKS.

| Value | Description |
| -- | -- |
| Value 1 | Description |
| Value 2 | Description |

{{% /tab %}}
{{< /tabs >}}

And this is an image.


How to run this code ?

Any demo video in youtube
And
Which ide did you use to run

No there's no demo video. The steps in README are self-explanatory. Any IDE which has Python installed can be used to run the codes. Closing the issue since the questions asked are pretty much answered in README.


chore(system_monitor): fix typos

Description

Fixed several spell-check errors.
If you feel this PR is large, you can revert 4f5e15ef017b4a146136f900d65d4b93ef589d39 and merge only e948de4ea5d761ae0c083b69c44aade00c91b77e.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

/__w/autoware.universe/autoware.universe/system/system_monitor/reader/hdd_reader/hdd_reader.cpp:57:8: error: redefinition of ‘struct HddInfo’
   57 | struct HddInfo
      |        ^~~~~~~
In file included from /__w/autoware.universe/autoware.universe/system/system_monitor/reader/hdd_reader/hdd_reader.cpp:20:
/__w/autoware.universe/autoware.universe/system/system_monitor/include/hdd_reader/hdd_reader.hpp:72:8: note: previous definition of ‘struct HddInfo’
   72 | struct HddInfo

I removed _ from the struct, but was it to avoid redefinition? :thinking:
I'll consider renaming it.

Since this PR is a little big, I'll split the refactoring part into a separate PR.

@ito-san Since I've drastically changed this PR, could you recheck it, please?


style(Mathlib/Data/Seq/Seq): rename Stream'.Seq to Sequence

Using Stream'.Seq without opening Stream' is tiresome, but if we open it, we must discriminating declarations in Stream'.Seq and Stream'.
So, we rename Stream'.Seq to Sequence.
Additionally, we rename Stream'.Seq1 & Stream'.WSeq to Sequence1 & WSequence.

This PR consists Continued fractions makes an isomorphism between irrationals and baire space
 project

I'm sceptical here, as I think primed names are user-unfriendly. I won't block if someone wants to proceed, but would prefer to see a better solution with usable names.

@semorrison
How about Sequence?

I'd prefer if someone with more expertise on this material continued the review here!

@semorrison
I renamed.

@Komyyy  Coming here for PR triage: thanks for your PR. It currently has a merge conflict, which means it won't show up in the review queue (and is less likely to be noticed). Can you merge the master branch, please?

❗This project is temporarily frozen, PRs are all closed for a moment. Detail


Escape latex to avoid cookiecutter/jinja2 problems

It looks like latex needs to be escaped:
https://github.com/ansible/ansible/issues/16443

And in turn, the cookiecutter {% raw %} and {% endraw %} escape sequences should be escaped in latex, i.e. % {% raw %} and % {% endraw %}

Great! Closed by #15


security/acme-client: certificate installing fails

HI,
Version:
OPNsense 17.7.1_2-amd64
FreeBSD 11.0-RELEASE-p12
OpenSSL 1.0.2l 25 May 2017
Acme Client now works, but the new certificate will not be installed.
15:32:37 CEST 2017] Installing full chain to:/var/etc/acme-client/certs/59a50a1d94e960.04057068/fullchain.pem
15:32:37 CEST 2017] Installing key to:/var/etc/acme-client/keys/59a50a1d94e960.04057068/private.key
15:32:37 CEST 2017] Installing CA to:/var/etc/acme-client/certs/59a50a1d94e960.04057068/chain.pem
15:32:37 CEST 2017] Installing cert to:/var/etc/acme-client/certs/59a50a1d94e960.04057068/cert.pem
15:32:36 CEST 2017] _on_issue_success
15:32:36 CEST 2017] And the full chain certs is there: /var/etc/acme-client/home/xxx/fullchain.cer
15:32:36 CEST 2017] The intermediate CA cert is in /var/etc/acme-client/home/xxx/ca.cer
...
15:32:36 CEST 2017] Le_LinkIssuer='https://acme-v01.api.letsencrypt.org/acme/issuer-cert'
15:32:36 CEST 2017] Your cert key is in /var/etc/acme-client/home/xxx/xxx.key
15:32:36 CEST 2017] Your cert is in /var/etc/acme-client/home/xxx/xxx.cer
15:32:36 CEST 2017] Cert success.

The webinterface shows me the old one.
I checked on the filesystem:
/var/etc/lighty-webConfigurator.conf
...
ssl.pemfile = "/var/etc/cert.pem"
...

So I checked the certifiate in /var/etc/cert.pem, but this is the old one, so not installed.
Also I checked /var/etc/acme-client/certs/59a50a1d94e960.04057068/cert.pem, this is the new one.
Kind Regards
Sören

Hi @smindorf, thanks for your report!
Let me begin with an important clarification: the acme plugin will never directly install the certificate outside of /var/etc/acme-client. Instead it will import the certificate to OPNsense's Certificate Manager.
Can you confirm that the new certificate is visible in System -> Trust -> Certificates?
So to actually use the new certificate you have to select it for the desired use-case. Two things are important:

If you want to use the certificate for the WebUI, go to System -> Settings -> Administration -> SSL Certificate and choose the Let's Encrypt certificate from the list.
To ensure that the WebUI certificate is updated when the certificate is renewed, you need to define a Restart Action in Services -> Let's Encrypt -> Restart Actions and select "Restart OPNsense WebUI" for Restart Command. Afterwards just add this new Restart Action to your certificate in Services -> Let's Encrypt -> Certificates. Note that this will automatically restart the WebUI everytime the certificate is renewed.

HTH.

Hi @fraenki,
Thank you for your reply.
Yes it is there, two times. The old one and the new one. Both are Let's Encrypt certificates.
The actually one is my old one from OPNSense 17.1.8.
And actually the certificate is set to the old one.
Also I've set the cron to restart the webUI.
As I understand this complete acme client is that I configure the client and it will be automatically renew the certificate and install it fpr the WebUI. But this was not working. Don't know why I've now two LE Certificates in my Tuststore.
Now I've select the new certificate an it was installed correctly.
Kind Regards,
Sören

As I understand this complete acme client is that I configure the client and it will be automatically renew the certificate and install it fpr the WebUI.

Yes, the acme plugin is able to do this, given that the requirements are all met, as described in my previous post.

But this was not working.

I've just retested all of this on my OPNsense test box and it's still working as expected. So even after the changes in #236 it still looks good to me. No regression afaict :)

Don't know why I've now two LE Certificates in my Tuststore.

Me neither. :( The logs are (hopefully) key to this issue. You can check System -> Configuration -> History to find out if the acme plugin updated an existing certificate or if it imported a new certificate to Certificate Manager. The acme plugin will use the following log messages:

when importing a new certificate: "Imported Let's Encrypt SSL certificate..."
when updating an existing certificate: "Updated Let's Encrypt SSL certificate..."

Besides that it might be useful to check System -> Log File for further acme log messages.

Ok @fraenki,
I've checked the History:
9/4/17 15:32:37	11.2	174 KB	(root): Updated Let's Encrypt SSL certificate: xxx
8/29/17 09:35:46	11.2	174 KB	(root): Updated Let's Encrypt SSL certificate: xxx
8/29/17 08:51:32	11.2	174 KB	(root): Updated Let's Encrypt SSL certificate: xxx
8/29/17 08:47:35	11.2	174 KB	(root): Imported Let's Encrypt SSL certificate: xxx

Now I understand the new Certificate but I don't know where this import comes from.
Maybe from finding the bug #236
Sorry...
Sören

8/29/17 08:47:35	11.2	174 KB	(root): Imported Let's Encrypt SSL certificate: xxx

Have you at some point deleted the certificate from Services -> Let's Encrypt -> Certificates and recreated it? This would explain why the certificate was imported a second time.
(FYI, the acme plugin records the hidden certRefId from OPNsens's Certificate Manager. This information is lost if the certificate is deleted from Services -> Let's Encrypt -> Certificates. And the acme plugin will never delete a certificate from the Certificate Manager. That's why a duplicate entry in Certificate Manager is easily possible if the user deletes and recreates the same certificate.)

Maybe from finding the bug #236

The issue in #236 would not trigger a re-import of the certificate, AFAICT.

Hello @fraenki,

Have you at some point deleted the certificate from Services -> Let's Encrypt -> Certificates and recreated it? This would explain why the certificate was imported a second time.

I didn't delete here anything. But I've deleted the acme-plugin and installed it again to test what is going wrong before I've reported the Bug #236
I also switch from production to stable and back.

The issue in #236 would not trigger a re-import of the certificate, AFAICT.

Yes you are right.
Regards
Sören

I didn't delete here anything. But I've deleted the acme-plugin and installed it again

A plugin reinstall does not trigger a re-import of the certificate.
I'm sorry, I have no explanation for the second certificate import :(
I'd suggest to delete the second certificate from the Certificate Manager, update to 17.7.2 (later this week) and watch if it happens again on next due/renewal date.

I also switch from production to stable and back.

Where did you change this setting? Do you mean the Let's Encrypt "Staging" vs. "Production" environment?

I'm closing this now. Feel free to comment or reopen this issue if it happens again or if you can provide further information.


ChatGPT integration Feature

Along with rich user interface in the system,allow chatgpt inclusion in creating description would ease the task of the user.

Will be taken in Roadmap 3


Compatibility issue with Dyn365 9.1 Update 1.3 - on-premises

Describe the bug
Latest (Dynamics365WorkflowTools_1_0_61_1_managed.zip) package cannot be imported in Dynamics 365 9.1 - Update 1.3
Expected behavior and actual behavior.
Expected behavior
Solution is successfully imported.
Actual behavior
Solution is not imported. See below error details.
Steps to reproduce the problem (screenshots are really usefull)
Steps to reproduce the behavior:

Download latest released package - Dynamics365WorkflowTools_1_0_61_1_managed.zip
Try to import solution in Dynamics 365
Error is displayed

CRM Version - online/onpremise - WorkflowTools version
Dynamics 365 On-premise:

Same issue here!
Any plans to release it for on-premise support?

Same error with Version 2104 (<IP_ADDRESS>) (DB <IP_ADDRESS>) on-premises.
Does the latest workflow tools support v9.1?

Same issue as those above.  I could really use this to use workflows with multiselect option sets.

Same issue. Cant install anything past 1.0.51.  which lacks the multiselect workflows.

Thank you for this note - this worked for me.

I edited the first line of the Solution.xml file to my version and I was able to import it and use it with no issue. Replace the first line with this: ImportExportXml version="<IP_ADDRESS>145" SolutionPackageVersion="9.1" languagecode="3082" generatedBy="Crm" xmlns:xsi=http://www.w3.org/2001/XMLSchema-instance


[DRAFT] Add in support for duplicate type names across modules

This PR solves the problem from #62 where types with the same name are blocked even if they are in different Rust modules. Currently this code is not production ready, but  I'll keep this PR up for reference and for when I get the time to productionize this code.
Currently this PR adds a new ModuleExportBehavior struct, that when enabled will append the module path of a struct to the beginning of its name. Take the following example:
mod app {
    mod protobufs {
        struct Waypoint { ... }
    }
}

The Waypoint struct would then be named app_protobufs_Waypoint within the generated code, allowing for duplicate type names.
Note: Tests are currently not passing

For personal reference. Emit types into dedicated files based on module or into ts namespaces within the same file, configurable by user.


Some fine tuning of the seasonizing experience

[ ]  Indicate in the dropdown which seasons will have interesting results or just straight elide the ones that will not (necro archetypes in S1, Mother of Runes card detail page in S7, etc.)
[x] Create a good experience at all of /seasons/, /seasons/<season_id>/, /seasons/<season_id><code>/decks/ and (if necessary) /seasons/<season_id>/decks/all/ and don't load all decks from a season until the point where someone has clearly asked us to because that's never going to be fast. Currently the relationship between /season/7/ and /season/7/decks/ is confused - clarify.
[ ] /seasons/0/, /seasons/-1/, /seasons/9/ should all 404 not produce empty decklist tables.
[x] /rotation/changes/ should have saner behavior for 'all time'

Mostly decided to not do these things but tidied up a little bit. That we can make these pages clearer, more interesting and faster is not a secret to anyone but not holding a task open for that.


ListHostedZones not found in .Net Core

The latest version of AmazonRoute53Client(); is different when comparing the .Net Standard to .Net Core.
For example:
        public ListHostedZonesResponse ListHostedZones(ListHostedZonesRequest request);
Is not available in .Net Core.

Are you looking for ListHostedZonesAsync()?  Also, the binaries always targeted netstandard1.3.  Did you mean to say .net45 vs .net core?

I am closing this.  Feel free to comment on this issue if you have unresolved problems.


Call on undefined (extract-text-webpack-plugin). Chain for .scss compilation

Laravel Mix Version: 5.0.4
Node Version (8.16.2):
NPM Version (6.14.4):
MacOS Catalina

Description:
Guys does anyone had same issue? Can't compile SCSS while hot server is running.
Some of the loaders gives wrong output I guess.
Only happens with hot command.
Steps To Reproduce:
const mix = require('laravel-mix');
const glob = require('glob');
const path = require('path');

const ENTRIES_DIR = './assets/modules';
const DIST_DIR = './public';

if (mix.inProduction()) {
    // Enabling source maps with default settings
    mix.sourceMaps();

    // Versioning in filename instead of query parameter
    require('laravel-mix-versionhash');
    mix.versionHash(); // IMPORTANT: Doesn't work with BrowserSync

    // Concatenates all used node_modules to vendor.js
    mix.extract();
} else {
    mix.webpackConfig({ devtool: 'inline-source-map' })
}

// Helpers
const getEntryName = fileName => path.basename(path.dirname(path.dirname(fileName)));
const makeSass = fileName => mix.sass(fileName, `${DIST_DIR}/css/${getEntryName(fileName)}.css`);
const makeJs = fileName => mix.js(fileName, `${DIST_DIR}/js/${getEntryName(fileName)}.js`);

// Processing
glob.sync(`${ENTRIES_DIR}/*/styles/index.s*ss`).forEach(makeSass);
glob.sync(`${ENTRIES_DIR}/*/scripts/index.js`).forEach(makeJs);

Console error:

Yes having the same issue but not sure how to override mix so that it doesn't use the extract-text-webpack-plugin. I believe this is related https://github.com/webpack/webpack/issues/6094#issuecomment-618235590

@orckid-lab thanks for your answer. I've already switched to webpack-encore


TwitchLink reinstalling itself

Twice TwitchLink opened as if I hadn't installed it, resulting on all my configurations being gone.

It seems like you are running the installer instead of the program itself.
Please make sure to run the program that was installed through the installer.

I's running into the same problem with the desktop shortcut. It popped up a window with a big "lunch" button and everything's gone like a newly installed after that.


remove/replace references to IMPORT

DOC-10549
DOC-10065

Removed IMPORT and cockroach import docs, links to those docs, and stale includes that concerned those docs
Added redirects from those docs to IMPORT INTO doc
Updated all examples (mostly in spatial data) to use alternatives to IMPORT (i.e., IMPORT INTO or piping a given dump file directly into cockroach sql)
These apply to 24.1 docs.

@Jeremyyang920 Can I ask your review on these commits from a general migrations perspective, keeping in mind that these examples are too small and specialized to use the MOLT tooling (i.e., no source database)? These were updated based on the GeoJSON solution provided by @michae2 in https://cockroachdb.zendesk.com/agent/tickets/21036.

https://github.com/cockroachdb/docs/pull/18674/commits/12e16757f8d7acc5a16a318d6804db0dfca43d4e
https://github.com/cockroachdb/docs/pull/18674/commits/45d085aeb71aa4418b400b7074ed122f2adbac32
https://github.com/cockroachdb/docs/pull/18674/commits/aabc4b2c98778610f2c3e22c5840022fd1cb8dbe

There's also an incremental update to the Oracle migration tutorial to replace the IMPORT references with IMPORT INTO. These only affected certain steps, but there's still a callout at the top of the page that hopefully makes it clear that the entire tutorial is generally outdated (too manual and no usage of MOLT).

https://github.com/cockroachdb/docs/pull/18674/commits/4d8299a9534ea4c801dd46cd7eff0496580ba0bb

@taroface Those 3 commits look good. We can update once MOLT supports geospacial types properly.

TFTR!!!!


Changing indent doesn't invalidate cache

The PHP version you are using ($ php -v):
=> 7.3.2
PHP CS Fixer version you are using ($ php-cs-fixer -V):
=> 2.14.0 Sunrise
Changing the indent style by changing (or initially setting) the value passed to setIndent() doesn't invalidate the cache, thus requiring either bypassing the cache or changing one of the rules to be able to get the indentation change to apply.

Thank for reporting,
This seems like a bug to me, when we calculate the cache signature we should also take these into account:

getIndent
getLineEnding

(i.e.PhpCsFixer\ WhitespacesFixerConfig)


feat: use environmental variable for pubsub

Description
after graphops/graphcast-sdk#96 closes
Issue link (if applicable)
update following graphops/graphcast-sdk#94

have to re-run tests after the sdk gets merged because of POI's cargo.toml dependency :')


Hello tls

Add a secure example to clarify how to use TLS with gRPC in C++ (greeter_server_ssl and greeter_client_ssl).
@veblush

I signed it


fixes for broken daily  build

fixes for broken daily  build

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

Needs approval from an approver in each of these files:

release/OWNERS

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

@rkpagadala: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
1a31e95be97cc073314262f747e88363377edc63
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Outline doesn't expand matching nodes when filtering on type

Testing #69299

enable filtering on type in the outline
collapse all nodes using collapse all
type a symbol that you know exists that but is collapsed

-> the parent node is not expanded
In this example the symbol type exists, but it is not expanded:

@joaomoreno is that something the filter controls? how do I achieve this? would I know when filtering happens/happened so that I can call expand all?

I have pushed a change that restores the old behaviour when filtering but for "type to find" more is needed. Moving to March to the remainder...

This is now fixed.


Add comment in the function and fix linting issue

FEATURE REQUEST?

What you expected to happen:
Add comments in the file and fix linting issue
Reference:- https://github.com/openebs/openebs/blob/master/e2e/ansible/files/bootstrapper.py
Anything else we need to know?:

I'll work on this.

Is this issue still open ?


[Feature][MOD-128] Update email notifications for new submissions

Purpose
Don't add the confirmation email to the Notification Digest, just send it. Keep the status updates and comments the same.
Why:
Users shouldn't be able to unsubscribe from confirmation emails and they should receive the email instantly.
Changes

Add new separate signal with handler for new preprint submissions
Update existing signal/code
Update the test

Side effects
Ticket
https://openscience.atlassian.net/browse/MOD-128

@sheriefvt Could you resolve the conflicts?

@aaxelb done 🔧


Bump workflow actions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.00%. Comparing base (b95028c) to head (b69ceb0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #66   +/-   ##
=======================================
  Coverage   74.00%   74.00%
=======================================
  Files           7        7
  Lines         808      808
=======================================
  Hits          598      598
  Misses        210      210

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


How to use latest XP toolset v141_xp?

How to use latest XP toolset v141_xp?

Which image are you using?
Looks like Windows XP support is pre-installed only in windows-2019, while for windows-2022 (= windows-latest) you need to install the Visual Studio component manually, like here: https://github.com/actions/runner-images/issues/6067#issuecomment-1213069040
The interwebs are also warning that you might need to get ahold of the old C++ runtime libs that still have Windows XP support.


Assertion 'mem_pools == NULL' failed in mem_pools_finalize

Jerry version:
Checked revision: 58851bc
Build: debug.linux

OS:
Ubuntu 15.04, x86_64

Test case:
JSON.stringify([], [ 'foo', 'foo' ]);

Note:
Although the assertion and the backtrace look similar to the issue will be fixed by #542 but this problem seems unrelated to that one.
Backtrace:
ICE: Assertion 'mem_pools == NULL' failed at jerryscript/jerry-core/mem/mem-poolman.cpp(mem_pools_finalize):89.
Error: ERR_FAILED_INTERNAL_ASSERTION

Program received signal SIGABRT, Aborted.
0x00000000004a8443 in syscall_2_asm () at jerryscript/jerry-libc/target/linux/jerry-asm.S:32
32    SYSCALL_2
(gdb) bt
# 0  0x00000000004a8443 in syscall_2_asm () at jerryscript/jerry-libc/target/linux/jerry-asm.S:32
# 1  0x0000000000400847 in syscall_2 (syscall_no=62, arg1=8145, arg2=6) at jerryscript/jerry-libc/target/linux/jerry-libc-target.c:96
# 2  0x00000000004009e8 in abort () at jerryscript/jerry-libc/target/linux/jerry-libc-target.c:175
# 3  0x0000000000483ac5 in jerry_fatal (code=ERR_FAILED_INTERNAL_ASSERTION) at jerryscript/jerry-core/jrt/jrt-fatals.cpp:65
# 4  0x0000000000483b14 in jerry_assert_fail (assertion=0x4cb3f3 "mem_pools == NULL", file=0x4cb3b0 "jerryscript/jerry-core/mem/mem-poolman.cpp",
    function=0x4a9450 <_ZZ18mem_pools_finalizevE8__func__.lto_priv.1071> "mem_pools_finalize", line=89) at jerryscript/jerry-core/jrt/jrt-fatals.cpp:92
# 5  0x00000000004a5827 in mem_pools_finalize () at jerryscript/jerry-core/mem/mem-poolman.cpp:89
# 6  0x00000000004a4265 in mem_finalize (is_show_mem_stats=false) at jerryscript/jerry-core/mem/mem-allocator.cpp:58
# 7  0x00000000004a2661 in jerry_cleanup () at jerryscript/jerry-core/jerry.cpp:1269
# 8  0x00000000004a137b in main (argc=3, argv=0x7fffffffdd08) at jerryscript/main-linux.cpp:299

PR: #575

This issue is solved by #575.


Alt - Tab / Cmd Tab Color Tag

When working with multiple projects and multiple windows are open in different screens, it is hard to find the target window when making alt - tab.
As of now one possible solution is using an extension like peacock but this time, when the window is active the color is becoming distractive.
A possible solution can be to have a small color tag, which is to be seen only when the window is being seen in alt - tab thumbnails. An alternative method can be the color tag to be active when the window does not have a focus.
The color tag can be placed anywhere on the window as long as it is visible when the alt - tab is showing a thumbnail.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


[Tree] Fix positioning for rootless trees

Here's the same fix for the master branch. It required a small adjustment concerning siblings but that was it. Thanks for merging!

thanks ;)


Add more functions to Interpreter to handle deeper Monad Transformer stacks

Current coverage is 82.54%

Merging #1054 into master will increase coverage by +0.14% as of d82120e

@@            master   #1054   diff @@
======================================
  Files          145     145
  Stmts         9363    9353    -10
  Branches       297     300     +3
  Methods          0       0
======================================
+ Hit           7716    7720     +4
  Partial          0       0
+ Missed        1647    1633    -14

Review entire Coverage Diff as of d82120e

Powered by Codecov. Updated on successful CI builds.

:+1:


Added OSGi dynamic import

Added definition to OSGi descriptor to let the MapDB bundle load the classes from other bundles. This is necessary when a bundle uses loadClass and want  to laod class definition exposed by other bundle, not included at compile time.

Agree the proper way of fixing this is adding support to specify a classloader.
However TCCL is not a workaround in many cases. @njbartlett wrote this up here: http://njbartlett.name/2010/08/30/osgi-readiness-loading-classes.html
Therefore I think in many cases we can only do a DynamicImport-Package: * right now.

@gravelld I can not reopen this issue. Could you please create a new bug for MapDB, with a few points what should be tested in 2.0?


Reinstate windows icon

Had to remove the windows app icon in #20 because of electron/rcedit#56.
We need to find a way of reinstating it.

@jacobtomlinson I would like to work on this issue. Will adding the same line again result in some issue?

Thanks @AdityaSrivast. Honestly I'm not sure, it needs investigating.

I am on it

I can work on this @jacobtomlinson but can you explain more about it, as I am a bit new here


Create Downloads page

https://www.dgidb.org/downloads

upping estimate to account for development and design time


Add README entry for GRIST_INCLUDE_CUSTOM_SCRIPT_URL variable

Context
We were looking for a way to inject custom scripts to the app's html templates, and thought grist had no such feature yet.
Ref issue https://github.com/gristlabs/grist-core/issues/904
Content
Turns out, the feature was implemented very recently (see here) but was not yet documented.
We could mention the related env variable in the README for discoverability.

Some tests are failing but I would assume due to some flakiness? Will put the PR back in draft until I get a green CI and retry

@hihuz Yes, you may open your PR directly. In order to get the review quickly from Gristlabs, you also may change the title to preprend with Tiny:.

Thanks for the tips @fflorent 😉

@hihuz also pro-tip: you may add a line beginning with Solves  / Fixes  followed by the issue number.
This will automatically close the associated issue when the PR is merged.
Example here: https://github.com/gristlabs/grist-core/pull/903


Enable swap file on Azure Linux

Fixes https://github.com/conda-forge/dgl-feedstock/issues/3
Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #3.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

@conda-forge-admin, please re-render

Looks like pip check failed
+ pip check
filelock 3.12.3 has requirement typing-extensions>=4.7.1; python_version < "3.11", but you have typing-extensions 4.5.0.

Weird since https://github.com/conda-forge/filelock-feedstock/blob/main/recipe/meta.yaml doesn't list typing-extensions

Probably pulled in by something else. Could also just disable pip check with a comment about this issue

Thanks Mike! 🙏


2.1.0 krb

@doktoric plz

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/101/

could you please rewrite you commit messages with jira id? (The pattern is CLOUD-)

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/102/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/103/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/104/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/105/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/106/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/107/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/108/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/ambari-rest-client-pull-request/109/


Juniper SRX plugin: Improperly parsed untagged VLAN when provisioning a guest network

ISSUE TYPE

Bug Report

COMPONENT NAME

Juniper SRX plugin

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

N/A
OS / ENVIRONMENT

N/A
SUMMARY

This is a direct follow-up to the following issue: https://issues.apache.org/jira/browse/CLOUDSTACK-9996
Previously addressed by this commit:
https://github.com/apache/cloudstack/commit/a8c09f2a1980757eb7a9da4a178dc27da08df2ed
A similar issue arises whenever Cloudstack tries to provision a Guest network; The following check fails because of the equals instead of contains:
https://github.com/apache/cloudstack/blob/main/plugins/network-elements/juniper srx/src/main/java/com/cloud/network/resource/JuniperSrxResource.java#L703
STEPS TO REPRODUCE

1. Configure an SRX network element. Choose any interfaces, it doesn't matter for this issue
2. Provision a public IP range inside your public network; make sure that you don't specify any VLANs (so that it becomes vlan://untagged)
3. Create a network offering for isolated networks with the following providers:
- DNS,DHCP,UserData: VR
- Firewall,SourceNAT,StaticNAT,PortForwarding: Juniper SRX
4. Create a guest network that uses the above network offering.
5. Try to restart the guest network; this will force the initial SRX configuration.

If you manually update the VLAN inside cloud.vlan database to be "untagged" instead of "vlan://untagged"; the above steps achieve expected results.

EXPECTED RESULTS

The guest network restart completes successfully

ACTUAL RESULTS

The guest network restart fails. DEBUG logs show the following:

2022-05-11 14:06:44,472 ERROR [c.c.n.r.JuniperSrxResource] (DirectAgent-2:ctx-f5e76969) (logid:b540c9d9) com.cloud.utils.exception.ExecutionException: Could not parse public VLAN tag: vlan://untagged

We don't have any SRX infra in the community CI/CD systems to verify/fix this, any pull request is welcome @vdombrovski

@vdombrovski As the iControl jar for juniper appliances is outdated we have removed the SRX plugin. If we reintroduce it we first need to find a replacement jar.

@DaanHoogland sounds good, I saw your PR #7023. Are you planning to delete this from the official documentation for the upcoming release as well?

Are you planning to delete this from the official documentation for the upcoming release as well?

good point didn't think of that.

created https://github.com/apache/cloudstack/issues/7690 for this

Are you planning to delete this from the official documentation for the upcoming release as well?

good point didn't think of that.

Awesome, I'll close this issue, as it is no longer in scope.


Update to Google Play Billing Library v3.0.3

It contains some bug fixes compared to 3.0.0
Google is forcing apps to update from version 2 by november 2021. Would be nice to have the latest version with bugfixes

Google now even released v4.0.0 - would be great if the bindings could get updated!

Hi @moljac
Are there any plans to upgrade soon?
Many thanks 🙏


[FLINK-5183] [py] Support mulitple jobs per plan file

Modifies the Python API to support running multiple jobs in one Python plan file.

Thanks for the comments, I've addressed them with the latest commit.

Yes, please continue to comment after pushing changes :)
I'll have to try this out on a cluster to be certain, but from the looks of it this is good to merge.

In the process of getting a more complex job to run, I've tested a basic multi-job file (seen below) on an Amazon EMR YARN cluster and it works successfully.
from flink.plan.Environment import get_environment
from flink.plan.Constants import INT, STRING, WriteMode
from flink.functions.GroupReduceFunction \
    import GroupReduceFunction
from flink.functions.Aggregation import Sum

import sys

if __name__ == "__main__":
    output_file = 'hdfs:/tmp/out.txt'
    output_file2 = 'hdfs:/tmp/out2.txt'

    env = get_environment()
    data = env.from_elements((0,1),(1,2),(2,3),(2,10))

    data \
        .group_by(0) \
        .aggregate(Sum, 1) \
        .write_csv(output_file, write_mode=WriteMode.OVERWRITE)
    env.execute()

    env2 = get_environment()
    data2 = env2.read_csv(output_file, (INT, INT))

    data2 \
        .first(2) \
        .write_text(output_file2, write_mode=WriteMode.OVERWRITE)

    env2.execute()

It may have worked with a smaller file, but there may be issues with heavier jobs. When I ran a more computationally intensive and time consuming job, the first job of the Python file ran successfully. The second job of the file was then submitted:
<snip>
02/09/2017 16:39:43	DataSink (CsvSink)(4/5) switched to FINISHED
02/09/2017 16:39:43	Job execution switched to status FINISHED.
2017-02-09 16:40:26,470 INFO  org.apache.flink.yarn.YarnClusterClient                       - Waiting until all TaskManagers have connected
Waiting until all TaskManagers have connected
2017-02-09 16:40:26,476 INFO  org.apache.flink.yarn.YarnClusterClient                       - TaskManager status (5/5)
TaskManager status (5/5)
2017-02-09 16:40:26,476 INFO  org.apache.flink.yarn.YarnClusterClient                       - All TaskManagers are connected
All TaskManagers are connected
2017-02-09 16:40:26,480 INFO  org.apache.flink.yarn.YarnClusterClient                       - Submitting job with JobID: b226f5f18a78bc386bd1b1b6d30515ea. Waiting for job completion.
Submitting job with JobID: b226f5f18a78bc386bd1b1b6d30515ea. Waiting for job completion.
Connected to JobManager at Actor[akka.tcp://flink@<snip>.ec2.internal:35598/user/jobmanager#68430682]

However, Flink does not receive or respond to this new job. Instead, the client terminates with a timeout error:
Caused by: org.apache.flink.runtime.client.JobClientActorSubmissionTimeoutException: Job submission to the JobManager timed out. You may increase 'akka.client.timeout' in case the JobManager needs more time to configure and confirm the job submission.
	at org.apache.flink.runtime.client.JobSubmissionClientActor.handleCustomMessage(JobSubmissionClientActor.java:119)
	at org.apache.flink.runtime.client.JobClientActor.handleMessage(JobClientActor.java:239)
	at org.apache.flink.runtime.akka.FlinkUntypedActor.handleLeaderSessionID(FlinkUntypedActor.java:88)
	at org.apache.flink.runtime.akka.FlinkUntypedActor.onReceive(FlinkUntypedActor.java:68)
	at akka.actor.UntypedActor$$anonfun$receive$1.applyOrElse(UntypedActor.scala:167)

I tried setting akka.client.timeout to 20 minutes, but Flink is still not receiving the second job. I suspect this may be an issue with this patch.

It may have worked with a smaller file, but there may be issues with heavier jobs.

How silly of me. This problem had nothing to do with this pull request, with YARN, with issues in Flink, or with the size of the input file at all. I was using ExecutionEnvironment.from_elements to generate a large sequence of indexed zeroes to fill in the gaps of another indexed DataSet with zeroes. However, when I was using large input files, I set larger parameters and generated larger zero sequences. Because I was using from_elements, the client needed to send all of those values (lots and lots of zeroes) to the runtime, which was very time-consuming and caused the timeout. I have replaced this with a generate_sequence call and a map function, which does not require sending lots and lots of values from the client to the runtime, and the job (and this pull request) seem to work just fine.
(change in question: https://github.com/quinngroup/pyflink-r1dl/commit/00a16d564bfad21fc1f4958677ada0a95fa9f088)

Resolved merge conflicts

I will now try this out and merge it if it workds.