how do i get from the operation replaceAsForeignKey to is.linkedEntity.identifier in a document

i am trying to add relationships to my documents and i am following these docs
https://learn.microsoft.com/en-us/common-data-model/sdk/projections/replaceasforeignkey
the entity1.cdm.json file now looks like this. how do i ex
 {
                    "name": "attr3",
                    "entity": {
                        "source": "entity2",
                        "operations": [
                            {
                                "$type": "replaceAsForeignKey",
                                "condition": "referenceOnly",
                                "reference": "attr2",
                                "replaceWith": {
                                    "dataType": "entityId",
                                    "name": "FK_address"
                                }
                            }
                        ],
                        "runSequentially": true
                    }
                },

how do i get the is.linkedEntity.identifier trait added?
i also tried running this example
https://github.com/microsoft/CDM/blob/6adbae0554f5cca0d6796558a85466f72d28b947/samples/8-logical-manipulation-using-projections/code-ts/index.ts
but the trait is not added there either or did i misunderstood the functionality?
additionally if call,
await this.corpus.fetchOutgoingRelationships(entity);

no relationship is added returned even if i manually add the trait

hello @dgcaron,
You're schema now is in what we call "Logical" form. In logical form you can have references to another entities that are added as relationships once you resolve them. Resolution is the process in which the CDM SDK transforms you logical definitions into resolved/physical definitions.
In this line we save the resolved document into disk, you can take a look there :)

ok, i get the difference between logical and resolved now. so i need to save the logical form first and then resolve it using the methods in the example. two things:

the example doesn't save the manifest, i need to resolve this too with same options?
the is.linkedEntity.identifier isn't added by the example

{
                        "name": "address",
                        "type": "attributeDefinition",
                        "parent": "referenceOnly_Person/attributeContext/referenceOnly_Person",
                        "definition": "resolvedFrom/Person/hasAttributes/address",
                        "contents": [
                            {
                                "name": "projection",
                                "type": "projection",
                                "parent": "referenceOnly_Person/attributeContext/referenceOnly_Person/address",
                                "definition": "resolvedFrom/Person/hasAttributes/address/entity/projection",
                                "contents": [
                                    {
                                        "name": "source",
                                        "type": "source",
                                        "parent": "referenceOnly_Person/attributeContext/referenceOnly_Person/address/projection",
                                        "contents": [
                                            {
                                                "name": "Address",
                                                "type": "entity",
                                                "parent": "referenceOnly_Person/attributeContext/referenceOnly_Person/address/projection/source",
                                                "definition": "resolvedFrom/Address"
                                            }
                                        ]
                                    }
                                ]
                            },
                            {
                                "name": "_generatedAttributeSet",
                                "type": "generatedSet",
                                "parent": "referenceOnly_Person/attributeContext/referenceOnly_Person/address",
                                "contents": [
                                    {
                                        "name": "operation/index1/operationReplaceAsForeignKey",
                                        "type": "operationReplaceAsForeignKey",
                                        "parent": "referenceOnly_Person/attributeContext/referenceOnly_Person/address/_generatedAttributeSet",
                                        "contents": [
                                            {
                                                "name": "_foreignKey",
                                                "type": "addedAttributeIdentity",
                                                "parent": "referenceOnly_Person/attributeContext/referenceOnly_Person/address/_generatedAttributeSet/operation/index1/operationReplaceAsForeignKey",
                                                "contents": [
                                                    "referenceOnly_Person/hasAttributes/addressFK"
                                                ]
                                            }
                                        ]
                                    }
                                ]
                            }
                        ]
                    },

is that the expected behaviour?

ok, so instead of resolving individual entities resolving the manifest actually yielded the expected result
const resolvedFolder = await this.corpus.fetchObjectAsync<cdm.types.CdmFolderDefinition>("local:/resolved/");

var manifestName = "default.manifest.cdm.json";
var options: cdm.types.copyOptions = {};
var resolvedManifest = await this.manifest.createResolvedManifestAsync(
      manifestName,
      "resolved/{n}.cdm.json",
      undefined
);

resolvedFolder.documents?.push(resolvedManifest);
var saved = await resolvedManifest.saveAsAsync(manifestName, true, options);

the attribute now looks like this
 {
  "name": "FK_attr1",
  "appliedTraits": [
    "is.dataFormat.character",
    "is.dataFormat.big",
    "is.dataFormat.array",
    "is.dataFormat.guid",
    "means.identity.entityId",
    {
      "traitReference": "is.linkedEntity.identifier",
      "arguments": [
        {
          "entityReference": {
            "entityShape": "entitySet",
            "constantValues": [
              ["/entity2.cdm.json/entity2", "attr1", "attr3_entity2"]
            ]
          }
        }
      ]
    }
  ],
  "attributeContext": "entity1/attributeContext/entity1/attr3/_generatedAttributeSet/operation/index1/operationReplaceAsForeignKey/_foreignKey",
  "dataFormat": "Guid"
}

what i don't fully understand is that it defaults to a GUID. the field being referenced is a int64.. do i need to set this explicitly somewhere?

the issue that remains is that when i read the resolved manifest the attributes and relationships remain empty, is there an additional call i need to do to construct theses values?

Hi, is the problem you saved the resolved manifest and the file looks good but when reading it, it has empty relationship and attributes? Can you post the code how you save (if it's not the same as above code) and read the manifest?

yes, that is correct.  i store the files using the createResolvedManifestAsync method as described above.
this is how i read the resolved manifest (simplified):
var objectPath = "default.manifest.cdm.json";
var manifest = await this.corpus.fetchObjectAsync<cdm.types.CdmManifestDefinition>(`local:/${objectPath}`);

for await (const entity of manifest?.entities.allItems.map((e) => .corpus.fetchObjectAsync<cdm.types.CdmEntityDefinition>(e.objectPath, manifest))) {
// use the entity
// the entity has a name and such but when i want to check the attributes or relationships they remain empty
}

It seems that the entity fetched is not properly defined. There are several things worth checking.

the resolved manifest file

please check if the resolved manifest's entities list has the entityPath defined properly, it should point to the resolved entity.

the resolved entity file

please check the resolved entity file has attributes.

You can also try modify the file manually and use fetchObjectAsync to check whether it works properly or not. From the error you get, it is most likely to be the file itself doesn't have attributes.

i have tried what you suggested and i might be overlooking something here.. i have attached the manifest and the (very) simple example json files
resolved.cdm.zip
maybe someone can spot the problem here?

ok, the issue with the attributes was caused by calling
corpus.fetchObjectAsync<cdm.types.CdmEntityDefinition>(e.atCorpusPath, manifest)))

instead of
// use the entityPath and not the corpuspath to fetch the entity
corpus.fetchObjectAsync<cdm.types.CdmEntityDefinition>(e.entityPath, manifest)))

so i can read the resolved entities now, however the relationships remain empty it that expected?

Hi @dgcaron,
When you create the resolved manifest that should make the correct calls to put the relationships in that newly created resolved manifest.
You have some other code where you call CalculateEntityGraphAsync, if that is done on the resolved manifest, it should not be necessary as the resolved manifest should have already contained this information. If you are doing this on the unresolved manifest, you will also need to call the api to add the relationships to the manifest like this:
// generate the relationship information
await corpus.calculateEntityGraphAsync(manifest);
// add the relationship information to the manfiest
await manifest.PopulateManifestRelationshipsAsync(CdmRelationshipDiscoveryStyle.Exclusive);

Again, these apis are called inside of CreateResolvedManifestAsync so the resolved manifest that is created should already contain the relationships. If the relationships still aren't created, then there may be an issue with the logical schema files. If that is the case, can you share the unresolved schema files? The zip you shared before only contained the resolved entity files

logical-replaceasfk.cdm.zip
Yes, Sure! thank you for looking into this

It looks like there may be an issue with generating relationships on resolved manifests and entities when using projections (such as calling CreateResolvedManifestAsync, I am still investigating that. Looking at the files you sent, this code should work for generating relationships on the unresolved entity/manifest:
// generates the relationships from entities in the manifest
await corpus.CalculateEntityGraphAsync(manifest);
// add any relationships that are relevant to the entities found in this manifest
await manifest.PopulateManifestRelationshipsAsync();

Hi! Thank you for taking the time to look into this. I'll give these pointers a go and let you know what the result is.

Closing this issue, please open a new one if there is any new issue.


CA-212659 : Break link to xenserver version / remove tools hotfix number

Can one of the admins verify this patch?

Can one of the admins verify this patch?

add to whitelist

retest this please


Bonfire: Mutations - wrong test case

Challenge Bonfire: Mutations has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function sortString(str){
  return str.toLowerCase().split('').sort().join('');
}

function countSorted(str,char){
  var c = 0;
  var pos = str.indexOf(char);
  while (str[pos++] === char) c++;
  return c;
}

function mutation(arr) {
  var lastChar = '';
  var s1 = sortString(arr[0]);
  var s2 = sortString(arr[1]);
  var l2 = s2.length;
  var res = true;
  for (var i=0; i < l2; i++){
    if (s2[i] === lastChar) continue;
    lastChar = s2[i];
    if (countSorted(s1,lastChar) < countSorted(s2,lastChar)){
      res = false;
      break;
    }
  }
  return res;
}

mutation(["Mary", "Aarmy"])

The testcase is mutation(["Mary", "Aarmy"]), the system says it should return true because it doesn't care about only one 'a' in the first string, but i think it should return false due to the problem description.

Return true if the string in the first element of the array contains all of the letters of the string in the second element of the array.

It says "all of the letters of the string", it doesn't say anything about quantity, just presence.  This is reinforced by all of the test cases which support that notion.  This is not an error, this is the definition.
You may want to try a solution that uses indexOf to search one string for the presence of another.  Your current solution does not conform well to the problem as described.
Happy Coding!


[Enum Request] For a field of Table 85 ("Acc. Schedule Line")

Hello ALAppExtensions-team,
Could It be possible to make option field "Totaling Type" in Table 85 ("Acc. Schedule Line") extensible using enums?
Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


PDF doesn't show

Nice website man. It's really amazing what is possible. Small issue: when clicking on "PDF" in the start menu, the viewer just appear to hang with a blue rotating cursor. I've tried both Firefox and Edge.

Thanks for mentioning this! I had indeed logged this but forgot to deal with it sooner as it's quite confusing. It looks to be loading but just never leaves that state because I forgot to unset it when a file isn't selected for the PDF viewer.

Fixed https://github.com/DustinBrett/daedalOS/commit/c601ce2a291dbc73dc39ed596bdb5f574bd54e9e


Cloud Pub/Sub example could result in message loss

In which file did you encounter the issue?
java-docs-samples/pubsub/cloud-client/src/main/java/com/example/pubsub/SubscriberExample.java

Did you change the file?
No.
Describe the issue
The example presented does not exhibit best practices for using Cloud Pub/Sub. The message is acknowledged as soon as it is added to the BlockingQueue. However, since the queue is not persistent, a crash of the task would result in message loss for any messages added to the queue that were not yet processed. I understand that it is a simple example, but we should make sure we are demonstrating best practices for using Pub/Sub.

@anguillanneuf, do you know who would be the right person to take a look at this?

I took a stab at it in #1534.

Thank you @kamalaboulhosn so much for reporting this.

Closing now as the PR has been merged.
Message loss can be dealt by a blocking queue with nack logic and a processing timeout. One of Kamal's commits shows exactly how to do that in the PR.

@anguillanneuf You don't need a blocking queue to handle message loss. The only thing required to avoid message loss is that the ack needs to happen after all processing of the message is complete. In the example prior to this PR, ack was being called at a point that was potentially before the message had been processed after being added to the blocking queue.

Thanks for calling this out!


kaggle datasets list results in TypeError: '>=' not supported between instances of 'NoneType' and 'int'

I installed kaggle via
pip3 install kaggle

I've verified the version is 1.5.6.
When running kaggle datasets list, I get the following traceback.

Traceback (most recent call last):
File "/usr/local/bin/kaggle", line 10, in
sys.exit(main())
File "/usr/local/lib/python3.7/site-packages/kaggle/cli.py", line 51, in main
out = args.func(**command_args)
File "/usr/local/lib/python3.7/site-packages/kaggle/api/kaggle_api_extended.py", line 940, in dataset_list_cli
max_size, min_size)
File "/usr/local/lib/python3.7/site-packages/kaggle/api/kaggle_api_extended.py", line 905, in dataset_list
return [Dataset(d) for d in datasets_list_result]
File "/usr/local/lib/python3.7/site-packages/kaggle/api/kaggle_api_extended.py", line 905, in
return [Dataset(d) for d in datasets_list_result]
File "/usr/local/lib/python3.7/site-packages/kaggle/models/kaggle_models_extended.py", line 69, in init
self.size = File.get_size(self.totalBytes)
File "/usr/local/lib/python3.7/site-packages/kaggle/models/kaggle_models_extended.py", line 109, in get_size
while size >= 1024 and suffix_index < 4:
TypeError: '>=' not supported between instances of 'NoneType' and 'int'

