[Feature] Support Sats Names to make lighting address easier to recognize, easier to read, easier to remember

Feature description
If you don't know what is Sats Names, you can read this doc:
https://docs.sats.id/sats-names/about
Sats Names is a standard for writing names to Bitcoin using ordinals. The goal is to build a name ecosystem for Bitcoin, that is built by Bitcoiners, and developed entirely on Bitcoin.
it can make lighting address easier to recognize, easier to read, easier to remember, and most important, it's decentralized.

All namestate lives on Bitcoin
The protocol will remain credibly neutral
Names are inscribed as ordinals, and the only fees involved are paid to miners on the Bitcoin network.

FAQs: https://docs.sats.id/sats-names/faqs
Why we need this?

easier to recognize, easier to read, easier to remember, and most important, it's decentralized
It seems that some Lightning Network wallets are currently using email as an address scheme, but this solution is not decentralized, it's not neutral enough

Describe the solution
I'm not an engineer, but perhaps you can discuss with the author of Sats how to implement this. https://twitter.com/satsnames
My idea is as follows:
for instance, an address that owns bitcoin.sats could mint an inscription declaring that the Lightning Network address/invoice corresponding to that bitcoin.sats is ****.
Afterwards, when transferring to this bitcoin.sats via the Lightning Network, one could send to the Lightning Network receiving address corresponding to this bitcoin.sats
Describe alternatives
No response
Additional context
No response
Are you working on this?
None

for instance, an address that owns bitcoin.sats could mint an inscription declaring that the Lightning Network address/invoice corresponding to that bitcoin.sats is ****.
Afterwards, when transferring to this bitcoin.sats via the Lightning Network, one could send to the Lightning Network receiving address corresponding to this bitcoin.sats

Invoices are not reusable and lightning addresses depend on a domain, so in your example bitcoin.sats would just be a proxy for the lightning address<EMAIL_ADDRESS>I don't see why this would gain anything in terms of decentralization and why you wouldn't just give out your lightning address to others?

for instance, an address that owns bitcoin.sats could mint an inscription declaring that the Lightning Network address/invoice corresponding to that bitcoin.sats is ****.
Afterwards, when transferring to this bitcoin.sats via the Lightning Network, one could send to the Lightning Network receiving address corresponding to this bitcoin.sats

Invoices are not reusable and lightning addresses depend on a domain, so in your example bitcoin.sats would just be a proxy for the lightning address<EMAIL_ADDRESS>I don't see why this would gain anything in terms of decentralization and why you wouldn't just give out your lightning address to others?

now, we don't need to be a proxy for the lightning address<EMAIL_ADDRESS>we don't need to use this kind of domain.
we just need to a decentralized indexer, like a bitcoin wallet that can read amount of my bitcoin address.
a bitcoin wallet can also know which Invoices this "test.sats" point to by reading ordinals.
actually, test.sats can also point to evm address, do not need to do any complex things. just need wallet to index.

Do you mind to elaborate with a concrete example of how this use case and how this would fit into Alby?

Firstly, users can inscribe (mint) a valid inscription on Ordinals like this json:
{    "p": "sns",   "op": "reg",   "name": "test.sats",   "lighting-invoice": "*******",   "BTC": "bc1p*******" }
then as a wallet, wallet can index these inscription, so they can know which invoice a sats name points to.
so, when a user use lighting wallet, they click "send" button, and input test.sats, then wallet can know which invoice test.sats points to. then wallet can send bitcoin(sats) to this invoce.
also, there are many rules about what is valid inscription, you can see this doc: https://docs.sats.id/
This experimental protocol was recently proposed and has some incomplete aspects, such as the update procedure. However, it does provide us with a possibility to make Bitcoin addresses/Lightning Network addresses more readable and user-friendly in a fully decentralized manner.
If a wallet is willing to collaborate with the protocol author and the Sats community, it will accelerate the adoption of the protocol and also become one of the standard setters for the protocol. There is no project entity behind this protocol, and the project itself cannot generate any revenue. Once a user registers a name, they can use it permanently for free. Moreover, upon reading the documentation, you will know that it is fully decentralized, which is very cool and aligns with the spirit of the Bitcoin community.
most important, a readable address is useful and important for The widespread adoption of Bitcoin and the Lightning Network.

Thanks for laying that out. However that's not how lightning invoices work, they are not reusable so it makes no sense to inscribe them. This means you would have to inscribe something that can generate lightning invoices on demand (such as a lightning address) which brings us back to the original question:
What is the benefit of having a .sats name (that is resolved to a lightning address) over having a lightning address?

