build errors when building the documentation

output from running .\build.cmd
Creating C:\src\Okanshi\docs\output\reference
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.StopwatchExtensions.Time because: not a named type
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.AverageGauge because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.BasicCounter because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.BasicGauge because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.BasicTimer because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.DecimalGauge because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.DoubleCounter because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.DoubleGauge because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.HealthCheck because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.LongGauge because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.LongTaskTimer because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.MaxGauge because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.MinGauge because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.PeakCounter because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.PerformanceCounter because: the type '[]`1' does not have a qualified name
FSharp.Formatting Error: 0 : Error while building member-name for Okanshi.OkanshiMonitor.DefaultTags because: the type '[]`1' does not have a qualified name
FSharp.Formatting Information: 0 : Starting razor engine

This is apparently an open issue in FSharp.Formatting, see https://github.com/fsprojects/FSharp.Formatting/issues/364. Waiting for it to been fixed

This has been fixed while updating the build script in 0d3a3229eb5894eea4e2746f54c9dafdb752496e


Unit Tests

Unit tests running with UnitTest's MagicMock functionality blocking the kivy window creations

🚀 [bumpr] Bumped! New version:v0.0.2 Changes:v0.0.1...v0.0.2


How to embed the viewer into a QT window?

Hi all,
I'm new to MuJoCo and I'm trying to use MuJoCo for realtime rendering.
In my project I have made a client GUI using pyQT, for the ease of use I would like to combine the passive viewer and the QT GUI together as a single window.
Currently I am able to create the QT GUI and the viewer as two seperate windows, the code structure looks like this:
class MuJoCoViewerThread(threading.Thread):
    def __init__(self):
        super().__init__()

    def run(self):

        model = mujoco.MjModel.from_xml_path(filename=xml, assets=None)
        data = mujoco.MjData(model)
        with mujoco.viewer.launch_passive(model, data) as viewer:
               #do some tasks

class UDPServerWindow(QMainWindow):
    def __init__(self):
        super().__init__()
        self.initUI()

    def initUI(self):
        self.viewer_thread = MuJoCoViewerThread()
        self.viewer_thread.start()
        #do some tasks

Waht should I do to combine them to a single window? Ideally I would like the window look like this:

Hey @Zzx-J, I am looking for a solution as well for exactly what you asked. Were you able to embed mujoco into a single Qt app?

Sadly, no. I am still actively looking for solutions. My program currently uses 2 windows.

Hey @Zzx-J,
Check these out, they might help you out.
mujoco_qt
qmujocosim
mujoco_viewer

Hey @Zzx-J, Check these out, they might help you out. mujoco_qt qmujocosim mujoco_viewer

Awesome! The first one seems really helpful! Thank you very much!!!


select * from csv error

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:
select * from 'testing/data/csv/aggregate_test_100.csv'
Expected behavior
A clear and concise description of what you expected to happen.
return result successfully
Additional context
Add any other context about the problem here.

this problem is caused by for overflow int, arrow-csv still consider it as a int64

this problem is caused by for overflow int, arrow-csv still consider it as a int64

this problem is caused by for overflow int, arrow-csv still consider it as a int64


Allow empty location as no location info

Package github.com/lib/pq converts Postgres type "Date" to time.Time with zero value location, not nil.
So, scanning it by date.Date returns error.
However I believe this is an issue on github.com/lib/pq, we can add this patch to date.Date.
Regarding the nil location, golang itself already handles this and returns UTC.

@nasermirzaei89 Do you feel equipped to add a unit test for this case? (I'm fine with the change but want to make sure we have a test that FAILS without this change but passes with it.)

OK, I'll add the test.

@nasermirzaei89 Testing this is quirkier than I realized! I starting experimenting with time.Time{}.Zone() in my local checkout and it started to become a big thing!
I'm sending a PR with test-only changes, then your feature will be easier to test and add tests for.

@nasermirzaei89 https://github.com/hardfinhq/go-date/pull/13 will merge shortly and then you can merge it in / rebase on top of it

@dhermes as the input for the function is time.Time, this field shouldn't be time.Time rather than string?
https://github.com/hardfinhq/go-date/blob/main/convert_test.go#L78

@nasermirzaei89 The input is intentionally string so the test author can just write a valid RFC 3339 string and then the test code will handle parsing it into a time.Time{}.
But makes it impossible to handle named timezones, so I added that in #13.

@dhermes could you please take a look at my second commit in this pull request and if it's not the desired solution, I can revert it and continue with the #13 approach.

@nasermirzaei89 So sorry, I wasn't intending for you to refactor the existing tests! Definitely not the direction I want to take that test in. Sorry we didn't have a chance to discuss before you put that time in.
As for your code itself, you'll need to tweak it to using t.Zone() and I think just checking that the offset is equal to 0 should be sufficient.

@dhermes If I get #13 correctly, it has a timezone that translates the time to its location. So when we translate a time with time zone -0500 by -21600, it goes to timezone zero or UTC. So it converts to Date without error.

So when we translate a time with time zone -0500 by -21600, it goes to timezone zero or UTC. So it converts to Date without error.

The issue is that it's a timezone with no name, so the t.Location().String() check is a false positive. Instead you should be using _, offset = t.Zone() and checking offset = 0.

I've updated the code to use zone and offset. However, since TestFromTime makes some assumptions beyond the FromTime itself, I can't precisely test the lib/pq case.


Make our configuration great again

Add ggandor/leap.nvim
And I forgot

Also echanovski/mini.ai

And glepnir/lspsaga.nvim

Also add trail-blazer.nvim i fogot

actually, flit.nvim sounds cool!

done done done


vulkan: Setting WLR_RENDERER to vulkan stretches the images

Relevant issue - https://github.com/emersion/slurp/issues/121 (Resolved since it's wayshot)
It seems setting WLR_RENDERER to vulkan is causing wayshot to cause a problem with images.
Captured with wayshot:
https://user-images.githubusercontent.com/66054069/213844246-8dbc2d73-cf48-42b0-a0e4-4626b1ab74de.png
Captured with grim:
https://user-images.githubusercontent.com/66054069/213844441-61828ffa-18c9-450b-bbbb-4c3a67797edf.png

Relevant issue: https://github.com/waycrate/wayshot/issues/16

@uncomfyhalomacro What's you compositor version and which version of wlr are you on?

@Shinyzenith it seems it fixed itself for idk what reason


Remove version: Icedrive.Desktop version 3.11

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://cdn.icedrive.net/static/apps/win/IcedriveSetup-v3.11.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 897)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 897.)


Load Checkpoint files from bundle.

Is your feature request related to a problem? Please describe.
Since stencils are not being used anymore and given resource bundles can be achieved in SPM. Develop the functionality that allows the SDK to have JSON files with generated checkpoints for testnet and mainnet as the android SDK has.
Describe the solution you'd like

Stencil files are deleted if still available
Huge Switch statements are deleted
the SDK includes two folders: mainnet and testnet for the corresponding checkpoints files
the checkpoint files have 1 to 1 match with the GetTreeState JSON output from lightwalletd
the files are called {CHECKPOINT_BLOCK_HEIGHT}.json
their content is JSON
the current public API does not change for the users

Alternatives you've considered

provide the option for wallet developers to supply their own checkpoints at runtime.

Additional context

corrupted or maliciously created checkpoints can be a threat to the user's funds. Procede with caution.

This is for Daniel, who needs to be added to ZcashLightClientKit. IT request is aware.


Support for table attributes in output mapping

Would like to be able to set ui.maxSampleRows attribute for a table.
Some cases with large text/blob columns the storage console will freeze while fetching data, if the component knows that it has a table like this (emails for example) it can override the maxSampleRows to a low number for better performance.

@pivnicek this should go through productboard I suppose. The right solution is a more clever data sample.

Related to https://github.com/keboola/kbc-ui/issues/512


Cannot compose a Derived component

Describe the bug
Unsure if this is a bug or if I'm doing something incorrectly but I can't seem to compose a new classed component from derived component
To Reproduce
code sample:
import { classed, deriveClassed, ComponentProps } from "@tw-classed/react";

const ButtonBase = classed("button", {
  variants: {
    color: {
      red: "red",
      blue: "blue"
    }
  }
});

type ButtonProps = ComponentProps<typeof ButtonBase>;

const DerivedButton = deriveClassed<typeof ButtonBase, ButtonProps>(
  ({ children, ...rest }, ref) => {
    return (
      <ButtonBase {...rest} ref={ref}>
        {children}
      </ButtonBase>
    );
  }
);

const Anchor = classed("a", ButtonBase);
const AnchorWithDerived = classed("a", DerivedButton);

// This works
<Anchor color="blue">Anchor</Anchor>

// This doesn't work
<AnchorWithDerived color="blue">AnchorWithDerived</AnchorWithDerived>

Codesandbox: https://codesandbox.io/s/funny-matsumoto-18bjiz?file=/src/App.tsx:700-769
Expected behavior
<AnchorWithDerived color="blue"> should render <a class="blue">

This is indeed correct behavior.
const AnchorWithDerived = classed("a", DerivedButton);

Classed has no internals flow the variants upward, which is required to make the above work. I guess we can call this an anti-pattern.
Essentially what happens here is this:

You create a button base
You derive it (wrap in forwardRef and provide some typesafe utils)
You then class it using classed("a", DerivedButton)
Step 3 actually creates a new classed component and attempts to extract variants from DerivedButton
Since DerivedButton is just a typesafe forwardRef wrapper, it doesn't extract variants from ButtonBase.
Thus AnchorWithDerived does not do anything.

If you want to produce a default as prop you'll need to pass it in your deriveClassed generic.
https://tw-classed.vercel.app/docs/derived-components#experimental-as-manual-prop-support
I see this is a documentation issue on exactly what the classed() function does. I will attempt to make a visual of when extraction happens and when rendering happens. The former is what happens here.

There's also a type error when deriving a derived component using default as prop, so if you wan't to do that you must cast the ref tag as any:
// This works:

const AnchorDefaulted = deriveClassed<typeof DerivedButton, ButtonProps>(
  (props, ref) => <DerivedButton as="a" {...props} ref={ref as any} />
);

The deriveClassed api has some rough edges as I didn't expect this advanced usage of it, lets keep this open until its optimised.
Thanks for the issue :)


[Firebase background message handler in flutter spawns duplicate isolate and runs the app twice]

When my app uses:
firebase_core: 2.5.0
amplify_analytics_pinpoint: 0.6.12
firebase_messaging: 14.2.2
when calling FirebaseMessaging.onBackgroundMessage(...) make main method run twice =. it causes all plugins in my app to run twice.

