OpenCL seems to almost work

@leejet @Green-Sky @ggerganov
I do not know cpp and do not have a solid grasp on how ggml works. , but building the repo with cmake -dggml_clblast=ON seems to work as the GPU utilization goes up and it’s very fast (10s vs 80s per step on a higher end CPU). It does complete all the steps and completes sampling too, but then crashes at line 1505 of ggml-opencl.
If it is a matter of spending time to make this work, is it simple enough for one of you to explain what needs to be done? If so, would be happy to give it a shot but don’t know where to start.
My limited understanding is that sampling is what takes all the effort, so is there a way to maybe switch from GPU to CPU to save the file? Or am I missing some context/knowledge?
Edit: Fixed typo. Flag used is clblast, not openblas.

Try this patch: https://github.com/ggerganov/llama.cpp/commit/6460f758dbd472653296044d36bed8c4554988f5

@ggerganov That worked, thank you!
Is it proper protocol to submit a pull request for a one-liner?

FYI: It does work, but GPU utilization is very low. Got any more simple speedups in your pocket? @ggerganov

I'm sorry to disappoint you but openblas doesn't use the gpu to accelerate the processing but it uses the cpu itself. If anything you should try DGGML-CLBLAST=ON but it still wouldn't work as the developer still hasn't integrated any gpu acceleration into the program.

@DaniAndTheWeb Thanks for pointing that out…it was a typo and the CLBLAST flag is what I was referring to.
How difficult/time-sensitive of a task is it going to be to incorporate OpenCL? With that flag, the gpu does get some kind of signal because utilization increases.
Just wondering if it’s a very involved process, or if we just need to copy/paste something from llama and/or ggml?

I'm no expert in opencl but it will require some time, it's not just a copy/paste. The good news is that with the current ram usage the gpu acceleration will probably be one of the more memory efficient.

@DaniAndTheWeb Can you tell me broadly speaking what tasks need to be completed, like I’m a 5?
Maybe CodeLlama can help me contribute a pull request to get it done, but I need a thread to grab onto.
(Not sure if tagging is necessary, new to Github)

As I told you I don't know a lot about how the OpenCL implementation works but you probably have to implement each computing kernel of the stock cpu code in opencl. You can take a look at llama.cpp's implementation but you will need to make lots of tweaks to the code to make it work with this project.

Try this patch: https://github.com/ggerganov/llama.cpp/commit/6460f758dbd472653296044d36bed8c4554988f5

I can confirm that really work!

I applied the patch and then added some ifdef SD_USE_CLBLAST include "ggml-opencl.h" ... etc, edited cmakelist file with bits from llama.cpp's clblast ported over and renamed/re-pointed, then configured with cmake .. -DGGML_OPENBLAS=ON -DGGML_CLBLAST=ON. Now compiled ./sd recognizes my AMD RX 580 GPU and I get about a 30% speed up. Not a huge increase, but my GPU is pretty old too. And it does take some load off CPU which is nice. Thanks!


Replace SubstreamCancelStrategy with SupervisionDecider for Split