another thing, some wallets ( popular wallets in the Ordinals community ) have support sats names,
such as unisat, [url] (https://chrome.google.com/webstore/detail/unisat-wallet/ppbibelpcjmhbdihakflkdcoccbgbkpo?utm_source=chrome-ntp-icon)

you can try it.

For on-chain addresses that could work of course, however keep in mind that the full transaction history of those names will be recorded and publicly accessible forever.

Thanks for laying that out. However that's not how lightning invoices work, they are not reusable so it makes no sense to inscribe them. This means you would have to inscribe something that can generate lightning invoices on demand (such as a lightning address) which brings us back to the original question:
What is the benefit of having a .sats name (that is resolved to a lightning address) over having a lightning address?

this is a good questions, but now, we have bolt12, https://bolt12.org/
with bolt12, we can reuse an "involce" if I understand correctly,  they call it "offer", it can create static Lightning Network invoices

what does this mean? I can not understand.

what does this mean? I can not understand.

That the full transaction history of this .sats name will be publicly visible to anyone?

what is transaction history? you mean Transaction history in the trading market？

what does this mean? I can not understand.

That the full transaction history of this .sats name will be publicly visible to anyone?

or the mint/update history in Ordinals? It is completely public, just like Bitcoin addresses. so what impact?

@reneaaron  so what's your opinion?

No, I was just trying to point out that any payments you receive to your .sats name will be publicly visible and that address reuse is something to avoid.

No, I was just trying to point out that any payments you receive to your .sats name will be publicly visible and that address reuse is something to avoid.

Your statement is rather arbitrary, neglecting user needs. Why should Bolt12 support static fixed addresses? There are real user demand scenarios for this, such as tipping. Avoiding the use of fixed addresses caters to the needs of some people, but there will still be other demographics or use cases that require fixed addresses.
if you are product designer, you should not neglect some use cases.

Why should Bolt12 support static fixed addresses? There are real user demand scenarios for this, such as tipping.

I wasn't referring to bolt12, but to on-chain payments. Sorry for not being clear about that. With bolt12 that could be an interesting use case for sure.

Why should Bolt12 support static fixed addresses? There are real user demand scenarios for this, such as tipping.

I wasn't referring to bolt12, but to on-chain payments. Sorry for not being clear about that. With bolt12 that could be an interesting use case for sure.

I do not understand, bolt12 is useful, if give it a readable name, it also should be useful！
also, may be I misunderstand your questions, if you use sats names, it also can not be publicly visible.
because the anonymity of sats names and bolt12 is the same.

Why should Bolt12 support static fixed addresses? There are real user demand scenarios for this, such as tipping.

I wasn't referring to bolt12, but to on-chain payments. Sorry for not being clear about that. With bolt12 that could be an interesting use case for sure.

Do you have any other concerns？

The main advantage is that a user can have a moritz.sats address that can be resolved to a lightning address such as<EMAIL_ADDRESS>Do I get that right?
And the idea is Alby works like explorer to find the inscription with the lightning address. How can Alby know which moritz.sats to pick from the blockchain?

The main advantage is that a user can have a moritz.sats address that can be resolved to a lightning address such as<EMAIL_ADDRESS>Do I get that right?

I think for lightning addresses the benefits are not too compelling as they are already human readable identifiers. It probably makes more sense for Bitcoin addresses / BOLT12.
And the idea is Alby works like explorer to find the inscription with the lightning address. How can Alby know which moritz.sats to pick from the blockchain?
You need to scan the blockchain for this particular inscription. Since we don't have blockchain data locally in the extension, we can only rely on APIs to that do that for us (or maintain our own indexer):
https://api.sats.id/names/satoshi.sats

now the lightning addresses has problems, it is centralized.
for example, if lots of users use<EMAIL_ADDRESS> then someday, btcpay.com is able to Steal all the money from his users. Or Hackers can steal all the money from their users, we need to believe this company.
you are right, BOLT12 + sats names, maybe it's a good solution.
Bolt12 is experimental now? not public?

I think all major node implementations are working on it, might still take some time until this is widely available and also wallets start supporting it.


Attempt to make the browser remember the current theme

Can be solved with cookies

Fixed in commit #f590f4


Use GitHub's native Dependabot to support "go mod tidy"

We've been using Dependabot (dependabot-preview) for upgrading Go
modules. However dependabot-preview has been replaced by the
GitHub-native Dependabot which supports "go mod tidy"!
This change introduces GitHub-native Dependabot by adding its new
configuration file.
https://dependabot.com/docs/config-file/
https://github.blog/changelog/2020-10-19-dependabot-go-mod-tidy-and-vendor-support/
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

No strong reasons. Let me update.


Feature hide duplicate days

This PR does two things.

Solves #2284 . As described in issue, keyboard focus does not update in inline version. By updating preSelection date when changing date by click the keyboard date and selected date will be in sync.
Before:

After:

While solving 1, i broke a the test: "should not switch months in inline mode when a day is clicked". After looking into it i saw that the test is not actually testing if the month is switched or not. But i found the case interesting. Firstly, i mean that in default inline case, if you click on a outside-month day, it should change month (and it currently does). Secondly believe it should be the case if its multiple months shown. The current version will switch between months when clicking on an outside-month day (the inline design of two months is another issue):

My take on a solution: Why show duplicate days at all when viewing multiple months?

This also effects non-inline version:

Hi, this is great. Thanks for putting this together. I agree with your assessment.

This didn't solve the issue with inline calendar rendering different dates twice, unfortunately. aka 2284


builtins: support current_timestamp with precision

PR: https://github.com/cockroachdb/cockroach/pull/42633
From release notes:

Users can now use the current_timestamp() function with a given precision from 0-6, e.g. SELECT current_timestamp(4). [#42633][#42633] {% comment %}doc{% endcomment %}

Not really called out in the auto-generated docs. Not sure if this is a problem.

https://www.cockroachlabs.com/docs/v20.1/functions-and-operators.html lists precision syntax for current_timestamp. Also, we call out supporting precision for TIMESTAMP in this PR: https://github.com/cockroachdb/docs/pull/6784


Use ncp?

Looking at cli.copy and wondering if ncp would be helpful for package maintainers. If my package has multiple migration files, or multiple config files in subfolders, using ncp under the hood rather than fs.copy would make publishing easier. Thoughts?

Yes seems like it makes sense to use ncp. Adding to enhancements and will publish with final release, coming first week of November

Please assign if you would like me to implement

Looking into it and found that there is catch with the package, that it will overwrite the existing files, I don't want that to happen

Here is a sample that does not clobber https://github.com/benallfree/ncp-test

@benallfree Could you create a PR with tests to verify that files aren't overwritten?

Closing since no response from the issue reporter and not actionable as well


[Enhancement] add camera-streamer option

Please add an option (and the code) to install and configure camera-streamer.
I realize this won't be simple (as I've tried it unsuccessfully) but I believe that with the cooperation with the camera-streamer author and various OctoPrint gurus (including Gina) this would be an excellent addition to this project.
For non-Raspberry Pi systems, this may expand to installing and configuring libcamera.

Everything is already there to install camera-streamer except for setting up service files. There are several issues with using camera-streamer, which is why I haven't actually included it as of yet. The primary reason is that since the new camera stack for OctoPi is using it, it will lead to confusion on HOW cameras are setup. Did someone create a camera-streamer service with octoprint_deploy, or did they use OctoPi scripts to add the camera? A second reason is that the more options that are present, the harder it is help people debug issues. This is the reason I removed mjpg-streamer entirely. I'm not convinced that camera-streamer is so much better than ustreamer that it warrants making this change.

Except that ustreamer doesn't support CSI cameras (i.e. the Raspberry Pi Camera 3). If there is an alternative that supports these cameras then I'd accept it.

This is the issue that I considered when I chose to not include camera-streamer. If you are using a RPi 3 camera, you can always just use OctoPi and have camera-streamer immediately ready to go. From there, you can use the OctoPi tools to install any additional cameras. I would like to keep octoprint_deploy image agnostic and constantly having to make exceptions for OctoPi didn't seem worth it anymore.

Except that RPi 5 support is currently missing from OctoPi 1.0.0 (either camera stack).
I have an Orange Pi 3B and an Pine64 Quartz 64 B both of which have CSI interfaces. I used octoprint-deploy on both successfully. There are other Orange Pi offerings that also include CSI interfaces. I just don't own any of them.
I also have a Libre Computer Le Potato AML-S905X-CC but no CSI so ustreamer is the only option (and it works).
I understand your reluctance but I believe some solution for the CSI will be "image agnostic" and apply to more systems that just Raspberry Pi (and OctoPi).

I would be looking for suggestions on how to actually setup CSI cameras with octoprint_deploy. Right now, I only have the picam command line argument, which basically just defaults to using /dev/video0 (I believe) for the next camera that is setup. It is much easier to do USB detection of cameras.


trident-csi pods stuck in ContainerCreating after node reboots

Describe the bug
multiple trident-csi pods are stuck in ContainerCreating after node reboots with the error:
Generated from kubelet on node:
2 times in the last 3 minutes
(combined from similar events): Unable to attach or mount volumes: unmounted volumes=[trident-csi-token-z59nn], unattached volumes=[pods-mount-dir dev-dir host-dir trident-tracking-dir plugin-dir sys-dir certs trident-csi-token-z59nn plugins-mount-dir registration-dir]: timed out waiting for the condition
Generated from kubelet on node
19 times in the last 23 minutes
MountVolume.SetUp failed for volume "trident-csi-token-z59nn" : secret "trident-csi-token-z59nn" not found
If I delete the pod a new trident-csi pod is created and started ok but without manual intervention the original pod hangs forever and other pods on that node that use trident persistent storage fails to start.
I also noted that when it hangs it references a secret trident-csi-token-z59nn that doesnt exist and after I manually delete the pod and it starts up that pod references another secret that actually exists.
Environment
Openshift 4.7.13

Trident version: 21.04.0
Trident installation flags used: default install using helm
Kubernetes version: v1.20.0+df9c838
Kubernetes orchestrator: Openshift 4.7.13
NetApp backend types: ONTAP-NAS

To Reproduce
Steps to reproduce the behavior:
Reboot node
Expected behavior
The trident-csi pod to start by using the correct secret
Additional context
Add any other context about the problem here.

Hello @ gorantornqvist
Thanks for reporting this issue. To give you some background, the secret token trident-csi-token-z59nn is created when Trident creates a service account name trident-csi. Trident deployment and daemonset pods use the service account token for API authentication.
The behaviour that exists in Kubernetes is that if a service account is re-created the corresponding token is refreshed but the pods using the old token are not automatically updated. So, what Trident does is automatically re-creates Trident deployment and the daemonset pods on service account recreation.
In your case, I am trying to understand:

If the service account trident-csi was re-created?
a. If yes, was it before node reboot, during node reboot or after the node reboot?
b. If not, can you consistently reproduce the behaviour and does it involves just rebooting the Kubernetes node?
The Trident operator logs may also be useful in getting some insights, if you can share them here or on Slack or via Support case, that would help as well. Using kubectl -n <trident installation namespace> logs <trident_operator_pod>.

Please let us know.
Thank you!

Reincarnation of #444 ?

@gorantornqvist, can you provide more information based on @ntap-arorar's comments?

Hi,
Nothing was really done with the trident configuration before this.
I actually encountered the same issue on 2 different clusters but after the restart of the pods the issue was resolved.
I  tried restarting each node in these 2 cluster and the issue didnt occur again - it cant be reproduced.
So I guess this could be hard to troubleshoot.
I am OK with closing this and if it occurrs again I will gather all logs from the trident operator ...

@gorantornqvist, thanks for the feedback. We will reopen this issue if you encounter the problem again.

Hi,
We encountered this issue again today when updating 2 different openshift clusters.
If I deleted a trident-csi pod it started working (no need for a operator pod restart)
Attaching operator logs from one of the clusters
trident-operator-86c5b968cb-gz6p9.log

@gorantornqvist, where you able to resolve your issue?

We havent encountered this problem again so this issue can be closed :)

Thanks, for the update!


ARM64v8 latest unavailable

Currently there is no build for 4.2, 4.2.2 and latest for the arm64v8 platform.
Was that platform deprecated?

See https://github.com/docker-library/official-images/pull/7094#issuecomment-564244244 -- it disappeared from the upstream repos, so the best bet would be to ask upstream why that happened.  Hopefully it's a simple oversight, but we don't really have any more visibility than that.


Revert "Add query.rate to the autoscaling metrics"

Reverts vespa-engine/vespa#16745
For some awkward reason this seems to have made the queries.rate gone missing in a system test here:
https://git.vzbuilders.com/vespa/systemtests/blob/2619415f7e7a41abd344731791d7e7ae3bd8fb2f/tests-internal/search/metricsproxyyamas/metricsend2end.rb#L84

hehe, ok


e2e nightwatch vulnerabilities npm audit fix

Version
3.0.4
Node and OS info
osx, node v10.10.0
Steps to reproduce

Install any package from NPM,
You'll get prompted to run npm audit fix for 6 vulnerabilities
Run npm audit fix
Problem still there
Run npm audit fix - f
Still something with vue-cli-plugin-e2e-nightwatch that keeps bugging

What is expected?
To not have any issues
What is actually happening?
Nightwatch keeps on yelling it has some problems

Well, since I don't event use nightwatch I don't really have a problem with it. But I thought it would be nice to let you know :-)
Cheers guys, you do an amazing job!

Thanks for the report.
Unfortunately we can't do much about this, it's nested depedencies of nightwatch that are affected, and the semver versions of these packages don't allow an update to a secure version.
Fortunately though, since nightwatch isn't generating any user-facing code, and not running any user-provided input, the type of security problem that's reported here - a DOS attack / command injection - isn't really an issue, at least that's how it seems to me.
So we (including you) can reach out to package maintainers and open an issue about this, hoping they adjust the dependency version ranges, but than than that we can't do much.

