BHT-8000

I have a BHT-8000 , and this one is not reconised as climate
Is this supported by this integration ?!

Hi,
the log messages from when you try to add the device to Home Assistant through this integration
Cette erreur provient d'une intégration personnalisée

Logger: custom_components.tuya_local.device
Source: custom_components/tuya_local/device.py:236
Integration: tuya_local (documentation, issues)
First occurred: 23:17:34 (1 occurrences)
Last logged: 23:17:34

Failed to refresh device state for Test.

device details
{
  "result": {
    "active_time":<PHONE_NUMBER>,
    "biz_type": 18,
    "category": "wk",
    "create_time":<PHONE_NUMBER>,
    "icon": "smart/icon/15264583415nnlzwfs1qd2wqz6i937op3nmi_0.png",
    "id": "xxxxxxxx",
    "ip": "xxxxxx",
    "lat": "xxxxx",
    "local_key": "xxxxx",
    "lon": "xxxx",
    "model": "",
    "name": "Thermostat_LCD",
    "online": true,
    "owner_id": "2694xxx",
    "product_id": "IAYz2WK1th0cMLmL",
    "product_name": "thermostat",
    "status": [
      {
        "code": "switch",
        "value": true
      },
      {
        "code": "temp_set",
        "value": 46
      },
      {
        "code": "upper_temp",
        "value": 46
      },
      {
        "code": "eco",
        "value": false
      },
      {
        "code": "child_lock",
        "value": false
      }
    ],
    "sub": false,
    "time_zone": "+01:00",
    "uid": "xxxxxxxx",
    "update_time":<PHONE_NUMBER>,
    "uuid": "xxxxxxxx"
  },
  "success": true,
  "t":<PHONE_NUMBER>988
}