This PR is the equivalent of https://github.com/akka/akka/pull/31415 but for Pekko, I would recommend reading the conversation thread from that PR to get the context/justification for the feature.
The very brief summary is that the SubstreamCancelStrategy abstraction that is only used when creating a SubFlow (via usage of splitWhen and other associated functions) is not needed since Pekko already provides a generic streams abstraction for this called a SupervisionDecider. It can even be said that the SubstreamCancelStrategy is worse than pointless duplication because unlike SupervisionDecider you can't do things like log the thrown exception in the SubFlow (this is actually the main impetus for the change, because as user I had the situation where SubFlow was throwing exceptions and I had no idea what was going on because it wasn't logging them via a custom SupervisionDecider which I defined, even worse because its not possible to change the already defined SubstreamCancelStrategy you have to do annoying workarounds to log any potentially thrown exceptions in a SubFlow).
Note that this change although not source/binary breaking it does change behaviour because the default SupervisionDecider is to stop in case of an exception where as the default SubstreamCancelStrategy default is to ignore and keep on going (this is also the reason behind the test changes in the PR). As stated in the original PR, even though it is a breaking behavioural change, since it fails more explicitly its much easier for a user to notice (see https://github.com/akka/akka/pull/31415#pullrequestreview-983031284).
Due to this behaviour change the PR was intended to be looked at by @raboof and @jrudolph to get buy in/opinion whether this change is acceptable (see https://github.com/akka/akka/pull/31415#issuecomment-1141293129), would be great to get feedback on this PR's premise (because otherwise we have to create an entirely new set of functions which overall I think is worse proposition).
PR is currently draft because I didn't update the docs for the behavioural change. If @jrudolph / @raboof are happy with this change (along with anyone else that has strong experience with akka/pekko-streams, i.e. @He-Pin ) then I will update the PR with proper documentation explaining all the changes. It will also target Pekko 1.1.x.

I will read this PR in more detail ,and add some feedback later.

The original issue is : https://github.com/akka/akka/issues/31394

Pekko-HTTP is using the splitAfter and splitWhen IIRC, how about running a snapshot against it too?

Pekko-HTTP is using the splitAfter and splitWhen IIRC, how about running a snapshot against it too?

Sure I can do this, I would expect that pekko-http 1.1.x would depend on pekko 1.1.x so if there are any breaking changes in pekko-http it would just be for 1
1.1.x.
I can check locally what impact this change would have against pekko-http

Pekko-HTTP is using the splitAfter and splitWhen IIRC, how about running a snapshot against it too?

See https://github.com/apache/incubator-pekko-http/pull/371 which is related

So I have decided to finish off this PR (which just involved updating the documentation) since it makes sense to release this as part of Pekko 1.1.0. Of note is that I had to add a new migration-guide-1.0.x-1.1.x.md which follows the same style Akka used (I copied it from https://github.com/mdedetrich/akka-apache/blob/main/akka-docs/src/main/paradox/project/migration-guides.md and https://github.com/mdedetrich/akka-apache/blob/main/akka-docs/src/main/paradox/project/migration-guide-2.5.x-2.6.x.md which is the latest Apache 2.0 release of Akka).
Regarding the behavioural changes, as you can see from https://github.com/mdedetrich/akka-apache/blob/main/akka-docs/src/main/paradox/project/migration-guide-2.5.x-2.6.x.md?plain=1#L17-L18, doing behavioural changes happen as long as there is a justification for it and I believe this PR does hit that threshold.

Pinging @He-Pin , @pjfanning , @jrudolph and @raboof since I forgot to do so earlier.

@pjfanning All apostrophe's have been removed

I will take a review today

@mdedetrich I have left some comments

@mdedetrich I have left some comments

I can't see any, I think you forgot to submit?

@He-Pin Responded to comments

I think this one is ok, I missed it's was internalApi.

Thanks
I will leave the PR open till the end of the month (given that its christmas) to see if anyone else wants to comment on it, at that point I will merge it.

The commit I just pushed was adding @deprecation in some places that I missed beforehand, otherwise PR is the same.

Im going to go ahead and merge this, largely so that migration-guide-1.0.x-1.1.x.md file which this PR introduces can be used for any future possible behaviourally incompatible changes but also so I can test the changes sooner rather than later to iron out any potential problems.

Just wanted to mention that since the changes in this PR were published as a snapshot, pekko-http tests ran against these changes and passed without any problems (see https://github.com/apache/incubator-pekko-http/actions/runs/7353051075/job/20018534864)


chore(build): Dev. Container Setup

Task Description
A clear and concise description of what the task entails.
Acceptance Criteria:

[ ] AC 1
[ ] AC 2

Expected Test:
Add any information about how this will be tested.

@copyonwrite, could you kindly fill in the Acceptance criteria and Expected Test sections of the issue?


CI: Enable sodium on Windows

Ref: #12296
Install libsodium to enable sodium feature and do related tests in test_crypt.
9.0.1486, Test_uncrypt_xchacha20v2() and Test_uncrypt_xchacha20v2_custom() in test_crypt fails on x86 environment.

Strange ASAN test errors, so I started a rerun.


security/acme-client: acme.sh runs arbitrary commands from a remote server

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
acme.sh runs arbitrary commands from a remote server related to this issue: https://github.com/acmesh-official/acme.sh/issues/4659
Version 3.0.6 is fixing the security issue.

https://twitter.com/opnsense/status/1667277622370344960

This is not a plugin issue.


Links to version-specific pages in headers of master docs at heptio.github.io/velero/master are broken

What steps did you take and what happened:
The latest version in https://heptio.github.io/velero/master/versions.html points to v0.10.0versions.html which is not found.
What did you expect to happen:
When I click on the link, it takes me to the latest version.

More generally - the master version of our docs site at https://heptio.github.com/velero/master has a warning at the top of each page:
"This is the documentation for the latest development version of Velero. Both code and docs may be unstable, and these docs are not guaranteed to be up to date or correct. See the latest version."
Where "latest version" is a link to the latest version-specific instance of that page.  These links are broken on all pages except the landing page -- looks like there's a missing / in the URL.  E.g. https://heptio.github.io/velero/v0.10.0versions.html instead of https://heptio.github.io/velero/v0.10.0/versions.html.

I'll test this.

Closed by #1239


[FEATURE REQ] List of servers to ping (maybe controlled by the user)

Implemented in 1.5 release.
Servers or websites can be added to the list "listServer" which currently has 3 servers. The list and all functions work with any amount of Servers. Servers can't be added or deleted by the user. This might change in the future.


why different  keyboard-shortcuts for Linux、mac、Windows？

VSCode Version:
OS Version:

Steps to Reproduce:

Reproduces without extensions: Yes/No

Because the OS itself uses different shortcut conventions. Plus, Mac keys don't usually perform the same functions as their Windows counterparts.


Blogger alternative

Please include alternatives to blogger

A possible alternative could be using Hugo which will produce a static website which you can host on a free provider like GitHub Pages, Netlify or Vercel. You have 100% control over the software, since Hugo is OSS.


CloudBlockBlob.StartCopyAsync fails with NotFound

CloudBlockBlob.StartCopyAsync fails with NotFound when using azurite, works properly using azure storage account.
CloudBlobContainer targetContainer = blobClient.GetContainerReference("target-container");
await targetContainer.CreateIfNotExistsAsync();

CloudBlobContainer sourceContainer = blobClient.GetContainerReference("source-container");
var sourceBlobs = await sourceContainer.ListBlobsAsync();

foreach (var sourceBlob in sourceBlobs)
{
    var targetBlob = targetContainer.GetBlockBlobReference(blob.Name);

    // fails with "NotFound" when using azurite, works properly with azure blob storage
    await targetBlob.StartCopyAsync(blob);
}

I was able to reproduce the issue using with WindowsAzure.Storage version 9.0.0 with NETStandard.Library version 2.0.1.

Small .NET Core project to reproduce the issue: https://github.com/thisisthekap/Azurite157Repro

Thanks for reporting this @thisisthekap ! I will look into this within the next few days and try to release a fix asap.

@thisisthekap : fix is released as part of v1.9.3. Please reopen in case it does not work for you. Thanks!

@arafato Thank you for your effort, I just tried your fix. Even tough StartCopyAsync proceeds without error, an DownloadTextAsync call made right afterwards blocks forever. Also, it seems that the blob storage gets entirely unusable after calling StartCopyAsync (until restart). I updated the test project hosted at https://github.com/thisisthekap/Azurite157Repro to reproduce the new issue.
// proceeds without error now
await targetBlob.StartCopyAsync(sourceBlob);

// This call blocks for azurite, the blob storage seems to get entirely unusable after using StartCopyAsync.
var str = await targetBlob.DownloadTextAsync();
Console.WriteLine("Copied: " + str);

@thisisthekap : this is strange, I cannot repro with your code. Code finishes (and outputs) as expected. Can you paste in the log output of Azurite? Also, what happens if you do a simple
$ curl localhost:10000/devstoreaccount1/target-container/the-blob ?

@arafato I have got no clue why, but now the issue is gone (the only thing which I have done in the meantime was to upgrade node, but I do not know the version it was before).

@arafato When using the docker container version 1.9.3, the NotFound issues arises again.

@thisisthekap strange again, I cannot repro. Running the code against arafato/azurite:1.9.3 works fine on my side. Are you sure that you are using v1.9.3?
Did you start the container with
$ docker run -d -t -p 10000:10000 -p 10001:10001 -v azurite-testdrive:/opt/azurite/folder arafato/azurite:1.9.3 ?

@arafato I am using it with docker compose. This is my compose file:
version: "2.1"

services:
    local-storage:
        image: arafato/azurite:1.9.3
        ports:
            - 10010:10000
        volumes:
            - ./Data/:/opt/azurite/folder
        environment:
            - executable=blob

Ok, just to make sure: can you repro the bug when you start Azurite like so
$ docker run -d -t -p 10000:10000 -p 10001:10001 -v azurite-testdrive:/opt/azurite/folder arafato/azurite:1.9.3 and run your program against it?

When using docker directly, everything works fine. When using docker-compose, I experience the issue described above. My suggestion: docker-compose creates a software defined network (SDN) by default. So I think that the address given to your validation logic is neither <IP_ADDRESS> nor localhost.

@thisisthekap ahhh, good point! In this case I will need to extend the validation rule to also include os.hostname() What do you think?

@arafato Yeah, so either you have to support all possible names of the host, or ignore the hostname entirely (which also would be not that good).

Maybe we could solve it by using an environment variable to specify the whitelisted hostnames. The appropriate environment variables could then be specified in the docker-compose yaml file.

@thisisthekap I think this might also be related to the port that is being exposed. You are exposing Azurite at 10010. The regex I'm using, however, checks for the port number 10000. At the end it depends how the SDK populates the x-ms-copy-source header... from my experience different storage SDKs (PHP, Java, .NET, Node, ...) often behave slightly different. So I will simply release a fix that ignores the hostname and the port being exposed. Do you see an issue with that?

@arafato As azurite is not a production system and therefore security is not that big issue, I am fine with just ignoring host and port.

@thisisthekap just released fix as part of v2.0.0 (initial table storage support). Let me know if it works for you!

@arafato Everything is working just fine now. Thank you once again for your effort!


small comment fix

small comment fix for the Main::Setup function

Hey there! While grammar is important for code comments, we prefer when PRs offer a bit more substance than a small typo fix, especially if it's not user-facing. I recommend taking a note of this typo and looking for more mistakes like this in the future, then opening a PR with a bunch of them.
Better yet, if you are interested in getting started with contributing to Godot, consider fixing some of the reported bugs! I'm going to close this PR with a hope that you'll find something to contribute in the future.

Understood. Thank you!Sent from my iPhoneOn Sep 20, 2023, at 12:15 PM, Yuri Sizov @.***> wrote:
Hey there! While grammar is important for code comments, we prefer when PRs offer a bit more substance than a small typo fix, especially if it's not user-facing. I recommend taking a note of this typo and looking for more mistakes like this in the future, then opening a PR with a bunch of them.
Better yet, if you are interested in getting started with contributing to Godot, consider fixing some of the reported bugs! I'm going to close this PR with a hope that you'll find something to contribute in the future.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

also doesn't look really like a grammar fault


Add JavaScript/WASM bindings

This is a very WIP PR to add WASM support. The goal is to have an isomorphic API that will work in the browser or in standalone executables. This will be implemented by using build tags and the syscall/js package.

Pull Request Test Coverage Report for Build 2327

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.2%) to 80.237%

Files with Coverage Reduction
New Missed Lines
%

internal/ice/candidate.go
2
92.22%

internal/ice/candidatepair.go
2
89.55%

internal/mux/mux.go
4
92.5%

lossy_stream.go
4
77.36%

Totals

Change from base Build 2322:
-0.2%

Covered Lines:
4068

Relevant Lines:
5070

💛  - Coveralls

I actually did come up with another advantage of using a common interface: It makes sure that both implementations share the same public API. I still believe we shouldn't expose such an interface to the public but it may be worth investigation options to make sure the public API is the same for both the native and JS/WASM implementations.

@backkem I was thinking the same thing. It's a good idea to make an unexported interface to enforce that the JavaScript/WASM bindings follow the same API.

There's still a lot of work to do, but I've reached the point where there is a runnable example in examples/data-channels-wasm. The example simply ports the JavaScript code from examples/data-channels to Go. There are potentially more interesting examples we could build down the road, e.g., having the same main.go file run in both the browser and the server.

The example is really cool already!
Things I noticed:

It would indeed be amazing to be able to build the exact same example for both targets. That definitely seems feasible.
We provide a web server to host the examples locally as documented here. We should restructure the example so it can be served like this as well.
I would like to avoid duplicating the accompanying go code example. The same example is already available. Maybe we can find a good way to merge the example with the corresponding original example. (There may be a relation to the previous point. E.g.: make the examples server detect a wasmfiddle folder to serve wasm examples.)

Let me know if you would like any help with that.

Cool I'm happy to get the WASM example more in line with the other examples. Will reach out to you on Slack for some of the details.
I'd like to focus on the minimum set of changes to get this PR ready to merge.

Currently, the PR supports DataChannels only. That means methods like OnTrack and NewTrack are not implemented. I would like to keep this PR small and address these features in a future PR. I'm also not sure if this is something I can implement myself since we only need DataChannels for the project I'm working on (but I would be happy to help point someone in the right direction).
There are not any tests for the WASM bindings (unless you count the example). Testing them will require some significant changes and new testing infrastructure which is not in place. I have some ideas for how to do this and it is something I want to help build, but I would prefer to do it in a separate PR in order to make changes easier to review.
As we discussed on Slack, it is not possible to implement certain properties of DataChannel and PeerConnection in the WASM bindings (e.g., PeerConnection.CurrentLocalDescription and PeerConnection.PendingLocalDescription). So unlike the Go version, these have been converted to methods in the WASM bindings. I think we are aligned on a plan to update the Go version to use methods too. I chose not to do that in this PR in order to make it easier to review. Additionally, my preference would be for someone else who is more familiar with the internals of the Go implementation to convert the properties to methods.
I would also like to delay handling Certificates and Credentials as they are a lot trickier to implement. Cryptography in JavaScript land is completely different from how it works in Go. It's not as simple as converting basic data structures.
I added two methods to PeerConnection in the WASM bindings that don't exist in the Go implementation: OnICECandidate and OnNegotiationNeeded. Both of which are part of the WebRTC spec and I would like to add these to the Go implementation too at some point.

With that said, here's what I have left to do:

[ ] Update WASM example to be more in line with existing examples.
[ ] Add better error handling in some places, especially recovering from exceptions in JavaScript land.

@backkem how does all this sound?

I agree with everything you said. I think we're fine with landing this in increments.

@backkem I'm not sure how to address this issue in Travis:
syscall/js
go build syscall/js: build constraints exclude all Go files in /home/travis/.gimme/versions/go1.12.linux.amd64/src/syscall/js

Any ideas?

@albrow I figured out the CI problem. At some point the CI (and go modules as well) seems to run go get ./... to download the dependencies. Because the examples/data-channels-wasm/browser package doesn't have the build flag it also tries to download and build syscall/js as well. The fix is pretty simple: ensure that all the uses of syscall/js are behind the js build flag. In this case that means adding the build flag at the top of  examples/data-channels-wasm/browser/main.go

FYI, The CI pointed out the following after fixing the dependency issue:

Please add yourself to the contributor section in the root readme.md.
We don't allow the use of 'panic'. Please change that to just returning an error or something similar.


Github wiki links to other markdown files opening in raw view

Take this site:
https://github.com/orientechnologies/orientdb/wiki/Pivoting-With-Query
Click on the first link there ("SELECT"):
https://github.com/orientechnologies/orientdb/wiki/SQL-Query.md
^^^
This is where you get.
Solution: Remove all ".md"-endings from links and we get to the correct view of markdown files on github!!!

The Home Wiki Page says: "This wiki is the main source of documentation for developers working with (or contributing to) the OrientDB Open Source project. If this is your first time hearing about OrientDB, we recommend starting with the Orient Technologies website."
We use GitHub only as documentation source, but everything is published on:
http://www.orientechnologies.com/docs/last/index.html

.md is because GitBook needs this suffix to work.


dockview-vue error Cannot read properties of null (reading 'appContext')

Describe the bug
I'am using dockview-vue.
In many instances i get this error
Cannot read properties of null (reading 'appContext')
To Reproduce
A simple way to reproduce this is just go to the documentation.
In many of the live examples for vue on the dockview site don't work like the react live examples. For instance adding a floating group/empty group
Steps to reproduce the behavior:

https://dockview.dev/docs/core/watermark
Click on 'vue' examples
Scroll down to 'live examples'
Press add empty group

Steps to reproduce the behavior:

https://dockview.dev/docs/core/groups/floatingGroups
Click on 'vue' examples
Scroll down to 'live examples'
Press add floating group

The error occurs when getCurrentInstance() is called and returns null, it is used as parent is this function in dockview-vue.
function findComponent(parent, name) {
  var _a, _b;
  let instance = parent;
  let component = null;
  while (!component && instance) {
    component = (_a = instance.components) == null ? void 0 : _a[name];
    instance = instance.parent;
  }
  if (!component) {
   // Error coming from here, parent is null and throws
    component = (_b = parent.appContext.components) == null ? void 0 : _b[name];
  }
  if (!component) {
    throw new Error(`Failed to find Vue Component '${name}'`);
  }
  return component;
}

Fixed in 1.14.2


Add QuickQuery

A simple SQL database query tool, built using Wails (Go & Vue.js)
https://github.com/mmghv/QuickQuery

Thanks. Please consider adding it to the discord channel for wider exposure: https://discord.gg/buhZcEDF

Thanks you, will do.


Warning

Hi,
Thanks for this great library. Right now I'm adding it to our react native app and sometimes I see this warning:
Warning: Can only update a mounted or mounting component. This usually means you called setState, replaceState or forceUpdate on an unmounted component. This is a no-op.

on line 151 of imageCacheHoc.js. At least that's what it says :). Am I doing something wrong or is there a way to fix that? Thanks

Since this.fileSystem.getLocalFilePathFromUrl() is an async function and setState() doesn't get called until after this.fileSystem.getLocalFilePathFromUrl() resolves, that warning will throw if the CacheableImage component is mounted and then quickly unmounted before the remote image can be downloaded.
This is a warning, so the issue will not show up in production, and as such there's no need for you to consider yourself "blocked" by this. That said, there's got to be a way to check if a component is currently mounted before calling setState(). I'll investigate this and let you know.
Thanks for bringing this to my attention.

This appears to be the solution.
https://reactjs.org/blog/2015/12/16/ismounted-antipattern.html

Oh, great!

Patching it right now ;-)
Thanks for offering though!

By the way, today I was trying to put an ActivityIndicator as placeholder but with an enclosing View of the same width and height than the Image I'm trying to load. Is that possible somehow? I tried during some time to access to the style property with no luck, I also tried to pass a placeHolderStyle property and assign it to the placeholder component but that also didn't work.

Okay this is resolved.

ActivityIndicator as placeholder but with an enclosing View of the same width and height than the Image I'm trying to load. Is that possible somehow?

I can't think of a way offhand, but is there a way to use the children prop?
Something like
const defaultPlaceholderObject = {
  component: View,
  props: {
    style: styles.viewStyleThatMatchesImageWidthAndHeight,
    children: // is there a way to pass activity indicator component into here.
  }
};

Great, I'll try that. And thanks a lot for the quick fix!!

Finally I got it by doing a wrapper ti imageCacheHoc and passing the same style to the Image and the placeholder!
Thanks again

Great!
Mind pasting example code of your solution so I can help if someone else asks how to accomplish this?

Of course, sorry. This is what I did:
const CacheableImagePlaceholder = ({ style }) => (
  <View style={[style, { alignItems: 'center', justifyContent: 'center' }]}>
    <ActivityIndicator color={colors.grey} />
  </View>
);

CacheableImagePlaceholder.propTypes = {
  ...View.propTypes,
};

const ImageCacheHoc = imageCacheHoc(Image, {
  defaultPlaceholder: {
    component: CacheableImagePlaceholder,
    props: {},
  },
});

ImageCacheHoc.propTypes = {
  ...Image.propTypes,
};

const CacheableImage = ({ source, resizeMode, style }) => (
  <ImageCacheHoc
    source={source}
    style={style}
    resizeMode={resizeMode}
    placeholder={{
      component: CacheableImagePlaceholder,
      props: {
        style,
      },
    }}
  />
);

CacheableImage.propTypes = {
  ...Image.propTypes,
};

So this way you only have to change your Image by this component and it works with the placeholder without needing anything.
But I have another problem, when testing the release version of the app it crashes when starting with this error:
com.facebook.react.common.JavascriptException: undefined is not an object (evaluating 's.View.propTypes.style')
It comes from this line:
    static propTypes = {
      fileHostWhitelist: PropTypes.array,
      source: PropTypes.object.isRequired,
      permanent: PropTypes.bool,
      // style: View.propTypes.style,  <---- THIS LINE
      placeholder: PropTypes.shape({
        component: PropTypes.func,
        props: PropTypes.object
      })
    };

But I haven't found the reason for the crash, but If I comment it it works. I'll investigate some more, but if you could help me, that'd be great.

Ah excellent solution and I'll keep it in mind for anyone else who asks.
As for that other issue, looks like View.propTypes has been deprecated in favor of ViewPropTypes (guess FB hasn't updated react docs appropriately since I pulled View.propTypes from their docs).
https://github.com/facebook/react-native/commit/53905a537a58d028fc5eb1c4e9a8ec967d8cd396
If that resolves it and you make a PR I can merge it in on my phone otherwise it will have to wait until next wednesday for me patch.
If you make a PR make sure to run the tests "npm test" and lint "npm run lint" so build doesn't get broke when I merge.
Happy Thanksgiving!

Ok, I'll take a look and make a PR tomorrow. Thanks

There you are: https://github.com/billmalarky/react-native-image-cache-hoc/pull/4

Excellent! Merged and thanks a ton!


form-data missing in package.json

Against the latest develop, when trying to run via npm run start, I get:
ERROR in ./node_modules/origin/dist/index.js
Module not found: Error: Can't resolve 'form-data' in '/repos/OriginProtocol/demo-dapp/node_modules/origin/dist'
 @ ./node_modules/origin/dist/index.js 894:17-37
 @ ./src/services/origin.js

(PR forthcoming)

https://github.com/OriginProtocol/demo-dapp/pull/133

@inkredabull Is this just a matter of the current develop branch not playing well with the currently-published npm package? I think that lib is required by origin-js only and maybe should not be installed in this app.

@inkredabull it does look like the package might be missing from origin-js's 'package.json. I guess it's possible that we all already have it in our package-locks, so we're not seeing the error?

These instructions work for me, with no errors.
#Terminal A
mkdir -p origin
cd origin
git clone https://github.com/OriginProtocol/origin-js.git
cd origin-js
git checkout develop
npm install
npm link
npm start

#Terminal B (start in the origin directory you just made)
git clone https://github.com/OriginProtocol/demo-dapp.git
cd demo-dapp
git checkout develop
npm install
npm link origin
npm run start

#Then with a metamask enabled browser, go to http://localhost:3000

It's possible thought that my computer is special case, and this working only here.

I had the same symptoms yesterday.  I can confirm that the work-around @DanielVF described also works for me.

@inkredabull, in case this helps you with the PR:
My original installation of demo-dapp does point to the correct version of origin-js:

Revision in develop I have checked out:

git show --oneline
66f139e (HEAD -> develop, origin/develop) Merge pull request #131 from OriginProtocol/upgrade-modules

Has the dependency on origin-js declared like this:

grep origin-js package.json
    "origin": "OriginProtocol/origin-js#develop",

The installed version reports being from SHA 44390dd:

grep _resolved node_modules/origin/package.json
  "_resolved": "github:OriginProtocol/origin-js#44390dda628d0670bd52d6f0d30f4cfc818aaad4",

The version of origin-js I have checked out from the earlier test matches 44390dd:

cd ../missing-form-data/origin-js/
git show --oneline
44390dd (HEAD -> develop, origin/develop) Merge pull request #138 from OriginProtocol/20180430/use_our_own_web3

Something in the way npm installs from github is different from what happens with the npm link.

Thanks @micahalcorn @DanielVF @dobbs. I noticed that after pulling develop for this repo and running npm install that package.json and package-lock.json are updated; I would expect them to not need to be.  I'll run the suggestion from @DanielVF to see if that clears things up.

Suggestion from @DanielVF works for build/running; no getting form-data issue. Will close this Issue/PR.
@DanielVF should the dev instructions be updated with your suggestion as the canonical way?

Just noticed that package.json and package-lock.json are modified after I run the commands in Terminal B

Verifying that after I run npm link origin in Terminal B I get updates in the package files:
➜  demo-dapp git:(develop) ✗ git status
...
	modified:   package-lock.json
	modified:   package.json

Upgraded npm. FWIW:
➜  bluxomelabs > cd ~/repos/origin
➜  origin node -v
v8.7.0
➜  origin npm -v
6.0.0

I guess that (mods in pkg files) is expected behavior; just makes it less intuitive to develop around.

Seeing LISTING_FETCH_IDS_SUCCESS in the console; realizing that empty listings have to do with Redux parsing. Investigating.


missing spring.thymeleaf.cache=false

Hi,
I've just try gs-serving-web-content-complete in SpringIDE and was disappointed when modifying the "greeting.html" template and do not see change in the browser (template was not reloaded).
After searching into the web I found that adding the file scr/main/resources/application.properties
with the line spring.thymeleaf.cache=false resolve the case.
I think you could add this tip to make people happy to discover that Java web app can reload changes without restarting web server (an old bad story).
Cheers.

That's a good point, although my temptation is to actually add spring-boot-devtools. It has the effect of flipping that flag as desired, but also switching some other features people may wish to use. And I think it would better align with the message we want to share with devs.

I can meet many devs that hate Web Java techs for those little missing tips ...

perhaps completing the page "http://blog.netgloo.com/2014/05/21/hot-swapping-in-spring-boot-with-eclipse-sts/" with your experience on "spring-boot-devtools" should be a way to an easy Web Java life ;-)

Your solution is explained in the Spring documentation on chapter 78 (after a long way...)
http://docs.spring.io/spring-boot/docs/current-SNAPSHOT/reference/htmlsingle/#howto-hotswapping


GUVNOR-2563 : Guided Decision Table V&V: Reports check status in Analysis Reporting screen

https://issues.jboss.org/browse/GUVNOR-2563
https://dl.dropboxusercontent.com/u/2111740/guvnor/status.png
For long running validation shows the current status of validation.
Yellow when running. Green when finished.
Not a progress bar. Tried those and they updating them did not work as expected.
Also cleaned up some redundant code from V&V tests.

A couple of comments... let me know your thoughts.

Good comments. I'll fix the lookupBean in the next PR and update this one with the progress bar and PatternFly alerts today.

New look: https://dl.dropboxusercontent.com/u/2111740/guvnor/analysis.png
Fixed all but the IOC.getBeanManager().lookupBean issue. Fixed that locally for the next PR, just need to finish it.

@Rikkola Please let me know when you want this PR reviewed again (nice new look BTW)

@manstis ok. This one can be reviewed, please. The issues found here are fixed in the next PR.

+1 to merge.


[Feature Request] Button to enable Avatar Scaling on all own words.

Instead of manually going on the website enabling it for each world separately, a 1 click solution.
Explain in detail what your suggested feature would be used for.
A button that changes the new world setting Avatar Scaling from the default off to on.
Describe how it would look if it requires a UI.
In User panel (you)/worlds next to the sorts, add a settings button that opens a new window for mass/batch editing. In that menu I would place the button.
While you are at it maybe at a batch toggle for making worlds private & so.
Explain why people would want to use it.
So they don't have to do it for every world 1 by 1.

seems a bit out of scope for vrcx, an a very niche feature. maybe at best adding an individual option to change world scaling settings as its just adding a tag to your world feature_avatar_scaling

But what about people with lots of worlds? Let's say they have a 100, changing it individually for every world is a lot of effort.
I just suspect that VRC is gonna add more & more features that are off by default, so the VRCX batch features is preparing for the future.

Being able to private all world & avatars with a few clicks seems not very niche to me. These are options that YT, Twitter, & alike have.

tag additions already exist and soon adding of the avatar scaling specifically will be added, so I don't see batch features being added anytime soon, that's more of a feature vrchat should be adding themselves to the sdk and world tools.

Indeed, there are many features VRC should add.


TabNine try to connect to different IPs everytime

please complete the following information:

OS version: Mac Mojave
Editor: vscode
Editor version:
Programming language: js

Issue Details:
I am curious why every time TabNine tries to connect to different IPs? and which data will be sent to those IPs?

That is weird. What IPs are you seeing Tabnine sending data to?
We use domain names for everything happening, so maybe you are seeing the ip of those?

That is weird. What IPs are you seeing Tabnine sending data to?
We use domain names for everything happening, so maybe you are seeing the ip of those?


How embed Vue2?

How can I embed Vue 2 into a component as a module?

You may now install the library with NPM:
npm i fluidscroll

Then import it into your component for use:
import { fluidScroll } from 'fluidscroll';


Improve language in README.md

Edited some of the README.md to fix typos or reformulate the sentences.
It's not perfect yet, since I wasn't always completely sure what was meant by the original author.

Needs rebase after recent changes

Thanks!


Solr and ES always needs the WGS84 bounding box

Currently the bounding box conversion to mercator needed by the search engine is totally wrong:
https://github.com/cga-harvard/HHypermap/blob/master/hypermap/aggregator/utils.py#L537
We should correctly project the bounding box to WGS84 if the layer is exposed with a different SRS.

working on this.

I think this must still be fixed.
@dkakkar @zhangmingda when you start being more familiar with the system can you look into this?


Document Python driver Twisted backend

The RethinkDB 2.1 Python driver will come with support for Twisted in addition to Tornado.
https://github.com/rethinkdb/rethinkdb/pull/4096
Hence http://rethinkdb.com/api/python/set_loop_type/ and http://rethinkdb.com/docs/async-connections/ need to be expanded.
I don't actually know how the Twisted syntax works. @Tryneus would you mind writing up some code examples similar to the ones in http://rethinkdb.com/docs/async-connections/#python-and-tornado ?

I'm going to need a quick writeup similar to the Tornado stuff to be able to get going on these…

Pinging @Tryneus on this again.

I used the existing Tornado examples as a starting point.  I skipped the sections on running tasks in the background and canceling background tasks because these are just generic Tornado/Twisted usage, and we probably shouldn't be telling users how to do this.  In addition, the Tornado section seems to end quite abruptly, so I added a bit more after the final example.
Basic Usage
The set_loop_type command should be used before connect to set the RethinkDB Python driver to use asynchronous event loops.  This should be set to 'twisted' to use a Connection compatible with the Twisted reactor.
import rethinkdb as r
from twisted.internet import reactor, defer
from twisted.internet.defer import inlineCallbacks, returnValue

r.set_loop_type('twisted')
connection = r.connect(host='localhost', port=28015)

After executing set_loop_type, r.connect will return a Tornado Deferred, as will r.run.
Example: Simple use
@inlineCallbacks
def single_row(conn_deferred):
    # Wait for the connection to be ready
    conn = yield conn_deferred
    # Insert some data
    yield r.table('test').insert([{"id": 0}, {"id": 1}, {"id": 2}]).run(conn)
    # Print the first row in the table
    row = yield r.table('test').get(0).run(conn)
    print(row)

# Output
{u'id': 0}

Example: Using a cursor
@inlineCallbacks
def use_cursor(conn):
    # Insert some data
    yield r.table('test').insert([{"id": 0}, {"id": 1}, {"id": 2}]).run(conn)
    # Print every row in the table.
    cursor = yield r.table('test').order_by(index="id").run(conn)
    while (yield cursor.fetch_next()):
        item = yield cursor.next()
        print(item)

# Output:
{u'id': 0}
{u'id': 1}
{u'id': 2}

Note that looping over a cursor must be done with while and fetch_next, rather than using a for x in cursor loop.
Error handling
If an error occurs during an asynchronous operation, the yield statement will throw an exception as normal. This may happen immediately (for example, you might reference a table that doesn’t exist), but your application might receive large amounts of data before the error (for example, your network might be disrupted after the connection is established).
One error in particular is notable. If you have a task that consume a changefeed indefinitely, and the connection closes, the task will experience a ReqlRuntimeError.
Example: Re-thrown errors
@inlineCallbacks
def bad_table(conn):
    yield r.table('non_existent').run(conn)

Unhandled error in Deferred:
Traceback (most recent call last):
Failure: rethinkdb.errors.ReqlOpFailedError: Table `test.non_existent` does not exist in:
r.table('non_existent')
^^^^^^^^^^^^^^^^^^^^^^^

Example: Catching runtime errors
@inlineCallbacks
def catch_bad_table(conn):
    try:
        yield r.table('non_existent').run(conn)
    except r.ReqlRuntimeError:
        print("Saw error")

# Output
Saw error

Subscribing to changefeeds
The asynchronous database API allows you to handle multiple changefeeds simultaneously by running multiple background tasks.  As an example, consider this changefeed handler:
@inlineCallbacks
def print_feed(conn, table):
    feed = yield r.table(table).changes().run(conn)
    while (yield feed.fetch_next()):
        item = yield feed.next()
        print("Seen on table %s: %s" % (table, str(item)))

We can schedule it on the Twisted reactor with this code:
reactor.callLater(0, print_cfeed_data, conn, table)

Now the task will run in the background, printing out changes.  When we alter the table, the changes will be noticed.
Now consider a larger example:
@inlineCallbacks
def print_feed(conn, table, ready, cancel):
    def errback_feed(feed, err):
        feed.close()
        return err

    feed = yield r.table(table).changes().run(conn)
    cancel.addErrback(lambda err: errback_feed(feed, err))
    ready.callback(None)
    while (yield feed.fetch_next()):
        item = yield feed.next()
        print("Seen on table %s: %s" % (table, str(item)))

@inlineCallbacks
def table_write(conn, table):
    for i in range(10):
        yield r.table(table).insert({'id': i}).run(conn)

@inlineCallbacks
def notice_changes(conn, *tables):
    # Reset the state of the tables on the server
    if len(tables) > 0:
        table_list = yield r.table_list().run(conn)
        yield defer.DeferredList([r.table_drop(t).run(conn) for t in tables if t in table_list])
    yield defer.DeferredList([r.table_create(t).run(conn) for t in tables])

    readies = [defer.Deferred() for t in tables]
    cancel = defer.Deferred()
    feeds = [print_feed(conn, table, ready, cancel) for table, ready in zip(tables, readies)]

    # Wait for the feeds to become ready
    yield defer.gatherResults(readies)
    yield defer.gatherResults([table_write(conn, table) for table in tables])

    # Cancel the feeds and wait for them to exit
    cancel.addErrback(lambda err: None)
    cancel.cancel()
    yield defer.DeferredList(feeds)

yield notice_changes(conn, 'a', 'b')

# Output
Seen on table b: {u'old_val': None, u'new_val': {u'id': 0}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 0}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 1}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 1}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 2}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 2}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 3}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 3}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 4}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 4}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 5}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 5}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 6}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 6}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 7}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 7}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 8}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 8}}
Seen on table a: {u'old_val': None, u'new_val': {u'id': 9}}
Seen on table b: {u'old_val': None, u'new_val': {u'id': 9}}