Thank you for your quick response Linus!


Switch to GitHub Releases

Hey,
the project is now on maintained on GitHub.
Maybe the recipe should switch the .tar.gz URL:
https://github.com/erikd/libsndfile/releases

I don't think this is a good idea, because those files have quite different contents:

http://www.mega-nerd.com/libsndfile/files/libsndfile-1.0.28.tar.gz
https://github.com/erikd/libsndfile/archive/1.0.28.tar.gz

For starters, the file configure isn't available in the second one.
The first one is the official release tarball, created by the author.
The second one just holds a snapshot of the repo, automatically created by Github.

Oh, maybe @erikd might clarify this. I guess that should not happen...

The official release files are on www.mega-nerd.com. I don't support the repo snapshots created by Github.

Ok thank you all for your clear explanation and clarification about this.

@erikd quick bot on release files: there are means to upload generated tarballs as github releases. Libzmq for example does that.


Collected components re-render when an unused property is created

Recollect version
5.10
Describe the bug
initStore({
  data: {accessedByA: 100}
});

const A = collect(({store}) => { ... accesses store.data.accessedByA ... });

const B = collect(({store}) => { ... creates & modifies store.data.createdByB ... });

const App = collect(({ store }) => {
  return [<A/>, <B/>];
});

Creating store.data.createdByB in B causes A to be re-rendered. This should not be the case because A never accesses the property.
Modifying store.data.createdByB in B does not cause A to be re-rendered.
Here is a demo.

Thanks for the input and the demo!
This is by design. A component that subscribes to changes in store.data.accessedByA is also subscribed to changes in store.data. Imagine if B did store.data = 'Oh no!', this would change the data used by A, even though it didn't change store.data.accessedByA directly.
Creating a new property on an object is considered a modification of the object, since a component could listen to if (Object.keys(store.data).length) /* do something */. Hence, if you change the number of keys on an object (or array length, or map or set size), it counts as modifying the object.
Check out When will my components be re-rendered? in the readme and this test: https://github.com/davidgilbertson/react-recollect/blob/master/tests/integration/updating.test.tsx, which includes the same logic as your demo.
I think there is potentially room for improvement here, since as you rightly point out, this can cause renders that aren't necessary. But it's a balancing act: every extra check when reading/writing adds a tiny bit of runtime overhead, and increases bundle size. And adding new props to the store is probably not something that's done with great frequency. And the renders caused by this aren't actual DOM updates, React will bail early because the VDOM won't have changed, so the impact should generally be negligible.
I'm open to suggestions but for now I'm happy with this balance.

Thank you for the information!
That makes sense, I hadn't thought about cases like if (Object.keys(store.data).length) /* do something */.
My original issue, an unnecessary re-render, can be fixed by wrapping the created property in an object, i.e. store.data.extra: {createdByB}, where store.data.extra already exists.

Oh there's so many weird edge cases :)
Your solution sounds good. I might add that to the readme.
Also, if you know in advance what props you'll be using, you can run this before mounting your components to minimise updates later. Same idea as 'initial state' in Redux reducers.
initStore({
  data: {
    usedByA: null,
    usedByB: null,
  },
});


regression: when running demeteorizer, the platform used is darwin

I'm running on a Linux 64bits machine:
# dfischer:phantomjs installs the npm module phantomjs-sun
# which reads the 'process.platform' variable (https://github.com/gadicc/phantomjs/blob/master/install.js)

$ meteor add dfischer:phantomjs
$ demeteorizer -a os.linux.x86_64
$ file .demeteorized/bundle/.../phantom/bin/phantomjs
.demeteorized/bundle/.../phantom/bin/phantomjs: Mach-O i386 executable

It is as if the platform was not used at all in the installation of the npm package. Any hint about that?
I'm saying it's a regression because with pre-v3.x versions, the package was added to the package.json, and during the npm install the correct platform was used by npm and I got the linux version of phantomjs

So, the architecture option is kind of weird... It's passed down to build, but
packages don't necessarily have to honor it. This is the case here. If the
module isn't compiled for the target arch and doesn't have a step for
recompiling on build, you get this. :(

Is this still an issue? It seems impossible for Meteor to compile for a target other than the current platform - so even specifying --architecture may nor work correctly. :(
It looks like you've come up with some workarounds for this anyway, correct?


AI gets spawn again if loading save that has streaming levels

If player saves after streaming level loads and then tries to load another set of ai characters will spawn along side the ones from save
They also will be broken as they won't have any controller

fixed in 0.4.4 by checking if Controller is valid
If Controller is not valid it checks if DeleteOnLoadIfNoController? is true and if it's dead(in case if lack of controller is intentional)
If all conitions are met AI gets deleted

Using dead as one of the options(to prevent unnessasary deletion of dead bodies) lead to creation of the clones of the dead bodies

Removing dead check solved the issue


[BUG] TimeoutException: Cannot make a working EventHubConsumerClient

Describe the bug

I have successfully built an EventHubProducerClient using the connection string from my Event Hubs instance. I was able to see in the Azure portal that my logs came in (seeing the correct number of Incoming Messages).

Now I am trying to consume back the log that I sent. I have followed the documentation to build out an EventHubConsumerClient and I am trying to first check my partition ids to check that the consumer is built out properly. When I try to print my partition ids, I get a timeout exception.

Note: I've also failed to receive events and it errors out. Which leads me to believe all operations with the consumer are timing out.
Exception or Stack Trace
Exception in thread "main" reactor.core.Exceptions$ReactiveException: java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'source(MonoWhen)' (and no fallback has been configured)
	at reactor.core.Exceptions.propagate(Exceptions.java:336)
	at reactor.core.publisher.BlockingIterable$SubscriberIterator.hasNext(BlockingIterable.java:162)
	at com.eis.kraken.EventHubProducerMain.main(EventHubProducerMain.java:84)
Caused by: java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'source(MonoWhen)' (and no fallback has been configured)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:288)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:273)
	at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:390)
	at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
	at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
	at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:834)

To Reproduce

Create an Event Hub name space.
Create an Event Hubs instance.
Create policy on the instance with Manage permissions and get the connection string
Build an EventHubConsumerClient with the connection string and default consumer group.
Get all the partition ids from the EventHubConsumerClient.
Run a foreach to print out your partition ids

Code Snippet
      Logger logger = LoggerFactory.getLogger(EventHubProducerMain.class);
      String connectionString = System.getenv("CONNECTION_STRING");

      EventHubConsumerClient consumer = new EventHubClientBuilder()
              .connectionString(connectionString)
              .consumerGroup(EventHubClientBuilder.DEFAULT_CONSUMER_GROUP_NAME)
              .buildConsumerClient();

      IterableStream<String> partitionIds = consumer.getPartitionIds();
      for (String partitionId : partitionIds) {
        logger.info(partitionId);
      }

Expected behavior
Expected to print out the 2 partition ids that are created in my instance
Setup (please complete the following information):

OS: macOSX
IDE : IntelliJ 2020 Professional
Version of the Library used : 5.0.2

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

@ShahmirAhmed Thank you for reporting this issue. Is your host behind a proxy? If so, you might have to configure proxy when building your client.

So I'm working on a corporate network and after talking to our cloud team, we realized that our network policy blocks port 9093, which is used by this sdk to receive logs from Event Hubs. Is there any way to override that?

@ShahmirAhmed The ports cannot be overridden from the SDK. You may, however, set up a proxy server that whitelists Azure Event Hubs ports and connect via proxy.

Hi @srnagar yes setting up the required transportType on my end has helped me. Thank you so much for your help. My issue has been resolved

@ShahmirAhmed Great to know! Thanks for the update.


/nllm forces recache

Eliminate with prejudice, or at the very least rate limit its use.

This doesn't apply only for nllm specifically, I only used it as an example. It's rather group managers behavior, where it will always attempt to load a group from the database, if it cant be found in the cache, so any commands with bogus group names will force a lookup in the db.

Oh, well, failure to find is very lightweight. I thought from what you said that giving it a valid group name would force a recache.
I'm not very concerned about bogus data causing cache hits and DB checks, those are pretty fast. Still not a bad idea to rate limit though...


[Feature] Revisit challenge card designs for those with longer titles

What projects is this feature for?
OpenChallenges
Description
Challenge cards with long titles (span over 3+ lines) doesn't look too great, e.g.

We could either:

increase the card height
overwrite the line height set for the card-title
set a character limit for the card title
hide the platform information if the title is too long

Of these, I like option 2 the best, followed by 1 or 3.
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I agree to just adjust the title height since we already have max character limits (60) for challenge name.

This challenge title spread over 4 lines. We probably don't want titles to spread on so many lines.
https://openchallenges.io/challenge/110/oropharynx-radiomics-hpv
Ideas:

Slightly decrease the font size
Fix the height of the title section so that only three lines are displayed.

The three lines displayed are probably enough to uniquely identify the challenge.

This challenge title spread over 4 lines. We probably don't want titles to spread on so many lines.