The problem stems from a few refs which don't appear to have any data files. For example this one bigquery/crypto-litecoin.
I've traced the problem to https://github.com/Kaggle/kaggle-api/blob/84895cea61af708a24f0e8ad8307d570e82d8097/kaggle/models/kaggle_models_extended.py#L67
This issue is that it seems in these cases, self.totalBytes will be None since it isn't in the parsed_dict.
I've created a PR for this. I've made get_size treat a size that is None to have the value 0.
This results in output looking like:

bigquery/crypto-litecoin                                   Litecoin Crypto Blockchain                             0B  2019-02-14 20:22:27              0         25  0.6764706
bigquery/crypto-bitcoin-cash                               Bitcoin Cash Blockchain                                0B  2019-02-14 18:08:11              0         54  0.6764706
bigquery/crypto-dogecoin                                   Dogecoin Crypto Blockchain                             0B  2019-02-14 20:22:27              0         23  0.6764706
rajeevw/ufcdata                                            UFC-Fight historical data from 1993 to 2019           3MB  2019-07-05 09:58:02          13989        530  0.9705882
bigquery/crypto-ethereum-classic                           Ethereum Classic Blockchain                            0B  2019-03-20 23:21:25              0        112  0.7058824
dgomonov/new-york-city-airbnb-open-data                    New York City Airbnb Open Data                        2MB  2019-08-12 16:24:45          49718       1163  1.0

Note some others have this issue: https://stackoverflow.com/questions/59445928/kaggle-api-in-colab-datasets-kaggle-datasets-list-error/59559533#59559533

This worked! Thanks


Fix basic template example on Getting Started wiki page

The example on https://github.com/tdewolff/canvas/wiki/Getting-Started has a number of syntax and functional issues that prevent it from working. This is an updated, working example that can be slotted into the page:
package main

import (
	"github.com/tdewolff/canvas"
	"github.com/tdewolff/canvas/renderers"
)

func main() {
	// Create new canvas of dimension 100x100 mm
	c := canvas.New(100, 100)

	// Create a canvas context used to keep drawing state
	ctx := canvas.NewContext(c)

	// Create a triangle path from an SVG path and draw it to the canvas
	triangle, err := canvas.ParseSVGPath("L60 0L30 60z")
	if err != nil {
		panic(err)
	}
	ctx.SetFillColor(canvas.Mediumseagreen)
	ctx.DrawPath(0, 0, triangle)

	// Rasterize the canvas and write to a PNG file with 3.2 dots-per-mm (320x320 px)
	err = renderers.Write("getting-started.png", c, canvas.DPMM(3.2))
	if err != nil {
		panic(err)
	}
}

(I'd create a PR but that isn't possible for GitHub wikis)

Thanks David, I've updated the example. Shouldn't the coordinates of the DrawPath operation remain (20,20) though, lest it'd start in the corner?

Good catch @tdewolff, exactly right. I was using this example as a quick base for testing and left that change.

Changed that. If you find other holes in the documentation or examples I'd be happy to hear about it. I haven't had time to sift through all of that (or add appropriate unit tests for all code). The project has become somewhat overwhelming hehe.


Hotfix: update the logic to get path of access tokens.

Add logic to read path of access tokens through environment variable.
Add fallback logic to default one.

Notice: this is short-term fix and require user to use latest version of provider in Cloud Shell.
Thank you for your contribution to Go-AutoRest! We will triage and review it as soon as we can.
As part of submitting, please make sure you can make the following assertions:

[ ] I've tested my changes, adding unit tests if applicable.
[ ] I've added Apache 2.0 Headers to the top of any new source files.
[ ] I'm submitting this PR to the dev branch, except in the case of urgent bug fixes warranting their own release.
[x] If I'm targeting master, I've updated CHANGELOG.md to address the changes I'm making.

I could not find any legacy unit test for AccessTokensPath method. Let me know if we need to add it.

@marstr let me create an issue as TODO item for the unit test.

@marstr the PR is updated, when it's merged, please let me know. I will work with Tom to update the dependency on provider repo.


Arrows extension

Hi, I'm trying to use the arrow extension, but it's failing due to props not being defined:
from Arrows.js
if (props.sprite) {
            Object.assign(this.props.sprite, props.sprite)
}

How is this props set? and what should it be its value, is there a default one?
This is how I'm setting up the chessboard:
    const config = {
      sprite: {
        url: 'assets/chess/images/chessboard-sprite.svg',
      },
      extensions: [
         { class: Markers },
         { class: Arrows }
      ],
      responsive: true,
    };
    this.chessboard = new Chessboard(this.chessboardElement, config);

Markers are working fine, but no luck with arrows. Thank you for your support!

I use arrows but don't have a "sprite" in my config.

I tried removing the sprite from my config and it doens't work either. This:
    const config = {
      // sprite: {
      //   url: 'assets/chess/images/chessboard-sprite.svg',
      // },
      extensions: [
         { class: Markers },
         { class: Arrows }
      ],
      responsive: true,
    };
    this.chessboard = new Chessboard(this.chessboardElement, config);

breaks in the same way:

I think the error is in the code, that it does not checks if arrow's props is undefined. I'm trying to see how to pass an empty object to props so that it will pass

Yes, the following works:
    const config = {
      sprite: {
        url: 'assets/chess/images/chessboard-sprite.svg',
      },
      extensions: [
         { class: Markers },
         { class: Arrows, props: {
            sprite: {
                //url: "./assets/images/chessboard-arrows.svg",
                size: 40,
                cache: true
            },
         } }
      ],
      responsive: true,
    };
    this.chessboard = new Chessboard(this.chessboardElement, config);

Hi, because it is working in this example, with the following code, without Arrows properties,
const chessboard = new Chessboard(document.getElementById("board"), {
    position: "rn2k1r1/ppp1pp1p/3p2p1/5bn1/P7/2N2B2/1PPPPP2/2BNK1RR w Gkq - 4 11",
    assetsUrl: "../../assets/",
    extensions: [{class: Arrows}]
})

I think you just forgot to set the assetsUrl. This property is important because the assets, also of extensions, are first searched in this folder.

you are right, I´m not using assetsUrl. Thank you!

Things go fast hehe my version is from 2 months ago I think , will try to update


Renaming definitions from Bulk Jobs to Import Jobs

Data Plane API - Pull Request
Renaming some definition names and description to suit our naming pattern. API remains same.
API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your engagement record.

Link to engagement record issue:

Is this review for (select one):

[ ] a private preview
[x] a public preview
[ ] GA release

Change Scope
This PR involves renaming certain artifacts so that the generated docs reflect proper naming scheme.

Design Document:
Previous Open API Doc:
Updated paths:

❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

We haven't released the SDK or Docs from this swagger yet, so these are not breaking changes.


Bug -Could not find declaration file, when TS has no errors

src/config/search_options.ts:1:24 - error TS7016: Could not find a declaration file for module 'typesense'. '/Users/Stephano/Development/code/app/archimedes/node_modules/typesense/lib/Typesense.js' implicitly has an 'any' type.
  Try `npm install @types/typesense` if it exists or add a new declaration (.d.ts) file containing `declare module 'typesense';`

1 import { Client } from 'typesense';
                         ~~~~~~~~~~~

But TypeScript itself shows no errors as I have a declaration file:

ts-node-dev version:
"ts-node-dev": "^1.0.0-pre.63",

try --files option.

Got the same issue and the --files option indeed solves it.
@whitecolor What is the magic behind? Should not be better to have the same result as compiling typescript (which works without option)? :thinking:

No huge magic, check if it runs with ts-node.

Got the same issue and the --files option indeed solves it.

Another way of compiling is used then and tsc, read ts-node docs if interested. Or read this package docs more carefully.


Allow setting transport settings for ports

What does this PR do?

Allows setting various transport options on ports as defined in the docs.
Note: based on #1043 so that should be merged and then I hope there'll be no merge conflicts...
Motivation
Wanting to set one myself with an interface slightly more convenient then using the additionalArguments value.
More

[x] Yes, I updated the tests accordingly
[x] Yes, I ran make test and all the tests passed

As a side note: I personally created this as to set requestAcceptGraceTimeout to a non-zero value for all ports by default, given that with how the Kubernetes API handles endpoints, there might in practice exist some interval where the pod has started terminating but the endpoint has not been deregistered and/or this has not been processed by other consumers yet. In such a case, consumers may attempt to send requests to the terminating pod, causing failures.
I think it might be reasonable to set a default non-zero value in this chart by default; let me know if you agree!