Hi,
Have you take a look to the new chenagelog version ?
Are you think that this change (https://www.home-assistant.io/blog/2021/11/03/release-202111/#entity-categorization) can help us to get the on/off option on the BHT-8000 separated from the Light entity ?
Thanks

Good !
Your recent modification work !
I will try some automations to test it to night !
Thanks a lot for the good job !


Qt creator qml edit autocompletion

Can you separate in different pull request example modification and QtCreator related code ?

@OlivierLDff i revert qml modification one


IPv6 from mDNS

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/yarl/_url.py", line 166, in __new__
    port = val.port
  File "/usr/local/lib/python3.10/urllib/parse.py", line 178, in port
    raise ValueError(message) from None
    ValueError: Port could not be cast to integer value as ':c6de:e2ff:fe11:1cac:8081'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/aiohttp/client.py", line 422, in _request
    url = self._build_url(str_or_url)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/client.py", line 356, in _build_url
    url = URL(str_or_url)
  File "/usr/local/lib/python3.10/site-packages/yarl/_url.py", line 168, in __new__
    raise ValueError(
    ValueError: Invalid URL: port can't be converted to integer

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/config/custom_components/sonoff/core/ewelink/local.py", line 257, in send
    r = await self.session.post(
  File "/usr/local/lib/python3.10/site-packages/aiohttp/client.py", line 424, in _request
    raise InvalidURL(str_or_url) from e
    aiohttp.client_exceptions.InvalidURL: http://fe80::c6de:e2ff:fe11:1cac:8081/zeroconf/info

from this https://github.com/AlexxIT/SonoffLAN/issues/1090

https://github.com/AlexxIT/SonoffLAN/releases/tag/v3.4.0


[WIP][SYSTEMML-1743] Create lightweight SystemML artifact

Create a lightweight SystemML jar file based on the execution of
a particular set of code, such as JMLC. In addition to generating
a minimal jar file that corresponds to this code, automatically
generate maven assembly dependency sets that can be used to build
a 'lite' jar that can be activated via a 'lite' maven profile.
This essentially creates an minimum-size uberjar that can perform
tasks such as scoring with JMLC, where distributed operations on
Spark and Hadoop are not desired. Note that this potentially addresses
the underlying requirements of situations such as that needed by
[SYSTEMML-453].

Lightweight jar can be built using:
mvn clean package -P distribution,lite

@mboehm7 Please let me know what you think of this approach and if other examples (such as those in BuildLiteExecution.java) should be supported.

Also, note that the artifact created by the maven assembly includes all log4j classes, commons-math3 classes, and SystemML classes.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/1771/

Regarding log4j, it would be great if we could also package a default log4j.properties into this jar; otherwise users won't see warnings from the compiler, etc or would need to include a custom log4j.properties into their classpath. Furthermore, I ran into this error (which, however, did not raise an exception):
java.lang.ClassNotFoundException: org.apache.hadoop.log.metrics.EventCounter         at java.net.URLClassLoader.findClass(URLClassLoader.java:381)         at java.lang.ClassLoader.loadClass(ClassLoader.java:424)         at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:335)         at java.lang.ClassLoader.loadClass(ClassLoader.java:357)         at java.lang.Class.forName0(Native Method)         at java.lang.Class.forName(Class.java:264)         at org.apache.log4j.helpers.Loader.loadClass(Loader.java:179)         at org.apache.log4j.helpers.OptionConverter.instantiateByClassName(OptionConverter.java:320)         at org.apache.log4j.helpers.OptionConverter.instantiateByKey(OptionConverter.java:121)         at org.apache.log4j.PropertyConfigurator.parseAppender(PropertyConfigurator.java:664)         at org.apache.log4j.PropertyConfigurator.parseCategory(PropertyConfigurator.java:647)         at org.apache.log4j.PropertyConfigurator.configureRootCategory(PropertyConfigurator.java:544)         at org.apache.log4j.PropertyConfigurator.doConfigure(PropertyConfigurator.java:440)         at org.apache.log4j.PropertyConfigurator.doConfigure(PropertyConfigurator.java:476)         at org.apache.log4j.helpers.OptionConverter.selectAndConfigure(OptionConverter.java:471)         at org.apache.log4j.LogManager.<clinit>(LogManager.java:125)         at org.slf4j.impl.Log4jLoggerFactory.getLogger(Log4jLoggerFactory.java:66)         at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:270)         at org.apache.commons.logging.impl.SLF4JLogFactory.getInstance(SLF4JLogFactory.java:155)         at org.apache.commons.logging.LogFactory.getLog(LogFactory.java:290)         at org.apache.sysml.hops.OptimizerUtils.<clinit>(OptimizerUtils.java:60)         at org.apache.sysml.api.DMLScript$DMLOptions.<init>(DMLScript.java:129)         at org.apache.sysml.api.DMLScript$DMLOptions.<clinit>(DMLScript.java:138)         at org.apache.sysml.api.DMLScript.<clinit>(DMLScript.java:161)         at <some testclass> log4j:ERROR Could not instantiate appender named "EventCounter"

List of updates:
Shrink lite.xml size by over 50%.
Add MRJobConfiguration.USE_BINARYBLOCK_SERIALIZATION check.
Switch isLite/remotePar checks in InfrastructureAnalyzer.
Update BuildLite IO to use try-with-resources.
Remove commented code in BuildLite.
Switch displayMatrix to use DataConverter.
Update LiteCheck isLite to use try-with-resources.
Cache isLite results for performance.
Add default log4j.properties to root of artifact.
Add EventCounter class to lite artifact.
Exclude lite jar from python artifact if built at same time.
Generate entire dependencySets section from BuildLite.
Automatically update lite.xml dependencySets if lite.xml is found.
Allow commons-math3 option for all or only required classes.
Switch BuildLiteExecution output to utilize log4j.

Note that I tried switching to MRJobConfiguration.addBinaryBlockSerializationFramework(_rJob) but with this minimal lite jar I saw Could not find a serializer for the Key class: 'org.apache.sysml.runtime.matrix.data.MatrixIndexes', so I kept the existing MRConfigurationNames.IO_SERIALIZATIONS call within the USE_BINARYBLOCK_SERIALIZATION check.
By handling the lite.xml differently for commons-logging and commons-math3, I was able to shrink the size of lite.xml by over 50%. WRT switching to calling BuildLiteExecution from the lite profile, I would say artifact maintentance would become very difficult (WRT issues such as licensing, included files, etc) since it would mean going from essentially our project's standard maven assembly technique for building our verified artifacts to a fairly convoluted programmatic technique. I think the lite.xml file used with the maven-assembly-plugin helps standardize how the artifact is built, which makes our release validation process easier. A side benefit of decoupling the actual BuildLiteExecution execution from the lite profile means that the artifact takes a few seconds to build rather than a few minutes.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/1788/

@mboehm7 When you have time, please let me know what you think of the latest updates.

I'll go ahead and merge this tomorrow if there are no objections.

can you please hold off a few more days. Thanks.
Regards,
Berthold Reinwald
IBM Almaden Research Center
office:<PHONE_NUMBER>; T/L: 457 2208
e-mail<EMAIL_ADDRESS>From:   Deron Eriksson<EMAIL_ADDRESS>To:     apache/systemml<EMAIL_ADDRESS>Cc:     Subscribed<EMAIL_ADDRESS>Date:   07/31/2017 12:10 PM
Subject:        Re: [apache/systemml] [WIP][SYSTEMML-1743] Create
lightweight SystemML artifact (#578)
I'll go ahead and merge this tomorrow if there are no objections.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

ping @bertholdreinwald

ping @bertholdreinwald

sorry for the delay.
I tried it on a bare bone machine. Got below class not found. Checked the lite jar. Several org.apache.hadoop.util classes are included but GenericOptionsParser is missing. Should it be included?
Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/hadoop/util/GenericOptionsParser
at org.apache.sysml.api.DMLScript.main(DMLScript.java:230)
Caused by: java.lang.ClassNotFoundException: org.apache.hadoop.util.GenericOptionsParser
at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:335)
at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
... 1 more

@bertholdreinwald Thanks for having a look at this. It looks like you're running SystemML using DMLScript (at org.apache.sysml.api.DMLScript.main(DMLScript.java:230)) rather than using the JMLC API, which is the focus of the lite jar. Please try with the JMLC API.

Rebased with latest code. Also updated l2svm predict input param due to a change introduced in the last month.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2030/

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2099/


Finished assigment 3

Good except you didn't do the exercise in the file

What are you talking about?
Sent from my Sprint Samsung Galaxy S7.
-------- Original message --------
From: cengique<EMAIL_ADDRESS>Date: 2/12/17 2:55 PM (GMT-05:00)
To: cengique/bash_basics<EMAIL_ADDRESS>Cc: Michael Holtmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [cengique/bash_basics] Finished assigment 3 (#99)
Good except you didn't do the exercise in the file
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/cengique/bash_basics/pull/99#issuecomment-279243634, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVl2OceHXj5UyJRVdoSGxZaOI1FoJH04ks5rb2PKgaJpZM4L3NYn.


Added multicast to ViewStore publisher to reduce equality checks

I don't think there is too much overhead using multicast and creating an additional CurrentValueSubject. It might be possible to implement similar behaviour inside StorePublisher as an alternative though.

Could consider storing the CurrentValueSubject inside the ViewStore and just call objectWillChange inside the sink...
public init(
    _ store: Store<State, Action>,
    removeDuplicates isDuplicate: @escaping (State, State) -> Bool
  ) {
    self._state = CurrentValueSubject(store.state.value)
    let publisher = store.state
      .removeDuplicates(by: isDuplicate)
      .multicast(subject: self._state)
      .autoconnect()

    self.publisher = StorePublisher(publisher)
    self._send = store.send
    self.viewCancellable = publisher
      .sink { [weak self] _ in self?.objectWillChange.send() }
  }

  /// The current state.
  public var state: State {
    self._state.value
  }

@iampatbrown Thanks for this! @mbrandonw and I took inspiration from #618 and wrote a very naive benchmark that showed that multicasting with a new subject does add a bit of overhead. We'd like to write a beefier benchmark with state that's a bit more real-world, though, to weigh this overhead against the equality check it's preventing. We're going to keep this open while we investigate further, but wanted to let you know where we're at 😄

Great idea :) Was looking at that discussion as well. I'll definitely investigate further using the benchmark branch.

@stephencelis and @mbrandonw storing the state in a CurrentValueSubject and forwarding changes seems to work better on my end.

@iampatbrown Looks like it! We're seeing a much more minimal cost with those changes. We'd like to flesh out some benchmarks before merging this, but assume that it will be a favorable change :smile: Thanks again for taking the time to dig into the problem and submit this PR!

No worries. Thanks for pointing me in the right direction.
I noticed with CurrentValueSubject state.send(newValue) benchmarked better than state.value = newValue and also guard let self = self worked better than self?. when weak self was referenced more than once inside a closure. It was fairly insignificant but interesting to learn more about.

@stephencelis Okay for me to close this one? Pretty straight forward for you guys to implement on your end when ready.

We're happy to leave it open and merge when we've weighed the performance. If being open is getting in the way for you, let us know. We can track and link to it in another issue.

Cool. We can leave it open. I'll just fix up the formatting. At some point I would have used swift format which made a change to the convenience initialisers.

@iampatbrown No need to worry about formatting 😄 We automatically run swift-format on main when branches are merged.

Hi @iampatbrown, I just merged main into this branch and did a small clean up pass here https://github.com/pointfreeco/swift-composable-architecture/pull/624/commits/48845a9bc0edc17209118ab6f53ec8b54b6a6d37. We will be merging soon!


API change

Just letting you know that API version 1 became unavailable (at least here in NL), so I did some research, and here are the updated endpoints:

Ticket list
Old: https://tickets.lidlplus.com/api/v1/NL/tickets/list
New: https://tickets.lidlplus.com/api/v2/NL/tickets
You can use the query parameter pageNumber to select the page of the paginated response, and onlyFavorite (true/false) to filter by favourite status.

Ticket details
Old: https://tickets.lidlplus.com/api/v1/NL/tickets/{id}
New: https://tickets.lidlplus.com/api/v2/NL/tickets/{id}

Response data structure also slightly changed, for example records in the list response is now tickets instead.
Also, methods for these aren't currently implemented in your library, but you might be interested in them:

Mark ticket (receipt) as favourite:
POST https://tickets.lidlplus.com/api/v2/NL/tickets/favorite, with JSON body ["ticketidhere"]
Mark ticket (receipt) as not favourite:
POST https://tickets.lidlplus.com/api/v2/NL/tickets/unfavorite, with JSON body ["ticketidhere"]
Coupon list: https://coupons.lidlplus.com/api/v2/NL
Coupon details: https://coupons.lidlplus.com/api/v2/NL/{id}
Coupon activation: POST (activate) or DELETE (deactivate) to https://coupons.lidlplus.com/api/v1/NL/{id}/activation

There's a bunch more endpoints for announcements, store details, shopping lists, etc, I'll document them all at some point, but hopefully the above will save you some trouble fixing the current methods in your library.

Hey, thanks for reporting and your research! 🙂
I'm on vacation until mid of next week. I will have a look then, or feel free to make a pr.

I don't actually have a use for this library myself, I just had to figure out the new API anyway for my own project, and wanted to share my findings with others who might be interested - so feel free to use or not use this info whenever you like :)
Enjoy your vacation!

Thank you for the quick fix!
A note for other users: The keys in the resulting json changed for example an item now looks like this:
      "currentUnitPrice": "2,29",
      "quantity": "1",
      "isWeight": false,
      "originalAmount": "2,29",
      "name": "Mozarelli",
      "taxGroupName": "A",
      "codeInput": "4056489442431",
      "discounts": [],
      "deposit": null,
      "giftSerialNumber": null

I merged it and published a new release, thanks guys! 👍
@ahormann At first glance it looks the same, can you point out what changed? Pr of the readme are also welcome 🙂

The description was renamed to name and the extendedAmount seems to be gone.

The description was renamed to name and the extendedAmount seems to be gone.

Thanks, I updated the readme :+1:


msg.sender address in initial state

Currenty, the KEVM configuration is initialized with a symbolic CALLER_ID instead of the address that Foundry uses as msg.sender. Perhaps we should include that account in the initial account state and set the concrete message sender.

Here is where in the code the msg.sender and msg.origin are set: https://github.com/runtimeverification/evm-semantics/blob/8960441a7bf0b9961833a47ad72826cd17792f19/kevm-pyk/src/kevm_pyk/foundry.py#L1005

We can use the pranks to set these in the test.


[Bug]: Some initializers are not being called.

What happened?
I have been doing some tests with hyperstack and realised that the initializers, beforeMiddleware, afterMiddleware, beforeControllers and afterControllers are not being called.
Before app and after app work as expected.
I haven't tested afterLogger and beforeWorkers yet.
From what I could see in the code it seems that the issue comes from the validations of the initializers object in createServer.
https://github.com/hyperstackjs/hyperstack/blob/829c843ba3141333283fc24b1b6b50ae37ab1d60/packages/hyperstack/src/controllers.ts#L22-L61
The initializers object has a key initializers inside.
https://github.com/hyperstackjs/hyperstack/blob/829c843ba3141333283fc24b1b6b50ae37ab1d60/packages/hyperstack/src/initializers.ts#L47-L49
I can make a pull request with a fix for it if you want.
What type of Operating System?
macOS
Steps to produce this issue.
Running the kitchensink example should reproduce the issue.

Yes, that's correct!.
Seems we need to fetch the inner initializers.initializers field and hand it over:
probably something like from:
  const initializers = store.initializers

to:
  const initializers = store.initializers?.initializers

I'll be happy to accept a PR. Let me know if you need any help

@jondot fixed in this PR, I think this issue be closed


DefinitelyTyped for salesforce REST api

in DefinitelyTyped-master there are so many definition available but i want to create hybrid app for mobile using SalesForce for that we need force.d.ts

Hi @yogeshksraj! I'm not sure whether you're in the right place. This is the repo for the tool, TSD, not for general definition support. I'd have a look at the TypeScript handbook to get started writing your own definitions, as well as the definitions on https://github.com/borisyankov/DefinitelyTyped for examples, and post any questions to StackOverflow or somewhere similar where people will be equipped to answer you.


opencv files, version question

Hi,
Does the example that you sent me last time (Linux) require a client-side opencv installation, or are all the necessary opencv files included? I am getting  the following error when executing:
./Test: error while loading shared libraries: libopencv_core.so.2.4: cannot open shared object file: No such file or directory
If are to install OpenCV do we do that with 3.0+ or match the 2.4, as above.

you need to install the opencv binary library by sudo apt-get install libopencv-dev

I did that and manually checked /usr/lib and /usr/local/lib and no libopencv_core.so.2.4.
I did system search as well for libopencv_core.so.2.4 as well, with negative results.
The system did install
a/usr/lib/x86_64-linux-gnu/libopencv_core.so.3.1.0
/usr/lib/x86_64-linux-gnu/libopencv_core.so
/usr/lib/x86_64-linux-gnu/libopencv_core.so.3.1
What next?

Should I uninstall the 3.1.0 and replace with the older distro opencv Unix 2.4?
I really could use the help here, I really want to move forward here.

I think you need to recompile the source code to run the ./Test example. If you link the source code with opencv 3, it won't require 2.4 version.

Awesome, that makes sense.
What compiler/linker did you use? I plan o using LinuxGCC toolchain unless you think it better to use something else.

There is comments on top of the cpp file about how to compile the code, something like this: g++ MT9V022_demo.cpp -o Test pkg-config --cflags --libs opencv -lArduCamLib -lpthread -lusb-1.0 -L. -I. -std=gnu++11
So you need g++ to compile the code.

Things going ok recompiling source code. It appears opencv library files were not installed wioth the orignal sudo command:
you need to install the opencv binary library by sudo apt-get install libopencv-dev
I can link to header files but not link to library files - they don't exist. I' ve done a thorough search--and manually checked /usr
Should I reinstall opencv?

The documentation is quite good, thanks for checking anyway. My only problem rightn ow is my opencv libraries have literally disappeared. If I can find them I can move on.

Hi Lee, me again. A lot of progress. Can you please tell me what to fix ?
directory>./Test
`Device found = 1
ArduCam_open successful
create capture thread successfully
create display thread successfully
frame available
No protocol specified
Unable to init server: Could not connect: Connection refused
(Display Image:1991): Gtk-WARNING **: cannot open display: :0
`

I have never seen "No protocol specified" and "Unable to init server: Could not connect: Connection refused" errors, did you modified our example ?


include type syntax in imports as required by frameworks like sveltekit

importing types and interfaces in typescript projects for frameworks like svelte-kit requires the "type" syntax appended for type imports e.g
import type { LinkedField, LinkedType } from './types'
setting
 "importsNotUsedAsValues": "remove"
does not help as this is internally overriden and set to "error".
These are the points i identified in my project where these errors were being thrown

Thank you! i will create a new release soon


Build breaks with webpack 2, postcss and svg-inline

I'm submitting a bug report
Webpack version:
2.x
Please tell us about your environment:
OSX 10.x
Current behavior:
With a specific file structure and when using postcss-loader, postcss-import and svg-inline-loader, I get the following error:
Module build failed: (SystemJS) ENOENT: no such file or directory, open '/Users/karolis/webpack-2-failing-build/loader-utils'

Expected/desired behavior:
The build should work.
To reproduce

Clone https://github.com/KidkArolis/webpack-2-failing-build
Run npm install && webpack --progress

I think the bug itself is caused by postcss-import, but this only happens in webpack@2, so I thought I'll post here first and cross link to postcss-import repo. It would be nice to dive deeper and understand what's causing this issue. It only seems to be happening with this very specific file structure. Moving the .css file around for example, might cause the build to work again. Any pointers on how to debug this further would also be appreciated.

Yep, postcss-import somehow breaks webpack's module resolution. It may be a bug with webpack's SystemJS support, but for now the workaround is to pin postcss-import to 8.1.0.

Yep, postcss-import somehow breaks webpack's module resolution.

postcss-import 9 removes pkg-resolve as a dependency, the addition of which happened at version 8.1.1 (when JSPM support was "fixed").
From my experience, this 🐞 only presents in all versions of postcss-import > 8.1.0 and < 9.0.0.

An additional note that if you upgrade, you'll need to make sure to delete the pkg-resolve from your node_modules folder, otherwise you'll still see the buggy behavior. postcss-import 9.0.0 will still require that package if it's present, it just won't install it anymore if it's absent.


Adding FA image class in Navbar doesn't display icon

If configuring like below, icon isn't added.

Based on #94 findings I think you can change the image class library to use MD instead of FA. Devs won't see the list of icons in APEX however they can declaratively define them as an icon class rather than custom attributes.

Closing this for now.


Logging enhancement and screenshotting logic to improve action debugging

To improve the observability and debugging capabilities of our deployment process, I propose introducing a new environment variable named FULL_LOG. When set to true, this variable will trigger the logging of additional information during the upload process. This enhanced logging could include verification of element discovery and other pertinent details, which are yet to be precisely defined.
Moreover, a critical enhancement to our error handling strategy could involve capturing a screenshot in the event of an exception. This will provide visual context to the errors, significantly helping in their diagnosis. Below is a snippet of the implementation for capturing a screenshot upon encountering an exception:
try {
  ...............
} catch (error) {
  const screenshotPathError = '/path/to/screenshots/error.png';
  await page.screenshot({path: screenshotPathError});
  console.log(`Screenshot taken: ${screenshotPathError}`);
}

To ensure these screenshots are readily accessible for analysis, I suggest adding the following step to our GitHub Actions workflow. This step will upload the captured screenshots as artifacts, making them available for review:
- name: Upload screenshots
  uses: actions/upload-artifact@v2
  if: always()
  with:
    name: screenshots
    path: /path/to/screenshots/

Looking good


http3: reject negative values for the Content-Length header

cc @WeidiDeng 🙏

LGTM :shipit: . By the way, this should also be done for response writers, I remember stdlib uses strconv.ParseUint as well.

By the way, this should also be done for response writers, I remember stdlib uses strconv.ParseUint as well.

That's correct. With https://github.com/quic-go/quic-go/pull/3969 we'll use the same code for header parsing for requests and responses.

:shipit:


Show last updated date on documentations

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
A few doc pages on kubernetes.io are relatively stale. While we are working hard to rearrange our docs structure and improving the existing docs, it would be nice to show last updated date along with the docs, which can give users a better sense of how old the docs are.
Proposed Solution:
Show last updated date on docs.
Page to Update:
http://kubernetes.io/docs/

/remove-lifecycle stale
/lifecycle frozen

/assign


Update README.md

minor fix

@gombrowicz94
Thanks for your comments to the previous review request! Since we had to close that pull request, I'll address them here:

Users don't need to be exposed to some of the functions in the base class. I will change them to private and add a new update to this pull request.
The shape of x and y inputs and scaling required are slightly different based on the model we are using. _model() is doing this transformation and calling the correct model based on the class.  It's similar for AR and regularised regression models but different for neural networks. Since we are not repeating the code more than twice, I did not make it into a function. Let me know what you think!
I think sklearn is better than glmnet because its a more common dependency and easy to install and work with than glmnet.

changing some functions to private, thanks for the comment @gombrowicz94
option to include PCA features  to run regression models, and AR+PCR model
Improving unittests


Can't access Documents folder in iOS

Terrific package but unfortunately it appears there is no way to store files in the iOS Documents folder. This is critical because if your app needs to allow users to sync files via iTunes, the files must be placed in Documents, not Library -- an iOS restriction. Unable to use PCLStorage for this reason. A third mode after LocalStorage and RoamingStorage is needed : - for Android it would be the same as Local, for iOS it would point to Documents.

LocalStorage defaults to the Library folder with the path "<app_storage_path>/Documents/../Library"
Files can be stored in the Documents folder by adding "/../Documents" to this path.

Hey thanks. I should have thought of that!


Console Error "VueDarkMode is not defined"

Hello
When I use "VUE SOCIAL" with a CDN, I get a title error.
How should I solve it?
Please check the following Github source for the tried source code.
https://github.com/kabanoki/vue/blob/master/vuesocial/index.html

Hello @kabanoki, can you try to import Vue Dark Mode with your CDN also?
https://www.growthbunker.dev/vuedarkmode/
<script src="https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>I recommend to use the NPM package if this is a possible solution for you though.


Infer podspec version from git command

This PR aims to provide a better solution to 'current version' inference for llbuild when used in CocoaPods directly from git.
When requiring a Pod directly from git, like so:
pod 'llbuild', git: 'https://github.com/apple/swift-llbuild.git'

CocoaPods relies on the .podspec to provide the version that is then used, among other things, for dependency resolution.
SwiftPM, on the other hand, relies solely on SemVer and git tags and implies no versioning in the code itself.
The two ways are ultimately incompatible, but this new approach attempts to bridge them. The main use cases:

When requiring a pod directly from git, the version inferred will be the latest SemVer-compatible tag.
When requiring a pod from a directory, it's implied that the directory is a git repo. The same logic applies.
When pushing a podspec to Trunk with pod trunk push, the validation will be performed for the most recent tag. If it was already pushed, an error will be raised.
For a podspec that is uploaded to Trunk, the logic will not be performed - Trunk podspecs are static JSON files with immutable versions.

It finds the most recent tag on the current branch. I've tested this on several different scenarios and it seemed to work okay.


Avatar picture not being correctly updated

If a user changes its avatar, their avatar on messages are being updated, but not the popover-avatar.

@zulipbot claim


The validation images are also used in training

Hi, thank you for your great open source and quick response to my previous questions.
I am trying to train mega-nerf to reproduce the results. I have a question related to the validation images. As in the code below:
https://github.com/cmusatyalab/mega-nerf/blob/306e06cc316dd4f5c84d0610308bcbc208228fc3/mega_nerf/runner.py#L620
It seems that the validation images are also used in training. To my understanding, it is because we need to learn the appearance embeddings of the validation images. I would to like confirm this setting and want to ask whether I understand this setting correctly. Thank you very much.
Best regards,
Zhenxing Mi

Your intuition is generally correct - we use half of the validation image to learn the appearance embeddings, and then validate solely against the other half of the image.

Hi, sorry to reopen this issue. I still have a question about the optimization of appearance embeddings of the validation images.
In NeRF in the Wild, "when evaluating error metrics on test-set images we optimize $l^{(a)}$ to match the appearance of the true image using only the left half of each image". It seems they optimize the embedding for each validation image in the validation phase.
In the code of Mega-MeRF, the left half of each validation image is mixed into the training data to learn appearance embeddings of the validation images. And I didn't find codes to optimize the embedding for each validation image separately before validation. So I think the Mega-NeRF optimizes the embedding for each validation image in training phase.
I just want to make sure whether you perform extra optimization for the embedding of each validation image before evaluating error metrics showing in the tables of Mega-NeRF paper.
Thank you very much.

I'm not sure what it would mean for NeRF in the Wild to optimize the embedding during the validation phase vs the training phase, but I think that your understanding of Mega-NeRF is correct. We just add the left half of the images as training pixels and then evaluate on the held out right half.

Got it, thanks!


Show nomad job prefix in the UI

Depends on #21

Won't fix because we implemented a regex which is already shown.


Kraken open orders by pair

The Kraken API does not allow for filtering by currency pair. This change filters by desired currency pair after retrieving all open orders from Kraken.

look good, some small remarks


Authentication failed: password is incorrect or there is no user with such name.

Working curl example
echo 'SELECT version()' | curl 'https://analytics:analytics3113@***:8123/?database=analytics' --data-binary @-

Error throwing javascript example
const client = createClient({
  host: "https://***:8123",
  username: "analytics",
  password: "analytics3113",
  database: "analytics",
});

Error
error: Uncaught Error: default: Authentication failed: password is incorrect or there is no user with such name.

In both cases da domain was used with valid ssl certificate.

Hmm, that's hard to tell straight away what might be wrong here.
But the port looks a bit odd, it is usually 8443 instead of 8123 for HTTPS.
I assume this is a self-hosted configuration?

True, should be 8443, however, with 8123 / http I am getting the same error.
My endpoint http://***:8123 returns Ok.
curl
➜  ~ echo 'SELECT version()' | curl 'http://analytics:analytics3113@***:8123/?database=analytics' --data-binary @-
<IP_ADDRESS>

working javascript connection
const db = createClient({
  host: "http://analytics:analytics3113@***:8123",
  // username: "analytics",
  // password: "analytics3113",
  database: "analytics",
});

not working connection
const client = createClient({
  host: "http://***:8123",
  username: "analytics",
  password: "analytics3113",
  database: "analytics",
});

Error
error: Uncaught Error: default: Authentication failed: password is incorrect or there is no user with such name.

I'd appreciate it if you could join our community slack and share your server configuration and exact code snippets with me there. It will help with the debugging of this scenario.

Issue seems to be with Deno. I am not sure what exactly causes the problem but it doesn't seem to be a direct issue with ClickHouse javascript module.


[CI] MeshModel Component Generator and Updater workflow timeout

Current Behavior
The Component Generator workflow no longer completes successfully.
Desired Behavior

Immediately, split workflow into two parts.
Then, rewrite Component Generator.
Then, expose Components Generator in mesheryctl system model command.

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation
Meshery documentation site and source
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack
🧪 Meshery Test Plan Spreadsheet

@Revolyssup can you take care of #1?
@MUzairS15, if @Revolyssup doesn't respond, will you take over this item?

Sir @leecalcote  i want to work on this issue.

@Revolyssup can you take care of #1?
@MUzairS15, if @Revolyssup doesn't respond, will you take over this item?

Ok

I can see the runner is being cancelled (most probably due to resource exhaution).
The workflow can be broken down lexicographically on model names.
So there needs to be passed another argument which limits the number of models generated. Like [a-j], [k-p],[q,z]

@Revolyssup or split this by generator and updater.

@leecalcote my slack is causing some issue the think is the issue according to logs is in line 148 of the code and the cred for spreadsheet was not provided in my last code in generator job so just added the cred of spreadsheet can you rerun once if the issue still persists I'll look for the timeouts

line 148 of code


Bug: Qr code prematurely deleted on edit command

The qr code of a client is currently being prematurely deleted before validation checks are performed during the execution of the edit command.
To reproduce this bug, change the name of an existing client 1 to the name of another existing client 2 using the edit command.
Then, run the list command to refresh the contact list. The qr code will be reflected as empty. Additionally, a new qr code file will be created with existing client 2's name concatenated with existing client 1's number, taking up unnecessary space.
Although the qr code will be re-generated again on app launch (since the relevant info to generate the qr code such as name and phone number is not deleted), this should not be the expected outcome.
The cause of this bug lies in the premature call to personToEdit.deleteQrCode(); in the EditCommand.java file.

I think we're only considering contacts to be same if both name and phone number are the same.
The current code only checks for the same name.

I think we're only considering contacts to be same if both name and phone number are the same.
The current code only checks for the same name.

Yes, I have created #95 to address this. Meanwhile, moving personToEdit.deleteQrCode(); to after the the duplicate client check would fix the premature deletion, but not the premature creation.


Update README.md

@mkz212 would you like a beta version to test?

@ilcato I only made changes to the documentation and GitHub Issues templates, so I don't need a beta. It is rather a question to the user who added the PM25 sensor and the presence sensor.


How can I have an @arrayProp with multiple types in it (aka are discriminators supported)?

In mongoose I've used discriminators to solve this, are discriminators supported in Typegoose at all?
I have a property with multiple possible types, how should I properly declare this so it resolves the right types?
class Example {
  @prop({ required: true })
  public kind!: string;
}

class Foo extends Example {
  public kind = "foo";
  ...
}

class Bar extends Example {
  public kind = "bar";
  ...
}

class Data extends Typegoose {
  @arrayProp({ items: ?? })
  public examples!: Example[];
}

When I fetch a Data I need to be able to spin through the examples, look at the kind and then cast it into the appropriate type so I can get the fields / behaviors off of it. How can I tell Typegoose the various item types that are available for this purpose?

I think the answer is that you can't, discriminators are not supported by typegoose.


Move react and react-dom to peerDependencies

Hi, it looks like having react and react-dom in dependencies can cause conflict with other packages with Next.js (in my case Persona). Moving react and react-dom to peerDependencies instead seems to fix the issue. Would it be possible to make that change?
  "peerDependencies": {
    "react": ">= 16",
    "react-dom": ">= 16"
  },
  "dependencies": {
    "detect-browser": "^5.1.0",
    "prop-types": "^15.7.2",
    "styled-components": "^5.3.3",
    "tslib": "^1.10.0"
  }

With stable version 2.0.0 of Web3Modal now released, we are officially dropping support for version 1.x
Due to this this issue/pr was marked for closing. It is highly recommended to upgrade as 2.x will be receiving further updates that will enable functionality for some of our newer sdks like auth and push as well as support for WalletConnect v2 (See this post about WalletConnect v1 being deprecated https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e)
If you need to continue using Web3Modal 1.x and require this feature/fix implemented, we suggest adding it via forking V1 branch.


Add --no-install-* to add/remove/run and add env vars

Some options were added in https://github.com/astral-sh/uv/issues/4028 to avoid the installation of some packages, including the project itself. Taking --no-install-project as an example, it works great when running uv sync --no-install-project, but as soon as we want to run a script/entrypoint through uv run later on, we'll still end up installing the project at some point.
For instance, if we want to build documentation for a project, running uv run mkdocs build will install the project itself, which could take some time if the project is using Rust.
Current workaround I found is to use uv run --no-project mkdocs build to avoid project discovery (assuming that uv sync --no-install-project was run beforehand), but I wonder if the 3 options added to uv sync should not also:

be added to other commands that call do_sync (add, remove, run)
be configurable via environment variables (this could be really useful on CI workflows)

Can you share more about the use-case here? We added --no-install for Docker image building and layer caching. It's not intended for you to maintain a partial environment long-term.
I'm not necessarily opposed, but I think we should be careful here — we moved fast to deliver that feature because it's critical for Docker workflows but I'm not sure I understand this use-case as much.

I also need this feature.  Our use case is that we frequently not creating python packages, but we are instead defining the dependencies using UV that are packaged alongside our source code which gets deployed into lambdas as part of serverless infrastructure.  The .egg-info file that gets created with each script that gets deployed is unnecessary, especially since the various lambdas that compose the service will each independently define their own entry points within the same src folder.
I'd also prefer to have the ability to set no_install_project = true in the pyproject.toml so that the project itself is never added regardless of uv command.

I don't think you're looking for --no-install-project. I mean, it might work for that use-case, but that isn't why it exists. You want something like #6585: the ability to mark projects as non-packages.

Can you share more about the use-case here? We added --no-install for Docker image building and layer caching. It's not intended for you to maintain a partial environment long-term.
I'm not necessarily opposed, but I think we should be careful here — we moved fast to deliver that feature because it's critical for Docker workflows but I'm not sure I understand this use-case as much.

Yep agree with not providing too many options unless they really make sense.
My use case is for https://github.com/fpgmaas/deptry, where we use maturin to build a project with a mix of Python and Rust. Building deptry itself takes quite some time, since we need to compile Rust.
In the CI, we have a dedicated job that validates documentation with mkdocs, that we set as a dev dependency. The job only needs mkdocs, but dependencies installation is so fast with uv that we don't mind installing other dependencies as well (like mypy for instance).
But when we run uv run mkdocs, we end up installing deptry itself, which takes quite some time, although we don't need to do that in this specific case.
We technically have other options, so it's not a blocker at all. For instance I guess we could:

move uv run mkdocs to another job that does require to install deptry itself
set the documentation specific dependencies in docs/requirements.in and use uv pip compile
use workspaces?
use the --no-project workaround that we currently use for now
use uvx<EMAIL_ADDRESS>(but we'd loose the ability to lock transitive dependencies)
...or other things I might be missing?

Of course, since deptry is a library, we cannot use virtual projects in this specific case.

Thanks for the details! We have the same problem here in the uv repository. We do
uvx --with-requirements docs/requirements.txt -- mkdocs serve -f mkdocs.public.yml

right now. Obviously this isn't ideal though. I think "project-level" tools is what you're looking for here (#3560) but we don't support that yet and need to do some design work first. I think --no-project is the "correct" workaround for now. This use-case is important, but I think uv run --no-install-* is probably not be a good long-term solution for it.

I don't think you're looking for --no-install-project. I mean, it might work for that use-case, but that isn't why it exists. You want something like #6585: the ability to mark projects as non-packages.

Ah! I think I'm just making some assumptions based on my unfamiliarity with the terminology in this project so far - thank you for pointing out that PR!  I'll just patiently wait for that one.

No prob, I mostly just think we can do a better job of solving your problem that way :)

@jfgordon2 the alternate approach is available in uv 0.4.0 now.

The change in uv 0.4.0 still doesn't fix the scenario when running uv run that it will install all --dev dependencies that aren't installed usually in a container. You still need to do uv run --no-project everytime. (think ssh into a container to debug)

There's uv run --no-dev as well. Might make sense to add UV_NO_DEV or UV_NO_PROJECT for containers. What happens if you delete the pyproject.toml and uv.lock after you setup the project? I presume we won't discover the additional dependencies then.

I presume we won't discover the additional dependencies then.

Yes, it runs normally.

There is --frozen but is there a uv run --no-sync? Just something to say don't do anything, just use the venv as it is (but it's not no-project, I'm running w.r.t the current project's workspace and virtualenv)

uv run --frozen should be effectively the same as uv run --no-sync

I thought frozen would be it, but it installs packages if they are missing. To clarify I'm looking for a flag that makes uv run never modify the virtualenv.
Here's a minimal reproducer. Yes I am deleting the venv -- making it empty -- in this example.
uv init --package testrun
# Initialized project `testrun` at `/home/user/testrun`
cd testrun/
uv sync
# Using Python 3.12.3
# Creating virtualenv at: .venv
# ...
uv venv
# Using Python 3.12.3
# Creating virtualenv at: .venv
# Activate with: source .venv/bin/activate
uv run --frozen hello
# Installed 1 package in 1ms
# Hello from testrun!

It would install more (dev-deps, other missing deps) if those existed in the example.

uv run --frozen just ensures that we don't update the lock. It still syncs the environment.

I want to add a use case. I have a project that's actually not virtual (I do want it to be an installable package). However, PyCharm becomes buggy when I have the package itself (maybe also because it's editable?) as a dependency of an SSH interpreter. So I want to add such a flag (--no-install-*) when I run some uv subcommand from this env.