The challenge name is 92 chars. Is it a bug?
https://github.com/Sage-Bionetworks/sage-monorepo/blob/9807dd9a5b68a6c2796b6994231e5e2377bd0692/libs/openchallenges/api-description/src/components/schemas/ChallengeName.yaml#L1-L4


python3Packages.cmd2-ext-test: init at 2.1.2

Motivation for this change
This plugin supports testing of a cmd2 application by exposing access cmd2 commands with the same context as from within a cmd2 pyscript.
https://github.com/python-cmd2/cmd2/tree/master/plugins/ext_test
Will be needed for faraday-cli (#138944) to enable the tests.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 138946 run on x86_64-linux 1

2 packages built:

python310Packages.cmd2-ext-test
python311Packages.cmd2-ext-test


how to add different user as "run in user's context" while registering Plugin, we are not able to using spkl

@scottdurow - is there anyway to do this?


Allow "allowed" parameter to be fetched from  body

On most "authorize" flows the authorization step is a form post to request permissions from a user, so this parameter should be fetched from the body. In fact, IMHO, it should not be fetched from the query.

I'll reopen for consideration!


Setup from source

Hi,
I'm using python 3.5 and I try to setup pyHarmonySearch from source by running python setup.py install, But I get the following error:

TypeError: chown() missing 1 required positional argument: 'numeric_owner'

How can I resolve the error? Is it maybe because I use python 3.5 and it's not compatible with the setup process from source?

Running Ubuntu on Windows, I just tried a completely fresh install with Python 3.5, and it's no problem:
gfairchild:~/projects$ git clone git@github.com:gfairchild/pyHarmonySearch.git
Cloning into 'pyHarmonySearch'...
remote: Counting objects: 255, done.
remote: Total 255 (delta 0), reused 0 (delta 0), pack-reused 255
Receiving objects: 100% (255/255), 79.83 KiB | 0 bytes/s, done.
Resolving deltas: 100% (150/150), done.
Checking connectivity... done.
gfairchild:~/projects$ cd pyHarmonySearch/
gfairchild:~/projects/pyHarmonySearch$ ls -la
total 36
drwxrwxrwx 0 gfairchild gfairchild  512 Apr 20 08:39 .
drwxr--r-- 0 gfairchild gfairchild  512 Apr 20 08:39 ..
-rw-rw-rw- 1 gfairchild gfairchild 1827 Apr 20 08:39 CHANGES.md
drwxrwxrwx 0 gfairchild gfairchild  512 Apr 20 08:39 examples
-rw-rw-rw- 1 gfairchild gfairchild 8596 Apr 20 08:39 ez_setup.py
drwxrwxrwx 0 gfairchild gfairchild  512 Apr 20 08:39 .git
-rw-rw-rw- 1 gfairchild gfairchild  726 Apr 20 08:39 .gitignore
-rw-rw-rw- 1 gfairchild gfairchild 1518 Apr 20 08:39 LICENSE.txt
-rw-rw-rw- 1 gfairchild gfairchild   20 Apr 20 08:39 MANIFEST.in
drwxrwxrwx 0 gfairchild gfairchild  512 Apr 20 08:39 pyharmonysearch
-rw-rw-rw- 1 gfairchild gfairchild 6796 Apr 20 08:39 README.md
-rw-rw-rw- 1 gfairchild gfairchild 3836 Apr 20 08:39 setup.py
gfairchild:~/projects/pyHarmonySearch$ pyvenv env
gfairchild:~/projects/pyHarmonySearch$ source env/bin/activate
(env) gfairchild:~/projects/pyHarmonySearch$ python --version
Python 3.5.2
(env) gfairchild:~/projects/pyHarmonySearch$ python setup.py install
running install
running bdist_egg
running egg_info
creating pyHarmonySearch.egg-info
writing pyHarmonySearch.egg-info/PKG-INFO
writing dependency_links to pyHarmonySearch.egg-info/dependency_links.txt
writing top-level names to pyHarmonySearch.egg-info/top_level.txt
writing manifest file 'pyHarmonySearch.egg-info/SOURCES.txt'
reading manifest file 'pyHarmonySearch.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'pyHarmonySearch.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
creating build
creating build/lib
creating build/lib/pyharmonysearch
copying pyharmonysearch/__init__.py -> build/lib/pyharmonysearch
copying pyharmonysearch/harmony_search.py -> build/lib/pyharmonysearch
copying pyharmonysearch/objective_function_interface.py -> build/lib/pyharmonysearch
creating build/bdist.linux-x86_64
creating build/bdist.linux-x86_64/egg
creating build/bdist.linux-x86_64/egg/pyharmonysearch
copying build/lib/pyharmonysearch/__init__.py -> build/bdist.linux-x86_64/egg/pyharmonysearch
copying build/lib/pyharmonysearch/harmony_search.py -> build/bdist.linux-x86_64/egg/pyharmonysearch
copying build/lib/pyharmonysearch/objective_function_interface.py -> build/bdist.linux-x86_64/egg/pyharmonysearch
byte-compiling build/bdist.linux-x86_64/egg/pyharmonysearch/__init__.py to __init__.cpython-35.pyc
byte-compiling build/bdist.linux-x86_64/egg/pyharmonysearch/harmony_search.py to harmony_search.cpython-35.pyc
byte-compiling build/bdist.linux-x86_64/egg/pyharmonysearch/objective_function_interface.py to objective_function_interface.cpython-35.pyc
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying pyHarmonySearch.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying pyHarmonySearch.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying pyHarmonySearch.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying pyHarmonySearch.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
zip_safe flag not set; analyzing archive contents...
pyharmonysearch.__pycache__.objective_function_interface.cpython-35: module MAY be using inspect.stack
creating dist
creating 'dist/pyHarmonySearch-1.3.3-py3.5.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing pyHarmonySearch-1.3.3-py3.5.egg
creating /home/gfairchild/projects/pyHarmonySearch/env/lib/python3.5/site-packages/pyHarmonySearch-1.3.3-py3.5.egg
Extracting pyHarmonySearch-1.3.3-py3.5.egg to /home/gfairchild/projects/pyHarmonySearch/env/lib/python3.5/site-packages
Adding pyHarmonySearch 1.3.3 to easy-install.pth file

Installed /home/gfairchild/projects/pyHarmonySearch/env/lib/python3.5/site-packages/pyHarmonySearch-1.3.3-py3.5.egg
Processing dependencies for pyHarmonySearch==1.3.3
Finished processing dependencies for pyHarmonySearch==1.3.3

So it's definitely not a 3.5 problem. The fact that it's complaining about chown tells me it's a permissions error. Are you trying to install within a virtual environment or globally on the system? If the latter, you may need to install using sudo. However, I highly recommend you don't install into the system Python. You should be using virtual environments to keep project dependencies separate.


feat: Configuration Method and expanded parameters for Database Model

SUMMARY
We are creating an alternative way of editing or creating a database, in order for this to be feasible we want to add a configuration_method column to the database model.
This PR adds the necessary logic to the model, with validation, and tests to make sure that it is working.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A
TEST PLAN
Unit tests added.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

This change has broken the superset-frontend though for master. configuration_method is not being passed from the database add modal. Works fine for edit since I'm guessing it's loaded from the api but when creating a new one it's giving an error like the attached.

My guess is superset-frontend/src/views/CRUD/data/database/types.ts needs to be updated, but I'm not too great at React to be sure.

@sujeetpillai I think this will be fixed with: https://github.com/apache/superset/pull/14583

@AAfghahi Thanks for the quick fix. I can confirm it's now fixed.

Thank you so much, I can confirm it's fixed as well
On Tue, May 18, 2021, 7:07 AM Sujeet Pillai @.***>
wrote:

@AAfghahi https://github.com/AAfghahi Thanks for the quick fix. I can
confirm it's now fixed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/superset/pull/14451#issuecomment-842840453,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3VSVXR7NJW2ZJBFXPHPSLTOHYXTANCNFSM44BTZ7MQ
.

Nice! Glad to hear it.


Refs #23919 --Replaced super(ClassName, self) with super() in forms

I suggest you replace all occurrences of the code base in a single commit.

OK. then allow me some more time. don't you think smaller moduler changes are better

I don't see any benefit in a large number of commits for such a change.

OK

By the way, you could probably write sed script to automate the process.

never used before, anything like http://askubuntu.com/questions/747450/how-do-i-call-a-sed-command-in-a-python-script this?

Over the 1371 instances of super() in the code base there's a risk we have a smart hack somewhere where super calls the grandparent instead of the parent.
I think the correct way to do this would be with a 2to3 transform or a script that checks the AST to ensure that ClassName is indeed the name of the class in which the super() call happens.
These changes have to be scripted; it's absolutely impossible to review them otherwise. For this reason I don't think it's productive to continue doing them one by one.

+1 to being careful

where super calls the grandparent instead of the parent.

We had a grandparent super call at work that came up when upgrading to Python 3. We actually used a regex one liner like sed to change them all, but luckily a test caught this one. Using a 2to3 transform/AST tool for Django though is sensible, we have to be as cautious as possible 🌵

I am checking the tool you mentioned in the article.

Adam and I are saying that while sed allows automating what you were doing manually, it's still insufficient because removing parameters from super() can change the semantics of the call in (uncommon) cases.
You can learn about sed -- it's a valuable tool -- but don't bother submitting a PR where parameters to super() are just just removed with sed: the consensus of committers is that it's insufficient.

Continued in https://github.com/django/django/pull/7905.


Remove version: Google.PlatformTools version 34.0.3

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

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

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Executable C:\Users\Validator\AppData\Local\Microsoft\WinGet\Links\adb.exe returned exit code: -1073741515
Executable C:\Users\Validator\AppData\Local\Microsoft\WinGet\Links\fastboot.exe returned exit code: -1073741515

Automated error analysis suggests -1073741515 may mean STATUS_DLL_NOT_FOUND - possibly due to a dependency on Microsoft.VCRedist.2015+.x64 (or .x86).
(Automated response - build 887.)

URL: - https://dl.google.com/android/repository/platform-tools_r34.0.3-windows.zip
Invalid URI: The URI scheme is not valid.
(Automated message - build 912)

Hi @drshtmstry,
The package is still available for download. Why is its manifest being removed?

URL: https://dl.google.com/android/repository/platform-tools_r34.0.3-windows.zip
Status Code: 200
(Automated message - build 887)

Hi @drshtmstry,
The package is still available for download. Why is its manifest being removed?

Old versions are giving validation error which new version don't have.

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\adb.exe returned exit code: -1073741515
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\fastboot.exe returned exit code: -1073741515

Automated error analysis suggests -1073741515 may mean STATUS_DLL_NOT_FOUND - possibly due to a dependency on Microsoft.VCRedist.2015+.x64 (or .x86).
(Automated response - build 887.)


Help needed to convert the model to Tflite

Please help me to convert the hand detection model to tflite.

I am needed help too!

Hi,
I have not tried convertiing this to tflite, but here is my suggestion.

The original model checkpoints are here . There is a savedmodel and checkpoint (you can export a frozen graph from the checkpoint) format
You can use the tflite converter to convert the above to tflite.

Hope this helps! And ofcourse, please share what you buiild!
-V.


Update logs with city and region names

Log file says “this” city or this region instead of actual names

When you hover over them it highlights the city/region/path in question.  Perhaps I should make it clickable though, so that one can use it from a tablet or phone.   We could add names but then we'd have to type them all in...

I guess I wasn’t able to see it since I was on my phone. I guess you can leave it as is. Not many would play on phone


Area2D's get_audio_bus() is a misnomer

Godot 3.0
Was reviewing documentation for Area2D, and I found the following:

A C++ property exists called audio_bus that holds the StringName of an audio bus associated with the AudioServer singleton.

C++ methods get_audio_bus and set_audio_bus act as getter/setter for this C++ property.

A GDScript property is bound by the name audio_bus_name and is given the above getter/setter as its own getter/setter.

This results in get_audio_bus() becoming a misnomer since it deals with an audio bus's name rather than an audio bus object. These methods should probably be renamed to get_audio_bus_name() and set_audio_bus_name(...).

Hey people should I change the documentation as well?

@paezao Yes, all instances of that function should be renamed 😃 .

@bojidar-bg Do I also need to change the docs? How?

@paezao If you don't know, you don't have to bother with the docs. If you really want, you can check the docs about it.

Fixed by #11497


Added a few visual features, pingbased backtrack, Fullbright light and other pullrequests extended and fitted to GUI.

Added:
-Ping Based Backtrack
-Draw all backtrack cham ticks
-Fullbright Light//mat_fullbright
-Ragdoll Gravity custom value changer
-Ragdoll Physics time scale (faster or slower moving body through time/air)//cl_phys_timescale
-Added remove weapon sway when moving/running option.//cl_use_new_headbob
-Viewmodel Move XYZ for weapons and knife independently(sv_minspec->setValue(1); is needed for more then 2.5 each xyz direction, now up to a max of 20.)
-cl_righthand in menu when selecting viewmodel for weapons and knife independent
-default (0.0.0) location for bomb equip viewmodel forced. (works 95% of the time)
-Hitmarker Damage numbers animation from chrosshair with menu settings integration.
-spec list on players you are spectating added
-Config save/load integration for all of the above.
////////////////////////////////////////////////////////////////////////
here is a link to compare
https://github.com/danielkrupinski/Osiris/compare/master...RyDeem:master
////////////////////////////////////////////////////////////////////////
if (config.human.reader) > (config.emotion.interested)=extended description{
-Pingbased backtrack is what it says on the tin, it makes the backtrack lenght match your ingame ping to server in ms. If you disable pingbased you can see the ping value in backtrack tick ms slider on the gui. Basically takes similar code like on the fps/ping watermark.
-Fullbright light = forces mat_fullbright 1 and disables brightness slider, as in my test it did nothing with fullbright on, change the world color for much crisper light now, and see chams better on no shadows and light changes in the enviroment.
-Gravity value changer = customize the gravity abount you get with the flip gravity gui button,
do -600(default or -2400 for fast moving flying upwards ragdolls.
-Added a gui interface thats not perfect onto ZerGo0'ds code for changing the hitmarker damage settings, would be better if i understood how he calls for movement direction in the code. not gud coder eh,? yas :/
-And i moved around things in the visuals window, hope people arent mad about that, i had to put my new GUI poppup sliders for gravity changer, viewmodel xyz and hitmarker at the bottom for max nice effect xd
-Added ragdoll Physics timescale float input window for when selecting the option in visuals, default is 1.0 and you can set it to 0.5 for half as fast ragdoll animation/physics or 2x for faster, onr anything inbetween. Goes up from 0.0x to 10x.
};
//////////
Thank you coders!
danielkrupinski#835 By ZerGo0 // Hitmarker Damage Indicator
danielkrupinski#894 By agshaidun // Spec List on other players
danielkrupinski#730 By JDeu // Draw All Backtrack Tick Chams
danielkrupinski#584 By effex1337 // View Model Offset XYZ

"if (config.human.reader) > (config.emotion.interested)=extended description{"
Lmao, made my day :p

shouldn't it be better if u pull request these changes one by one?

shouldn't it be better if u pull request these changes one by one?

it could be probbably better, but i dont really wanna go over it a 3rd time, but i can do that.

Im having fun now adding console commands that are behind sv_cheats 1 and clientsided for my personal branch, i got tons of shite i can add here, but like this is small amount of additions now.

Was thinking of building a model mover xyz for each of the guns individually or groups like in aimbot window, to have max customization.
would be cool.
the grenades look wierd a bit with default "weapon" location, so that would be 1st id do, set default 000 for grenades, but i dont know if my way of autoupdating is good, for me it doesnt drop frames, but i can imagine its changing the cvars every tick/cycle of code in its current state, so i dont really know how to fix it/make it call only on changes.

stop
Why did you open another useless PR like this, you're stealing code from the other branches when daniel could easily just accept all the pull requests.
You could have easily just updated your old pull request, instead you closed it and made another which will never get accepted.
What you could have done:
Make the feature you want and don't try to merge pulls for daniel, he is in control of the project. He will decide if he wants them in or not.

okboomer w/e not like im trying to help the cause by contributing code n stuff, developing stuff further and making life easyer by having them in one place, and for other people to see this modification of the master that has more features with easy access with one merge, instead of doing it themselves. Noone would see them even if they are not on master branch, as noone would see that they exist if not for this pull request.

10 tips for better Pull Requests. < link you should open

Make it small
A small, focused Pull Request gives you the best chance of having it accepted.
Do only one thing
the rest is irrelevent but you should read up on it either way.


Textual change homepage

From Desiree:
Yesterday I noticed that the Start page of the portal contains the text:
The regulations of the General Chamber or those of the Linguistics Chamber.
The two links do not work anymore. Could this also be adjusted to:
The regulations of the [FEtC-H] (https://fetc-gw.wp.hum.uu.nl/en/regulations-fetc-h/)

This was actually already solved in a previous branch, it just has not been pushed to production yet


removing xalan

Removing dependency on xalan by updating dependency on sign service pdf commons
pdf commons is currently on SNAPSHOT and must be released first.
Resolving issue #49

Fixed in PR https://github.com/idsec-solutions/signservice-integration/pull/52.


Adds module for the director style QuinticWalk

Adds a new QuinticWalk module in module/skill
Pretty much the same logic as the original QuinticWalk but using director <Provide> .
Updated walk related roles but left director stuff commented for now.

Make sure you add the walk to the webots/directormotion role otherwise CI won't build it.

Make it compile. and put utility/motionWalkEngine* into utility/skill/WalkEngine*


Merge of resource set

    private static string RemoveTrailingZerosFromMillisecondsForAGivenDate(DateTimeOffset date)

I am not sure why we need to have this special method. There should be a place that converts date to desired format when we create raw resource string. Why cannot we use that conversion here too?

Refers to: src/Microsoft.Health.Fhir.SqlServer/Features/Storage/SqlServerFhirDataStore.cs:427 in e78dbeb. [](commit_id = e78dbeb04876519e13ca8a406b3a570afdeed8f9, deletion_comment = True)

/azp run

/azp run


fix(imperva_securesphere): Support broken CEF header

In some cases Imperva sends a malformed header support this condition

:tada: This PR is included in version 1.51.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Files are opened but not closed

As an example, when a TiffAdapter is created it holds a handle to an open TIFF file (via tifffile.TiffFile).  After __init__() completes, that file handle is used by calls to metadata(), read(), and read_block(). The latter two already cache the array data for future reads.  I believe the file handle never gets closed until the server shuts down. I see this as hundreds of ResourceWarnings when running the test suite.
I expect other file-backed adapters act similarly.
With cached read operations already in use (and assuming metadata() results could be cached), is it necessary to keep the file open for performance (server response time)?  Did earlier versions already try opening the file on demand when the cache misses?
Alternatively, I suppose the server could have a cold timeout...so that if a file hasn't been accessed for X minutes, then the file handle gets closed.

Some of the ResourceWarnings can be addressed by improving our resource handling in the tests.  PR https://github.com/bluesky/tiled/pull/678 has been submitted to correct those.

I wonder whether object_cache should be overhauled ~now. Like the old (now replaced) implementation of tiled.client.cache.Cache, it's a proof of principle put in place when we were still evaluating whether Tiled was even viable as a concept.
I think something much, much simpler would address the use case. Perhaps a TTL cache of file handles.

I think something much, much simpler would address the use case. Perhaps a TTL cache of file handles.

That makes sense.


No XML documentation file in publish folder

I have checked XML documentation option in project settings. The XML file is in output folder but not in publish folder.
Using VS.NET 2017 RC4.
dotnet --info
.NET Command Line Tools (1.0.0-rc4-004771)

Product Information:
 Version:            1.0.0-rc4-004771
 Commit SHA-1 hash:  4228198f0e

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.14393
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\1.0.0-rc4-004771

Dupe of https://github.com/dotnet/sdk/issues/795


Stuck on NVIDIA screen at boot

I have connected the 20.04 img to the jetson nano in many ways, but it always gets stuck on the NVIDIA screen at boot. The original nvidia img works, only this custom img gets stuck on login.

Sorry to hear.
It looks a hardware issue. Which Nano do you use? The 2 or 4 GB version. Or are you using the Orin Nano?
The image only works on the 4 GB Jetson Nano. Not on the Orin, nor on the 2 GB version.

I use the 4 GB Jetson Nano version. I have done it before for multiple times but now it is not working, I do not know whats the problem is.

I have been able to get Ubuntu 20 image working on my Nano, but now it fails.
d738f1fe20088a1bdbd10e2358b512f7  My md5sum
D738F120088A1BDBD10E2358B512F7  original md5sum

I see your md5sum differs from the original one. A typo? (f1fe20 vs F120)
It is strange the image has worked. The only "advise" is by starting with a fresh image all over again.
There are several files involved during booting. It seems one of them fails.
Tip: If possible use another new SD card. When it works, you may copy your software from the old one.

Thanks for reply, I will try and get back to you

You're welcome

Hey @meliheyuboglu , I'm also getting the same md5 as you. How did you manage to get it to work?

I followed the instructions under the "Split image." title.


soft_deactivation: Stop computing "new messages" in Python.

Rather than fetch all UserMessage rows for all streams, and subtract those out in Python-space from the list of all Message rows the user may have received -- do this via a "NOT EXISTS" subquery.  This is much better indexed (performing in fractions of milliseconds rather than hundreds), and also consumes much less memory.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Looks great, marked the merge once CI passes, thanks @alexmv!


[network] Prioritize peer connections and Remove VFN 2nd public network interface

Motivation
Having a 2nd public network interfaces causes tons of confusion, as well as makes it hard to keep track of the number of connections to individual nodes.  This removes it, and allows us to run a single network interface for known and unknown connections and simplifies this confusion.
Additionally, this prioritizes connections to other peers, now that there is only one connectivity manager due to there only being one public interface.
Currently builds upon https://github.com/diem/diem/pull/7925

/canary

:sunny: Canary successful

/canary

:sunny: Canary successful

overall looks good!!!! how well is it working? Did we take care of the issue of over dialing wherein we already have 3 inbound upstreams (public network, vfns dialed in), we don't want to also make 3 dials out...

This now just makes 3 dials out and vfns can dial in on the public network.  There's only one public interface now so it guarantees exactly 3 VFN out dials for a VFN.  There's additionally the one dial out for the validator.
Public full nodes similarly follow the same pattern, without the validator.

Did you push this into the next release set of partners repo? is that possible?

No, but I'll make sure it does.

Afaiu, there shouldn't be any quirkiness if we leave the old config as is... this requires tweaks to adopt this awesomeness, right? If there will be weirdness for old configs, let's figure out how we address them.

There's backwards compatibility for the old configs.
I think there might be one weirdness (which is why my performance change didn't work), but I needed a place to test it.

/land


Getting encrypted message

Yesterday for some reason i Re-registered my number with WART and after that we received encypted (junk) message, i dont know why and how this happing with us.. bcoz before register this was work fine... pls help us.. we receive message as below
3���!a�{4�$}T{X���R�pS@���I7�
^!�\�P��W�DR%7F�=��B�+��"B3
!���L��,�0s�U0%��t�J0�BY7��

@jaysonroni register with a different password (salt)

We already tried that , but not getting solutions.. pls help us


Update Spineomatic

Good afternoon everyone,
I have spineomatic 7.0 at my workplace and the API may break soon so I need to update it.  Unfortunately, the check for update functionality is not working.  Is there an easy way to upgrade what we currently have so I do not need to reconfigure this software?  It was installed before I started this position so I'm completely unfamiliar with it.  Thank you in advance.

To be safe, you can make a copy of the settings.som file in the folder in which the SpineOMatic program resides.  But the installation of a newer version is just copying a binary file.  You may download the latest version from https://github.com/ExLibrisGroup/SpineOMatic/releases.  It shouldn't affect your settings.

I expected this to be much worse.  I have already upgraded the programs and want to thank you for your assistance.  You saved me a lot of time and effort. Have a great weekend.


Appium inspector timeout issue

Problem
we recently updated:

Xcode to 9.1
Appium to 1.7.1
iOS simulator to 11.0
Device - Ipad Air 2

and noticed the appium inspector timeout most of the time while loading the view.

We never encountered so much delay or timeout in the previous version:

Xcode 8.3
Appium 1.6.5
iOS simulator 10.3
Device - Ipad Air 2

Is this a known issue ? Are you planning to push any changes ?

Yes this is known WDA issue. Apple did several changes in the recent iOS SDK and this made the stuff so slow


586 create twitter feed analyzer - Initial setup

Added stubs to identify location of work to be done, added link to page to display retrieved tweets

Approved. More notes here: https://docs.google.com/document/d/1LpRijA7ijR-VV7xfjhCfib12CIUz9tq_sA3igxRG1M4/edit#heading=h.jiccduxc3yjp


Example for different topics

It would be easier for someone reading the guide if we could create a example project (on github) and walk them through this for different aspects of best practices (licensing, issues, PR, github pages, etc)

This would be a considerable maintenance burden for us though and it is quite generic, so maybe this would be more suitable for The Turing Way? Maybe there are existing examples?

The Python template in a way does this, although specifically for Python. I agree it's a nice idea, but I would say it's a whole new project, outside of the direct scope of the Guide, mostly for the reasons @bouweandela mentions, i.e. maintenance burden. The Guide itself already has big maintenance issues.
That said, "maintenance burden" is partly guesswork. Why don't you try it @cpranav93? See how far you get and then we have an idea of how feasible it is.
In fact, I think this could be a nice little project to work out together with a few other RSEs. If you would like to do it like that, I will announce in a LLT on 23 November a new periodic activity focused on software quality. This idea would be a perfect candidate for that... Sorry for the vagueness, DM me if you want more details.

If someone wants to pick this up, feel free to reopen.


Ensures empty todo directories are deleted

Relates to https://github.com/ember-template-lint/ember-template-lint/issues/1684. If a directory contains no files, we ensure it's deleted. While this does introduce some extra I/O for the removal and potential recreation of the same directory, this does remove ghost directories remaining.

Should this target next instead of master?

Should this target next instead of master?

I'm going to go ahead and merge this. We can fix forward if we think we can optimize the deletion.

I'm going to go ahead and merge this. We can fix forward if we think we can optimize the deletion.


chore: Use node: protocol in imports

The official docs have used the prefix in examples for years.
According to the NodeJS docs for Modules (§Core Modules), the node: “protocol” was added in:

NodeJS v16.0.0 (initial release 2021-04-20; source) and
(probablypossibly experimentally…?) v14.18.0 (released 2021-09-08; source)

Personally, I like how it makes it easy to distinguish:

3rd party modules (no prefix) from
core modules (node: prefix) and
local modules (./ or ../ prefix).

Okay, thanks! 🙏🏽


Bai 17

Hi,
Your homework 17 has been reviewed.
Great! Keep up the good work!
Best wishes from Automation Test Pro,
Khanh


DirectSecp256k1HdWallet.fromMnemonic in Cosmos SDK (Golang)

I am using DirectSecp256k1HdWallet.fromMnemonic in cosmosjd.
I need to use it in Cosmos SDK.
Help me please.

Can you please be a bit more specific about your problem? It's unclear what you need help with.

I think he wants some example of this classes DirectSecp256k1HdWallet to generate account address from mnemonic.
I need that too. Can you give me some? Thank a lot!

Thanks, my problem is solved.


Porsche Unleashed - Crash while loading Schwarzwald

I've had this crash consistently across different hardware with Verok's patch, and while it doesn't happen on an unpatched version of the game using WINE on Linux (while Verok's patch on Linux exhibits the same issue), I'd like to figure this out so I can play this game on Windows finally. I was hoping that the Silent Patch would fix it somehow, but it doesn't. The last I looked into this using a debugger, it seems to be crashing like loading one of the "forest.*" files, but IIRC swapping the sky box like some fixes for other levels didn't work.
Do you have any ideas on how to debug this? Thanks!

I'm not sure how you could debug this, unfortunately. The only idea I'd have is to try playing around with different graphical settings, perhaps some specific setting triggers/solves that.


Add support for @LinkingObjects

Similar to Realm Java, we should add support for mapping inverse relationships. The Java annotation of @LinkingObjects seem fine for this.
class Dog : RealmObject {
  var name: String = "Fido"
  @LinkingObjects("dog")
  val parents: RealmResult<Person> = RealmResults.empty()
}

class Person : RealmObject {
  var name: String = "Jane"
  var dog: Dog? = null
}

In Java, the LinkingObjects implementation was highly annoying in Kotlin as it wasn't possible for end users to construct RealmResults themselves which meant that they were forced to do var parents: RealmResults<Person>? = null
We should definitely lift that restriction in the Kotlin SDK. Suggestion would be provide a RealmResults.empty() factory method that basically emulate the empty Results from Core, i.e. it always contains 0 elements. Semantics around queries, adding changelisteners etc. needs to be defined.

Any update on this? We have a database with ~500 relationships we'd like to model. If I can help let me know.

Hi @CarsonRedeye . No update yet...it is on our short-term roadmap to implement, but we don't have any updates right now.

Throwing some API ideas in here, since I know @nhachicha had some ideas as well
a) Similar to Realm Java
@LinkingObjects("child")
val parents: RealmResults<Person> = RealmResults.empty()

The only difference is we need to expose the possibility to create an empty RealmResults. This should be rather easy though with only a few caveats around how to handle RealmResults.query() and RealmResults.version() when calling parents on the unmanaged object
b) Use delegation
val parents: RealmResults<Person> by LinkingObjects("child")

Maybe more Kotlin idiomatic? But I'm not sure we can validate the input at compile time as we do for annotations? This has the advantage we can keep the empty RealmResults as an internal implementation that doesn't have to leak into the public API
Open Questions:

[ ] Choose between A or B...or something else
[ ] Determine semantics for unmanaged objects. Either throw on all methods or try to mirror a RealmResults as closely as possible?

Still no news about this? We'd like to migrate from java to kotlin but we are missing that

Throwing some API ideas in here, since I know @nhachicha had some ideas as well
a) Similar to Realm Java
@LinkingObjects("child")
val parents: RealmResults<Person> = RealmResults.empty()

The only difference is we need to expose the possibility to create an empty RealmResults. This should be rather easy though with only a few caveats around how to handle RealmResults.query() and RealmResults.version() when calling parents on the unmanaged object
b) Use delegation
val parents: RealmResults<Person> by LinkingObjects("child")