Here, we can listen for changes on multiple tables at once.  We simultaneously write into the tables, and observe our writes appear in the changefeeds.  We then cancel the changefeeds after we've written 10 items into each of the tables.

Do you think it'd be a good idea to remove the "running a coroutine in the background" examples from the Tornado documentation, and leave the "larger example" of changefeeds as sufficient?

@chipotle - yes, while reading the Tornado docs to prepare this, I was very confused as to why we were going over that.  If someone wants to learn how to run Twisted/Tornado tasks in the background and cancel them correctly, there are plenty of generic tutorials out there.  That section doesn't even mention ReQL.


43 add security compatibility

Security compatibility using JWT.
Temporary token generator/validator is JwtTokenUtil.
Security classes (by package):
com.revature.config: WebSecurityConfig
com.revature.repositories: UserRepository
com.revature.model.security: Authority, AuthorityName, User
com.revature.security: JwtAuthenticationEntryPoint, JwtAuthenticationFilter, JwtAuthenticationRequest, JwtAuthenticationSuccessHandler, JwtTokenUtil (TEMP), JwtUser, JwtUserFactory
com.revature.security.controllers: AuthenticationRestController, UserRestController
com.revature.security.exceptions: JwtTokenMissingException
com.revature.security.service: JwtAuthenticationResponse, JwtUserDetailsServiceImpl