@manhtran-itr

amplify_analytics_pinpoint: 0.6.12

This is a third party plugin for Flutterfire, so can you check by removing this plugin and its corresponding code in your app to see if you still get same behavior or not ?

I tried to remove third party plugin as you asked me for checking it but it still causes the same issue.  But when I remove FirebaseMessaging.onBackgroundMessage(...)  it doesn't happen anymore. Please help me to fix it.

@manhtran-itr
Make sure you have configured following:

targetSdkVersion & compileSdkVersion targeting android API 33.
android:exported="true" property set in app manifest on apps running on android 12 or higher.
@pragma('vm:entry-point') annotation just before calling background message handler.

@manhtran-itr Make sure you have configured following:

targetSdkVersion & compileSdkVersion targeting android API 33.
android:exported="true" property set in app manifest on apps running on android 12 or higher.
@pragma('vm:entry-point') annotation just before calling background message handler.

I checked all the information. My app configured those.

@manhtran-itr I think your issue probably is similar to #8175 or #9964 so please check that. Also see if this could be also one of the reasons as why you are seeing the behavior: #9816

yes, it's the same behavior. I read those and try all the solutions but it won't work.

Both the linked issues are open, so I suggest you to follow-up in them for further updates. You may upvote the linked issues and also share your findings for better tracking.
Closing this as a duplicate.


VS Code 1.14 notification

VS Code 1.14 provide new keys for coloring labels and buttons of heads-up notification
https://github.com/Microsoft/vscode/issues/28280

@ajitid Thank you. We already customised badge, foreground and background. I will fix the butotn background 👍


[Technical Question] Device Library for gateways?

Is the Device Library indicated to run in gateways? We need to connect a LoRaWAN network containing some hundreds of devices to Azure IoT HUB but I have the impression that this library is not made for this purpose.
I know there is the IoT Edge device that can be run as a gateway, but our gateway is not really on the edge but in the cloud and I wanted something more straightforward than the IoT Edge approach, requiring us to deploy modules in containers.
Thanks in advance.

We support using connection strings that have a GatewayHostName property specifying a gateway, and will use that information when connecting to the Hub.
Does that sound sufficient for your scenario?

Thanks Carter,
I didn't know there was such gateway property. The most important thing for a cloud gateway is to be able to send messages on behalf of other lower devices so we need multiple devices on IoT HUB to be updated from a single gateway. I was hoping to do so with this SDK by means of AMQP transmissions but now it looks like MQTT is the default technology used by this SDK. Am I right?

That is correct, we do not support AMQP, however I believe MQTT should be sufficient for your purposes so long as you have a connection string for your device that includes a GatewayHostName

Thanks Carter,

Documentation regarding this approach seems to be pointing to the use of an IoT Edge device as a gateway, but we want to host the gateway in the cloud, not in the Edge and avoid the hardness and limitations about deploying all the Edge stuff.

Since AMQP is not supported by this SDK, our gateway should create and maintain as many MQTT connections to the IoT HUB as devices connected to the gateway. This is not an option when we have hundreds of connections to maintain.

The issue is that MQTT can not be used in Azure IoT HUB in multiplexing scenarios (multiple devices being reported through a single connection). All the docs seem to be pointing to the use of AMQP in this case, and this SDK does not support AMQP. On the other hand, the Java and C Device SDK's are indeed supporting AMQP and device multiplexing so I think we are moving to one of those SDK's. Or am I missing something?

@dberenguer
While the documentation does focus on an Edge scenario, as long as you have a GatewayHostName in your Connection String, it should work regardless.
As for multiplexing, I will have to look into that and get back to you.

@cartertinney and @dberenguer  - The GatewayHostName field is not what you're looking for.  You use GatewayHostName when you want to talk to a gateway.  This conversation is about being a gateway.
@dberenguer - your understanding of multiplexing is correct.  MQTT does not support multiplexing.  For that, you would need to use AMQP, which does support multiplexing.  This library does not support AMQP, so you appear to be at a dead end.
The term that we use for this is "Identity Translation".  You can find some documentation here: https://docs.microsoft.com/en-us/azure/iot-edge/iot-edge-as-gateway?view=iotedge-2018-06#translation-gateways
IoT Edge has some functionality in preview to support this, but, because it's in preview, it's not particularly well documented or supported.  It's also expected to change before being released as an official feature.  Details are here: https://docs.microsoft.com/en-us/azure/iot-edge/how-to-publish-subscribe?view=iotedge-2020-11
IoTHub may also be supporting the notion of identity translation in some future release.

Thanks @BertKleewein and @cartertinney for your help.
For other people wanting to implement gateways, the C and Java SDK's do support AMQP and multiplexing. I don't know about the Node.js version though.


FileNotFoundError: No such file or directory: README.rst

$ ./bin/pip --version
pip 8.0.2 from /webapps/planexo/lib/python3.4/site-packages (python 3.4)

$ ./bin/pip install django-compat
Collecting django-compat
  Using cached django-compat-1.0.8.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "/tmp/pip-build-1wztbpt3/django-compat/setup.py", line 33, in <module>
        long_description=read('README.rst')
      File "/tmp/pip-build-1wztbpt3/django-compat/setup.py", line 17, in read
        return open(get_path(fname)).read()
    FileNotFoundError: [Errno 2] No such file or directory: '/tmp/pip-build-1wztbpt3/django-compat/README.rst'

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-1wztbpt3/django-compat/setup.py", line 36, in <module>
        long_description=read('README.md')
      File "/tmp/pip-build-1wztbpt3/django-compat/setup.py", line 17, in read
        return open(get_path(fname)).read()
      File "/webapps/planexo/lib/python3.4/encodings/ascii.py", line 26, in decode
        return codecs.ascii_decode(input, self.errors)[0]
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 8678: ordinal not in range(128)

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-1wztbpt3/django-compat

Thank you for reporting this issue, @SebCorbin. I have tried to fix the errors with the latest release. Please report back whether they still occur in your environment.


Feature Reqest: metadata callback value prevents toFile

When piping data there it's not possible to execute toBuffer in metadata callback:
img = sh();
img.webp().quality(75)
	.metadata((err, data) => {     // check image dims before processing
		if (err) return res.json({err});
		img.toFile('uploads/img', (err) => {
			res.json({err});
		})
	});
req.pipe(img);

in above example toFile is never called because of piping.
But when they are chained both of them work, so it would be nice if metadata callback returns false toFile execution is prevented.
img = sh();
img.webp().quality(75)
	.metadata((err, data) => {    // check image dims before processing
		if (err) return false; // this must prevent toFile execution
	})
	.toFile('uploads/img', (err) => {
		res.json({err});
	});
req.pipe(img);

I don't know if there is another way to do it now other than buffering data!

Hello, did you see the clone feature to split incoming piped image data into two Streams?

@lovell yes I did, the clone feature works on both instance at once, there is no way to stop one of them. I want for example check the width or height of a pipe in metadata and if it doesn't match requirements cease toFile or toBlob. for now I store the pipe in a buffer and process using the first code which doesn't work on a pipe!

OK, thanks for confirming, your approach seems sensible to me. Perhaps the feature outlined at #236 may also help.

@lovell I saw #236, and that is even better somehow as it's possible to change the output base on input. But it also has bring some complexities, I think beside that a false return from metadata works for most scenarios!

I've reworked your example a little, which may also provide what you need (note: untested):
const img = sharp().webp().quality(75);
img.metadata((err, data) => {
  if (!err) {
    img.toFile('uploads/img', (err) => {
      if (err) {
        res.json({err});
      }
    });
  }
});
req.pipe(img);

@lovell I tested, doesn't work! toFile is never called!

@rahbari Perhaps an err was passed to the metadata() callback?
const img = sharp().webp().quality(75);
img.metadata((err, data) => {
  if (!err) {
    img.toFile('uploads/img', (err) => {
      if (!err) {
        res.json('OK');
      } else {
        res.json({err});
      }
    });
  } else {
    res.json({err});
  }
});
req.pipe(img);

@lovell No, I didn't even check for metadata error. When piping only the main chain works!

How odd. Running the following locally seems to work for me:
const fs = require('fs');
const sharp = require('sharp');

const img = sharp().webp().quality(75);
img.metadata((err, data) => {
  if (!err) {
    console.log(data);
    img.toFile('img', (err, data) => {
      if (!err) {
        console.log(data)
      } else {
        console.log(err);
      }
    });
  } else {
    console.log(err);
  }
});

fs
  .createReadStream('input.jpg')
  .pipe(img);

Output:
{ format: 'jpeg',
  width: 2725,
  height: 2225,
  space: 'srgb',
  channels: 3,
  hasProfile: false,
  hasAlpha: false }
{ format: 'webp',
  width: 2725,
  height: 2225,
  channels: 3,
  size: 253128 }

@rahbari Were you able to make any progress with this?

Closing due to inactivity but please feel free to reopen if there's still a problem.


Use "system.autoUpgrade" with krops

This is not a real issue and maybe I'm just missing something… but what is the best way to achieve automatic system updates (like system.autoUpgrade) with krops?
Currently, I've pinned nixpkgs using niv and manually deploy (e.g. update) my machines. This has the obviously advantage of deterministic deployments because I'm not just tracking a channel like nixos-20.09 but, instead,  a specific point/hash of this channel. The downside is that system.autoUpgrade does not match this workflow.
How do you guys achieve automatic system updates? Updating nixkgs sources and deploying using a CI/CD system? Is this something where Flakes could help out?

I have one system I run with auto upgrades, the way I do it, is to not deploy nixpkgs with krops but have it as a channel. then system.autoUpgrade works as expected.
I guess flakes could also do that.
Also I have a repo where nixpkgs.json is created by a script with the current rev and then commited into the repo and pushed onto CI.


x/pkgsite: page with the stdlib code is not UX-friendly (as it was before)

What is the URL of the page with the issue?
https://pkg.go.dev/encoding/base64#Encoding.Decode which redirects to
https://go.googlesource.com/go/+/go1.16/src/encoding/base64/base64.go
But it doesn't matter, 'cause the problem is related to https://go.googlesource.com
What is your user agent?

Mozilla/5.0 (Macintosh; Intel Mac OS X 11_2_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:86.0) Gecko/20100101 Firefox/86.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0.3 Safari/605.1.15