Maybe more Kotlin idiomatic? But I'm not sure we can validate the input at compile time as we do for annotations? This has the advantage we can keep the empty RealmResults as an internal implementation that doesn't have to leak into the public API
Open Questions:

[ ]  Choose between A or B...or something else
[ ]  Determine semantics for unmanaged objects. Either throw on all methods or try to mirror a RealmResults as closely as possible?

@cmelchior

I vote the solution with the delegation, it's more a kotlin approach, I don't see how it can cause a problem but we can try anyway. 😅
And for the unmanaged objects, the fact to return a realmResult as long as possible I don't see much interest since in the end we want to query only managed objects

Hi @sirambd Thank you for the feedback. We just picked this up, and that is also the direction we are leaning in for the reasons you mentioned.

So, is there a date for it to be released?

Hi @ganfra We just started the implementation of this, so hopefully we have something to share soon.

@cmelchior Another question, in Realm java you can create a query from a RealmList, I checked the doc and I didn't see how I could do it with Realm kotlin.
Is it a choice of yours not to have it here and just have the LinkingObject ?

@sirambd No, that is an oversight. It will be added.


Add MIT License

The MIT License is nice because it is simple and permissive - it lets anyone use your code, but also protects you by not making you liable if the code causes someone problems.
I put your name (from github profile) in the copyright section.
See more: https://en.wikipedia.org/wiki/MIT_License or http://choosealicense.com