SonarQube analysis reported 31 issues

 5 blocker
 1 critical
 22 major
 3 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 19 issues

 5 blocker
 11 major
 3 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 19 issues

 16 major
 3 minor

Watch the comments in this conversation to review them.

@tjkemper please review this.

This is a lot in one PR.
Initial reactions

The Swagger endpoint /swagger-ui.html should load without a JWT in the request
Each request must have a JWT signed with some secret: dog

The API doesn't handle passwords or generate tokens

Claims have a secret: mySecret (see application.yml and JwtTokenUtil)
This secret is validated through the JwtAuthenticationFilter.
The password/token generation is handled by JwtTokenUtil, a test class placed in the structure to test the rest of it. It can be replaced later with an external generator.

SonarQube analysis reported 27 issues

 24 major
 3 minor

Watch the comments in this conversation to review them.

Build fails for non-existent dependency (JwtAuthenticationFilter [8,37])
The dependency is a local second project. The dependency is in the POM and the directory is in the classpath. Any ideas as to why it can't find this dependency would be appreciated.
POM:

com.revature
RevatureSecurityLogin
1.0-SNAPSHOT
compile

Classpath:

SonarQube analysis reported 20 issues

 16 major
 4 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 18 issues

 15 major
 3 minor

Watch the comments in this conversation to review them.


