[storage] add convenience methods exists, createIfNotExists, deleteIfExists

Fix #7701

Added createIfNotExists for ContainerClient, AppendBlobClient and PageBlobClient. Added deleteIfExists for ContainerClient and BlobClients.

Added exists and deleteIfExists() to ShareClient, ShareDirectoryClient, and ShareFileClient. Added createIfNotExists() to ShareClient and ShareDirectoryClient. (As FileClient.CreateIfNotExists can't be implemented safely.)

Added exists, createIfNotExists and deleteIfExists to QueueClient.

Added createIfNotExists, deleteIfExists to DataLakeFileSystemClient, DataLakePathClient, DataLakeDirectoryClient, and DataLakeFileClient.

API shape looks good modulo one typo.
General question: is returning Promise super useful or common? It makes it hard to get at the request metadata, so I wonder if maybe returning some empty response object or other sentinel value would be better. Harder to check if e.g. the thing wasn't deleted because it didn't exist, but more uniform - I can still do res._response. Thoughts?

@XiaoningLiu  suggested we return void instead of null for now and upgrade if needed.
When deleting but it doesn't exists, a RestError is returned to us, which has roughly the same set of info as the normal response but in a different structure.
To return more useful information like requestId header and provide a more uniform usage, probably we should extend the original response by adding a bool field to indicate whether we have successfully deleted it, and extract all the info in the RestError and fit them into the original response.
But I wonder if we can do the fitting in a neat way.  @jeremymeng  any idea?
For example,
export type BlobDeleteResponse = BlobDeleteHeaders & {
  /**
   * The underlying HTTP response.
   */
  _response: coreHttp.HttpResponse & {
      /**
       * The parsed HTTP response headers.
       */
      parsedHeaders: BlobDeleteHeaders;
    };
};

export class RestError extends Error {
  static readonly REQUEST_SEND_ERROR: string = "REQUEST_SEND_ERROR";
  static readonly PARSE_ERROR: string = "PARSE_ERROR";

  code?: string;
  statusCode?: number;
  request?: WebResourceLike;
  response?: HttpOperationResponse;
  details?: unknown;
...
}

export interface HttpOperationResponse extends HttpResponse {
  /**
   * The parsed HTTP response headers.
   */
  parsedHeaders?: { [key: string]: any };

  /**
   * The response body as text (string format)
   */
  bodyAsText?: string | null;

  /**
   * The response body as parsed JSON or XML
   */
  parsedBody?: any;

  /**
   * BROWSER ONLY
   *
   * The response body as a browser Blob.
   * Always undefined in node.js.
   */
  blobBody?: Promise<Blob>;

  /**
   * NODEJS ONLY
   *
   * The response body as a node.js Readable stream.
   * Always undefined in the browser.
   */
  readableStreamBody?: NodeJS.ReadableStream;
}

How about a RestResponse and set _response to restError.response?  I think RestError.response should have all the information. Other fields are just information extracted from the response.
export interface RestResponse {
  /**
   * The underlying HTTP response containing both raw and deserialized response data.
   */
  _response: HttpOperationResponse;

  [key: string]: any;
}

I was thinking more along the lines of an actual BlobDeleteResponse (which is a subtype of RestResponse, without the indexer, and with the headers). Users could check _response.statusCode to determine if the entity existed in the first place or not. We could add a friendly property in the future if so desired.
Unfortunately I don't think returning void for now will help much - users will still take a dependency on the falsey value indicating that the object didn't exist in the first place, so moving to anything non-falsy like some RestResponse subtype would be breaking.

I would like to make these methods convenient enough in the first place.
Does this look good to you?
@bterlson @jeremymeng
return type:
BlobDeleteIfExistResponse extends BlobDeleteResponse {
   succeeded: boolean;
}

fit RestError in:
catch (err: RestError) {
  return {
      succeeded: false,
      ...err.response?.parsedHeaders,
      _response: err.response
  }
}

I think @bterlson suggested just returning a BlobDeleteResponse for now. User can check _response if they want to.  One question is: do we have enough information to fill the BlobDeleteHeader?
catch (err: RestError) {
  if ( /* error matches */) {
    return {
        ...err.response?.parsedHeaders,
        _response: err.response
    }
  }
}

I think @bterlson suggested just returning a BlobDeleteResponse for now. User can check _response if they want to. One question is: do we have enough information to fill the BlobDeleteHeader?
catch (err: RestError) {
  if ( /* error matches */) {
    return {
        ...err.response?.parsedHeaders,
        _response: err.response
    }
  }
}

@jeremymeng

do we have enough information to fill the BlobDeleteHeader?

I think we do. First of all, fields in the Header interface are all optional.  When the error is returned, some fields will be undefined but I think it's acceptable. For example, for container.create, if it failed due to "ContainerAlreadyExists", etag and lastModified will be undefined while errorCode is "ContainerAlreadyExists" (which is undefined when the container is created successfully).
export interface ContainerCreateHeaders {
  /**
   * The ETag contains a value that you can use to perform operations conditionally. If the request
   * version is 2011-08-18 or newer, the ETag value will be in quotes.
   */
  etag?: string;
  /**
   * Returns the date and time the container was last modified. Any operation that modifies the
   * blob, including an update of the blob's metadata or properties, changes the last-modified time
   * of the blob.
   */
  lastModified?: Date;
  /**
   * If a client request id header is sent in the request, this header will be present in the
   * response with the same value.
   */
  clientRequestId?: string;
  /**
   * This header uniquely identifies the request that was made and can be used for troubleshooting
   * the request.
   */
  requestId?: string;
  /**
   * Indicates the version of the Blob service used to execute the request. This header is returned
   * for requests made against version 2009-09-19 and above.
   */
  version?: string;
  /**
   * UTC date/time value generated by the service that indicates the time at which the response was
   * initiated
   */
  date?: Date;
  errorCode?: string;
}

I think version 12.2 (minor increment) is appropriate given the new feature set.

Fixed.

I thought about not adding an extra succeeded field and using the errorCode in the original response as an indicator.
It's basically fine except for one scenario - QueueClient.CreateIfNotExists,
where when creating a queue with the specified name already exists, the Queue service checks the metadata associated with the existing queue. if the existing metadata is identical to the metadata specified on the Create Queue request, status code 204 (No Content) is returned, in which case, no errorCode when the queue is not newly created.

/check-enforcer reset

/check-enforcer override


Importing cluster fails with beta.kubernetes.io/os is deprecated since v1.14; use kubernetes.io/os instead

Rancher Server Setup
standalone docker container, new installation 2.7.9
User Information
admin
Describe the bug
trying to import an existing cluster (created by rancher as well, about 9 months ago, fails with
kubectl apply -f https://rancher.my.server/v3/import/6qzk57bkdthftgwnj5glrn79qlc6gv8gvxl7kzll5w7t9cst6wsmdd_c-m-thbfkmmr.yaml
Warning: spec.template.spec.affinity.nodeAffinity.requiredDuringSchedulingIgnoredDuringExecution.nodeSelectorTerms[0].matchExpressions[0].key: beta.kubernetes.io/os is deprecated since v1.14; use "kubernetes.io/os"                                                     instead
I downloaded the yaml file and edited it, the error disappeared, cluster was imported after some time. However not as a "rancher"-cluster, but as a simple "3rd party", i.e. without node templates, node pools, no update via rancher possible.

Hi guys
I believe it's the same issue I see on my fresh 2.8.0 Rancher, installed an 1.27.8-rancher1 RKE Kubernetes, and trying to edit
the cattle-cluster-agent deployment (for testing purpose only):
$k edit deployment -n cattle-system         cattle-cluster-agent
Warning: spec.template.spec.affinity.nodeAffinity.requiredDuringSchedulingIgnoredDuringExecution.nodeSelectorTerms[0].matchExpressions[0].key: beta.kubernetes.io/os is deprecated since v1.14; use "kubernetes.io/os" instead
deployment.apps/cattle-cluster-agent edited


[Testing] "Welcome to Swift" does not always print LLVM/clang version

@swift-ci please test platform


HAWKULAR-341 URLs with 2 x-powered-by header fail

As you are separating the values by comma, wouldn't it be easier to skip the multi-value case and have a map<header,String> where you check if get(header) is not null and if so you just append ", " + other value?  Or do you fear duplicates here?

@pilhuhn I guess, duplicates are not that frequent in the wild web (although this solves the problem well, when they occur). My main motivation for the present approach has to do with how we look at the identity of the values:
(1) Note that the present approach sorts the multiple values alphabetically. Having some kind of canonical order should make aggregation over x-powered-by more reliable. Let's say we want to count instances by used technology. With sorted values, both "ARR/2.5, ASP.NET" and "ASP.NET, ARR/2.5" will be counted together, which is good.
(2) The same holds for the detection of changes. I have no idea how reliable is it to assume that the servers are sending the headers always in the same header. I doubt that there some spec prescribing them to send the headers in the same order for two identical (or even in some sense similar) requests. I think it is theoretically possible that a server sends "ARR/2.5, ASP.NET" once and "ASP.NET, ARR/2.5" other time. With the present approach, there is no change for the inventory, which is good IMO again.
Does that sound acceptable?

Sure, was just asking :)


permissions

Hi,
we just used this documentation, which is great, many thanks!
We noticed that following permissions was not sufficient:
"the Storage Blob Data Contributor role on the Azure Data Lake Storage Gen2 account."
It would be great if you can check with the responsible engineering team and confirm if there was a change. Once we provided "owner" access, the connection worked fine.
Thank you
Michael

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

ID: 46a924e8-1757-5b5a-ad6d-930e316c3196
Version Independent ID: 6fac1727-bb8a-f993-e480-d9bcfc8ef9f7
Content: Create an Azure Synapse Link for Dataverse with your Azure Synapse Workspace - Power Apps
Content Source: powerapps-docs/maker/data-platform/azure-synapse-link-synapse.md
Service: powerapps
Sub-service: dataverse-maker
GitHub Login: @sabinn-msft
Microsoft Alias: matp

Thanks for your comment, @MichaelWeberMSFT. This issue has been assigned to the doc author for review.


Playground zooming way too much on mobile version of Chrome

For some reason, the playground started dooing microscopic zooms when clicking on any part of it, it is very annoying. I used it some months ago to test some snippets, and some time ago it became that annoying.

In general, the fancy editors don't support mobile. Try switching to a native textbox instead:

Simple does work without zooming to the pixels of a letter, but even though i don't mind losing syntax highlighting, things like auto indentation and closing brackets (+ everything else) are very convenient.
I'll be using simple as a workaround, but could an editor not be as fancy as ace but work fine with those utilities like the one that that Programiz website uses? I mean, if it was done, it can be done again.

like the one that that Programiz website uses

Interesting, as it appears that site even uses Ace!
<div id="editor" class=" ace_editor ace_hidpi ace-tm">

I've opened #847 to check into that.


Update FluentStorage to Microsoft.IO.RecyclableMemoryStream 3.0.1

Fixes
The breaking change in Microsoft.IO.RecyclableMemoryStream  which now returns a RecyclableMemoryStream instead of a stream
Issue #84
Description
Updated the SinkedStream to return a RecyclableMemoryStream in place of a MemoryStream stopping the error being thrown.  The class inherits from MemoryStream so the code works without any further changes

Since it is required for future proofing this library, I am accepting this as is. Thanks for this contribution.

Released.

https://www.nuget.org/packages/FluentStorage/5.6.0
https://www.nuget.org/packages/FluentStorage.Azure.Blobs/5.3.0
https://www.nuget.org/packages/FluentStorage.AWS/5.5.0


Add animation for spikes

Maybe Medic Kneel?

Added Medic Kneel for player, working on animation for the spikes themselves.


Top level beforeAll, beforeEach(), afterAll, and afterEach

Right now, when these are declared outside of a describe(), they don't run. This would be useful to setup o mocking HTTP server like https://mswjs.io/ and adding it to setupFiles in the peeky config to automatically add this to all test suites:
import { server } from './mock/server.ts' // msw specific

beforeAll(() => server.listen())
afterEach(() => server.resetHandlers())
afterAll(() => server.close())

@posva I'm using it outside a describe() and it's working fine

I think @Akryum implemented this but forgot to close it


[BUG] GetProcessorCount for webjobs service bus extension does not override processor count for flex consumption

Library name and version
Microsoft.Azure.WebJobs.Extensions.ServiceBus <IP_ADDRESS>
Describe the bug
Logic that uses the processor count to figure out default value of maxConcurrentCalls needs to special case the flex consumption SKU similar to how it special cases the consumption SKU.
This logic is in the GetProcessorCount helper.
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/servicebus/Microsoft.Azure.WebJobs.Extensions.ServiceBus/src/Utility.cs#L14
Here is some relevant code from functions host for detecting flex consumption via environment variables:
https://github.com/Azure/azure-functions-host/blob/f58df65944a2eca87c7ce0e837b2141f4b0a95bf/src/WebJobs.Script/Environment/EnvironmentExtensions.cs#L225
https://github.com/Azure/azure-functions-host/blob/f58df65944a2eca87c7ce0e837b2141f4b0a95bf/src/WebJobs.Script/ScriptConstants.cs#L119
Expected behavior
Flex consumption apps default to using maxConcurrentCalls = 16.
Actual behavior
Flex consumption apps default to using maxConcurrentCalls = 64, because the system thinks the processor count is 4.
Reproduction Steps
Deploy a service bus triggered function on flex consumption and observe the concurrency behavior.

@pragnagopa  I think it makes more sense for someone on the functions team to make this fix


How to create nested objects on the fly?

I have started developing a demo app for DunglasApiBundle. I tried to deserialize a new User with a new Group:
POST api/users

{"_format": "json",
"groups": [{
"name":"newgroup"
}],
"name": "test12311"
}

I have configured serialization groups, however, I'm receiving the following error:
hydra:title: "An error occurred"
hydra:description: "Nested objects for attribute "groups" of "AppBundle\Entity\User" are not enabled. Use serialization groups to change that behavior."

Here's my entity: https://github.com/felicitus/DunglasApiBundleDemo/blob/master/src/AppBundle/Entity/User.php

Hi,
There was some issues I fixed in felicitus/DunglasApiBundleDemo#1:

annotations where not enabled: https://github.com/felicitus/DunglasApiBundleDemo/pull/1/files#diff-bf0e70bcef1a5d5b2f87289220a51108R21
the service definition was not OK: https://github.com/felicitus/DunglasApiBundleDemo/pull/1/files#diff-1728bfc1c274f341afc1a0275fca694dR14
cascade wasn't properly configured: https://github.com/felicitus/DunglasApiBundleDemo/pull/1/files#diff-0f7d9a537f1062976ab4c8e2fa39ad72R28

It should be OK now. Can I close?

Yes you can close this, however, it's not clear to me when to use array brackets [] in the service definition as arguments for the "initDenormalizationContext" service and when not. I initially tried without [] and got an error, then I added them which worked for me. Now, with your pull request, it seems to work without [] again.

Before the call, something like the following (which is invalid) was generated:
$r->initDenormalizationContext(array(array('groups' => user)));


Front carousel images

3 good images for the carousel?

Do we resize the carousel a little?

Are you are thinking smaller or larger? It could go a little taller but I like the ability to see the top of the text on the page. I'm old-fashioned though and I don't like web sites like this: http://billbyronwines.com/
I want to read what you have to say without having to scroll around for it.
I originally opened this ticket to confirm that the three images we have are the ones we want to show.

Resolved


Tidied up bob

I have stripped out the one line extension methods, and attempted to simplify the logic. This might make the solution less maintainable, but for the exercise is more simplistic.

That's a lot neater, thanks Doug!


Crash: Error while updating property 'source' of a view managed by: RNCWebView

Upgraded from react-native-webview 11.26.1 to 12.4.0 and since then I'm getting this crash when the webview is opened:

Error while updating property 'source' of a view managed by: RNCWebView
null
null

If that can make a difference the source is first set to undefined in our code, then soon after to {"baseUrl": "file:///data/user/0/net.cozic.joplin/files/", "uri": "file:///data/user/0/net.cozic.joplin/files/NoteEditor.html?r=36630425"}

It doesn't happens on iOS
It happen on Android

Ok, found the issue, it's because when the source is null, our code set it to { uri: null }. That works on iOS but crashes on Android. However when setting it to { uri: undefined } it works without crashing.
So that's another breaking change in version 12. Perhaps it should be documented?

That's very unexpected, let me have a look

Probably that line: https://github.com/react-native-webview/react-native-webview/blob/master/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManagerImpl.kt#L384
It's expecting that if there is a "uri" field it will parse the string

Anyone found a solution for this? getting it on android as well, on version 12.0.2, trying to load a local file. iOS has no issues

@Titozzz I still am unable to resolve this issue on android.
Its working fine on IOS but not on android. Please look in to this

@webraptor were you able to solve this?

@laurent22 were you able to solve this?

<WebView
    mediaPlaybackRequiresUserAction={true}
    source={{uri: url || undefined}}
/>

<WebView source={{uri: url || undefined}} />

This worked for me. Thank you.

<WebView source={{uri: url || undefined}} />

This worked for me. Thank you.

Where do i put this

<WebView source={{uri: url || undefined}} />

This worked for me. Thank you.

Where do i put this

Here is an example of how I am using this. This is its own component and I call it in another component.
  return (
    <>
      <SafeAreaView style={styles.safeContainer}>
        <TouchableOpacity
          onPress={() => navigation.navigate('Main')}
          style={styles.button}
        >
          <Text style={styles.submitText}>Close</Text>
        </TouchableOpacity>
      </SafeAreaView>
      {(!webviewUrl && !shouldShowWebView) || !isVisible && (
        <View style={[styles.loadingPage, {zIndex: 10}]}>
          <ActivityIndicator size={'small'} color={greenBackgroundColor}/>
          <Text style={styles.loadingText}>Connecting to WebView...</Text>
        </View>
      )}
      {webviewUrl && shouldShowWebView && (
        <WebView
          source={{ uri: webviewUrl }}
          injectedJavaScript={...} // JS provided to me by the creator of the web view I am using
          onNavigationStateChange={_onNavigationStateChange} // a method I implemented
          onMessage={_onMessage} // a method I implemented
          thirdPartyCookiesEnabled={true}
          userAgent={...} // provided by the creator of the web view I am using
          mediaPlaybackRequiresUserAction={true}
        />
      )}
    </>
  );

I have fixed this, turns out when I updated expo cause of SDK 52, I didn't install it's dependencies but thanks tho

<WebView source={{uri: url || undefined}} />

This worked for me. Thank you.

Where do i put this

Here is an example of how I am using this. This is its own component and I call it in another component.
  return (
    <>
      <SafeAreaView style={styles.safeContainer}>
        <TouchableOpacity
          onPress={() => navigation.navigate('Main')}
          style={styles.button}
        >
          <Text style={styles.submitText}>Close</Text>
        </TouchableOpacity>
      </SafeAreaView>
      {(!webviewUrl && !shouldShowWebView) || !isVisible && (
        <View style={[styles.loadingPage, {zIndex: 10}]}>
          <ActivityIndicator size={'small'} color={greenBackgroundColor}/>
          <Text style={styles.loadingText}>Connecting to WebView...</Text>
        </View>
      )}
      {webviewUrl && shouldShowWebView && (
        <WebView
          source={{ uri: webviewUrl }}
          injectedJavaScript={...} // JS provided to me by the creator of the web view I am using
          onNavigationStateChange={_onNavigationStateChange} // a method I implemented
          onMessage={_onMessage} // a method I implemented
          thirdPartyCookiesEnabled={true}
          userAgent={...} // provided by the creator of the web view I am using
          mediaPlaybackRequiresUserAction={true}
        />
      )}
    </>
  );

I have fixed this, turns out when I updated expo cause of SDK 52, I didn't install it's dependencies but thanks tho


infinite loop when connecting to orientdb that's running in distributed mode

I'm using orientDB v.2.2.7 on AWS in distributed mode.
When connecting using pyorient 1.5.4, the connection freeze up.
Inside the OrientSerializationCSV class, on the "_parse_collection" method, the while loop goes into an infinite loop.
It looks like "_parse_set(self, content)" can't really handle unexpected formatting. It seems very unreliable. If "_parse_value" returns the same content as what was passed into it, as per the last "else" statement, the entire "_parse_set" method ends up in an infinite loop.
From my tests, it looks like "_parse_set" is only called when connecting to a distributed database. In my case, the code tries to parse the following string:
<"MyDatabase">,
usedMemory: 143389672l,
freeMemory: 375589912l,
maxMemory: 518979584l,
latencies: (node-0AbxA4VVo9PcEmOC: (entries: 200l,
last: 788930l,
min: 0l,
max: 788930l,
average: 788930.0f,
total: 788930l,
firstExecution:<PHONE_NUMBER>720l,
lastExecution:<PHONE_NUMBER>139l,
lastReset:<PHONE_NUMBER>139l,
lastResetEntries: 1l)),
messages: (heartbeat: 199l,
deploy_db: 1l)),
(id: 0,
uuid: "6c9ab948-547d-4e91-b2a2-0e8381d91f0e",
name: "node-0AbxA4VVo9PcEmOC",
startedOn:<PHONE_NUMBER>365t,
status: "ONLINE",
connections: 2,
listeners: [
  {
    "protocol": "ONetworkProtocolBinary",
    "listen": "<IP_ADDRESS>:2424"
  },
  {
    "protocol": "ONetworkProtocolHttpDb",
    "listen": "<IP_ADDRESS>:2480"
  }
],
user_replicator: "4047485640755908123",
databases: <"MyDatabase">,
usedMemory: 145392192l,
freeMemory: 373587392l,
maxMemory: 518979584l,
latencies: (node7096-5453: (entries: 200l,
last: 815985l,
min: 0l,
max: 1488559l,
average: 983307.0f,
total: 3933231l,
firstExecution:<PHONE_NUMBER>530l,
lastExecution:<PHONE_NUMBER>562l,
lastReset:<PHONE_NUMBER>562l,
lastResetEntries: 4l)),
messages: (heartbeat: 200l))
],
database: (autoDeploy: true,
hotAlignment: false,
executionMode: "asynchronous",
readQuorum: 1,
writeQuorum: 2,
failureAvailableNodesLessQuorum: false,
readYourWrites: true,
servers: (*: "master"),
clusters: (internal: (),
index: (),
*: (servers: [
"node-0AbxA4VVo9PcEmOC",
"node7096-5453",
"<NEW_NODE>"
]),
posts_base_35: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
posts_base_24: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
posts_base_34: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
posts_base_23: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
posts_base_6: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
posts_base_33: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
posts_base_7: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
posts_base_10: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
---------------------------------
Keeps going for each cluster
---------------------------------
post_9: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
]),
post_26: (servers: [
"node7096-5453",
"node-0AbxA4VVo9PcEmOC",
"<NEW_NODE>"
])),
version: 410)

