Bluetooth: hci-usb uses non-standard interfaces

The Bluetooth specification (5.2v4B2.1.1) requires that a USB HCI primary controller provide the bulk and interrupt endpoints as interface 1, and a scalable isochronous endpoint at interface 2.  The latter supports the synchronous connection oriented capability required for Bluetooth audio.
Zephyr doesn't have the ability to specify which interface the BT HCI controller ends up at, and it doesn't provide an SCO endpoint at all.
Linux btusb verifies this requirement.  And it happens that if the only USB interface in the Zephyr application is the HCI one, samples/bluetooth/hci_usb works fine as a Linux bluetooth HCI device.
If some other devices are added, for example a CDC-ACM intended to serve as a shell, or a console that can be used to get log messages out, it does not.  This appears to be because Zephyr sorts the candidate interface descriptors by size, and the HCI one comes after the others, breaking the validation check in Linux.
While it's possible to patch the btusb module to bypass this check (an existing exception is in place for Apple Broadcom), the resulting interface is not usable:
raspi3bp[92]$ hciconfig
hci1:	Type: Primary  Bus: USB
	BD Address: 00:00:00:00:00:00  ACL MTU: 0:0  SCO MTU: 0:0
	DOWN INIT RUNNING
	RX bytes:0 acl:0 sco:0 events:0 errors:0
	TX bytes:0 acl:0 sco:0 commands:1 errors:1

My theory is this is because Linux issues the probe while Zephyr is busy setting up the earlier interfaces.
There should be a way to ensure that the HCI-USB implementation conforms to the specification:

The existing bulk HCI-USB interface should be installed as interface 0;
A new an empty interface number 1 added that can be detected by btusb for the ISOC endpoint.

IMO that part's a bug.  If, an enhancement may be necessary to delay announcing existence of the interface to Linux until Zephyr's ready to respond to the on-probe commands.
May relate to the original goal of #4767

The Bluetooth specification (5.2v4B2.1.1) requires that a USB HCI primary controller provide the bulk and interrupt endpoints as interface 1, and a scalable isochronous endpoint at interface 2. The latter supports the synchronous connection oriented capability required for Bluetooth audio.

Actually, first interface and second interface, I did not found where bInterfaceNumbers are claimed to be specific value. This would also defeat the purpose of an interface descriptor.

Zephyr doesn't have the ability to specify which interface the BT HCI controller ends up at, and it doesn't provide an SCO endpoint at all.

Zephyr does not need it, nobody needs it, that is what the descriptors are for. But within the local descriptor content of a class you can define the order of interfaces.

Linux btusb verifies this requirement. And it happens that if the only USB interface in the Zephyr application is the HCI one, samples/bluetooth/hci_usb works fine as a Linux bluetooth HCI device.

If btusb has problems finding the right interface then it is a problem in btusb not Zephyr. But if remember correctly we can change the order on the Zephyr side so that hci_usb always appears as first entity.

the resulting interface is not usable:

raspi3bp[92]$ hciconfig
hci1:	Type: Primary  Bus: USB
BD Address: 00:00:00:00:00:00  ACL MTU: 0:0  SCO MTU: 0:0
DOWN INIT RUNNING
RX bytes:0 acl:0 sco:0 events:0 errors:0
TX bytes:0 acl:0 sco:0 commands:1 errors:1

My theory is this is because Linux issues the probe while Zephyr is busy setting up the earlier interfaces.

My theory is this is because the function to generate a random address requires "User-space interface for hash algorithms"
from Linux kernel. This option is not enabled in the RPi kernel image (tested 4.19.85-v7+) and the BlueZ failes to bring up the device.

For most of that, all I can say is that the specification appears to be pretty clear.

My theory is this is because the function to generate a random address requires "User-space interface for hash algorithms"
from Linux kernel. This option is not enabled in the RPi kernel image (tested 4.19.85-v7+) and the BlueZ failes to bring up the device.

No, it's not that.  CONFIG_CRYPTO_USER_API_HASH=m is set in the 5.4.69 config I'm using under Yocto, and the driver works just fine if the first interface has bInterfaceNumber=0 as required.

For most of that, all I can say is that the specification appears to be pretty clear.
if the first interface has bInterfaceNumber=0 as required.

Where did you get this condition that bInterfaceNumber must be 0, where is it in the specification?
You can force hci_usb class to be first element with:
diff --git a/subsys/usb/class/bluetooth.c b/subsys/usb/class/bluetooth.c
index 5511b24dc0..5982784efd 100644
--- a/subsys/usb/class/bluetooth.c
+++ b/subsys/usb/class/bluetooth.c
@@ -50,7 +50,7 @@ struct usb_bluetooth_config {
        struct usb_ep_descriptor if0_in_ep;
 } __packed;