Hi @Dania02525, did you have a moment to look this over?


jitsi-videobridge: 2.2-45-ge8b20f06 -> 2.2-79-gf6426ea0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jitsi-videobridge is: "A WebRTC compatible video router"
meta.homepage for jitsi-videobridge is: "https://github.com/jitsi/jitsi-videobridge"
meta.changelog for jitsi-videobridge is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2-61-g98c9f868 with grep in /nix/store/fgn7scy5p0wmc6ycx4p5nlw4bzzvsnqj-jitsi-videobridge2-2.2-61-g98c9f868
found 2.2-61-g98c9f868 in filename of file in /nix/store/fgn7scy5p0wmc6ycx4p5nlw4bzzvsnqj-jitsi-videobridge2-2.2-61-g98c9f868

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jitsi-videobridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fgn7scy5p0wmc6ycx4p5nlw4bzzvsnqj-jitsi-videobridge2-2.2-61-g98c9f868 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fgn7scy5p0wmc6ycx4p5nlw4bzzvsnqj-jitsi-videobridge2-2.2-61-g98c9f868
ls -la /nix/store/fgn7scy5p0wmc6ycx4p5nlw4bzzvsnqj-jitsi-videobridge2-2.2-61-g98c9f868/bin

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

1 package built:

jitsi-videobridge