Copied the values from the test to values.yaml as example.

Wdyt of making them appear in values, so third party tools are able to use it ?

Just tried it, but appearently my if (toString .X) check fails on nil/null, as it is converted to the string <nil> rather than empty string. I figured using just "if" is also not the way to go, as then numeric 0 is ignored (it's falsey).
So there's two choices here AFAICS:

Change the checks to if ne (quote .X) "\"\"", which looks even uglier to me, but does seem to be more accurate in result;
Don't support empty values, and use literal "" values in the examples, which are not passing the existing test.

So there's two choices here AFAICS:

I have a third choice to suggest:
With values like this:
    transport:
      respondingTimeouts:

should work with template code like this:
                {{- with .readTimeout }}
          - "--entryPoints.{{ $entrypoint }}.transport.respondingTimeouts.readTimeout={{ . }}"
                {{- end }}

The with operator would not allow passing 0 (numeric) as argument, as it's falsey, so e.g.
    transport:
      respondingTimeouts:
        readTimeout: 60
        writeTimeout: 0
        idleTimeout: 42s

results in
                                - --entryPoints.websecure.transport.respondingTimeouts.readTimeout=60
                                - --entryPoints.websecure.transport.respondingTimeouts.idleTimeout=42s

(with no writeTimeout argument printed).

\o you're right. Sorry, the correct template code should be:
                {{- with .readTimeout | toString }}
          - "--entryPoints.{{ $entrypoint }}.transport.respondingTimeouts.readTimeout={{ . }}"
                {{- end }}

Ah yeah, an explicit nil check works too. So 3 styles to pick from I guess:
1:
              {{- if ne .readTimeout nil }}
                {{- with .readTimeout | toString }}
          - "--entryPoints.{{ $entrypoint }}.transport.respondingTimeouts.readTimeout={{ . }}"
                {{- end }}
              {{- end }}

2:
                {{- if and (ne .readTimeout nil) (toString .readTimeout) }}
          - "--entryPoints.{{ $entrypoint }}.transport.respondingTimeouts.readTimeout={{ .readTimeout }}"
                {{- end }}

3:
                {{- if ne (quote .readTimeout) "\"\"" }}
          - "--entryPoints.{{ $entrypoint }}.transport.respondingTimeouts.readTimeout={{ .readTimeout }}"
                {{- end }}

Style 1. or 2. is fine. Choose which one you prefer.
Style 3. seems a door open to more not-yet-discover problems.


added new script get-spo-invalid-user-accounts

Checking existing user accounts in a Site Collection against disabled accounts in AAD or is no longer present in the UPA

Happy New Year @kasperbolarsen - I have processed the sample and will be live shortly 🎉😃 thank you again for another awesome sample.


How to get URL from firebase firestore

I'm trying to do this:
        imageList.add(SlideModel(FirebaseUtils().firestore
            .collection("emergencies")
            .document(emergenciesId)
            .collection("imageUrl")
            .get()
            .toString()))

It does not give any error, but it does not show the image on the ImageSlider. How do i resolve it? or is it impossible to do it? @denzcoskun

Firestore queries doesn't work like that i think .. Do the query first and confirm that you are getting what you want from firebase (in your case a string url im assuming), and in onSuccess set that value to a new slidemodel to your imagelist.

hi @herva2003 I think this tutorial can help you, based on this library : https://youtu.be/2GEVSqyAkIA


dev server: node_modules assets are not served from relative path if kit.paths.relative is true

Describe the bug
We have an app, where we now have to set the config option kit.paths.relative to true in the svelte.config.js. The base import from $app/paths works great, and after vite build everything is rendered as expected, too. But when running our app in dev-mode with vite dev, the SvelteKit app does not work for our use-case. The problem is that SvelteKit loads several resources directly from the node_modules folder with an absolute path:
                    Promise.all([import("/node_modules/@sveltejs/kit/src/runtime/client/start.js"), import("/@fs/Users/.../.svelte-kit/generated/client/app.js")]).then(([kit,app])=>{
                        kit.start(app, element, {
                            node_ids: [0, 3, 12],
                            data,
                            form: null,
                            error: null
                        });
                    }
                    );

This also happens, if you import for example fonts from a node_module.
If we are setting kit.paths.base then the base is prepended to the path. But for kit.paths.relative, the path is not relative.
Reproduction
Set kit.paths.relative to true in your svelte.config.js. Then open the page in the browser and look into the raw HTML that is served from the Vite dev-server (For the above example look at the last lines of the raw HTML).
Logs
No response
System Info
System:
    OS: macOS 13.6
    CPU: (10) arm64 Apple M2 Pro
    Memory: 73.97 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.0/bin/npm
  Browsers:
    Chrome: 117.0.5938.132
    Safari: 17.0
  npmPackages:
    @sveltejs/adapter-node: 1.3.1 => 1.3.1
    @sveltejs/kit: 1.25.1 => 1.25.1
    svelte: 3.59.2 => 3.59.2
    vite: 4.4.9 => 4.4.9

Severity
serious, but I can work around it
Additional Information
To "work around" this, currently, we have to use vite build --watch and restart vite preview after any change. This is an absolute pain.

Also looking for an update on this. Running the dev server behind a proxy.


Refactor the run script and pre-commit hooks to reduce duplication

This PR refactors the run script and pre-commit hooks make it easier to share functionality and reduce duplication. It is based on apologies PR #48.

Coverage decreased (-0.02%) to 98.1% when pulling 3c0befee07307ed411323d3077cd492ff91b41c0 on ken/run-refactor into 37bdbe4c27973a88704a043ccc393c271039dd82 on master.


HasColumnName in IEntityTypeConfiguration is ignored

.NET Core Version 2.2 on macos
EntityFrameworkCore.Design 2.1.8
EntityFrameworkCore.Postgresql 2.1.2
I recently observed a very strange behavior. I use IEntitiyTypeConfiguration<T> to configure the entities in the DbContext. I register them using modelBuilder.ApplyConfiguration(new StopTimeConfiguration); for example. When I use the Microsoft.EntityframeworkCore.RelationalPropertyBuilderExtensions.HasColumnName(string) extension method inside my configuration class it is ignored in the queries on the DbSet. But it works when I use the the same extension method inside the OnModelCreating(ModelBuilder)method.
My concrete example occurs when using the following table:
CREATE TABLE StopTimes (
  trip_id VARCHAR NOT NULL,
  arrival_time TIME NOT NULL,
  departure_time TIME NOT NULL,
  stop_id VARCHAR NOT NULL,
  stop_sequence INTEGER NOT NULL,
  stop_headsign VARCHAR,
  pickup_type SMALLINT,
  drop_off_type SMALLINT,
  shape_distance_traveled DOUBLE PRECISION,
  timepoint SMALLINT,
  next_day BOOLEAN,
  PRIMARY KEY (trip_id, stop_id, stop_sequence)
);

with this entity class:
public class StopTime {
        public string TripId { get; set; }
        public TimeSpan ArrivalTime { get; set; }
        public TimeSpan DepartureTime { get; set; }
        public string StopId { get; set; }
        public int StopSequence { get; set; }
        public string StopHeadsign { get; set; }
        public short PickupType { get; set; }
        public short DropOffType { get; set; }
        public double ShapeDistanceTraveled { get; set; }
        public bool IsNextDay { get; set; }
    }

When I now use:
builder.Property(x => x.IsNextDay).HasColumnName("next_day");

In an the Configure(EntityTypeBuilder<T> method is is completely ignored. Whereas I use it in the OnModelCreating(ModelBuilder) it works.
Queriying the DbSet using the first method results in this query:
SELECT st.trip_id, st.stop_id, st.stop_sequence, st.arrival_time, st.departure_time, st.drop_off_type, st.is_next_day, st.pickup_type, st.shape_distance_traveled, st.stop_headsign
      FROM stoptimes AS st

As you can see, it uses Entity Frameworks default naming convention for the column name. But the second way produces a query in which my configuration is reflected:
SELECT st.trip_id, st.stop_id, st.stop_sequence, st.arrival_time, st.departure_time, st.drop_off_type, st.next_day, st.pickup_type, st.shape_distance_traveled, st.stop_headsign
      FROM stoptimes AS st

I am missing something or should this work?

So I figured out that we have a default set of mappings which are applied in reflective code which is executed as the last part of Configure(). So the mapping works our setup overwrites my mapping. Never mind, sorry to disturb.


UI not rendering

I recently updated bevy_egui to 0.12.1 (from 0.11) and bevy-inspector-egui to 0.9 (from 0.8). Both are now using egui 0.17, as checked via cargo tree.
Code that previously rendered fine does nothing. It runs (evidenced by debugging print statements inside a Window::new("").show() function. All the bevy systems are still running, but the egui code yields nothing.
There are no errors in compilation, no panics.
Perhaps an issue with providing the right contexts/data for bevy and egui in #78 ?

Hi! Any chance I could take a look at the project you're having the issue with or a minimal reproducible example?

My project is a bit complicated, but I managed to make a MRE: https://gist.github.com/BrettWitty/8887a4d2405eafe5343f00d9dcdb41f9

If I change bevy_egui to 0.11 in my Cargo.toml (and not touch the bevy-inspector-egui version), it runs as expected.

Oh, I see. You should either remove .add_plugin(EguiPlugin) or move it above .add_plugin(WorldInspectorPlugin::new()). bevy-inspector-egui initialises bevy_egui itself if it hasn't been added. But if you add it once more, that'll break things.

Ah that's great. Thanks for solving it.


feat: Split bulk for bulkupsert

Split bulk of BulkUpsert if the bulk greater then max grpc message size:
transport/ResourceExhausted (code = 8, source error = \"rpc error: code = ResourceExhausted desc = trying to send message larger than max (75139668 vs. 67108864)\"

as first step - doesn't retry the error.

it will have problem with non protobuf formats, for example arrow and csv.
In the ticket will fix bulk upsert with native sdk types only.


ngf-max-size not populating form.field.$error

I have the following input:
<form name="someForm">
<input type="file" ngf-select name="logo"
              ng-model="logo.file" ngf-change="selectedLogo()" ngf-max-size="50000" accept="image/*"/>
</form>

When I select a file with size greater than 50kb it doesn't get selected. This is correct behaviour. However, I can't show feedback to the user, as the someForm.logo.$error property remains empty (e.g. {} )

Nevermind, this appears fixed in 7.3.4, my version was outdated.


fix getPackagedAppPath for mac package

This small fix enable to build macOS package with dependencies.

Codecov Report

Merging #237 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #237   +/-   ##
=======================================
  Coverage   58.63%   58.63%
=======================================
  Files          20       20
  Lines        1494     1494
=======================================
  Hits          876      876
  Misses        618      618

Continue to review full report at Codecov.

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


Create Contact us page

Hey, I think there must be a "Contact Us" page in the nav bar so that anyone who wanted to work with you all like if anyone wants to be a front-end developer of your website "Kitabe" then they can easily contact you all through the contact form . Also if any viewer want to ask some questions about any book or author then they can contact you easily. I think this would be good for your websitw, if you find it useful you can assign this issue to me , I  will definitely make a contact form page for your website.
Thank you

@fatma-sia27 We had a similar issue #45


Soak test based on serve_hostname image

This is a simple soak test that performs the following actions when executed for the GCE provider:

Obtains information about what cluster to use for the test from $HOME/.kube/.kubeconfig
Examines how many nodes are on the cluster (say N).
Creates a fresh namespace for the test using a random suffix.
Instantiates M (default 1) serve_hostname pods on each node.
Creates a service that captures the pods serve-hostname-N-M.
Repeatedly issues Q (default 1,000) queries to the service.
Reports the distribution of how many queries were handled by each pod (we expect and even distribution).
It does this in a loop forever.

Keeping this running constitutes a simple soak test.
$ ./serve_hostnames --per_node=5
Nodes found on this cluster:
0: kubernetes-minion-8ezp.c.kubernetes-satnam.internal
1: kubernetes-minion-fga5.c.kubernetes-satnam.internal
2: kubernetes-minion-p2u9.c.kubernetes-satnam.internal
3: kubernetes-minion-viip.c.kubernetes-satnam.internal
Using namespace serve-hostnames-8826 for this test.
Creating pod serve-hostnames-8826/serve-hostname-0-0 on node kubernetes-minion-8ezp.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-0-1 on node kubernetes-minion-8ezp.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-0-2 on node kubernetes-minion-8ezp.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-0-3 on node kubernetes-minion-8ezp.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-0-4 on node kubernetes-minion-8ezp.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-1-0 on node kubernetes-minion-fga5.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-1-1 on node kubernetes-minion-fga5.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-1-2 on node kubernetes-minion-fga5.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-1-3 on node kubernetes-minion-fga5.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-1-4 on node kubernetes-minion-fga5.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-2-0 on node kubernetes-minion-p2u9.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-2-1 on node kubernetes-minion-p2u9.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-2-2 on node kubernetes-minion-p2u9.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-2-3 on node kubernetes-minion-p2u9.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-2-4 on node kubernetes-minion-p2u9.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-3-0 on node kubernetes-minion-viip.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-3-1 on node kubernetes-minion-viip.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-3-2 on node kubernetes-minion-viip.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-3-3 on node kubernetes-minion-viip.c.kubernetes-satnam.internal ... [done]
Creating pod serve-hostnames-8826/serve-hostname-3-4 on node kubernetes-minion-viip.c.kubernetes-satnam.internal ... [done]
Creating service serve-hostnames ... [done]
Wait for the serve-hostname pods to be ready
serve-hostnames-8826/serve-hostname-0-0 is running
serve-hostnames-8826/serve-hostname-0-1 is running
serve-hostnames-8826/serve-hostname-0-2 is running
serve-hostnames-8826/serve-hostname-0-3 is running
serve-hostnames-8826/serve-hostname-0-4 is running
serve-hostnames-8826/serve-hostname-1-0 is running
serve-hostnames-8826/serve-hostname-1-1 is running
serve-hostnames-8826/serve-hostname-1-2 is running
serve-hostnames-8826/serve-hostname-1-3 is running
serve-hostnames-8826/serve-hostname-1-4 is running
serve-hostnames-8826/serve-hostname-2-0 is running
serve-hostnames-8826/serve-hostname-2-1 is running
serve-hostnames-8826/serve-hostname-2-2 is running
serve-hostnames-8826/serve-hostname-2-3 is running
serve-hostnames-8826/serve-hostname-2-4 is running
serve-hostnames-8826/serve-hostname-3-0 is running
serve-hostnames-8826/serve-hostname-3-1 is running
serve-hostnames-8826/serve-hostname-3-2 is running
serve-hostnames-8826/serve-hostname-3-3 is running
serve-hostnames-8826/serve-hostname-3-4 is running
Waiting for the endpoints to propagate... finished waiting
serve-hostname-1-2: 61  serve-hostname-0-3: 51  serve-hostname-2-1: 46  serve-hostname-2-0: 42  serve-hostname-1-1: 44  serve-hostname-0-1: 42  serve-hostname-0-0: 62  serve-hostname-1-0: 49  serve-hostname-1-4: 39  serve-hostname-2-3: 64  serve-hostname-3-3: 40  serve-hostname-2-2: 56  serve-hostname-1-3: 43  serve-hostname-0-4: 58  serve-hostname-0-2: 46  serve-hostname-2-4: 57  serve-hostname-3-1: 59  serve-hostname-3-2: 51  serve-hostname-3-4: 42  serve-hostname-3-0: 48
Took 51.191636354s
serve-hostname-3-1: 55  serve-hostname-1-3: 50  serve-hostname-1-4: 52  serve-hostname-1-0: 49  serve-hostname-2-1: 41  serve-hostname-0-1: 42  serve-hostname-2-2: 46  serve-hostname-0-4: 47  serve-hostname-0-3: 38  serve-hostname-2-3: 38  serve-hostname-0-0: 58  serve-hostname-1-2: 63  serve-hostname-3-3: 59  serve-hostname-2-4: 65  serve-hostname-3-0: 51  serve-hostname-1-1: 49  serve-hostname-3-2: 48  serve-hostname-2-0: 48  serve-hostname-0-2: 48  serve-hostname-3-4: 53
Took 52.022849323s
serve-hostname-3-2: 58  serve-hostname-1-2: 48  serve-hostname-1-1: 48  serve-hostname-2-1: 53  serve-hostname-1-0: 46  serve-hostname-1-3: 52  serve-hostname-3-1: 45  serve-hostname-2-4: 43  serve-hostname-2-3: 48  serve-hostname-2-0: 70  serve-hostname-3-4: 47  serve-hostname-0-1: 51  serve-hostname-0-3: 64  serve-hostname-0-0: 50  serve-hostname-0-4: 41  serve-hostname-2-2: 44  serve-hostname-3-3: 43  serve-hostname-3-0: 54  serve-hostname-0-2: 54  serve-hostname-1-4: 41
Took 51.382949407s
serve-hostname-0-0: 45  serve-hostname-0-3: 51  serve-hostname-3-2: 40  serve-hostname-1-3: 61  serve-hostname-2-1: 47  serve-hostname-1-2: 49  serve-hostname-3-0: 57  serve-hostname-2-3: 53  serve-hostname-1-4: 54  serve-hostname-3-1: 58  serve-hostname-2-4: 42  serve-hostname-0-1: 64  serve-hostname-0-2: 40  serve-hostname-1-0: 55  serve-hostname-2-0: 43  serve-hostname-3-3: 50  serve-hostname-0-4: 53  serve-hostname-2-2: 57  serve-hostname-1-1: 44  serve-hostname-3-4: 37
Took 52.76047085s
serve-hostname-1-0: 45  serve-hostname-3-2: 56  serve-hostname-0-1: 48  serve-hostname-2-3: 55  serve-hostname-0-4: 62  serve-hostname-3-3: 58  serve-hostname-2-0: 59  serve-hostname-0-0: 47  serve-hostname-3-1: 42  serve-hostname-1-1: 43  serve-hostname-2-1: 69  serve-hostname-3-0: 48  serve-hostname-1-2: 53  serve-hostname-1-3: 47  serve-hostname-1-4: 43  serve-hostname-2-4: 48  serve-hostname-2-2: 44  serve-hostname-0-2: 48  serve-hostname-3-4: 41  serve-hostname-0-3: 44
Took 51.717851856s
serve-hostname-2-2: 46  serve-hostname-1-3: 43  serve-hostname-1-1: 49  serve-hostname-3-4: 55  serve-hostname-0-3: 52  serve-hostname-1-0: 47  serve-hostname-3-1: 45  serve-hostname-0-1: 54  serve-hostname-1-4: 32  serve-hostname-0-2: 64  serve-hostname-2-1: 52  serve-hostname-3-2: 47  serve-hostname-3-3: 51  serve-hostname-3-0: 55  serve-hostname-2-3: 54  serve-hostname-0-0: 42  serve-hostname-1-2: 66  serve-hostname-0-4: 55  serve-hostname-2-0: 46  serve-hostname-2-4: 45
Took 52.904848788s
serve-hostname-0-0: 46  serve-hostname-2-1: 54  serve-hostname-0-3: 30  serve-hostname-3-4: 52  serve-hostname-2-0: 55  serve-hostname-1-0: 49  serve-hostname-3-0: 55  serve-hostname-0-1: 51  serve-hostname-1-2: 43  serve-hostname-3-1: 56  serve-hostname-1-1: 48  serve-hostname-0-2: 51  serve-hostname-3-2: 46  serve-hostname-1-3: 52  serve-hostname-1-4: 46  serve-hostname-2-3: 62  serve-hostname-3-3: 48  serve-hostname-2-2: 56  serve-hostname-2-4: 49  serve-hostname-0-4: 51
Took 52.712764611s
serve-hostname-3-4: 61  serve-hostname-0-4: 52  serve-hostname-0-3: 44  serve-hostname-1-0: 60  serve-hostname-1-1: 42  serve-hostname-2-2: 45  serve-hostname-1-3: 55  serve-hostname-3-2: 56  serve-hostname-3-3: 59  serve-hostname-3-1: 53  serve-hostname-1-4: 62  serve-hostname-0-2: 40  serve-hostname-0-0: 45  serve-hostname-1-2: 46  serve-hostname-2-1: 36  serve-hostname-0-1: 44  serve-hostname-3-0: 62  serve-hostname-2-4: 45  serve-hostname-2-0: 54  serve-hostname-2-3: 39
Took 53.611146825s
serve-hostname-1-4: 55  serve-hostname-0-0: 37  serve-hostname-3-2: 51  serve-hostname-3-0: 58  serve-hostname-1-2: 41  serve-hostname-2-3: 47  serve-hostname-1-1: 53  serve-hostname-1-3: 50  serve-hostname-0-2: 52  serve-hostname-1-0: 49  serve-hostname-0-1: 52  serve-hostname-0-3: 53  serve-hostname-2-0: 43  serve-hostname-3-3: 48  serve-hostname-3-1: 58  serve-hostname-3-4: 49  serve-hostname-0-4: 60  serve-hostname-2-4: 47  serve-hostname-2-2: 54  serve-hostname-2-1: 43
Took 52.354442773s

@roberthbailey

PTAL. I think I've dealt with all the excellent suggestions. For now, I have changed some Fatals to Warnings and then I return to make sure the defer cleanup occurs. I report any nodes that did not respond with a Warningf. In a future iteration I will handle SIGINT and SIGKILL to make sure the defer occurs. It is now possible to specify the number of iterations (see below) so this can be added to some suite or regularly run builds for examples etc. to avoid bit-rot.
$ ./serve_hostnames --up_to=1
I0321 11:12:06.071849   76721 serve_hostnames.go:53] Starting serve_hostnames soak test with queries=1000 and perNode=1 upTo=1
I0321 11:12:06.481369   76721 serve_hostnames.go:78] Nodes found on this cluster:
I0321 11:12:06.481386   76721 serve_hostnames.go:80] 0: kubernetes-minion-6gen.c.kubernetes-satnam2.internal
I0321 11:12:06.481391   76721 serve_hostnames.go:80] 1: kubernetes-minion-ivc1.c.kubernetes-satnam2.internal
I0321 11:12:06.481395   76721 serve_hostnames.go:80] 2: kubernetes-minion-nkl2.c.kubernetes-satnam2.internal
I0321 11:12:06.481399   76721 serve_hostnames.go:80] 3: kubernetes-minion-qjrh.c.kubernetes-satnam2.internal
I0321 11:12:06.481417   76721 serve_hostnames.go:86] Using namespace serve-hostnames-1091 for this test.
I0321 11:12:06.481423   76721 serve_hostnames.go:89] Creating service serve-hostnames
I0321 11:12:06.569275   76721 serve_hostnames.go:123] Creating pod serve-hostnames-1091/serve-hostname-0-0 on node kubernetes-minion-6gen.c.kubernetes-satnam2.internal
I0321 11:12:06.649259   76721 serve_hostnames.go:123] Creating pod serve-hostnames-1091/serve-hostname-1-0 on node kubernetes-minion-ivc1.c.kubernetes-satnam2.internal
I0321 11:12:06.737740   76721 serve_hostnames.go:123] Creating pod serve-hostnames-1091/serve-hostname-2-0 on node kubernetes-minion-nkl2.c.kubernetes-satnam2.internal
I0321 11:12:06.858153   76721 serve_hostnames.go:123] Creating pod serve-hostnames-1091/serve-hostname-3-0 on node kubernetes-minion-qjrh.c.kubernetes-satnam2.internal
I0321 11:12:06.966499   76721 serve_hostnames.go:158] Waiting for the serve-hostname pods to be ready
I0321 11:12:53.014380   76721 serve_hostnames.go:175] serve-hostnames-1091/serve-hostname-0-0 is running
I0321 11:13:18.486994   76721 serve_hostnames.go:175] serve-hostnames-1091/serve-hostname-1-0 is running
I0321 11:13:18.596576   76721 serve_hostnames.go:175] serve-hostnames-1091/serve-hostname-2-0 is running
I0321 11:13:18.670425   76721 serve_hostnames.go:175] serve-hostnames-1091/serve-hostname-3-0 is running
I0321 11:14:28.024996   76721 serve_hostnames.go:202] serve-hostname-2-0: 265
I0321 11:14:28.025029   76721 serve_hostnames.go:202] serve-hostname-0-0: 261
I0321 11:14:28.025039   76721 serve_hostnames.go:202] serve-hostname-3-0: 264
I0321 11:14:28.025049   76721 serve_hostnames.go:202] serve-hostname-1-0: 210
I0321 11:14:28.025065   76721 serve_hostnames.go:213] Iteration 0 took 1m9.35461398s for 1000 queries
I0321 11:14:28.025091   76721 serve_hostnames.go:150] Cleaning up pods
I0321 11:14:28.349487   76721 serve_hostnames.go:111] Cleaning up service