make TBStorage.clearCache work across subdomains.

I have noticed it causing some issues with new modmail.
What I am thinking is:

Using it will send out an event through the background page to all open reddit pages triggering it there as well.
It will add an extension storage item with an integer as value, this one will be increased every time the cache is cleared. A similar item will be added to the cache. On page load the two are compared, when the one in cache is lower the cache will be cleared. This will make sure that even if there is no page open on a subdomain the cache always will be cleared.

test

https://github.com/creesch/reddit-moderator-toolbox/commit/6751e255dc02982ec269f3c0588cf8e3ccfdd4b1 fixes this.


XmlClassTest.java:85-86: this method appears three times...

The puzzle 297-5540e446 from #297 has to be resolved:
https://github.com/yegor256/jpeek/blob/79004d0cf652c23f98ae6d0209ade3cd1cce8d5d/src/test/java/org/jpeek/skeleton/XmlClassTest.java#L85-L86
The puzzle was created by Gnusin Pavel on 19-Nov-19.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@paulodamaso/z please, pay attention to this issue

Job #359 is now in scope, role is DEV

@paulodamaso/z not enough funds available in the project, can't set budget of job #359, see §21; @thomas-melville/z will get no money on completion; in order to fix that, add funds to the project and assign the job again

Resigned on delay, see §8: -30 point(s) just awarded to @thomas-melville/z

