Adding CSS class to geojson file

I have a simple request, I don't know if it exists or not but, I want to add ripple effect to my geojson file.
I have a list of points like this:
"type": "FeatureCollection",
   "features": [
       {
           "type": "Feature",
           "properties": {
               "scalerank": 8,
               "natscale": 10,
               "labelrank": 3,
               "name": "Vatican City",
               "namepar": null,
               "namealt": null,
               "diffascii": 0,
               "nameascii": "Vatican City",
               "adm0cap": 1,
               "capalt": 0,
               "capin": null,
               "worldcity": 1,
               "megacity": 0,
               "sov0name": "Vatican (Holy Sea)",
               "sov_a3": "VAT",
               "adm0name": "Vatican (Holy Sea)",
               "adm0_a3": "VAT",
               "adm1name": "Lazio",
               "iso_a2": "VA",
               "note": null,
               "latitude": 41.9000122264,
               "longitude": 12.4478083889,
               "changed": 4,
               "namediff": 0,
               "diffnote": "Changed scale rank.",
               "pop_max": 832,
               "pop_min": 832,
               "pop_other": 562430,
               "rank_max": 2,
               "rank_min": 2,
               "geonameid": 6691831,
               "meganame": null,
               "ls_name": "Vatican City",
               "ls_match": 1,
               "checkme": 0,
               "featureclass": "Admin-0 capital"
           },
           "geometry": {
               "type": "Point",
               "coordinates": [
                   12.453386544971766,
                   41.903282179960115
               ]
           }
       },

and I just want to add ripples like THIS to the coordinates in this geojson, as you can see here I have coordinates of Vatican city and my style is:
export const geojsonTheme = [
    {
        when: "$geometryType == 'polygon'",
        technique: "fill",
        renderOrder: 10000,
        attr: {
            color: "#7cf",
            transparent: true,
            opacity: 0.8,
            lineWidth: 1,
            lineColor: "#003344"
        }
    },
    {
        when: "$geometryType == 'polygon'",
        technique: "solid-line",
        renderOrder: 10001,
        attr: {
            color: "#8df",
            metricUnit: "Pixel",
            lineWidth: 5
        }
    },
    {
        when: "$geometryType == 'point'",
        technique: "circles",
        renderOrder: 10002,
        attr: {
            size: 10,
            color: "#5ad",

        },

    },
    {
        when: "$geometryType == 'line'",
        technique: "solid-line",
        renderOrder: 10000,
        attr: {
            color: "#8df",
            metricUnit: "Pixel",
            lineWidth: 5
        }
    }
];

precisely it used the 3rd geometryType which is point and it renders a blue point, any way to make that point ripple? in any way possible?

@sirakc , thanks for the issue.
I spoke with a colleague @robertoraggi and this should be possible.
You can use a dynamic-property, i.e. you would set the size to be something like: ”size”: [“get”, “size”, [“dynamic-properties”]]
And then you can set the size as: map.setDynamicProperty(“size”, Math.sin(time) * 5);
Does that work?

Hi @nzjony ! I am a bit new to the geojson world, can you guide a little more?, my project is pretty simple right now, just a  globe with a theme on it, just like harp.gl's example on geojson and now just like you said, changing the size of the circle will make it look like a ripple, so the question is, where and how to update it,  like do I put the map.setDynamicProperty(“size”, Math.sin(time) * 5); do I put it inside an event that updates the map? I have an event that i use to update the map and make it rotate
map.addEventListener(MapViewEventNames.AfterRender, () => {
        options.target = new GeoCoordinates(0, options.target.lng + 0.1);
        map.lookAt(options);
        map.update();

    });

so I am trying this from my side and my findings will be coded here and if you have time please do share an example, so that I can get an ideas of properties and where to put them, also do share if you know any docs/reference guide to these properties that are changable, like
{
when: "$geometryType == 'point'",
technique: "circles",
renderOrder: 10002,
attr: {
size: 10,
color: "#121275",
    },

}

so that i can know what properties can be added

@sirakc , would he happy to help.
Best change AfterRender to Render, i.e. you set the lookAt and the dynamic property before rendering.
Also, you don't need the map.update() in the event listener. You can, outside of the event listener call map.beginAnimation() onece and then at least you will get the render calls without having to manually call update.
In your case, you would change the value 10, with the expression: [“get”, “size”, [“dynamic-properties”]], then you would need to set the value each frame in the Render event listener.

@nzjony I did this and it worked changed the animation, things are running smoother.
I would like to know if there is an official method for adding markers something you guys recommend, or change the render of the geojson points, you know how the geojson refers to the technique of rendering the geojson coordinates
{
when: "$geometryType == 'point'",
technique: "circles", //<-- HERE! referring to make a circle
renderOrder: 10002,
attr: {
size: ["get", "size", ["dynamic-properties"]],
color: "#121275",
},
},
so can we change that to a separate three.js object or something? like a square

@sirakc , what do you mean by Markers? Do you mean 2D icons? Take a look at: https://github.com/heremaps/harp.gl/issues/1718#issuecomment-664254883
We also have the idea of mapAnchors on the MapView, see: https://www.harp.gl/docs/master/examples/#threejs_animation.html, if you have a separate three.js object, you can add it here.
Does that help, or did I misunderstand your question?

@nzjony  hey, a bit late reply, yea 2D icons is what I am talking about what are those 2D icons made of? html tags? three js objects?
cause then we can find out what can be done with that icon, what property can be changed

I found this newly added example to the latest harp.gl version of August 23,2020,
https://www.harp.gl/docs/master/examples/#getting-started_hello-world_npm.html
this example shows how to add markers over json data

Thanks @ninok  for the example!


docs: document rate limiting and retry for pipeline promotions

What changed?
Documentation

Why was this change made?
To document Helm options for Promotion with Pipelines.

How was this change implemented?
NA

How did you validate the change?
I run and check the site with yarn start

Release notes
Exclude from release notes

Documentation Changes
This is the documentation change.
References:

https://github.com/weaveworks/pipeline-controller/pull/147

I don't know how to resolve that snyk alert. Those dependencies are dependencies of docusaurus and I coulnd't find a way to update those dependencies without updating docusaurus, and docusaurus has no updates yet.

I don't know how to resolve that snyk alert. Those dependencies are dependencies of docusaurus and I coulnd't find a way to update those dependencies without updating docusaurus, and docusaurus has no updates yet.

It's fixed now.


[9.x] Allow jsonSerialize() to return other types

This PR updates jsonSerialize()'s return type to mixed, matching PHP's definition.
Use case
Allows models to override jsonSerialize, and serialize into strings rather than always having to be an JSON object.
Example
We have a simple model, Permission, who's only attribute is name:
class Permission extends Model
{
    protected $fillable = [
        'name',
    ];
}

When we fetch all permissions, the collection is automatically serialized into JSON objects.
Controller
class PermissionsController
{
  public function list()
  {
     return Permission::all();
  }
}

Would currently return
[
  {name: "foo"},
  {name: "bar"},
  {name: "baz"},
]

With this change, Permission, can override jsonSerialize, and return a string.
Updated response after this PR:
["foo", "bar", "baz"]

Ping @X-Coder264 this is what I meant on that pr your sent in ^

Updated it to be a union type with string instead. JsonSerializable only specifies that the return type has to be compatible with json_encode, which a string is.

With array|string it won't be possible to override jsonSerialize and return an integer, a float or a boolean, right? json_encode's $value can have any type but a resource.

@X-Coder264 my ping was more about you being okay with this change?

@driesvints There's nothing intrinsically wrong with this change. From my perspective it's a bit weird for a collection (which is just arrays on steroids) to be JSON serialized as anything else but an array. But if people want to do that I guess there's no reason from a framework perspective to block that so on that level I'm fine with the proposed change.

@X-Coder264

it's a bit weird for a collection (which is just arrays on steroids) to be JSON serialized as anything else but an array.

It's actually the Collection item's jsonSerialize that I'm hoping to override. The collection can (and should?) still expect an array to be returned.
I've updated it to still return an array for collection in: https://github.com/laravel/framework/pull/42133/commits

If we do this then I suggest we move to the mixed return type like the PHP interface and not bother with union types.


ColorPickerButton.text not visible?

Godot version
v3.x +
System information
Windows 11, Core i5-12400F, RX 6600
Issue description

Is there some way to make the text property of a ColorPickerButton show?
The text seems to always be beneath. No matter size or min size etc...
You can see a couple of pixels of text poking out here at default size is the most I can see.
Thanks.
Steps to reproduce

make a color picker with text

Minimal reproduction project
n/a

The button's text and icon are drawn first, in the Button class, the ColorPickerButton base class. The background texture (checkerboard pattern) and the color rectangle are drawn later and therefore overlap it.
In theory, ColorPickerButton could be inherited directly from BaseButton, since the text and icon are not visible anyway. But then we would have to duplicate the implementation of the button styles. In any case, this is not subject to change in 4.x for compatibility reasons.
I'm not sure if there's an easy way to fix this other than draw the text and icon again on the ColorPickerButton. Perhaps we should just document this limitation?

Will this be fixed in a future version?
This still does not work in 4.3-dev6.


AcceleratorParserStage is not compatible with Stream Accelerator objects

Currently, stream uses the zigzag AcceleratorParserStage stage, which either accepts a string or a ZigZag's Accelerator object.
If the field accelerator is a ZigZag's Accelerator, then the stage finishes without errors but later stages error out. If a Stream's Accelerator object is passed, the stage does not recognize such object as an Accelerator.

Hi @Andful,
Is this still a problem or can I close this issue?

I personally steered away from using an accelerator object and have been using accelerators defined in a separate path. So, to answer your question, personally it has not been an issue for me.

This has been fixed in a recent ZigZag update


Update manage-loop-component.md to clarify Fluid is default on

Only modification was changing sentence ""Microsoft Fluid Framework defaults to ON for all targeted release organizations." to "Microsoft Fluid Framework defaults to ON for all organizations.". Original sentence is outdated and doesn't reflect the current state of the experience.

@MikePlumleyMSFT PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


feat: create a custom form for stripe payments

fixes #334
Changes: More padding added to make it comparable with Eventbrite, Changed the accent color from pink to light blue, postal code and country added to the card object
Screenshot:

@iamareebjamal moved the logic to the viewModel. Please review

@nikit19 travis is failing


Add panels showing visual and olfactory experience to save_video by default

Get rid of save_video_with_vision_insets and add the same logic to NeuroMechFly.save_video by default
Do the same for olfaction
Update the tutorials accordingly

Closed due to inactivity


ci: unify imports aliases via importas

What this PR does / why we need it:
This PR addresses inconsistent import alias naming throughout the code base.
The proposed aliases and imports set are subject for debate.

LGTM.
This and other lint configurations we've added recently should probably get fanned out to our other repositories as well.

I was thinking about using a sync action that would propagate changes from a "template repo" (in quotation since I don't think there's an option to make a repo be based of a template post-creation).
So basically we could create an opinionated k8s version 😆 of https://github.com/Kong/template-generic and make this sync to designated repos like it's done in: https://github.com/Kong/template-generic/blob/175a2db550d41bae4af8a9dbd6a2bf85ecb2d8cd/.github/template-sync.yml#L2-L6


rdns=False never achieved since   dest_host variable is not used

https://github.com/romis2012/python-socks/blob/d657fa1ef467cf9fefdf8eaa23123e3b4d3374eb/python_socks/_connectors/socks5_async.py#L59
dest_host variable holding resolved domain is never used.
In case rdns=False  domain will be resolved locally but original domain will be used anyway when connecting through proxy,
so there will be another DNS resolution made by proxy.
Should this be fix ?  _, host = await self._resolver.resolve(

Fixed in v2.4.4. Thanks!


Fixed margin-bottom as a result of introducing the strong tag to the …

Fixed additional panel as a result of introducing the strong tag. Last child was not working as a result of this addition.
Also fixed the incorrect colour being used for the success panel
Before

After

 #1353 Remove unwanted padding from warning panel and fix tick icon colour


Biowulf resources location changed

/data/CCBR_Pipeliner/Exome-seek/ no longer exists for both mm10/hg38 resources.

@dnousome have moved(hard-linked) the data to /data/CCBR_Pipeliner/XAVIER/resource... Can we close this?

Fixed with hardlinks to original resources locations


Integrated terminal tries to read '/proc/1/environ'

VSCode Version: 1.39.0
OS Version: Arch Linux

Steps to Reproduce:

Disable "terminal.integrated.inheritEnv"
Run code from cli
Start a new ternimal

Does this issue occur when all extensions are disabled?: No
This function tries to read environ variables from the parent of the main process, however the ppid of code procces is 1 (init), which is diened for normal user process.

/duplicate https://github.com/microsoft/vscode/issues/76542


Result given by mxnet.model.FeedForward.predict() in Wrong Size

It seems that when the number of data points is not divisible by the numpy_batch_size, mxnet.model.FeedForward.predict() will give results in wrong size - either larger or smaller. In order to get the right result, I can only load model by specifying numpy_batch_size = 1. Is this a bug?
In addition, when using mxnet.io.NDArrayIter, if the numpy_batch_size does not equal to that of the loaded model, it will result in errors. It is kind of weird that one has to specify a "batch size" when loading a model or creating a data iter and the two sizes have to be the same.

Hmm, this seems to be not the right behavior. I guess it was a bug in the arrayiter.
Thanks for the catch. Can you check if you change https://github.com/dmlc/mxnet/blob/master/python/mxnet/io.py#L192 to self.current_batch == self.batch_num instead, the result will become correct. If so, please submit a PR :)

@tqchen Line 192 is correct and shouldn't be changed. The problem is line 160 https://github.com/dmlc/mxnet/blob/master/python/mxnet/io.py#L160. The correct padding size is self.num_pad = batch_size - data.shape[0] % batch_size. This should be clear with very simple calculation :)
Besides, the if condition may not be necessary, since batch_size - data.shape[0] equals to batch_size - data.shape[0] % batch_size when data.shape[0] <= batch_size.
If the above is agreed, I will submit a PR then.

Yes you are right! Please submit a PR on this. Thanks!

I am so happy finding the following useful links:

How do you update a GitHub repo fork in the web interface?
Development workflow with Git: Fork, Branching, Commits, and Pull Request
Create a new branch with git and manage branches

I am accumulating experiences of contributing in Github. ^_^
I will try to update contribute-to-mxnet steps some time later.

That is great. Thanks for the contribution. I am closing this due to #372

I am more than pleased to be able to make some contributions.
Sure, I was waiting for you to close this issue :-)


yii\mongodb\Exception contains non utf-8 character

What steps will reproduce the problem?
in chinese win7, if mongo is unavailable
What's expected?
What do you get instead?
yii\mongodb\Exception:Failed to connect to: <IP_ADDRESS>:27018:
�������ӷ���һ��ʱ���û����ȷ�𸴻����ӵ�����û�з�Ӧ�����ӳ���ʧ�ܡ�
i need to encode the  yii\mongodb\Exception message into json, but the string contains non utf-8 character. change it into utf-8.
Additional info

Q
A

Yii version
2.1

Yii MongoDB version
newest

MongoDB server version
no mongodb

PHP version
5.6

Operating system
win7, XAMPP

Yii does not perfrom any encoding conversion for the Exception messages they are return as they are.
If you need a specific representation for it - you should handle it yourself.


Blog on #740

What would you like to be added:
A blog on the memory leak issues worked on in falcosecurity/falco#740
Why is this needed:
To share a high level digest with the community

/area blog
/kind content

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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


Drop usage of deprecated 'render_to_response' method

Well, this is basically the same as #3 with additionnal code formatting.


Increase test coverage to approximately 80%.

This issue involves choosing what test framework & runner to use and then using those tools to increase test coverage. We will probably use Mocha + Chai + Karma. Here are some testing packages:

Karma
Jasmine
Mocha with Chai and Sinon.
Jest
QUnit

Annotated Resources
https://stackoverflow.com/questions/26032124/karma-vs-testing-framework-jasmine-mocha-qunit
https://stackoverflow.com/questions/9710297/jasmine-vs-mocha-javascript-testing-for-rails-3-1

(Mocha + Chai) seems equivalent to Jasmine.

https://npmcompare.com/compare/jasmine,mocha,qunit

Mocha is the most popular, Jasime follows it, QUnit is quite far behind in popularity.

https://stackoverflow.com/questions/23272521/difference-between-karma-and-mocha

Karma appears to provide more features including spawning web browsers.

https://developer.telerik.com/featured/journey-client-side-testing-javascript/

Testing Frameworks: Mocha, Jasmine, Chai. suite, describe, test, it let us group tests into suites.
Assertion Library: Assert (comes with NodeJS), Chai, Expect, should.js...
Test runner: These wrap test suites to be run in specific contexts (e.g. Web browser).

Roll our own (e.g. TestRunner.html that imports mocha.js and calls mocha.run())
Additionally, call TestRunner.html with Phantom.js
Instead of rolling our own, use Karma because it provides runner features.

Closed by https://github.com/shaunluttin/aurelia-open-id-connect/pull/37 and https://github.com/shaunluttin/aurelia-open-id-connect/pull/36


helios-testing shaded artifact is broken

#1137 added a "shaded" artifact for helios-testing to attempt to relocate Guava classes to a new package, so that users of helios-testing are not impacted whenever helios updates the Guava version it uses (which tends to cause a chain of headaches because of version incompatibilities).
The shaded helios-testing jar appears to be broken and unusable, at least for users of HeliosSoloDeployment: a NoSuchMethodError is thrown in a simple test case like
private static final HeliosDeploymentResource soloResource =
      new HeliosDeploymentResource(HeliosSoloDeployment.fromEnv().build());

java.lang.NoSuchMethodError: com.spotify.docker.client.messages.NetworkSettings.ports()Lcom/spotify/helios/testing/shaded/com/google/common/collect/ImmutableMap;
  at com.spotify.helios.testing.HeliosSoloDeployment.getHostPort(HeliosSoloDeployment.java:462)
  at com.spotify.helios.testing.HeliosSoloDeployment.<init>(HeliosSoloDeployment.java:141)
  at com.spotify.helios.testing.HeliosSoloDeployment$Builder.build(HeliosSoloDeployment.java:879)
  at com.spotify.foobar.ServiceTestIT.<clinit>(ServiceTestIT.java:27)

This looks to be due to the fact that the maven-shade-plugin is rewriting the bytecode for the getHostPort method in com.spotify.helios.testing.HeliosSoloDeployment to look like the following (this is from the 0.9.146 JAR):
private java.lang.String getHostPort(java.lang.String, int) throws com.spotify.helios.testing.HeliosDeploymentException;
  descriptor: (Ljava/lang/String;I)Ljava/lang/String;
  flags: ACC_PRIVATE
  Code:
    stack=7, locals=7, args_size=3
      ...
      35: invokevirtual #733                // Method
      38: invokevirtual #737                // Method com/spotify/helios/testing/shaded/com/google/common/collect/ImmutableMap.entrySet:()Lcom/spotify/helios/testing/shaded/com/google/common/collect/ImmutableSet;
      41: invokevirtual #742                // Method com/spotify/helios/testing/shaded/com/google/common/collect/ImmutableSet.iterator:()Lcom/spotify/helios/testing/shaded/com/google/common/collect/UnmodifiableIterator;

This last 3 lines are the problem - it is trying to invoke a method named com/spotify/docker/client/messages/NetworkSettings.ports with no arguments (()) which is expected to return com/spotify/helios/testing/shaded/com/google/common/collect/ImmutableMap.
It seems as if the plugin does not realize that in code like this
final NetworkSettings settings = dockerClient.inspectContainer(containerId).networkSettings();
for (final Map.Entry<String, List<PortBinding>> entry : settings.ports().entrySet()) {

the invokevirtual operations that invoke the com.google.common.collect.ImmutableMap NetworkSettings.ports() method should not be rewritten to return com.spotify.helios.testing.shaded.com.google.common.collect.ImmutableMap since this class is supplied from another library and will never actually return this rewritten type.
In looking at this, I noticed that we are using version 2.4.3 of maven-shade-plugin and that 3.0.0 exists, I'm not sure yet if upgrading there would fix this.
On the other hand, upgrading docker-client so that the NetworkSettings.ports() method does not return a type from the com.google.common.collect package (#1138 or #1141) should also fix this.
But in general simply relocating Guava classes is not as simple as thought and might have to be scrapped as unworkable.

Confirmed that this is fixed in #1141 (if merged):
        35: invokevirtual #733                // Method com/spotify/docker/client/messages/NetworkSettings.ports:()Lcom/spotify/docker/client/shaded/com/google/common/collect/ImmutableMap;
        38: invokevirtual #739                // Method com/spotify/docker/client/shaded/com/google/common/collect/ImmutableMap.entrySet:()Lcom/spotify/docker/client/shaded/com/google/common/collect/ImmutableSet;
        41: invokevirtual #744                // Method com/spotify/docker/client/shaded/com/google/common/collect/ImmutableSet.iterator:()Lcom/spotify/docker/client/shaded/com/google/common/collect/UnmodifiableIterator;

On the other hand, upgrading docker-client so that the NetworkSettings.ports() method does not return a type from the com.google.common.collect package should also fix this.

We can return the built in Collection types, but AutoValue docs recommend returning the Immutable types. See https://github.com/google/auto/blob/master/value/userguide/howto.md#mutable_property

I think they mean that as a general principle and not something directly related to using auto-value.
I started on a branch to reduce our usage of Guava in the public API of docker-client, but it might break a lot of methods on classes in the 'messages' package that use the immutable Builders or offer varargs setters.

Can I close this issue since the original problem is now fixed?

I think they mean that as a general principle and not something directly related to using auto-value.

Are you sure? The example they give here has ImmutableList as the return type of the getter.
@AutoValue
public abstract class ListExample {
  public static ListExample create(String[] mutableNames) {
    return new AutoValue_ListExample(ImmutableList.copyOf(mutableNames));
  }

  public abstract ImmutableList<String> names();
}

@davidxia my point was more that it is a recommendation and not a requirement when using auto-value, and we don't have to follow it if it leads to a problematic design like in this case.
I'm not sure if this is still a problem or not, I haven't tried building a new project that uses helios-testing:shaded since #1141 was done - there might be other problems.

To elaborate a little bit:
Returning Guava types (ImmutableList, ListenableFuture, etc.) from the public API surface of helios-testing or docker-client means that the users of these libraries have to be aware of Guava too. This has caused a lot of pain internally for projects that use a number of clients that depend on differing Guava versions, since there have been a number of incompatibilities between recent Guava versions recently.
Trying to alleviate this by shading and relocating the Guava classes in helios-testing or docker-client would seem like an obvious solution, but if the public API is returning Guava types, then these uses get relocated too. This means that someone using the normal helios-testing passes in a com.google.guava.whatever.ImmutableList to a docker-client/helios-testing method, but if they want to use helios-testing:shaded then now they have to change their code to pass in a com.spotify.helios.relocated.com.google.guava.whatever.ImmutableList instance. This makes it harder for someone to switch to using the shaded artifact as they have to update their code and not just the coordinates of the artifact in their pom. It gets more complicated if helios-testing (regular) is to use docker-client:shaded. So in the end the relocation is still painful to users, as the use of Guava is not purely internal to the library.
For a value class like NetworkSettings I think it is a bad idea (from the lessons we've learned with all the Guava upgrade pains internally) to return Guava collection types as opposed to the core types like Set or List for just this reason. It also makes it harder to ever decide to remove Guava from the library, as now that is a breaking API change for users.
Should note that if you decide to not expose fields as Immutable/Guava types, you can still use them internally in the auto-value static factory - accept a List and wrap it in a ImmutableList.copyOf() - without exposing this to users.
Also for a class like NetworkSettings I don't think there is any reasonable expectation that a user could add elements to a field like List ports(), especially since I don't think the client ever accepts a user-supplied NetworkSettings as an input to an operation. I would be a bigger fan of communicating immutability with javadocs or annotations rather than using the Guava type.

Makes sense. Should we change the return types to regular collections? Did
you have a branch somewhere?
On Wed, Nov 15, 2017 at 8:47 AM Matt Brown<EMAIL_ADDRESS>wrote:

To elaborate a little bit:
Returning Guava types (ImmutableList, ListenableFuture, etc.) from the
public API surface of helios-testing or docker-client means that the users
of these libraries have to be aware of Guava too. This has caused a lot of
pain internally for projects that use a number of clients that depend on
differing Guava versions, since there have been a number of
incompatibilities between recent Guava versions recently.
Trying to alleviate this by shading and relocating the Guava classes in
helios-testing or docker-client would seem like an obvious solution, but if
the public API is returning Guava types, then these uses get relocated too.
This means that someone using the normal helios-testing passes in a
com.google.guava.whatever.ImmutableList to a docker-client/helios-testing
method, but if they want to use helios-testing:shaded then now they have to
change their code to pass in a
com.spotify.helios.relocated.com.google.guava.whatever.ImmutableList
instance. This makes it harder for someone to switch to using the shaded
artifact as they have to update their code and not just the coordinates of
the artifact in their pom. It gets more complicated if helios-testing
(regular) is to use docker-client:shaded. So in the end the relocation is
still painful to users, as the use of Guava is not purely internal to the
library.
For a value class like NetworkSettings I think it is a bad idea (from the
lessons we've learned with all the Guava upgrade pains internally) to
return Guava collection types as opposed to the core types like Set or
List for just this reason. It also makes it harder to ever decide to
remove Guava from the library, as now that is a breaking API change for
users.
Should note that if you decide to not expose fields as Immutable/Guava
types, you can still use them internally in the auto-value static factory -
accept a List and wrap it in a ImmutableList.copyOf() - without exposing
this to users.
Also for a class like NetworkSettings I don't think there is any
reasonable expectation that a user could add elements to a field like List
ports(), especially since I don't think the client ever accepts a
user-supplied NetworkSettings as an input to an operation. I would be a
bigger fan of communicating immutability with javadocs or annotations
rather than using the Guava type.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spotify/helios/issues/1142#issuecomment-344596815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdVbeCcf4eejaeUStFkGAOBg5sJ5-Knks5s2utggaJpZM4O5fkl
.

--
David Xia
Software Engineer
<EMAIL_ADDRESS>

No, I hadn't started on that. That type of change might be considered breaking, since code like the following would fail to compile:
ImmutableList<?> ports = networkSettings.ports() // if ports() now returns java.util.List

Yea we can bump the major version. Seems like solving the relocation
problem might be worth it to some users.
David Xia
Software Engineer
<EMAIL_ADDRESS>

I would’ve never followed the autovalue examples of returning immutable
types if I knew it was going to be such a headache later on.
David Xia
Software Engineer
<EMAIL_ADDRESS>


Set ctx ns before publish

Fixes #1497
This sets the namespace on the context when deleting a namespace so that
the publish event does not fail.
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1509   +/-   ##
=======================================
  Coverage   40.82%   40.82%
=======================================
  Files          23       23
  Lines        2920     2920
=======================================
  Hits         1192     1192
  Misses       1451     1451
  Partials      277      277

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 9f28040...14c36cc. Read the comment docs.

@crosbymichael Thanks for the fix here!


sqllint installation fails for pre-commit

pre-commit log below:
version information
pre-commit version: 2.17.0
git --version: git version 2.17.1
sys.version:
    3.9.5 (default, Dec  1 2021, 10:34:33)
    [GCC 7.5.0]
sys.executable: /home/saurav/.pyenv/versions/3.9.5/bin/python3.9
os.name: posix
sys.platform: linux

error information
An unexpected error has occurred: CalledProcessError: command: ('/home/saurav/.rvm/rubies/ruby-3.0.0-cruby/bin/ruby', '/home/saurav/.rvm/rubies/ruby-3.0.0-cruby/bin/gem', 'build', 'sqlint.gemspec')
return code: 1
expected return code: 0
stdout: (none)
stderr:
    Invalid gemspec in [sqlint.gemspec]: cannot load such file -- pg_query
    ERROR:  Error loading gemspec. Aborting.

Traceback (most recent call last):
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/error_handler.py", line 70, in error_handler
    yield
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/main.py", line 396, in main
    return run(args.config, store, args)
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/commands/run.py", line 416, in run
    install_hook_envs(to_install, store)
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/repository.py", line 224, in install_hook_envs
    _hook_install(hook)
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/repository.py", line 82, in _hook_install
    lang.install_environment(
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/languages/ruby.py", line 132, in install_environment
    helpers.run_setup_cmd(
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/languages/helpers.py", line 52, in run_setup_cmd
    cmd_output_b(*cmd, cwd=prefix.prefix_dir, **kwargs)
  File "/home/saurav/.pyenv/versions/3.9.5/lib/python3.9/site-packages/pre_commit/util.py", line 154, in cmd_output_b
    raise CalledProcessError(returncode, cmd, retcode, stdout_b, stderr_b)
pre_commit.util.CalledProcessError: command: ('/home/saurav/.rvm/rubies/ruby-3.0.0-cruby/bin/ruby', '/home/saurav/.rvm/rubies/ruby-3.0.0-cruby/bin/gem', 'build', 'sqlint.gemspec')
return code: 1
expected return code: 0
stdout: (none)
stderr:
    Invalid gemspec in [sqlint.gemspec]: cannot load such file -- pg_query
    ERROR:  Error loading gemspec. Aborting.

I have also tried with different ruby versions. 2.6.5 as mentioned in the .ruby_version file. Can you help me?
Also this is how my pre-commit-config looks like
  - repo: https://github.com/purcell/sqlint
    rev: 0.2.1
    hooks:
      - id: sqlint
        name: SQL linting

I don't know anything about pre-commit, but it seems to be running an RVM-installed Ruby that does not have the pg_query gem installed. That gem is a dependency for sqlint, which is what the error indicates.

Hey Thanks for the answer. I was able to pinpoint the issue. Apparently rvm-installed ruby gave this env error for me.
https://stackoverflow.com/questions/18276701/getting-warning-path-is-not-properly-set-up-when-doing-rvm-use-2-0-0-defaul
rvm reset
rvm get stable --
rvm @global do gem install pg_query

I followed this to fix it. Will leave this here for anyone else looking. Can be closed.

Another option would be to just gem install sqlint directly and stop trying to get sqlint magically from pre-commit using its Ruby support.


No loading screen is displayed while instrument is loading sounds from XMLHttpRequests into arrayBuffer

Currently unnecessary, sound loading is fast enough to not require a loading screen. May need to reopen if our requests take more than a second.


chore: support customize docker repository name

Type of change:

[ ] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches
[ ] Documentation
[ ] Refactor
[x] Chore
[x] CI/CD or Tests

What this PR does / why we need it:
Currently, the default REGISTRY is localhost:5000 in CI and Makefile, but it does not work on my local machine(MBP with Arm arch). I find if we change the localhost to <IP_ADDRESS>, it works well, so I created this PR to solve the issue to keep it compatible with MacOS.

Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[x] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #1958 (a357e78) into master (43444f8) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a357e78 differs from pull request most recent head d721a03. Consider uploading reports for the commit d721a03 to get more accurate results

@@           Coverage Diff           @@
##           master    #1958   +/-   ##
=======================================
  Coverage   36.87%   36.87%
=======================================
  Files          93       93
  Lines        7851     7851
=======================================
  Hits         2895     2895
  Misses       4568     4568
  Partials      388      388

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hi, @tao12345666333, cloud you help me review this PR? Thanks❤


Grabbing builders by platform-api tag is no longer working

It looks like the n-1 and n-2 platform API tags are no longer working as expected.
When I run pack inspect-builder gcr.io/paketo-buildpacks/builder:tiny-platform-api-n-2 I see the following output
Inspecting builder: gcr.io/paketo-buildpacks/builder:tiny-platform-api-n-2
REMOTE:

Description: Tiny base image (bionic build image, distroless run image) with buildpacks for Golang
..........
Lifecycle:
  Version: 0.7.2
  Buildpack API: 0.2
  Platform API: 0.3

Expected behavior here is to see that the builder platform API is 0.1
It looks passing an explicit platform API tag still works. e.g pack inspect-builder gcr.io/paketo-buildpacks/builder:tiny-platform-api-0.1
Inspecting builder: gcr.io/paketo-buildpacks/builder:tiny-platform-api-0.1

REMOTE:
......................
Lifecycle:
  Version: 0.5.0
  Buildpack API: 0.2
  Platform API: 0.1

Looks like this if fixed now. Closing this.


KeyError in get_candles_by_connector_trading_pair with different exchange and candle exchange configurations

Describe the bug
While using different configurations for the exchange and candle exchange in configuration, a KeyError is thrown in the function get_candles_by_connector_trading_pair. Specifically, this occurs when setting the exchange to binance_perpetual_testnet and the candle connector to binance_perpetual.
Steps to reproduce

Update the Hummingbot script from v2_market-making_dman_v2.py to use a different exchange as shown below:config_v2 = DManV2Config(
    exchange="binance_perpetual_testnet",
    trading_pair="ETH-USDT",
    order_levels=[...],  # Omitted for brevity
    candles_config=[
        CandlesConfig(connector="binance_perpetual", trading_pair="ETH-USDT", interval="15m", max_records=1000),
    ],
    ...  # Additional configuration omitted
)

Run the script.

Observed Result:
Upon the network status changing to connected, an error was thrown as captured in the log:
2023-11-25 18:15:51,560 - 15 - hummingbot.core.rate_oracle.rate_oracle - INFO - Network status has changed to NetworkStatus.CONNECTED. Starting networking...
2023-11-25 18:15:52,002 - 15 - hummingbot.core.utils.async_utils - ERROR - Unhandled error in background task: 'binance_perpetual_testnet_ETH-USDT'
[Traceback omitted for brevity]
KeyError: 'binance_perpetual_testnet_ETH-USDT'

Possible Root Cause:
The KeyError seems to cause from a mismatch in the naming conventions used in the candles dictionary. The system attempts to retrieve candle data using the key 'binance_perpetual_testnet_ETH-USDT', which does not exist as the candle name was set to binance_perpetual_ETH-USDT.
Release version
1.21.0
Type of installation
Docker
Attach required files
logs_v2_market-making_dman_v2.log

Tested. It is now working on development branch.
Thank you.


When ABI call output decoding fails, give more information what happened

What was wrong?
When a call failed (ABI mismatch, etc.) one had little or no information to work on why this happened.
How was it fixed?
Have an exception with a friendlier error message.
Use Python 3 raise from pattern to preserve the underling eth_abi exception.

Nice, thank you!


Map view

Check your own project file -> bundle identifier and other content if app doesn't run on own device


activate renv

Hi all,
I am looking forward to the course this evening.
Reading the README, I would ask if the project could benefit (in terms of friction in the installation phase from students) from activating renv. I am undecided what is better between a list of installation steps for setup or a new tool (from students' point of view, i.e., renv) to adopt that is not related to the content of the course.

That is a great point, thanks for filing an issue! I confess, I have never used renv, but it probably would benefit from this approach.
If you'd be happy to send me any further thoughts (or even a PR 😁) I would be very grateful. If not I will investigate myself some time.
Thanks! 🙏

Closed by #7


Stats on the wiki

It would be great if the wiki could include statistics about who's making contributions so that we can say "thanks" to members of the community that participate. I found out that you can git-clone the wiki with the following incantation:
git clone https://github.com/incf/bids-starter-kit.wiki.git

We can use whatever git-analytics approach we like to pull out user stats from this. Alternatively we could create a new dummy repository within INCF called something like wiki-stats and the only thing we'd do is occasionally update that repository with the git repo for the wiki. Then I think GitHub's pre-existing repo stats dashboarding would work.
Thoughts? cc @KirstieJane

actually turns out stats etc is quite easy. I just tried this with a test repo on my account. All you need to do is run the clone command above, then:
git remote add newrepo https://github.com/incf/wiki-stats
git push newrepo master

and it'll update newrepo with the latest state of the wiki. E.g. here's what it looks like on my temp repo:
https://github.com/choldgraf/tmp_test/graphs/contributors

I LIKE IT! I'd love to be able to show off anyone contributing to the wiki ✨
I don't have time to work on this, but I'm going to leave the issue open and anyone in the community is welcome to make it their own. The only thing I care about is making it easy to celebrate people adding to the BIDS starter kit 🎉

so I think the path forward here is to:

Create a repo called "bids-starter-kit-wiki-stats" (or something like this)
Occasionally clone the wiki for this repo (git clone https://github.com/incf/bids-starter-kit.wiki.git) and push it to the wiki-stats repo we've created.
Create links in the wiki / other places online to the dashboard for wiki-stats so that we can say "thanks" to people

Gotcha - sounds good to me. Thanks @choldgraf.
I wonder if that's something that could be automated? To update somewhat regularly?
This issue is open for anyone who wants to expand on the plan! I think it's likely a good first contribution so if you think you can help, please do! ✨

for sure - definitely scriptable at the least (e.g. python update_wiki_stats.py) though you'd need push access to the repo to do it. It could be automated to run on cloud infastructure somewhere...maybe a cron job that runs on whatever hosts OpenNeuro? cc @chrisfilo

What about https://support.circleci.com/hc/en-us/articles/115015481128-Scheduling-jobs-cron-for-builds- ?

First of all: whoah, I hadn't seen that before but it looks super useful ✨ ✨ ✨
Secondly: yeah we should totally look into this for automating the process :-)

Seems like it would be cool idea to have these stats!
I just looked into both circleCI and travisCI:
for pricing, it seems like they'd both be free circle(one free container) travis(all open-source free)
for the technical process, I haven't used either of them, so I'm inclined to go with whichever is more proven.
What are your opinions on this?

Hey @choldgraf! I was chatting with @Park-Patrick at OHBM and he mentioned that he wasn't super sure where to start on this front! I also don't really know how to get this off the ground.
I can tweet the issue and see if anyone from the community wants to help? What do you think?
The alternative is that you might be able to do the work, but I know you're super busy so I'm really just giving your first refusal here ✨

Wooooo! I did it! https://github.com/INCF/bids-starter-kit-wiki
Thank you to @rmarkello for helping me with an additional git rebase wiki/master and git push --force which I think happened because I added a README before linking the wiki to the new repo.
Easy to see the stats, as you said @choldgraf, at https://github.com/INCF/bids-starter-kit-wiki/graphs/contributors

We do not have a continuous integration way of doing this - I think we need a server for that - so at the moment my plan is to basically git pull wiki/master; git push whenever I'm curious about the updates and....erm....that might have to be the best we do for now 😆 😬 🤷♀

https://support.circleci.com/hc/en-us/articles/115015481128-Scheduling-jobs-cron-for-builds-

Thanks @chrisfilo - I do know that it's technically possible but it's outside of my skillset and I don't have time to figure it out.
If you do, that would be amazing ❤


ARP/ProductCode: martinchrzan.ColorPicker version 1.1.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This is actually wrong and needs special treatment

Since this is something you can fix instead of spamming Metadata PR's you can fix this:
Replace all the MSI's with the EXE's found on their GitHub Release page, the silent switch is --silent
The version is 3 digit

Move the MSI's to martinchrzan.ColorPicker.MachineWide with the PackageName Machine-Wide Installer to match against any version or since you have the ProductCode you can do Color Picker Machine-Wide Installer to match against any version which have a ProductCode.

Ok, so you mean to move the MSI installers to martinchrzan.ColorPicker.MachineWide and add EXE installers to martinchrzan.ColorPicker with 3 digit versions and Scope:user.

Ok, so you mean to move the MSI installers to martinchrzan.ColorPicker.MachineWide and add EXE installers to martinchrzan.ColorPicker with 3 digit versions and Scope:user.

Yes, could technically be martinchrzan.ColorPicker.User but we'll see if they find a way to merge Machine Wide installers and User Installers
My issue on this: https://github.com/microsoft/winget-pkgs/issues/14115
Related: https://github.com/microsoft/winget-cli/issues/982
Suggestion: https://github.com/microsoft/winget-cli/issues/1073
Included in Manifest Schema v1.1.0 https://github.com/microsoft/winget-cli/issues/1243
Nevermind the martinchrzan.ColorPicker.MachineWide suggestion, add the User install in the Manifest and keep the MSI version, then we can fix this once v1.1.0 comes out

Add uninstallPrevious as the upgrade behavior for the MSI

Well, I will have to delete the existing manifests and then add all manifests again in separate PRs.

If you can approve #22411 #22412 #22413 #22414 #22415 #22416 #22418 #22419 #22420 #22421 otherwise new PRs will be flagged as duplicates.

If you can approve #22411 #22412 #22413 #22414 #22415 #22416 #22418 #22419 #22420 #22421 otherwise new PRs will be flagged as duplicates.

Hold up, what kind of folder structure are you going to do?

I will out EXE installers under martinchrzan.ColorPicker.User and MSI installers under martinchrzan.ColorPicker.MachineWide

I will put EXE installers under martinchrzan.ColorPicker.User and MSI installers under martinchrzan.ColorPicker.MachineWide

Ahh sorry for confusing you, because https://github.com/microsoft/winget-cli/issues/1243 will be implemented Soon™ we can do both in martinchrzan.ColorPicker

Please explain to me from the beginning what you want to say.

When i follow this thread:

My issue on this: #14115
Related: microsoft/winget-cli#982
Suggestion: microsoft/winget-cli#1073
Included in Manifest Schema v1.1.0 microsoft/winget-cli#1243
Nevermind the martinchrzan.ColorPicker.MachineWide suggestion, add the User install in the Manifest and keep the MSI version, then we can fix this once v1.1.0 comes out

i realized the fix was close, therefore you can add the EXE and MSI in martinchrzan.ColorPicker under the MSI Version name

Ok, so you approve the PRs and I will create new PRs with EXE and MSI installers under the MSI version name. Because I deleted my manifests it will time taking to revert commits rather than creating them from scratch.

@OfficialEsco the bot just can't keep up with you 😂


Deprecate matlab

Deprecating matlab functions from modules and tests...
Suggestions are welcome if they can be better organized or have other ideas.

So, where's the part that lets you turn off testing for deprecated?

That would have to be setup in setup.py which I do not want to do now before everyone completely signs off matlab totally. That change will not be in this PR. Doesn't make sense to have tests written for deprecated functions and NOT run them here.


[DB authentication failure] Docker containers seem to only work with empty passwords - with generate creds, they cannot connect to DB

Describe the bug
The steps with generating the credentials and copying the corresponding service.env files into the respective component directories cause the docker containers to fail to start.
They fail by connecting to the database, i.e., for some reason, those new credentials do not work, and the DB can only accept connections with empty passwords.
To Reproduce
Steps to reproduce the behavior:

Follow the steps in the README.md file
generate the credentials with the keygen script and copy the service.env files into the respective component folders
follow the docker-compose-based approaches and see whether the containers come up - after finishing the compilation, docker ps -a shows the containers as exited.
Issue docker logs pri-fdo-<component> to see the error logs

Docker logs output
03:37:17.033 [INFO ] HHH000400: Using dialect: org.hibernate.dialect.H2Dialect
03:37:17.067 [WARN ] SQL Error: 28000, SQLState: 28000
03:37:17.068 [ERROR] Wrong user name or password [28000-210]
03:37:17.069 [ERROR] database session factory setup failed

Test Environment
Regular Ubuntu Linux
Additional context
NA

After setting up the containers with the empty passwords, accessing the manufacturer service fails with authentication error.
This is the instruction from the README.md
POST https://host.docker.internal:8038/api/v1/rvinfo (or http://host.docker.internal:8039/api/v1/rvinfo) The post body content-type header text/plain Authorization DIGEST with "apiUser" and api_password defined in the manufacturer's service.env POST content

Checking the service.yml of the manufacturer, I can see the username is apiUser (just as indicated above), but the empty password does not work and I get an Error 401:
WWW-Authenticate: Digest realm="Authentication required", qop="auth", nonce="1658464401861:5ec2d32654920cc8cb25ad3bff2f990f", opaque="3B2D5B875381C78ACF4EC0AA71A54F1F"
Content-Type: text/html;charset=utf-8
Content-Language: en
Content-Length: 673
Date: Fri, 22 Jul 2022 04:33:21 GMT
Keep-Alive: timeout=20
Connection: keep-alive

Did anyone encounter this behavior?

@cslev  Thanks for your Contribution. Can you delete the existing db file from demo//app-data folder?

Thanks for the comment. The trick indeed work for my first issue!
So, I deleted both emdb.mv.db and emdb.trace.db files, copied the generated creds/. into their corresponding directories, and rebuild the containers from scratch to be sure.
The containers came up without any error, so it worked.
However, when I try to Create Ownership Vouchers using Individual Component Demos, then I bump into the same error.
I tried both using HTTP and HTTPS, POST message, set authentication to digest, and content-type to text/plain.
The browser (where my REST client sits) even ask for the password (so the URLs are indeed working), but then the response is similar:
Response 401

WWW-Authenticate: Digest realm="Authentication required", qop="auth", nonce="1658891804361:fe5f0888c75456e714a1ff8f994c2bcf", opaque="7C912350EAB1868F3D8BAD04BCA34D36"
Content-Type: text/html;charset=utf-8
Content-Language: en
Content-Length: 673
Date: Wed, 27 Jul 2022 03:16:44 GMT
Keep-Alive: timeout=20
Connection: keep-alive

I have a sense, though, that something might be still working, because if I supply the wrong password (e.g., empty or randomly chosen other passwords from the service.env file), the browser immediately asks for the credentials again.
Once I set the correct api_password, then I get back the message above...but then why HTTP error 401 (Unauthorized access).
Sounds weird.

I still tried to continue the tutorial, and run java -jar device.jar with the modified service.yml pointing to di-url: http://host.docker.internal:8039.
The output is still the same as in the beginning. The debug message still says T01 only with the built-in host of http://host.docker.internal:8080.
12:12:26.842 [WARN ] Using SSL self-signed certificate trust strategy for Http Clients
12:12:27.011 [INFO ] Starting Fdo Device
12:12:27.085 [INFO ] credentials loaded, GUID is caf69057-be3f-4bc4-9f77-af59e0461a07
12:12:27.162 [INFO ] RVBypass flag not set, Starting TO1.
12:12:27.163 [INFO ] TO1 URL is http://host.docker.internal:8080
12:12:27.179 [INFO ] Type 30 [h'CAF69057BE3F4BC49F77AF59E0461A07', [-7, h'']]
12:12:27.393 [INFO ] Type 31 [h'0BDABB02A4414F80868E68572399A8F6', [-7, h'']]
12:12:27.927 [INFO ] Type 32 [h'A10126', {}, h'A20A500BDABB02A4414F80868E68572399A8F61901005101CAF69057BE3F4BC49F77AF59E0461A07', h'F4F5F13E995AA351D319B6B3773B52A0F268581581F1B956A705BB91B56377DE8689E7F460416477DF7C1ECCCF31629EE442932ACF283F89F87E4714C845E2A9']
12:12:28.032 [INFO ] Type 255 [6, 32, "guid: caf69057-be3f-4bc4-9f77-af59e0461a07",<PHONE_NUMBER>,<PHONE_NUMBER>]
12:12:28.032 [INFO ] Starting Fdo Completed

I still tried to continue the tutorial, and run java -jar device.jar with the modified service.yml pointing to di-url: http://host.docker.internal:8039. The output is still the same as in the beginning. The debug message still says T01 only with the built-in host of http://host.docker.internal:8080.
12:12:26.842 [WARN ] Using SSL self-signed certificate trust strategy for Http Clients
12:12:27.011 [INFO ] Starting Fdo Device
12:12:27.085 [INFO ] credentials loaded, GUID is caf69057-be3f-4bc4-9f77-af59e0461a07
12:12:27.162 [INFO ] RVBypass flag not set, Starting TO1.
12:12:27.163 [INFO ] TO1 URL is http://host.docker.internal:8080
12:12:27.179 [INFO ] Type 30 [h'CAF69057BE3F4BC49F77AF59E0461A07', [-7, h'']]
12:12:27.393 [INFO ] Type 31 [h'0BDABB02A4414F80868E68572399A8F6', [-7, h'']]
12:12:27.927 [INFO ] Type 32 [h'A10126', {}, h'A20A500BDABB02A4414F80868E68572399A8F61901005101CAF69057BE3F4BC49F77AF59E0461A07', h'F4F5F13E995AA351D319B6B3773B52A0F268581581F1B956A705BB91B56377DE8689E7F460416477DF7C1ECCCF31629EE442932ACF283F89F87E4714C845E2A9']
12:12:28.032 [INFO ] Type 255 [6, 32, "guid: caf69057-be3f-4bc4-9f77-af59e0461a07",<PHONE_NUMBER>,<PHONE_NUMBER>]
12:12:28.032 [INFO ] Starting Fdo Completed

Can you try with this PR? https://github.com/secure-device-onboard/pri-fidoiot/pull/529

), the browser immediately asks for the credentials again.
Once I set the correct api_password, then I get back the message above...

Which client are you using for your Digest request creation?

Thanks, I will try!
If that is only about the fallback mechanism that I already tried to overcome such an issue by setting the host.docker.internal to my localhost IP in /etc/hosts file.

), the browser immediately asks for the credentials again.
Once I set the correct api_password, then I get back the message above...

Which client are you using for your Digest request creation?

I tested with RESTer extension of firefox. I have been using it for a while and it always worked as expected. But I am a fan of postman too. So, will try with postman; maybe it will work out with it. Keep you posted. Thanks

), the browser immediately asks for the credentials again.
Once I set the correct api_password, then I get back the message above...

Which client are you using for your Digest request creation?

I tested with RESTer extension of firefox. I have been using it for a while and it always worked as expected. But I am a fan of postman too. So, will try with postman; maybe it will work out with it. Keep you posted. Thanks

Can you try with Postman?  We test with postman and cURL.

I tried with postman, same error 401:
See postman output here

Are you running the service as java -jar aio.jar ?

I tried with postman, same error 401: See postman output here

Are you running the service as java -jar aio.jar ?

Nope, I am running them in containers following the docker-compose-based instructions. All containers are up and running properly.
This part: Staring the FDO PRI HTTP Device works as expected. I could also telnet to the services running on the specified ports. So, there should not be any issue with the services running.

Restart the service using docker-compose down && docker-compose up --build
or
Can you try updating the password in the service.yml file's $(api_password) field ? and restart the service.

++, check whether the service.env file contains the correct api_password.

I just checked all! rebuild images, double-checked the passwords in the corresponding services.env files, but still the same result.

screenshot

screenshot

if check the post address; it is 127.0.01:8039. Shouldn't it be <IP_ADDRESS>:8039 ?

Oh, yes. There was a typo, however, it did not make any difference. Postman figured out the right IP address. Otherwise, I would have gotten a different networking error instead of error 401.
Just to be sure, I put that missing dot there, but nothing changed...


JOBS_GRAPH is not defined

ReferenceError: JOBS_GRAPH is not defined
    at updateStatusJob (/usr/src/output/app/lib/utils.js:174:43)
    at updateAndFilterTimedOutJobs (/usr/src/output/app/lib/utils.js:196:17)
    at run (/usr/src/output/app/jobs/dump-publication-graph.js:25:63)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
     at async CronJob.<anonymous> (/usr/src/output/app/app.js:85:7)
     ```
Getting this error, and I don't see JOBS_GRAPH defined in that file so I think is an error of the service

should be fixed in 1.0.1


IntelliJ generator for Scala native lacking ill.scalanativelib dependency

Importing the project into Intellij, the following packages are not found:
import mill.scalanativelib._
import mill.scalanativelib.api._

Could you please provide a minimal working example? I would like to help but I am not versed in the Scala native world.

Looks like the marked duplicates of this issue were closed with #616 .


Two FIFO deq's assigned to same muxPort,muxOfs

In MultiplexedWrites, one of the readers to wt seems to have no bufferPort defined but I think it should be on bufferPort 0. Assuming this were correct though, what does a broadcasting write mean?
Also in Scan_fifofull.  It has an NBuf FIFO.  One of the dequeuers is on bufferPort 0, the other has bufferPort None.  I'm not sure what this is supposed to create.  Pairwise, these readers are time multiplexed.
Accel{
  Pipe.Foreach{
    // deq
    // enq
  }
  // deq
}

Actually this seems to just be the case with any read that is from a reduction, regs and srams are the same but I erroneously used to default to bufferPort 0 in these cases so it used to accidentally work.

The broadcasting writer here looks correct - its a write which is sequential with a double buffer. I can't remember if the correct behavior we settled on before was we should either write only to the first buffer, or write to all buffers.

Just pushed a fix for attempting to buffer FIFOs - should have a reasonable port layout for these dequeues now.


Introduce an API to get catchup status of peers

This commit introduces an API that can be used to query a particular
partition on a storage server (or all partitions) to find whether the
remote replicas of that partition have caught up to within a specified
range of bytes (i.e. lag is "acceptable").

Codecov Report

Merging #741 into master will increase coverage by 0.02%.
The diff coverage is 69.84%.

@@             Coverage Diff              @@
##             master     #741      +/-   ##
============================================
+ Coverage     65.94%   65.96%   +0.02%
- Complexity     3842     3869      +27
============================================
  Files           331      333       +2
  Lines         25392    25507     +115
  Branches       3180     3198      +18
============================================
+ Hits          16744    16826      +82
- Misses         7755     7781      +26
- Partials        893      900       +7

Impacted Files
Coverage Δ
Complexity Δ

.../com.github.ambry/tools/admin/ServerAdminTool.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...in/java/com.github.ambry.server/ServerMetrics.java
95.02% <100%> (+0.28%)
2 <0> (ø)
:arrow_down:

...hub.ambry.protocol/AdminRequestOrResponseType.java
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

...n/java/com.github.ambry.protocol/AdminRequest.java
91.42% <50%> (-3.58%)
17 <2> (ø)

...hub.ambry.protocol/CatchupStatusAdminResponse.java
75% <75%> (ø)
6 <6> (?)

...thub.ambry.protocol/CatchupStatusAdminRequest.java
77.77% <77.77%> (ø)
5 <5> (?)

.../java/com.github.ambry.protocol/AdminResponse.java
77.27% <87.5%> (+5.84%)
9 <6> (+6)
:arrow_up:

...in/java/com.github.ambry.server/AmbryRequests.java
68.12% <97.67%> (+2.49%)
51 <6> (+9)
:arrow_up:

...c/main/java/com.github.ambry.network/Selector.java
78.48% <0%> (-3.59%)
44% <0%> (-1%)

...in/java/com.github.ambry.store/BlobStoreStats.java
84.41% <0%> (-1.44%)
91% <0%> (-2%)

... and 10 more

Continue to review full report at Codecov.

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

@cgtz comments addressed

@nsivabalan comments addressed


Switzerland: ch/contrywide fix missing postcodes and city names

The latest ch/countrywide data was having blank POSTCODE and CITY columns.
There was a mismatch with the column names of the source data, so this PR fixes it.

Looks like this is being affected by the lat/lon bug in https://github.com/openaddresses/batch/issues/90

@iandees anything blocking the merge? We are waiting on that to re-enable openaddresses.

There's a bug that's preventing our system from processing the source properly. I can merge it, but the data output will be affected by the bug mentioned above.

@iandees Thanks. This is why we get the image rotated and flipped?

Yep.


dm: support dm worker incremental validator cutover switch

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/9868
What is changed and how it works?
Part of adding interfaces to support worker incremental switch
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/retest

/test engine-integration-test

/retest

/lgtm

/retest

/retest

/cherry-pick release-7.5


Changes in metadata broke backward compat of metric name

The metric name for STS & co changed in the metadata as previously it was created via merging the cos_sim & spearman names i.e. https://github.com/embeddings-benchmark/mteb/blob/2cc222efd84c29cbb5ff04fb8e6703674d53dda9/scripts/mteb_meta.py#L113 and now it is defined as cosine_spearman i.e. https://github.com/embeddings-benchmark/mteb/blob/b16d7d5620b987992e16ee663e5d82ee007acd8e/mteb/tasks/STS/eng/BiossesSTS.py#L21C21-L21C36 which is then taken as is i.e. https://github.com/embeddings-benchmark/mteb/blob/b16d7d5620b987992e16ee663e5d82ee007acd8e/mteb/cli.py#L265
For old models like https://huggingface.co/w601sxs/b1ade-embed it was called cos_sim_spearman.
For new models like https://huggingface.co/w601sxs/b1ade-embed-kd_3 it is now called cosine_spearman.
This now breaks the leaderboard as people are starting to submit with the new metric name but the old one is asked for in the code via https://huggingface.co/spaces/mteb/leaderboard/blob/a812c3bbcc3cd88510b60c6c5788d95668d69dc6/config.yaml#L38 which is then used to get the scores via
https://huggingface.co/spaces/mteb/leaderboard/blob/a812c3bbcc3cd88510b60c6c5788d95668d69dc6/app.py#L245
Same will happen for PairClassification and others I think. 🤔
Options are:

Allow both metric names; Requires some changes to the leaderboard code
Revert cosine_spearman to cos_sim_spearman & others
Maybe sth else reasonable that I'm missing?

Not sure what's best, cc @KennethEnevoldsen & @tomaarsen who probably know better! 🙌

1 and 2 both work I am up for either. Assuming that some models might in the future use their own similarity metric we might want to allow for that. E.g. use "similarity_spearman" instead? (and then the "similarity" is defined by the model but defaults to "cosine"

Are there any notable models thus far that have a similarity metric that is not native in MTEB anyways (i.e. cosine, dot & co)?
Another big difference is that previously the metadata would include all metrics which is nice in case someone wants to fetch different scores, while now it only includes the main score? I think we want to revert this, no?

Maybe we want to add a test that checks that the metadata generation is working to prevent such issues in the future? 🤔

Are there any notable models thus far that have a similarity metric that is not native in MTEB anyways (i.e. cosine, dot & co)?

Binary still uses cosine? (then I don't think so). It is simply from the perspective of limiting modeling.

Another big difference is that previously the metadata would include all metrics which is nice in case someone wants to fetch different scores, while now it only includes the main score? I think we want to revert this, no?

All metrics are totally doable, we can totally add that in.

Maybe we want to add a test that checks that the metadata generation is working to prevent such issues in the future? 🤔

Def.! A test will solve this going forward (I actually added a test based on the example in the readme of the previous script, but I can see that it is not comprehensive).
So a solution would be:

Convert cosine_spearman > cos_sim_spearman

We can optionally add similarity_spearman which uses the model's custom similarity, without breaking compatibility. At a later stage we might want to change the main metric while still calculating cosine (might e.g. be interesting for vector databases, which can be limited to online certain metrics)

Add in all scores in the metadata

Update the test to reflect compatibility with previous results (do you have a gold standard + results that lead to it)

Seeing https://github.com/embeddings-benchmark/mteb/issues/885 it makes sense to me that you want to separate similarity measure and metric. Indeed, maybe there should be a tab on the leaderboard similar to Max Tokens that says what similarity metric a model uses and we switch to defining similarity metrics at the model, not dataset/task level 🤔
Maybe having: spearman (uses default similarity metric) ; cosine_spearman ; dot_spearman etc?
(I don't have a preference for cos_sim_spearman over cosine_spearman; I think cosine_spearman is fine, similar to how we don't say dot_prod_spearman)

Sounds good

Should we just take some result files of different formats from https://huggingface.co/datasets/mteb/results?

two things to add for creating metadata:
hf gives trouble because the create meta calls it "metric" and not "metrics". and also it must be an array so instead of:
type: ...
value: ...
it needs to be

type: ...
value

then it seems to work :-)

two things to add for creating metadata:
hf gives trouble because the create meta calls it "metric" and not "metrics". and also it must be an array so instead of:
type: ... value: ...
it needs to be

type: ...
value

then it seems to work :-)

Thanks for finding that! Do you want to open a PR to fix it & the other issues mentioned here? Would be amazing 🙌

I have added a pr #931


docs(apps/legacy-api): Add C2: Subgraph Sample

What this PR does / why we need it:

Add C2: Subgraph Sample
Also rename Legacy API to api.defichain.com/v1 and Ocean API to ocean.defichain.com/v0 for API explicity

The link titles look a little disconnected from the original format, perhaps expecting something like Ocean v0 and Legacy v1?

The link titles look a little disconnected from the original format, perhaps expecting something like Ocean v0 and Legacy v1?

Yeah, I have reverted this!


added List methods push, pop, unshift, shift to List IndexedCursor

I tried to add some methods to the Cursor API because they are missing. This solves #346.
I'm not sure if this is the right way to do it, because all other methods extend both KeyedCursorPrototype and IndexedCursorPrototype and this only extends IndexedCursorPrototype. So all feedback is more then welcome!

This is big for me, really hinders ability to use of cursors until this is fixed! :+1: :+1:

@natew If there are other methods that are missing, let me know, I'm happy to implement them too.

wow, this is so great, exactly the things I need. Can't wait for it

Nice work, thanks for your patience!

I'm not sure how to push regular JS items onto an immutable cursor List, this doesn't work:
immutableStore.push.apply(null, jsarray);

I've tried a few variations I keep getting:
Possibly unhandled TypeError: Cannot read property 'updateIn' of undefined
    at updateCursor (webpack:///./~/immutable/contrib/cursor/index.js?:279:37)

@natew can you give a full testcase? Somewhere as a gist or something?


Add FHEM Home Automation

FHEM for Home automation
https://www.fhem.de
iOS: 3rd party e.g. https://itunes.apple.com/de/app/fhem-remote/id652723412?mt=8
And Android 3rd party https://play.google.com/store/apps/details?id=li.klass.fhem&hl=de

Added. Thank you.


buttons to control break timer

Is your feature request related to a problem? Please describe.
Once in a while I need to skip, pause or restart breaks. The app don't have this feature.
Describe the solution you'd like
Just control buttons for breaks, like work timer have.

I'm sorry but it has.
Just don't enable Strict Mode feature, it will blocks you from using those buttons when timer started.

Thanks and sorry.


Update versions to 0.15.1

I also changed all dependency versions so nothing ever tries to pull in 0.15.0. I'll also yank 0.15.0 from crates.io after 0.15.1 is published.


Fastlane not running

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Running anything to do with Fastlane throws this message:
/usr/local/bin/fastlane: /usr/local/Cellar/fastlane/2.177.0/libexec/bin/fastlane<EMAIL_ADDRESS>bad interpreter: No such file or directory
/usr/local/bin/fastlane: line 4: /usr/local/Cellar/fastlane/2.177.0/libexec/bin/fastlane: Undefined error: 0

Command executed
Anything but for example: fastlane env
Complete output when running fastlane, including the stack trace and command used
/usr/local/bin/fastlane: /usr/local/Cellar/fastlane/2.177.0/libexec/bin/fastlane<EMAIL_ADDRESS>bad interpreter: No such file or directory
/usr/local/bin/fastlane: line 4: /usr/local/Cellar/fastlane/2.177.0/libexec/bin/fastlane: Undefined error: 0

Environment
Can't run the command

Got it to works, thanks!


#342 - Add wiki URL to lint message

Resolves #342

Append wiki URL to the lint message
Note this is only applicable in the browser

Hi @cvrebert would you be okay merging this PR in?

Merged. Thanks! Sorry for the delay.


Enable PID namespace isolator in mesos for the UCR.

High Level Description
This change enables the pid isolator for all containers launched by the UCR.
Related Issues

DCOS_OSS-790 Always enable the pid isolator for the UCR in Mesos.

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: Pending
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

Not very likely -- it's only relevant for containers launched via the UCR
-- it also is likely the "expected" behavior and anything that relied on it
not being set should be considered a bug.
On Mon, Mar 20, 2017 at 7:50 PM Adam<EMAIL_ADDRESS>wrote:

@adam-mesos commented on this pull request.
How likely is this to break existing users' apps/tasks?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dcos/dcos/pull/1380#pullrequestreview-28013045, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAF4o8XEsUr1cZGU1e0EQY6wV7QxuuoBks5rnzrbgaJpZM4Mh6HN
.

Can I get a milestone added to this? @adam-mesos @spahl @lingmann

TBD which milestone/release. Let's discuss.

1.10 would be my vote for milestone. people have lived with the existing behavior for quite some time now so enabling it with the next release seems the safest option.

@mesosphere-mergebot bump-ee


Disable version check by default

Its very inconvenient to me that its not possible to disable online version checking by a flag but it requires a ENV. It complains about API KEY right away. It is possible to disable this by default and enable it on request or thru the config file?

Have u taken a look at https://github.com/interlynk-io/sbomasm?tab=readme-ov-file#edit-sboms, or is this something that if available in assemble would be better?

We will soon be make a 1.0 version of this tool, in that version we would have this check off by default.

This needs to be done for sbomqs, sbomasm and sbomgr.


MqttClientOptionsBuilder: Method WithWillMessage() missing

Describe the feature request
After an update to MQTTnet v4 I noticed that the method WithWillMessage() is missing from MqttClientOptionsBuilder.
A lot of methods have been re-added after the initial release of v4, but not this one.
Which project is your feature request related to?

Client

Describe the solution you'd like
An answer whether this decision is intentional and it is required for consumers to adapt to the new API or this is not intentional and can and will be fixed in the future allowing consumers to wait before integration.
Describe alternatives you've considered
Previous implementation:
options.MqttClientOptionsBuilder.WithWillMessage(msg);

Alternative implementation with v4 (for me, your results may vary depending on what is important to you):
options.MqttClientOptionsBuilder.WithWillTopic(msg.Topic)
    .WithWillPayload(msg.Payload)
    .WithWillRetain(msg.Retain)
    .WithWillPayloadFormatIndicator(msg.PayloadFormatIndicator)
    .WithWillContentType(msg.ContentType);

Additional context
none

This message was removed in version 4 because the will message is not 100% the same as a regular message. So according to the RFC all fields in the package were converted to corresponding methods in the connection options builder like "WithWillTopic" etc. Properties which are missing there like "ReponseTopic" etc. are not supported in will messages.


Changing MFOLD not altering output

Hello, I wanted to try varying the upper and lower bound for the MFOLD parameter and I do not see any changes in the results. Can you tell me why this is? I should note that I do not have a control, and I realize that in that case, macs2 infers a noise level (i.e. background) and thus generates a control.
This is my command: macs2 callpeak -t /out.sorted.bam -B -f AUTO --nomodel --SPMR --keep-dup all -g dm --trackline -n out.sorted.bam --cutoff-analysis --call-summits -p 0.01 -m 40 100 --outdir /outdir/ 2> /outdir/out.sorted.bam.log &
I tried 3 different MFOLD boundary limits: 5 50, 25 100, and 40 100, and when examining the narrowPeak and pileup.bdg files there is no difference.
Thank you!
Ashley:)

MFOLD is used to estimate the shifting model in order to predict the 'fragment length' for smoothing pileup signals, and the actual setting may or may not have an impact on the final result. If you want to tweak 'fragment length' to see if there is a different result, tweak the parameter '--extsize' while setting '--nomodel'.


Add guide to show how to enable remote-upgrade on CloudRun

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Cool
/lgtm

/changelog


[Production] C# room not getting emails

Closing as cannot fix. There's something wrong with the sender's side.


Homework

create a class movie with the following instance variable and methods. instance variables : title ( string ) ,rating (int) . constructors: movie(string title , int ratings) ,Movie ( Movie movie ). methods:getTitle(),getRarings(),setTitle(),setRatings(),getCategroy(). For setRating(),the user must input rating between 0 and 10 . For getCategroy, it is a methods that return character such that: A : rating is equal to 9 or 10 B : rating is equal to 7 or 8 C : rating is equal to 5 or 6 D : rating is equal to 4 or 3 F : otherwise, Write the main program for variables and methods declaration.

this belongs on discussion boards, properly contextualized as a PAT


merge package from arlac77/npm-package-template

rollup.config.js

chore(rollup): import cleanup

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


License of published surfaces is never shown in the web interface

The license of a published surface is listed in the downloaded data file but not on the web interface. Do we need to show it somewhere?
Any opinions @mcrot and @tdbjacobs ?

Seems like it would be useful to show. Just so people know up front how
they can reuse a surface.
-Tevis
Tevis Jacobs
William Kepler Whiteford Faculty Fellow and Associate Professor
Mechanical Engineering and Materials Science
University of Pittsburgh
http://www.engineering.pitt.edu/JacobsLab/
Mail: 636 Benedum Hall, 3700 O'Hara St., Pittsburgh, PA 15261
Office: 538-E Benedum Hall,<PHONE_NUMBER>
<EMAIL_ADDRESS>On Thu, Dec 3, 2020 at 7:21 AM Lars Pastewka<EMAIL_ADDRESS>wrote:

The license of a published surface is listed in the downloaded data file
but not on the web interface. Do we need to show it somewhere?
Any opinions @mcrot https://github.com/mcrot and @tdbjacobs
https://github.com/tdbjacobs ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ComputationalMechanics/TopoBank/issues/566, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFOUYWQD7FPSVJWELQQLIH3SS57F3ANCNFSM4UL4YIMQ
.

Yes, I fully agree, I simply forgot this. That's easy to fix and will be included in the next release.

I've implemented this and it looks like this:

Let me know if you need sth. else here. I would prefer not to explain the license myself but only refer to the description on the creative commons page, which is easy to read, I think.


Instantiate chaincode error - book Hands-On Blockchain with Hyperledger

Hi,
I've installed chaincode successfully but when I tried to instantiate the chaincode as per instruction on page 250 but received the following error:
...
2018-07-03 02:42:35.654 UTC [chaincodeCmd] checkChaincodeCmdParams -> INFO 05f Using default vscc
2018-07-03 02:42:35.654 UTC [chaincodeCmd] getChaincodeSpec -> DEBU 060 java chaincode enabled
2018-07-03 02:42:35.655 UTC [msp/identity] Sign -> DEBU 061 Sign: plaintext: 0AB0070A6A08031A0C089BC5EBD90510...6E740A000A04657363630A0476736363
2018-07-03 02:42:35.655 UTC [msp/identity] Sign -> DEBU 062 Sign: digest: 26532ECA4A0FFA52BA8554EB203C324CF77E9B2E888F0890954F5E8C5DD4F70F
Error: Error endorsing chaincode: rpc error: code = Unknown desc = failed to init chaincode: handler not found for chaincode tw:0
Usage:
peer chaincode instantiate [flags]
Background:
i started the network in development mode:
$> ./trade.sh up -d true
then i build the chaincode:
$> docker exec -it chaincode bash
go build
install chaincode:
$> docker exec -it cli bash
peer chaincode install -p chaincodedev/chaincode/trade_workflow_v1 -n tw -v 0
instantiate chaincode:
peer chaincode instantiate -n tw -v 0 -c '{"Args":["init","LumberInc","LumberBank","100000","WoodenToys","ToyBank","200000","UniversalFreight","ForestryDepartment"]}' -C tradechannel
then i got the above-mentioned errors..
Any suggestion on the cause of this error?
nizam

@nizamk I don't see any instructions for chaincode instantiation on Page 250. (I only have a PDF copy at this point.) I'm guessing it's Chapter 4 you are referring to, based on the nature of your query? Page 103-104?
@jcmldev FYI.

@nizamk Based on the commands you listed above, you seem to have missed the chaincode start command, which you need to run immediatedly after go build in the chaincode container:
2. Run the chaincode with the following command:
$ CORE_PEER_ADDRESS=peer:7052 CORE_CHAINCODE_ID_NAME=tw:0 ./trade_workflow_v1

Can you verify?

Also, for invocations and queries, please use the following commands (the invoke command listed in the book in page 104 is missing an ID argument):
1. Put a new trade agreement on the ledger with the following command:
    $ peer chaincode invoke -n tw -c '{"Args":["requestTrade", "trade-12", "50000", "Wood for Toys"]}' -C tradechannel
2. Retrieve the trade agreement from the ledger with the following command:
    $ peer chaincode invoke -n tw -c '{"Args":["getTradeStatus", "trade-12"]}' -C tradechannel

The parameter trade-12 refers to the ID of a trade instance.

@VRamakrishna You're correct. Actually right after my post i just realised i had killed the chaincode after starting it. It's working now. Thanks.
BTW I also had problems with Invoke command as you had correctly pointed out. It was complaining insufficient arguments. It's working now as per your suggestion.

@VRamakrishna I have another question. When starting fabric network in dev mode, we have to start the chaincode container manually and issue start command: $ CORE_PEER_ADDRESS=peer:7052 CORE_CHAINCODE_ID_NAME=tw:0 ./trade_workflow_v1. In this command we actually starts the chaincode executable residing in ./trade_workflow_v1.
My question is why are we installing chaincode again in the following command from CLI container?
$ peer chaincode install -p chaincodedev/chaincode/trade_workflow_v1 -n tw -v 0
Earlier we had already started the chaincode in chaincode container, and now it seems to me we're installing the chaincode source codes since the path "chaincodedev/chaincode/trade_workflow_v1 " is referring to source codes.

@nizamk The goal of the dev mode is to enable control of redeployment of the chaincode by the developer, without stopping the network or any of the containers. The command is there to show how the chaincode can be installed and updated after the source code was modified.

@VRamakrishna yes sure, you can close this one. Thanks.


Migrating blobs from go-ssb

I think the plan is to simply leave them where they are for now. I need to change the way they are stored by scuttlego so that it is identical to the way go-ssb does it.

What needs to be done:

change the way scuttlego stores blobs so that it is done identically to the way go-ssb does it and uses the same directory
change the swift function back so that it keeps using the old directory


Adding mulimedia button to change the episode

Adding an button to change to the next or previous episode through the player only
Please assign me this issue under ssoc'23

@ansh7432 already done.


Is it possible to store partitioned files on s3 without partition keys in path?

Hi, when I storing partitioned data on S3, I use only value in path (without keys)
for example
I use
s3://bucketname/spectrum/tablename/2008-01-01/
not
s3://bucketname/spectrum/tablename/saledate=2008-01-01/
As I can see awswrangler offer only partition naming with key=value
https://github.com/awslabs/aws-data-wrangler/blob/2f16e613d240b55869147060cba9edf023188570/awswrangler/s3/_write_dataset.py#L42
Could you help with it?

I am afraid that is not supported. The method uses hive-style partitions (key=value) because it interacts with the Glue catalog. Without that information, the metadata for partition columns in the catalog could not be populated.


fix clippy error

I just ran these local sanity checks:
cargo check && cargo check --tests && cargo fmt && ./utils/trailing-whitespace.sh reject && cargo clippy
warning: Git command output: Output { status: ExitStatus(unix_wait_status(0)), stdout: "mob-release-1.2.5_126-251-g53d592b3-dirty\n", stderr: "" }
    Checking zingolib v0.2.0 (/home/nattyb/src/zingolabs/zingolibs/dev/zingolib)
    Checking zingo-testutils v0.1.0 (/home/nattyb/src/zingolabs/zingolibs/dev/zingo-testutils)
    Checking zingo-cli v0.2.0 (/home/nattyb/src/zingolabs/zingolibs/dev/zingocli)
    Finished dev [unoptimized + debuginfo] target(s) in 1.16s
warning: Git command output: Output { status: ExitStatus(unix_wait_status(0)), stdout: "mob-release-1.2.5_126-245-ge4ef4e1b-dirty\n", stderr: "" }
    Finished dev [unoptimized + debuginfo] target(s) in 0.10s
No trailing whitespace detected.
warning: Git command output: Output { status: ExitStatus(unix_wait_status(0)), stdout: "mob-release-1.2.5_126-264-g51a2c63b\n", stderr: "" }
    Finished dev [unoptimized + debuginfo] target(s) in 0.12s
1694547382 || shofth nattyb dev[ ?? ]
/home/nattyb/src/zingolabs/zingolibs/dev: 0 $
git show
commit e6341a1e22921a84887e5a367742d2b84a5c009d (HEAD -> dev, zancas/fix_clippy_error)
Author: zancas<EMAIL_ADDRESS>Date:   Tue Sep 12 13:35:37 2023 -0600

    fix clippy error

diff --git a/zingolib/src/commands.rs b/zingolib/src/commands.rs
index 5cae2c32..7b4dcffa 100644
--- a/zingolib/src/commands.rs
+++ b/zingolib/src/commands.rs
@@ -904,8 +904,8 @@ impl Command for SendCommand {
                     )
                 })
                 .collect::<Vec<_>>();
-            if error.is_some() {
-                return error.unwrap();
+            if let Some(err) = error {
+                return err;
             }

             match lightclient.do_send(tos).await {

Codecov Report
Patch coverage has no change and project coverage change: -0.01% :warning:

Comparison is base (51a2c63) 59.80% compared to head (e6341a1) 59.79%.

:exclamation: Current head e6341a1 differs from pull request most recent head f5f16cc. Consider uploading reports for the commit f5f16cc to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #480      +/-   ##
==========================================
- Coverage   59.80%   59.79%   -0.01%
==========================================
  Files          33       33
  Lines        5411     5410       -1
==========================================
- Hits         3236     3235       -1
  Misses       2175     2175

Files Changed
Coverage Δ

zingolib/src/commands.rs
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

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

This fixes a clippy error. We shouldn't be able to merge without fixing clippy errors.

i have added clippy and format to our branch protection. could you resolve the conflicts and resend for review?


Add Azure (AAD) auth

Waiting for #366 to be determined before we move on with this issue

So, I've taken a first stab at adding Azure AD as an OIDC auth provider for Sign In.
I've opened #1990 as draft as I have hit some issues and looking for some help resolving them, and feedback on what I've done so far.

I've abandoned #1990 as I was getting nowhere with passport-azure-ad (it didn't conform to passport.js anyways).
I switched to using passport-microsoft and everything is now all good 👍
#2056 is the working implementation.


durable-deterministic-mode Added durable mode to seed()

See https://github.com/fzaninotto/Faker/issues/1913

As mentioned in the issue, better solution without code change found

What was that the better solution?


#question x64 build on appstudio.windows.com

Just a small question : )
The site appstudio.windows.com generates packages for ARM and x64 platforms.
I am wondering that generating x64 package is not big deal for you guys.
This is just a curiosity.
Are there any potential problems with the generation of x64 assemblies?

It shouldn't be any problems with x64 packages. We are not generating this because of storage space saving.


Gangwei/ppp

Codecov Report

Merging #315 (9adc4cc) into master (1fd190e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #315   +/-   ##
=========================================
  Coverage     54.19%   54.19%
  Complexity      761      761
=========================================
  Files           120      120
  Lines          3194     3194
  Branches        481      481
=========================================
  Hits           1731     1731
  Misses         1260     1260
  Partials        203      203

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 1fd190e...cde904d. Read the comment docs.


Fullscreen preview mode

Added a fullscreen preview mode to go with https://github.com/mozilla/thimble.webmaker.org/pull/1266

Also, please update the README for this: https://github.com/humphd/brackets/blob/bramble/README.md#bramble-instance-methods.
r=me with those 2 changes.

@humphd Made the requested changes, R?

I restarted the build, which looks like it failed due to npm issues.  That needs to pass and you need to fix the submodule change.

@humphd Fixed the submodule issue (thanks @gideonthomas) - is the NPM issue something I can fix?

Looks great.


feature request - http status endpoint for external monitoring

Please add some kind of status endpoint that would return HTTP 200 only if gogs and it's dependencies are running smoothly. (Probably easy solution would be to attempt to SELECT 1 from database). Such endpoint would be useful for infrastructure monitoring tools.

Hi, you can use /healthcheck, e.g.: https://try.gogs.io/healthcheck


Move the stealth.js to a separate repo

Just like it was done in the #206, we need to move the stealth.js to a separate repo and use modern build process for it.
What needs to be done:

Stealth script repo: EXTENSIONS/repos/stealth-script/browse
Use a modern build system (choose whichever you like more)
Use a transpiler (the stealth script needs to work in IE10)

The content script interface (configuration) needs to be clearly described in the repo README.

Belongs to https://github.com/AdguardTeam/StealthMode, or a different repo?

No, this is a module that is used in AG for Windows, and, potentially, can
be used in AG for Mac/Android.
On Sun, Apr 29, 2018, 06:28 TPS<EMAIL_ADDRESS>wrote:

Belongs to https://github.com/AdguardTeam/StealthMode, or a different
repo?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AdguardTeam/CoreLibs/issues/232#issuecomment-385222928,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFq-m0WXinMLxWA6JVyxwx9wIk88DcOoks5ttTNggaJpZM4TriVD
.


Fix list all uniq container images

Fix https://github.com/kubernetes/website/issues/27475

@sftim Done

@ms-choudhary
Could you remove language/* labels except for language/en?
Because current code includes just content for en.

Could you remove language/* labels except for language/en?
I don't seem to have access to update the labels actually. Might have been auto picked based on my first commit. Should I raise a new PR?

@ms-choudhary
In situation of this PR, you can refer this comment of PR(#27993).
That command will remove labels.
You can use prow commands in some k8s repository for updating and collaborating ..., please see it :smile:

/remove-language fr
/remove-language id
/remove-language ja
/remove-language ko
/remove-language zh

thanks @atoato88

Thanks.
/lgtm
/approve


(2) Separate out Processor from BlockProcessor

Second step towards #737, builds on #736.
Introduces a new Processor that is used by BlockProcessor. In the future, we should be able to use it in VmProcessor as well. All the code is moved without any changes.

Rebased


docs: fix docs typos and spelling mistakes

Description
I found a few small typo's and spelling mistakes when I was reading through the docs.
They are under:
Migration ui_template
Migration ui_toast
Related Issue(s)
N/A
Checklist

[x] I have read the contribution guidelines
[ ] Suitable unit/system level tests have been added and they pass
[x] Documentation has been updated

[ ] Upgrade instructions
[ ] Configuration details
[ ] Concepts

[ ] Changes flowforge.yml?

[ ] Issue/PR raised on FlowFuse/helm to update ConfigMap Template
[ ] Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

[ ] Includes a DB migration? -> add the area:migration label

Gave it a better title😁