Cool idea !  So, this is a specific kind of soak, i.e. i think this test is specifically testing the amount of times a service is forwarded to one machine, versus another.
So shall we maybe call it svc_soak.go ?
then i think the other tests can be on the lines of net-soak, cpu-soak, all-soak (this i think is what I'm working on atm)...
Otherwise lgtm and hope to try it out soon.

I am currently running it on a 50 node cluster with N pods per node. I am also producing rc_serve_hostnames which does not explicitly pin pods to nodes but instead makes use of a replication controller and it also randomly kills pods and adds pods and checks that the overall behaviour of the system is fine. I am calling this test cauldron because it is means to keep the number of pods bubbling along at roughly the same number.

Even 10 pods per node is too much.
I0322 05:28:18.910522    1886 serve_hostnames.go:175] serve-hostnames-7439/serve-hostname-49-3 is running
I0322 05:28:18.917045    1886 serve_hostnames.go:175] serve-hostnames-7439/serve-hostname-49-4 is running
Timeout, server <IP_ADDRESS> not responding.

5 pods per node also does not work.


Sync Server beginning of documentation

First attempt at Site Sync documentation.
Its a question if it should be separated as I did it(admin and user separated) or if everything should be in single place.

I've tweaked some wordings and added more stuff. I reckon let's merge this to have a starting point and keep iterating on it.