The user @thomas-melville/z resigned from #359, please stop working. Reason for job resignation: It is older than 10 days, see §8

The job #359 assigned to @monomonedula/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@monomonedula/z this job was assigned to you 7days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

Resigned on delay, see §8: -30 point(s) just awarded to @monomonedula/z

The user @monomonedula/z resigned from #359, please stop working. Reason for job resignation: It is older than 10 days, see §8

The job #359 assigned to @akryvtsun/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@akryvtsun/z this job was assigned to you 8days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat assign @akryvtsun

@0crat assign @akryvtsun (here)

@paulodamaso The job #359 assigned to @akryvtsun/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; we should be aware that @akryvtsun/z is on vacation; this ticket may be delayed; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @paulodamaso/z

@0crat assign @akryvtsun (here)

@paulodamaso There is an unrecoverable failure on my side. Please, submit it here:
PID: 11@9c2ebbbaf9da, thread: PQ-C7JGJ00DP
com.zerocracy.claims.ClaimIn[214] java.lang.IllegalArgumentException: Parameter "age" not found in "Order was canceled" among: [comment, job, issue, repo, cause, _parent_sig, flow, login]

0.50.38: CID: d25c1093-bc67-4aeb-896a-bfbf32048417, Type: "Order was canceled"

The puzzle 297-5540e446 has disappeared from the source code, that's why I closed this issue.