"_parse_set" is not able to parse that set of values and results in the connection locking up.
Could we not use another deserializer instead?

"_parse_collection" has a similar issue. Any while loop that makes use of "_parse_value" run the risk of infinite looping if the format is unexpected. "_parse_value" is not a friend to us :(

Rather than hacking away at the current parser, I want to use another deserializer instead but that would introduce a dependency into pyorient. I'm ok with that for my project but that would probably not be ideal for the pyorient project in general.
I'm proposing that instead of specifying a "serialization_type" on "connect", why not pass the serialisation class instead?
That way we can use our own implementation. The current OrientSerializationBinary is bogus anyways.
Also, can't we pass a serialization on instantiation of pyorient? It's silly to pass it on "connect" when it might as well be passed to "db_open". In some cases, "connect" is not needed to be called.
it could look like this:
client = pyorient.OrientDB('localhost', 2424, serializer=MyCustomSerializer)

I tried using the development branch with the new Binary serialization. It works perfectly with the servers running in distributed mode.

There is a bad memory leak in the development branch. Use with caution.

The memory leak is caused by "pyorient_native". It's a shame because the binary serialize implementation is much faster than the CSV one and it's the only option that works in distributed mode :(

The Memory leak with pyorient_native has been resolved. Merging the binary serializer into the master branch is now a viable option for solving the distributed mode issue.

Here is a solution to the original bug.
There is currently a development branch on pyorient that implements the missing binary serializer (OrientSerialization.Binary). I have another branch that has some fixes merged into it.
Install it with:
pip install https://github.com/anber500/pyorient/tarball/17f5e42e83859a661c6483f7fa812226194694dd#egg=pyorient

Set your serialiser as follows:
client = pyorient.OrientDB("localhost", 2424, serialization_type=pyorient.OrientSerialization.Binary)

You will also need an updated version of pyorient_native. The first release had a memory leak so use the version from the master branch:
pip install https://github.com/nikulukani/pyorient_native/tarball/master#egg=pyorient_native

This works perfectly on AWS in distributed mode and is much faster than the CSV serializer.

@anber500 could do regular push request against develop branch with your fixes?
Thanks a lot!!

I would but I didn't do any of the fixes. All the fixes has been stuck in push request for a while and we couldn't wait weeks for a fixed release. I only forked the code to merge a patch into the develop branch that was given to me by the developer of pyorient_native. I saw the code was missing form the develop branch and I patched it.
To be honest, I would prefer to pull from an official build but I missed 2 sprints waiting for this fix and I'm sure other developers are just as desperate for a fix.

I have also had some chats with the people from OrientDB. They have their own forked version of pyorient  which they point to in their documentation. It's looks like they "officially" support that fork but they don't. It would be really great if the fixes could make it's way back upstream. As a newcomer to OrientDB, I must admit that it's very confusing to have to deal with all these forks and no one knows which ones work and which ones doesn't.

@anber500 ok now it's merged in the develop branch.
Yes I know it's confusing we have to clean up all the messy with forks and the likes, we are chatting with people from OrientDB too to understand how to do it
It will be better I promis :D

We have experienced exactly this same problem here...
Installing pyorient_native as you have instructed worked fine (thanks!) but only in the linux environment (in our development environment, windows systems, pip wasn't able to install it cause of a broken dependency, stdc++.lib).
Hope to have this patch merged into the main pyorient branch soon!
Regards
Joao

@marigonda I think the branches have been merged. We honestly haven't run into this issue because we deploy via dockers. I looks like stdc++.lib is a mingw dependency. Have you tried manually adding the lib to your environment?

Thanks for your reply. I will stay in debt with you about the answer regarding the stdc++.lib on windows ... btw, our option would be for the linux environment anyway....
About the merge, pip3.5 on CentOS7 (Python 3.5 from IUS Repository) did not install the native version with binary deserialization support, I had to do it manually, as you have instructed earlier (thanks again).
Now things are starting to work fine but it seems we do not have OGM support with binary deserialization, do we?
Thanks very much for your time!

You can use the OGM with binary deserialization if you are using the develop branch of pyorient and have pyorient_native installed
from pyorient.ogm import Graph, Config from pyorient.serializations import OrientSerialization graph = Graph(Config.from_url('my_db','root', 'root', initial_drop=False,                                      serialization_type=OrientSerialization.Binary))


No module named 'datasets.COCO.nms.cpu_nms'

I followed the instruction and tried to run a test.
However, something seems to be wrong with coco api.
I have installed coco api before which was working.  I used it in my other work before.

Traceback (most recent call last):
File "main.py", line 2, in
import builder
File "/scratch/liu.shu/codesPool/Pytorch-Human-Pose-Estimation/builder.py", line 5, in
import dataloaders as dataloaders
File "/scratch/liu.shu/codesPool/Pytorch-Human-Pose-Estimation/dataloaders.py", line 1, in
import datasets
File "/scratch/liu.shu/codesPool/Pytorch-Human-Pose-Estimation/datasets/init.py", line 2, in
from datasets.coco import *
File "/scratch/liu.shu/codesPool/Pytorch-Human-Pose-Estimation/datasets/coco.py", line 2, in
from datasets.COCO.coco import COCODataset
File "/scratch/liu.shu/codesPool/Pytorch-Human-Pose-Estimation/datasets/COCO/init.py", line 11, in
from .coco import COCODataset as coco
File "/scratch/liu.shu/codesPool/Pytorch-Human-Pose-Estimation/datasets/COCO/coco.py", line 24, in
from .nms.nms import oks_nms
File "/scratch/liu.shu/codesPool/Pytorch-Human-Pose-Estimation/datasets/COCO/nms/nms.py", line 13, in
from .cpu_nms import cpu_nms
ModuleNotFoundError: No module named 'datasets.COCO.nms.cpu_nms'

I encountered the same problem, have you figured out how issue?
Cheers

I encountered the same problem, have you figured out how issue?
Cheers

I thought it missed the necessary Makefile for nms.
I assumed this part was originated from this repo.
You can use their nms instead.

Yes I believe @ShuangjunLiu is right. You need to build nms for COCO dataset as per the instructions here


[BEAM-8258] basic metric feature for nexmark

added beam-metric based performance monitoring
performance are logged to console after the query is done or canceled by nexmark suite

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @y1chi
R: @pabloem

Run Portable_Python PreCommit


Metrics for G1 garbage collector need to be parsed correctly

Currently, heron-instance does gather the metrics gc-count and gc-time-ms for all types of gc in JVMMetrics (https://github.com/twitter/heron/blob/a905adde3888f508dfd6c9b57e1959f093810674/heron/common/src/java/com/twitter/heron/common/utils/metrics/JVMMetrics.java#L411).
And the metric name is: __jvm-gc-count/{GarbageCollectorMXBeanName}.
For instance, for G1GC:
__jvm-gc-time-ms/G1-Old-Generation
__jvm-gc-count/G1-Old-Generation
__jvm-gc-time-ms/G1-Young-Generation
__jvm-gc-count/G1-Young-Generation
Notice, these metrics are accumulative values so you would have to do a rate if you want to know the gc behavior for every particular interval.
Also, we have gc metrics summing up all types of gc for a jvm process:
__jvm-gc-collection-time-ms
__jvm-gc-collection-count
Attached an example generated by the file-sink in metrics-mgr:
metrics.txt
localhost:1/exclaim1/container_1_exclaim1_1 is using G1GC by adding "-XX:-UseConcMarkSweepGC -XX:+UseG1GC -XX:MaxGCPauseMillis=200" to Config.TOPOLOGY_WORKER_CHILDOPTS

"-XX:+UseConcMarkSweepGC" is removed in JDK 9, a humble FYI!

I believe this issue has been resolved. Closing it for now. Please add a new issue if there is still an issue.

I believe this issue has been resolved. Closing it for now. Please add a new issue if there is still an issue.


Per clients entity

Describe the problem related to your feature request.
Sometime, you need to have entities that can only be seen by only one or a group of players.
What solution would you like?
A composant that reference the group of client that the packet will be send to.
If the composant is not present, the entity is normally process.
What alternative(s) have you considered?
Maybe dispawn the entity if all client are disconnected

Is this a duplicate of #342?

Yea it basically is


Text Format

Hello rfinochi,
Is there a way to retrieve messages in Text format ? Or did I miss something ?
Thank you for this project.
Sabelle

Hi!
Message are always retrieved in Text Format, check the Body property of PopMessage class.


[ BUG ]

Hello Zero,
I just hosted the bot for my server the bot works fine by its not showing hosted by, i enabled in the config too but still its not showing in giveaway.

Solved:
If anyone facing this try this method!
Try putting hostedBy: interaction.user in start.js file in giveaway data section after message,


Update Node.js version to 4.8.3

As one cannot submit an issue in this repo I post here my problem (it's an emergency). If I use base OR node-4.8.4-base for my meteor 1.5 app the error is : sh: 1: node-pre-gyp: Permission denied. And if I use with binbuild (which has pre-gyp right ?) I have the "/bundle/bundle/programs/server/npm/node_modules/sshpk/bin\sshpk-conv" error

Hello, @NitroBAY.
Can you check the tag node-8.1.4-base with your app?

I've finally the "bin" section of the package that's causing trouble while keeping your docker image. I've no clue about the utility of this image in my fix but nevertheless it's working so I'm alright.


PowerPC Mac OS X Guests refuse to connect with Software Update server.

Describe the issue
PowerPC Mac OS X Guests being connected to the network and loading webpages as old web browser can, they refuse to connect with Software Update server, tested every Mac OS X except Tiger (since my instal discs don't really contain "latest the greatest"), I'm kinda stuck to looking around for Combo update files. I've included Panther debug.log but each VM is just copy-paste of settings and issue with Software Updates.
Configuration

UTM Version: 2.4.1
OS Version: macOS 12.1 Monterey
Intel or Apple Silicon? Apple Silicon M1 MacBook Air

Debug log
Panther_Debug.log
Upload VM
config.plist.zip

Your configuration looks good, bridged mode network is the best for avoiding network problems with some guests. That said this does not sound like a UTM-related problem. Are you sure this Software Update still works on real PPC Macs? Servers or SSL certificates might have changed.
Which Mac OS X specifically have you tested and which of them had this problem?
Do you have a way to test with a real USB network card passed to the VM via USB Sharing?

Your configuration looks good, bridged mode network is the best for avoiding network problems with some guests. That said this does not sound like a UTM-related problem. Are you sure this Software Update still works on real PPC Macs? Servers or SSL certificates might have changed.

On real PPC Macs I have no idea if it still works because I don't have one (I'm planning to get iMac G4 but its kinda pricey, I guess the design is a thing) but so far I found official combo updates hosted by Apple so I assume it still should work but in 10.5 for example I ended up boot looping after installing Combo (tried a few times on VM copies, same thing).
Out of like 7 cloned VMs with different Mac OS 9 (but since MacOS9Lives ISO exists I won't bother bringing its topic) and X - Software Update works perfectly on Tiger only which is a bit of funny since all my VMs are cloned from each other and upgraded. I checked, all of them have different IPs assigned by DHCP so they don't fight with each other on the network.

Which Mac OS X specifically have you tested and which of them had this problem?
10.0 - Immediately kicks me out with message I'm up to date or gets stuck on findings without any errors (I'm on 4K78 as the CD was shipped with it, it's not latest).
10.1 - Internet connection works as its expected on 20 yo OS but yeah, Software Update just ignores me.

10.2 - Failed. Error occurred while checking. {Shows my correct timezone}
10.3 - Update server could not be found. Make sure you're network is configured properly, then try again.

10.4 - No issues.
10.5 - Error -1001 connection with server gets timed out, sometimes after deleting two plists it finds the updates but fails to install them because connection issues even tho network diagnostics is all green. Sometimes it's able to find updates after deleting com.apple.SoftwareUpdate.plist from /Library/Preferences/ (https://discussions.apple.com/thread/4752087) but fails to install them because of the same error with connection timed out.

Do you have a way to test with a real USB network card passed to the VM via USB Sharing?

Unfortunately, no.

In #3404 you wrote something to the effect of, you were able to get online in the VM by switching to emulated VLAN mode. Please close this issue if that is the case.

In #3404 you wrote something to the effect of, you were able to get online in the VM by switching to emulated VLAN mode. Please close this issue if that is the case.

Yeah, will do that, forgot, it's weird because QEMU used from command line applies that by default, UTM even if replicated in GUI setting doesn't work, needs to be added as QEMU argument.


Remove TODO tag for closed issue

Remove TODO tag for issue #2689.

There are no changes in this PR. I found where this comment now and will remove it in my PR


🚀 Feature: Support pnpm as alternative package manager to yarn

🔖 Feature description
Offer support for the existing "packageManager" configuration setting for Node projects.
Please consider removing the explicit dependency on yarn in the "backstage-cli" and support alternative package managers like pnpm.
🎤 Context
We have a monorepo built with a lot of pnpm automation, but the Backstage instance directories including all projects / workspaces require yarn and must be excluded with special treatment.
pnpm can offer some benefits to teams over yarn depending on the features they value most.
✌ Possible Implementation
Read the preferred package manager tool from the "package.json" file and the configuration setting "packageManager". Support this tool in all automation including "backstage-cli".
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Some loosely related issues just for history:

https://github.com/backstage/backstage/issues/11570
https://github.com/backstage/backstage/issues/8054

There's a lot of customization around the yarn.lock file in the backstage-cli as you noted, the DevTools plugin also uses it for its Info tab feature

Would love to add support for more package managers! This is a capacity/priority issue, happy to help guide any contributions in this area. We don't want the CLI to require you to use Yarn, in fact there are many parts of the CLI that should work without it as long as you're using package manager with the workspaces monorepo setup. Things like version:bump will for sure not be supported though so we'd need more work to get there.

Very simple and scoped PR for npm support at https://github.com/backstage/backstage/pull/20670 but fully working to just build everything without yarn which was my main use case. Will take a stab at making it slightly more extensible as that seems the overall interest.

I am happy to add pnpm to #20670 if someone can test it (see https://github.com/backstage/backstage/pull/20670#discussion_r1369750361)

Please add support for pnpm to the backstage project and corresponding backstage-cli.

@ESmithaaS, can you share some reasons why you'd like this? Would be helpful to have more information as this is something the Community would need to help with. Being able to learn about the potential benefits would be great 👍

I'm also looking into this, we run a pnpm monorepo and would like to pull our Backstage implementation into that monorepo. Primary benefits for us are mostly monorepo-related, ie being able to use our centralized tooling and easier usage of shared libraries (workspace:*).
PNPM has a bunch of its own benefits, like stricter peer dependency resolution (#25307) and better space efficiency than yarn/npm. https://rushjs.io/pages/maintainer/package_managers/#considerations-for-pnpm

Thanks for the details @aramissennyeydd, super helpful!

@awanlin, here's another use case. My company uses Bazel for building and testing code, although the currently maintained Bazel rules for JavaScript rely on pnmp, so wrapping Bazel rules around Backstage's code has proven challenging.

Thanks @jdswalker, that's great. Have you considered contributing support?

It looks like the work here is to separate the existing Lockfile class into 2 (or more) versions, 1 for yarn and 1 for pnpm. https://github.com/backstage/backstage/blob/master/packages/cli/src/commands/versions/bump.ts#L80-L81, and update all uses to use that new Lockfile with implicit / maybe config based locations for the lockfile. One complication here is that our build system uses a non-root lockfile location (common/config/rush/pnpm-lock.yaml), so the location discovery would need to be configurable somehow.

@ESmithaaS, can you share some reasons why you'd like this? Would be helpful to have more information as this is something the Community would need to help with. Being able to learn about the potential benefits would be great 👍

@freben I find Performant NPM (PNPM) to be a easier package manager to work with than Yarn. It has better documentation. It also provides better management of dependencies, and disk space. Take a look at Benchmarks and Feature Comparison I also link to a few call-outs below that the community may find beneficial.
A few noticeable call-outs:
Better Docker Builds
CI -- Github Example
More CI examples
Git Branch Lockfiles
Workspaces (Monorepos)
Workspace Catalogs
peer dependency management

Here are an article that makes a compelling reason to support PNPM
Medium Post

So here is something ridiculous, if I install packages inside backstage with yarn, everything works as expected. however if I install them with pnpm, I get this error:

What has visibility to do with package management?
Missing required config value at 'backend.baseUrl' in 'app-config.yaml' this error is actually covered by the warning in the console: Failed to read configuration value at 'backend.baseUrl' as it is not visible. See https://backstage.io/docs/conf/defining#visibility for instructions on how to make it visible.

@adminy That's likely a version mismatch between packages causing a context or similar up the tree to not be found. Initial thought here might be mismatched versions of @backstage/core-app-api and the ApiProvider. Ran into a similar thing with #25278 for MUI. The cost of correctness :)

I've developed enough of an interest in this to start work on refactoring backstage-cli to allow for pnpm support. Note this is still a draft PR because it's a complete mess. Still lots of work left to get this ready for review. https://github.com/backstage/backstage/pull/27095

To anyone here who has expressed an interest in PNPM support, I would greatly appreciate your help! Please take a look at https://github.com/backstage/backstage/pull/27269 and have a go at implementing these interfaces for PNPM. You can see a full POC for Yarn with these interfaces in https://github.com/backstage/backstage/pull/27095 if it helps. I have zero experience working with PNPM programatically and from what I've seen the lockfile parsing in particular is kind of tricky. Thanks!

hmm, Yarn actually supports pnpm style module linking but it's not used here: https://yarnpkg.com/configuration/yarnrc#nodeLinker


Add classes for plotting ROC curve and displaying confusion matrix

check out if the confusion matrix looks somehow okay, too


Depth First Search (DFS)

Hey !
I want to add the Depth First Search (DFS) algorithm . Depth first traversal is a recursive algorithm for searching all the vertices of a graph.

refer #186


Testing session

Several issues detected and fixed

Duplicate SkipToLink on the archived news page
Trailiing / for the root pages in the language switch
Lot's of warning for images

Created a new issue for 404 trouble with the archived news https://app.zenhub.com/workspaces/energy-vision-5fb6386c578f1200222dd6c2/issues/equinor/energyvision/576


Error: Unexpected type of example_input

Describe the bug
Can't use dictionary type for example_input when trying to convert pytorch model
Expected behavior
Not giving this - "Error: Unexpected type of example_input" and successfully convert the pytorch model to openvino
Screenshots
See attached for screenshot.

Installation instructions (Please mark the checkbox)
[ x ] I followed the installation guide at https://github.com/openvinotoolkit/openvino_notebooks#-installation-guide to install the notebooks.
Additional context
I'm trying to convert the UNet model from diffusers library's StableDiffusionPipeline but with an additional parameter called "added_cond_kwargs" that is supposed to be Dict[str, torch.Tensor]. But openvino's ov.convert_model(unet, example_input) function is giving the error "Error: Unexpected type of example_input".
It seems openvino's pytorch_frontend_utils.py script raises an error if the type of example_input doesn't match the expected type (see https://github.com/openvinotoolkit/openvino/blob/9dad09aea42977a9b237b717da82bdbe5cefb0ff/tools/mo/openvino/tools/mo/moc_frontend/pytorch_frontend_utils.py#L158) but the diffusers library's UNet model requires the added_cond_kwargs  parameter to be a Dict (see https://github.com/huggingface/diffusers/blob/645a62bf3bcba47f97bf2a36a8689f40bd6b04fd/src/diffusers/models/unet_2d_condition.py#L843).
Please help! Thank you.

@kolabearafk, Hello, support directly passing dict as parameter  of example input is planed feature for openvino release 2023.3. You can test it using openvino-nightly  package(please note, it can be incompatible with openvino and openvino-dev installed in your env, it is recommended to delete these packages before installing openvino-nightly or use this package in separated virtual env).
With existing release, this issue can be workarounded creating wrapper on model with passing keys of dict as arguments and building dict from them inside wrapper forward as demonstrated in this notebook (in LCM related part) https://github.com/openvinotoolkit/openvino_notebooks/blob/main/notebooks/248-stable-diffusion-xl/248-ssd-b1.ipynb

Hi @eaidova,
Thank you for your reply! I tried a similar way mentioned in the above notebook in the LCM part with openvino 2023.2. Below is the code used. There's still a problem with TorchScript not being able to do tracing with the exception: mat1 and mat2 shapes cannont be multiplied (2x1280 and 1024x3072). Please see the attached screenshot below for the error message.
`
from diffusers import StableDiffusionPipeline
import torch
import gc
import numpy as np
import openvino as ov
pipe = StableDiffusionPipeline.from_pretrained("runwayml/stable-diffusion-v1-5")
pipe.load_ip_adapter("h94/IP-Adapter", subfolder="models", weight_name="ip-adapter_sd15.bin")
def convert(model: torch.nn.Module, xml_path: str, example_input):
xml_path = Path(xml_path)
if not xml_path.exists():
xml_path.parent.mkdir(parents=True, exist_ok=True)
with torch.no_grad():
converted_model = ov.convert_model(model, example_input=example_input)
ov.save_model(converted_model, xml_path)
    # cleanup memory
    torch._C._jit_clear_class_registry()
    torch.jit._recursive.concrete_type_store = torch.jit._recursive.ConcreteTypeStore()
    torch.jit._state._clear_class_state()

UNET_OV_PATH = Path('model/unet_ir.xml')
class UNETWrapper(torch.nn.Module):
def init(self, unet):
super().init()
self.unet = unet
def forward(self, sample=None, timestep=None, encoder_hidden_states=None, image_embeds=None):

    return self.unet.forward(
        sample,
        timestep,
        encoder_hidden_states,
        added_cond_kwargs={'image_embeds': image_embeds}
    )

example_input = {
'sample': torch.rand([1, 4, 128, 128], dtype=torch.float32),
'timestep': torch.from_numpy(np.array(1, dtype=float)),
'encoder_hidden_states': torch.rand([1, 77, 2048], dtype=torch.float32),
'image_embeds': torch.rand([2, 1280], dtype=torch.float32),
}
pipe.unet.eval()
w_unet = UNETWrapper(pipe.unet)
convert(w_unet, UNET_OV_PATH, example_input)
`

I also tried the openvino-nightly version after uninstalling the openvino 2023.2 version. But that one gives a different error when passing a dictionary for added_cond_kwargs parameter in the unet. Please see the below screen for the error message.

Ultimately, I'm trying to see if OVStableDIffusionPipeline can be extended with IP-Adapter feature because I don't see any notebook example for it in this repo.
Thanks so much!


compose: Increase click target area around banner close button.

Using padding instead of margin allows a wider area around the button where the user can click from to still functionally click the button.

Hm, would it be better for it to be symmetric around the x? It currently feels like way more space on the left than the right.
@terpimost for any thoughts here.

Oh you're totally right, good catch. There's no right padding -- I can add that!

Padding is now on the right side as well!

Could you post a screenshot that shows what the target area is now? Thanks!

ah sorry -- here it is!

That looks good to me (I haven't re-tested manually). @timabbott could you please take a look? Hopefully a relatively quick one.

Merged, after adding a comment to help explain that right_edge class.
It's possible there's something nicer/fancier we could do involving a padding-left on an element that sits at the right border, but I'm not sure it's worth more attention to try to do that.


⚠ Minecraft has degraded performance

In d8ce2dd, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1136 ms

Resolved: Minecraft performance has improved in 98b4bd8.


Hyperlink to support period update

Hyperlink to support period update

@xmudrii

/assign @xmudrii

/approve

/label tide/merge-method-squash
This PR covers part of #32930
(from https://github.com/kubernetes/website/pull/32945#issuecomment-1100644227)

/lgtm


"Cuveșdia, Labașinț, Varnița" are villages in "Șiștarovăț " common, "ARAD" county

Precondition
Empty the search bar after every step

Steps 1
Acces to the link https://peviitor-ro.github.io/ui.orase/
Expected result
The page opens


Add underscores to long literals in test code

Updating to rust nightly 2017-08-31 made clippy start
failing builds on long literals. (e.g., 72400 should be 72_400).
This adds underscores to long literals in test code.
See https://github.com/diesel-rs/diesel/issues/1143 on why this wasn't caught in CI.

Awesome, thanks @willmurphyscode!


Can this script work on CDH5

Hi Team,
Thanks for the great work. The readme said These defaults work for CM/CDH 6 and CDP 7. A different set of directories will be used for HDP. I'm wondering to ask does this script work on CDH5. Thanks.

From the current TSB, Cloudera does not believe CM5 or CDH5 to be affected by the log4j2 vulnerability. CM6 and above with CDH5 clusters are affected, and there script can be run. As CM5 uses a different set of directories, the script cannot be run on CM5 with CDH5 as-is.


4732 Party model improvements

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4788


How can we use 'highcharts-exporting' for ReactHighstock?

Hello! I am trying to use exporting for ReactHighstock, but I have no idea how can I do this, nothing works for me. Can you give an example or show how we can use add-ons with ReactHighstock?

I am having the same issue and I have also had no luck, nothing works either.

@1Pauletto Found the way:
import ReactHighstock from 'react-highcharts/ReactHighstock';
HighchartsExporting(ReactHighstock.Highcharts);

On a side note: I think the documentation should more clearly show that ReactHighstock (and I suppose ReactHighMaps as well) expose the Highcharts attribute so you can use the modules/add-ons on them.

@paufabregat Thank you very much!

@paufabregat tell me please where can I get HighchartsExporting?

import HighchartsExporting from 'highcharts-exporting';

@paufabregat Thanks!

Hi, I have the same problem and I have a question about the HighchartsExporting because in the page said that is deprecated and the first option don't working for me

Having the same problem trying to get exporting working. Any pointer is appreciated.

Closing and there has been no activity in this issue for a while.
if this is still relevant, please feel free to reopen


TAP, DNS, Unbound ?

Hello,
Thank you all for your work!
I'm using softether for 1 day, before i was on openvpn only.
Everything works as I want it only I have a DNS leak that I did not have on openvpn because I used unbound with it.
Do you know if some happens to use unbound with softether?
And
Do you know why my TAP does not have an ip address?
Thank you for all.
Debian 9 stretch at oneprovider

I'm not sure whats the problem with DNS, but about you not having an ip address on TAP is right.
I highly suggest reading this issue: https://github.com/SoftEtherVPN/SoftEtherVPN/issues/822
If you want DHCP with tap you should apply the patch of davidebeatrici yourself.
Patch: https://github.com/SoftEtherVPN/SoftEtherVPN/pull/827/files


Merging and improving the documentation with other material.

Thanks for reminding me. I believe the comments are addressed -- added more details in the other review.


Missing documentation for weight file and multi-reporter-config.json

I would love to see some documentation with an example of using a weight file correctely.
Furthermore the multi-reporter-config.json file which pops up after execution is not documented at all.

While running test cases, I know few of my test cases are heavy and few are light ones, if I could get a functionality to run all heavy ones in parallel, instead of random selection, that could help in reducing a lot of time. And if weight file could help in this, then it would be really great.

Digging trough the code, weights seems to go from the highest number to the lowest. Since the file is generated after each test, my understanding is that you need to always run the tests with the same number of threads to get any real benefits from it.
It is really nice for repeated tests as it makes them really faster. I have a couple of heavier tests in my suite, and once the file is generated, the run time is cut in half. It will depend on your test suite and how each test behave for you.
What I have not figured out for now is the imapct of the --weightsJson argument on the CLI. If it simply read the file and won't replace it, that would be how we can set a default weight file for our tests.


Mawile's ability stuck after battle

Following a gym challenge my mawile's ability stayed as huge power despite changing back to normal mawile from mega mawile, this should have reverted back to intimidate.

Has this been fixed/addressed? Can we reproduce the bug? We need more information, perhaps the admins can test this one too @WisconsinWinters

The ability on the mon was fixed by cpt but i'm not sure how to reliably reproduce the bug

i tried this earlier and wasn't able to reproduce in gyms

Could not be reproduced, closing.


Aspira Health - API Sender Onboarding

What You Need to Know

[x] Terms of Service agreement signed

[x] Kickoff meeting completed

[x] Company’s intake form: ReportStream API – format HL7

Q: What is your expected message volume?
A: We currently report anywhere between 60-200 daily results
Q: Are you currently reporting to any other states outside of your state?
A: No just DE
Q: What is your current process for reporting COVID-19 data?
A: Manual entry through Redcap, located here: https://redcap.dhss.delaware.gov/surveys/?s=CH478NH4A7
Q: What state do you report to?
A: DE
To Do

[x] create individual tickets for this epic

[x] RS team member who originates this ticket: complete the following when creating this Issue.

[x] RS team member who originates this ticket: add any dependencies, as needed

Berni:
Completed intro/kickoff meeting on Wed 9/14/2022 with William Albanese

Berni:
Ok to move to O&O backlog until they're ready to re-engage @BerniXiongA6 can follow up on email chain down the road.

@brick-green When will we know it's time to reengage?

Oh I do not remember! @BerniXiongA6 Any memories here?

When we completed Kickoff call, they said they were told that RS can help with system set-up. @brick-green explained that we help with integration to RS, but not set up. Aspira's EMR is Athena. They wanted to know if Athena currently integrates with RS. The sender contact William Albanese sounded as though he was pretty early on with their company, so it might be a good idea to reach out again after the new year unless we hear from them first. @brandonnava

Christopher from Delaware reached out for an update on the onboarding status. I responded as:
Hi Christopher,
I'm looking in through the notes in our system and see that Aspira's EMR is Athena.  We are waiting on WIlliam to confirm that Athena can integrate with ReportStream and have not heard back.  Please advise if you'd like us to reach back out for an update.
Best regards,
The ReportStream Support Team
Contact us at<EMAIL_ADDRESS>Support hours:  8:00 am through 9:00 pm Eastern time
Monday-Friday, excluding US public holidays

@sliu1000 Didn't you share an update on this earlier this week? Are we still going to onboard them? Thanks

@brandonnava Yes, we've had some back-and-forth conversation with William from Atracare.  Please see below the communications:
Hi Sharon,
I'd like to add Nijanthan to this email chain.  He is a Project Coordinator for Athena and will collect all information and then assign this project to a team that will develop the necessary interface.
He is going to need a physical address for the CDC in order to get the process started, so if you could recommend the best address, that would be great.  Then he can instruct us on the next steps to build the interface.
Thanks,
On Wed, Jan 18, 2023 at 11:55 AM Report Stream (CDC<EMAIL_ADDRESS>wrote:
Hi William,
Can you please provide days and times that work for you to discuss this further? Also, if you could include a few of the Athena folks on the call that would be helpful.
Best regards,
Sharon
The ReportStream Support Team
Contact us at<EMAIL_ADDRESS>Support hours:  8:00 am through 9:00 pm Eastern time
Monday-Friday, excluding US public holidays
From: William Albanese<EMAIL_ADDRESS>Sent: Wednesday, January 4, 2023 1:17 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>White, Sarah (CDC/DDPHSS/OD/HITSSU<EMAIL_ADDRESS>Biskie, Joel (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Green, Brick (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Xiong, Bernadette (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Subject: Re: Aspira Health Kickoff Meeting
Yes!  Your timing is perfect as I met again today with the Athena team and we tried to figure out how to get the syntax into the EMR.  Their initial ask from the IT department yielded no responses.  They are following up.
In the meantime, if you can connect us with anyone that is able to assist with Athena or knows how to integrate the code, it would benefit everyone nation-wide on Athena as they attempt to integrate this interface.
We will be working on this and hope to get it going soon.
Thank you,

Updates:
Reeves, Maurice (CDC/OD/OCS)
Nijanthan AV anijanthan@athenahealth.com;William Albanese walbanese@atracare.com;Report Stream (CDC)
lalbanese@atracare.com;Green, Brick (CDC/DDPHSS/OD/HITSSU) (CTR)
Good evening Nijanthan,
My name is Maurice, and I’m the product owner for the Report Stream project. I apologize in advance, but I am just following up to confirm: are you saying that you aren’t able to send HL7 messages? Reading through the documentation for your software I saw a specification listed for the ORU_R01 message type, among others, here: https://docs.athenahealth.com/interfaces/guides/hl7v2-documents
HL7v2 Documents | Interfaces - Athenahealth
Health Level Seven ® (HL7) is a standard format used to exchange data within and across organizations. For more information on this standard, visit the HL7 website. athenahealth supports the following HL7v2 message types: Demographics - ADT. Specification and Sample Messages. Appointments - SIU. Specification and Sample Messages
docs.athenahealth.com
Is it possible for us to schedule a quick call over Zoom so we could discuss what we’re trying to accomplish, and see if we can figure out if it is possible?
Thank you very much,
Maurice
From: Nijanthan AV<EMAIL_ADDRESS>Date: Monday, January 23, 2023 at 15:57
To: William Albanese<EMAIL_ADDRESS>Report Stream (CDC<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Green, Brick (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Reeves, Maurice (CDC/OD/OCS<EMAIL_ADDRESS>Subject: RE: Aspira Health Kickoff Meeting
Hello William,
I had internal discussion with my manager on this he confirmed we unable to procced via HL7 like external care facility. We have only option is via report builder tool also we have existing Global orders report name "COVID-19 Orders Report" which is in our report builder tool library. Please use the report builder tool to activate the existing report to your practice. Which is the option you would be using to establish the report using report builder tool. Thanks.!
Regards,
Nijanthan AV
Senior Analyst
(he/him/his
A picture containing logo
Description automatically generated
Global Infocity • Chennai • India – 600 096
athenahealth.com

@sliu1000 can we reach out to Aspira to confirm if they are willing to move forward?

@TaneaY Hi Tanea, can we please assign a team member to follow up on this? Thank you Sharon

Update:
To: William Albanese<EMAIL_ADDRESS>Cc: Reeves, Maurice (CDC/DDPHSS/OD) qva8@cdc.gov;Green, Brick (CDC/DDPHSS/OD/HITSSU) (CTR)
Hi William,
Just following up on next steps. Were you able to connect with Athena Team? If so, are you still interested in signing up for ReportStream? If not, please reply to this email and we will close your ticket.
Please let us know if you have any further questions/concerns.
Best Regards,
Olivia
The ReportStream Support Team
Contact us at<EMAIL_ADDRESS>Support hours:  8:00 am through 9:00 pm Eastern time
Monday-Friday, excluding US public holidays


add profile links

Add icons & links to social profiles. Will be displayed between site name and slogan.

Try latest code and the demo https://examplesite.lvv.me/post/markdown-syntax/

It is better. Thank you.


Castling notation fix

Fixes #94

Current coverage is 61.88%

Merging #95 into master will decrease coverage by -0.33% as of 58e6390

@@            master     #95   diff @@
======================================
  Files           50      50
  Stmts         1003    1010     +7
  Branches       171     175     +4
  Methods          0       0
======================================
+ Hit            624     625     +1
  Partial         27      27
- Missed         352     358     +6

Review entire Coverage Diff as of 58e6390

Powered by Codecov. Updated on successful CI builds.


Adding support of JSON validation scheme #25

@kahboom this changes the API. Merge if it doesn't conflict your code.

Codecov Report

Merging #26 (95a3f29) into main (cb1605d) will decrease coverage by 0.23%.
The diff coverage is 70.00%.

@@             Coverage Diff              @@
##               main      #26      +/-   ##
============================================
- Coverage     47.44%   47.21%   -0.24%
============================================
  Files            27       27
  Lines           843      843
  Branches        119      119
============================================
- Hits            400      398       -2
- Misses          410      412       +2
  Partials         33       33

Impacted Files
Coverage Δ

...data/parser/step/kamelet/KameletFileProcessor.java
72.82% <70.00%> (-2.18%)
:arrow_down:

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 cb1605d...95a3f29. Read the comment docs.

Sonarcloud complains because the test percentage coverage, but that really depends on the kamelet catalog we are testing...


Added "pack-applications-%" to allow packaging applications directly from the /Applications directory.

Don't we already have this?  https://github.com/unixorn/luggage/blob/master/luggage.make#L714-717

There are 2 different stanzas, and I can add comments to them if it helps, but one is for .app[s] in the actual /Applications directory, and the other is for .app[s] that are in the same directory as the Makefile.

I follow you.  So, I'd probably name it pack-local-application-%: or something OTHER than just applicationS, plural, since it DOES pull from a different location.  That would make it stand out more for folk trying to figure out the difference (and comments always help! :).  What do you think?

I agree that naming it something different would help, but i'm stuck on what to name it. All the other stanza's in luggage assume that the files are in the same working directory as the Makefile. So I would like to keep the current stanza for packaging a .app in the same working directory "pack-application-%." This is where I am stuck, as there is currently no other stanza that pulls files from a directory other than the same working directory. I am concerned that "pack-local-application-%" might be confused between being local to the system, or local to the Makefile.
Would something like "pack-appdir-%" or "pack-applicationsdir-%" or "pack-l_Applications-%" seem more appropriate?

Possibly "pack-from-applications-%" or 'from-applications-dir'?

I like "pack-from-applications-%". I'll edit it the next time I get to a computer and I'll add some comments.

I like pack-from-applications or even pack-from-system-applications

Are there any further recommended changes that should be made before this pull request is completed?

lgtm


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:29: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
ERROR: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:29: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:35: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:35: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description
ERROR: /tmp/abs_390eaiul0a/clone/recipe/meta.yaml:35: missing_dev_url: The recipe is missing a dev_url
Errors were found


flutter docter check crash

Steps to Reproduce
╰ flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 11.6 20G165 darwin-x64, locale
zh-Hans-CN)
• Flutter version 2.8.1 at /Users/liuyanfeng/Library/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 77d935af4d (34 hours ago), 2021-12-16 08:37:33 -0800
• Engine revision 890a5fca2e
• Dart version 2.15.1
• Pub download mirror https://pub.flutter-io.cn
• Flutter download mirror https://storage.flutter-io.cn
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/liuyanfeng/Library/Android/sdk
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for
more details.
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.10.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[!] Android Studio (not installed)
• Android Studio not found; download from
https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/macos#android-setup
for detailed instructions).
[☠] IntelliJ IDEA Ultimate Edition (the doctor check crashed)
✗ Due to an error, the doctor check did not complete. If the error message
below is not helpful, please let us know about this issue at
https://github.com/flutter/flutter/issues.
✗ FormatException: Unexpected extension byte (at offset 5)
• #0      _Utf8Decoder.convertSingle
(dart:convert-patch/convert_patch.dart:1789:7)
#1      Utf8Decoder.convert (dart:convert/utf.dart:318:42)
#2      InputStream.readString
(package:archive/src/util/input_stream.dart:207:30)
#3      new ZipDirectory.read
(package:archive/src/zip/zip_directory.dart:40:30)
#4      ZipDecoder.decodeBuffer
(package:archive/src/zip_decoder.dart:19:30)
#5      ZipDecoder.decodeBytes
(package:archive/src/zip_decoder.dart:14:12)
#6      IntelliJPlugins._findPluginXml
(package:flutter_tools/src/intellij/intellij.dart:130:44)
#7      IntelliJPlugins._readPackageVersion
(package:flutter_tools/src/intellij/intellij.dart:141:40)
#8      IntelliJPlugins.validatePackage
(package:flutter_tools/src/intellij/intellij.dart:63:35)
#9      IntelliJValidator.validate
(package:flutter_tools/src/intellij/intellij_validator.dart:103:15)
#10     asyncGuard.
(package:flutter_tools/src/base/async_guard.dart:111:32)
#11     asyncGuard.
(package:flutter_tools/src/base/async_guard.dart:109:18)
#12     _rootRun (dart:async/zone.dart:1428:13)
#13     _CustomZone.run (dart:async/zone.dart:1328:19)
#14     _runZoned (dart:async/zone.dart:1863:10)
#15     runZonedGuarded (dart:async/zone.dart:1851:12)
#16     runZoned (dart:async/zone.dart:1782:12)
#17     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
#18     Doctor.startValidatorTasks
(package:flutter_tools/src/doctor.dart:197:9)
#19     Doctor.diagnose (package:flutter_tools/src/doctor.dart:301:47)
#20     DoctorCommand.runCommand
(package:flutter_tools/src/commands/doctor.dart:53:47)
#21     FlutterCommand.verifyThenRunCommand
(package:flutter_tools/src/runner/flutter_command.dart:1290:12)

#22     FlutterCommand.run.
(package:flutter_tools/src/runner/flutter_command.dart:1140:27)

#23     AppContext.run.
(package:flutter_tools/src/base/context.dart:150:19)

#24     CommandRunner.runCommand (package:args/command_runner.dart:209:13)

#25     FlutterCommandRunner.runCommand.
(package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)

#26     AppContext.run.
(package:flutter_tools/src/base/context.dart:150:19)

#27     FlutterCommandRunner.runCommand
(package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)

#28     run..
(package:flutter_tools/runner.dart:62:9)

#29     AppContext.run.
(package:flutter_tools/src/base/context.dart:150:19)

#30     main (package:flutter_tools/executable.dart:94:3)

[✓] Connected device (1 available)
• Chrome (web) • chrome • web-javascript • Google Chrome 96.0.4664.110
! Error: Ha0 is not connected. Xcode will continue when Ha0 is connected.
(code -13)

Execute flutter run on the code sample
...
...

Expected results:
Actual results:

Code sample

Logs

Duplicate of #94060


problem with styling

hi phill
thank's guy for your cool editor
i have a problem
i used your editor in my vue project but i have a same problem with all of your templates
styles not loading!!!
as a sample i decide to use bubble menu
but i saw a simple box with three button
how can i load your editor styles in my project
i installed it with npm command
is there any dependencies that i must install it first?

Hey @Saeid-Barzegar, tiptap is renderless and comes without any styling. So it's a feature – not a bug ;)
But feel free to take the styling from the examples page.

thanks man.
but i think its better release it with style and every body can customize it
or write this feature in your github first page
its so useful and beautiful  but in first look seems  a bug...
cause if installed it from npm  repository i didn't see your repository files
thank you so much  and
Have a good year...

I second this. I really love the look of the editor as it is on the examples page, so much so that I ended up borrowing all your sass for one of my projects. Thank you!


#174143979 update user profile info

What does this PR do?
Updating user profile info
Description of Task to be completed?

Getting the user from DB
Getting the updated info of the user
Submitting the updated user

How should this be manually tested?

Clone the repo git clone https://github.com/YvesIraguha/lost-and-found-api.git
Switch the working directory cd lost-and-found-api
Checkout the branch git checkout ft-update-profile-info-#174143979
Install all dependencies npm install
Start the server npm run dev
Using postman Create an account on http://localhost:3000/api/v1/users/signUp to get token
Create an auth-token  with your token as the value
Using postman on http://localhost:3000/api/v1/users/profile as an endpoint, send a put request containing the data you want to update

You should see a successful message and the updated user object

Any background context you want to provide?

If you had a signed up user, You can only log in or copy the given token
Data to be sent for user signup are:

{
"firstName":"your first_name"
"secondName":"your second_name"
"username":"your username"
"phoneNumber":"your phone"
"email":"your email"
"password":"your password"
}

If you want to use google for creating an account paste http://localhost:3000/api/v1/users/auth/google in your browser the user the provided token,
Data to be updated only are

firstName

secondName

username

phoneNumber

What are the relevant pivotal tracker stories?
#174143979
Questions

Is it right to allow a user to update the email they used to create an account?
Is there any other field that can be put there to allow user to have a consistent profile?

I do think that user can also have to change the password. That would be nicer if you add this field too coz s/he might wish to change password s/he used to signup.


Info for 0.x packages

Getting a weird bug in my dependency-lister tool where it can't find licenses for certain 0.x modules. For example three-effectcomposer.
Looking at the link directly definitely provides a "license" field:
https://skimdb.npmjs.com/registry/three-effectcomposer
Whereas the stats.module('three-effectcomposer').info(callback) returns this instead, with no "license" field available:
[ { _id: 'three-effectcomposer',
    _rev: '3-3c1efe1436eb561c81fd17b3321fed32',
    name: 'three-effectcomposer',
    description: '@alteredq\'s EffectComposer plugin for three.js ported for use with Browserify',
    'dist-tags': { latest: '0.0.1' },
    versions: { '0.0.1': [Object] },
    readme: '# three-effectcomposer #\n\nBrowserify-friendly version of `THREE.EffectComposer`, which offers a quick\nGLSL post-processing implementation.\n\nFull credit goes to [@alteredq](http://github.com/alteredq) for writing this,\nthe original source can be found\n[here](http://mrdoob.github.com/three.js/examples/webgl_postprocessing.html).\n\n## Installation ##\n\n``` bash\nnpm install three-effectcomposer\n```\n\n## Running the Demo ##\n\nInstall the dependencies and build the script:\n\n``` bash\ngit clone git@github.com:hughsk/three-effectcomposer.git\ncd three-effectcomposer\nnpm install -d\nnpm run demo\n```\n\nThen just open up `index.html` to see the results.\n\n## Usage ##\n\nThis module doesn\'t touch the `THREE` object, instead you access the different\npass classes through `EffectComposer`. For a working example, see\n[`demo.js`](https://github.com/hughsk/three-effectcomposer/blob/master/demo.js).\n\n``` javascript\nvar THREE = require(\'three\')\n  , EffectComposer = require(\'three-effectcomposer\')(THREE)\n  , DotScreenShader = require(\'./shaders/dotscreen\')\n  , RGBShiftShader = require(\'./shaders/rgbshift\')\n\nvar renderer\n  , scene\n  , camera\n  , composer\n\ninit()\nanimate()\n\nfunction init() {\n  renderer = new THREE.WebGLRenderer\n  scene = new THREE.Scene\n  camera = new THREE.PerspectiveCamera(70, window.innerWidth/window.innerHeight, 1, 1000);\n\n  // ...\n  // The rest of your setup code, as per usual\n  // ...\n\n  // Create your composer and first RenderPass\n  composer = new EffectComposer(renderer)\n  composer.addPass(new EffectComposer.RenderPass(scene, camera))\n\n  // Redraw with a shader\n  var effect = new EffectComposer.ShaderPass(DotScreenShader)\n  composer.addPass(effect)\n\n  // And another shader, drawing to the screen at this point\n  var effect = new EffectComposer.ShaderPass(RGBShiftShader)\n  effect.renderToScreen = true\n  composer.addPass(effect)\n};\n\n// Instead of calling renderer.render, use\n// composer.render instead:\nfunction animate() {\n  requestAnimationFrame(animate)\n  composer.render()\n};\n```\n',
    maintainers: [ [Object] ],
    time:
     { modified: '2013-01-28T03:56:40.101Z',
       created: '2013-01-28T03:56:36.529Z',
       '0.0.1': '2013-01-28T03:56:40.101Z' },
    author: { name: 'alteredq', url: 'http://alteredqualia.com/' },
    repository:
     { type: 'git',
       url: 'git://github.com/hughsk/three-effectcomposer.git' } },
  { etag: '"3-3c1efe1436eb561c81fd17b3321fed32"',
    date: 'Wed, 11 Feb 2015 17:34:22 GMT',
    'content-type': 'application/json',
    'cache-control': 'must-revalidate',
    'status-code': 200,
    uri: 'https://skimdb.npmjs.com/registry/three-effectcomposer' } ]

Ideas? Going to poke around a bit more.
EDIT: omg i am a tool, the data is there in info[0].versions
Weird that 0.x packages don't have it as a top-level field.

Hehe, all good :)


Make output generation deterministic. Fixes #414.

This patch changes all uses of HashMap/HashSet in Registry into
BTreeMap/BTreeSet and adds derive(PartialOrd, Ord) to all types contained
in them so that the generated sources will be deterministic.
This is generally just good hygiene, but it also makes things more cacheable with sccache.
I didn't add any new tests (maybe I should have?) but this passes all the existing tests on my machine, and I verified that building gleam twice resulted in the exact same generated files from its build script.

Oops, I realize I only ran the tests in the gl_generator subdirectory, not elsewhere, and some of the gl_tests are failing.

Seems like this is related to https://github.com/brendanzab/gl-rs/pull/411#issuecomment-270483188 ... maybe new warnings have been added since gl_generator was last deployed!

Oh good! I was worried that I had broken something unintentionally. I've successfully built webrender and servo with my gl_generator branch, so I don't think it's actually broken. :)

I opened https://github.com/brendanzab/gl-rs/issues/416 and did a little digging there, but I didn't find a simple fix. In light of this being existing breakage, are you OK with merging this?

Done! Will have to look into that. Thanks again 🤔


chore: update default config values for bn.format

Summary
This PR updates the default config values for the bn.format function as requested in #595.
// constants.ts
DEFAULT_PRECISION = 9 // previously 3
DEFAULT_MIN_PRECISION = 3 // previously 1

Closes #595

Perhaps it's not a breaking change per se, but it could break something that's using it.
I'd suggest a minor bump.


Error during Crystools installation

Describe the bug
Error during Crystools installation
Screenshots

Error in console:
[Crystools INFO] Crystools version: 1.15.1
Traceback (most recent call last):
File "D:\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1931, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 940, in exec_module
File "", line 241, in call_with_frames_removed
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools_init.py", line 24, in
from .server import *
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools\server_init_.py", line 1, in
from .monitor import *
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools\server\monitor.py", line 4, in
from ..general import cmonitor
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools\general_init_.py", line 1, in
from .monitor import *
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools\general\monitor.py", line 5, in
from .hardware import CHardwareInfo
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools\general\hardware.py", line 4, in
from .gpu import CGPUInfo
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools\general\gpu.py", line 6, in
from pyrsmi import rocml
ModuleNotFoundError: No module named 'pyrsmi'
Cannot import D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools module for custom nodes: No module named 'pyrsmi'
Versions:
** Python version: 3.11.8 (tags/v3.11.8:db85d51, Feb  6 2024, 22:03:32) [MSC v.1937 64 bit (AMD64)]

Total VRAM 12282 MB, total RAM 31895 MB
Set vram state to: NORMAL_VRAM

[Crystools INFO] Crystools version: 1.15.1

### Loading: ComfyUI-Manager (V2.48.2)
### ComfyUI Revision: 2425 [c24f8973] | Released on '2024-07-31'

Hi, I had the same problem. You can install pyrmsi by yourself with 'pip install pyrsmi', it seems to work well.

Hi, I deleted and reinstalled, but the error, as I understand it, remains. Maybe a screenshot will help to better understand what's wrong.
Requirement already satisfied: pyrsmi in c:\users\artem\appdata\local\programs\python\python310\lib\site-packages (0.0.6)

Crystools Updated! Please try to updated (should be 1.16.0)


Update README.md

https://giwa.com/
zendesk.com/auth/v2/login/registration?auth_origin=212034%2Ctrue%2Ctruell

https://giwa.com/ zendesk.com/auth/v2/login/registration?auth_origin=212034%2Ctrue%2Ctruell

#6


Document platform_lib usage.

Fix: #607
Signed-off-by: Qi Zhang<EMAIL_ADDRESS>

I will let @steven-bellock review


If the Unity splash screen is disabled for an OpenXR hololens 2 app where spatial awareness is enabled, the spatial mesh is not built.

Describe the bug
If the Unity splash screen is disabled for an OpenXR hololens 2 app where spatial awareness is enabled, the spatial mesh is not built. In the same project, if the splash screen is enabled, the spatial mesh is built and functions properly.
To reproduce
Steps to reproduce the behavior:

Create an MRTK app in Unity for hololens 2 with spatial awareness enabled by following the MRTK project setup instructions  here.
Switch to UWP build platform, then build and run the app on hololens 2.
See that spatial awareness and the spatial mesh are running properly
Disable the unity splash screen in the project settings
Build and run the app again
See that the spatial awareness is not functioning properly and the spatial mesh is not created

To make it easy to reproduce this error, I have created a nearly-empty MRTK project repository on github.
The only modifications to this project as compared to the MRTK tutorial instructions are:

Cloned the default hololens 2 MRTK configuration to enable spatial mesh and set mesh visibility to occlusion
Added an ErrorManager that receives any exceptions thrown and displays them on a camera-anchored canvas

The repository for that project is here.
Expected behavior
Disabling the Unity splash screen should not affect the proper functioning of the MRTK spatial awareness mesh.
Your setup

Unity Version: 2020.3.17 and 2020.3.8
MRTK Version: 2.7.2
MRTK OpenXR: 1.0.1, 1.0.3
com.unity.xr.openxr 1.2.8

Target platform (please complete the following information)

HoloLens 2, ARM or ARM64

For some reason, I wasn't able to repro this 🤔 based on looking at the code though, i think there's a good chance this will be fixed via #10366

Going to close this out, as I'm unable to repo with the meshing fix in 2.7.3


JSON parse error when receiving error messages from test cloud upload endpoint

Errors are in some cases send as text and not JSON. The cli can look at the content type of the reply  to determine the format.
Steps to reproduce:
Upload ipa with architecture that is not compatible with the device selection.
Result:
Error "Unexpected token N in JSON at position 0" occurred in deserializing the responseBody - "No valid devices in selection (fb9a78d3: 1 devices)" for the default response.
Expected result:
Error output with the following content:
No valid devices in selection (fb9a78d3: 1 devices)

Is this issue resolved or not? I'm still getting same error while uploading iOS app to appcenter.
Error: Error "Unexpected token N in JSON at position 0" occurred in deserializin
g the responseBody - "No valid devices in selection (e1439c1a: 1 devices)" for t
he default response.

We talked about this a bit in the test cloud team. The issue is that the swagger file in appcenter does not tell the truth entirely. on HTTP code 200 we return JSON alright, but on != 200 we return plaintext. The swagger declares that it is always JSON.
IOW: the fix is to update the server side swagger, then regenerate the client libs in this project.
(@prashantchoubey16 the issue is not fixed yet - sorry)

I get the same error

This issue is now fixed on the server-side, which should now be returning proper JSON in all error cases. @prashantchoubey16 /cc @glennwester

Will close for now per @jacobe request. Thanks!


Interactor: introduce expose to replace automagic instance variable collection

Hello,
Once again, thanks you all for the great work on Lotus!
At some point on Gitter I mention replace the automagical instance variable conversion to mimic Lotus::Action expose behavior:
class SignIn < Struct.new(:params)
  include Lotus::Interactor

  expose :user

  def call
    @user = User.find params[:id]
  end
end

result = SignIn.new({ id: 15 }).call
result.exposures # => { user: <User:0x007f965c1d0318 @id=15> }
result.user # => <User:0x007f965c1d0318 @id=15>

The only difference will be the direct mapping as methods between exposures (as part of payload).
I've a patch for this locally, but I'm concern about the change in behavior, which is why I'm submitting this question first:
Should I keep current behavior (expose all instance variables) unless expose is used? That way existing code will not break.
What would be the proper approach to handle such scenario?
Looking forward your feedback.
Thank you.

/cc @jodosha

Hi @luislavena
Nice feature but in which scenarios do you use this feature? This expose api only add visibility to know what instance variables you exposed, somethin like public and private instance variables.

That is correct.
We use several instance variables that should remain private to the
service, for example memoize an api call.
We would like to decide what to expose instead of the arbitrary all
instance variables without having to use special naming.
Having a defined exposure also helps with the contract for the service.
Thank you.
Sorry for top posting. Sent from mobile.
On May 15, 2015 11:38 AM, "Alfonso"<EMAIL_ADDRESS>wrote:

Hi @luislavena https://github.com/luislavena
Nice feature but in which scenarios do you use this feature? This expose
api only add visibility to know what instance variables you exposed,
somethin like public and private instance variables.
—
Reply to this email directly or view it on GitHub
https://github.com/lotus/utils/issues/77#issuecomment-102437258.

@jodosha I agree with @luislavena I created several "interactors/services" in rails (without gem) and some times I use memoization and I only expose some instance variables.
What do you think to support this feature?

@luislavena Thanks for brining this discussion on the table. My reply comes a bit later because of the 0.3.1 release of lotusrb. I'm totally :+1: with this, but because it's a breaking change, I'd bump lotus-utils 0.5.0.

@jodosha @luislavena told us this
Should I keep current behavior (expose all instance variables) unless expose is used? That way existing code will not break.

@luislavena can you make a PR with this new feature? thanks ;)

I like this idea too, :+1:

@AlfonsoUceda @joneslee85 great, will send the pull request shortly! (just back from some time off :smile: :sunny:)

@luislavena PING. I would love to include this change in v0.5.0, which will be a dependency for Lotus v0.4.0 (June 23rd). Do you will have time to work on this?

Yeah!
I'm traveling right now but will send my pull request this weekend.
On May 27, 2015 1:54 PM, "Luca Guidi"<EMAIL_ADDRESS>wrote:

@luislavena https://github.com/luislavena PING. I would love to include
this change in v0.5.0, which will be a dependency for Lotus v0.4.0 (June
23rd). Do you will have time to work on this?
—
Reply to this email directly or view it on GitHub
https://github.com/lotus/utils/issues/77#issuecomment-106012824.

@luislavena I took this over, as we have so little time to wrap up this release. Would you mind to review? Thank you! #80

@jodosha sorry, had a few setbacks on my flights and limited internet over the past few days.
Checking your changes now, thank you.

@luislavena No worries, just wanted to avoid to rush for this change one or two days before the release.


Feature - TRSV2-597 - Caseworker ROI approval merge process

Codecov Report
Patch coverage: 36.36% and no project coverage change.

Comparison is base (8445c84) 55.14% compared to head (aee72c8) 55.14%.

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

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #398   +/-   ##
========================================
  Coverage    55.14%   55.14%
========================================
  Files          210      210
  Lines        12308    12320   +12
  Branches      1599     1602    +3
========================================
+ Hits          6787     6794    +7
- Misses        5521     5526    +5

Impacted Files
Coverage Δ

trade_remedies_api/organisations/models.py
55.88% <33.33%> (-0.12%)
:arrow_down:

...de_remedies_api/organisations/services/v2/views.py
59.67% <40.00%> (+0.66%)
:arrow_up:

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Removed Arbitration Profiles from Classic UI

Removed view file & marked product features as hidden for Arbitration Profiles as they are in use by API.
PR https://github.com/ManageIQ/manageiq-ui-classic/pull/426 removes Arbitration Profiles  from classic UI
https://www.pivotaltracker.com/story/show/137999337
original PR https://github.com/ManageIQ/manageiq/pull/13980 got messed up somehow, opened a new PR
@dclarizio please review.
cc @jntullo

@abellotti Please review.


enhance: Add flag to control skip bucket exist check

The bucket existence check need extra permission, this could block online check.
This PR enable to skip bucket existence check.

/lgtm


Add map build height as a world property.

It just seems like having the map build height be a property that can be modified would be pretty nice.

If there is max-build-height in server.properties, why not have it here as well?


--import-db could accept data from /dev/stdin

Is your feature request related to a problem? Please describe.
I have my SQL data stored in several files. I would like to import them to a DB.
Describe the solution you'd like
It would be nice to be able to pipe SQL data in ddev --import-db like
cat db/sql/struct/* db/sql/data/* | ddev --import-db

Describe alternatives you've considered
I use
cat db/sql/struct/* db/sql/data/* | docker exec -i ddev-<PROJECTNAME>-db mysql --default-character-set=utf8 db

as a workaround to pipe SQL data into the "mysql" command in the container.
Notice the docker exec -i which connects the host's stdin to the command.

Thanks!

Actually, #1700 gets us set up for this, but doesn't actually solve this problem, reopening.

The example in ddev import-db -h doesn't have any information about this usage, still needs to be added. Also we should check the docs.


Very slow to finish cataloguing when Keypirinha started on boot

Hardware: i7-7700HQ, 16GB RAM, 256GB m.2 SSD for OS/all applications, ~60% full
Environment: Windows 10 22H2
Version: Keypirinha 2.25, latest via PackageControl for all third-party packages
Third-party packages loaded: ColorPicker, Currency, Cvt, Kill, PackageControl, Shell, Steam, Svc, Symbols, WindowsApps, WinSys
Steps to Reproduce:

Enable Keypirinha start on boot, boot PC
Monitor load time for WindowsApps.WindowsApps via Keypirinha console

Expected Result: Reasonable load times, a minute or two at most, for 32 items
Actual Result: Extremely long and highly variable load times, usually ~200s+, up through the current record of 46295.5s, per the console. Average seems to be ~5-10 minutes assuming start-on-boot.
Notes: Far lower times are achieved when restarting Keypirinha without rebooting, on the order of 10-30s, even with the machine fairly heavily loaded. Other startup programs exist, and the load is relatively moderate - a few minutes is sufficient to reach an idle state. I have no idea if the 46 thousand seconds result can even be accurate - 12 hours seems impossibly long.

That seems very odd to me. I also start Keypirinha on boot and cataloging times are around 5-18 seconds for 50 items (over the last 3 months).
To analyze the problem better, can you configure the following in the WindowsApps config (open with Keypirinha: Configure Package: WindowsApps) and provide your keypirinha log:
[main]
debug = yes

This enables debug output for the WindowsApps package. It gets very spammy in the Keypirinha
console, but it should give an idea where all that time is spent on your system.


HTTPError

Something has changed...
It was working ok but now, the simplest example is failing with HTTPError...

Just trying to run example and get this

I think it depends on your computer.
I logged url that you send request :
'https://translate.google.com/translate_a/single?client=gtx&sl=auto&tl=uk&hl=uk&dt=at&dt=bd&dt=ex&dt=ld&dt=md&dt=qca&dt=rw&dt=rm&dt=ss&dt=t&ie=UTF-8&oe=UTF-8&otf=1&ssel=0&tsel=0&kc=7&q=проверка&TypeError='
'
and got  this
'Our systems have detected unusual traffic from your computer network. Please try to submit your request again later. Why did this happen?'

and after few tries it starts working again.
maybe smth in token generator?

Our systems have detected unusual traffic from your computer network. Please try to submit your request again later. Why did this happen?

That's a rate limit error. That means you're making more requests than you're allowed to.


Resolving response-timeout values set as place holder values in spring profiles

Hi, add this problem in the next release?