fix(runtime/manual/tools/init): remove bench

deno init no longer creates the bench file.
Output also does not include the instruction to run benchmarks.

thanks


Use boson buildpacks for Quarkus

Use boson (downstream, non-paketo) buildpack.
Force legacy-jar packaging until boson buildpack support fast-jar packaging.

Signed-off-by: Matej Vasek<EMAIL_ADDRESS>

Recently I switched to the paketo buildpacks in upstream, however for midstream we still should use boson buildpacks.

/lgtm


Add Purge ~K~ by Kubatko

Song Name
Purge ~K~
Artist Name
Kubatko
Source
Youtube
Youtube Link
ffLs4E6FD4w
Direct File Link
No response
Song ID
354826
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 3072765

accept

accept

accept

accept

reject
popular songs like toe xl are too inconvenient to be replaced by smth unrelated and are therefore usually rejected.
Note that you can still use the song yourself, but it won't be public


The accuracy of the hdl_graph_slam

I ran the datasets that you provided, and used the rosservice to save the map, as you can see  the map in some where has the thick point cloud. So can you tell me how to adujst the parameters. and I saw the code and found that IMU is only used in the optimization part, so is there no use of IMU for distortion correction of point cloud data in the front end?  Look forward to your reply! thanks

Hi @Blackshaddock,
You may want to tune the following params which have a big impact on the mapping result:

registration_method: NDT_OMP or GICP (note that GICP in PCL 1.7 may not work)
ndt_resolution: larger values are good for outdoor envionments (e.g. 2-10m)

BTW, I'm going to release a new SLAM framework which allows to interactively correct mapping errors like the one you pointed out. If you are interested in it, please check out the following repository in the next week:
https://github.com/koide3/interactive_slam

I saw the code and found that IMU is only used in the optimization part, so is there no use of IMU for distortion correction of point cloud data in the front end?

In this package, we use IMU data only in the backend optimization part (I assumed the user moves the LIDAR “gently” during mapping). In the localization package (hdl_localization), we used IMU to deal with quick sensor motion. The frontend of this package may be improved in the same way, but it has not been done yet.

Hi @koide3 , will the new SLAM framework be used an enhancement to the hdl_graph_slam? And it would be interesting to know how to use these packages together, for example, hdl_graph_slam and hdl_localization can launch them simultaneously do they complete each other or not? If not, then how would it be possible to add the IMU implementation into hdl_graph_slam, what steps need to be taken, and which classes to be modified or added?

Hi @GLkode,
The new framework (interactive_slam) is built on top of hdl_graph_slam. You first create a map with hdl_graph_slam (or any other ROS package), then you can edit the created map with interactive_slam.
Since hdl_localization requires a complete map beforehand, you cannot launch them simultaneously. You need to first create a map with hdl_graph_slam (and interactive_slam), and then launch hdl_localization with the created map.
If you want to incorporate IMU information in the frontend of the mapping process, you need to modify scan_matching_odometry_nodelet. You can use PoseEstimater class which performs a UKF-based IMU and range data fusion to estimate the sensor pose and then give it to the scan matching as the initial guess. If you are going to implement it, I would be happy to provide more detailed information.
https://github.com/koide3/hdl_graph_slam/blob/master/apps/scan_matching_odometry_nodelet.cpp
https://github.com/koide3/hdl_localization/blob/master/include/hdl_localization/pose_estimator.hpp

Thank you very much @koide3 for the detailed explanation, I believe there are many requests for implementing IMU in frondend, I would be happy to implement it, could you please provide me with more detailed information, so I can start.

Hi @GLKode ,
Thanks a lot. I try to provide more information:
In scan_matching_odometry_nodelet, we apply a scan matching method between consecutive input frames to estimate the sensor ego-motion. In the current implementation, we just use the last sensor pose as the initial guess for the scan matching (scan_matching...cpp, L156). So, it doesn't take sensor motion into account so far.
https://github.com/koide3/hdl_graph_slam/blob/c5d5e6720403cb8445725d6eb30ac6e81be6b004/apps/scan_matching_odometry_nodelet.cpp#L156
In hdl_localization_nodelet, we use IMU data to predict the sensor displacement from the last scan matching result. We have a class named PoseEstimater which exploits UKF to predict the currenct sensor pose from IMU data and integrate it with the scan matching result.
https://github.com/koide3/hdl_localization/blob/master/include/hdl_localization/pose_estimator.hpp
This class has two important methods, predict and correct. "predict" method takes an IMU data and its timestamp and updates the estimated sensor pose. "correct" method takes a point cloud and updates the estimation with the scan matching result. What we need to do is to adapt this class to scan_matching_odometry_nodelet. Currently, PoseEstimater performs map-vs-current-frame scan matching, and we need to modify the class so it performs previous-frame-vs-current-frame scan matching.
https://github.com/koide3/hdl_localization/blob/624a7613d1639fefc0f67934e4f57dd2a8c9878f/include/hdl_localization/pose_estimator.hpp#L97
(we need to set previous frame as the input target at this line)
Then, we can put the class in scan_matching_odometry_nodelet. We can just feed IMU and LIDAR data to the class and publish the result as the estimated odometry. The following codes in the localization nodelet may be helpful to know how we can feed data to the class.
https://github.com/koide3/hdl_localization/blob/624a7613d1639fefc0f67934e4f57dd2a8c9878f/apps/hdl_localization_nodelet.cpp#L153
https://github.com/koide3/hdl_localization/blob/624a7613d1639fefc0f67934e4f57dd2a8c9878f/apps/hdl_localization_nodelet.cpp#L160
In case you need more information please let me know. You can also refer to the paper which describes the theoretical side in detail.
Thanks again for your cooperation :)

Thank you very much @koide3 for the detailed information, by end of the next week I will be having time and will start working on it. will keep you updated.