-USBD_CLASS_DESCR_DEFINE(primary, 0)
+USBD_CLASS_DESCR_DEFINE(0primary, 0)
        struct usb_bluetooth_config bluetooth_cfg = {
        /* Interface descriptor 0 */
        .if0 = {

BLUETOOTH CORE SPECIFICATION Version 5.2 | Vol 4, Part B, section 2 "USB endpoint expectations".

The Primary Controller configuration consists of two interfaces. The first
interface has no alternate settings and contains the bulk and interrupt
endpoints. The second interface provides scalable isochronous bandwidth.

I will grant you that first and second may not require interface numbers 0 and 1, and it's just chance that in the twelve years Linux has had HCI-USB driver support, with dozens of explicitly supported devices and uncounted standard ones, only one device hasn't used interface numbers 0 and 1.
In any case, if that isn't a hard requirement of the specification, I've already worked around that, but it'll still hit anybody who can't build their own custom driver in Linux.

You can force hci_usb class to be first element with:

No, as you rediscovered, it isn't, because the entries are sorted by size first.
I do think that if I could make sure that the bluetooth stuff was up and running before Linux tries to communicate with the endpoints it might work.

You can force hci_usb class to be first element with:

No, as you rediscovered, it isn't, because the entries are sorted by size first.

Other way around, you can try to force CDC_ACM to be behind hci_usb:
diff --git a/subsys/usb/class/cdc_acm.c b/subsys/usb/class/cdc_acm.c
index 1d1fb0b191..2bd6c61b05 100644
--- a/subsys/usb/class/cdc_acm.c
+++ b/subsys/usb/class/cdc_acm.c
@@ -1045,7 +1045,7 @@ static const struct uart_driver_api cdc_acm_driver_api = {

 #if (CONFIG_USB_COMPOSITE_DEVICE || CONFIG_CDC_ACM_IAD)
 #define DEFINE_CDC_ACM_DESCR(x, int_ep_addr, out_ep_addr, in_ep_addr)  \
-       USBD_CLASS_DESCR_DEFINE(primary, x)                             \
+       USBD_CLASS_DESCR_DEFINE(primary, 1.##x)                         \
        struct usb_cdc_acm_config cdc_acm_cfg_##x = {                   \
        .iad_cdc = INITIALIZER_IAD,                                     \
        .if0 = INITIALIZER_IF(0, 1, COMMUNICATION_DEVICE_CLASS,         \
@@ -1070,7 +1070,7 @@ static const struct uart_driver_api cdc_acm_driver_api = {
 }
 #else /* (CONFIG_USB_COMPOSITE_DEVICE || CONFIG_CDC_ACM_IAD) */
 #define DEFINE_CDC_ACM_DESCR(x, int_ep_addr, out_ep_addr, in_ep_addr)  \
-       USBD_CLASS_DESCR_DEFINE(primary, x)                             \
+       USBD_CLASS_DESCR_DEFINE(primary, 1.##x)                         \
        struct usb_cdc_acm_config cdc_acm_cfg_##x = {                   \
        .if0 = INITIALIZER_IF(0, 1, COMMUNICATION_DEVICE_CLASS,         \
                              ACM_SUBCLASS),

You can check the order of the entities in zephyr.map, for example for the hid-cdc sample:
*(SORT_BY_NAME(SORT_BY_ALIGNMENT(.usb.descriptor*)))
 .usb.descriptor_primary.0
                0x0000000020000488       0x1b libzephyr.a(usb_descriptor.c.obj)
 .usb.descriptor_primary.1.0
                0x00000000200004a3       0x19 subsys/usb/class/hid/libsubsys__usb__class__hid.a(core.c.obj)
 .usb.descriptor_primary.1.1
                0x00000000200004bc       0x19 subsys/usb/class/hid/libsubsys__usb__class__hid.a(core.c.obj)
 .usb.descriptor_primary.1.1.0
                0x00000000200004d5       0x42 libzephyr.a(cdc_acm.c.obj)
 .usb.descriptor_primary.1.1.1
                0x0000000020000517       0x42 libzephyr.a(cdc_acm.c.obj)
 .usb.descriptor_primary.4
                0x0000000020000559       0x70 libzephyr.a(usb_descriptor.c.obj)
 .usb.descriptor_primary.5
                0x00000000200005c9        0x2 libzephyr.a(usb_descriptor.c.obj)
                0x00000000200005cb                __usb_descriptor_end = .

@holtmann could you please weigh in on this discussion?

It looks like Zephyr now supports ISO bluetooth operations, so maybe this is still worth doing?

It looks like Zephyr now supports ISO bluetooth operations, so maybe this is still worth doing?

AFAIK BR/EDR USB audio interface cannot be used for ISO or not yet.

I leave https://github.com/zephyrproject-rtos/zephyr/issues/29107#issuecomment-706770001 as workaround and will not fix it for now.

A more generic solution beyond workaround posted above will come in the future under a USB rework.

My opinion for what it's worth:
While there's a proposed workaround that requires modifying the Zephyr OS code itself, the problem remains: it is not confirmed that Zephyr's behavior is consistent with the specification, and a dongle application combining a Zephyr HCI-USB class with other USB classes like a console or shell will not work on Linux without a change to Zephyr's USB framework.
That a solution "will come in the future" suggests it's still recognized as a bug (or an enhancement) that really does need to be addressed.  However, because this is closed, there's nothing to help ensure that future work will actually provide a solution.
Closing this as "Wont Fix" with a comment acknowledging intent to fix it later may reduce the bug census, but it doesn't make sense from a software quality process point of view.

what is the news on this issue ?
in the most recent release the workaround on subsys/usb/class/cdc_acm.c seem not work anymore.
I try to modify on line 1031
USBD_CLASS_DESCR_DEFINE(primary, 1.##x)				\
and check the zephyr.map
 *(SORT_BY_NAME(SORT_BY_ALIGNMENT(.usb.descriptor*)))
 .usb.descriptor_primary.0
                0x000000002000090c       0x1b zephyr/libzephyr.a(usb_descriptor.c.obj)
 .usb.descriptor_primary.1.0
                0x0000000020000927       0x1e zephyr/libzephyr.a(bluetooth.c.obj)
 .usb.descriptor_primary.1.1.0
                0x0000000020000945       0x42 zephyr/libzephyr.a(cdc_acm.c.obj)
 .usb.descriptor_primary.4
                0x0000000020000987       0x58 zephyr/libzephyr.a(usb_descriptor.c.obj)
 .usb.descriptor_primary.5
                0x00000000200009df        0x2 zephyr/libzephyr.a(usb_descriptor.c.obj)
                0x00000000200009e1                __usb_descriptor_end = .

some suggestions ?

Error on btusb:
[   38.262123] cdc_acm 1-1.4:1.1: ttyACM0: USB ACM device
[   38.262288] usbcore: registered new interface driver cdc_acm
[   38.262301] cdc_acm: USB Abstract Control Model driver for USB modems and ISDN adapters
[   38.262406] btusb: probe of 1-1.4:1.0 failed with error -16
[   38.262503] usbcore: registered new interface driver btusb

The workaround to change what class interface appears at the end still works, but that will not help you with btusb, since it simple does not work with composite configuration.
diff --git a/subsys/usb/class/cdc_acm.c b/subsys/usb/class/cdc_acm.c
index 9adc89a9c4b..df207d4a3a0 100644
--- a/subsys/usb/class/cdc_acm.c
+++ b/subsys/usb/class/cdc_acm.c
@@ -1010,7 +1010,7 @@ static const struct uart_driver_api cdc_acm_driver_api = {
        }

 #define CDC_ACM_CFG_AND_DATA_DEFINE(x)                                 \
-       USBD_CLASS_DESCR_DEFINE(primary, x)                             \
+       USBD_CLASS_DESCR_DEFINE(primary, 1.##x)                         \
        struct usb_cdc_acm_config cdc_acm_cfg_##x = {                   \
                INITIALIZER_IAD                                         \
                .if0 = INITIALIZER_IF(0, 1,                             \

So it's not possible to use HCI USB + Console or shell at the same time ? I don't understand for which problem the workaround works.

What can I patch on the zephyr side to avoid changing my btusb driver?

So it's not possible to use HCI USB + Console or shell at the same time ? I don't understand for which problem the workaround works.

The workaround to change what class interface appears at the end still works

What can I patch on the zephyr side to avoid changing my btusb driver?

Nothing, you cannot use hci_usb and CDC ACM in composite configuration with btusb.


iOS - framework not found error

Hello.
I've installed<EMAIL_ADDRESS>from npm, linked it to a fresh project with react-native link, added RNSocialAuth.xcodeproj to my project libraries and made sure that libRNSocialAuth.a was added to the "Link Binary With Libraries" Build Phase (everything according to the iOS setup doc).
Now when I try to run the project on my iPhone 7 device, I get the following error:

ld: framework not found FBSDKLoginKit for architecture arm64

If I move the libRNSocialAuth.a to the top of the "Link Binary With Libraries" list, then I get:

ld: framework not found Bolts for architecture arm64

If I try to run it in iPhone 6 simulator, then I get:

ld: framework not found FBSDKCoreKit for architecture x86_64

I'm not sure if this is a problem with the package or if I'm doing something wrong. Any ideas?

Oh, I needed to add the Frameworks and Framework Search Paths listed in the "Facebook" section in order to get the app to compile.
My bad.

@UltCombo I have the same problem, how did you add Framework?

Hello @Luckygirlllll, if I recall correctly you have to follow the instructions in the iOS setup. Note that you need to set up Facebook even if you are only using Twitter auth, that was the problem when I tried to install it a year ago.


fix(TDI-38194): use http instead of socks as the proxy type for the proxy setting in salesforce components and adjust the test case to detect it

Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-38194
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull request test result: success :star:

thank you, igonchar

This PR should not be on master, the customer replied it works in 6.3.1 (while it was not working on 6.2.2)


🐞[Bug]:

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
BEFORE :

The "Find Doctor" button seems to be causing disruption to the webpage layout.
AFTER :

Expected behavior
The webpage layout appears to be disrupted.
Add ScreenShots
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Anishkagupta04 pls assign this to me, I have raised multiple issue son your websites, pls check them too


feat(generate): specify class type via dot notation

Closes #2155
BREAKING CHANGE: The ability to specify a class type via an additional arg has been replaced by combining the name and type args separated by a dot

Well crap, I just lobotomized CLA bot. @Brocco can you open a new PR for this?

I signed it


fix member select highlight issue

Details
Called onSelectRow function instead of selectRow function in renderItem function of BaseSelectionList component.
Fixed Issues
$ https://github.com/Expensify/App/issues/27377
PROPOSAL: https://github.com/Expensify/App/issues/27377#issuecomment-1718163069
Tests

Go to Settings > Workspace.
Create new workspace if not created.
Go to members.
Invite any 2 members.
Click on select all check box.
Deselect last selected member.
Deselect second selected member.
Check if the member is not highlighted.

[x] Verify that no errors appear in the JS console

Offline tests
Same as above.
QA Steps
Same as above.

[X] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
Screencast from 2023年09月18日 16时49分36秒.webm

Mobile Web - Chrome

Mobile Web - Safari

Desktop
https://github.com/Expensify/App/assets/53784473/a881fd16-d798-4695-ae07-de90ab36a6c8

iOS

Android

Let's hold this for https://github.com/Expensify/App/pull/27246. We may need to do nothing here

@s77rt Could you please let me know the progress of this issue?

@akamefi202 For now we have to do nothing but wait for https://github.com/Expensify/App/pull/27246 to get merged then recheck if the bug is still reproducible

I think the issue is no longer reproducible. Can you please confirm the same?

@s77rt I've tested in staging and it is still reproducible.

The PR has not been deployed to staging yet, it's only merged on main

@s77rt I tested with main branch and it's not reproducible now.

In that case I think we can close this PR.


wish: GNU Bison

https://www.gnu.org/software/bison/ | https://github.com/akimd/bison

Bison is a general-purpose parser generator that converts an annotated context-free grammar into a deterministic LR or generalized LR (GLR) parser employing LALR(1) parser tables.

Should be implemented as a build system module if I understand the documentation correctly.
Build prerequisites

m4 (https://github.com/build2-packaging/WISHLIST/issues/24)
flex (https://github.com/build2-packaging/WISHLIST/issues/23)

I looked into this and it's not going to be straightforward because it requires gnulib which is deeply integrated with autotools. It (or flex, I don't remember exactly) also requires m4 though I think it doesn't have to be GNU m4, just reasonably compatible.
What I did is package byacc (Berkley yacc) and reflex (a flex fork) which are reasonably compatible with GNU versions (there is also openbsd-m4). I used them instead of bison/flex in liblkc for example: https://github.com/build2-packaging/kconfig/blob/master/liblkc/liblkc/buildfile
Regarding the build system module, we may want to do it one day but it feels like using an ad hoc recipe/rule is a lighter-weight solution for most projects.

I think let's keep it open even if things work out for thrift -- we will need to package bison and flex sooner of later.


Why is this no longer in operation?

Hi!
I was in the process of making something similar to this when a friend told me of its existence. But I see it's no longer in the chrome web store. There's a reddit thread on the subject, but it's been deleted. So, if I may ask, what happened to it?

Looking at the most recent commit from back in March 2019, it seems that the latest version of the extension (v3.0.13) was broken due to large changes made to Rotten Tomatoes. Upon updating to that version of Heirloom, the user would receive this message:

Heirloom Rotten Tomatoes is under construction 😱
My apologies that the extension is broken!
Rotten Tomatoes has made some big changes to the layout of their movie pages and Tomatometer. You can read about the changes on their blog. As a consequence, I'll need to make some big changes to the Heirloom Rotten Tomatoes extension to make it work again.
Thank you for your continued support
This extension is a labor of love, and I am grateful for all of your bug reports and messages of encouragement. I hope to get things running again soon, but that may take a couple of months because of my other commitments. Until then, if you keep the extension installed, the Rotten Tomatoes website will continue to work normally. I apologize for the inconvenience!
Once you tap "Got it" below, this message will disappear forever until I have more news to share. If you have any questions, feel free to DM me on Twitter: @mattthew

I also found this response on Twitter made by Mattthew back in September 2019:

is the app gone for good?

I never say never, but if I did work on it again, it would be a long time from now. Thanks for letting me know that you used it!

Many thanks for your help HatScripts! It is very helpful. Looking at your repos I can tell you have some experience in this domain. I have not gotten started on this yet, but I hope to do so in short order!


chore(monorepo): bring in revery-text-wrap

This brings in @OhadRau's revery-text-wrap into the monorepo. In order to make fallback work properly, we have to make this lib Unicode/UTF-8 aware. Since we already have Zed in here as of #941, I think it makes sense to also bring this into the monorepo.
Note that this PR does not patching to the lib for Zed, just brings it into the repo.
Thanks again @OhadRau for your text-wrap implementation!

Thanks for bringing this in @zbaylin , and @OhadRau for the initial implementation!
Is there a reason the test wasn't brought over? https://github.com/OhadRau/revery-text-wrap/blob/master/wrapTest.ml
It'd be great to have some test cases in place, so that we can build / add them for the UTF-8 implementation.

Thanks for bringing this in @zbaylin , and @OhadRau for the initial implementation!
Is there a reason the test wasn't brought over? https://github.com/OhadRau/revery-text-wrap/blob/master/wrapTest.ml
It'd be great to have some test cases in place, so that we can build / add them for the UTF-8 implementation.
The wrap tests might be a good fit for inline tests, maybe?

Thanks for reminding me! I added them in 059bcd0 as ReveryTextWrapCli and added the native and bytecode tests to CI in 059bcd0!

Hey sorry! Haven't had a whole lot of time to look at this today, but one thing I did notice is that we're not carrying over the OPAM package right now? I'm not sure what the situation looks like for our unicode lib (Zed I guess?) but I think it would make sense to make both of these available as OPAM libs. Not a huge priority but it might be a little confusing to keep developing our internal version while not publishing those changes back to the existing OPAM package.

Thanks @OhadRau! As of right now it's bundled with the Revery opam package -- not sure if we can include it in two separate packages but it's something I can look into. Since it will soon depend on Zed I'm not sure how feasible that would be either.
Thanks for the approval!


Error "Unknown XML node type: XMLElement" occurred in serializing the payload - "BlockLookupList"

Package Name: @azure/storage-blob
Package Version: 10.3.0
Operating system:OSX
[x] nodejs

version: 10.15.3

[ ] browser

name/version:

[x] typescript

version:3.4.5

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Unable to upload a stream using uploadStreamToBlockBlob
To Reproduce
Steps to reproduce the behavior:
1)Execute the following code to upload a file to existing container.
const main = async () => {
 // Enter your storage account name and shared key
  const account = accountname;
  const accountKey = key;

  // Use SharedKeyCredential with storage account and account key
  const sharedKeyCredential = new SharedKeyCredential(account, accountKey);

  // Use sharedKeyCredential, tokenCredential or anonymousCredential to create a pipeline
  const pipeline = StorageURL.newPipeline(sharedKeyCredential);

  // List containers
  const serviceURL = new ServiceURL(
    // When using AnonymousCredential, following url should include a valid SAS or support public access
    `https://${account}.blob.core.windows.net`,
    pipeline
  );

  const containerName = containername;
  const containerURL = ContainerURL.fromServiceURL(serviceURL, containerName);

  const blobName = 'newblob' + new Date().getTime();
  const blobURL = BlobURL.fromContainerURL(containerURL, blobName);
  const blockBlobURL = BlockBlobURL.fromBlobURL(blobURL);

 //upload stream
  const uploadBlobResponse = await uploadStreamToBlockBlob(
    Aborter.none,
    fs.createReadStream(pathToFile),
    blockBlobURL,
    4 * 1024 * 1024,
    5,
    {
      progress: ev => console.log(ev),
    }
  );
  console.log(`Upload block blob ${blobName} successfully`, uploadBlobResponse.requestId);
}

Additional context
Error log
`
Error "Unknown XML node type: XMLElement" occurred in serializing the payload - "BlockLookupList".
  at serializeRequestBody (node_modules/@azure/ms-rest-js/lib/serviceClient.ts:329:13)
  at e.Object.<anonymous>.ServiceClient.sendOperationRequest (node_modules/@azure/ms-rest-js/lib/serviceClient.ts:281:7)
  at e.blocks [as commitBlockList] (node_modules/@azure/storage-blob/dist-esm/lib/generated/lib/operations/blockBlob.js:48:13)
  at r.<anonymous> (node_modules/@azure/storage-blob/dist-esm/lib/BlockBlobURL.js:206:97)
  at y (node_modules/@azure/storage-blob/node_modules/tslib/tslib.es6.js:98:36)
  at Object.RetriableReadableStream [as next] (node_modules/@azure/storage-blob/dist-esm/lib/utils/RetriableReadableStream.js:12:5)
  at __generator (node_modules/@azure/storage-blob/node_modules/tslib/tslib.es6.js:75:17)
  at __awaiter (node_modules/@azure/storage-blob/node_modules/tslib/tslib.es6.js:67:37)
  at r.commitBlockList (node_modules/@azure/storage-blob/dist-esm/lib/BlockBlobURL.js:203:48)

`

Hi @Kirushanr - I've encountered the same error and am curious why you closed this?
Did you solve it?

Hi @Kirushanr & @tonytamps, Did you find a solution? It is happening randomly - Thanks

@m12m3r I did solve it but I don't remember how.
I'm using @azure/storage-blob and it's pinned to version 10.3.0 - Don't remember why.
Here is the code I'm successfully working with:
const UPLOAD_OPTIONS = {
  bufferSize: ONE_MEGABYTE * 40,
  maxBuffers: 5
}

function base64PhotoToStream(photoString) {
  const stream = new Readable()
  stream.push(photoString, 'base64')
  stream.push(null)
  return stream
}

function isBase64(str) {
  return Buffer.from(str, 'base64').toString('base64') === str
}

function base64PhotoContentType(photoString) {
  const buffer = Buffer.from(photoString, 'base64')
  return fileType(buffer)
}

// Based on https://docs.microsoft.com/en-us/azure/storage/blobs/storage-upload-process-images?tabs=nodejsv10
async function uploadPhoto(container, photoAsBase64, prefix) {
  if (typeof photoAsBase64 !== 'string' || !isBase64(photoAsBase64)) {
    throw new Error('Invalid base64 image provided')
  }

  const fileInfo = base64PhotoContentType(photoAsBase64)
  const uploadOptions = fileInfo
    ? { blobHTTPHeaders: { blobContentType: fileInfo.mime } }
    : {}
  const fileExtension = fileInfo ? fileInfo.ext : undefined

  const stream = base64PhotoToStream(photoAsBase64)
  const blobName = createUniqueName(prefix, fileExtension)
  const blobURL = BlobURL.fromContainerURL(container, blobName)
  const blockBlobURL = BlockBlobURL.fromBlobURL(blobURL)

  await uploadStreamToBlockBlob(
    Aborter.none,
    stream,
    blockBlobURL,
    UPLOAD_OPTIONS.bufferSize,
    UPLOAD_OPTIONS.maxBuffers,
    uploadOptions
  )

  return blobName
}

I have a vague memory that it had something to do with a dependency version mismatch. Maybe even to do with Azurite. Not sure if you're using Azurite @m12m3r ?


chore: add localizations to French, Turkish and German languages

Related Issue
Closes #383
Proposed Changes

Add localization to French
Add localization to German
Add localization to Turkish

Just reviewed FR + DE (german) translations in case it was needed


Scale istiod dynamically with VPA

How to categorize this issue?
/kind bug
What would you like to be added:
Scale istiod with VPA (or HPA, or HVPA).
Why is this needed:
Currently istiod Deployment is deployed with static memory and cpu limits - 2Gi and 1 CPU core. On multiple large Seeds we observe istiod Pods being OOMKilled:
$ k -n istio-system get po
NAME                     READY   STATUS             RESTARTS   AGE
istiod-99d6648c8-4dssv   0/1     CrashLoopBackOff   52         41h
istiod-99d6648c8-6h4ml   0/1     Running            53         20h
istiod-99d6648c8-fmtg2   0/1     CrashLoopBackOff   73         12d
istiod-99d6648c8-kdw9b   0/1     CrashLoopBackOff   71         13d
istiod-99d6648c8-t52r4   0/1     CrashLoopBackOff   70         10d

$ k -n istio-system describe po istiod-99d6648c8-4dssv

    State:          Waiting
      Reason:       CrashLoopBackOff
    Last State:     Terminated
      Reason:       OOMKilled
      Exit Code:    137
      Started:      Fri, 19 Feb 2021 16:47:43 +0200
      Finished:     Fri, 19 Feb 2021 16:48:16 +0200

The memory usage of istiod Pods from the last 14 days.

/assign @mvladev


Changing sort order changes column width

Describe the bug
Changing the sort order causes the column width to change, which is jarring. This is because the sort indicator toggles between being displayed and hidden.
To Reproduce

Go to https://codesandbox.io/s/github/tannerlinsley/react-table/tree/master/examples/sorting
Change the const data = React.useMemo(() => makeData(2000), []) to make only 10 rows.
Click on any table header 3 times.

Expected behavior
Column width should never change when changing the sort order on a table that fits fully on the screen.
Screenshots

The react-table itself is headless, and this behaviour is just one of the possible implementations of this kind of control and a part of an example. I don't think the library itself should do anything there.

Yep ☝

So how do we implement the behavior where the sort doesn't reset the width, are there any docs I can take a look at?

Hey!
A simple way is to change the style of the span like this:
 <span                     style={{ position: "absolute", right: "8px", top: "5px" }}                   >                     {column.isSorted                       ? column.isSortedDesc                         ? " 🔽"                         : " 🔼"                       : ""}                   </span>


Cleanup logging util a bit more. Add additional tests

first step to potentially moving out to api builder validation library
add tests

[info] ScalaTest
[info] Run completed in 15 seconds, 625 milliseconds.
[info] Total number of tests run: 37
[info] Suites: completed 9, aborted 0
[info] Tests: succeeded 37, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[info] Passed: Total 37, Failed 0, Errors 0, Passed 37
[success] Total time: 16 s, completed Jul 16, 2017 9:06:21 AM


ARROW-2428: [Python] Support pandas ExtensionArray in Table.to_pandas conversion

Prototype for https://issues.apache.org/jira/browse/ARROW-2428
This included the a squashed version of https://github.com/apache/arrow/pull/5162 as the first commit, as this builds further upon that PR (it can also be reviewed together at once in this PR).
What does this PR do?

Based on the pandas_metadata (stored when creating a Table from a pandas DataFrame), we infer which columns originally had a pandas extension dtype, and support a custom conversion (based on a __from_arrow__ method defined on the pandas extension dtype)
The user can also specify explicitly with the extension_column keyword which columns should be converted to an extension dtype

This only covers use case 1 discussed in the issue: automatic roundtrip for pandas DataFrames that have extension dtypes.
So it eg does not yet provide a way to do this if the arrow.Table has no pandas metadata (did not originate from a pandas DataFrame)

I also don't know whether exposing the extension_columns parameter publicly makes sense.

I also don't know whether exposing the extension_columns parameter publicly makes sense.

Yes, I agree, I am also not yet fully clear about that part of the API. For now, it was mainly for being able to test this a bit more.
Currently, I provide an "automatic detection" based on the stored pandas_metadata (which enables automatic roundtrip).
But, I think we need other mechanisms as well:

arrow extension types need to be able to specify a custom pandas dtype to convert to as well (one option is that they can point to a pandas ExtensionDtype object, and then use the same ExtensionDtype.__from_arrow__ logic).
It would be nice to have some way to override all this logic. For example, consider the use case where you have a parquet file (or arrow memory in general) that does not originate from pandas (and thus has no pandas metadata), and you want to ensure that all integer columns are converted to the pandas nullable integer type instead of the default numpy int64 dtype. One option could be to specify (somehow somewhere) a type map like {pa.int64(): pd.IntegerDtype} (but how would that work with parametrized types?)

Seems like a little more work is needed here, at least some docstrings. Let me know when you want me to take another look

I think https://github.com/apache/arrow/pull/5162 could already be merged (based on your comment there https://github.com/apache/arrow/pull/5162#issuecomment-538056824)? Then I will update this PR after that.

OK, this should be ready for review now.
I added now support for 2 of the cases of https://issues.apache.org/jira/browse/ARROW-2428:

If there is pandas_metadata, we will check which columns originated from pandas extension dtypes
If you defined a pyarrow.ExtensionType, we check if the ExtensionType.to_pandas_dtype method returns a pandas extension dtype

In both cases, the actual conversion of the pyarrow array to the pandas ExtensionArray is dispatched to the pandas ExtensionDtype.__from_arrow__ method, which is a method on the pandas extension dtype that knows how to convert a arrow array into an extension array of its dtype.


scheduler is  getting halt after first run

Hi ,
I have installed 3.2.0 version of this gem(https://rubygems.org/gems/rufus-scheduler/versions/3.2.0).
I have tried to run the below code  :
require 'rufus-scheduler'
scheduler = Rufus::Scheduler.new
scheduler.in '3s' do
  puts 'Hello... Rufus'
end
scheduler.join

But after printing first 'Hello...' statement , the scheduler halts.After 3 seconds it is not printing the 'Hello..' statement again.
Not sure about the issue.
Below is my system configuration :
CentOS in Vagrant : CentOS Linux release 7.2.1511 (Core)
Ruby Version : ruby 2.0.0p598 (2014-11-13) [x86_64-linux]
Below are the gems installed in my system :
bigdecimal (1.2.0)
bundler (1.13.6)
concurrent-ruby (1.0.2)
daemons (1.2.4)
i18n (0.7.0)
io-console (0.4.2)
json (1.7.7)
lps (0.2.1)
minitest (5.10.1, 5.9.1)
option_initializer (1.5.1)
psych (2.0.0)
rdoc (4.0.0)
redis (3.0.2)
rufus-scheduler (3.2.0)
thread_safe (0.3.5)
tzinfo (1.2.2)

Please upgrade to rufus-scheduler 3.3.1 and try again.

Well, no need to upgrade...
try with
require 'rufus-scheduler'
scheduler = Rufus::Scheduler.new
#scheduler.in '3s' do
scheduler.every '3s' do
  puts "Hello... Rufus #{Time.now.inspect}"
end
scheduler.join

scheduler.in '3s' will run once after 3 seconds. scheduler.join will let the current (here main) Ruby thread join the scheduler thread so that the Ruby process doesn't terminate unless expressly told so.
Best regards.


fix example Trainer API calls

My last PR passes all tests but broke make doc because I didn't update the API in the examples folder. This PR fixes that.

Excellent, thanks @dhpitt, merging.


Auto-create PR with blessed output from latest nightly

Closes #217
Also change CI.yml to trigger its nightly build via Nightly.yml rather than by itself, to make things more neatly organized.

This code managed to create https://github.com/Enselic/cargo-public-api/pull/223 via CI. So with some luck this will also work when used in the wild. Merging and then re-triggering to create a real auto-bless PR.

Does this create another pr if the change is not merged before the next day?

Yep it will
Still tweaking this a bit before I call it a day
not that aspect though


Correction for raw affiliation NARBONNE HOSPITAL Narbonne  France

Correction needed for raw affiliation NARBONNE HOSPITAL Narbonne  France
raw_affiliation_name: NARBONNE HOSPITAL Narbonne  France
new_rors:
previous_rors: 022m8c549
works_examples: W1950693590
contact: 682f5e08c3e7ca91a29eaaff3a99b324:4413ebb1182ac45553459b78d33c9dbf0156 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


SysAdm should alert the user when Life Preserver fails

Not a nagging alert, but there should be at least an unmistakable exclamation mark in lieu of (or overlaying) the SysAdm icon.
Yellow alert for anything that requires non-urgent attention.
Red alert when urgent attention is appropriate.

Housecleaning time.  Many changes have occurred so I'm closing old tickets. If the issue still remains we can reopen.


String.indent

I got what looked like some funky behavior, it didn't look like:
String("\nFOO\n").indent(1)

was indenting as expected, so I should add some tests to make sure it is doing what is expected, I would expect something like (period represents spaces):
....\n
....FOO\n

and I think it might be stripping that last \n because it is the last character

I'm pretty sure the self.splitlines(False) in indent is the problem:
>>> s = "\nFOO\n"
>>> s.splitlines(False)
['', 'FOO']
>>>
>>> s.splitlines(True)
['\n', 'FOO\n']


[BUG] pino and rxjs should be peer dependencies of nestjs-pino

[X] I've read the docs of nestjs-pino
[X] I've read the docs of pino
[X] I couldn't find the same open issue of nestjs-pino
What is the current behavior?
When using pnpm, installing using the --prod flag omits dependencies used within the code, namely, the following:

pino
rxjs

As these are only defined in devDependencies, they are not installed correctly by pnpm.
at the moment, this can be solved with a pnpm readPackage hook to add the correct dependencies in:
function readPackage (pkg, context) {
  if (pkg.name === 'nestjs-pino') {
    pkg.peerDependencies = {
      ...(pkg.peerDependencies ?? {}),
      'pino': '^7.5.0',
      'rxjs': '^7.2.0',
    };
  }

  return pkg;
}

What is the expected behavior?
pino, pino-http, and rxjs are included in "dependencies", so they are included in production installs of the package.
Please provide minimal example repo. Without it this issue will be closed
I will attempt to get this for you soon.
Please mention other relevant information such as Node.js version and Operating System.
pnpm: 7.3.0
node: 14.19.0
nestjs-pino: 2.6.0

Edit: Updated, fix can be achieved by simply adding as peerDependencies and installing those packages within project

Sorry, I'm not a user of pnpm. But rxjs is a dependency of nestjs, and pino-http is a peer dependency, that should be installed by end user, and it includes pino. This peer dependency allows control of it's versions by end user. Do you install both nestjs-pino and pino-http as mentioned in the docs?

I can't claim to say I understand pnpm's resolution patterns fully, but if I had to guess, the nested node_modules structure which pnpm uses places packages in the places a consumer expects them to be based on the package.json of that lib.
In practice, there are issues resolving any pkg which is imported and used directly by a project's codebase if it is not defined either as a peer dependency or a dependency by that project.
The places I had issues:

https://github.com/iamolegga/nestjs-pino/blob/master/src/PinoLogger.ts#L3

PinoLogger imports 'pino' directly

https://github.com/iamolegga/nestjs-pino/blob/master/src/LoggerErrorInterceptor.ts#L7

LoggerErrorInterceptor imports 'rxjs' directly

This is to say, these same modules have no issue with importing @nestjs/common 🤷
I updated my solution above, pino-http was not an issue, only pino and rxjs. Additionally, they were only required to be added in peerDependencies.

So, the solution is to add both pino and rxjs to peer deps, right? So, rxjs version in peer deps should be the same as used in nestjs's minimal supported version. And the same for pino, which is used by pino-http. I believe both of them should be set as a range, right? As this package depends on @nestjs/common ^8.0.0 and pino-http ^6.4.0 || ^7.0.0 If you want to fix that feel free to open PR, but please add the links for me to understand that versions are set properly for both packages.

Since this library is to log nestjs log errors through pino, it seems reasonable that pino be exposed as a peerDependency instead of a devDependency like currently.
The workaround works great but really adds to the friction of your library. My two cents.

So, the solution is to add both pino and rxjs to peer deps, right? So, rxjs version in peer deps should be the same as used in nestjs's minimal supported version. And the same for pino, which is used by pino-http. I believe both of them should be set as a range, right? As this package depends on @nestjs/common ^8.0.0 and pino-http ^6.4.0 || ^7.0.0 If you want to fix that feel free to open PR, but please add the links for me to understand that versions are set properly for both packages.

Hi!
I had the same issue when building an application for prod, with npm, within a docker container.
So I've opened this PR: https://github.com/iamolegga/nestjs-pino/pull/2103

Similar issue with yarn, work around is to use yarn packageExtensions:
packageExtensions:
  "nestjs-pino@*":
    peerDependencies:
      "pino": "*"
      "rxjs": "*"


Add DPH and NPH and cleanup a little

What value is there for having this as a dedicated command over just providing it with the rest of the player stats?


'value' in set from computed get/set is actually 'key'

@computed('estimatedStart', 'realStart')
start: {
  set(value, estimatedStart, realStart) {
     // value => "start"
  }
}

this is fixed as of the latest release, thanks for reporting!


affected/package: method:growsSlice  from runtime/slice.go

What version of Go are you using (go version)?

$ go version
1.17.6

Does this issue reproduce with the latest release?
when i use the keyword of append,according to the slice capacity expansion policy,i choose a slice with the capacity and length of 2 and use the append method to expanse its data capacity,but the result dont match with theory.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
the programming code

What did you expect to see?
the theory result
the 3 line data should be "arr1: 9" but the running result is "arr1:10"
What did you see instead?
the running result

This is expected behavior. The capacity of the array doesn't always match the length of the slice. Since expanding an array (allocate + copy the content) is expensive the array will be somewhat bigger than the slice.

The Go project doesn't use the issue tracker for questions. See https://golang.org/wiki/Questions.


[8.x] Add pg_dump --no-owner and --no-acl to avoid owner/permission issues

Hey,
https://github.com/laravel/framework/commit/fcf175b246a646e81c13fb0e6c212f0b21cdfe51 changed PostgreSQL schema loading to use pg_restore but it fails when you dump schema on one computer and restore it on another (let's say one developer dumps schema, another one loads it) due to owner/permission issues (see below).
You can add --no-owner and --no-acl options to either pg_dump or pg_restore but I think it doesn't make sense to commit a schema with owners/permissions in the case (might be wrong, no problem to change - let me know).
Symfony\Component\Process\Exception\ProcessFailedException

The command "PGPASSWORD=$LARAVEL_LOAD_PASSWORD pg_restore --host=$LARAVEL_LOAD_HOST --port=$LARAVEL_LOAD_PORT --username=$LARAVEL_LOAD_USER --dbname=$LARAVEL_LOAD_DATABASE $LARAVEL_LOAD_PATH" failed.

Exit Code: 1(General error)

Working directory: /Users/rihardssceredins/Code/api-diet

Output:
================

Error Output:
================
pg_restore: while PROCESSING TOC:
pg_restore: from TOC entry 8; 2615 415790 SCHEMA public janisozolins
pg_restore: error: could not execute query: ERROR:  schema "public" already exists
Command was: CREATE SCHEMA public;

pg_restore: error: could not execute query: ERROR:  role "janisozolins" does not exist
Command was: ALTER SCHEMA public OWNER TO janisozolins;

pg_restore: from TOC entry 420; 1255 415886 FUNCTION datediff(character varying, timestamp without time zone, timestamp without time zone) janisozolins
pg_restore: error: could not execute query: ERROR:  role "janisozolins" does not exist

Setting those options on pg_restore would make sense to fix schema dump restoration for those who already upgraded to Laravel 8.8.0, but the release hasn't been marked as latest/release notes published. Let me know what option you think is best.


correct static-color attribute in readme

Thanks for making this component available...
The attribute expected (and used in the example) seems to be "static-color" rather than "color".
Martin

Thanks for your time!!!
Merged it...


Change sidebar colors

Hello everybody,
How I change sidebar color?

Just change the classes_sidebar in config/adminlte.php, if you use Laravel-AdminLTE v3 (AdminLTE v3).

Thank You REJack!!! I will test new version.
I alter color in css files, because My client request a specific color that not exists in AdminLTE

classes_sidebar

Hello friend, you could change it to another color that is not what it comes by default as green, red


[BUG] [fix]: Update Date to Dynamically Display Current Year

Description
The current copyright section of the website reflects the year "2023". To avoid the need to manually update this section every year, I propose implementing a JavaScript solution. The goal is to have the year dynamically displayed, ensuring it automatically updates to the current year.
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
Microsoft Edge
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[ ] I am participating in Hacktoberfest 2023

Sounds interesting @purnasth. Can you elaborate on how this feature would enhance the project?

The website no longer has to hardcode the year after resolving this issue with a straightforward dynamic updating approach, which will undoubtedly enhance this project.

Should I begin working on this issue, @CBID2?

@purnasth
You should see it in the codebase to actually verify.

This might not be the best way to use inspect in next.js project.
Closing this as it already exists!


Per environment state file

It would be really nice if you could use terragrunt with a environment argument, and have it map the argument value to a state file configured.
Something like terragrunt -environment=staging plan and then be able to do something like
remoteState = {
  backend = "s3"
  backendConfigs = {
    encrypt = "true"
    bucket = "my-bucket"
    key = "${environment}.tfstate"
    region = "us-east-1"
  }
}

One of the goals for Terragrunt was to make the state management completely declarative for a given folder. That is, you don't have to remember to set any flags or do anything manually, as managing state manually is precisely what leads to mistakes with vanilla Terraform. Having a flag that changes where state is stored seems to bring some of that manual effort back, which would lead to errors.
It's also worth mentioning that when we use Terraform, we keep separate environments in totally separate folders (and therefore, totally separate tfstate files). This ensures you have true isolation (you can never change prod while working on stage) and if a tfstate file becomes corrupt, it only affects one environment and not all. See Terraform, VPC, and why you want a tfstate file per env for a fun read.
Our typical folder structure looks like this:
stage
 └ main.tf
 └ .terragrunt
prod
 └ main.tf
 └ .terragrunt
global
 └ main.tf
 └ .terragrunt

That way, the key in each .terragrunt file can be hard-coded to the environment you're in, and there is no chance of setting it to the wrong value by accident.

That makes a lot of sense, thanks for the input.


DOC: documentation for swagger workflow

This is just a tiny pr to act as a reminder for how to run this script! I also added a requirements.txt file for local install.

do we also want to add a license here? i tend to default to MIT but we could also use the same license that stravalib uses for consistency which may make more sense.

thanks @jsamoocha !!  i know i've been quiet. pycon recovery took some time. i'll do my best to get to your open pr's and to contribute a bit more in the upcoming week!


[experiment] Check for extended empty tuples in isTupleLikeType

I want to export this in #52467 but it doesn't work when you write:
type EmptyTuple = []

interface MyEmptyTuple extends EmptyTuple {
	extraInfo: any;
}

Because isTupleLikeType determines tuple-ness by checking for a property named 0 which won't exist on an empty tuple.
I don't think this is the final code but I want to see what this breaks. It seems like there's a lot of code which depends on empty tuples not being handled, and I don't know how much that matters.

@typescript-bot test this
@typescript-bot test top100
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this

The above definitely implies this PR is wrong as-is, of course.

#52617 does this but correctly.


assorted gnome updates

Motivation for this change
See commit msg's for links to News
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

We should probably add gdk-pixbuf to clutter-gst dependencies.

We should probably add gdk-pixbuf to clutter-gst dependencies.

Yep, since it's explicit now.

Fixed the gnome-desktop bubblewrap patch.

Thanks @hedning

So the thumbnailing script refuse to run due /var/cache/fontconfig not being present. A simple, but ad-hoc fix is telling gnome-desktop to use PKG_CONFIG_FONTCONFIG_CACHEDIR = "/etc/fonts";.
I guess a more robust fix is only mounting the fontconfig cache if it actually exists.


Fix #33 Maven mojo change ignored

Fix #33
A more finegrained strategy to fix #33

Replaced by #38


Expose MemoVar type

Unfortunately we can't refer to MemoVar otherwise.

(Would it be possible to have a patch release once this is merged?)


frontend: Add dashboard templates, temp-endpoint

This commit adds the basic dashboard templates for the various dashboards. It also adds a new endpoint for temperature data and updates the URL configuration accordingly.

AdminLTE looks good and looks like it's actively maintained.
Their License hint is a little concerning, but I don't think that the risk of a re-licensing to a proprietary license is high. I have not seen them using Contributor License Agreements (CLAs) for contributors, so they would have to get the permission from all contributors (which hold the copyright) in case of a future license change. So all good from my side:
AdminLTE is an open source project by AdminLTE.io that is licensed under MIT. AdminLTE.io reserves the right to change the license of future releases.

@jonas-rem made all the changes as requested. The base frontend is good to go 👍

As discussed, I merged the backend PRs that have been open. As those renamed some database resources (e.g. device -> endpoint) you have to rebase this PR and update the references.

@jonas-rem the following changes have been made in this PR for the first version of the frontend.

[x] Have a minimalist login page
[x] Have the footer only in the licenses page
[x] Update the logo to the latest design
[x] Add a serialiser file that allows to configure a general mapping for the graph dashboard
[x] Update any and all iterations according to the latest changes in the main branch

This should be merge-ready now 👍😄

The server-side functionality (such as buttons for OTA, graph updates, and selection) will be implemented in the next PR. However, that PR can only be reviewed once we address the testing issue with POSIX frameworks, which currently prevents running the simulation.py script.

@jonas-rem the graph functionality now works according to the latest schema. However, since the initial frontend PR was limited to a single client configuration, I have restricted the current setup to filter only based on resource_name, which doesn’t account for different clients.
Should be ready to merge.... Lesss goooo 😁🎉

@jonas-rem The branch should be good for merging and release now. Squashing all commits into one would have been quite challenging, as it would require resolving conflicts from previous merges, so I left the history as it is. I have fixed all the failing tests and pushed the latest changes. The lines of code are now back to 1300. Everything is good to go! 👍😄

@jonas-rem merging this would successfully finish all tasks for release v0.1.0. Less gooo 🎉🎉

Closing. Please catch updates on #68.


v3.0 Beta test results - Download Station

Synology_app_mover v3.0.15
DS920+ DSM 7.2.1-69057-4
did not backup the /volume1/@download directory.
This is where all of the in progress and completed downloads are kept.
I would add a warning when backing up the - /volume1/@download directory that it could take a VERY long time depending on how many gigs or even TB are actively being downloaded and or seeded.
perhaps add a question if the person wants to backup their downloads?

The script checks the size of the @docker folder and the free space on the target volume. I'll make that check a function and use it for all extra folders. And warn that it could take a (long) while if the size is large.
I probably should ask if they want to backup the @downloads folder, but because it would take up double space until the @downloads folder is moved to the target volume it be better to:

Instead of moving the folder copy it to the target volume.
Then rename the source volume's @downloads to @downloads_backup.

I skipped moving the @downloads folder because mine is a few TB. But I don't how I forgot to include code to move it (and @calendar).

Fixed in v3.0.19

confirmed the red text warning that this could take a while
Stopping Download Station

Backing up /volume1/@appconf/DownloadStation
Backing up /volume1/@appdata/DownloadStation
Backing up /volume1/@apphome/DownloadStation
Backing up /volume1/@appshare/DownloadStation
Backing up /volume1/@appstore/DownloadStation
Backing up /volume1/@apptemp/DownloadStation

WARNING Moving @download could take a long time
Backing up /volume1/@download/DownloadStation

Starting Download Station

Finished backing up Download Station

Nice. I haven't actually seen it myself.
Was the read text on the black background easy to read?

Nice. I haven't actually seen it myself.
Was the read text on the black background easy to read?

i thought so, it should at least be enough warning to people that the script is not stuck or something while working.


Debugger doesn't allow code to clean up and hangs

#2  Environment data

VS Code version: 1.37.1
Extension version (available under the Extensions sidebar): 2019.8.30787
OS and version: macOS 10.14.6
Python version (& distribution if applicable, e.g. Anaconda): 3.7.4
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pipenv
Relevant/affected Python packages and their versions: sanic==19.6.3
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): Python Language Server version <IP_ADDRESS>

Behaviour
I have a Sanic server, where when it receives a kill signal it will clean up things such as database connections etc. These can last about a second to complete.
When I run this application via the python debugger in vscode, it doesn't appear as though the runner gives enough time for the process to wrap-up and will hard kill the process during this period of time.
This ends up producing a  RuntimeError: Event loop stopped before Future completed. in the output as well as leading to a broken terminal buffer in the integrated terminal (not just this is the right terminology). It will basically make the terminal responsible, but unable to present the next command prompt. Appears to never return from the command that printed the Event loop error.

Is there a way to increase this grace period? It seems to be set to around 0.1 from my testing.
Steps to reproduce:
Here's a simple Sanic application that produces the error.
import asyncio

from sanic import Sanic

app = Sanic()

@app.listener('after_server_stop')
async def clean_up(app, loop):
    await asyncio.sleep(1)  # set to 0.1 and all is well

if __name__ == '__main__':
    app.run(host='<IP_ADDRESS>', port=8000)

Here's an example project as well:
example.zip

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
User belongs to experiment group 'ShowPlayIcon - control'
> conda --version
> pyenv root
> python3.7 -c "import sys;print(sys.executable)"
> python3.6 -c "import sys;print(sys.executable)"
> python3 -c "import sys;print(sys.executable)"
> python2 -c "import sys;print(sys.executable)"
> /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/bin/python -c "import sys;print(sys.executable)"
> python -c "import sys;print(sys.executable)"
> pipenv --version
cwd: /var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
> conda info --json
> ~/.asdf/shims/python3.6 ~/.vscode/extensions/ms-python.python-2019.8.30787/pythonFiles/interpreterInfo.py
> ~/.asdf/shims/python3.6 ~/.vscode/extensions/ms-python.python-2019.8.30787/pythonFiles/interpreterInfo.py
Starting Microsoft Python language server.
> pipenv --version
cwd: /var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
> pipenv --venv
cwd: /var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
> /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/bin/python -m flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/main.py
cwd: /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
> /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/bin/python -m flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/main.py
cwd: /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
> pipenv --venv
cwd: /var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
> pipenv --py
cwd: /var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
##########Linting Output - flake8##########
[Info  - 15:04:06] Analysis cache path: /Users/james/Library/Caches/Microsoft/Python Language Server
[Info  - 15:04:06] GetCurrentSearchPaths /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/bin/python
> pipenv --py
cwd: /var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p
[Info  - 15:04:06] Interpreter search paths:
[Info  - 15:04:06]     /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/tloul1pa.bgi
[Info  - 15:04:06]     /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/lib/python3.7
[Info  - 15:04:06]     /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/lib/python3.7/lib-dynload
[Info  - 15:04:06]     /usr/local/Cellar/python/3.7.4/Frameworks/Python.framework/Versions/3.7/lib/python3.7
[Info  - 15:04:06]     /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/lib/python3.7/site-packages
[Info  - 15:04:06] User search paths:
[Info  - 15:04:07] Microsoft Python Language Server version <IP_ADDRESS>
[Info  - 15:04:07] Initializing for /private/var/folders/5v/x8dcvr4n2wl0dlq2pxd35y2w0000gn/T/pyground.XXXXXXX.DBw9qa7p/.venv/bin/python
> conda info --json
> conda --version

Issue moved to microsoft/ptvsd #1746 via ZenHub


Moved the version in DownloadCommand

This fixes #204

:+1: Always wondered why it was like it. BTW, you could also move $downloadedFilePath and $requirementsErrors IMO.
Also, it makes no sense the DownloadCommand has methods with @return NewCommand in the PhpDoc block and deals bad with autocompletion.

@ogizanagi done :)

@Pierstoval thanks for providing such a fast bugfix!


Added ability to build odk-docs via dockerfile

To help us preview API docs without need to setup getodk/docs repo.
You should be able to see ODK docs by running make api-docs
Before submitting this PR, please make sure you have:

[x] run make test-full and confirmed all checks still pass OR confirm CircleCI build passes
[x] verified that any code from external sources are properly credited in comments or that everything is internally sourced

What do you think about adding a sentence about make api-docs to the readme or the contributing guide? I feel like it's more likely to be discovered that way.

any quick ideas why i'd get this error?
$ sudo make api-docs
(test "" || docker build --file odk-docs.dockerfile -t odk-docs .) && \
	docker run --rm -it -v ./docs/api.yaml:/docs/docs/_static/api-spec/central.yaml -p 8000:8000 odk-docs
[+] Building 0.3s (2/3)                                                        docker:desktop-linux
[+] Building 0.3s (3/3) FINISHED                                               docker:desktop-linux
 => [internal] load .dockerignore                                                              0.0s
 => => transferring context: 578B                                                              0.0s
 => [internal] load build definition from odk-docs.dockerfile                                  0.0s
 => => transferring dockerfile: 385B                                                           0.0s
 => ERROR [internal] load metadata for docker.io/sphinxdoc/sphinx:latest                       0.3s
------
 > [internal] load metadata for docker.io/sphinxdoc/sphinx:latest:
------
odk-docs.dockerfile:1
--------------------
   1 | >>> FROM sphinxdoc/sphinx
   2 |
   3 |     RUN apt-get update && apt-get install -y libenchant-2-2 git
--------------------
ERROR: failed to solve: sphinxdoc/sphinx: error getting credentials - err: exit status 1, out: ``
make: *** [api-docs] Error 1

Did you switched from Docker Desktop to OrbStack? Can you please try running docker pull sphinxdoc/sphinx, it will ask for the access for osxkeychain

Credential store
OrbStack uses the open-source macOS keychain credential store (osxkeychain) to store registry credentials securely.
Docker Desktop uses a different credential store (desktop), so you may need to log in again after switching to OrbStack. If you use them side-by-side, you may need to fix the store in ~/.docker/config.json.

I don't know why OrbStack need credentials to pull public image 🫤


Example code is crashing

Hi, thanks for sharing this library. I've been having problems getting it to work on my arduino uno.
Below is the example file, just with a few Serial.println commands added. It runs fine, until I uncomment the kin.inverse line.  Then, I just get junk on the serial console, like:
⸮)⸮⸮⸮⸮⸮)⸮⸮⸮⸮

Any ideas what could be going wrong? The arduino works fine with other code.
#include "Kinematics.h"
#include "MatrixUtils.h"

#define N 3

void setup() {
    // put your setup code here, to run once:
    Serial.begin(9600);
    Serial.println("hello");

    Kinematics kin(N);
    MatrixUtils mat_utils;
    Serial.println("well.");

    kin.add_joint_axis(0, 0,  1,  4, 0,    0);
    kin.add_joint_axis(0, 0,  0,  0, 1,    0);
    kin.add_joint_axis(0, 0, -1, -6, 0, -0.1);
    Serial.println("added joints.");

    kin.add_initial_end_effector_pose(-1, 0,  0, 0,
                                       0, 1,  0, 6,
                                       0, 0, -1, 2,
                                       0, 0,  0, 1);
    Serial.println("added nose pose.");

    float desired_transform[4][4] = {
        {0, 1,  0,     -5},
        {1, 0,  0,      4},
        {0, 0, -1, 1.6858},
        {0, 0,  0,      1}
    };

    float jac[6][N];
    float jac_t[6][N];
    float AA_t[6][6];
    float A_tA[N][N];
    float pinv[N][6];

    float joint_angles_0[N] = {1.0, 2.5, 3};
    float joint_angles[N];

    // kin.inverse((float*)desired_transform, (float*)jac, (float*)pinv, (float*)jac_t, (float*)AA_t, (float*)A_tA, joint_angles_0, 0.01, 0.001, 20, joint_angles);
    // mat_utils.print_matrix(joint_angles, 1, N, "Joint angles");
}

void loop() {
  // put your main code here, to run repeatedly:

}

Hi, I'm sorry to inform you that unfortunately, this library does not support boards like the Arduino Uno right now. The inverse kinematics step requires large amounts of matrix manipulations and optimization therefore the memory on a board like the Arduino Uno is just not enough. You could try running the program on an Arduino compatible board that has a larger memory such as the Mega or Due and it should run fine.
In the meanwhile, I am working on certain memory optimization steps which could help resolve the issue.

Aha, thanks for your fast and helpful reply @kousheekc! That explains my issue. I'll look for another arduino.


Fixed an issue where parsed coordinates are strings

Better matches the format from here: http://geojson.org/

Thanks for this. Going to merge this in, add some tests, and then push a release.

I just pushed v0.4.0 to npm, which includes this change. Thanks again.

No problem :-)


pythonPackages.osqp: remove mkl dependency

Motivation for this change
Removing mkl allows this to be built in Hydra (mkl is unfree-ish license), and it seems the mkl dependency is somewhat optional given downstream packages build cleanly when removing mkl.
Also add explicit numpy/scipy dependency.
Cleanup commit slightly for formatting.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python37Packages.osqp python37Packages.cvxpy python38Packages.osqp python38Packages.cvxpy

CC @matthewbauer, who is looking at making this type of thing parameterizable in https://github.com/NixOS/nixpkgs/pull/83888

They seem to be used in the code, it's just not listed in requirements
which I used when originally packaging.
On Thu, Apr 2, 2020, 2:01 PM Frederik Rietdijk<EMAIL_ADDRESS>wrote:

@FRidh commented on this pull request.
In pkgs/development/python-modules/osqp/default.nix
https://github.com/NixOS/nixpkgs/pull/84124#discussion_r402508396:

 future

numpy

Why add these new dependencies? Are they required for the package?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/84124#pullrequestreview-386669119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNZYI6HH3236ZXOZ7WZ6O3RKTHGBANCNFSM4L2VTSEQ
.

This case is a little different because osqp doesn't use MKL for BLAS, but MKL PARDISO (https://software.intel.com/en-us/mkl-developer-reference-fortran-intel-mkl-pardiso-parallel-direct-sparse-solver-interface) which is unique to MKL.
Anyway, it looks like this library just needs MKL to test the MKL functionality. Since osqp just uses dlopen, you can always enable MKL support through LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$(nix-build '<nixpkgs>' -A mkl)/lib.

This case is a little different because osqp doesn't use MKL for BLAS, but MKL PARDISO (https://software.intel.com/en-us/mkl-developer-reference-fortran-intel-mkl-pardiso-parallel-direct-sparse-solver-interface) which is unique to MKL.
Anyway, it looks like this library just needs MKL to test the MKL functionality. Since osqp just uses dlopen, you can always enable MKL support through LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$(nix-build '<nixpkgs>' -A mkl)/lib.

Ack, thanks for the good tips. I wasn't familiar with what MKL PARDISO did. pythonPackages.osqp does just use MKL PARDISO for testing PARDISO functionality AFAICT, it probably also enables some runtime functionality but I view that as a downstream thing. I prefer for build purposes to have no unfree dependencies so as much as possible can be built/cached by Hydra.

@GrahamcOfBorg build python37Packages.qiskit-aer python37Packages.qiskit-ignis
@GrahamcOfBorg build python38Packages.qiskit-aer python38Packages.qiskit-ignis

Result of nixpkgs-review pr 84124
9 package built:
python27Packages.osqp python37Packages.cvxpy python37Packages.osqp python37Packages.qiskit-aer python37Packages.qiskit-ignis python38Packages.cvxpy python38Packages.osqp python38Packages.qiskit-aer python38Packages.qiskit-ignis

Can you send a 20.03 backport PR as well?

@bhipple python3Packages.osqp isn't in 20.03.

Ah ok, my bad!  For some reason I thought we'd had this packaged for longer, but I was thinking of another related pkg.


getLatestTweet and getTweets not showing results

Does anyone experience issues with getLatestTweet and getTweets showing no results when the user clearly has posts? I am not authenticated when I make these function calls.
I attempted it with username: devjumzzz and while the user has posts on their twitter account, I am getting no results when running these two functions.
Note, I am using a JS implementation of the twitter-scraper codebase, which may account for some differences, but I am thinking it may be due to something else.
Thank you

Twitter removed ability to see user feed without auth, you can check by open any profile in browser incognito mode. It's making response to UserTweets but return result with no tweets, so it doesn't throw any error.

Thank you for the quick response. Since authentication is required, it seems that rate limits will be more impactful. Do you happen to know what the rate limits are for calls to the API for UserTweets?

@im-kent maximum 150 requests per 15 minutes though i recommend use less around 100 request should be safe per account

Thank you!


Text edits

I added the following texts:

vertical widget use
api Earthdata login explanation
more on dataviz
caps/italic fixes

@lisakaser the method for checking the order status in the API notebook  is wrong, I forgot to change it to check_order_status currently says order_status  could you update it and we merge this.

ok, now I think it should be good to go!


I have tried every tutorial but ntellisense is not working

Hi,
I am using Macbook with Visual Studio Code. I have installed and followed every instructions but still i am unable to work the auto suggestions.
So basically Eloquent Autocompletion and route autocompletion not working all others are working.

Hi @khanakia!
What are you using on your development environment (Valet - Docker - Homestead ..)?

@simoebenhida
No i am not using any Development Environment. I am using simple Laravel PHP installation on my local macbook.
Please check the below link i have created a video.
https://www.dropbox.com/s/la4lv60nncjwr8o/Screen Recording 2020-07-20 at 9.08.15 AM.mov?dl=0

What do you get when you run:
php artisan tinker
App\User::first();

What do you get when you run:
php artisan tinker
App\User::first();

Please check the screenshot.

Hmm, it looks good!
Can I get a screenshot of your vscode process explorer when writing the query autocompletion

Hmm, it looks good!
Can I get a screenshot of your vscode process explorer when writing the query autocompletion

Please find the video link
https://www.dropbox.com/s/la4lv60nncjwr8o/Screen Recording 2020-07-20 at 9.08.15 AM.mov?dl=0

Sorry! I couldn't see the process explorer on the video! I think you sent me the first video again

Sorry. Check this one https://www.dropbox.com/s/xwybctisu96pja1/Screen Recording 2020-07-20 at 10.11.22 AM.mov?dl=0

Thanks! It looks that the script is not even fired!
Hmm, make sure that the user table has at least one record! Because the package gets the columns from the database!
Also, the autocompletion will not happen when you do whenColumn('') because it autocompletes columns after all!

From looking at web.php it makes sense that you don't have any route autocompletion! You should have named routes!

@simoebenhida  Thank you very much 💯 👍
It worked. You should have mentioned this on the docs also. So new users will know that it will only work with named routes instead of opening a issue.

@khanakia glad it works! Yeah, I mentioned the database thing on the Readme! Feel free to contribute if you think it needs something else!


Why is there an inverted triangle in the transparent background?

I think it's because of android system shadow.

Try adding bnbElevation=0 in your xml to BottomNavigationBar.
If you are not using xml for you can add "bottom_navigation_elevation" to your dimens file with 0 as value


DDL not include default value type sequence: nextval ('sequence_name':regclass)

System information:

Windows 10 64 Bit
DBeaver 21.3.4

Connection specification:

PostgreSQL
Server: PostgreSQL 13.3 (Debian 13.3-1.pgdg100+1)
Driver: PostgreSQL JDBC Driver 42.2.20
Not using tunnels or proxies

Describe the problem you're observing:
Generated DDL not include default value type sequence: nextval ('sequence_name':regclass)
Steps to reproduce, if exist:
Databases -> Tables -> <table_name> -> Properties -> DDL
Databases -> Tables -> <table_name> -> right click -> Generate SQL -> DDL
Columns:

DDL:

Hello @peterprvy,
The SERIAL types family are implicitly autoincremented by default. The default value may not be specified for such attributes.

@ShadelessFox
Intellij Idea identify column as integer, not serial:

DDL:

Integers do require the nextval expression as their default values.

@ShadelessFox
pgAdmin identify columns as integer, not serial

please reopen issue

@ShadelessFox
Dbeaver badly found data type, Pleasr reopen issue

It's a feature. The original type is indeed an integer.
If the attribute contains the nextval expression as its default value, DBeaver changes its type to serial.

But there is no mention of this in DDL and when creating a table in another database I forget to create a seqeunce.

The SERIAL type is a sugar that PostgreSQL provides.
You can try creating a new database containing an attribute of type SERIAL, and it will erase it down to int with nextval as its default value. A sequence is created automatically.


Tensor property

I don't understand why my branches always include commits that have already been submitted. I always pull and branch from master so I don't know what is wrong

https://github.com/google/TensorNetwork/issues/63


No versions to install?

Thanks for providing this.
When I try to install I get this error:
Tried pip3 install excel_solver and pip3 install excel-solver
ERROR: Could not find a version that satisfies the requirement excel_solver (from versions: none)
ERROR: No matching distribution found for excel_solver

Any ideas?

It's likely you need a newer python version. It's listed as requiring >=3.10 on pypi, but I think it would work with 3.9 if you fork/clone and update the version in the pyproject.toml


Add transitions for undo/redo slide add/delete/move

On undo and redo, make sure the slide list transitions occur. If a user clicks undo/redo multiple times fast, make sure the slide list doesn't lag or get into an unhealthy state.

On Undo, the slide list doesn't transition.


Passing arguments to underlying invocations of cargo

I wonder if it is possible to run dylint lint pass with -Z build-std passed to all underlying cargo invocations. One of the lints I am writing depends on having standard library MIR accessible during the compilation. While it is possible to achieve this by configuring the crate on which the lint runs, it would be ideal to have this behavior out of the box for the end user.

Might this work?
cargo dylint ... -- -Z build-std

The arguments after -- are forwarded to cargo check, which cargo dylint runs under the hood.

That worked! Thanks!

A follow-up on the previous question: is it possible to do the same thing with dylint_testing?

Hmm. dylint_testing is a wrapper around compiletest-rs, which runs rustc directly. Furthermore, dylint_testing has a "test builder" that allows one to pass flags to rustc.
For example, here it used to pass the --test flag to rustc: https://github.com/trailofbits/dylint/blob/39c6cd94fc697f90bf9c00e3956ecfb2f6688778/examples/general/abs_home_path/src/lib.rs#L155-L160
So if one could figure out what flags Cargo passes to rustc when -Z build-std is enabled, one could pass them to rustc using a test builder.
At least, I think that should work.

Thanks again! Unfortunately, it was more complicated than just passing flags to rustc, so we ended up building a simple test harness that runs dylint directly and diffs the outputs.

Thanks, I was curious to know how this went. Can I ask what more was required?

From what I could gather, it would require manually resolving std, setting up a fake workspace for it, and then linking it as an extern. If you are interested, this is the first PR for cargo build-std that briefly explains how they do it

Thanks for the explanation. It is unfortunate that dylint_testing did not help here, but I am glad you were able to find a solution.


Using 2021.1-eap-3 and the JavaScriptLanguage JB plugin, the built in transitive dependency CSS is not activated at runtime

Using version v0.7.2 of the gradle-intellij-plugin and 2021-eap-3:
intellij {
    ...
    plugins 'JavaScriptLanguage'
}

Fails at runtime because it does not pull the required transitive CSS plugin. Using a 2020.3 profile with v0.7.2 of the gradle-intellij-plugin, this works as expected.

@hunterwerlla Could you please provide your Gradle configuration file?

Yep, here is where we specify the plugins: https://github.com/aws/aws-toolkit-jetbrains/blob/master/buildSrc/src/software/aws/toolkits/gradle/IdeVersions.kt#L80
And here is where they are applied: https://github.com/aws/aws-toolkit-jetbrains/blob/28244203b9acd17eea5f31626f01a8a94b51801c/jetbrains-ultimate/build.gradle.kts#L23
Which we just OR together from the list: https://github.com/aws/aws-toolkit-jetbrains/blob/28244203b9acd17eea5f31626f01a8a94b51801c/buildSrc/src/software/aws/toolkits/gradle/IdeVersions.kt#L132

is this still an issue with 2021.1 final?

is this still an issue with 2021.1 final?

I will check Yann after the Rider 2021.1.0 sdk issue is resolved, see https://github.com/aws/aws-toolkit-jetbrains/pull/2551

Yes, it still appears to be an issue:
    2021-04-12 15:52:39 WARN  PluginManager:26 - Problems found loading plugins:
      The JavaScript and TypeScript (id=JavaScript, path=/codebuild/local-cache/custom/e3753f6254b925d6ff833e213fc1705723cd512ae537636977eeb06cc91c3c94/root/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIU/2021.1/fb73193bad7ac7619a8bfac40db2cb035e38bd13/ideaIU-2021.1/plugins/JavaScriptLanguage) plugin Plugin 'JavaScript and TypeScript' requires plugin 'com.intellij.css' to be installed
      The JavaScript Debugger (id=JavaScriptDebugger, path=/codebuild/local-cache/custom/e3753f6254b925d6ff833e213fc1705723cd512ae537636977eeb06cc91c3c94/root/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIU/2021.1/fb73193bad7ac7619a8bfac40db2cb035e38bd13/ideaIU-2021.1/plugins/JavaScriptDebugger) plugin Plugin 'JavaScript Debugger' requires plugin 'JavaScript and TypeScript' to be enabled


Feature Idea - Bounce

Hi there! Just wanted to make a suggestion for a new animation. Id like an element to be visible on page but jump or bounce or wiggle when the user scrolls to the element. I love the library and thank you!!!

Hey @kristenzirkler ! Thanks for the suggestion, however I'd like to keep animations as simple as possible and those not very often used leave for the users to add in their projects on their own. You have an example of how to create your own animations in readme (there is even example codepen)  :)


Tooltips needed everywhere possible

We have a tooltip component in webclient and it's easy to implement for any components.
Would like you to specify the places for displaying tooltips.

@elite0226 when a button is disabled we need to show the user why, for example

I think this issue is also duplicated, right?

The issue is too generalized, we will need to make it more specific @alwyn-ixo lets figure out where we can add specific tooltips.

tested as far as possible without knowing exactly what is required


Secret credentials support for CreateVolume(From)Snapshot

Change Overview

Pull request type
Please check the type of change your PR introduces:

[ ] Work in Progress
[ ] Refactoring (no functional changes, no api changes)
[ ] Trivial/Minor
[ ] Bugfix
[X] Feature
[ ] Documentation

Overall looks good to me @hakanmemisoglu will let @tdmanv accept the PR.


Issue/PR Search Tables broken by new Labels columns

Describe the bug
Receive the below error when attempting to utilise the github_search_issue table with query org:turbot is:open

ERROR: rpc error: code = Unknown desc = failed to populate column 'labels': rpc error: code = Internal desc = transform getIssueLabels failed with panic interface conversion: interface {} is struct { TextMatches []models.TextMatch; Node struct { models.Issue "graphql:\"... on Issue\"" } }, not models.Issue

Steampipe version (steampipe -v)
Steampipe v0.20.9
Plugin version (steampipe plugin list)
v0.30.0
To reproduce
select
  *
from
  github_search_issue
where
  query = 'org:turbot is:open';

Expected behavior
Obtain data, not error.

Issue stems from the getIssueLabels func which has
issue := d.HydrateItem.(models.Issue)

However in the case of the search tables it's being passed a different struct type - will look to alleviate this issue.

Swapped transform function to take input from labels_src column instead of attempting to parse from main struct.


How to read Object in nlog Layouts

I wanted to maintain some common class for logging related information like as below.
 public class LogObject
    {
        public string AppName { get; set; }
        public string StackTrace { get; set; }
        public string Message { get; set; }
        public string JobNumber { get; set; }
        public string TransactionType { get; set; }
    }

Adding entire log object in API, like as below.
 logMgr.Info<LogObject>(logObject);

now i need to print those information into log like as below.
 <layout xsi:type="SimpleLayout"
               text="AppName-- ${logObject.AppName}
                     Message-- ${logObject.Message}"
              >
      </layout>

But above layout render is not working, i am unable to read the object like logObject.Message.
i know other way is adding individual properties into variable and events, i am more looking addding entire object into log and reading in layout.

Maybe you can to do something like this:
nLogger.Info("{0}", logObject);
Then register a custom layoutrenderer for each LogObject-property:
NLog.LayoutRenderers.LayoutRenderer.Register("logObject-AppName", (logEvent) =>
{
    if (logEvent.Parameters?.Count == 1)
    {
        LogObject logObject = logEvent.Parameters[0] as LogObject;
        if (logObject != null)
              return logObject.AppName;
    }
    return string.Empty;
}

Then your layout can look like this:
 <layout xsi:type="SimpleLayout"
               text="AppName-- ${logObject-AppName}
                     Message-- ${logObject-Message}"
              >
 </layout>

Thanks Snakefoot for quick response and providing the solutions.
I am new to Nlog,  i  am looking for is there any possibility of accessing object members directly in Layout without writing custom code.

I assume your question has been answered, if not, let us know!

NLog 4.6.3 introduces objectpath= as option, so you can specify ${event-properties:item=logObject:objectpath=AppName}. See also #3329


Add custom css for docs

Current dependencies on/for this PR:

main

PR #26   👈

This stack of pull requests is managed by Graphite.

Pull Request Test Coverage Report for Build #63
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build #62:
0.0%

Covered Lines:
249

Relevant Lines:
249

💛  - Coveralls


Cannot read property 'getHostNode' / 'Text' of null

Getting this error while importing the data into DejaVu UI.

It would be great if you can share

Which version of ElasticSearch you are using
What kind of file you are trying to import
Whether you are using the Web version of Dejavu, a chrome extension or a self-hosted version.

ES: 7.4.2
JSON file
Web version

@brendanwheble thank you for providing this information, I will try replicating the issue with the above version and let you know.


[Async] Use of async function throws Syntax error

Adding code form http://www.reactnative.com/react-native-meets-async-functions/
 async function fetchJSONAsync(url) {
    let response = await fetch(url);
    let body = await response.json();
    return body;
 }
The above code throws a syntax error

You need to have babel configured. There's a section in the post about how to do that.

Sorry my bad. Did not see the link to medium. Will look into it. thanks for the timely response


Add end-to-end checksum to outgoing requests. flag controlled so user…

…s can turn it off

I am not able to add reviewers, so for some reason.
Would like to add these reviewers: @BenWhitehead @crwilcox and @kolea2

Thanks for the review..

urlencoding of the checksum http header - that is a can of worms, it seems.
I need to step back and present performance consideration aspects of this step.
give me a little time to put that answer together.

I think this encoding + performance issue is not a complicated one. let me explain.
checksum is MD5. 16 bytes. and then it is converted to a hex string of 32 bytes. it is, by default, http transmision-safe (they are 32 hex chars).
so encoding in the header is a non-issue.
is performance of converting 16 bytes to 32 hex chars an issue? IMO, not that concerning.
Your comments please.. thanks for taking time to review!

Hello Reviewers, I think I have addressed the comments. PTAL. appreciate your help

Hello Reviewers, I have added checksum on the response side also.
PTAL when you can.
thanks


Install chef-client omnibus on docker volume (#26)

This patch allows to install chef-client omnibus on docker volume instead copying and installing it inside container. Image size after is reduced by about 150Mb. See #26 for more.
It is also allows to use volumes during convergence.

Looks good on a quick scan--I'll read it through carefully tomorrow.
Can you get some tests around this to ensure we don't regress? I just merged an RSpec scaffold to master, and I'm happy to help you get up and running with it.

Sure, will do.

@randomcamel , added feature tests.

I have discovered that this feature doesn't work with machine_batch at the moment. Have to make certain adjustments to fix that.

Closing due to it is incompatible with #56 .


Fixes #60 Removed "operators" section from online help

Pull Request Test Coverage Report for Build 136

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 90.489%

Files with Coverage Reduction
New Missed Lines
%

.tox/py2/lib/python2.7/site-packages/friendlyshell/shell_help_mixin.py
1
98.67%

Totals

Change from base Build 134:
-0.1%

Covered Lines:
1626

Relevant Lines:
1782

💛  - Coveralls


xc contract

add Ink-tone contract.
add Ink-xc contract.
add xc-plugin contract.
add README.md

xc contract

modify sign


chore: list command namespace

Adds namespace test to list command
Moves namespace flag into list command for

Better help text
Easier testing

Sets static default namespace for flag to "default"

/kind techdebt

@lkingland unit test running with current main merged in are failing.

Assert need adjustment and/or you need also rebase.

/approve
/lgtm

/lgtm

/approve

@lance can you override the coverage thing? It's obviously broken.


Minor: Allow to change ruby used to run native helpers #6517

Fix for #6517
I have added a ruby_cmd option to the FileParser class so that updating the gem is done with the context of jruby for example (thus installing proper native extensions)
It can be used like this:
@parser = Dependabot::FileParsers.for_package_manager('bundler').new(dependency_files: @files, source: @source, credentials: @credentials, options: { ruby_cmd: '/home/ylecuyer/.rbenv/versions/jruby-<IP_ADDRESS>/bin/jruby' })

This implementation makes sense to me. Are you self-hosting Dependabot with JRuby installed?
GitHub hosted Dependabot doesn't ship with JRuby, so I would expect the custom ruby_cmd to fail

I do have dependabot self-hosted indeed. It isn't running with jruby but instead with ruby.
Here I need custom jruby because for jruby project the diff created by dependabot brings dependencies with the ruby platform and not Java platform. In https://github.com/dependabot/dependabot-core/issues/6517#issuecomment-1483581368 I give an example diff of the Gemfile.lock with this change.

GitHub hosted Dependabot doesn't ship with JRuby, so I would expect the custom ruby_cmd to fail

The default isn't changed and will keep using ruby unless you explicitly override it, so in the end the change should still be compatible with github hosting.

I don't know a lot about jruby, but I'm curious why you don't run Dependabot entirely using Jruby? That'd solve this for you.

We have multiple projects some running ruby and some jruby, our dependabot setup runs from one place only and is configured to scan all projects and create associated PRs.
Contrary to other dependabot users, we do not update all dependencies when an update is available. We only update dependencies having vulnerabilities. Thus when we receive a CVE alert (like nokogiri in the example which led to this PR) we run the dependabot pipeline which will patch all projects. That's why we do not have some automated process running from every projects.

We could ship this as a short-term thing

That would be nice, I'm already using my fork with the patch with good results so far.


Sharpen alignment between map layers

Users want to toggle between layers of the core map to understand context
There appears to be misalignment between the layers which can reduce confidence in the fidelity of the layers
How closely can we get these layers to align?

How closely can we get these layers to align

Some misalignment is expected in any warped layers. The control points will have to be continuously tweaked for a more perfect fit
Let's capture screenshots of examples of misalignments with location link here. Will give a better idea of how to resolve this.

And maybe we should add a prominent disclaimer mentioning one needs to check authenticity of the data with the respective sources.
We don't want to give a false impression of accuracy when in reality none of the data sources are fully accurate. Each layer has a different dimension of error and it becomes more prominent when stacked together.

Ok, yes. Starting to want an intro page or short explainer somewhere

There is nothing actionable here since alignment will be improved over time with feedback.


save_dvec() and load_dvec() are not exported

https://github.com/joachimbrand/Rimu.jl/blob/cf90d2cca8545adc1b9751ef11777605827beb84/src/RimuIO.jl#L16
I guess this is not intentional, I see no reason to not export these two functions.

You're right. I had the wrong idea that these functions were unexported and only to be used as RimuIO.(load|save)_(df|dvec). Fixed in #114