Design a passive mode switch that would only listen and record sounds for training on them

Have an easy way to toggle a mode on the master raspberry that would not distribute the sound to the players. That way we can have a "detect and do nothing" mode.

Use the only_monitor_no_detect var when deploying the solution with Ansible.


Fix the build on Ubuntu 24.04

The individual commits have more information, but essentially this PR is required to allow this package to build on Ubuntu 24.04 (what ROS 2 Rolling is targeting right now).  Let me know if you need more information about these changes.

Thank you for the PR!

Thank you for the PR!

You are welcome.  When you get a chance, it would be great to do a release of this package into Rolling to fix the build: https://build.ros2.org/view/Rbin_uN64/job/Rbin_uN64__fields2cover__ubuntu_noble_amd64__binary/


Could we have DrawBoxShadows api, or DrawGeometry with BoxShadows parameter?

It would be very handy to have DrawGeometry with BoxShadows parameter in the same way as DrawRectangle? Or something like DrawBoxShadows (with clipping lile DrawRectangle).
The idea is to be able to draw such geometry without relying on ICutstomDrawOperation like I did:

Thanks a lot!

In Avalonia (and CSS for that matter) box shadows do make sense only with rectangles.
And we already have an API for that:
https://github.com/AvaloniaUI/Avalonia/blob/43f1b9d8ca9389fea51a5463b003af325b171349/src/Avalonia.Base/Media/ImmediateDrawingContext.cs#L119
This DrawRectangle accepts box shadows argument, and can be used to achieve the same result.


Edge

Merging code... no changes other than documentation.

Had to correct lack of documentation in pypi.


Issue with k8s.io/docs/setup/production-environment/tools/kubeadm/install-kubeadm/

What's the new yum repo of kubeadm and related binary packages? It seems no kubeadm packages in https://packages.cloud.google.com/yum/repos.

/language en
/kind support

Hi @chee-ip , can you check again now, it has been updated now.

/triage need-information

/triage needs-information

/kind bug
/remove-kind support

I don't think this needs to stay open
/close


Many Dialogic errors such as "Parse Error: Could not resolve class" and "Invalid call. Nonexistent function" on editor start, Dialogic panel Character selection and edit

The problem
Describe the bug
I get many errors each time I open the Godot editor:

res://addons/dialogic/Editor/Events/Fields/field_vector2.gd:0 - Parse Error: Could not resolve class "DialogicVisualEditorFieldVector".
modules/gdscript/gdscript.cpp:2788 - Failed to load script "res://addons/dialogic/Editor/Events/Fields/field_vector2.gd" with error "Parse error". (User)
res://addons/dialogic/Editor/CharacterEditor/char_edit_section_portraits.gd:16 - Invalid call. Nonexistent function '_load_display_info' in base 'HBoxContainer (field_vector2.gd)'.
...
res://addons/dialogic/Editor/CharacterEditor/char_edit_section_portraits.gd:51 - Invalid call. Nonexistent function 'set_value' in base 'HBoxContainer (field_vector2.gd)'.
res://addons/dialogic/Core/DialogicUtil.gd:405 - Invalid set index 'property_name' (on base: 'HBoxContainer (field_vector2.gd)') with value of type 'String'.
res://addons/dialogic/Modules/StyleEditor/style_layer_editor.gd:449 - Invalid set index 'size_flags_horizontal' (on base: 'Nil') with value of type 'int'.

I suspect that Godot is responsible for this as it doesn't parse things in order (I saw some issues on Godot repo although not sure which specific one it would be). If so there is not much choice to fix it from plugin side, unless you know a workaround.
Note that the game runs fine at runtime. It's just a one-time issue that doesn't prevent development.
To Reproduce
Steps to reproduce the behavior:

Install Dialogic plugin and enable it
Restart project
Check errors in Output

Expected behavior
Start without errors
Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

OS: Linux Ubuntu 22.04
Godot Version: v4.2.1.stable.official [b09f793f5]
Dialogic Version: 2.0-Alpha-13 (Godot 4.2+) (installed manually from GitHub)

Solutions
Workaround
No workaround known
Possible fixes
It may be on Godot side... but it would be nice to find out what specifically triggers this, as not all plugins cause such errors

I got a more consistent repro during project edit, not on start, with similar but a slightly different set of errors:
Open Dialogic panel and select character:

res://addons/dialogic/Editor/CharacterEditor/char_edit_section_portraits.gd:51 - Invalid call. Nonexistent function 'set_value' in base 'HBoxContainer (field_vector2.gd)'.

Edit Display Name and try to save:

res://addons/dialogic/Editor/CharacterEditor/char_edit_section_portraits.gd:66 - Invalid get index 'current_value' (on base: 'HBoxContainer (field_vector2.gd)').
res://addons/dialogic/Modules/Text/character_settings/character_moods_settings.gd:36 - Invalid get index 'custom_info' (on base: 'Nil').
res://addons/dialogic/Modules/Style/character_settings_style.gd:23 - Invalid get index 'custom_info' (on base: 'Nil').
res://addons/dialogic/Editor/CharacterEditor/character_editor.gd:114 - Invalid get index 'resource_path' (on base: 'Nil').

It is possible that one of the errors prevent immediate character saving, therefore causing https://github.com/dialogic-godot/dialogic/issues/2227

Very weird. When adding a new character portrait (the entry, not the picture), I got a similar issue:

res://addons/dialogic/Editor/CharacterEditor/char_edit_p_section_layout.gd:16 - Invalid call. Nonexistent function 'set_value' in base 'HBoxContainer (field_vector2.gd)'.

and noticed that field_vector2.gd defines a _set_value with initial underscore indeed, not set_value. So I fixed the call in char_edit_p_section_layout.gd from:
%PortraitOffset._set_value(data.get('offset', Vector2()))

to:
%PortraitOffset.set_value(data.get('offset', Vector2()))

but then I got:

res://addons/dialogic/Editor/CharacterEditor/char_edit_p_section_layout.gd:16 - Invalid call. Nonexistent function '_set_value' in base 'HBoxContainer (field_vector2.gd)'.

which I don't understand at all.

Before, I also had an error in res://addons/dialogic/Editor/CharacterEditor/character_editor_portrait_tree.gd when creating a new Portrait entry for a character:

res://addons/dialogic/Editor/CharacterEditor/character_editor_portrait_tree.gd:32 - Invalid get index 'default_portrait' (on base: 'Nil').
res://addons/dialogic/Editor/CharacterEditor/character_editor.gd:345 - Cannot call method 'set_meta' on a null value.

But this one was easy to avoid, just checking for null:
func add_portrait_item(portrait_name: String, portrait_data: Dictionary, parent_item: TreeItem, previous_name := "") -> TreeItem:
	var item: TreeItem = %PortraitTree.create_item(parent_item)
	item.set_text(0, portrait_name)
	item.set_metadata(0, portrait_data)
	if previous_name.is_empty():
		item.set_meta('previous_name', get_full_item_name(item))
	else:
		item.set_meta('previous_name', previous_name)
	# LOCAL FIX by hsandt: check for null current_resource
	# This avoids errors:
	# res://addons/dialogic/Editor/CharacterEditor/character_editor_portrait_tree.gd:32 - Invalid get index 'default_portrait' (on base: 'Nil').
  	# res://addons/dialogic/Editor/CharacterEditor/character_editor.gd:345 - Cannot call method 'set_meta' on a null value.
	if editor.current_resource:
		if portrait_name == editor.current_resource.default_portrait:
			item.add_button(0, get_theme_icon('Favorites', 'EditorIcons'), 2, true, 'Default')
	return item

And another error I don't understand at all, when refreshing the portrait image field:
			print("current_portrait_data: ", current_portrait_data)
			var mirror: bool = current_portrait_data.get('mirror', false) != current_resource.mirror