Job was finished in 0 hours, bonus for fast delivery is possible (see §36)

@fabriciofx/z please review this job completed by @akryvtsun/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

There is an unrecoverable failure on my side. Please, submit it here:
PID: 11@9c2ebbbaf9da, thread: PQ-C7JGJ00DP
com.jcabi.xml.StrictXML[124] java.lang.IllegalArgumentException: 1 error(s) in XML document: -1:-1: Duplicate unique value [gh:yegor256/jpeek#359] declared for identity constraint of element "speed".

0.50.38: CID: 9e587d77-5745-4a41-9d21-b8cabaa5b3a9, Type: "Order was finished"

The job #359 is now out of scope


I18n support

Hi,
currently some labels are hard-coded in the JS file ("Start typing a name" etc.). There should be a way to localize all strings, maybe using the options object which is passed to the jfmfs() call.

"Start typing a name" is still hard-coded in two places.
You may want to reopen this issue.


CF template for additional account / region starts reciprocal scan

Describe the bug
After running the supplied template to add a new account and region choice, reciprocal scanning started which was not expected
To Reproduce
Instantiate main perspective CF stack in Account A; region eu-west-2
Set up Cognito user and log into UI
Follow Account Management dialog to add a new account (Account B) region eu-west-2
Download cloud formation template
Apply template to get new 'listener authority' CF stack in Account B
Expected behaviour
Account A AWS Config is allowed to scan region in Account B, this shows in list of resources in Aggregator in AWS Config
Account B AWS Config shows authorisation it is granting to Account A
Actual behaviour
Account A AWS Config is allowed to scan region in Account B, this shows in list of resources in Aggregator in AWS Config
Account B AWS Config shows authorisation it is granting to Account A
Plus!...
Account B AWS Config aggregator is recording resources owned by Account A and shows a list of resources

Ah yes, the template is creating an aggregator in the second account but it doesn't actually need to do that does it?


[IOTDB-3277] [IOTDB-3310] [IOTDB-3311] Fix some SQL parser bugs in new cluster

support CREATE STORAGE GROUP
fix bug when combining GroupBytime and Fill
fix bug when using nested expressions in aggregate query

Coverage decreased (-0.02%) to 45.037% when pulling 7dec34a373dd31a72b7eeb0bdc8ef2e45de34aa3 on lmh/mppSqlParserBug into 9ae4c8841501dff1c9e761a2e421369fcf3f5dbe on master.


Update tool tip on contract/offer dialogs to be 'Send to Contact'

H/T to @benthecarman for finding this one. Previously it would be Add Contact which confused him as he didn't want to add a contac, he wanted to send to a contact.

"Select Contact"?

"Select Contact"?

Yes, add contact makes users think they are adding a new contact

Same needs to change in contact-detail.component.html and accept-offer.component.html

Done in #185


Add rule for using the Player CLI to compile XLRs

Adds rule xlr_compile to use the Player CLI to compile XLRs for a package.
Didn't integrate with the js_library_pipeline since its not applicable in all use cases.

Build failure is just due to a missing tag.

:rocket: PR was released in v0.10.0 :rocket:


ANSI escape codes in Miri output

Started happening only recently. Since a day ago or so?

Caused by https://github.com/rust-lang/miri/issues/2277

See also #645

I think this issue here has long been fixed. Also the screenshot from the OP doesn't exist any more so we can't even try to reproduce it.
@shepmaster can you close this?

If the fix is "don't show them", then yes, that PR resolved it.  If its "show colored output", then it seeems #645 would be the issue to track that.

This issue is about a Miri specific issue. That one got fixed.


kvm stage1 should return pod exit status

While writing test cases, I realized that kvm flavors always return 0 even if the application exit with some other exit codes. Behavior should be aligned with non-kvm flavors, where user/caller gets a meaningful exit status upon termination.
I think @alban and others were already aware of this, but I didn't find any report tracking it.
/cc @jellonek

@pskrzyns can you take a look at this one?

I'm on vacation till monday :)
@jjlakis @lukasredynk could you look at it ?

@tmrts @alban can we create team kvm on this repo ?

@pskrzyns :+1:

@coreos/rkt-kvm-maintainers can you set a proper milestone here?

I'll move it to v1.12.0

@iaguis Thanks

Bump, @jjlakis any chance you can have a look at this?

I see no other way to support this than running hypervisor engine (lkvm, qemu) by other process, and then collect status from remains of pod as this is already done by rkt status. So rkt binary would not exec directly to hypervisor engine, but to this overlapping helper binary. This looks annoying but seems to be very simple in implementing...
Imo this binary should be statically linked C program, small binary like enter, prepare-app because of golang overhead...

I see. I was probably too optimistic asking for stage1 to just bubble-up the return value from systemd, but skimming a bit I realized lkvm/bzimage have no concept of a return value (uml has, though).
Thinking a bit more on this, the binary wrapper may not even be such a great solution because we don't want to introduce another long-running component, and it will miss anyway errors from failed stage1 setup (while stage2 status can be already retrieved the way you mentioned).
The original context for this was that we have some test units using this feature on the coreos flavor, and we end up not running some tests on kvm due to this. At this point, however, I think it is better to adjust the testsuite to be aware of this, instead of introducing other moving parts here.
De-milestoning but keeping this around in case we get better support from the intermediate layers in the future.

Just as an update/note here - the same situation exists for other VM container systems. In Clear Containers we have gone with the 'intermediate binary' approach, where we have an extra process that reaps the exit code and returns it to the higher level.
A fundamental issue is that (for instance) qemu does not return the exit code of its payload, but of itself - thus, directly running qemu will never get you the payload exit code, unless we could add a feature to qemu.
When we come to re-visit this issue we can look at re-using the Clear Containers code.
CC @jodh-intel


Add support for Open Yesterday's note

Closes #33 :smiley:

@svsool happy to get any feedback on this PR :smile_cat:

:facepalm: #33 talks about Tomorrow's notes. Not yesterday :facepalm:


Fixed turtlebot3_house.launch.py by using robot_state_publisher.launc…

There's no "turtlebot3_state_publisher.launch.py" in turtlebot3_gazebo. It should be changed to "robot_state_publisher.launch.py" instead.

Closing as this change has been applied.


Use of outdated brew cask commands by Ansible

The actual version of Ansible uses a deprecated command from actual brew commands.
TASK [common : Install homebrew cask packages] *********************************************
failed: [<IP_ADDRESS>] (item=google-chrome) => changed=false
  ansible_loop_var: item
  item: google-chrome
  msg: 'Error: Calling brew cask install is disabled! Use brew install [--cask] instead.'

There is actually a PR on ansible-collections / community.general to fix this issue.
https://github.com/ansible-collections/community.general/pull/1481
Homebrew version has to be downgraded manually while this issue is not fixed.
https://github.com/ansible-collections/community.general/issues/1524#issuecomment-749226927
I ran a manual brew upgrade as the script was breakinb due to an outdated version of curl on my environment.
failed: [<IP_ADDRESS>] (item=filezilla) => changed=false
  ansible_loop_var: item
  item: filezilla
  msg: |-
    Updating Homebrew...
    Warning: curl is outdated!
    To avoid broken installations, as soon as possible please run:
      brew upgrade
    Or, if you're OK with a less reliable fix:
      brew upgrade curl
    Error: Calling brew cask install is disabled! Use brew install [--cask] instead.

Maybe we will have to upgrade the version of Ansible when the fix will be released ?

I see that, I had the same issues on my personal mbp.

Maybe we will have to upgrade the version of Ansible when the fix will be released ?

Yes, we will have to update the requirements.txt then dev upgrade

It's fixed

It's fixed


etcher-image-stream was moved to the main etcher repo.

But I think its still an interesting module to mention.

Thanks for the contribution! Could you update your commit message to adhere to our commit guidelines – then it'll be good to go :)


[Bug]: Cluster autoscaler not creating any nodes

Description
A nodepool with autoscaling is created with minimum nodes >0, but no nodes are created.
I tried deploying a fresh cluster with and without the image that fixes the CX11 deprecation problem
Kube.tf file
network_region = "eu-central" # change to `us-east` if location is ash

  control_plane_nodepools = [
    {
      name        = "control-plane-nbg1",
      server_type = "cax11",
      location    = "nbg1",
      labels      = [],
      taints      = [],
      count       = 1

    },
  ]

  agent_nodepools = [
    {
      name        = "workers",
      server_type = "cax11",
      location    = "fsn1",
      labels = [
        "node.kubernetes.io/role=workers"
      ],
      taints = [
      ],
      floating_ip = false
      count = 1
    },
    {
      name        = "ingress",
      server_type = "cax11",
      location    = "fsn1",
      labels = [
        "node.kubernetes.io/role=ingress"
      ],
      taints = [
        "node.kubernetes.io/role=ingress:NoSchedule"
      ],
      floating_ip = false
      count = 1
    },
  ]
  #load_balancer_type     = "lb11"
  load_balancer_type     = "lb11"

  load_balancer_location = "fsn1"

  autoscaler_nodepools = [
    {
      name        = "autoscaled-egress"
      server_type = "cax11"
      location    = "fsn1"
      min_nodes   = 3
      max_nodes   = 5
      labels = {
        "node.kubernetes.io/role" = "egress"
      }
      taints = [
        {
          key    = "node.kubernetes.io/role"
          value  = "egress"
          effect = "NoSchedule"
        }
      ]
    }
  ]

  cluster_autoscaler_image = "docker.io/hetznercloud/cluster-autoscaler"
  cluster_autoscaler_version = "v1.29.4-hcloud1"

Screenshots
│ I0909 18:32:01.723771       1 static_autoscaler.go:291] Starting main loop                                                                                │
│ I0909 18:32:01.723893       1 hetzner_node_group.go:599] Set node group k3s-autoscaled-egress size from 0 to 0, expected delta 0                                                                                │
│ I0909 18:32:01.724104       1 hetzner_node_group.go:380] Build node group label for k3s-autoscaled-egress                                                                                │
│ I0909 18:32:01.724129       1 hetzner_node_group.go:394] k3s-autoscaled-egress nodegroup labels: map[beta.kubernetes.io/instance-type:cax11 csi.hetzner.cloud/location:fsn1 hcloud/node-group:k3s-autoscaled-eg │
│ I0909 18:32:01.724365       1 filter_out_schedulable.go:63] Filtering out schedulables                                                                                │
│ I0909 18:32:01.724391       1 filter_out_schedulable.go:120] 0 pods marked as unschedulable can be scheduled.                                                                                │
│ I0909 18:32:01.724410       1 filter_out_schedulable.go:83] No schedulable pods                                                                                │
│ I0909 18:32:01.724418       1 filter_out_daemon_sets.go:40] Filtering out daemon set pods                                                                                │
│ I0909 18:32:01.724426       1 filter_out_daemon_sets.go:49] Filtered out 0 daemon set pods, 0 unschedulable pods left                                                                                │
│ I0909 18:32:01.724465       1 static_autoscaler.go:548] No unschedulable pods                                                                                │
│ I0909 18:32:01.724506       1 static_autoscaler.go:571] Calculating unneeded nodes                                                                                │
│ I0909 18:32:01.724524       1 pre_filtering_processor.go:57] Node k3s-control-plane-nbg1-boo should not be processed by cluster autoscaler (no node group config)                                               │
│ I0909 18:32:01.724538       1 pre_filtering_processor.go:57] Node k3s-ingress-yhe should not be processed by cluster autoscaler (no node group config)                                                          │
│ I0909 18:32:01.724551       1 pre_filtering_processor.go:57] Node k3s-workers-cqp should not be processed by cluster autoscaler (no node group config)                                                          │
│ I0909 18:32:01.724612       1 static_autoscaler.go:614] Scale down status: lastScaleUpTime=2024-09-09 17:26:49.693658128 +0000 UTC m=-3597.837867675 lastScaleDownDeleteTime=2024-09-09 17:26:49.693658128 +000 │
│ I0909 18:32:03.743981       1 reflector.go:800] k8s.io/client-go/informers/factory.go:159: Watch close - *v1.Job total 47 items received                                                                        │
│ I0909 18:32:08.743847       1 reflector.go:800] k8s.io/client-go/informers/factory.go:159: Watch close - *v1.Node total 26 items received                                                                       │
│ I0909 18:32:09.738820       1 reflector.go:800] k8s.io/client-go/informers/factory.go:159: Watch close - *v1.CSIStorageCapacity total 6 items received                                                          │