Screenshot
Nothing to show.
What did you do?
I would like to see the source code for some stdlib packages. https://pkg.go.dev now redirects to https://go.googlesource.com but https://godoc.org (and early pkg.go.dev ?) has redirected to https://github.com
What did you expect to see?
Same features as on Github:

download 'raw' code
manually copy the whole file
see commits history
see git blame
file history (commits, links to the issues and so on)

What did you see instead?
Sadly current UX and UI looks like this:

Link to the raw code (named 'txt' at the end of the page) gives base64 of the file, which is a strange format for the code (txt!) to be read by a human.
Manual copy removes empty lines (same behaviour between 3 browsers, no addons) (UPD: copy file content on the page, paste into IDE or any other editor)
No link to commits
No link to git blame
Nothing at all :(

I understand why Go stdlib is served via https://go.googlesource.com but still it's too feature limited to make daily work easy and simple.
Thank you.
UPD: updated after https://github.com/golang/go/issues/44832#issuecomment-791937266 comment, my bad.

Link to the raw code (named 'txt' at the end of the page) gives base64 of the file, which is a strange format for the code (txt!) to be read by a human.
Manual copy removes empty lines (same behaviour between 3 browsers, no addons)
No link to commits
No link to git blame
Nothing at all :(

Not sure what you mean. I see all of this:

@fzipp my bad, it's. [log] and [blame] works as expected (whoever links to the issue aren't supported, just golang/goXXX or #XXX).
[file] redirect to the same page, but not a raw file as expected (only by me?)

This and #44829 are duplicates. (This issue has more engagement and detail from what I can tell).

an option would be codesearch: https://cs.opensource.google/go which I think has better linking

@seankhliao even better!


any use case for utilsScript?

Hi, Thanks for creating such a great component!
I just wonder if it is really necessary to load the libphonenumber.js using useScript. I highly suggest that the libphonenumber should just be used as direct dependence, it's just easier to use.
The user cases that I can come up with that anyone wants to use a different util Script are:

(1) They have a more performant utils function to provide these utils
(2) The logic for validating numbers or other things related to international numbers are changed

For situation (1), instead of injecting a different library, it will be easier and more straightforward just providing utils function hooks, like customIsNumberValid(), customNumberParse(), which will decouple your library and the external code.
If at the end the so called customUtilsFunction becomes an convention, you can just replace libphonenumber with that library.
For situation (2), if it changes, I think libphonenumber will just follow up and release a new version for it, you just need to update the version.
In conclusion, utilsScript is now just an obstacle standing in most of your users' way by trying to cover an unknow user case.
Let me know if I miss something : )

What if I don't want to require a 500+kbs library inside my project and use a smaller one, like https://github.com/catamphetamine/libphonenumber-js ?

utilsScript is removed in v6.0.0. #245


Feature/admin track progress

LJ - Added aditional Admin feature for tracking progress for the trip.

Closing PR as SonarCloud isn't picking up that new commit has new coverage.


DraggableScrollableController should dispatch creation in constructor.

Description
This PR adds DraggableScrollableController creation dispatching in the constructor for memory leak tracking according to https://github.com/dart-lang/leak_tracker/issues/134.
Tests

Updates draggable_scrollable_sheet_test.dart to test DraggableScrollableController creation dispatch;
Updates draggable_scrollable_sheet_test.dart to add disposal of DraggableScrollableControllers in tests;
Updates material/scaffold_test.dart to add disposal of DraggableScrollableControllers in tests;

Pre-launch Checklist

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[X] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[X] I signed the [CLA].
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is [test-exempt].
[X] All existing and new tests are passing.

@polina-c, @Piinks, thanks a lot for the review!


Use CI

@SergeyTeplyakov  I'm happy to do some CI work, what are you looking for outside of the basic restore-build-test?

I think just that. I was thinking just to configure azure devops pipeline to see if something is broken when PRs are created. I.e. absolutely nothing fancy:)

Could go with Github Workflows if that's fine with you, easy to setup and fully integrated w Github

Totally fine with me as well.


Pass app as a parameter to plugins

Currently to access the application within a plugin you have to use this in .attach.
This becomes painful when you use a function like _.bindAll on your plugin. Since you no longer have access to the app in your attach function.
It's also painful because I can't call other methods of the plugin in the attach function using this.
I suggest changing the signatures to

attach(options, app);
init(done, app);
detach(app);

This will give plugin writes a little more freedom in how they arrange their plugin and how they interact with the application.

+1 We'll probably change this in the v0.2.0 release. It will be breaking though so we have to be wary


There is a 2GB limit to the size of the nugget package, so push is not possible.

Describe the bug
I am trying to push to the baget server for packages over 2GB. Packages under 2GB are uploaded, but for packages over 2GB, the error "500 (Internal Server Error)" appears.
The way I'm using the baget is in a docker. So I took an error log of the docker and confirmed that it was an error, "System.IO.InvalidDataException: Multipart body length limit<PHONE_NUMBER> exceeded." Can't I upload more than 2GB of packages?
Please let me know if there is a way to set the related config.
To Reproduce
Steps to reproduce the behavior:

Using this version of BaGet : v0.4.0-preview2
the way to use Baget : Ubuntu 20.04, docker
Run this code : ../nuget.exe push -Source http://<IP_ADDRESS>:5555/v3/index.json -ApiKey  -Verbosity detailed *.nupkg
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

nuget push error log

my baget server error log

Additional context
Add any other context about the problem here.

I've been investigating this issue and have found its cause.
At this point https://github.com/loic-sharma/BaGet/blob/5a8502ca4e3b21acd369d4a98c2529dac2d81cfe/src/BaGet/ConfigureBaGetOptions.cs#L67 the max body length is set to<PHONE_NUMBER> which is approximately 2.14 GB. When processing a form which is larger than this value a InvalidDataException is thrown as shown by the docs.
The ideal fix would be to accept this value from the appsettings.json file, but a temporary fix would be to increase this value to a number greater than int.MaxValue (if you have a fork of this repo).
In terms of better error handling, I think it would make sense to catch the InvalidDataException in the controller and then return a bad request indicating that the max size of NuGet packages in currently int.MaxValue;

Fixed in this repo https://github.com/bagetter/BaGetter/blob/7c9f535e7bd6f49b49335de07e3aaa6ee3a0ecf6/src/BaGetter/ConfigureBaGetterServer.cs#L38


mypy.ini small bug

This isn't dedicated to this PR, but I would suggest a small change in mypy script
from
python3 -m mypy .
to
python3 -m mypy ./pytest_motor ./test_data ./tests
because otherwise mypy scans venvs directory.

The mypy.ini has an exclude line to prevent that:
exclude=/venvs/

And generally, I prefer excluding files to check rather than including because it is very easy to add new files and not realize that mypy isn't checking them if you list includes.
Originally posted by @AustinScola in https://github.com/AustinScola/pytest-motor/issues/36#issuecomment-881872646

Maybe you misinterpretted my comment? I meant that mypy.ini already has an exclude:
https://github.com/AustinScola/pytest-motor/blob/7da7da0e67b7c2c826f4cd2b49d32a975307b28b/mypy.ini#L5

Here is the thing:
venvs isn't the top directory, so exclude=/venvs/ doesn't exclude venvs folder, it should be exclude=venvs/ instead.
Here is the image with and without the change on my PC:

Ahh, you are right. We either want ./venvs/ or venvs/!

Ahh, you are right. We either want ./venvs/ or venvs/!

./venvs/ also doesn't work for some reason (at least, on my machine) but venvs/ does

Might need some escaping? Docs say that it is a regex.

Might need some escaping? Docs say that it is a regex.

I'm not very good at regex and can't find any way to make ./venvs/ work. I think, though, venvs/ might be the thing.

Sounds good to me. Thanks for finding this!


Monetization: Paid perk / Subscription / Status etc

Is your feature request related to a problem? Please describe.
In the long run, if nostr succeeds, nostr.watch should have a way to monetize independent of grant sources.
Describe the solution you'd like
I am not sure what it is, but nostr.watch should have a paid feature, status, perk etc. My gut feel is that a freemium model is the way.
It could be that paid relays:

can afford to share some sats flow with nostr.watch
would be interested in having premium positioning on nostr.watch listings

I suppose some may call this the ad model.
Describe alternatives you've considered
Rely on the goodwill of @dskvr and Opensats forever.
Additional context
What are other ways might we monetize nostr.watch?


UI always renders at DPI scale of primary screen, not current screen