Maintainer pings
cc @cleeyv @ryantm @yu-re-ka for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jitsi-videobridge is: "A WebRTC compatible video router"
meta.homepage for jitsi-videobridge is: "https://github.com/jitsi/jitsi-videobridge"
meta.changelog for jitsi-videobridge is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2-63-g252d14bc with grep in /nix/store/pm17qcpz2k6sg7s2sgicapl6sqa08nvf-jitsi-videobridge2-2.2-63-g252d14bc
found 2.2-63-g252d14bc in filename of file in /nix/store/pm17qcpz2k6sg7s2sgicapl6sqa08nvf-jitsi-videobridge2-2.2-63-g252d14bc

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jitsi-videobridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pm17qcpz2k6sg7s2sgicapl6sqa08nvf-jitsi-videobridge2-2.2-63-g252d14bc \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pm17qcpz2k6sg7s2sgicapl6sqa08nvf-jitsi-videobridge2-2.2-63-g252d14bc
ls -la /nix/store/pm17qcpz2k6sg7s2sgicapl6sqa08nvf-jitsi-videobridge2-2.2-63-g252d14bc/bin

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

1 package built:

jitsi-videobridge

Maintainer pings
cc @cleeyv @ryantm @yu-re-ka for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jitsi-videobridge is: A WebRTC compatible video router
meta.homepage for jitsi-videobridge is: https://github.com/jitsi/jitsi-videobridge
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2-67-gc7f2b2d5 with grep in /nix/store/lzw1dskkhfrrx3a6zi17qhsnja7clvvz-jitsi-videobridge2-2.2-67-gc7f2b2d5
found 2.2-67-gc7f2b2d5 in filename of file in /nix/store/lzw1dskkhfrrx3a6zi17qhsnja7clvvz-jitsi-videobridge2-2.2-67-gc7f2b2d5

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jitsi-videobridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lzw1dskkhfrrx3a6zi17qhsnja7clvvz-jitsi-videobridge2-2.2-67-gc7f2b2d5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lzw1dskkhfrrx3a6zi17qhsnja7clvvz-jitsi-videobridge2-2.2-67-gc7f2b2d5
ls -la /nix/store/lzw1dskkhfrrx3a6zi17qhsnja7clvvz-jitsi-videobridge2-2.2-67-gc7f2b2d5/bin

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

1 package built:

jitsi-videobridge

Maintainer pings
cc @cleeyv @ryantm @yu-re-ka for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jitsi-videobridge is: A WebRTC compatible video router
meta.homepage for jitsi-videobridge is: https://github.com/jitsi/jitsi-videobridge
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2-69-gad606ca2 with grep in /nix/store/yd4vxxxm0r27m6qwacg15g9y1h3pyvin-jitsi-videobridge2-2.2-69-gad606ca2
found 2.2-69-gad606ca2 in filename of file in /nix/store/yd4vxxxm0r27m6qwacg15g9y1h3pyvin-jitsi-videobridge2-2.2-69-gad606ca2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jitsi-videobridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yd4vxxxm0r27m6qwacg15g9y1h3pyvin-jitsi-videobridge2-2.2-69-gad606ca2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yd4vxxxm0r27m6qwacg15g9y1h3pyvin-jitsi-videobridge2-2.2-69-gad606ca2
ls -la /nix/store/yd4vxxxm0r27m6qwacg15g9y1h3pyvin-jitsi-videobridge2-2.2-69-gad606ca2/bin

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