Platform
Linux

Just noticed that this may be caused due to account limits...

  cluster_autoscaler_extra_args = [
    "--enforce-node-group-min-size=true",
  ]

As seen in https://github.com/kubernetes/autoscaler/issues/6564 this is intended behavior. The fix is even commented out in the kube.tf
IMO it should be default, default ignore makes no sense, even if it is how upstream works.


Add UiImage offset property

Objective
Currently UiImage component is just a unit struct for Handle<Image> . At it's core, it draws the image and that's it. If someone needs to have a more complex use case, like implementing a nine patch, one have to use TextureAtlas which is good, but is a overkill for ui works.
This PR is related to https://github.com/bevyengine/bevy/pull/5070, but it aims to make TextureAtlas to work with UiImage which is a different use case IMO, since one should use TextureAtlas and UiImage together on same entity.
Solution
Add an offset property to UiImage and enable to render a portion of the image, much like a TexturaAtlas but simplified, so users may create it's own image atlas and reuse on many UiImage components.
Changelog

Added offset property on UiImage and changed to from unit struct to normal struct;
Removed Deref and DerefMut from UiImage since it's no longer a unit struct;
Changed extract_uinodes to use UiImage::offset when it's non-zero;
Added image_button example:

Migration Guide
UiImage is now a normal struct so it can't be directly dereferenced anymore and should use normal struct initialization.

Ping @ManevilleF for review here :)

Code quality and docs seem fine, minus a few nits. I'm not fully sold on the grand vision here, so I'll defer to @mockersf and @ManevilleF who've thought about this space more.

The new property offset should be called rect and I think it should be a separate component, overriding the default behaviour to be more ECS friendly.
But to customize Rect of a texture is what a TextureAtlas is for, and I think #5072 is the way, as UI and sprites shouldn't behave differently in that scope

I can't do a complete review anytime soon, but anything that impact rendered size should impact layout, and I didn't see layout files being modified in this PR

I can't do a complete review anytime soon, but anything that impact rendered size should impact layout, and I didn't see layout files being modified in this PR

Closing in favor of #5103; let's collect our efforts there.


createFilterString escape method fix

createFilterString uses ldap_escape() instead of escapeValue() to escape filter values which fixes an issue where certain guids are not escaped correctly.

Pretty sure I did that incorrectly. This needs to go to the develop branch then correct? Incompatible with php5.5 obviously as ldap_escape() was added in php5.6 so that check was failed.

Thank you for your contribution! I'm really sorry that it took me so long to get back to you on this issue!

No problem, thanks!

Do you by any chance have an example GUID that didn't escape correctly? I'd like to include a test with such example data to prevent a regression.

I'll find one really quick.

e208c55daefed349830e23870f797e00

Thanks for the GUID. The issue is that filtering it works perfect with the current mechanism. At least I could not reproduce an issue :frown:
Without being able to reproduce the issue it's really tricky to resolve that once and for all…
I'm checking further though…

It get's somewhat weirder. I just had a look at the source code of ldap_escape. It does almost exactly the same as the str_replace in the escapeValue-method. The only difference is that ldap_escape also escapes NULL-bytes (\0). So the only idea I currently have why ldap_escape worked but escapeValue not is that there has been a NULL-byte in the GUID.
I'll check whether I can reproduce that in the tests!

OK, thank you for looking into it.
It was a consistent issue for me, until I extended the Filter class and overwrote the createFilterString method. If I go back to the Filter class, the issue still remains. I can try to step through it again and see if I can dig up some more details.
I wonder if it could be a configuration issue on our AD server end... But I know very little about AD and LDAP, so that might not make sense.

Let me know if another guid would help

When you can still reproduce the issue I'd be more than happy when you could provide me with the data that goes into your ``escapeValue``` method so I can add that to a test-set. That would be absolutely awesome!

Sure thing, will do.

Hey @KaelBaldwin Any news here? I'D love to resolve this issue but I'm in need of your data here…

Sorry, I haven't spent the time yet to pull that up. I'll have something here shortly.

ok, so...
Before escapeValue: Å]®þÓI83♫#87 y~\000
After escapeValue: â\08Å]®þÓI83\0e#87 y~\000
now if I use ldap_escape as I do in this pull request I get:
After: Å]®þÓI83♫#87 y~\00

Thanks! That definitely helps!! I'll dig further into it!

From what I've found out after digging deeper:
I've taken your provided GUID and inserted that as HEX-Input into an HEX-Editor:

I saved the result as file and opened that with a default Text-Editor using ISO-8859-1 Text-Encoding:

From that I'm asking myself different things:

Where is that starting â in your unescaped and in the ldap_escape'd value gone to?
Where does the ♫ come from? Which encoding are you using?

For me all that looks very much like an encoding issue. Which encoding are you using? What is the default-charset and encoding set in PHP and in what charset and encoding is the file providing the GUID stored? What Filesystem on what operating system are you using?
And besides that: Why are you handling the GUID in binary? Usually GUIDs (actually any UUIDs) are handled using a 32 character-String containing HEX values.
Thanks for your insight!

Thanks,i'll take a look at that tomorrow and see what I can find out.

So, is it safe to use current version of zend-ldap (without ldap_escape) now?


Add Embedded Review Checklists for Various Analyses

To support design and other review, there should be a method for the user to work through a checklist to verify the completeness of an analysis.  A list of basic questions should be provided by RTK "out of the box", but this list should be editable and addable by the end-user.  At a minimum, the following checklists should be provided:
1. LCC analysis
2. Reliability assessment.
3. FMEA/FMECA (functional and hardware)
4. Allocation
5. Hazards analysis

In addition, there should be design review checklists provided for PDR, CDR, and TRR.  A checklist should be associated with the following:
1. Revision module to assess the overall RAM program.
2. Requirements module to assess the quality of the requirement/specification (analysis page already provides base questions, will need to re-format this to a RTKTreeView() to allow editing and addition of questions by the end-user).
3. Hardware module to assess the individual Hardware item.
4. Software module to assess the individual Software item (risk analysis page already provides base questions, will need to re-format this to a RTKTreeView to allow editing and addition of questions by the end-user).

The purpose of making these checklists addable by the end-user is to allow institutional knowledge to be captured for posterity.

The following will be needed to implement this feature:
* Design database tables to hold question text and answers; keep in RTK Common DB.
* Add page to appropriate module's Work View.


Name of $Foundation