VSCode Version: Code 1.14.1 (2648980a697a4c8fb5777dcfb2ab110cec8a2f58, 2017-07-13T19:18:47.188Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions: none

Steps to Reproduce:

Have a hardware setup where your primary display is at a different DPI scale than your external
Launch VSCode and move the window to the secondary display

Expected:
App content renders at the DPI scale factor of the external display
Actual:
App content continues to render at the DPI scale factor of the original display. Here's an example where my primary display is at 100% but my secondary is at 225% (exaggerated slightly to make the problem extra obvious). Note the radical difference between the title bar and the entire rest of the UI:

https://github.com/Microsoft/vscode/issues/6196


Update jib plugin version: 0.9.7 -> 0.9.8

Not really necessary but I want it to be consistent with Maven, which uses 0.9.8.

Maybe we can update the other version numbers in this PR too (https://github.com/GoogleContainerTools/jib/issues/765#issuecomment-409694279)

Updated. See if this makes sense.


Reorganize Concepts

Remove everything that's not a conceptual overview. Separate task and reference information and move them to the Operator Guide and the Reference section respectively.
This issue tracks recommended changes resulting from an analysis of the KEDA documentation commissioned by CNCF. The analysis and supporting documents are here: https://github.com/cncf/techdocs/tree/main/assessments/0011-keda/
Use-Case
The conceptual overview (also variously called a technical overview or architectural overview depending on the product) describes how the product works at a high level. It does not go into detail (for example, it might use pseudo-code; providing syntax is not the point here); rather, it tries to help the user build a mental model of the product's architecture and subsystems so that tasks and reference information will "make sense" and "stick" when the reader goes on to use the product.
Specification
Here is a proposed outline. Links are to existing pages that can be used as-is or provide a starting point.

KEDA Concepts

What is KEDA?
How KEDA works
KEDA Architecture
KEDA Custom Resources (CRDs)
Scaling Deployments, StatefulSets and Custom Resources

Overview
Scaling Deployments and Stateful Sets
Scaling Custom Resources
Scaling Jobs
Authentication
External Scalers
Admission Webhooks

Hello! May I work on this issue?

Certainly!

Thank you @tomkerkhove!
@dwelsch-esi, before start working on this issue, I want to clarify some points.
I see in KEDA Concepts page,there is these sections available -

What is KEDA?
How KEDA works
KEDA Architecture
KEDA Custom Resources (CRDs)

However, there is another two sections -

Event sources and scalers
Deploy KEDA are

Shall I remove them? Or should I move it to another page? Or just leave them at the bottom of concept page?
For Scaling Deployments, StatefulSets and Custom Resources section -

Scaling Jobs
Authentication
External Scalers
Admission Webhooks

These pages are too big to move them inside Scaling Deployments, StatefulSets and Custom Resources page, then shall I just have a small pera about them and then put link to their pages?

@mhmohona Good questions. Below are a couple of general comments, then responses to your questions.
General comment 1: My outline and suggested improvements are guidelines. Don't follow them to the letter if you see a better way. The important thing is to create an overview that introduces KEDA to new users, explains what it's for, outlines its architecture, and describes its components. Just rearranging the existing material might not be sufficient. You should feel free to revise any parts that you don't think are clear, and write new sections if you think something is missing.
General comment 2: The links you provide are to the 2.13 version of KEDA. It's probably best to work with the most recent published version and/or the upcoming release.
On to your questions:
"Event sources and scalers" IMO should be a reference page. Readers don't need a list of scalers to understand the product; a couple of examples should suffice. I'd describe what a scaler is, give an example, and link to the full list in case they're curious.
"Deploy KEDA" is a link to instructional information about how to get started. That information is included in the main table of contents. I'd leave it out of a conceptual overview, so yes, you can remove this section.
I wouldn't try and fit everything on one page. In fact, I'd try to keep to one page = one topic. My outline is meant to show the information structure for the conceptual overview and not to literally define the pages. I'd probably write this part in a series of separate pages, one page per topic, then link them into a sequence (with "previous" and "next" buttons at the bottom – the doc framework might even do that automatically). Also have a top-level landing page where you can access any of the page from a list of links. (So yes, your suggestion to include a short paragraph about each one and link to their pages is the right idea IMO 😄 ).
/cc @tomkerkhove

I'm not sure this has been completed, if it was just the bot that closed this could we reopen it?


Nested projection using wildcard problem

OrientDB Version: 3.0.31
Java Version: 1.8
OS: linux
Expected behavior
If there is a name attribute then I would expect na* in a nested projection to include something like "name":"some name"
Actual behavior
In my case it returns "na":null
Steps to reproduce
create class Test_1 extends V
create class Test_2 extends V
INSERT INTO Test_1 CONTENT {"name": "Joe", "surname": "Miner"}
INSERT INTO Test_2 CONTENT {"name": "Jay", "surname": "Miner"}
CREATE CLASS test_edge1 EXTENDS E
create edge test_edge1 from (select from Test_1 where name ="Joe") to (select from Test_2 where name ="Jay")

select out(test_edge1):{*} from Test_1  --> Works
select out(test_edge1):{*,!in_*} from Test_1  --> Works
select out(test_edge1):{!in_*} from Test_1  --> Works
select out(test_edge1):{name} from Test_1  --> Works
select out(test_edge1):{surname} from Test_1  --> Works
select out(test_edge1):{!na*} from Test_1  --> Works
select out(test_edge1):{*,!na*} from Test_1  --> Works

select out(test_edge1):{na*} from Test_1  --> Doesn't Work
select out(test_edge1):{sur*} from Test_1  --> Doesn't Work

I don't think this bug report should be labeled as a question, at least I meant to provide a bug report

I agree with you. I'm flagging it as a bug, I'll try to check it ASAP
Thanks
Luigi


Multiple markers dynamically

I know blade tags dont have to use the {{ and }} to pass data
https://github.com/LarsWiegers/laravel-maps/issues/6
Currently, there's no way to add markers dynamically. For instance, I query the DB and get an array of coordinates. Currently you cannot loop through each location to place a marker on a single map (not multiple maps).
May be there's a way but I can't find it.

can you give some example code of what you are trying to do?

Sorry for the late reply. If I wanted to make place multiple markes on the SAME map, it won't work:

<x-maps-leaflet :markers="[
@foreach ($locations as $location)
['lat' => $location->latitude, 'long' => $location->longitude',
@endforeach
]"></x-maps-leaflet>

So it is probably a good idea to this loop in your controller that way you can pass a single array to the component.

So it is probably a good idea to do this loop in your controller that way you can pass a single array to the component.

Well, yeah. I actually did that after the first code I posted didn't work. I didn't work, the map was blank for some reason.

Without more code / errors in the console you are getting its going to be hard for me to diagnose the problem

Alright...

can you give me an example of how you would make this loop as i keep getting the error: Undefined array key 0


Setup semantic-release

Setting up semantic-release is perfect for this library, because it means we're very strict about SemVer and don't have to think about it. Might also have to setup commitizen?

Done in #7, including commitizen!


[I18N] Add traditional Chinese(zh-TW) translation file

@RocketChat/android
Add traditional Chinese(zh-TW) translation file.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


force coin gecko IDs to be undefined

Currently the chainRegistryChainToKeplr returns currencies with coinGeckoId: "". This leads to errors when doing things like adding chains to Kepler.
Context here https://github.com/chainapsis/keplr-wallet/issues/657 if you're interested.

thanks @0xArbi !!


Velocity resets when an object is disconnected from joints.

Hey, I'm working on an XR interaction toolkit that uses joints to connect the hand to objects. When the joint is unset and disconnected from the object the hand is holding, it appears to lose all velocity while in Godot Physics it keeps its momentum.

I'm not able reproduce this issue from your description. In fact, I use this method for the picking in the example scenes camera, and when unsetting the joint nodes it keeps the velocity just fine, allowing you to flick the objects away.
Would you be able to share a minimal repro project that showcases your problem?

Weird. I'll double check that the bug is reproducable in my project before providing a minimal project.

FYI, this very much sounds like: https://github.com/jrouwe/JoltPhysics/discussions/268

FYI, this very much sounds like: jrouwe/JoltPhysics#268

That looks like exactly what I'm running into! I'm using the built in Generic6DOFJoint node but that looks to be the exact same.

Right, then I suspect you're likely making the same mistake I did when I first implemented the picking, by using a StaticBody3D as your hand. If you try using an AnimatableBody3D instead you should see better results.

Noted, testing out Jolt with this change.

Damn, can't test it inside my main project since it's in 4.3, but I'm gonna go ahead and assume that was the issue since I'm using StaticBody3D nodes. Closing.


WITH RECURSIVE clause may define non-recursive CTEs

Describe the bug
DataFusion did not recognize this scenario, treating it uniformly as recursive CTEs, which resulted in incorrect results.
This is a reasonable usage. Users can define multiple CTEs using a WITH clause, one of them might be recursive, while others are not.
 Query took 0.016 seconds.

Expected behavior
The result should be similar to PostgreSQL, with one row instead of zero rows.
psql=> WITH RECURSIVE cte AS (
    SELECT a FROM (VALUES(1)) AS t(a) WHERE a > 2
    UNION ALL
    SELECT 2
) SELECT * FROM cte;
 a
---
 2
(1 row)

Additional context
The following is documentation from BigQuery. https://cloud.google.com/bigquery/docs/reference/standard-sql/query-syntax#recursive_keyword
A WITH clause can optionally include the RECURSIVE keyword, which does two things:

Enables recursion in the WITH clause. If this keyword is not present, you can only include non-recursive common table expressions (CTEs). If this keyword is present, you can use both recursive and non-recursive CTEs.
Changes the visibility of CTEs in the WITH clause. If this keyword is not present, a CTE is only visible to CTEs defined after it in the WITH clause. If this keyword is present, a CTE is visible to all CTEs in the WITH clause where it was defined.

take


Uncaught TypeError: t.request is undefined

Problem Description
Seems like Instagram app crashes mitmweb
Steps to reproduce the behavior:

Click some items in the Instagram app
Type some keys into Google Search also, to add more requests
Try scrolling - entire screen goes white, and above error in console

System Information
Mitmproxy: 7.0.3 binary
Python:    3.9.7
OpenSSL:   OpenSSL 1.1.1l  24 Aug 2021
Platform:  Windows-10-10.0.18363-SP0

Could you please post a full stack trace?

I dont know how to do that

Where do you see the error message? There should be some more context. Are you seeing this in mitmproxy or mitmweb? For mitmweb, maybe open your browser's developer tools and check the JS console for a detailed error.

ok give me a minute - I have to set up the emulator again

Yes, thank you!
I believe this should be fixed in the latest development snapshots already, which you can get at https://mitmproxy.org/downloads/#branches/main/. Or alternatively you wait for the next release. 😃

I do not believe this issue has been resolved in the latest release.
I am running the latest official docker image (7.0.4) as mitmweb in reverse mode
The webpage initially loads fine at first but as soon as a client connects then the page goes white.
Refreshing the browser brings back the webpage for a split second then it immediately goes white again.
During this the proxy still functions as expected. It is just the web portion that is failing.
Let me know if you need more info or if this should be in it's own issue.
Console log on Firefox

Console log on Chrome

Ah, actually i think there are multiple closed issues for this already.
This one suggested that the fix would be in the next release, which would be 7.0.4, but in the other issues, it is said it is not released yet. Only available in main branch.
Of course i read the wrong previous issue ticket first and responded, heh
I will give the master branch a try and follow up if the problem still exists.


Do we need to add another base token different from OCEAN token?

I got a question after some fixed-rate-exchange README refactors if we can create an exchange and buy data tokens using another base token address different from the OCEAN one. Do we need this right now in ocean.py?

Agreed, it's considered as low prio task for the moment. Then #532 can be merged after a review.

This is done in v4 and any base token can be used. @trentmc do you want us to add to the FRE readme explaining that?

I could add in the READMEs extra info regarding this aspect.


Provides the hability to include tempusdominus js from local service

Hi,
It would be nice to add some settings to the app to use install tempus dominus locally with node, and avoid CDN, like done in https://github.com/tutorcruncher/django-bootstrap3-datetimepicker by example.
You could add a dependancie to django-appconf to provide settings...
Thanks !

This feature has been included the latest release. Cheers!


fix: return error for  unsupported SQL

Which issue does this PR close?

Closes #3873
Rationale for this change

Refer to this issue #3873
What changes are included in this PR?

Make some unsupported SQL queries return errors
Are there any user-facing changes?

no.

Looks good @Kikkon  -- thank you!
Is there any chance you can write some tests for this code?

Is there any chance you can write some tests for this code?

Sure, I will add tests later.

Some tests related to create view depend on physical plan & logical plan, change to use only logical plan


mask APr problem

Hi @mlzxy ,
I am working on the OVD task, and when I evaluate the LVIS dataset, I set MASK_ON to True, but the evaluation results do not show the mask APr results for the mask task; only the evaluation results for the box task appear. How can I resolve this issue?
best，
hongliang

Have you used MODEL.MASK_ON True or just MASK_ON True?  The first one is correct.

Have you used MODEL.MASK_ON True or just MASK_ON True? The first one is correct.

Yes, I set the command line as 'task=ovd dataset=lvis bash scripts/train.sh MODEL.MASK_ON True', but the mask APr did not appear correctly during evaluation.

Could you try setting a break point at this line and see whether it was reached?

Could you try setting a break point at this line and see whether it was reached?

Hello, I found an issue. I'm using your new model framework devit_update.py, and it seems that the code doesn't include if self.use_mask: . Could this be the reason why the mask isn't working?

Yeah I haven’t added that part yet.
On Oct 29, 2024 at 12:35:59 PM, ZhouHongLiang6 @.***>
wrote:

Could you try setting a break point at this line
https://github.com/mlzxy/devit/blob/9cab390fbe78b3bc8d62c224411dce1c71056cf9/detectron2/modeling/meta_arch/devit.py#L1473
and see whether it was reached?
Hello, I found an issue. I'm using your new model framework
devit_update.py, and it seems that the code doesn't include if
self.use_mask: . Could this be the reason why the mask isn't working?
—
Reply to this email directly, view it on GitHub
https://github.com/mlzxy/devit/issues/69#issuecomment-2444794943, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2OBPXM2M6YAWAAGSLHQO3Z562O7AVCNFSM6AAAAABQZ4WVFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBUG44TIOJUGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


add proxy env for each agent addons

task: https://github.com/open-cluster-management/backlog/issues/16121
Signed-off-by: Zhiwei Yin<EMAIL_ADDRESS>

@rokej Will this change be easily consumable by AppMgr?

@jnpacker It is now apparent to me how HTTP_PROXY, HTTPS_PROXY and NO_PROXY environment variables are made available to appmgr container but if https://github.com/open-cluster-management/klusterlet-addon-controller/pull/60/files#diff-3e6c977cb9b2d1b24ff2d0e765c862c9b3a04aa7789f129cdf83ec382ee69265R80 this line does it, YES.

/unhold

/lgtm


add run_once command

Run_Once
tg-signer run_once [chat_id] [send_text]
For a one-time sign-in, which can be used for crontab scheduled tasks.Before use run_once, you need to use login command to login telegram.
when you use login,you can get the chat_id you want.
send_text is send to chat_id's content,like "/sign"

第一次开PR，感谢大佬指点

main分支增加了run_once和send_text命令，有机会可以测试下


Update to holochain_persistence_api 0.0.8

PR summary
Closes #254
Review checklist

[ ] The story has unit or integration tests
[ ] No new bugs, and any tech-debt is identified and justified
[ ] There is enough API documentation (how to use)
[ ] There is enough code documentation (how the code works)

why is this not merged?  I thought it was deployed.

why is this not merged? I thought it was deployed.

I deployed the new versions for persistence_* but wanted to follow proper procedure for lib3h.


Reland - Wire up hot restart and incremental rebuilds for web

Reland https://github.com/flutter/flutter/pull/33197
Blocked by https://github.com/flutter/flutter/pull/33225

Tests?

This did a lot of damage to our consumer_dependencies_count benchmark...


Baremetal  auto  enlist/discover

now we can add hardware by only one way:  collect the mac and assign ip  in  advance.
Is there any plan to add a feature that  hardware can enlist automaticlly?

Dupe of #178. @jacobweinstock to close.


Validate only returning first error

What version of Ajv are you using? Does the issue happen if you use the latest version?
6.5.4
Ajv options object

JSON Schema

{
   "properties": {
       "email": { "type": "string" },
       "password": { "type": "string" },
        "firstName": { "type": "string" },
        "lastName": { "type": "string" }
    },
   "required": [ "email", "password", "firstName", "lastName" ],
   "type": "object",
   "definitions": {}
}

Sample data

{}

Your code

https://runkit.com/myrddraall/5bc4e37afa061f0012b5bf75
const ajv = new Ajv();
ajv.validate(schema, data);
console.log(ajv.errors);

Validation result, data AFTER validation, error messages
[ { keyword: 'required',
    dataPath: '',
    schemaPath: '#/required',
    params: { missingProperty: 'email' },
    message: 'should have required property \'email\'' } ]

What results did you expect?
I would expect an error for each of the required fields rather than only email
Are you going to resolve the issue?

That’s a feature, see options.


Update DynamicButtonPathVector.swift

This is to fix the following problem in Swift 4 : https://github.com/yannickl/DynamicButton/issues/19

Thank you for your help!


Update feature request template

add impact description.
add prompt to list example usecases

could we also add "Self-Service" option to know if person wanna contribute themself in fixing the bug or enhancing the feature example inspiration: https://github.com/facebook/docusaurus/blob/main/.github/ISSUE_TEMPLATE/feature.yml

could we also add "Self-Service" option to know if person wanna contribute themself in fixing the bug or enhancing the feature example inspiration: https://github.com/facebook/docusaurus/blob/main/.github/ISSUE_TEMPLATE/feature.yml
other thing is "Priority" like whats the priority of the issue to be fixed (by us or themself)
May be better to have a template in google doc 1st and get reviewed in terms of content with a CD

really great suggestions! ill have that in a gdoc and land this first since its approved and changes wouldnt break anything


xml view expression binding behaves weirdly if one of the parameters returns undefined

Found with: SAPUI5 1.28.1
take the following example:
var jsonModel = new sap.ui.model.json.JSONModel({
showButton: true
});

<Button
           text="test"
           visible="{= ${myModel>/showButton} && undefined)}"/>

As you can see the visible property of the Button is set by an expression binding. As the second parameter in the logical && is a literal undefined the expression should always evaluate to undefined which coerces to false, so the button should never be shown.
What happens instead is that the second parameter undefined does not get evaluated at all and the button still remains visible. Only If I set showButton to false then the button get's invisible.
This is of course just a simplified example. The general bug is that whenever a part of an expression  returns undefined this part seems to be ignored for the determination of the eventual result  (at least I can reproduce this for boolean expressions with the && operator).
If you need a live code example please tell me where I can put it.
Thanks and Best Regards Christian

here's a live example in jsbin (using latest openui5beta):
http://jsbin.com/datoza/1/edit?html,output

This IMHO is not a bug in the expression binding. Expression binding works straight forward here and should determine a value of "undefined" for the above binding. It explicitly doesn't do type coercion, nor does the UI5 runtime. Instead, data binding interprets "undefined" or "null" as "reset property to default value" which is true for most "visible" properties. And, BTW, this is also the reason for not implementing type coercion here. It would break the important "reset" feature of the data binding.
You would observe the same by calling oButton.setVisible(false).setVisible(undefined);
You could modify the expression so that it delivers the result that you expect ( (your expression) || false) , but I don't see that this should be fixed in the expression binding.

ok, got it. Was still some gotcha to me and my colleagues. Is it somewhere documented that undefined resets the value to it's default?

I fixed it btw in our case by doing !!{myModel>/potentiallyUndefinedProperty} which coerces undefined to false explicitly.

That's indeed nicer and shorter :-)
To be honest, I had to search for the documentation first and the place might not be the most obvious one: ManagedObject.validateProperty:
	/** ...
	 * In case null or undefined is passed, the default value for
	 * this property is used as value. If no default value is defined for the property, the
	 * default value of the type of the property is used.
	 */

ok thanks :-)
Would be good if you can add this to a more obvious place in the guides and add also some example where undefined is passed to a setter.

I've created a internal incident<PHONE_NUMBER>. You can check its status on GitHub.

Hi,
the documentation is already adapted. Please see the API reference.
BR,
Jan


Extract and inline the minimal CSS to style the rendered App Shell.

Implemented by @topaxi. Thanks!


Quick switch between shapes (rect <-> circle, etc.)

Add the ability to switch between different shapes quickly, keeping the original shape style and bound text size.
The current way of doing this includes a lot of manual steps:

turn on snapping and create a new shape of a similar size
copy the styles from the original shape and paste the style into the new shape
unbind text from the original shape and bind it to the new shape (usually the size of the bound text is different than the built-in size)

Instead, changing from rect <-> circle/diamond should be doable with a single click of a button, for all the selected elements.

so basically you want to copy past the text  when we give styles to new shapes , currect me if i am wrong? because when we copy the style from one shape and past it to another the binded text does not apper right?

Not exactly, as copying the styles (with or without the bound text) implies that you are going to apply the styles to another element, that you manually created.
Instead what I am suggesting is to have the possibility to change the shape/type of all the existing element/s without creating a new one. It should also preserve all the bindings, position, z-index, text wrapping, etc. Given that we could come up with some simple & reasonable UX.

hi @Mrazator  let me know your, is it fine?

Hey @Mrazator, can i start working on this issue.

Hey @Mrazator, can i start working on this issue.

@its-kunal There is a PR waiting to be merged delivering this functionality: #8533
You can check it out and test it if you'd like to help.


Add Cyborg screams and set emote pitch to 0.125

About the PR

Adds 4 screams to cyborgs. They're just edited versions of malescream_1.ogg thru malescream_4.ogg.
Very crunchy, very cyborg-y!
Sound files were edited by @Kezu02
Why / Balance

Now cyborgs can scream just like EVERY other normally playable mob in the game.
Discussion here: https://discord.com/channels/310555209753690112/1228890407749025854
Variation was changed to 0.125 to match the other emote variations.
Technical details

No code change, only .yml editing
Media

https://github.com/space-wizards/space-station-14/assets/144283718/1284cc46-df24-4fa4-a6ff-2fe993a2731d

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Cyborgs can now scream!

why does it sound like it's saying "raaauughhh" or "raaawwwrr", its weird. I would prefer it to sound a bit more flat.

why not add the screams from TG?
Also, these sound like the tubes you flip over that make the sound, if you know, you know

why not add the screams from TG? Also, these sound like the tubes you flip over that make the sound, if you know, you know

I didnt know /tg/ had borg screams. Could you link them in the discord discussion?

Not tg, my bad. They're from goonstation. I see now you linked them in the discussion. They sound much more robotic and fitting for borgs IMO.

I kinda prefer a short version of those goonstation robot screams

I don't like any of these (neither the ones here nor any suggested in the discussion) for the simple reason that cyborgs don't have an inherent gender. Some cyborgs want to be male, some female, others genderless. So any sound used should accommodate that.
I think if cyborgs get a scream (and they should, it's funny) it needs to be something that perhaps isn't a voice at all but a synthetic sound like the talk sounds already are. I wonder if @MilenVolf has any ideas for a scream, they made some new sounds for #27205.
Or alternatively, just randomly play one of the male or female sounds in a way that make them sound like it's playing back poorly made recordings of people screaming (noisy, low bandwidth, etc).

Try this sound filter with the moth screem, there's a chance it'll be way better than the human scream because it's more of a bleet which may be more inline with what a screaming Borg would sound like

Really dont like these screams. Prefer those goonstation ones.


sparkleshare: Remove fallback to Flatpak image

The upstream .desktop file has a fallback to flatpack. It is only intended to run if sparkleshare isn't in the user's path. But to make matters even worse it will also run if sparkleshare exists with a failure code.
Remove this fallback to ensure that users don't download and run unpinned and untrusted code from the internet.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@moritzschaefer can you review?

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

1 package built:

sparkleshare

Never reviewed on this repository/project so not sure if there are special guidelines but I had a look and it looks fine to me!

No strong requirements. I just grabbed you from and old commit to this package.


Add .install files for directories

If a directory has any files updated in it then we should execute .install files for that directory.

+1


ScrollView autohide keyboard is laggy on low grade devices

Description
ScrollView with on-drag keyboardDismissMode freezes for approximately 200-300ms on low grade devices before keyboard hide animation is triggered (iPhone 5 for example). This issue cannot be reproduced on normal devices.
React Native version:
0.65.1
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Focus on on any input so that the keyboard becomes visible
Scroll the view

Expected Results
No lag, freeze, etc
Snack:
https://snack.expo.dev/@vlad_matvei/scrollview-bug-repro

Added snack


Add a basic SMF manifest for SunOS platforms

Hi,
A basic SMF file which might come handy for people use SunOS platforms like SmartOS, OmniOS, etc.
Regards,
Antonio Huete

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4238   +/-   ##
=======================================
  Coverage   19.96%   19.96%
=======================================
  Files         153      153
  Lines       30521    30521
=======================================
  Hits         6094     6094
  Misses      23513    23513
  Partials      914      914

Continue to review full report at Codecov.

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

Nice, that saves me the effort of writing one. This one works perfectly on SmartOS.

@sjorge I've also made a port in pkgsrc WIP :-)

@tuxillo nice, I could not find it on the joyent/pkgsrc-wip repo though?

@techknowlogick need your approval

@sjorge it's in pkgsrc-wip (https://github.com/NetBSD/pkgsrc-wip/commit/6f84130e7cb5ec214241a861b2a450db57613a78)


feat(nodes): simple custom nodes

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[x] Yes
[ ] No, because:

Description
feat(nodes): simple custom nodes
Custom nodes may be places in $INVOKEAI_ROOT/nodes/ (configurable with custom_nodes_dir option).
On app startup, an __init__.py is copied into the custom nodes dir, which recursively loads all python files in the directory as modules (files starting with _ are ignored). The custom nodes dir is now a python module itself.
When we from invocations import * to load init all invocations, we load the custom nodes dir, registering all custom nodes.

This is a super minimal MVP and can be easily changed or ripped out. It doesn't handle anything besides just loading python files as modules. Loading is handled as indicated in the python docs for programmatic loading of modules.
Custom node installation method
You can just paste custom nodes into the folder, but ideally the nodes are in a git repo which can be cloned into the folder. A git repo is the easiest way to facilitate updates to custom nodes.
There is no logic in this PR to update git repos - that's on the user to go into each one and run git pull, but maybe we could also maintain a shell script to to walk the dir & git pull all repos found?
Type hints issue
One issue - static type checkers (pylance, mypy) apparently can't analyze imports from editable installs. If your venv has the invokeai package installed in editable mode, you won't get type hints in the custom nodes. This means it's unpleasant to dev custom nodes if you also work on the app itself and use an editable install.
You can do pip install . from the repo root to make it a normal install, then you'll get type hints in the custom node definitions.
QA Instructions, Screenshots, Recordings
Create some custom nodes and put them into the custom nodes dir. Here's a simple one:
from invokeai.app.invocations.baseinvocation import (
    BaseInvocation,
    InputField,
    InvocationContext,
    invocation,
)
from invokeai.app.invocations.primitives import IntegerOutput

@invocation("my_test_invocation")
class MyTestInvocation(BaseInvocation):
    a: int = InputField()

    def invoke(self, context: InvocationContext) -> IntegerOutput:
        return IntegerOutput(value=self.a)

Be sure to give them different types in the @invocation() decorator, and name the classes differently if you copy and paste multiples.

Added/updated tests?

[ ] Yes
[x] No : I don't know how to test this but it's probably a good idea

Closes #4950

Is there any way to limit inclusion to some files, or to scan for a particular file first and only include that if it exists? I'm thinking that a node author might not want to include all node support files or user-defined node subdirectories.

Updated structure based on feedback, see the new README


[5.7] Simplify collection average

Let's take a look at the Collection::avg() method!
We have already got a valid callback via the Collection::valueRetriever() method, so we needn't to create a new Closure.

If it is not broken let's just leave it.


How to load yml registry configuration when using docker-compose

Hey,
I'm trying to figure out how to configure the registry configuration using yml syntax when running a container with docker-compose.
I understood one could use the environment fields in docker-compose.yml to override the container's environment variables, but this is terrible as it doesn't use the easy-to-use yml syntax.
Thanks.

I ended up mounting the config.yml to /go/src/github.com/docker/distribution/cmd/registry/config.yml
hope this helps someone.

@c0rnflex

but this is terrible as it doesn't use the easy-to-use yml syntax.

A compose file is YML.
Furthermore, we are likely going to deprecate the use of config.yml for the registry, and allow only environment variables. YML is a terrible idea for config files, regularly tripping people over.
See for example http://12factor.net/config


[FLINK-34066][table-planner] Fix LagFunction throw NPE when input argument are not null

What is the purpose of the change
Fix LagFunction throw NPE when input argument are not null.
This issue is related to https://issues.apache.org/jira/browse/FLINK-31967. In FLINK-31967, the NPE error has not been thoroughly fixed. If the select value  LAG(len, 1, cast(null as int)) and  LAG(len, 1, 1) exists together in test case AggregateITCase.testLagAggFunction() as:
val sql =
  s"""
     |select
     |  LAG(len, 1, cast(null as int)) OVER w AS nullable_prev_quantity,
     |  LAG(len, 1, 1) OVER w AS prev_quantity,
     |  LAG(len) OVER w AS prev_quantity
     |from src
     |WINDOW w AS (ORDER BY proctime)
     |""".stripMargin

before is:
val sql =
  s"""
     |select
     |  LAG(len, 1, cast(null as int)) OVER w AS prev_quantity,
     |  LAG(len) OVER w AS prev_quantity
     |from src
     |WINDOW w AS (ORDER BY proctime)
     |""".stripMargin

NPE will throw.
Brief change log

Fix LagFunction throw NPE when input argument are not null
Adding test to cover the npe case.

Verifying this change

Adding test to cover the npe case.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? no docs

Why are we no longer testing the previous behavior?

Thanks, @MartijnVisser . I add more details about how the NPE error will be produce.  Thanks for kindly remainder.

@flinkbot run azure


Support for the Glorious Model I

Is your feature request related to a problem? Please describe.
Currently, my profile is saved properly from my previous windows install with the exception of the 2 extra mouse buttons not being registered. I'm unsure if this is addressable inside the context of libratbag, but I'm open to be as cooperative as possible in developing a solution.
Describe the solution you'd like
I would like the extra buttons on this mouse to be properly programmable. RGB does not concern me particularly, but I'm guessing that would be on the agenda as well.
Describe alternatives you've considered
I tried checking the raw input of the mouse with the intention of simply writing an AHK script or similar, but it looks like my linux distro doesn't even register the buttons being pressed, indicating there might be firmware issues? I'm not super savvy with this.
I was able to find the information from lsusb as expected.
Additional context
OS: EndeavourOS

Hello! There is #1327 already for Model I.
It's protocol is completely different from other Glorious mice, and we would need to write a completely new driver for it.

I tried checking the raw input of the mouse with the intention of simply writing an AHK script or similar, but it looks like my linux distro doesn't even register the buttons being pressed, indicating there might be firmware issues?

Things could have changed, but I remember AHK being Windows-only, if you are running AHK through Wine, especially if it's on Wayland, I won't be surprised it doesn't work.
Not sure if there are any alternatives for desktop Linux, as I have never needed such functionality.


Not edit  Buttons

When create new channel using "/setup",gives this error & don't edit buttons "pause to resume" & not send song resumed message.
p.s:- 395:20 is in you given command line.

.

ok we will fix it

it give error when msg is undefined

I didn't delete msg,how could it undefined.

sometimes bot can't cache message

Do it again


SPI doesn't seem to be working with OTBR image

Description
Ive connected an nrf52840 board, with the NCP firmware compiled with SPI as the transport, on the correct pins for both nrf52840, and rpi3. I also had to slow the clock of the raspberrypi to for my saleae to capture the data.

To Reproduce

Compile and flash ot-ncp-ftd compiled with make -f examples/Makefile-nrf52840 NCP_SPI=1
Connect nrf52840 to rpi

pin name
nrf52840 pin
Rpi pin

MOSI
4
GPIO10

MISO
28
GPIO9

SCK
3
GPIO11

CSN
29
GPIO7

IRQ
30
GPIO25

Run docker image with docker run --sysctl "net.ipv6.conf.all.disable_ipv6=0 net.ipv4.conf.all.forwarding=1 net.ipv6.conf.all.forwarding=1" -p 8080:80 --dns=<IP_ADDRESS> -it --volume /dev/ttyACM0:/dev/ttyACM0 --privileged openthread/otbr --ncp-path /dev/spidev0.1

Expected behavior
wpantund should not end in error.
Console/log output
pi@raspberrypi:~ $ docker run --sysctl "net.ipv6.conf.all.disable_ipv6=0 net.ipv4.conf.all.forwarding=1 net.ipv6.conf.all.forwarding=1" -p 8080:80 --dns=<IP_ADDRESS> -it --volume /dev/ttyACM0:/dev/ttyACM0 --privileged openthread/otbr --ncp-path /dev/spidev0.1
WARNING: Localhost DNS setting (--dns=<IP_ADDRESS>) may fail in containers.
--ncp-path
NCP_PATH: /dev/spidev0.1
TUN_INTERFACE_NAME: wpan0
NAT64_PREFIX: 64:ff9b::/96
AUTO_PREFIX_ROUTE: true
AUTO_PREFIX_SLAAC: true
Current platform is ubuntu
* Applying /etc/sysctl.d/10-console-messages.conf ...
kernel.printk = 4 4 1 7
* Applying /etc/sysctl.d/10-ipv6-privacy.conf ...
net.ipv6.conf.all.use_tempaddr = 2
net.ipv6.conf.default.use_tempaddr = 2
* Applying /etc/sysctl.d/10-kernel-hardening.conf ...
kernel.kptr_restrict = 1
* Applying /etc/sysctl.d/10-link-restrictions.conf ...
fs.protected_hardlinks = 1
fs.protected_symlinks = 1
* Applying /etc/sysctl.d/10-magic-sysrq.conf ...
kernel.sysrq = 176
* Applying /etc/sysctl.d/10-network-security.conf ...
net.ipv4.conf.default.rp_filter = 1
net.ipv4.conf.all.rp_filter = 1
net.ipv4.tcp_syncookies = 1
* Applying /etc/sysctl.d/10-ptrace.conf ...
* Applying /etc/sysctl.d/10-zeropage.conf ...
vm.mmap_min_addr = 32768
* Applying /etc/sysctl.d/60-otbr-ip-forward.conf ...
net.ipv6.conf.all.forwarding = 1
net.ipv4.ip_forward = 1
* Applying /etc/sysctl.conf ...
 * Starting userspace NAT64 tayga                                                                                [ OK ]
/usr/sbin/service
 * Starting domain name service... bind9                                                                                [ OK ]
/usr/sbin/service
 * dbus is not running
 * Starting system message bus dbus                                                                                [ OK ]
   ...fail!
wpantund[132]: Starting wpantund 0.08.00d (Nov 21 2018 21:32:05) . . .
wpantund[132]: 	BUILD_VERSION = a1e0dfe
wpantund[132]: Configuration file "/etc/wpantund.conf" read.
wpantund[132]: Ready.  Entering fault state.
wpantund[132]: State change: "uninitialized" -> "uninitialized:fault"
``

Attaching Saleae logic analyzer exported file also.
https://drive.google.com/file/d/1hNA-SW0ZhLQYMxe7AuXh3eHYGlvkfiWK/view?usp=sharing

Hello @ntavish
At first glance, it seems that you have an incorrect path to NCP device in wpantund.conf file.
As README.md states, the correct value of Config:NCP:SocketPath variable for your case (spidev0.1 used) is:
Config:NCP:SocketPath "system:/usr/bin/spi-hdlc-adapter --gpio-int /sys/class/gpio/gpio25 /dev/spidev0.1"
(of course with the assumption that spi-hdlc-adapter is located in the /usr/bin directory).
We have recently checked SPI communication with NCP and RCP on RPi 3B with the latest OpenThread libraries, so it should work fine. Please let me know about the progress.

Hi @LuDuda, thanks for the response, that solved the problem for me, not sure how I missed that!
Though one problem still persists for me, half the times wpantund fails to initialize, and sometimes when it does, it fails:
spi-hdlc-adapter[137]: INFO: sHdlcRxBadCrcCount=0
wpantund[132]: [-NCP-]: NCP was reset (RESET_SOFTWARE, 114)
wpantund[132]: [-NCP-]: NCP was reset (RESET_SOFTWARE, 114)
wpantund[132]: Unexpected reset during NCP initialization.
wpantund[132]: The NCP is misbehaving: Repeatedly unable to initialize NCP. Entering fault state.
wpantund[132]: State change: "uninitialized" -> "uninitialized:fault"

It might be because the clock speed for SPI is too high, it seems to be 125MHz in the raspian image I'm using:
Distributor ID:	Raspbian
Description:	Raspbian GNU/Linux 9.8 (stretch)
Release:	9.8
Codename:	stretch

The workaround for me has been to reduce default clock speed by loading a device tree overlay to slow down the clock before running the docker image. Listing steps in case someone else needs to do this also.
File: slowspi-overlay.dts
/dts-v1/;
/plugin/;

/ {
   compatible = "brcm,bcm2835", "brcm,bcm2836", "brcm,bcm2708", "brcm,bcm2709";

   fragment@0 {
      target = <&spi0>;
      frag0: __overlay__ {
         #address-cells = <1>;
         #size-cells = <0>;
         status = "okay";
         cs-gpios = <&gpio 8 1>, <&gpio 7 1>;

         spidev@0{
            compatible = "spidev";
            reg = <0>;   /* CE0 */
            #address-cells = <1>;
            #size-cells = <0>;
            spi-max-frequency = <100000>;
         };

         spidev@1{
            compatible = "spidev";
            reg = <1>;   /* CE1 */
            #address-cells = <1>;
            #size-cells = <0>;
            spi-max-frequency = <100000>;
         };
      };
   };

   fragment@1 {
      target = <&spi0_cs_pins>;
      frag1: __overlay__ {
        brcm,pins = <8 7>;
        brcm,function = <1>; /* out */
      };
   };
};

Compile it with dtc -O dtb -o slowspi-overlay.dtbo -b 0 -@ slowspi-overlay.dts
Load it with sudo dtoverlay slowspi-overlay. (If you need to remove it later, run sudo dtoverlay -r slowspi-overlay).

Hmm, I'm not familiar with such issue. @pszkotak any thoughts?

@LuDuda this is a different case than ours. Let me explain.
@ntavish does not run the OTBR image that is provided with nRF5 SDK for Thread and Zigbee, but some other distribution with docker. The image that we do test and share to public is based on raspbian and native wpantund. The SPI clock is set to 1MHz.
I propose to update the configuration/documentation for the SPI to:

for NCP:
Config:NCP:SocketPath "system:/usr/bin/spi-hdlc-adapter --gpio-int /sys/class/gpio/gpio25 --spi-speed 1000000 /dev/spidev0.1"
for RCP:
Config:NCP:SocketPath "system:/usr/bin/ot-ncp /usr/bin/spi-hdlc-adapter -- '--stdio -i /sys/class/gpio/gpio25 --spi-speed 1000000 /dev/spidev0.1'"

@ntavish, could you tell us more details about the OS you run on the RPi? Is it Raspian, Ubuntu or anything else? We build custom Raspian ourselves - it defaults the SPI clock to 1MHz.

@pszkotak I'm running Raspbian too, Raspbian 9.8 stretch
Distributor ID: Raspbian
Description:    Raspbian GNU/Linux 9.8 (stretch)
Release:        9.8
Codename:       stretch

Default clock on this seems to be 125MHz on this. I don't think I might have changed this, just enabled SPI (using raspi-config) and checked the clock speed. I checked it on another pi with release 9.6, clock seems to be 125MHz on that also.
pi@prod:~ $ cat /sys/class/spidev/spidev0.1/device/of_node/spi-max-frequency | od -t d --endian=big
0000000   125000000
0000004
pi@prod:~ $ lsb_release -a
No LSB modules are available.
Distributor ID: Raspbian
Description:    Raspbian GNU/Linux 9.6 (stretch)
Release:        9.6
Codename:       stretch

@ntavish Thank you for a quick response.
I will check our build settings then, maybe we have some old code everyone forgot about that sets this ;).
Could you check if the configuration I posted (Config:NCP:SocketPath "system:/usr/bin/spi-hdlc-adapter --gpio-int /sys/class/gpio/gpio25 --spi-speed 1000000 /dev/spidev0.1") can be applied to your wpantund.conf in the docker environement? It should solve the issue without recompiling the dts.

That doesn't seem to be working for me. wpantund fails to initialize. Lower clocks are working though, tested with 700KHz and 500KHz.
"system:/usr/bin/spi-hdlc-adapter --gpio-int /sys/class/gpio/gpio25 --spi-speed 500000 /dev/spidev0.1"


Update meta.yaml

Bump build to 1

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Small change to description.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Stack overflow when making change to project

FCSWatch is exiting with a stack overflow when watching this project.
It is fine when editing and saving any of the code in the mentioned project, however if you change something in this project, which it relies on, a stack overflow occurs.
I may try and look into this myself this weekend if I can figure out a good approach to debugging this issue.

https://github.com/humhei/FCSWatch/blob/master/tests/FcsWatch.InteractionTests/InteractionTests.fs#L34
        ftestCase "auto reload test" <| fun _ ->
            FcsWatch.Cli.Main.main [|"--project-file"; entryProjPath|]
            |> ignore

I'm not seeing this bug anymore, not sure why.
@humhei did you happen to change anything in regards to this? closing either way


[FEATURE] - Android and OpenaiAPI first structure

Android Features:

Android set and connect function
Get all widget view avaiable in screen
Filter to get only action widgets
Create a prompt for each widget view

OpenaiAPI features:

Able to manupulete the Assistant API
Async function to wait the API response
Create, "talk", em have a reponse from the assistant

General:

Create the setup file
Utils file with the timeit decorator to monitor function performance


OAK-9553: Add config support for multiple mounts in CompositeNodeStore

Add MountInfoConfig configuration factory, which can be used to
configure multiple read-only mounts.
MountInfoProviderService can now be configured with the list of expected
mounts, so it can wait for all of them before registering a
MountInfoProvider.
For backward-compatibility, MountInfoProviderService can still be used
to configure a single read-only mount, but all such properties are now
marked as deprecated.

Added integration tests using Pax Exam in oak-store-composite, validating the new configuration of multiple read-only mounts, as well as backward compatibility with previous configuration.


spawn fail when file path contain "(" on XP and WIN10

spawn fail when file path contain "(" on XP and WIN10 .
such issue not appear on .exe file, but appear on .bat file.
I first report it at https://github.com/nodejs/io.js/pull/1869.
my resolve way is :
change each '(' in arg[0] to "^(".

/cc @piscisaureus

@sxyizhiren
I think it would be better if you just added the character to the "search list" in https://github.com/libuv/libuv/blob/c619f37c3986b0ec5d13fb7f2437dff0db41c72b/src/win/process.c#L461.

hi @piscisaureus
i think you mean that, if contain '(', then add quote '"' to the string.
i have tested it, adding quote to the arg[0] couldn't resolve it.

Spawning .bat files is kind-of unsupported right now - parameter passing is completely unreliable as soon as any special characters are involved. But I'm not objected to making it slightly better.
HOWEVER I'm pretty sure that this will break certain .exe files, because their argv[0] will become unreliable. So in order to land this patch I would want it to only apply to .bat and .cmd files.

@piscisaureus ok, got it. so , should we keep it , or i will try to find a way that "only apply to .bat and .cmd files"  ?

MSDN for CreateProcessW says you should spawn cmd.exe and pass the script as parameter - e.g. the way Node does this.
I say we close the PR, because of this and because its almost 2 years old.

Sounds good, I'll close.  Thanks anyway @sxyizhiren.


Reduxify pull request for Language Downloading and Settings

I encountered an issue which I haven't been able to figure out yet. The individual language downloads work after a fresh launch of the app. However, if I use Reload from the Developer menu, the download requests never complete (even after adding the error handlers I can think of), and the created language files in the app/locales folder end up being zero bytes. I am not exactly sure if this is a result of how the batch actions are being handled, but I am still looking into it.

Additionally, could you eliminate the hardcoded-references to app/locales? It looks like this can be read directly from the i18n object. If not, it should be defined as a constant... somewhere (I'm so helpful).

@kauffj Pushed a new commit.

@akinwale just checked this out again and I'm not seeing any language selections other than English.
Additionally, whatever http is doing under the hood, it somehow makes requests without them showing up on the Chrome debug toolbar?! This makes no sense to me, but I don't see any requests at all, despite data being returned. Any idea what is going on there?

@kauffj Does this happen if you launch the app or if you reload using the Reload option from the Developer Menu? There seems to be an issue where the latter doesn't work properly for some reason. Launching the app seems to be fine, however.
Regarding the http requests, this may be related: https://stackoverflow.com/a/44890425/61385

Please switch to the i18n branch. All code, including your latest code, and some refactoring by me, is in that branch.
This code is extremely difficult to test and basically unshippable until the reloading issue is worked out. I spent a significant amount of time looking into it without resolution. What I learned:

The requests will eventually timeout.
The requests do actually get made and sent to the correct URL

@kauffj I was also stuck on the same reloading issue. I can try to move the language load logic to the settings page component componentDidMount method. I'll probably also have to add some sort of loading indicator to the settings page. What do you think?

@akinwale I think the load issue is deeper than that. The requests are getting made properly, they are just timing out. I suspect it is somehow related to the previous request not getting properly closed or cleaned up.
You can observe the requests happen with something like:
sudo tcpflow -p -c -i wlp4s0 port 80 | grep -oE '(GET|POST|HEAD) .* HTTP/1.[01]|Host: .*'
(Use ifconfig to determine the -i argument)
One fix that would definitely fix it is to replace the use of Node's http.request with browser-native xhr requests. But http.request shouldn't be behaving the way it is and I hate giving up without understanding :P

Pushed commit https://github.com/lbryio/lbry-app/commit/efd2b99cda762496dc718fca0132643f8e28125f with doDownloadLanguages modified to use XMLHttpRequest instead of http.request which seems to have fixed reload issues.

See comment on https://github.com/lbryio/lbry-app/commit/efd2b99cda762496dc718fca0132643f8e28125f

Follow up comment at https://github.com/lbryio/lbry-app/commit/efd2b99cda762496dc718fca0132643f8e28125f#commitcomment-23806852


Aborted can be triggered in  tf.raw_ops.TensorListConcat

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
tf 2.15
Custom code
Yes
OS platform and distribution
Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
The following code sometimes runs normally when calling tf.raw_ops.TensorListConcat, but sometimes encounters an "aborted" error.
To reproduce the issue, it may be necessary to execute several times.
Standalone code to reproduce the issue
import tensorflow as tf

input_data_1 = tf.constant([[1, 2, 3], [4, 5, 6]])
input_data_2 = tf.constant([[7, 8, 9], [10, 11, 12]])

input_handle = tf.raw_ops.TensorListReserve(element_dtype=tf.int32, element_shape=[2, 3], num_elements=2)
input_handle = tf.raw_ops.TensorListSetItem(input_handle=input_handle, index=0, item=input_data_1)
input_handle = tf.raw_ops.TensorListSetItem(input_handle=input_handle, index=1, item=input_data_2)

concatenated_data, lengths = tf.raw_ops.TensorListConcat(input_handle=input_handle, element_dtype=tf.int32, element_shape=[6])

print(concatenated_data)

Relevant log output
tf.Tensor([1 2 3 4], shape=(4,), dtype=int32)
malloc_consolidate(): invalid chunk size
Aborted (core dumped)

Hi @vwrewsge ,
I tried to run your code on colab using tf 2.15, 2.16 and tf-nightly versions faced same issue. Please find the gist here for reference. We need to work on it, In few days i will provide a solution for it.
Thank you!

Hi @vwrewsge ,

Apologies for the delay,  I think this was resolved in recent tf-nightly. Please check the gist here. It will resolve in upcoming version.

Thank you!


Allow not setting vm.max_map_count

We set vm.max_map_count via config management on all of our kubelets to 1000000, so the setting here is simply unnecessary.

Ahh interesting, we can add a flag to make that optional.

Closed via #242


Cannot coerce_timestamps when writing parquet with pyarrow

When trying to take a pandas dataframe and save a parquet file, forcing a timestamp coercion throws an error:
import pandas as pd
import pyarrow as pa
import pyarrow.parquet as pq
from io import BytesIO
... # Creating a pandas dataframe with timestamps
b = BytesIO()
table = pa.Table.from_pandas(df)  # df is the dataframe in question
pq.write_table(table, b, coerce_timestamps='ms')
...

That throws the following error message and the script errors out:
pyarrow.lib.ArrowInvalid: Casting from timestamp[ns] to timestamp[ms] would lose data:<PHONE_NUMBER>115534000

Is there a way to suppress this error message as a warning instead of failing out?
Using pyarrow version 0.9.0.post1, python version 3.6.2 and pandas version 0.22.0.

Parquet has a nanosecond timestamp type that is discouraged to use for future projects which you can also enable to use when writing Parquet files using pyarrow, see https://arrow.apache.org/docs/python/generated/pyarrow.parquet.write_table.html.
pq.write_table(table, b, use_deprecated_int96_timestamps=True)

Alternatively, if you're happy with millisecond precision, just round your Pandas column so that the timestamp is a multiple of a million (1million ns = 1ms ;) ).

I have created a ticket in JIRA to track this https://issues.apache.org/jira/browse/ARROW-2555 Closing this issue here, please subscribe on JIRA if you want to have updates or implement it.

I am encountering this issue without ever explicitly creating a pandas DataFrame myself. It seems that when trying to use pyarrow.parquet.write_to_dataset, if one specifies an argument for partition_cols, pyarrow (for whatever reason) creates a pandas.DataFrame object for each partition, then converts that DataFrame back to a pyarrow.Table (see source code). Why it does this, I have no idea.
One consequence of this design decision, however, is that trying to write tables that have columns of type date32[day] not part of the partition columns, writing will often fail. I presume that the reason it fails is because of floating-point precision issues going back and forth between pandas and pyarrow (i.e., from date32[day] -> datetime64[ns] when creating each partition of the table in pandas, then from datetime64[ns] -> date32[day] when going back to pyarrow).

Can you please post your comment on JIRA? Thanks

I'm adding this because the cumulative online discussion is misleading. Parquet has deprecated support for nanoseconds with the INT96 encoding (32-bit Julian day, 64-bit nanoseconds since midnight). Parquet 2.x now has support for nanosecond annotated INT64 timestamps, and Arrow supports this as well (at least mostly). However, Pyarrow does not yet support Parquet 2.x format files fully.
Net of net - 64-bit nanosecond support is coming, probably in a quarter or three (just a guess).

There is now allow_truncated_timestamps=True to suppress that error

Thanks this solved it for me.

See: https://stackoverflow.com/questions/59682833/pyarrow-lib-arrowinvalid-casting-from-timestampns-to-timestampms-would-los/72111737#72111737

What about the same issue when reading? I have a source that provides parquet files with INT96 timestamps with ms precision. I can read this in arrow by adding coerce_int96_timestamp_unit="ms". But then it goes wrong when converting to pandas. I get the same exception as above.


[protocol] Parse manifest flags into structs

fixes #60
Hi! This PR seems OK to me, but likely it will just be the starting point. Obviously feedback is appreciated, and here are some thoughts I had:

Existing code doesn't seem to use new often, I could reduce the usages that I introduce?
Unsure exactly how manifest::owned::pfm and the elements in manifest::pfm really related to each other, should I make changes to manifest::pfm as well?
I specifically wrote the code to be forward/backward compatible with new flags, wire_enum didn't seem to be a perfect fit here especially because these flags don't have the whole byte, just a bit or two at the moment.
The zeroth_bit_zero and the << 6 >> 6 shifting don't seem great.. but they're relatively simple so I thought why try something more complicated
I could pull these things out to a flags module if you'd like, the code is a bit messy all together
Yes my tests test all u8 values, but its not that many. I heard someone once say "why not test them all?" when referring to testing variants on numbers less than like 32 bits, so I just went along with that logic.

Thx!

I just did pretty much an entire re-write of the logic, so a lot of the old comments don't apply.

Addressed the invariant situation
Tried to add better documentation comments per the info I had available, bit if you had more background you'd like me to add, let me know!
updated bit logic
tests have many more cases
add getters, and setter on the owned PFM

Thanks!


Question about load pretrained model

Hello,
When I wanted to load pretrained model, I faced this problem:

INFO - custom_bert -   Weights from pretrained model not used in BertForMaskedLM: ['bert.pooler.dense.weight', 'bert.pooler.dense.bias', 'cls.seq_relationship.weight', 'cls.seq_relationship.bias']

Sorry it is my fault, The code is ok.

Hello, I also encountered this problem, what is the cause? Or, how did you solve it?
Moreover, can you tell me the dependencies to run this code?
Thank you so much!

Hello, I also encountered this problem, what is the cause? Or, how did you solve it? Moreover, can you tell me the dependencies to run this code? Thank you very much!

I'm so sorry. It's been so long, and I lose the code.


Fix slideshow script on boot up problem

Note - Get slideshow script running on pi boot up. The script works when ran however on boot start-up of the script needs to be added.

Commit that resolved boot up problem https://github.com/akilhylton/slideshow/commit/eacbc423197d540f756b130620e9507bfabda7bf