1 package built:

jitsi-videobridge

Maintainer pings
cc @cleeyv @ryantm @yu-re-ka for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jitsi-videobridge is: A WebRTC compatible video router
meta.homepage for jitsi-videobridge is: https://github.com/jitsi/jitsi-videobridge
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2-79-gf6426ea0 with grep in /nix/store/gkz38fmpva7vmwgmlxxmr1y98d86n6rv-jitsi-videobridge2-2.2-79-gf6426ea0
found 2.2-79-gf6426ea0 in filename of file in /nix/store/gkz38fmpva7vmwgmlxxmr1y98d86n6rv-jitsi-videobridge2-2.2-79-gf6426ea0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jitsi-videobridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gkz38fmpva7vmwgmlxxmr1y98d86n6rv-jitsi-videobridge2-2.2-79-gf6426ea0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gkz38fmpva7vmwgmlxxmr1y98d86n6rv-jitsi-videobridge2-2.2-79-gf6426ea0
ls -la /nix/store/gkz38fmpva7vmwgmlxxmr1y98d86n6rv-jitsi-videobridge2-2.2-79-gf6426ea0/bin

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

1 package built:

jitsi-videobridge

Maintainer pings
cc @cleeyv @ryantm @yu-re-ka for testing.


prototype: delayed precommit

This PR forms the first step towards https://github.com/celestiaorg/celestia-core/issues/1416

A couple of tests are failing, and I am investigating them before running a network test to ensure there are no serious issues with the consensus in this modification.

Closing this as I am not actively working on this branch.


Use jupyterlab instead of jupyter notebook?

This had been suggested earlier (Harry?) and I dismissed on the basis that undo cell delete and undo cell edit didn't work. It turns out that it does both operations
Undo cell delete:
Esc (to leave edit mode) Z
Undo cell edit:
Ctrl+Z
Or use the Edit menu for both
Jupyter lab supports console and editor py files, which would be convenient for the exercise on writing a module, and is the way most people are used to writing python. It also does everything else notebooks do. So ... should we use Jupyterlab instead or it it too late?
I have updated the environment.yml just in case and anyone that want to install it just has to:
conda install jupyterlab
or
mamba install jupyterlab

Too late this year I think but can consider for next year? Wasn't me that suggested it - I've never used it


Editing a message with an @-mention of a group removes the message from "@ Mentions"

This can happen in 2 ways but only applies to @-mentions of "User Groups" (i.e. "Users" function correctly)
Method 1:

User writes a message with an @-mention.
User edits the message to fix a typo.
The group that was @-mentioned doesn't see the message highlighted and it no longer shows up in the "@ Mentions" menu.

Method 2:

User write a message without an @-mention.
User edits the message adding the @-mention.
Nobody is notified of this message.  It's not highlighted and it's not in the @-Mentions menu.

Here's an example of the highlighting:

And this is what I see in my "@-mentions" menu

This issue no longer reproduces, so we must have fixed it somewhere along the way.


Bug: action with command that exits with errorcode ! 0 is reported as OK

Description
While trying to use cp to copy a file to /etc (since file.copy doesn't support privilege elevation at this point) comtrya reports a completed execution, no errors.
If you run it in trace mode however, you see that the command failed with exit code 1,
Steps to reproduce
actions:
  - action: command.run
    command: cp
    args:
      - -f
      - manifests/tools/zsh/files/zshenv
      - /etc/zsh/

The above action tries to copy a file into /etc without sudo.
Trace will show:
TRACE execute:{manifest="manifests.etc"}:{action=command.run}: exit code: 1
TRACE execute:{manifest="manifests.etc"}:{action=command.run}: stdout:
TRACE execute:{manifest="manifests.etc"}:{action=command.run}: stderr: cp: cannot remove '/etc/zsh/zshenv': Permission denied
INFO execute:{manifest="manifests.etc"}: Completed

Expected behaviour
Visible error
Environment

OS: manjaro
Version: 0.8.3

btw there's a proper file copy and directory copy action that needs more than just me complaining about :>

Well aware, but since the privileged/sudo was not documented I was trying it this way (hence /etc as target).
I have since moved on to using copy.
Original remark stands though, the exit code should be evaluated.

@Blizzke I'll fix the error propagation, no problem.
@airtonix I'm not really sure how we could support privilege elevation for file.* actions, as a running process cannot elevate its privilege. Instead, you'd need to run sudo comtrya ...
In-order for us to elevate privileges, we'd need to shell out to execute commands on the host and I don't feel like that's a very good solution.
I'd really love to learn more about sudo comtrya vs comtrya and how you're using the tool.

@Blizzke PR submitted with a fix


3141- fixing the why section to include more descriptive

3141

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Adi Polak seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks! Updated according to suggestions :)


UPTM-13 - Dashboard Display on vertical 4K screen

Here is the GitStart Ticket for this pull request: UPTM-13

Please mark your PR as "ready for review" when you have completed all requirements.
Please ensure that your PR is reviewed by at least 2 people after you have completed it.
Comments for review or requested changes from the upstream client will be notified to you by email and will also show on the ticket page.
After the PR is merged on the upstream, please merge this PR.
If there exist merge conflicts on your current PR and you have to close the PR instead of merging it, please go to the ticket page to manually finish the task that is related to this PR.

Loom video
https://www.loom.com/share/ef13e3e6a9b34f7d84f568ec1d1fdd6c
Happy hacking! 🎉

This code was written and reviewed by GitStart Community. Growing great engineers, one PR at a time.

Why they merge the remote changes to this pull?

Please read my last comment: https://github.com/louislam/uptime-kuma/pull/3353#issuecomment-1635595265

This implementation is actually a bit more reasonable, but again the base branch is wrong...

This implementation is actually a bit more reasonable, but again the base branch is wrong...

Is it new? I thought it is the same pr as #3353. I didn't check as there are too many file changes.

This implementation is actually a bit more reasonable, but again the base branch is wrong...

Is it new? I thought it is the same pr as #3353. I didn't check as there are too many file changes.

Yeah this is new PR instead of making a branch and then make a PR they simply make the pr from the master branch of their fork.

Just check the video, it is actually better, but yes, it needs to be rebased.

noted. Rebased @louislam

Just check the video, it is actually better, but yes, it needs to be rebased.

noted. Rebased @louislam

I don't think so check the changes again

Honestly I don't know why these commits are here. They are supposed to be in the master branch already. Maybe somethings messed up in your branch (UPTM-13).
My suggestion is that you should give up this branch and open a new branch that based on master. Then copy and paste your code to this new branch and commit/push to here again.

Honestly I don't know why these commits are here. They are supposed to be in the master branch already. Maybe somethings messed up in your branch (UPTM-13).
My suggestion is that you should give up this branch and open a new branch that based on master. Then copy and paste your code to this new branch and commit/push to here again.

Exactly


Gatsby page down?

Hi,
Not sure if you're aware but for some reason your Gatsby starter page is not being found: https://www.gatsbyjs.com/starters/konstantinmuenster/gatsby-starter-portfolio-minimal
Excellent work by the way.

Hey! Thanks for mentioning it! Apparently, they changed their Starter library requirements. Since this repository here is deprecated, I won't update this starter project. But I will update it accordingly in the new Portfolio Minimal project: https://github.com/konstantinmuenster/gatsby-starter-portfolio-minimal-theme


popovers: Convert invite modal to tabbed format.

This PR replaces the radio buttons in invite user modal with tab component.
CZO discussion
Fixes #29392

Screenshots and screen captures

When the user is not allowed to generate invite links

When the user is not allowed to send email invitations

On very narrow widths:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks! I didn't see a purpose to the new discussion thread you started (there was no question for discussion posted, which might have merited a separate topic), so I moved your message to the original thread for this issue, and changed the link your PR description.

Thanks! In terms of the UI:

There is too little space above the Emails field. (Compare with spacing above the "Invitation expires" field.)
For disabled buttons, let's center the tooltip on top of the button, not in the middle.

What happens if the tab label doesn't fit on one line (in a narrow window / another language)?

What happens if the tab label doesn't fit on one line (in a narrow window / another language)?

I've attached screenshots in the PR description for when the screen is very narrow.
The component word-wraps the label text. If an individual word is too long, it becomes problematic.
This concern would be applicable to all the places this component is currently being used.

Ready for review!

Besides that, looks good to me in manual testing! @sahil839 are you up for reviewing this one?

I've implemented the suggestions as per the review and responded to the comments. The PR is ready for another review.

@timabbott ready for your review.

This looks great, merged, thanks @kuv2707!


CC: Edit org and user profile

Design doc: https://docs.google.com/document/d/1NiCzhuq3hN-t7IhRa4jP562ne3OZrhfdIsNs8OT9UlI/edit?usp=sharing
GH issue: https://github.com/cockroachlabs/managed-service/issues/1978

Closing in favor of #7358


Separate style and typing test runs from functional test runs in GitHub workflow

It's a bit inefficient not to be able to see what's going on with functional tests when style tests fail. They could be broken into their own jobs or workflows. We might separate style and type tests too, for that matter.

Also, enable linting (black) in the prehooks.