Now that we're making good progress on the other topics, it's a good time to think about proposing a name for $Foundation.
I propose we meet separately to discuss this, since naming can be a time consuming process and we don't want to derail the progress being made in the main bootstrap discussions.
If you want to participate, +1 this issue and I'll add you to my list.  When we get critical mass (hopefully in a few days or so), I'll work with you to find a good time for a call.
I can say with 100% confidence the outcome of this discussion will be one of the following:

Use the Node.js Foundation name for the merged entity
Use the JS Foundation name for the merged entity
Come up with something new that's more appropriate

...I just can't say which one it'll be.  So let's discuss and put together a proposal!

Closing as this has been decided 🎉

@MylesBorins what's the name?

what's the name?

@ljharb "Open JS Foundation" https://youtu.be/DLzh65FrZEg?t=402

The formal announcement hasn't been made yet - despite it being discussed in a meeting that was broadcast on youtube 😉So please keep this quiet until you see the proper announcement from the Foundation.


make android mobile version of fnf online

What feature do you want to get added on the base engine?
I would like to tell you to make the mobile version of this game, both Android and IOS, this game is very good, and I'm sure there will be a lot more downloads after the mobile version from this version. Thanks
To test your sight, and reliability, please select the option of what should NOT be requested.


(PUP-5575) Fail if cannot read puppet.conf

CLA signed by all contributors.

I don't think this is the ideal way to solve this, but I couldn't figure out anything better :( Any feedback would be swell


[bug] '-or' flag ignores filters

Output files are still written if your using filters, like auto calibrate, or status code filters.
I was having to do this manually so now that there's a flag for it, I decided to try and edit my scripts so it was definitely a good idea.

Could you give the full command line of what you are running and what you except the outcome to be.

Closing the issue in order to keep the issues backlog readable, and I do consider the issue either resolved or there's no communication. If you have something to add to the issue, please comment and we'll consider reopening it.


Refactor display-performance-test-app

I want to add support for testing hypermodeling views, but the app is a mess of spaghetti code, global variables, data of type any, exclamation marks to bypass null checks, etc. This makes it extremely difficult to understand the ramifications of even a small change, or what the data actually looks like, or at what point in the code a given variable is null, gets reassigned/mutated, etc.
Refactored to remove all global state. Functions now take well-defined inputs and return well-defined outputs. For example, there is no global theViewport: ScreenViewport | undefined; instead, we create a viewport when we start a test, pass it into the function that runs the test, and then dispose of it.

I ran @DStradley's tests against my branch. No differences except for 2 emphasis/hilite tests - I don't appear to be overriding those settings correctly.
The performance numbers in the CSV files look as expected.
Unless anyone objects I'll merge this after fixing the aforementioned issue.


Allow disabling wrapping for arrays with only one element

Support this option with both a comment and an config value in options.

Resolved with multilineArrayWrapThreshold option and prettier-wrap-threshold comment trigger.


Update lingo workflow to most recent release versions

We would like to update the GitHub Actions workflow for LingVo to the most recent release versions including Python.

@GiulioZizzo Thank you for the proposal!


Delphi 12.0

https://altd.embarcadero.com/download/radstudio/12.0/RADStudio_12_0_4915718.iso
https://altd.embarcadero.com/download/radstudio/12.0/RADStudio_12_0_esd_4915718.exe
I want to touch it, but I won’t install it until a release from you comes out with support)

yes i m on it :) I plan to migrate next week

Hi, any plans for updating the DeployProjNormalizer for Delphi 12 too?
Love this utility for scripting the build process for iOS and Android, but otherwise it's back to generating the deploy files from inside the IDE, I guess?!

Yes i m on it right now, hope everything ready in the next 2 weeks

https://quality.embarcadero.com/browse/RSP-41451
a couple of lines of code, but it's really faster: bitmap.Canvas.Clear
11.2 was still normal there.

can you use it already? or wait until information about d12 support appears in README.md ?

Can it be used?

Hi, everything as been migrated to Delphi 12.0 Athens in the latest commit. I m still working now to make everything working with Skia

Thank you for the amazing jobMarco CarboniOn 13 Dec 2023, at 13:08, Zeus64 @.***> wrote:
Hi, everything as been migrated to Delphi 12.0 Athens in the latest commit. I m still working now to make everything working with Skia
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Awesome job !

On 13 Dec 2023, at 13:08, Zeus64 @.***> wrote:
Hi, everything as been migrated to Delphi 12.0 Athens in the latest commit. I m still working now to make everything working with Skia
—
Reply to this email directly, view it on GitHub https://github.com/MagicFoundation/Alcinoe/issues/310#issuecomment-1853798323, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADAS3VJL3KEP6NOO672FQVDYJGLCBAVCNFSM6AAAAAA7BRGIJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJTG44TQMZSGM.
You are receiving this because you are subscribed to this thread.

This is really great! I'm using the DeployProjNormalizer for Delphi12 now.
Note sure if the updated version is supposed to work with Delphi11.3 projects as well?
I noticed that using the updated DeployProjNormalizer built with Delphi12, on a Delphi11.3 project generates a slightly different .deployproj for Android while for iOS the files are identical.
The new .deployproj is missing the Android_LauncherIcon48,..,Android_LauncherIcon96, but as we have to switch to Delphi12 for Android anyway this is not an issue for us.
Again really appreciate this utility, thanks a lot!

To work with Delphi 11.3 better to use this version of alcinoe : https://github.com/MagicFoundation/Alcinoe/releases/tag/v1.0.7
Delphi Athens introduce new resources in Android (like Android_VectorizedSplash, Android_VectorizedSplashV31Dark, etc.) and theses resources go in conflict with the r.jar that the androidmerger tool must build from all aar. so i decide in android to not deploy any of theses ressources :
//     be added via the AndroidMerger tool:
//       Android_Strings
//       Android_Colors
//       Android_ColorsDark
//       AndroidSplashImageDef
//       AndroidSplashImageDefV21
//       AndroidSplashStyles
//       AndroidSplashStylesV21
//       AndroidSplashStylesV31
//       Android_AdaptiveIcon
//       Android_AdaptiveIconV33
//       Android_VectorizedNotificationIcon
//       Android_AdaptiveIconMonochrome
//       Android_AdaptiveIconForeground
//       Android_AdaptiveIconBackground
//       Android_VectorizedSplash
//       Android_VectorizedSplashDark
//       Android_VectorizedSplashV31
//       Android_VectorizedSplashV31Dark
//       Android_LauncherIcon36
//       Android_LauncherIcon48
//       Android_LauncherIcon72
//       Android_LauncherIcon96
//       Android_LauncherIcon144
//       Android_LauncherIcon192
//       Android_SplashImage426
//       Android_SplashImage470
//       Android_SplashImage640
//       Android_SplashImage960
//       Android_NotificationIcon24
//       Android_NotificationIcon36
//       Android_NotificationIcon48
//       Android_NotificationIcon72
//       Android_NotificationIcon96
If you need this resource you must add them with the androidmerger tool (see somes exemples in for exemple Demos\ALFmxControls_Source\Android\MergeLibraries.bat)

To work with Delphi 11.3 better to ...
Thanks Zeus64... I've just kept a backup .exe version for Delphi11.3 just in case


NIP-60 cashu wallets

Is your feature request related to a problem? Please describe.
NIP-60 specifies a cashu wallet that stores its unspent proofs on nostr relays.
It's another wallet one could attach since it can pay any bolt11. It's also meant to make receiving immediately available:

The purpose of this NIP is:

ease-of-use: new users immediately are able to receive funds without creating accounts with other services.
interoperability: users' wallets follows them across applications.

 So if stackers are already using nostr and don't mind custody, this might become a popular option as a "starter wallet" since it requires little to no configuration.
Describe the solution you'd like
Add NIP-60 as a wallet
Describe alternatives you've considered
n/a
Additional context
n/a

this is a bit tricky to do on the receiver side.
The flow is like this:
quote, invoice = GET_MINT_QUOTE(sats)
PAY_INVOICE(invoice)
token = MINT_TOKENS(quote, pubkey)
SEND_NUT_ZAP(tokens)

after the tokens are minted  only the receiver can spend them, but there is nothing stopping the server from minting tokens for a different pubkey after they invoice is paid.
So this kinda break the no-trust contract of the receiving attachments.

There's nothing stopping us from giving senders an invoice to SN's node today too though, right?

I think the issue is that it's not possible to forward cashu tokens without trust unlike lightning invoices

request invoice from mint for tokens bound to pubkey

this is a multi step process, first you ask a quote, then you pay its invoice, and finally you mint the ecash bound to the pubkey.
Then you send the ecash to nostr with a nutzap to the pubkey.
Speaking of, this will need a job running on the server to claim and send the ecash once it is mintable.
Is that something I can do just to support a wallet attachment?

Oh, P2PK might work. Does the mint return locked tokens or do we need to lock them ourselves? I don't know enough about Cashu to answer that question but it should be somewhere in these NUTs:

https://github.com/cashubtc/nuts/blob/main/04.md
https://github.com/cashubtc/nuts/blob/main/10.md
https://github.com/cashubtc/nuts/blob/main/11.md

this is a multi step process, first you ask a quote, then you pay its invoice, and finally you mint the ecash bound to the pubkey.
Then you send the ecash to nostr with a nutzap to the pubkey.

Unfortunately, this sounds like we have custody after the invoice is paid, ie we can choose to steal the sender's money.

I asked in the Cashu chat and @cashubtc replied to me:

So receive looks possible to me

Yes if they're initially locked, this will work.

The problem is not with the tokens that are locked, but with the quote that is not locked.
My understanding reading NUT-04 is that if you have the quote id you can mint the tokens you want up to the quota.
Our use case is strange because we are minting from the server, usually you mint from the client.

Speaking of, this will need a job running on the server to claim and send the ecash once it is mintable. Is that something I can do just to support a wallet attachment?

Assuming the locking stuff is true, you can add an optional function to receive wallets that gets called when onPaid runs possibly.

My understanding reading NUT-04 is that if you have the quote id you can mint the tokens you want up to the amount.

Ah, so the client needs to be aware that it's zapping using ecash. This might be a function of gateways I guess? Lame.

Afaict, this would enable "locking quotes to pubkeys".

nice find, yes, and we need some code to run client-side to mint the tokens:

we generate only the quote+invoice on the server
then somehow we send the quote back to the client
the next time the stacker is online the ecash is minted and sent to nip-60

this is how i can see this working currently