Hi @koide3, I am very sorry for the late reply, I have had an accident last month and could not work on the code, today I have access to my PC and went through the code, in the PoseEstimater, I must set the previous_frame as registration->setInputSource(previous_frame);
With frame did you mean keyframe?  from the first look i didn't know where to access to the previous frame, can you please give me a hint?
Thank you in advance!

Hi @GLKode ,
It doesn't matter. I'd be glad if you could do it in your spare time...

in the PoseEstimater, I must set the previous_frame as segistration->setInputSource(previous_frame);
With frame did you mean keyframe? from the first look i didn't know where to access to the previous frame, can you please give me a hint?

In the localization method, since the registration target is the globalmap which never changes, setInputTarget is called when the globalmap is loaded. See: https://github.com/koide3/hdl_localization/blob/624a7613d1639fefc0f67934e4f57dd2a8c9878f/apps/hdl_localization_nodelet.cpp#L186
To adapt the PoseEstimater to the SLAM frontend, I suppose we need to change it to the last keyframe.

Was this implemented?


BTRoot : setting blackboard from script does not use the given blackboard instance

Scenario:

Create a node
Add a BTRoot as child of this node, and some behavior tree logic in it
In another scene script, add an instance of the above node
Create a blackboard in script, add some values in it, and assign this blackboard to the node's tree
Run the scene

The assigned blackboard is not the same as the tree blackboard at runtime ; its a copy. By doing this, it's not possible to share a blackboard between trees if the blackboard is not assign from Godot Editor.
One way to fix that is to change blackboard type in BTRoot from BTBlackboard to NodePath, like the actor

Some precisions:

We can't share a blackboard because, when setting the blackboard variable in BTRoot, it does not propagate to the real blackboard (_blackboard) : easy to fix
Once we do that, we can share a blackboard quite easily. BUT, there is still an issue ; trees that share the blackboard will store datas in it

When it's user data, it may be important or not, but it's shared
When this is a BTRoot, it contains data that make the tree work (running children ,...). We can't share that sort of thing

To solve that, the blackboard should have a sort of namespace in it ; when storing / retrieving data, you can set in which namespace the blackboard is working. Data in different namespaces should be totally separated.
To preserve the current API, the get_data and set_data can have a thrid parameter, namespace with a default value, to store datas in the default namespace.

Available in main branch :)


add mid-word and multi emphasis

I thought it might be useful to let people know you can use bold and italics in the middle of words as well as in combination with each other.

Deploy preview ready!
Built with commit 153ae13733cba578ba3a5df92ce5a13f940a1908
https://deploy-preview-2--markdown-guide.netlify.com

Thanks! This looks great!


Add automated tests to Dockerfile and configure a Travis account for the project

Just closed by latest commit. Check travis at https://travis-ci.org/github/flaviostutz/ruller