res://addons/dialogic/Editor/CharacterEditor/character_editor.gd:577 - Invalid get index 'mirror' (on base: 'Nil').
current_portrait_data: { "scene": "", "export_overrides": { "image": """" }, "scale": 1, "offset": (0, 0), "mirror": false }

I debug printed the current_portrait_data (note that print appears after error due to how streams are handled), and I do see mirror is there. And clearly the variable is not null.
Despite all these @tool errors I managed to make portraits work with the Premade Style layer "Textbox with Portrait", so I suppose I can go on with development. The biggest issue was character asset not saving properly, I had to re-add the Portrait once or twice to finally save it. But it seems related to another issue https://github.com/dialogic-godot/dialogic/issues/2227
For now I'll just ignore further tool errors and focus on actual runtime.


Create an intro

Add an intro about yourself: who are you, what is your strength, why are you here.

[x] I am from Merida Venezuela

Done!


No Option to Add Pair in BTC or ETH

Currently only USDT Pairs are allowed. There should be option to add pair in BTC or ETH. Even if we make it as base could not add pair.

do you find a way that add BTC or ETH pair?

option is added, try to create BTC and ETH pairs. After set is_base option for BTC or ETH.


Remove TChannel support from Jaeger Receiver

See for context and path forward if required
https://github.com/open-telemetry/opentelemetry-service/pull/264#discussion_r314504172

We might want to keep the receivers for a while and remove the legacy exporters. Most Jaeger Clients still can't send data via gRPC to the collector.

Thanks for the info @jpkrohling - let's keep this for later then.

I think it's time to remove this already. We'll be removing TChannel soon from the collector, and we shouldn't have it here by then.

Let's create one on the contrib repo first so there will be a fallback for anyone that needs it (we did this for zipkin scribe).


Improve pw providers on FreeBSD

I've improved the pw user and group providers on FreeBSD. Here's a summary of the changes:
The user provider now supports managing passwords and user expiry. A bug which caused a changed home directory not to be created when managehome is enabled has been fixed.
The group provider now supports managing group members. I've also re-added the allowdupe feature which works on FreeBSD 7, 8 and 9.
I've also added tests for both the user and group providers.
All of the changes to the group provider are mine and so are all the tests. The changes to the user provider have been partially taken from elsewhere, but the other changes are mine - see the commits for details.

It'd be great if the BSD provider had tests - https://github.com/puppetlabs/puppet/tree/master/spec/unit/provider/user - would you be interested in taking a stab at doing a basic set as well based on the useradd ones?
I'd also probably squash a couple of these commits down to make it a bit cleaner.

Hi James. Yes, I'll have a stab at a set of tests. I'm new to Ruby, but I've got enough experience in other languages to make it an interesting challenge :-)
Yes, I should squash the last two at least. I was unsure about squashing after I'd already pushed it and done the pull request. Is it fine to do that?

That'd be great. Yeah fine to squash and re-push to branch since it's not merged etc.

Here's my first stab at providing tests for both the user and group providers. I've also done a few other tweaks along the way, and even uncovered a bug when writing the tests!
I wouldn't say this is merge ready yet. It needs more testing at my end and undoubtedly feedback from some more knowledgable puppet people. What I've done seems to work, but whether I've done it correctly or not I'm not sure.

I'd be happy to squash commits 2, 3 and 4 if retaining the origins of changes isn't important.

It may be a good idea to open a Redmine ticket for any bugs you found and then squash and note the ticket number of the fix in the commit message, just so that it's noted and other people can pitch in with verification and testing.

I've done some further testing an ironed out a few issues. I'm now happy with the code and I'll hold off pushing any other changes until somebody has had a chance to review it.
Jason - thanks, good point. I submitted #10962 for the bug I found.

I'm getting four failures when I run the new tests.
Squashing commits is fine as long as they go together, but for the original pw changes that someone else wrote could you keep those separate and give credit?  (I think the original is https://github.com/nutbunnies/puppet/commit/5f1efd644da64971635728f4e51ab0f304abdfc4)  If you squash commits then you might want to add in some more ticket numbers (#11318, #11046) for the relevant commits. If not that's fine and I can add them in later.
If you can fix these tests I'll try to get this in. Let me know if you need any help with it.

The issue is that these tests depend on being run on freebsd, which isn't what I'm on. I'll have a patch for you for this later today.

That's at https://github.com/pcarlisle/puppet/commit/a29d9af6de6486801d9faed49ef0c44089d81855
I wasn't clear from your earlier comments whether you are finished or plan to add more to this. If you're done I'll squash commits and try to get it merged, if not I'll wait. Let me know.

Yes, that is the original source.. I am the author of the blog entry (I am brd@FreeBSD.org).

Sorry to leave this for so long. I've cleaned it up, made sure the tests work at each commit, and submitted a new pull request at https://github.com/puppetlabs/puppet/pull/338. I'm closing this as that is being merged now.


Any plans to support ON DUPLICATE KEY UPDATE?

For instance:
    INSERT INTO user_credits (user_uuid, credits)
    VALUES (?, 1)
    ON DUPLICATE KEY UPDATE credits = credits + ?

or is there a workaround so I can still use this project?

+1

+1

Should this issue be closed now?


docs: translations react-dom/client: Client APIs: hydrateRoot

Closes #455
Description
Done, waiting for review, thanks!
Translation for react-dom/client: Client APIs: hydrateRoot, reference by #330

I'll review again later.

@resir014 can approve so this can be merged soon


Import X2 attendance

Import attendance data from X2 for student profile. Related to #38 #25.

Thank you @marimuraki for the data QA on this import! :+1:

Do we have an indicator for whether the absence is excused vs unexcused? Per Nicole's feedback, excused absences may not be counted towards risk as (much as) unexcused absences.

We have an ATT_EXCUSED_IND column. Out of 1,772,100 attendance events, 78,135 (4.4%) were excused.
Also of interest: ATT_CODE_VIEW, ATT_DISMISSED_IND, ATT_REASON_CODE, ATT_COMMENT

Attendance import is functional and happening on a nightly basis. Still need to handle some irregularities and questions about the attendance data model, issue documented here: #96


Favicon helper

Implement favicon helper.
See Lotus::Assets::Helpers for reference.
Usage
Basic Usage
<%= favicon %>

<link href="/assets/favicon.ico" rel="shortcut icon" type="image/x-icon">

Path
It also accepts an optional path
<%= favicon 'myfavicon.ico' %>

<link href="/assets/myfavicon.ico" rel="shortcut icon" type="image/x-icon">

URL Prefix
This should take account of the prefix defined in Lotus::Assets::Configuration.
That means, if an application has a prefix: /admin, when one this helper is used, the prefix should be used.
<%= favicon %>

<link href="/assets/admin/favicon.ico" rel="shortcut icon" type="image/x-icon">

Digest Mode
If "Digest Mode" is enabled (usually in production), it should generate the file name with the digest suffix (see https://github.com/lotus/assets/blob/master/lib/lotus/assets/configuration.rb#L37).
<%= favicon %>

<link href="/assets/favicon-78a21a377046e9ddb3bd9b509c22d4c3.ico" rel="shortcut icon" type="image/x-icon">

Testing Notes
Add an integration test with Lotus::View, in order to make sure that the output isn't escaped twice and produces a valid HTML.

I'd be glad to pick this one up.

@jasoncharnes Please do.

I'm a little more tied up than I thought I would be. If someone else wants to take this, feel free. If not, I'll get to it as soon as my schedule opens.

@jasoncharnes Sure, thank you.

I would love to give this a shot!

I have opened a PR #409.
One question I did have was with regards to mobile Safari. In the Rails world, favicon_link_tag allows for the passing of options rel="apple-touch-icon". Is this something you would want to see happen in this helper or is the intent to keep it separate?

@deepj to avoid duplicating works, please ask before claiming the issue.
@michaeldeol I have merged #408, but I would love to get your integration in too, please update your PR with just the integration spec and I'll merge

@joneslee85 done, just waiting on travis.

@joneslee85 Thanks for notice. I hadn't have the refreshed issue in a tab so I didn't notice @michaeldeol claim. I'm sorry for that.


Linter does not recognize same docstring issues as flake8 on command line

On Atom version 1.0.5
The linter always raises "D100 - Missing docstring in public module" even if said docstring is present, but does not raise any other valid docstring warnings.
Example:

vs.
ff/team/__init__.py:5:1: D101 Missing docstring in public class
ff/team/__init__.py:12:1: D102 Missing docstring in public method
ff/team/__init__.py:18:1: D105 Missing docstring in magic method

This just forwards on the messages from flake8, are you sure Atom is using the same flake8 you are testing with? (Different virtualenv?)

I am also spotting this problem all the time. I have specified the path of flake8 but it seems the error still does not match what outputted from flake8 directly. For example,

Again, are you sure that the flake8 running in Atom is the same one you are testing in the terminal?

Also, when testing from the command line make sure you are using stdin as the input:
cat dep_w2v.py | flake8

yes... I only have one flake:
$ for p in ${(s.:.)PATH}; do echo $p; ll $p | grep flake8; done
/sw/bin
ls: /sw/bin: No such file or directory
/sw/sbin
ls: /sw/sbin: No such file or directory
/Library/Frameworks/Python.framework/Versions/2.7/bin
ls: /Library/Frameworks/Python.framework/Versions/2.7/bin: No such file or directory
/Library/Frameworks/Python.framework/Versions/3.3/bin
ls: /Library/Frameworks/Python.framework/Versions/3.3/bin: No such file or directory
/usr/local/bin
-rwxr-xr-x    1 xiaolong  admin   232B Nov 26 14:08 flake8
/usr/bin
/bin
/usr/sbin
/sbin
/opt/X11/bin
/Library/TeX/texbin
/Users/xiaolong/local
ls: /Users/xiaolong/local: No such file or directory
/Library/Frameworks/Python.framework/Versions/3.3/bin
ls: /Library/Frameworks/Python.framework/Versions/3.3/bin: No such file or directory
/Library/Frameworks/Python.framework/Versions/2.7/bin
ls: /Library/Frameworks/Python.framework/Versions/2.7/bin: No such file or directory
/Users/xiaolong/local
ls: /Users/xiaolong/local: No such file or directory
/Library/Frameworks/Python.framework/Versions/3.3/bin
ls: /Library/Frameworks/Python.framework/Versions/3.3/bin: No such file or directory
/Library/Frameworks/Python.framework/Versions/2.7/bin
ls: /Library/Frameworks/Python.framework/Versions/2.7/bin: No such file or directory
/Users/xiaolong/local
ls: /Users/xiaolong/local: No such file or directory
/Library/Frameworks/Python.framework/Versions/3.3/bin
ls: /Library/Frameworks/Python.framework/Versions/3.3/bin: No such file or directory
/Library/Frameworks/Python.framework/Versions/2.7/bin
ls: /Library/Frameworks/Python.framework/Versions/2.7/bin: No such file or directory

I tried to pipe in the file but flake8 seems not work that way:
$ cat dep_w2v.py | flake8
Usage: flake8 [options] input ...
flake8: error: input not specified

Ah, sorry you need a - at the end: cat dep_w2v.py | flake8 -

Oh yeah, I figured that out. But it's wired... After piping the file to flake8, it seems that flake8 output indeed throws D100:
$ cat dep_w2v.py | flake8 -
stdin:1:1: D100 Missing docstring in public module
stdin:2:1: F401 'nltk' imported but unused
stdin:8:1: F401 'numpy' imported but unused
stdin:9:1: F401 'sys' imported but unused
stdin:124:5: F841 local variable 'W' is assigned to but never used

Which pep257 plugin are you using? flake8-docstrings or flake8-pep257?

$ flake8 --version
2.5.0 (pep8: 1.6.2, pyflakes: 1.0.0, flake8-docstrings: 0.2.4, pep257: 0.7.0, mccabe: 0.3.1) CPython 2.7.10 on Darwin

Seems flake8-docstring. This comes from flake8 I believe

Can you try downgrading pep8 to 1.5.7? That is the latest currently released version that is compatible with flake8-docstrings.
(Requirement already up-to-date: pep8!=1.6.0,!=1.6.1,!=1.6.2,>=1.5.7 in c:\python27\lib\site-packages (from flake8->flake8-docstrings))

sure, but sorry can you let me know how to downgrade pep8 in flake8?

pip install pep8==1.5.7

I can only downgrade flake8 as a whole to 2.3.0
'''
$ flake8 --version
2.3.0 (pep8: 1.5.7, pyflakes: 1.0.0, flake8-docstrings: 0.2.4, pep257: 0.7.0, mccabe: 0.3.1) CPython 2.7.10 on Darwin
'''
And using that I am still getting:
$ cat dep_w2v.py | flake8 -
stdin:1:1: D100 Missing docstring in public module
stdin:2:1: F401 'nltk' imported but unused
stdin:8:1: F401 'numpy' imported but unused
stdin:9:1: F401 'sys' imported but unused
stdin:124:5: F841 local variable 'W' is assigned to but never used

I'm pretty sure v2.3.0 of flake8 doesn't support the latest pep257 :stuck_out_tongue:. You should be able to just do pip install -U flake8-docstrings and it should update that to the latest version again.
In any case this seems like an issue with flake8-docstrings so I'm marking this as closed. You should file an issue over there about this.

You can also try flake8-pep257, they just released 1.0.5 which should fix stdin on there.

thanks! after uninstall flake8-docstrings and install flake8-pep257:
$ flake8 --version
2.5.0 (pep8: 1.5.7, pyflakes: 1.0.0, flake8-pep257: 1.0.5, mccabe: 0.3.1) CPython 2.7.10 on Darwin

I can confirm that the problem seems no longer present.
$ cat dep_w2v.py | flake8 -
stdin:2:1: F401 'nltk' imported but unused
stdin:8:1: F401 'numpy' imported but unused
stdin:9:1: F401 'sys' imported but unused
stdin:25:1: D103 Missing docstring in public function
stdin:38:1: D103 Missing docstring in public function
stdin:53:1: D103 Missing docstring in public function
stdin:62:1: D103 Missing docstring in public function
stdin:77:1: D103 Missing docstring in public function
stdin:98:1: D103 Missing docstring in public function
stdin:109:1: D103 Missing docstring in public function
stdin:124:5: F841 local variable 'W' is assigned to but never used
stdin:131:1: D103 Missing docstring in public function

Great news, thanks @Arcanemagus!

FYI, looked into it a bit and re-opened my original issue over on flake8-docstrings: https://gitlab.com/pycqa/flake8-docstrings/issues/2#note_2807528
It's simply reporting a D100 error for any file passed in via stdin.

I installed flake8-pep257, and the warning gone.
pip install flake8-pep257


kiab 0.7.3

Point to KIAB 0.7.3 release and update control flags table with self hosted git functionality (gitea).

Thanks @sergiohinojosa for the PR - looks good.
Only thing: you need to sign your commit to comply with the DCO. Learn how to do this: https://github.com/keptn/tutorials/pull/100/checks?check_run_id=1424490423
(if you need help please reach out)

I hope it's ok now, if not I would need a bit of help with this DCO.

Thanks @sergiohinojosa ! It's merged now and will be live within minutes.


Update for CUDA 12.4.1

xref: https://github.com/conda-forge/cuda-feedstock/issues/23

@conda-forge-admin, please restart ci


Object destructuring support

It works, but the editor is a bit rough now.
I wanted to add a right side transformation for JSBindingPatternBinding to add initializer on "=" but that didn't work.
We also need a transformation to add ...rest pattern, but I don't know how to do that yet.
Going to allow destructuring in more places now (funtion declaration parameters, var bindings, etc.)

Did you subscribe to the Slack channels? Traffic and users are constantly growing and many posts are getting answer, at least much more than one year ago.
The slack is here:   jetbrains-mps.slack.com

Will check it out now. Thanks!
I have added destructuring to variable declarations and reworked scope resolution to support that.

When you think you have done with restructuring let me know, I would try to
merge asap.
On Fri, Aug 30, 2019 at 3:01 PM Alexey Shmalko<EMAIL_ADDRESS>wrote:

Will check it out now. Thanks!
I have added destructuring to variable declarations and reworked scope
resolution to support that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mar9000/ecmascript4mps/pull/5?email_source=notifications&email_token=AACQ26BCO55ITLWBQBEGFFLQHEK2JA5CNFSM4ISL4BGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5RSXXA#issuecomment-526592988,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACQ26HPE5TXWRJUAQBST4TQHEK2JANCNFSM4ISL4BGA
.

--
twitter.com/mar9000
www.mar9000.org

It's mergeable now. I'll take a break now. Hope to add destructuring to function declaration and function expression tomorrow.

@mar9000 it's ready

With respect to JSBindingProperty you could have choosen a different concept name that match production alternative "PropertyName[?Yield, ?Await]:BindingElement[?Yield, ?Await]" better but it's ok, I'm saying it just to avoid misundertanding when looking at the language after months from the implementation. The implementation of SingleNameBinding and JSIBindingProperty perfectly implement the TC39 grammar in my personal vision.
I saw labeled statements with missing label in all-examples, Do you know why?

BTW you are doing an impressive job!

I've opened this issue https://github.com/mar9000/ecmascript4mps/issues/6
Can you comment on it?
Thx

I saw labeled statements with missing label in all-examples, Do you know why?

I've accidentally messed them up while implementing the migration. Thank for spotting!

With respect to JSBindingProperty you could have choosen a different concept name that match production alternative "PropertyName[?Yield, ?Await]:BindingElement[?Yield, ?Await]" better

I thought about that but I don't have a better idea for the name.

I've opened this issue #6
Can you comment on it?

Well, this is not something I am particularly interested in—at least for now. Pasting javascript text looks like a quite big endeavor but we're missing a lot of basic staff.

Ok, I'm going to merge.

BTW for the "we're missing a lot of basic stuff",  feel free to open an issues for the most wanted features. I stopped development because I saw few to nothing interest in this project and I delay its adoption in my daily activities.

Yeah, I just started using ecmascript4mps for my other project (ce3) and I discover and implement missing features on the go :slightly_smiling_face:

Awesome!
On Tue, Sep 3, 2019 at 10:00 AM Alexey Shmalko<EMAIL_ADDRESS>wrote:

Yeah, I just started using ecmascript4mps for my other project (ce3
https://github.com/rasendubi/ce3) and I discover and implement missing
features on the go 🙂
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mar9000/ecmascript4mps/pull/5?email_source=notifications&email_token=AACQ26BMJVQ6XBRNHQ7I2CDQHYKQDA5CNFSM4ISL4BGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5XLQ4Q#issuecomment-527349874,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACQ26EBD7EIUAHWJFRGAGDQHYKQDANCNFSM4ISL4BGA
.

--
twitter.com/mar9000
www.mar9000.org


Silent Sign-In causes Loop

Firstly, thanks a lot for your package and all the work that went into it.  We are using it at the moment to implement single-sign-on in conjunction with a KeyCloak server. We want to integrate a silent sign-in, but are running into an issue. The OIDC config looks like the following:
const oidcConfig = {
  authority: "<url>",
  clientId: "<clientId>",
  responseType: "code",
  scope: "openid",
  redirectUri: "http://localhost:3000",
  autoSignIn: false,
};

The login and logout work flawlessly with that configuration. However, the following component will cause a loop of requests to the KeyCloak server if the user is not logged in before finally stopping when a request runs into a timeout. The code of the component looks as follows:
import React, { useEffect } from "react";
import { useAuth } from "oidc-react";

const LoggedIn = () => {
  const auth = useAuth();

  useEffect(() => {
    const authenticate = async () => {
      if (!auth.userData) {
        try {
          await auth.userManager.signinSilent();
        } catch (e) {
          console.error(e);
        }
      }
    };

    authenticate();
  }, []);

  if (auth.userData) {
    return
Logged in!
;
  }
  return
Not logged in!
;
};

export default LoggedIn;

Which will cause the following cascade of requests in the IFrame the package creates:

The console logs:
LoggedIn.js:13 ErrorTimeout: IFrame timed out without a response
    at http://localhost:3000/static/js/bundle.js:11242:54
authenticate @ LoggedIn.js:13
await in authenticate (async)
(anonymous) @ LoggedIn.js:18
commitHookEffectListMount @ react-dom.development.js:23150
commitPassiveMountOnFiber @ react-dom.development.js:24926
commitPassiveMountEffects_complete @ react-dom.development.js:24891
commitPassiveMountEffects_begin @ react-dom.development.js:24878
commitPassiveMountEffects @ react-dom.development.js:24866
flushPassiveEffectsImpl @ react-dom.development.js:27039
flushPassiveEffects @ react-dom.development.js:26984
(anonymous) @ react-dom.development.js:26769
workLoop @ scheduler.development.js:266
flushWork @ scheduler.development.js:239
performWorkUntilDeadline @ scheduler.development.js:533
LoggedIn.js:13 TypeError: Failed to fetch
    at JsonService.fetchWithTimeout (JsonService.ts:50:1)
    at JsonService.getJson (JsonService.ts:89:1)
    at MetadataService.getMetadata (MetadataService.ts:52:1)
    at MetadataService._getMetadataProperty (MetadataService.ts:100:1)
    at MetadataService.getAuthorizationEndpoint (MetadataService.ts:64:1)
    at OidcClient.createSigninRequest (OidcClient.ts:113:1)
    at UserManager._signinStart (UserManager.ts:386:1)
    at UserManager._signin (UserManager.ts:379:1)
    at UserManager.signinSilent (UserManager.ts:246:1)
    at async authenticate (VM3354 bundle.js:201:11)
[...]

The individual redirects from the KeyCloak server all the form http://localhost:3000?error=login_required&state=<state-that-was-sent> which look to me correct, though I don't have too much experience with OIDC to say for sure.
I have a few questions concerning this behavior:

Is the KeyCloak server seemingly behaving correctly?
Is this an error that you have seen in the past?
Can you see something in the code snippet that might be wrong and causing this issue?
Can this be related to the browser that is used?

If one of the maintainers or anyone in the community can take a look at this, this would be much appreciated! 🙂

This is very old, but I figured I'd drop in what I do in this scenario:
Basically, the KeyCloak server in your screenshots appears to be responding with an error of login_required which implies you cannot do a silent sign-in. A silent sign-in works if you're already authenticated, or have a token that hasn't expired to allow you to perform sign-in again. So, what I do in a very similar looking component is the following:
try {
   await auth.userManager.signinSilent();
} catch (e) {
  if (e.error === 'login_required') {
    await auth.signIn();
  }
}

I also technically do a few steps before calling auth.signIn() to save things like where the user was navigating to, but that doesn't really matter for the specific issue at hand.

Hey @pseudoramble, thanks for your reply! You are exactly right, that I am not authenticated when I get the errors. However, I would think, that this error should only occur once 🤔
We have since I opened this issue swapped the frameworks, thus I will close this. But thank you for your information  and hints here!


Use unique sequenced id for metrics instead of full name

A significant amount of our traffic is metric ids. Another workaround might be to use prepared statements with "hardcoded" metric id and transmit a prepared statement id instead of whole metric.

Duplicate of #208


update finalzier to CRD doc

Depends-On: https://github.com/openstack-k8s-operators/lib-common/pull/519
Depends-On: https://github.com/openstack-k8s-operators/mariadb-operator/pull/231

/test ironic-operator-build-deploy-kuttl


spawn recipes on startup accorging to @trigger annotations

implements: https://github.com/PolymerLabs/arcs/issues/3823

addressed comments, PTAL


Initial cut at making the module and the example.

This pull request creates a Terraform module to deploy the reactive governance solution.
It includes the following major changes:

Moves the Java source code used in the Cloud Functions to a subdirectory inside of reactive-governance.  This makes it easier to create a zip file of the source code.
Templatizes the BigQuery view
Creates a Google Cloud Storage bucket in the host project to store the Cloud Function source code instead of pulling from a remote bucket.
Adds a simple sample deployment in the example directory.

This fixes Issue #4 and #5


New naming scheme

Counterpart of https://github.com/prebuild/prebuildify/pull/27. Major.

Looking really good! See my one comment


[event-hubs] Addressing feedback from Brian

Renames and consolidation:

on renamed to process
Moving the one positional parameter that was an event handler (onReceivedEvents) into the options structure along with the other event handlers. Made it required.
Changed the EventProcessorOptions interfaces to make it so batch options weren't required. I had to do this in a less terse way than I prefer because of typedoc but the important thing is that the public interfaces just use straight inheritance.

/azp run js - eventhubs-client – tests

/azp run js - eventhubs-client - tests


display: flex; makes the pagination not centered

Describe the bug
#133 introduces display: flex; which makes my page's pagination no longer centered.

To Reproduce
without display: flex: here
with display: flex;: here
Expected behavior
Without display: flex:

Versions:

Pagination: 2.3.0
jQuery 3.6.1
Browser Edge

Additional context
My HTML code is

I fix it by adding the style display: unset. I'd like to know if there are any suggestions.
.paginationjs {
    display: unset!important;
}

@njzjz justify-content also is an option:
.paginationjs {
    justify-content: center;
}

Thanks, it looks good to me.


Can't see BLE Long Range on Samsung Galaxy S10+

I downloaded your project and build it. I am not able to see my BLE device. But I have used NRF Connect android app it is showing the device. What could be that will be going wrong. Can you please further guide me on this.
Thanks

Maybe the discussion here will help you move forward?
https://github.com/opendroneid/receiver-android/issues/7
If not, please describe in a bit more detail what exactly you are doing and I might be able to help.


Copy row with generated columns

Description
If you copy a row from a table with a generated column, than this generated column would "saved/changed", too.
I mean, the generated column will be in the INSERT INTO statement, this throws an error.
DBeaver Version
CE 24.1.2 and all before
Operating System
any
Database and driver
10.11.8-MariaDB-log
Steps to reproduce
Sample:
CREATE TABLE test1 (
  first varchar(20)
, last varchar(20)
, together varchar(40) GENERATED ALWAYS AS (CONCAT(first, ' ', last))
)

INSERT INTO cockpit.test1 (`first`,`last`)
	VALUES ('Frank','Glück');

SELECT *
FROM test1

-- copy row (Ctrl+Alt+Insert)
-- edit some
-- generates following SQL:
--  Auto-generated SQL script #202407121253
INSERT INTO cockpit.test1 (`first`,`last`,together)
	VALUES ('Frank','Pech','Frank Glück');

Additional context
No response

thanks for the report.
The issue is reproducible if changes are not saved.
https://github.com/user-attachments/assets/ad35fd48-2ad4-47dc-ad2c-53536475c6b7

yep, a really annoying thing for me, too. I have to always manually set the cell to [Null] to temporarily solve this problem. Of course, it can be avoided by using SQL code, but as a layman, I use Dbeaver coz I want to use a visual interface to operate.