Yes, that is also my understanding now.
But let's start with a PR that only implements paying BOLT11 via NIP-60.

Note for receives: we should try to use ephemeral keys so the mint can't link minted tokens together. Ideally, the receiver would give us some kind of xpub.


Add column(s) to Sheets exporter

I'm opening this as a draft because I think it still needs thinking/work.
Background: I'm using Caspion with just Google Sheet as exported. Having no "Currency" field is a blocker for me as I can't differentiate when a new "USD" or alike transaction appears.
I've added the changed described in this PR, they do work, but the problem is it's not backward compatible.
(Note: The field "Currency" is discussed here, but once we agree on something I can add the rest of fields as appear in the bank scraper or CSV, if needed).
I see a few possible options to make it backward compatible:

"Quick and ugly": Instead of adding the field(s) columns in the middle, we can add those in the end (most right column).  column. Also, it doesn't promise it won't cause issues for users who are not expecting it.
"Migrating date": Analyzing the structure of the current sheet and manipulating it to extend to new columns. Problem here is that also we're not sure if the sheet itself was manipulated. Also, this is a lot of work with introducing new features on the sheet itself.
"Only for new sheets": We can introduce the new fields only for new "sheeters" (🤣). Meaning, if we are creating the sheet for you now, we'll pour in the new fields. This is the most elegant solution IMO, but will require some leftovers in the codebase itself.
"Let the users shift the data manually" - we keep the code as it is now, and users will need to shift the code as they see right. meaning a breaking change, but easy to fix.

WDYT?
P.S. Thank you @brafdlog @baruchiro for this amazing code! much appreciated.

Hi @SeanZoR,
Thanks for bringing this up, these are good points.
I have been thinking about how to handle breaking changes in the structure of the configuration object but haven't yet thought about the spreadsheet structure changing.
Here are my thoughts about the options you suggested, will start with the easier ones :)
(2) Doing an analysis of the structure of the sheet and acting according to that - sounds fragile and not worth the ROI. I don't think we should go that route.
(4) Keeping the code as is - not great. This will break the hashing logic so transactions will be duplicated. The value we gain from the nicer order of columns is not worth IMO such a breaking change.
(3) only for new sheets - this option is interesting. It relates to a thought I was having about versioning. We could make the sheets have versions and each change in the schema will tick up the version in the code. If we do this, we can easily detect that the sheet is not up to date and display a notification to the user with the required changes to comply to the new version (or do the changes for him if he wants). I think long term this is a good option, but given the beta state of the project, I wouldn't go there yet.
(1) Move the column to the end - I think this is the right thing to do now. This gives support for the currency use case at a minimal effort and it doesn't break the sheets backwards because the hashing mechanism will still work so old txns that were created in the old format would not change and just new txns will be created in this format.
@baruchiro  @SeanZoR  WDYT?

@SeanZoR you can proceed with the first option of moving the column to be the last. @baruchiro is not available this week and we won't wait. If you are happy with this option go ahead, if not, lets discuss

I agree with @brafdlog, I think we should solve the problems in an easy way, because we don't have enough effort to spend on the best implementation.
Thanks a lot @SeanZoR! Well-defined PR.
Looking forward to seeing more PRs for you, feel free to take care and suggest things, or just contribute for fun :-)
If you need any help, you know where to find us...
(BTW you should set this PR to be Ready for review if you want us to approve it..)

Hey all, thank you for your notes. I'm on a vacation right now but will
follow through with the changes once I'm back 🔥
On Thu, Mar 31, 2022, 14:11 Baruch Odem (Rothkoff) @.***>
wrote:

I agree with @brafdlog https://github.com/brafdlog, I think we should
solve the problems in an easy way, because we don't have enough effort to
spend on the best implementation.
Thanks a lot @SeanZoR https://github.com/SeanZoR! Well-defined PR.
Looking forward to seeing more PRs for you, feel free to take care and
suggest things, or just contribute for fun :-)
If you need any help, you know where to find us...
(BTW you should set this PR to be Ready for review if you want us to
approve it..)
—
Reply to this email directly, view it on GitHub
https://github.com/brafdlog/caspion/pull/333#issuecomment-1084433348,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVENBPSNQBUWUQEWY7BMLTVCWB5DANCNFSM5RRXJTTA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey, OK this looks good. meaning we are going with option (1) Move the column to the end.
i had a look on other potential fields and haven't found them useful at this point.

@baruchiro I've made the change now, I believe it's ready.
I've tested :
A. a migration of it, meaning starting with a sheet that was in master branch, and then switching to this branch. Works fine 👍 Looks like this, note the most right column

B. Starting a sheet straight from this branch, also looks good and also has the desired col title

About amount and chargedAmount it's unrelated to currency, but rather just shows the total amount and if you have תשלומים

Thanks @baruchiro , I don't have write access so can't merge, but go ahead and merge for me 👍
And kudos again for you and @brafdlog on building this project! 👑   It's not easy to dedicate the time to it.
I don't have any other outstanding issues but if will..


Module 2 exercises [YAKUBU HUZIEMATA]

Please "

" exercise 1 file below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 2 and make sure to take
the post-training survey.

Hi Yakubu, great work with the screenshots! Could you please fill in the questions in the
template to receive full credit? Thank you!


Avoid waiting for Consumer.close when removing resources

In an E2E test run, we saw that failing to close a consumer
will cause the reconciler to stop reconciling resources.

[1] the data plane reconciler saw that we need to remove a
resource
since the previous removal never "completes" (as we're
waiting for the callback to complete), newer contract
generations [2] are never reconciled

[1]
{"@timestamp":"2022-06-06T11:46:44.775Z","@version":"1","message":"Set new contract contractGeneration=107","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerMessageHandler","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000,"contractGeneration":107}
{"@timestamp":"2022-06-06T11:46:44.775Z","@version":"1","message":"Reconciling contract contractGeneration=107","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerMessageHandler","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000,"contractGeneration":107}
{"@timestamp":"2022-06-06T11:46:44.775Z","@version":"1","message":"Reconcile egress diff DiffResult{added=[], intersection=[], removed=[d5f66d67-03f1-440e-a7b1-d8408107b5dc]}","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerImpl","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000}
{"@timestamp":"2022-06-06T11:46:44.775Z","@version":"1","message":"Stopping consumer","logger_name":"dev.knative.eventing.kafka.broker.dispatcher.impl.consumer.BaseConsumerVerticle","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000}

in particular, only one egress (KafkaSource) was removed at contract generation 107
Reconcile egress diff DiffResult{added=[], intersection=[], removed=[d5f66d67-03f1-440e-a7b1-d8408107b5dc]

[2]
{"@timestamp":"2022-06-06T11:46:55.564Z","@version":"1","message":"Set new contract contractGeneration=108","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerMessageHandler","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000,"contractGeneration":108}
{"@timestamp":"2022-06-06T11:46:56.27Z","@version":"1","message":"Set new contract contractGeneration=109","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerMessageHandler","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000,"contractGeneration":109}
{"@timestamp":"2022-06-06T11:46:57.28Z","@version":"1","message":"Set new contract contractGeneration=110","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerMessageHandler","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000,"contractGeneration":110}
{"@timestamp":"2022-06-06T11:47:02.216Z","@version":"1","message":"Set new contract contractGeneration=111","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerMessageHandler","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000,"contractGeneration":111}
{"@timestamp":"2022-06-06T11:47:10.103Z","@version":"1","message":"Set new contract contractGeneration=112","logger_name":"dev.knative.eventing.kafka.broker.core.reconciler.impl.ResourcesReconcilerMessageHandler","thread_name":"vert.x-eventloop-thread-0","level":"INFO","level_value":20000,"contractGeneration":112}

we will never see:
Reconciling contract contractGeneration=108 or
Reconciling contract contractGeneration=109

Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>Fixes #

Proposed Changes

Release Note

Docs

/cherry-pick release-1.5

/cherry-pick release-1.4

/cherry-pick release-1.2


limit number of jobs in tube

We limit the numbers of jobs through parameter -m and already use it in our project. It can reduce the pressure on producers, consumers and disk. Is it necessary to merge it?

I have submitted a issue at #327 , welcome to leave comments for this PR. : )


Adds expression re-writing capabilities in LSP InitialFixup method.

    private readonly Dictionary<string, CodeFixHandler> _handlers = new Dictionary<string, CodeFixHandler>();

Can fixup be here (instead of engine)?
EditorContextScope already has other state, like code fix handlers.

Refers to: src/libraries/Microsoft.PowerFx.LanguageServerProtocol/Public/EditorContextScope.cs:42 in 30406d6. [](commit_id = 30406d6a7951c5d9b955098b09d5d3bea2d5d3a9, deletion_comment = False)

✅ No public API change.

✅ No public API change.


Request for countersunk holes

I am using the led part for a variety of other purposes, including pushbuttons:
https://www.digikey.com/en/products/detail/e-switch/TL1105SPF250Q/271559
with caps:
https://www.digikey.com/en/products/detail/e-switch/1RBLK/271579
That seems to work fine if I simply set the hole size to 5mm.
I also use it to make round 3mm holes for hex spacers like these:
https://www.amazon.com/dp/B08DMSNB8X?psc=1&ref=ppx_yo2ov_dt_b_product_details
but it would be handy if the holes could be countersunk.  Is that possible?  The plastic is too think for the standard screws to reach all the way through, and countersinking would also make them flush

Good suggestion! I also have these standoffs in stock
It'll probably make sense to countersink them from the back. That should keep the print layers from separating when pushing buttons.
I'll try some things out. 👌

Two thoughts about it:

It's trickier to print with overhangs. Since we print these face down, I'd try to keep the front panel flat.
I was just thinking about the way layers are printed when FDM printing. Having the head of the screws supported by a middle layer adds a risk of ripping that layer off (It did happen to me in some earlier prototypes).

I see what you mean now.  I have never seen a part delaminate like that.  Though I am much newer to 3d printing panels than you are, I have been doing a fair amount of printing for some years.  I would suspect a printer issue if my layers could not hold up to the kinds of forces you are talking about.
I notice however that I am unlikely to have the problem listed at the end of this article, since I live in the desert:
https://all3dp.com/2/3d-print-delamination-tips-tricks-to-avoid-layer-separation/

Yeah. I just prefer my designs to be more forgiving.
Not requiring people to tune their printers perfectly makes the designs more accessible.

@Efroymson done! Test print works great so far. 😁