nixos/tests/chromium: Switch to nodes.machine (vs. deprecated machine…

… attribute)
The old attribute is deprecated:
trace: warning: In test chromium-stable': The machine' attribute in NixOS
tests (pkgs.nixosTest / make-test-pyton.nix / testing-python.nix / makeTest) is
deprecated. Please use the equivalent `nodes.machine'.
Note: This is only a refactoring.
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg test chromium


Error to update to PHP 7.2

When send email apperace this error after update my server to PHP 7.2
message: count(): Parameter must be an array or an object that implements Countable"
file: "/var/www/html/tfd_rh_api/vendor/sendgrid/sendgrid/lib/SendGrid/Email.php"
References:
https://github.com/yiisoft/yii/issues/4167
https://github.com/Automattic/jetpack/issues/8420
https://github.com/invoiceninja/invoiceninja/issues/1767

@bryglen, there is a way to solve this, did you prefer @Macbenach and me make a PR?

this is a known issue on 7.2, as of now no support for 7.2

@bryglen dude, plz upgrade your composer.json at least 6.++, see reference https://github.com/sendgrid/sendgrid-php/releases, you library are to old, plz make the update.

Hello @bryglen what you think about this snippet
  public function getHeadersJson() {
    if ($this->getHeaders() == NULL) {
     return "{}";
    }

    if (count($this->getHeaders()) <= 0) {
     return "{}";
    }
    return json_encode($this->getHeaders(), JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_AMP);
  }

Maybe exist some better code but it seems to solve the problem in php 7.2, i test it in PHP 7.2 and 7.0

@bryglen, help us

please create PR

Hello guys,
Can use this package https://github.com/ramprasadm1986/yii2-sendgrid


Remove experimental from gateway infrastructure

What type of PR is this?

/kind gep
/kind cleanup
What this PR does / why we need it:
Removes experimental from gateway infrastructure type.
Which issue(s) this PR fixes:

Fixes #1867
Does this PR introduce a user-facing change?:

Promoted GEP-1867 Per Gateway Infrastructure to Standard Channel

#3197 actually probably blocks this to some extent, because we need to resolve that before we release. Doesn't stop us merging this, but we need to make sure we do both.

Thanks for the reminder! @snorwin happy to wait on your change or make add the new LabelValue field in this PR. Up to you 🙂

@keithmattix I created a proposal in #3284. It would be great if you have a look at it too.

@keithmattix @howardjohn We got a good question about this GEP at SIG-Network yesterday. How do we handle mutability? If these labels were to map to Pods, it could be rather disruptive to update labels.

In the Istio implementation, we create a deployment for the gateway, so any pod restarts would be via a rolling restart. I think as long as you're not changing a label used for deployment or service selection, you're probably fine in most cases. I think the GEP reserves the gateway.networking.k8s.io prefix (i.e. users can't set labels with that prefix), and that's what we use for deployment + service selection in Istio.

@keithmattix do you mind adding a note somewhere in the GEP as part of this PR to at least call out the risk that changing labels could lead to underlying dataplane Pods being replaced, depending on the underlying implementation? I'm assuming that at least some Gateway implementations may not be able to do that without at least some minor disruptions to traffic.

@robscott I added some language about dataplane pods; let me know if that's sufficient. I also removed references to gatewayclass.infrastructure because I believe that direction was abandoned

Thanks @keithmattix! LGTM once presubmits pass - looks like another round of codegen is needed.
/approve

/lgtm
once the presubmits pass. Nice work @keithmattix!

Oh the cel isn't getting copied over hmmm

It was because I didn't rebase on top of #3197 🤦🏾 presubmit should pass now

It was because I didn't rebase on top of #3197 🤦🏾 presubmit should pass now

Good catch, thanks @keithmattix!
/lgtm

/hold cancel


internal/uidriver/glfw: Use glfw.WaitEventsTimeout instead of time.Timer and glfw.PollEvents

I forgot the context.
https://github.com/hajimehoshi/ebiten/blob/4509f2aee4c88d556a1f46dca011b034b52b6fd4/internal/uidriver/glfw/ui.go#L1318-L1332
WaitEventsTimeout would not work well with framebufferSizeCallbackCh.


[EAK-504] Limited Parsys: add functionality to limit parsys-like components child component count

Closes: #504

Current TODO list:

Rename limited-parsys to max-children in JS API
Get rid of isParsys check if possible, add the ability to handle other placeholders, then 'newpar'
Move the isEditConfig check to a generic place from MaxChildrenHandler or remove todo
Investigate the possibility of displaying a message if the limit is riched


I have a problem that the devices are not detected on the bus.

I have a problem that the devices are not detected on the bus.
This is in the OneWire32::search() as the first reset fails.
and then it will report that there are no items.
08:57:54.652 -> Setup Started
08:57:54.652 -> reset () found =0 item=1 rx_size=4 Level0=1 Duration0=1 Level1=0 Duration1=481
08:57:54.652 -> reset () found =1 item=1 rx_size=8 Level0=0 Duration0=481 Level1=1 Duration1=33
08:57:54.694 -> 0: 0xc90000001b94f828,
I have resolved this by adding an extra reset() before starting the search.
#Device ESP32-S3
PS: Add some default TX,RX buffer for the diffrent Devices. as not all devices support RX/TX on all buffers.
OneWire32 ds(ONEWIRE_PIN, 0, 4, 0); //gpio pin, tx, rx, parasite power
uint8_t OneWire32::search(uint64_t addresses[], uint8_t total) {
uint8_t last_src;
int8_t last_dev = -1;
uint8_t found = 0;
uint8_t loop = 1;
if(!drvrx){return found;}
reset(); // reset the Bus once before discovery
while(loop && found < total){
uint64_t addr = 0;
loop = 0;
last_src = last_dev;
if(!reset()){
found = 0;
break;
}

done,
what do you mean by TX RX buffers? mem_block_num?


update to fluent 1.9.8

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/test all

/test all

/test all


Line width marker not moved on startup

Prerequisites

[x] Can you reproduce the problem in safe mode?
Not applicable, but probably.
[x] Are you running the latest version of Atom?
Yes.
[x] Did you check the debugging guide?
Not applicable, but yes.
[x] Did you check the FAQs on Discuss?
Not applicable, but yes.
[x] Are you reporting to the correct repository?
Yes.
[x] Did you perform a cursory search to see if your bug or enhancement is already reported? Yes, using the line in:title search selector

Description
When changing the font size on startup, the line width marker is in the incorrect (old) place.
Steps to Reproduce

Edit your Init script (Edit > Init script...) to set font size on startup (e.g. by adding atom.config.set('editor.fontSize', 1)).
Zoom a little, so the line width marker is clearly visible near the middle.
Reopen atom.

Expected behavior: The line width marker is in the correct position, near the left when setting editor.fontSize to one.
Actual behavior: The line width marker is still in it's old positition. Zooming in or out will move the marker to the correct position.
Versions
Atom 1.11.1
Ubuntu 16.04.1 LTS

This is a duplicate of atom/wrap-guide#25 - feel free to subscribe there for updates.
(Also, testing in safe mode and checking the debugging guide are both very applicable here.)

@50Wliu Thanks, I will.


[BUG] A problem occurred configuring project ':google_api_headers'.

Steps to Reproduce
flutter run
Expected behaviour
I was expecting not to throw an error.
Actual behaviour
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':google_api_headers'.

Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

 If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant.

Flutter doctor output
[√] Flutter (Channel stable, 3.24.3, on Microsoft Windows [Version 10.0.22631.4169], locale en-IN)
• Flutter version 3.24.3 on channel stable at C:\src\3.24.3
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 2663184aa7 (10 days ago), 2024-09-11 16:27:48 -0500
• Engine revision 36335019a8
• Dart version 3.5.3
• DevTools version 2.37.3
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
• Android SDK at C:\Users\nishant\AppData\Local\Android\sdk
• Platform android-35, build-tools 35.0.0
• Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[X] Visual Studio - develop Windows apps
X Visual Studio not installed; this is necessary to develop Windows apps.
Download at https://visualstudio.microsoft.com/downloads/.
Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2024.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.10+0--11609105)
[√] VS Code (version 1.93.1)
• VS Code at C:\Users\nishant\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.96.0
[√] Connected device (4 available)
• Windows (desktop)   • windows          • windows-x64    • Microsoft Windows [Version 10.0.22631.4169]
• Chrome (web)        • chrome           • web-javascript • Google Chrome 128.0.6613.138
• Edge (web)          • edge             • web-javascript • Microsoft Edge 129.0.2792.52
[√] Network resources
• All expected network resources are available.
Additional Context
No response

i suffer from he same issue

I'm unable to reproduce this and the example app within this repo runs fine for me, can you provide a minimal reproducible example? Please also confirm if the issue still remains in the latest Flutter and google_api_headers package.


Intranet usage requirements

Requested feature
Using Docling requires downloading models via huggingface_hub, which poses challenges in internal network environments without internet access. Could you modify the system to detect and utilize locally stored models? This adjustment would facilitate internal usage and allow for the integration of more advanced models, such as ChatGPT-4, potentially enhancing document parsing performance.

This is available. You actually have two options:

Download the model artifacts before, e.g. when building a Docker image. This will store them in the default cache location.
converter = DocumentConverter()
converter.initialize_pipeline(InputFormat.PDF)

Point the conversion pipeline to an existing artifacts folder. In case you have a custom location with the artifacts.
pipeline_options = PdfPipelineOptions(artifacts_path="your location")
converter = DocumentConverter(
    format_options={
        InputFormat.PDF: PdfFormatOption(pipeline_options=pipeline_options)
    }
)

thanks！

Sorry to bother you again. After installing EasyOCR by pip, I still encounter network download errors. Is there a parameter to specify the model storage directory? Like EasyOcrOptions.model_storage_directory?

Yes, the EasyOCR option has already arguments for 1) do not download models, 2) set the location.
See the options here: https://github.com/DS4SD/docling/blob/main/docling/datamodel/pipeline_options.py#L35

Yes, the EasyOCR option has already arguments for 1) do not download models, 2) set the location.
See the options here: main/docling/datamodel/pipeline_options.py#L35

Thanks!


[interoperator] Fix bug in mcd provisoner

Stateful rest replica count was not getting updated if master is used as sister

Pull Request Test Coverage Report for Build 3306

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 89.884%

Files with Coverage Reduction
New Missed Lines
%

data-access-layer/db/DBManager/DBManager.js
1
98.48%

Totals

Change from base Build 3300:
0.0%

Covered Lines:
9390

Relevant Lines:
10095

💛  - Coveralls


SegmentationImage gets immediately cleared by clearing at later time step

Describe the bug
It looks like a previously logged SegmentationImage gets cleared by a rr2.Clear at a later time. Only observed for SegmentationImage so far.
To Reproduce
import rerun as rr
import numpy as np

rr.init("test clear", spawn=True)
rr.set_time_sequence("frame", 0)
rr.log_image("image", np.random.rand(100,100,3) * 0.2)
rr.log_segmentation_image("image/segmentation", np.random.randint(0,10, size=(100,100)))
rr.set_time_sequence("frame", 10)
rr.log_cleared("image/segmentation")

This works fine in 0.8.2, but not in the prerelease. Same with the new API.
Expected behavior
Segmentation image should disappear at frame 10, not at frame 0.
Desktop (please complete the following information):

OS: Ubuntu 20.04

Rerun version
rerun_py 0.9.0-alpha.4 [rustc 1.72.1 (d5c2e9c34 2023-09-13), LLVM 16.0.5] x86_64-unknown-linux-gnu 2e9df4c, built 2023-09-23T09:59:35Z

I'm actually reproducing the same issue on both 0.8.2 and 0.9-alpha :thinking:


Test Bug 47

Summary
Short summary of what is going on or to provide context.
Steps To Reproduce:

This is step 1.
This is step 2. All steps should start with '1.'

Expected result
Describe what should have happened.
Actual result
Describe what actually happened instead.
Additional information
Feel free to attach a screenshot.
Salesforce Extension Version in VS Code: 60.12.0
Salesforce CLI Version: @salesforce/cli/2.41.8 darwin-arm64 node-v18.18.2
OS and version:
VS Code version:
Most recent version of the extensions where this was working:

Hello @daphne-sfdc :wave: None of the versions of the Salesforce Extension Pack you shared match the latest release.
Shared: 60.12.0
Latest: 60.13.0
Update to the latest version of the Salesforce Extension Pack in the VSCode Extensions view and confirm that you're still seeing your issue.
After updating, share your current extensions version.

Thank you for filing this issue.  We appreciate your feedback and will review the issue as soon as possible. Remember, however, that GitHub isn't a mechanism for receiving support under any agreement or SLA. If you require immediate assistance, contact Salesforce Customer Support.

Salesforce Extension Version in VS Code: 60.13.0


[WIP] Enable GRPC support

Description

Fix #1149
How Has This Been Tested?

Test Impact

Request review criteria:

[ ] The commits have meaningful messages (squashes happen on merge by the bot).
[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work
[ ] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

@heyselbi @Jooho This is the WIP PR, it seems that the feature is not ready in the backend side so we'll hold this for now.

Closing this as it would need a broader scope based on latest conversations in the thread


Dependency library to up version

Hello. Thank you for providing a nice library.
Since the major version of the dependent package emojilib is uped, I would like to you to follow it.
Can you please upgrade the version or can I submit a pull request from here?
Thank you.

A PR would be welcome! 🌟

Pull request has been created! Please check it out. #17


[Snyk] Upgrade axios from 1.6.4 to 1.6.5

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade axios from 1.6.4 to 1.6.5.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2024-01-05.

Release notes

Package name: axios

1.6.5 - 2024-01-05Release notes:
Bug Fixes

ci: refactor notify action as a job of publish action; (#6176) (0736f95)
dns: fixed lookup error handling; (#6175) (f4f2b03)

Contributors to this release

 Dmitriy Mozgovoy
 Jay

1.6.4 - 2024-01-03Release notes:
Bug Fixes

security: fixed formToJSON prototype pollution vulnerability; (#6167) (3c0c11c)
security: fixed security vulnerability in follow-redirects (#6163) (75af1cd)

Contributors to this release

 Jay
 Dmitriy Mozgovoy
 Guy Nesher

    from axios GitHub release notes

Commit messages

Package name: axios

6d4c421 chore(release): v1.6.5 (#6177)
0736f95 fix(ci): refactor notify action as a job of publish action; (#6176)
f4f2b03 fix(dns): fixed lookup error handling; (#6175)
1f73dcb docs: update sponsor links

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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

:exclamation: No coverage uploaded for pull request base (develop@cda7774). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #213   +/-   ##
==========================================
  Coverage           ?   63.69%
  Complexity         ?      315
==========================================
  Files              ?      124
  Lines              ?     1099
  Branches           ?       50
==========================================
  Hits               ?      700
  Misses             ?      356
  Partials           ?       43

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


add alias domain proxy

Hello
Please add alias domain proxy in the next version.
Thanks

Can do :)

Version 2 is ready for someone to test besides me. Have a read of the documentation including the installation instructions and importing guide.
The app has only a few minor bugs I want to change but I'm using it in production myself on 4 instances.
If you have issues, probably be to report them as new issues and mention that they affect version 2 in the description.
Thanks


Changing proj API from v4 to v6

https://github.com/osmcode/libosmium/issues/277

https://git.neumann-andreas.de/andreas/libosmium-docker/blob/master/Dockerfile
Test-Docker-Build-Container with alpine:latest and proj v6

Thanks for the pull request. Please check what happened that all lines show up as changed. Also for backwards compatibility the old API must still work, please add some kind of ifdef-magic or so to check for that.

Also for backwards compatibility the old API must still work, please add some kind of ifdef-magic or so to check for that.

@cyper85 You might have a look at https://github.com/Nakaner/osmium-geos-factory/blob/master/include/osmium_geos_factory/geos_factory.hpp#L47 where I deal with changes in GEOS's C++ API. proj.h defines the macros PROJ_VERSION_MAJOR, PROJ_VERSION_MINOR and PROJ_VERSION_MAJOR patch for that purpose.

Codecov Report

Merging #284 into master will increase coverage by 0.01%.
The diff coverage is 97.67%.

@@            Coverage Diff             @@
##           master     #284      +/-   ##
==========================================
+ Coverage   83.21%   83.23%   +0.01%
==========================================
  Files         282      282
  Lines       23933    23935       +2
==========================================
+ Hits        19916    19922       +6
+ Misses       4017     4013       -4

Flag
Coverage Δ

#data_tests
60.41% <ø> (ø)
:arrow_up:

#examples
56.23% <ø> (ø)
:arrow_up:

#unit_tests
79.84% <97.67%> (+0.02%)
:arrow_up:

Impacted Files
Coverage Δ

include/osmium/geom/projection.hpp
97.67% <97.67%> (+0.11%)
:arrow_up:

include/osmium/thread/queue.hpp
100% <0%> (+12.5%)
:arrow_up:

Continue to review full report at Codecov.

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

It still shows all lines as changed. Is there a problem with line endings?
__has_include is only available since C++17, but libosmium should work from C++11 on.

Okay,
I had to modify .trafis.yml. Due the little Change I add osx10.2 and osx11 to the installation.
In osx will build proj in v6.1, so you can see in your tests, that it works.

We have a temporary solution (see #277) now.
This patch is problematic on several levels. Some problems I have mentioned above. Then there is the use of const_cast. And even if we solve all this, the API that libosmium provides just can't be implemented suitably with the new proj API. We need a better solution.


Regression from d2b75b3: pre-existing disambiguate files are not accounted for

cc @LMannarino
d2b75b3 changed the names of the files during disambiguation. This is a backwards-incompatible change, however, which means that pre-existing runs will have to run through disambiguation again, even if it was done already.

Luca -- sorry about the extra re-run work here. Do you have an example directory from an old run and what the names we should look for are? Apologies, I'm not very familiar with the disambiguate code and the re-write work from @tetianakh. It looks like she tried to improve the naming of files but we could add a check for the old case to skip all the additional work and limit the re-running. Thanks for starting this discussion.

The new files are in work/align/<name>/disambiguate_bwa and they have names like human, explant and ambiguous.
The old ones were in the <name> folder and they had suffixes like disambiguatedSpeciesA.sorted.

Luca -- thanks for this, I've been reading through the changes more and am thinking about reverting this to the more general previous version. Some of the changes are cleaner but in general it does change things quite drastically which I can to do.
@schelhorn -- the disambiguation changes came in as part of Tetiana's Ericscript work and I'm not entirely sure if there are specific needs it addresses for you or if it was a general cleanup. Thanks for any background help and I'll work to tackle resolving it.

Brad, I would vote for a revert: I just noticed that the presence of BAM indices is not accounted for, so the files aren't indexed (or the index is not correctly symlinked) causing failures due to "missing" BAM indices.

General cleanup only, I believe. Going back to a previous version should be fine. Thanks for the integration.

Luca and Sven-Eric;
Thanks so much for the feedback and thoughts. I've reverted this back to the previous version prior to integration. This should now be fully back-compatible and behave as previously. Thanks again for the help talking through the pros and cons here.


clusterisation during setRegion

Hello. I have a problem. I have 10 -30 points in one city. When user tap on point
(void)mapView:(MKMapView *)mapView didSelectAnnotationView:(MKAnnotationView *)view
I call ... [self.mapView setRegion:region animated:YES]; ..
And after this clusterisation works. But every time with different values, because region is different a bit. And this looks not good - after each user's selection clusters are redrawing.
but! When user just swipe map regionDidChangeAnimated works too, but clusters are the same!

Could you please try out this recipe and see if this helps?

Which recipe did you mean?

Check the link

O_o
[self.mapView setVisibleMapRect: instead of [self.mapView setRegion works like I want. Thank You!


TTA callback fix

Fixes: AttributeError: 'Learner' object has no attribute 'values' when running TTA on a new learner.
Looks like event.begin_fit doesn't happen if inner=True

I think it should only been done once at the beginning and then have a try for all the rest and
except CancelFitException:             self('after_cancel_fit')
finally:                               self('after_fit')

at the end.

Ah good point. Forgot get_preds should be included too - 40436de
Feel free to close this and push the correct fix - if that's easier.

Thanks!


Search by title, link is true but no title and rate

Hi,
When i try this:
$oIMDB = new IMDB('Scary movie 3');
if ($oIMDB->isReady) {
    echo '
getUrl() . '">' . $oIMDB->getTitle() . ' got rated ' . $oIMDB->getRating() . '
';
} else {
    echo '
Movie not found!
';
}

Output showing me: n/A got rated n/A
But link is true.

I tested exactly your snippet and got the following output:

Scary Movie 3 got rated 5.5

Sorry, there is nothing I can (have to) fix.

Thanks for respond. I'm on localhost, all of other names working fine except "Scary movie, Scary movie 2, Scary movie 3". Not sure why. Anyway thanks ;)

Enable debug and post this output. Without an error I just can't help :(


Suspected incorrected calculation of buffer value for live edge

https://github.com/CDrummond/lms-material/blob/691accf0d045e5592929f05c2e1b3f7ea050d89b/MaterialSkin/HTML/material/html/js/nowplaying-functions.js#L86
I believe the current time should be included in the calc, so shouldn't that be? :
   let pc = view.playerStatus.current && undefined!=view.playerStatus.current.duration && view.playerStatus.current.liveEdge && view.playerStatus.current.duration>0
                 ? 100*Math.floor((view.playerStatus.current.liveEdge+view.playerStatus.current.time)*1000/view.playerStatus.current.duration)/1000 : 100.0;

        if (pc!=view.playerStatus.current.bufpc) {
            view.playerStatus.current.bufpc = pc < 100 ? pc : 100.0;
        }

Ah, thought live_edge was from 0. Code updated. Is it correct now?

Yeah, it;s a bit confusing, I know.  Its the distance from the live edge.   So for the buffer on the progress bar  needs to include the elapsed time as its total buffer, not just additional buffer..
Therefore its also possible for it to be over 100 percent,  so I added :
          view.playerStatus.current.bufpc = pc < 100 ? pc : 100.0;

All OK now? Seems to work for me. Slight oddity when seeking, but only for a second or so - probably things slightly out of sync.

Yes, all looks good.
BBC Sounds was a bit odd as well, so I've fixed that and will be doing another release.  The 2 changes together make it work well👍

Ah,  the buffer value doesn't appear to be updating at the same time as the time is updating.
Does it need to be triggered as the time updates?

live_edge is only updated on status messages. But Material has a timer that fires every second to update the playback position. I've updated the code to also set buffer % every second.
However, I've noticed that live_edge is not updated when not playing? I would expect it to be. I tested by pausing a stream and refreshing Material in the browser - live_edge is always same value. But if I the press play a new value is sent.

I've added a timer to refresh status every 10seconds if stream has live edge and not currently playing. This is when I would expect to see the buffer grow - but it does not.

Ah yes,  good point, thanks.  I'll see about correcting that.

Let me know when you do, and I'll close this. If its not something you can do I'll remove the 10second timer.

Yes, that's fine, I can make that change.

This is now working with BBC Sounds 2.35.12.  Closing.


respect 332 Need account for login response during authorization

Fixes #642
Logs after fix:
# ConnectAsync()
Status:   Connecting to ***
Response: 220 *** FTP server (Version <IP_ADDRESS> - 2003/05/11 20:39:33) ready.
Command:  USER ***
Response: 331 Password is required
Command:  PASS ***
Response: 332 Account is required
Command:  ACCT ***
Response: 230 ***Logon ok
Command:  FEAT
Response: 500 'FEAT': command unrecognized.
Status:   Text encoding: System.Text.ASCIIEncoding
Command:  SYST
Response: 215 UNIX Type: L8

# GetWorkingDirectory()
Command:  PWD
Response: 257 "***" is the current directory.

Thank you for this!

Released! https://www.nuget.org/packages/FluentFTP/33.0.3


Merge Release/2.13.0

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

Project coverage is 69.97%. Comparing base (eb6c153) to head (8c60347).
Report is 37 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2250      +/-   ##
==========================================
- Coverage   70.14%   69.97%   -0.17%
==========================================
  Files         727      727
  Lines       27051    27036      -15
  Branches     4558     4555       -3
==========================================
- Hits        18973    18916      -57
- Misses       6816     6840      +24
- Partials     1262     1280      +18

Files with missing lines
Coverage Δ

...id/core/internal/data/upload/DataOkHttpUploader.kt
90.43% <100.00%> (-0.30%)
:arrow_down:

.../android/core/internal/data/upload/UploadStatus.kt
88.46% <ø> (ø)

.../android/core/internal/data/upload/UploadWorker.kt
73.68% <100.00%> (-12.80%)
:arrow_down:

...dog/android/core/internal/metrics/RemovalReason.kt
91.67% <100.00%> (ø)

... and 30 files with indirect coverage changes


Roadmap for currently unsupported scenarios?

Are there any plans to support any of the currently unsupported scenarios in the future? If so, is there any publicly viewable roadmap? Particularly interested in RDP and loging in to a server using a security key. Thanks!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8b7e1d9a-fdaf-0a29-0746-e3a32826eb6b
Version Independent ID: 78e9b684-1121-6193-387f-fd8b6040cc0a
Content: Passwordless security key sign-in Windows - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-authentication-passwordless-security-key-windows.md
Service: active-directory
Sub-service: authentication
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@chrisoc5 Thanks for your feedback! We will investigate and update as appropriate.

@chrisoc5 We do not have that roadmap available for these specific features, however, you can get the latest updates on Azure updates for any in development, upcoming or in GA features.

@chrisoc5 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Liquidity provision link in orders datagrid

Story
As a liquidity provider
I want to be able to click and see the full liquidity provision
So that I know what orders belong to what provision
Acceptance Criteria

[ ] I can click through to the relevant liquidity provision from the order list

Tasks

[ ] Dev
[ ] Test

Sketch
Additional details / background info
I've broken this issue out from #1716

Suggest rendering a link in the type column of the order list if the order is an LP order. We might need to add the ID of the liquidity provision to the order query


fix: set planetscale's href to the correct link

Thanks for the PR! I just made the fix for the next commit.

o7


Validation Layer name change to "VK_LAYER_KHRONOS_validation"

VK_LAYER_LUNARG_standard_validation was deprecated in favor of VK_LAYER_KHRONOS_validation.
While they both have the same functionality (as its the same code under the hood) the former was re-architected to have less overhead, and VK_LAYER_LUNARG_standard_validation will be removed in a future SDK version.

I'm not sure what you're referring to, this project doesn't make use of the validation layers itself, it's up to the user. Please reopen this if you feel like this was a mistake

I'm not sure what you're referring to, this project doesn't make use of the validation layers itself, it's up to the user. Please reopen this if you feel like this was a mistake

I see my mistake now, I didn't note that it was the README.md which has the issue, not the project itself. You are very right, the project makes validation layers a user done option. If this project targets SDK versions 1.1.121 or later, I would suggest updating the readme.

I see my mistake now, I didn't note that it was the README.md which has the issue, not the project itself. You are very right, the project makes validation layers a user done option. If this project targets SDK versions 1.1.121 or later, I would suggest updating the readme.

Thanks for spotting this, I've updated the readme

Thanks for spotting this, I've updated the readme

No problem, and apologies for not making that clear in the issue post. I momentarily forgot other people can't read my mind it seems.

No problem, and apologies for not making that clear in the issue post. I momentarily forgot other people can't read my mind it seems.


Group overrides for treesitter don't seem to work

My configuration is very simple for this plugin:
local colors = require('vscode.colors').get_colors()
require('vscode').setup({
  group_overrides = {
    ['@property'] = { fg = '#DCDADC', bg = colors.vscNone },
    ['@interface'] = { fg = '#B8D7A3', bg = colors.vscNone },
    ['@structure'] = { fg = '#86C691', bg = colors.vscNone },
  }
})

I'm coming from VS22 and use this colorscheme since it's the most similar to it. Only the three changes above would make it pretty much perfect. But They don't seem to get applied at all.
Am I missing something?

Try using @lsp.type.interface and equivalent

@Kyykek that worked, thanks for the help!


README's "A quick example" get error.

"from tensornets import utils
img = utils.load_img('cat.png', target_size=256, crop_size=224)
assert img.shape == (1, 224, 224, 3)"
This script get error.

AttributeError                            Traceback (most recent call last)
 in ()
1 from tensornets import utils
----> 2 img = utils.load_img('cat.png', target_size=256, crop_size=224)
3 assert img.shape == (1, 224, 224, 3)
~/.conda/envs/tensor/lib/python3.6/site-packages/tensornets/utils.py in load_img(path, grayscale, target_size, crop_size, interp)
176         if isinstance(target_size, int):
177             hw_tuple = tuple([x * target_size // min(img.shape[:2])
--> 178                               for x in img.shape[1::-1]])
179         else:
180             hw_tuple = (target_size[1], target_size[0])
AttributeError: 'NoneType' object has no attribute 'shape'

got the same under windows i copy one of my pic and defined the complete path to find the image. worked well

@inoue0426, It doesn't seem to be a TensorNets issue. You can try the sample codes on the directory where the cat.png exists. Or, you can also give the complete path as @alain2208 said.