DNM: dts: bindings: usb: Add maximum-speed property

Add maximum-speed property to usb node.
It configures USB controllers to work up to a specific speed.
Valid arguments are 0 for super-speed, 1 for high-speed, 2
for full-speed and 3 for low-speed.
Depends on #8858

@jli157 and @aurel32 could you also review it and test is on your boards?

Codecov Report

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

@@           Coverage Diff           @@
##           master    #9062   +/-   ##
=======================================
  Coverage   52.41%   52.41%
=======================================
  Files         198      198
  Lines       24912    24912
  Branches     5185     5185
=======================================
  Hits        13058    13058
  Misses       9754     9754
  Partials     2100     2100

Continue to review full report at Codecov.

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

@ydamigos it is weird that I still have to add the line
@@ -256,6 +256,7 @@ static int usb_dc_stm32_clock_enable(void)
 #else
        /* Disable ULPI interface (for external high-speed PHY) clock */
        LL_AHB1_GRP1_DisableClock(LL_AHB1_GRP1_PERIPH_OTGHSULPI);
+       LL_AHB1_GRP1_DisableClockLowPower(RCC_AHB1LPENR_OTGHSULPILPEN);
 #endif /* USB_HS_PHYC */

here to make my STM32F4 board work on its USB-HS full-speed mode.  I found a link which also stated the similar issue.

@jli157 Thanks for testing. The reset value for RCC_AHB1LPENR register is 0x7EEF 97FF which enables USB OTG HS ULPI clock during Sleep mode. I will fix it.

@jli157 I updated it. Could you test it again?

Thanks @ydamigos for this patch. I have tested it on my STM32F723 board and it works. I have a few comments though:

Setting speed to low doesn't work for both USB ports on the board, FS is used instead. After looking quickly at the code, it seems to be a limitation from the STM32 HAL.
When an unsupported speed is requested (e.g. super speed on the STM32F723) board, the selected speed is FS instead of HS on the port with the HS PHY. On the other hand, I think it might be better to change the code to fail at compile time instead of printing a message to the console that can be missed.
At some point we'll need to add a function to get the actual USB speed used, as some descriptors might be slightly different. For example interrupt and isochronous endpoints give their polling rate in term of frames, ie every 125µs in HS or 1ms in FS. I therefore think the name usb_dc_stm32_get_speed can be confusing, you can maybe use something like usb_dc_stm32_get_max_speed.

@aurel32 Thanks for testing. Could you test it again?

Setting speed to low doesn't work for both USB ports on the board, FS is used instead. After looking quickly at the code, it seems to be a limitation from the STM32 HAL.

The reference manual also mentions for the OTG_HS/FS_DCFG register:
DSPD[1:0]:  Device speed
Indicates the speed at which the application requires the core to enumerate,
or the maximum speed the application can support. However, the actual bus
speed is determined only after the chirp sequence is completed, and is based
on the speed of the USB host to which the core is connected.

When an unsupported speed is requested (e.g. super speed on the STM32F723) board, the selected speed is FS instead of HS on the port with the HS PHY. On the other hand, I think it might be better to change the code to fail at compile time instead of printing a message to the console that can be missed.

Done

At some point we'll need to add a function to get the actual USB speed used, as some descriptors might be slightly different. For example interrupt and isochronous endpoints give their polling rate in term of frames, ie every 125µs in HS or 1ms in FS. I therefore think the name usb_dc_stm32_get_speed can be confusing, you can maybe use something like usb_dc_stm32_get_max_speed.

Done

I think this PR is ready to be merged.  Can any reviewer give approval to it? I like to have this one in my project. :)

@galak I'll do the changes to use string in max-speed property. Right now the driver fails to compile if unsupported speed is defined in device tree:
#if (CONFIG_USB_MAXIMUM_SPEED != USB_DEVICE_MAX_SPEED_HIGH) && \
    (CONFIG_USB_MAXIMUM_SPEED != USB_DEVICE_MAX_SPEED_FULL) && \
    (CONFIG_USB_MAXIMUM_SPEED != USB_DEVICE_MAX_SPEED_LOW)
#error "Unsupported maximum speed defined in device tree"
#endif

I would like to inform user if an unsupported speed is defined in device tree. I can't use #if directive with strings. Should I use an error message using the SYS_LOG or LOG subsys at runtime or is it possible to restrict the values supported on device tree level?

I would like to inform user if an unsupported speed is defined in device tree. I can't use #if directive with strings. Should I use an error message using the SYS_LOG or LOG subsys at runtime or is it possible to restrict the values supported on device tree level?

We should probably put this check/parsing in the device tree generation script.  Having some means in the yaml to describe a set of know string values that get checked against.

We should probably put this check/parsing in the device tree generation script. Having some means in the yaml to describe a set of know string values that get checked against.

I am not sure it's something possible as the allowed values depends on the controller version, which is not known at the time the device-tree is parsed. For example some STM32F7 have an HS internal PHY, while on others the internal PHY is limited to FS. This is known by checking the USB_HS_PHYC defined in the HAL code.

I am not sure it's something possible as the allowed values depends on the controller version, which is not known at the time the device-tree is parsed. For example some STM32F7 have an HS internal PHY, while on others the internal PHY is limited to FS. This is known by checking the USB_HS_PHYC defined in the HAL code.

We are responsible the device tree to have the correct values for each SoC. The check/parsing is just to avoid typos or invalid values (e.g. "super-fast-speed").

I am not sure it's something possible as the allowed values depends on the controller version, which is not known at the time the device-tree is parsed. For example some STM32F7 have an HS internal PHY, while on others the internal PHY is limited to FS. This is known by checking the USB_HS_PHYC defined in the HAL code.

Its annoying, but we should have different compatibles for this, and have different yaml if we want to error check the values.

Its annoying, but we should have different compatibles for this, and have different yaml if we want to error check the values.

Could we use phy bindings like it is done in linux, instead of dfferent compatibles and different yaml?
For example in stm32429i-eval.dts in linux::
usbotg_hs_phy: usbphy {
	#phy-cells = <0>;
	compatible = "usb-nop-xceiv";
	clocks = <&rcc 0 STM32F4_AHB1_CLOCK(OTGHSULPI)>;
	clock-names = "main_clk";
};

&usbotg_hs {
	dr_mode = "host";
	phys = <&usbotg_hs_phy>;
	phy-names = "usb2-phy";
	pinctrl-0 = <&usbotg_hs_pins_a>;
	pinctrl-names = "default";
	status = "okay";
};

Could we use phy bindings like it is done in linux, instead of different compatibles and different yaml?

that works for me.

I'll study a bit more the phy bindings in linux and I'll create a different PR to add it in Zephyr asap.

@ydamigos did you create a new PR to address the same issue on this PR? I want to follow it until it is merged.

@galak any more comments? Could we merge it?

@galak I opened PR #10359 to add support for USB PHYs.

Rebased to resolve conflicts.


更新webpack模块热替换翻译文档

Read and sign the CLA. This needs to be done only once. PRs that haven't signed it won't be accepted.
Make sure your PR complies with the writer's guide.
Read through the PR diff carefully as sometimes this can reveal issues. The work will be reviewed, but this can save some effort.
Remove these instructions from your PR as they are for your eyes only.

收到，最近比较忙，周末抽空审核。


⚠ Bofrost.at has degraded performance

In e80d196, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1595 ms

Resolved: Bofrost.at performance has improved in a15e6a7.


This is not supported and can not save data, when the screen oriention has changed

[ ] I have verified there are no duplicate active or recent bugs, questions, or requests
[ ] I have verified that I am using the latest version of Material Dialogs.
[ ] I have given my issue a non-generic title.
[ ] I have read over the documentation (before asking questions on how to do something).

Include the following:

Material Dialogs version: 0.x.x.x
Device OS version: 7.1.1
Device Manufacturer: Google
Device Name: Pixel XL

Also, please wrap Java with correct syntax highlighting.
System.out.println("Hello, world!");

Reproduction Steps

Expected Result
Actual Result

Wrap the dialog in a DialogFragment. This is not an issue specific to this library, that's how Android works.


Add a developing with spectral-cube docs page

Adding a page that lists out suggestions for incorporating spectral-cube in new packages, or adding custom functions to apply to cubes.
Also adds a bit more detail into how we use dask.Array under the hood.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d5ab192). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 42dd417 differs from pull request most recent head 055f9e0. Consider uploading reports for the commit 055f9e0 to get more accurate results

@@            Coverage Diff            @@
##             master     #798   +/-   ##
=========================================
  Coverage          ?   77.94%
=========================================
  Files             ?       24
  Lines             ?     5853
  Branches          ?        0
=========================================
  Hits              ?     4562
  Misses            ?     1291
  Partials          ?        0

Continue to review full report at Codecov.

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


Build and test operator on windows

Release the operator image as a multi-architecture container image supporting windows. Create end-to-end test on a GKE cluster with windows nodes.

This has been open for almost a year without any user interest. Let's close it and revisit only if there's a compelling reason to.


Update to live

removal of PS C:> from syntax

OPS Build status updates of commit 64f2e20:
:clock10: Preparing: average preparing time is 1 min(s) 13 sec(s)

OPS Build status updates of commit 64f2e20:
:clock10: Incremental building: average incremental building time is 12 min(s) 23 sec(s)

OPS Build status updates of commit 64f2e20:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/exchange-management-shell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/open-the-exchange-management-shell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/Get-CASMailboxPlan.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/set-CASMailboxPlan.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/devices/New-DeviceTenantRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Get-MailboxAutoReplyConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Get-MailboxPlan.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/New-SiteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Set-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Set-MailboxPlan.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/policy-and-compliance-dlp/Get-PolicyTipConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-PublicFolderClientPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-SiteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-SiteMailboxDiagnostics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-SiteMailboxProvisioningPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/New-SiteMailboxProvisioningPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Remove-SiteMailboxProvisioningPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Set-SiteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Set-SiteMailboxProvisioningPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Test-SiteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/sharing-and-collaboration/Update-SiteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/New-MailUser.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/New-UnifiedGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Set-UnifiedGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPAppDeniedEndpoint.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPProfileSyncConnection.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPServerScaleOutDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPServiceApplicationProxyGroupMember.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPShellAdmin.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPSiteSubscriptionFeaturePackMember.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPSiteSubscriptionProfileConfig.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPSolution.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPThrottlingRule.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPUserLicenseMapping.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPUserSolution.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPConfigurationDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPEnterpriseSearchServiceApplicationIndex.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPFarm.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPAppDeniedEndpointList.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPBusinessDataCatalogEntityNotificationWeb.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPContentDatabaseSyncData.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPDistributedCacheItem.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPLogLevel.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPMetadataWebServicePartitionData.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPPerformancePointServiceApplicationTrustedLocation.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPScaleOutDatabaseDeletedDataSubRange.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPScaleOutDatabaseLog.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPScaleOutDatabaseTenantData.md
:white_check_mark:Succeeded
View (sharepoint-ps)

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


SPIKE: Examine Cognito Adaptors

Goal: for customers to be able to quickly and easily integrate their project with an app client on Cognito managed by the SSO team.
Come up with some common criteria for adaptors to fulfill (or not)
Look at the "official" adaptors provide by AWS (amplify project?)
See if there are any good open source ones

One thing that might be a criterion is whether or not the adaptor can be used with a client-secret or if it needs to be used with a public app client.

The evaluation work is documented in
https://github.com/bcgov/cognito_example_client_app/blob/main/docs/adapter-evaluation.md


Arewemobileyet: make this site mobile responsive

Is your feature request related to a problem? Please describe.
Currently, this site isn't mobile responsive. Accommodating mobile screens is a must on all websites. Critical information may be hidden when viewing from smaller viewports.

Describe the solution you'd like
This will involve adding media queries and possibly separate, mobile-only components such as a mobile navbar.

Hi @Maniktherana, you may check the responsiveness in #469
The PR has not yet been merged as some minor changes are to be made still
But you may check if there are any missing portions of responsive behavior in that PR's head

@marsian83 Ah didnt realize, I'll check it out

this has been fixed/implemented by @Jad31


Optimization and refactoring of java integration tests

Combined multiple use cases into a single test where its appropriate, so that a new domain is not started for every use case. Some refactoring like test method names, domain input file names are changed. We still cover all the use cases, takes less time than before. On Jenkins, full test suite on develop branch takes 1 hr 35 mins. With the optimized tests, it takes 1 hr.
New test methods - testDomainOnPVUsingWLST, testDomainOnPVUsingWDT, testTwoDomainsManagedByTwoOperators, testCreateDomainWithStartPolicyAdminOnly, testCreateDomainPVReclaimPolicyRecycle, testCreateDomainWithDefaultValuesInSampleInputs, testAutoAndCustomSitConfigOverrides, testOperatorRESTIdentityBackwardCompatibility, testOperatorRESTUsingCertificateChain
Jenkins run - http://wls-jenkins.us.oracle.com/view/weblogic-operator/job/weblogic-kubernetes-operator-javatest/1031/consoleFull

LGTM overall - I only had some relatively minor naming and code simplification suggestions (provided in other comments).

@rjeberhard This PR is ready to merge.


update lab 8 images redwood UI

WMS ID: 11325
Lab 8 images updated to new redwood UI

@topformat - I do not see you on the workshop team. Could you check with the workshop team if we can approve this PR? Thanks!


Empty repo?

???

Yep! Have not taken the time to create the GitHub routines to automatically create it from the Flipper repo. “Soon. ™”


Baseline not catching multiline string literals correctly

Expected Behavior
A MaxLineLength with multiline string literals will be written in the baseline and not causing problems when running detekt.
Observed Behavior
It's written in the baseline file as follows:
<ID>MaxLineLength:MainActivity.kt$MainActivity$ </ID>

The task fails. I think the signature is incorrect.
Steps to Reproduce

Checkout my project: https://github.com/jelleheemskerk/detekt-bug
Run ./gradlew detekt

Context
I'm trying to integrate detekt with our existing codebase. We use multiline string literals with JSON data for our tests. I don't seem to have this issue on 1.18.1.
Your Environment

Version of detekt used: 1.19.0
Version of Gradle used (if applicable): 7.0.2
Gradle scan link (add --scan option when running the gradle task): https://gradle.com/s/ix7enmucwqthe
Operating System and version: Mac OS 12.1
Link to your project (if it's a public repository): https://github.com/jelleheemskerk/detekt-bug

I was able to reproduce the issue but there are numerous ways to fix it. I would like to opinion of the maintainers first.
Violating code
class C {
    fun foo() {
        """
          "First very very very very very very very very very very very very very very very very very very very long line"
        """

        """
          "Second very very very very very very very very very very very very very very very very very very very long line"
        """
    }
}

Baseline
<?xml version="1.0" ?>
<SmellBaseline>
  <ManuallySuppressedIssues></ManuallySuppressedIssues>
  <CurrentIssues>
    <ID>MaxLineLength:C.kt$C$ </ID>
  </CurrentIssues>
</SmellBaseline>

There are a couple of things that come together here:

The element that is used to create the entity of the finding is the white space before the quotes resulting in the same signature for both violations
When reading the baseline file, the signature is trimmed and therefore will not match the signature of the finding

Possible solutions

Do not `trim()' when reading the baseline file. This does not sound like a viable solution and might break uses baseline files.
Use the first "meaningful" element (meaning anything non-blank and not a quote) to report the finding.
Extend the way signatures are generated to support entire line contents.
???

Any thoughts?

Use the first "meaningful" element (meaning anything non-blank and not a quote) to report the finding.

I think that this is the best option.


npm WARN install Refusing to install aptr-uikit as a dependency of itself

sudo npm install aptr-uikit --save
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No license field.
npm WARN install Refusing to install aptr-uikit as a dependency of itself
Change the name in the package.json file to resolve this issue

@salali in general not able to install the UI kit on my Mac. any reasons for it?

Hi, apologies if it wasn't clear from Sal's label. We don't support or troubleshoot installation issues on this project, especially as we use this every day without issue.  We recommend you explore or raise any generic node based setup errors on StackOverflow.

👍

@wesatkinson got it to work finally. Had sometime today. Its a cool collection! Great Job by team Aptr! Cheers!


[23.2] Add option for setting memory limit for user job containers

This adds controller agent operation options:
"set_container_memory_limit" and "container_memory_overhead".

e6ab32794f88327f5dfd30970cafc79c82a9f8af
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/359
(cherry picked from commit 54bdde3388037e9744a8efe66c276b6bd721b070)

I observe merge conflits in Arcadia, could you please rebase on the latest stable/23.2?

Done. Now should be ok.


Insert row above another one - Tables in Templates - Links to note in Tables

Problem
I started using Notion Like Table and it works great after the recent update.
Still I face 3 issues.

how do I insert a row above another?

when I set up a table in a template, how do I prevent the table in the template note to get filled when I filled the table in the note based on that template?

Also it doesn't seem that I can insert a link to another note within a table. Is that right?

Thank you
Solution
As said above.
Additional details
No response

Fixed in 6.3.0
No longer relevant
Fixed in 6.2.0


Items in expanded SubChoreos are not shown after reimporting

How to reproduce:

Create new SubChoreo
Put elements Into SubChoreo
Save bpmn file
Open bpmn file in editor by setting it as xml in app.js
-> Elements added in step 2 are not displayed, however they are (at leas partially) in the .bpmn file.

I can no longer reproduce this. It has probably been closed in #94.


[RFC] Support 20 bytes address

The current address is hard-coded to be 16 bytes. This works well for Diem, but could be insufficient for other systems. A common requirement is 20 bytes.
This PR adds a feature that gates the size of the address to be either 16 or 20 bytes, and change everywhere this is used to be using the right length. If we want to support arbitrary configurable size, we would need to add a build.rs to generate it. Most of the changes here are still needed.
There are a few more things to polish/warnings to fix.
Sending out early to get feedback and see if we think this is something reasonable to do.

While you can use features this way it's generally discouraged because you'd need a feature flag for each length and then the features are no longer additive.
I wonder if a more scalable solution would be to use const generics to determine the size of the address.

I wonder if a more scalable solution would be to use const generics to determine the size of the address.

Would love to make this more generic. Thought about const generics, but couldn't see a way to do this without changing everything (all uses of AccountAddress and classes that have it). Would love to hear ideas on how to do this!
I could use a build.rs to generate the length definition based on a config, if that works for the team.

I wonder if a more scalable solution would be to use const generics to determine the size of the address.

Would love to make this more generic. Thought about const generics, but couldn't see a way to do this without changing everything (all uses of AccountAddress and classes that have it). Would love to hear ideas on how to do this! I could use a build.rs to generate the length definition based on a config, if that works for the team.

I think any technique, let it be generated build.rs or generic const or like done here, will lead to that there are two exclusive flavors of the Move binaries. The only way out of this seems to me dynamically sized addresses, which has performance and other far reaching implications, so I guess I'm good with this one here.

I could use a build.rs to generate the length definition based on a config, if that works for the team.

Could you show more what that would look like? Even a mock up of some sorts is fine for now.
Just not entirely sure what goes in the build.rs

Turns out that I don't have to use build.rs (I was initially thinking about having build.rs generating a rs file that contains a definition of LENGTH based on an environment variable).
I discovered a crate called "const_env", which allows you to declare a const integer from an environment variable.
Updated the change. Now we could specify the address length through environment variable ADDRESS_LENGTH. It's default to 16.

Turns out that I don't have to use build.rs (I was initially thinking about having build.rs generating a rs file that contains a definition of LENGTH based on an environment variable). I discovered a crate called "const_env", which allows you to declare a const integer from an environment variable. Updated the change. Now we could specify the address length through environment variable ADDRESS_LENGTH. It's default to 16.

I think this just makes it worse compared to the exclusive cfg feature you have used before. const_env evaluates environment variables at compile time. Thus compilation of the Move code now depends magically on some environment variable, which is less transparent than the cargo feature, but comes exactly with the same disadvantages which are listed for exclusive features here.
With the feature, one can at least specify in the toml on which flavor of Move one wants to depend. With the environment variable, it is even not possible to request an address flavor via a dependency.
While exclusive features are discouraged in general,  there are cases (like this here) where they are needed. Lets use a feature instead of some obscure dependency on an env variable at compile time. As I said, the only way to do this better than with features is via dynamic (at runtime) configurable address length, but this is to big of a refactoring.

I'd still like to better understand how the build.rs solution would work, but I agree with @wrwg that the env based approach is much much worse.
build.rs aside. Could we at least make the cfg feature based solution cleaner?
I think in the previous iteration, @lxfind, you had conditionally declared the constant depending on the feature. While functionally equivalent, could we do something like const C: u64 = if cfg!(feature = "my_cool_addr_length_feature") { 0 } else { 1 }; instead?

I'd still like to better understand how the build.rs solution would work, but I agree with @wrwg that the env based approach is much much worse.
build.rs aside. Could we at least make the cfg feature based solution cleaner? I think in the previous iteration, @lxfind, you had conditionally declared the constant depending on the feature. While functionally equivalent, could we do something like const C: u64 = if cfg!(feature = "my_cool_addr_length_feature") { 0 } else { 1 }; instead?

Thanks for the feedback! @wrwg @tnowacki
The approach with build.rs will only be worse in this regard (I was planning to generate a rust file that contains the definition of LENGTH based on environment variables and include it in account_address. So I won't go that route.
The closest I can do with if/else would be something like this:
pub const LENGTH: usize = match () {
        #[cfg(feature = "address20")]
        () => 20,
        #[cfg(not(feature = "address20"))]
        () => 16,
    };

which isn't much better than
#[cfg(feature = "address20")]
pub const LENGTH: usize = 20;
#[cfg(not(feature = "address20"))]
pub const LENGTH: usize = 16;

I'd still like to better understand how the build.rs solution would work, but I agree with @wrwg that the env based approach is much much worse.
build.rs aside. Could we at least make the cfg feature based solution cleaner? I think in the previous iteration, @lxfind, you had conditionally declared the constant depending on the feature. While functionally equivalent, could we do something like const C: u64 = if cfg!(feature = "my_cool_addr_length_feature") { 0 } else { 1 }; instead?

Thanks for the feedback! @wrwg @tnowacki The approach with build.rs will only be worse in this regard (I was planning to generate a rust file that contains the definition of LENGTH based on environment variables and include it in account_address. So I won't go that route.
The closest I can do with if/else would be something like this:
pub const LENGTH: usize = match () {
        #[cfg(feature = "address20")]
        () => 20,
        #[cfg(not(feature = "address20"))]
        () => 16,
    };

which isn't much better than
#[cfg(feature = "address20")]
pub const LENGTH: usize = 20;
#[cfg(not(feature = "address20"))]
pub const LENGTH: usize = 16;

Sorry, I don't understand why you can't do
pub const LENGTH: usize = if cfg!(feature = "address20") { 20 } else { 16 };

Or
pub const LENGTH: usize = if cfg!(feature = "address20") {
    20
} else if cfg!(feature = "address16") {
    16
} else {
    panic!("address feature not specified")
};

Sorry, I don't understand why you can't do
pub const LENGTH: usize = if cfg!(feature = "address20") { 20 } else { 16 };

Or
pub const LENGTH: usize = if cfg!(feature = "address20") {
    20
} else if cfg!(feature = "address16") {
    16
} else {
    panic!("address feature not specified")
};

Ha didn't know cfg! could be used this way! TIL. Will use it. Thanks!

Sorry, I don't understand why you can't do
pub const LENGTH: usize = if cfg!(feature = "address20") { 20 } else { 16 };

Or
pub const LENGTH: usize = if cfg!(feature = "address20") {
    20
} else if cfg!(feature = "address16") {
    16
} else {
    panic!("address feature not specified")
};

Ha didn't know cfg! could be used this way! TIL. Will use it. Thanks!

Instead of panic!, you may want to use compile_error!("msg")

LSR dependency change summary:
target packages:
  M diem-types 0.0.3 (workspace, path 'types')
    * added features: address20
    * (unchanged features: default)

LEC dependency change summary:
target packages:
  M diem-types 0.0.3 (workspace, path 'types')
    * added features: address20
    * (unchanged features: default)
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M diem-types 0.0.3 (workspace, path 'types')
    * added features: address20
    * (unchanged features: default)
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

LSR dependency change summary:
target packages:
  M diem-types 0.0.3 (workspace, path 'types')
    * added features: address20
    * (unchanged features: default)

LEC dependency change summary:
target packages:
  M diem-types 0.0.3 (workspace, path 'types')
    * added features: address20
    * (unchanged features: default)
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M diem-types 0.0.3 (workspace, path 'types')
    * added features: address20
    * (unchanged features: default)
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

LEC dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

LEC dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

LEC dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

LEC dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

LEC dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

LEC dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

release binaries dependency change summary:
target packages:
  M num-bigint 0.4.0 (direct third-party, crates.io)
    * status changed from transitive third-party
    * added features: default
    * (unchanged features: std)

All tests finally passing now. Ready for review. I initially wanted to make it such that it can pass all tests when address20 is enabled. But realized that it would be likely impractical. The move release are precompiled modules that use 16 bytes, and they cannot work with 20 bytes. So my current goal is just to make sure that move builds with address20, and we will test everything separately when depend on it.


Uninstall script does not remove Django users from PostgreSQL database

Actually, it does. However, there was another problem that caused this. Opened #37, closing this.


Skipping test_map_scalars_supported_key_types because of distributed …

…failures
This PR skips a test to unblock CI https://github.com/NVIDIA/spark-rapids/issues/8922.
I am not closing the issue as I'd like to rewrite the test possibly, but I ran out of time today and want to give this more thought.

build

build

build


Binary Content Crawling

Hello,
I would like to post a suggestion for Readme file. It is not very obvious that the crawler won't fetch images or pdfs basically binary content by default unless we set the config to includeBinaryContentInCrawling.
Though there is an example for image crawling explicitly, but to me it is not obvious that this example config is generic to include any binary content not just image. I had to debug the code for a while to understand about this setting.
Hence, I feel this setting should be a part of the Configuration Details section on the readme page.
Please let me know what do you think. If you agree on the same, I would like to raise a PR for this change.
Regards,
Harshita Kasera

Hello, please just raise the PR, thank you!

Thanks Harshita,
That would be very good to have.
On Tue, Oct 3, 2017 at 9:58 AM, Federico Tolomei<EMAIL_ADDRESS>wrote:

Hello, please just raise the PR, thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yasserg/crawler4j/issues/249#issuecomment-333756854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABrbW_8jWqY6PCKdsTOswwF07ZUNsd1Uks5sodsEgaJpZM4Pritv
.

That is great! Thank you! I will start working on it and send a PR in a couple of days.


Modifying the UI for Helpline numbers page

The font should be a normal one instead.
I would like to work on styling of the table too.
Replace some illustrations with newer ones.

@tanishq-arya The font is better I think. Can you share some designs you have thought of if possible?

Sure. I will share it by today.

I have redesigned something like this. But I am unable to make changes for mobile view because of the css.

Hey @tanishq-arya,
Go ahead with the issue.


The leap year warning for gt06

Please help my case :
2017-02-02 19:26:17 DEBUG: [B8CEFBA7: 5023 < <IP_ADDRESS>] HEX: 78780a130406040002007ab7860d0a
2017-02-02 19:26:17 DEBUG: [B8CEFBA7: 5023 > <IP_ADDRESS>] HEX:<PHONE_NUMBER>7a24a50d0a
2017-02-02 19:26:17  WARN: Data truncation: Incorrect datetime value: '1970-01-01 00:00:00' for column 'fixtime' at row 1 - MysqlDataTruncation (... < QueryBuilder:473 < DataManager:296 < DefaultDataHandler:27 < ...)
2017-02-02 19:26:17  INFO: [B8CEFBA7] id:<PHONE_NUMBER>56310, time: 1970-01-01 00:00:00, lat: 0.00000, lon: 0.00000, speed: 0.0, course: 0.0
2017-02-02 19:31:17 DEBUG: [B8CEFBA7: 5023 < <IP_ADDRESS>] HEX: 78780a130406040002007ba60f0d0a
2017-02-02 19:31:17 DEBUG: [B8CEFBA7: 5023 > <IP_ADDRESS>] HEX:<PHONE_NUMBER>7b352c0d0a
2017-02-02 19:31:17  WARN: Data truncation: Incorrect datetime value: '1970-01-01 00:00:00' for column 'fixtime' at row 1 - MysqlDataTruncation (... < QueryBuilder:473 < DataManager:296 < DefaultDataHandler:27 < ...)
2017-02-02 19:31:17  INFO: [B8CEFBA7] id:<PHONE_NUMBER>56310, time: 1970-01-01 00:00:00, lat: 0.00000, lon: 0.00000, speed: 0.0, course: 0.0

With my positions table:

I using the latest version.

It also occured on web.

Most likely your MySQL connection URL is incorrect.

My connection URL is correct. In accordance with documentation (https://www.traccar.org/mysql/).
Is this issue related to https://github.com/tananaev/traccar/issues/1537 ?

Basically the problem is that MySQL doesn't accept zero timestamp:

The TIMESTAMP data type is used for values that contain both date and time parts. TIMESTAMP has a range of '1970-01-01 00:00:01' UTC to '2038-01-19 03:14:07' UTC.

So, can you tell me how to solve my case?
Thank you.

You can just wait till your device sends valid GPS data.

Ok,
so, we can ignore this warning?

Yes.


Request - Add support for Rackspace Cloud

Hi,
Simple request. I'd like to see these tools also be usable on Rackspace's cloud computing infrastructure. Thanks!
Scott

Goods news... Carlos (cced) from Rackspace is a big contributor.  He can
likely get you started today with the static machine config we support.
That said I'd like to see Rackspace as one of the default clouds so all you
have to do is --cloud=Rackspace
Not sure of the timeline on that.
On Mar 17, 2015 12:04 AM, "Scott"<EMAIL_ADDRESS>wrote:

Hi,
Simple request. I'd like to see these tools also be usable on Rackspace's
cloud computing infrastructure. Thanks!
Scott
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/173.

Yes, the "--cloud=Rackspace" is the kind of support I'd like to see too. And I am almost certain RS would want to have it too.;)
Scott

Assigning to Carlos to eval.

Communicated this to the right channels internally at Rackspace. Developer Relations team at Rackspace will take a look into this for their next sprint.

So as to not keep everyone in the dark, this is currently being developed (by me).

@jrperritt Awesome thanks for taking on this task.

Nice!  Thank you.  Any planned ETA?
On Wed, Apr 15, 2015 at 10:19 AM, jrperritt<EMAIL_ADDRESS>wrote:

So as to not keep everyone in the dark, this is currently being developed
(by me).
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/173#issuecomment-93491315
.

--
Anthony F. Voellm (aka Tony)
Google Voice:<PHONE_NUMBER>
https://www.google.com/voice/b/0?pli=1#phones
Blog: http://perfguy.blogspot.com/

So as to not keep everyone in the dark, this is currently being developed (by me).

:+1:

@voellm Not really, but it would go a lot faster if there was a guide for adding cloud providers. Does such a thing exist?

We dont have a guide by you are welcome to ping<EMAIL_ADDRESS>for
help.
On Wed, Apr 15, 2015 at 12:56 PM, jrperritt<EMAIL_ADDRESS>wrote:

@voellm https://github.com/voellm Not really, but it would go a lot
faster if there was a guide for adding cloud providers. Does such a thing
exist?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/173#issuecomment-93548966
.

--
Anthony F. Voellm (aka Tony)
Google Voice:<PHONE_NUMBER>
https://www.google.com/voice/b/0?pli=1#phones
Blog: http://perfguy.blogspot.com/

Closing, #336 is merged and appears on track to be included in the upcoming 0.18.0 release.
FYI re @voellm's earlier comment, we've started on documentation for adding cloud providers in the Wiki - let me know if you have feedback:
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/wiki/Adding-a-new-cloud-provider

Awesome, thanks for merging this!
@klausw I'll go over the wiki, and leave my feedback. Thanks


iOS cannot load images from local xcassets

Description
react-native-skia on iOS cannot load images from local xcassets. Images included in android drawable work fine. Images from xcassets work fine using the built in react-native Image component. It's only images from xcassets that don't get loaded by react-native-skia.
Related issues:
https://github.com/Shopify/react-native-skia/issues/1069
https://github.com/Shopify/react-native-skia/issues/861
Looking at the implementation I tried to debug with
Skia.Data.fromURI(uri)
        .then(data => {
            console.log(`got data`, JSON.stringify(data))
           //data is always {"__typename__":"Data"}
            const img = Skia.Image.MakeImageFromEncoded(data)
           //img is always null
            console.log(`got skimage `, img)
            return img
        })
        .catch(err => {
           //error does not get called
            console.error(`Error loading skia image at "${uri}"`, err)
        })

Again, passing the image name without extension to load from local assets works perfectly on iOS. It works perfectly with the built in react-native image component. It's just skia that isn't resolving the image for some reason.
Version
0.0.173
Steps to reproduce
Add image to iOS xcassets
Add image to Android drawables/nodpi folder
Load image with react native image tag <Image src={{uri: 'filename_without_extension'}} />
Observe that the image loads correctly.
Attempt to load same image with skia
//results in console error " [Error: Could not load data]"
const img = useImage('filename_without_extension', err => console.error(err))

return <SkiaImage image={image} width={50} height={50} ....other props />

Observe that this works on Android
Observe that this does not work on iOS
Snack, code example, screenshot, or link to a repository
See above steps

@chrfalch Thanks for the quick patch! Are you planning to release a new version soon or should I apply that patch locally for now?

@chrfalch Thanks for the quick patch! Are you planning to release a new version soon or should I apply that patch locally for now?

@evelant this is available now in v0.1.174


Move children/ endpoint to history API

As children is not part of the Core Node API anymore, we should think about adding it to the history API and update the corresponding TIP accordingly.

History seems like a weird place for it. Is there really any reason we can't include additional routes in our impl of the core API?


Fix quoting for stringy test configs

resolves #3458
Description

Be smarter about how we wrap test configs in quotes, for inclusion in the templated {{ config() }} call
Add a test. I confirmed that the test yields the same error as the one reported in #3458 without this fix, and passes with it.

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Would this fix also support double quotes, for instance due to column names that are reserved words or are case-sensitive? For example, Segment has this unfortunate column name timestamp that's a reserved word in Redshift. So you could get a where condition like:
where: "timestamp" >= '2021-06-16'

@ilmari-aalto Yes! Check out the test case in this PR. Be mindful that, in order to pass values with mixed quoting in YAML, you need to escape it one way or another:
where: "\"timestamp\" >= '2021-06-16'"
# or
where: '"timestamp" >= \'2021-06-16\''

@ilmari-aalto Yes! Check out the test case in this PR. Be mindful that, in order to pass values with mixed quoting in YAML, you need to escape it one way or another:
where: "\"timestamp\" >= '2021-06-16'"
# or
where: '"timestamp" >= \'2021-06-16\''

Excellent! Thanks for confirming!

I'll cherry-pick the squashed commit onto 0.20.latest after merging into develop


Guild issue

Good Night, I am trying to use your project, and I made it work in a discord server with all the name together, but in a discord server with separate words like this example: 'Eurovision Discord <3' doesn't work, how I make this work in -guild?

use quotes, e.g. rankbot -guild "Eurovision Discord <3"

Still not working =/  - Still giving the same error : https://image.prntscr.com/image/R6jzyvUHQZat6YwV9R5OGQ.png

What does the error say? I can't read it.

System couldn't find the specified file

Must be something specific to the windows command line.
Google "how to send an argument with spaces in cmd.exe" and you'll likely find the answer.

Oh sorry, didn't specify but I think its misinterpreting the double quotes as a file read.

Tried with the 2 quotes but still persists. https://image.prntscr.com/image/ggZ8CZd3S6_zPoyRPDTJ4w.png

Don't use a space between the quotes.

https://image.prntscr.com/image/WZbXqpNdTjaS1IPBv0CUlQ.png - still same

sry for being annoying to you, I am newbie in this but already make this work

Sorry, I've got no idea then. I'm not familiar with windows. Your best bet is to keep googling around or ask someone familiar with the windows command line. You could also try using the linux on windows sub system.

Ok thanks :) and thanks for making this amazing script, this is really useful

Can use another command line like the git cmd or something like that? I am noob with this sorry :/

I wrote a quick Go program to do this for you so you wouldn't need to use a shell.
package main

import (
	"os/exec"
	"io"
	"log"
	"os"
)

func main() {
	c := exec.Command("echo", "-email"<EMAIL_ADDRESS>"-pass", "secret", "-guild", "server", "-chan", "channel")
	cmdStdout, err := c.StdoutPipe()
	if err != nil {
		log.Fatal(err)
	}
	cmdStderr, err := c.StderrPipe()
	if err != nil {
		log.Fatal(err)
	}
	err = c.Start()
	go func() {
		_, err := io.Copy(os.Stdout, cmdStdout)
		if err != nil {
			log.Fatal(err)
		}
	}()
	_, err = io.Copy(os.Stdout, cmdStderr)
	if err != nil {
		log.Fatal(err)
	}
}

Just edit the details (pass, server etc) to fit what you need.

That should fix your problem so I'm gonna close this issue.

Created a new issue: https://image.prntscr.com/image/qO_iBZKWTLenyMV7gSrl7g.png

PS: Sorry for reopening

Oh, haha I missed a small detail. See where it says "echo" on the first line of the main function? Change that string to the path of the rankbot executable.

c := exec.Command("go run C:\Users-KikoGrilo\Desktop\rankbot-master\main.go", "-email"<EMAIL_ADDRESS>"-pass", "password", "-guild", "Eurovision Discord <3", "-chan", "bot_commands")
Like this?

No, first run go build inside of C:\Users-KikoGrilo\Desktop\rankbot-master. That will generate an executable. Then c := exec.Command("C:\Users-KikoGrilo\Desktop\rankbot-master\rankbot-master", "-email"<EMAIL_ADDRESS>"-pass", "password", "-guild", "Eurovision Discord <3", "-chan", "bot_commands")

Like this?

not making it

Ok, forget all of this. Delete that file. I'll make it more simple.
Just edit the main.go in this repo. Change the following lines.
https://github.com/nhooyr/rankbot/blob/072e50b1a2e8407ba013981c05e97c4e9818a5d7/main.go#L11-L18
E.g.
var (
	email    = "some email"
	pass     = "pass"
	guild    = "server"
	channel  = "channel"
	message  = "_"
	interval = 60
)

And then just go run main.go

ok
in this case this?
var (
email    =<EMAIL_ADDRESS>pass     = "password"
guild    = "Eurovision Discord <3"
channel  = "bot_commands"
message  = "_"
interval = 60
)

lol, I missed one more detail. Do the following instead
var (
	email    = flag.String("email"<EMAIL_ADDRESS>"account email")
	pass     = flag.String("pass", "password", "account password")
	guild    = flag.String("guild", "Eurovision Discord <3", "guild (server) to join")
	channel  = flag.String("chan", "bot_channels", "channel to join")
	message  = flag.String("msg", "_", "message to be sent")
	interval = flag.Int64("int", 60, "interval between messages in seconds")
)

IT WORKED! just saying thank you for your amazing support and thanks for being nice to me since I am a big amateur in this programming world :) 👍 to you!

Glad it worked out.

Now you can close the thread 😂


Hotfix for sporadic ec2 instance bootstrap failure

Ensure splunkdConnectionTimeout setting is applied to prevent Splunk restart from returning a warning due to splunk web interface taking a long time to become ready. This sporadic warning happens mostly with small EC2 instances where Splunk startup time is higher than usual.
Make Master template (new vpc) outputs consistent with sub template (existing vpc) outputs.

@sshvans, this issue was encountered after additional testing with various EC2 instances.
Could you please take a look, and if possible merge this hotfix before upcoming release is cut?


Minimum changes to build on clang-11.0.3 (clang-1<IP_ADDRESS>)

Minimum changes to build on clang-11.0.3 (clang-1<IP_ADDRESS>) for x86_64-apple-darwin19.5.0

minimal customization of stock Apple:

install XCode from Apple App Store (XCode 11.6 onto MacOS-10.15.6 "Catalina")

clone the repo per the README (see updated "--recurse-submodules")
build warning-free per the README

NOTE: I'm not a fan of cmake-ing the world, my personal experience hasn't lead to great cmake support experiences, I'd rather bazelify-the-world personally or drop back to autotools for most compatibility :)  For this reason, I see @rumpeltux's https://github.com/darsto/brother-scand/pull/6 but would prefer autotools or bazel to cmake.
To avoid resistance based on replacing the build tool, I've cut a smaller PR to simply update this repo to the minimum required to build.
Apple's Clang-11.0.3 is:
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 11.0.3 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.5.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Hi, would you mind basing this on top of my https://github.com/rumpeltux/brother-scand version of #6 to avoid diverging this further? (Unfortunately this @darsto’s repo seems dead…)
In my branch, CMake is not required for building, just for running the unit tests. Regular building continues to use plain Makefiles.
Also maybe you can extend the README for Apple build instructions, otherwise it seems like the commands from this PR’s message would be hard to find for future users.

I might base it on a branch that has zero requirement for cmake if that helps you.
I don’t use cmake, so I cannot test anything that uses cmake, so I cannot test a PR based on something that needs cmake unfortunately.

I do agree that fragmentation splits effort across the board: 10 engineers working on one project get further than 10 working on 3.  We all move forward that way

I'm surprised to see any activity here. I'm obviously not maintaining this repo anymore so I'm probably not going to merge any pull requests, but I can add a note to the main readme saying that development is continued on @rumpeltux fork
@rumpeltux do you want me to do that?

Thank you, yes, that’d be great. I’m happy to address incoming pull requests.

@rumpeltux Thank you, done. I'm going to archive this repository now, which should make it read-only.


Callables in settings are not verified at start-up if they are strings

If you accidentally set, e.g., AXES_LOCKOUT_CALLABLE to an invalid import path, the server will still boot up fine and you won't find out about it until Axes tries to invoke the callable here. In my case, AXES_LOCKOUT_CALLABLE was misconfigured for months before I saw the error in the logs.
It would be ideal if Axes did some kind of verification of the callable settings at start-up. I can think of two ways:

Verify settings in conf.py
Verify settings in a custom system check

If this seems like a good idea to you I'd be willing to work on it. Not sure if crashing the server on misconfiguration counts as a backwards-incompatible change or not; if so could print a warning instead.
I am using version 5.39.0.

Thanks for reporting!
This would also be a good opportunity to refactor the callable resolution to use one common function to return one verified callable object from a string path or an object itself. At the moment there is duplicate code in many places and it would be nice to clean that up.
@iafisher and @hirotasoshu would you happen to be interested about improving this and possibly baking a new release as you are familiar with the code base?

@aleksihakli should it be new checks in axes/checks.py?

@hirotasoshu I'd be happy to write the system check if you want to work on the common function for resolving callables. I can get it done today or tomorrow.

@hirotasoshu I'd be happy to write the system check if you want to work on the common function for resolving callables. I can get it done today or tomorrow.

Ok, I created #1068 for this

Thank you for the work on the PR 👍 Upstreamed, I'll bake a new version to PyPI as well.

Released in version 6.0.2.


Improve randomness uniformity

This PR is for improving the randomness calculation for the percentage of actors. It's a port of the implementation on https://github.com/fetlife/rollout/pull/120.
As the new implementation reduces the fails in a really small decimal part, I haven't added a spec for this scenario. I also believe that it could become an expensive spec.
On the rollout implementation, there is a benchmark exposing the approximate accuracy of this implementation.
Closes #272

Hey thanks! Sorry I haven't responded to this. I was gone on vacation. Thank you so much for tackling it! I really apprecate the help. I'll try to look it over this weekend or first thing next week.
One question: will I need to note this as a breaking change in the changelog? I'm assuming that individual actors who were enabled before, may or may not be following this change. I understand that the same percentage will be enabled, but would upgrading flipper with any % of actors enablements were not 100% cause the set of enabled actors to differ? Just want to make sure I note it correctly for people.
I'll probably test it with 100 or 1k actors or something and get the enabled ones before this change and after and then do a set diff. If you have time to tackle that and drop the work here in a comment, it would help getting this merged faster, but if not I'm happy to look at it as soon as I have time.

Hey @jnunemaker, so, it was my turn to be on vacation this weekend, sorry for the delayed answer.
As you mentioned, the percentage will probably be similar, but with a really small decimal difference. In few cases, it will be higher & other cases lower than the old implementation %, so it makes sense to mention on the changelog this possibility.
I can do the test batch, just give me some time, so I  work on it and drop the result here. Not sure about this weekend, but I do my best to get it done asap. :-)

@rodrigopinto sounds great. If I get a chance I'll whip something together first, but if not I'd love the help. Thanks!

It has a really long time since I opened this PR and it got stale, so I will close it but keep the fork implementation in case I come back to work with this lib again.


[WIP] feat(pin-code): new component

Description
A new component: the PinCode!
Scope

[ ] Patch: Bug Fix
[x] Minor: New Feature
[ ] Major: Breaking Change

Who needs a spell-checker when I have @leyanlo? 😜
But for real - I'll do a pass on all the text before I un-WIP it.

This looks great! A few things to consider/look into:

using type=number on the input, so on mobile you'll get the mobile keyboard
for some reason, Chrome tries to autocomplete the inputs with my ZIP code
"Keep in mind that folks may have mistyped a digit so auto-submitting might be a bit aggressive." - I believe lots of apps work that way - should we change the wording here / add an example for that use case too?

type=number results in the little up and down ticks on the input and scrolling through numbers which i think would be a little annoying. i'm trying out type=tel with a pattern="\d*" to get the number pad on mobile. will see how that goes
autocomplete - not sure about this - i recreated an input in codesandbox with same type/pattern/autocomplete/etc and it does not appear - so i'm still working this out. i think in my case i clicked on the zip code at one point and i think chrome will never let me get away from it now.
so that last example is to showcase that you can do something when all the inputs are filled in - whether its submitting or focusing something else is kind of up to the app. i was going to add an onCompletion to encourage this but figured this example did the trick. i could change the example to prompt an alert instead of focusing a button but i feel like either way the point is made?

I also spent a while trying to add an e2e test for the copy/paste & autocomplete handling but puppeteer doesn't make it obvious how to do pasting - i may have to just generate an input event directly on the element but that is pretty hacky and getting close to implementation. not sure its worth it at that point.

type=number results in the little up and down ticks on the input and scrolling through numbers which i think would be a little annoying. i'm trying out type=tel with a pattern="\d*" to get the number pad on mobile. will see how that goes
autocomplete - not sure about this - i recreated an input in codesandbox with same type/pattern/autocomplete/etc and it does not appear - so i'm still working this out. i think in my case i clicked on the zip code at one point and i think chrome will never let me get away from it now.
so that last example is to showcase that you can do something when all the inputs are filled in - whether its submitting or focusing something else is kind of up to the app. i was going to add an onCompletion to encourage this but figured this example did the trick. i could change the example to prompt an alert instead of focusing a button but i feel like either way the point is made?

I also spent a while trying to add an e2e test for the copy/paste & autocomplete handling but puppeteer doesn't make it obvious how to do pasting - i may have to just generate an input event directly on the element but that is pretty hacky and getting close to implementation. not sure its worth it at that point.

cool, that sounds great - yeah, feel free to ignore the 3rd point

ok - i think i've got the silly autocomplete and keyboard issues under control

Looks very nice and the examples are great!


MJPEG Encode grayscale

I was wondering if there is a way to have the MJPEG encoder only encode grayscale images during a stream/video recording?

Hi, thanks for the question. The easiest thing would be just to set the whole camera system to give you greyscale images. To do this, just before or after starting the camera use
picam2.set_controls({"Saturation": 0.0})

It's more complicated if you wanted to receive full colour images from the camera but then only encode greyscale, I think you'd probably have to change the pixel colour values, which would cost CPU.

Hi, just wondering if there's anything else for us to investigate here? Thanks!

I'll close this one for the time being. Please open another issue if there's anything else. Thanks!


Missing notebook.

Description of the bug:
In page: https://ai.google.dev/gemma/docs/gemma_chat
There is a link to github (this repository)
https://github.com/google/generative-ai-docs/blob/main/site/en/gemma/docs/gemma_chat.ipynb
and also a run in colab:
https://colab.research.google.com/github/google/generative-ai-docs/blob/main/site/en/gemma/docs/gemma_chat.ipynb
but the notebook is missing.
Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

Fixed with PR - https://github.com/google/generative-ai-docs/pull/463


Lighter line number foreground color

IMO is current LineNr foreground color quite unreadable. I prefer higher contrast. I suggest same color as for normal text (#cbe3e7 aka clouds). What do you think about it?
P. S. Thanks for you colorscheme, I love it so much (I had tried several colorschemes before).

Thank you for the feedback! I just tried the suggested change and prefer the current visual appearance. From my point of view its subtle, but not hard to read.


Suddenly “this.audiocontext.createScriptProcessor is not a function” error in p5js-sound

I have a website, that should play mp3 file after clicking on a logo. Problem is with this line. Without any reason after two years of proper work it started to give this error.

Uncaught TypeError: this.audiocontext.createScriptProcessor is not a function.

I found out that it works in firefox and on some computers with older versions of chrome. How to solve this problem?

Hmm, the Script Processor has been on its way to deprecation for 2+ years. I was actually kind of excited to see this issue because I thought it meant that its long awaited and more efficient replacement, the AudioWorklet, was now available. Unfortunately, the AudioWorklet, as far as I can find, is still not available in any browsers yet. The ScriptProcessor should continue to work in the meantime.
Can you share a link to your project?
What version of Chrome? I just checked in latest versions of Chrome 55.0.2883.95, Canary 57.0.2960.0 and Chromium 57.0.2961.0 and createScriptProcessor, and p5 scripts that rely on it, are working.
What version of p5.sound.js? If it's two years old, there could be some other browser incompatibility issues that would be resolved by updating to a newer version.

http://bassfunk.moscow/ link to the project, It was working with the version 54.0.2840.71 m , after I updated to Version 55.0.2883.87 m  it stopped working

Thanks for the link. I'm also seeing this error on your site. It's running a version of p5.sound from 2015, and the issue is fixed if you update to a newer version of p5.sound, for example from here.
It seems to be related to https://github.com/processing/p5.js-sound/issues/148. In Chrome, AudioContext.prototype.hasOwnProperty('createScriptProcessor'); now returns false and it looks like in p5.sound v0.2.0 2015-05-30 it would have been getting overridden by a patch that was meant to fix conflicts between different implementations of the Web Audio API.

i'll leave this open as you're probably not the only one who'll run into this issue

thanks for your help.

I'm also getting this error on Chrome, with the 11/01/2016 version of p5.sound.

ps that's on Chrome  56.0.2924.87, amd the page that is generating the errors, www.thisisnotlaselva.net, is playing html5 audio normally

@df6000 Hmm, this issue is fixed in the 11/01/2016 version of p5.sound, and I'm not seeing the error when I visit your site with Chrome 56.
I am seeing a different error when I click the mouse. It seems that preload is not finishing so setup is not being called. I believe if you remove the loaded callback from loadSound it will work. (preload works with asynchronous functions because it adds setup as the callback once everything has loaded, so if you provide your own callback it gets tripped up). If that doesn't solve the issue, feel free to create a new issue. I'd really like to experience This is not La Selva!

Hey thanks for the answer!  I embarrassingly didn't notice your response until this week, finally had a chance to check it out, and you're absolutely right!  I had several errors in the code which I fixed....there are still a few glitchy things and some general interface stuff that I need to fix/rework but it's more or less working again!  thank you!

because of the comments on this thread, i understand the issue was fixed, so i am closing it for now ~


hosting with docker

Adds Dockerfile to create hosting image.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (713bff9) 99.73% compared to head (bb35119) 99.73%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   99.73%   99.73%
=======================================
  Files          13       13
  Lines         754      754
  Branches       20       20
=======================================
  Hits          752      752
  Misses          1        1
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Kkatanaga add tokenizer

-Issue: #3
-Added tiktoken and pytorch in requirements
-Created Tokenizer class
-Added default values when initializing Tokenizer
-Tokenizer can encode text during initialization or by calling tokenize(text)
-Embeddings can be retrieved by calling embeddings() or as an output of tokenize(text)

@kkatanaga
Thanks for the PR I have taken a look and have added some comments. Here are some additional notes:

Would you be able to come up with small test case (e.g. a random paragraph of text), put it in the tokenizer and then print out the output of the embeddings(self) method?
Either you or @TayoAlalade can work create this. Then just post the results of test case to the #3 issue page

@TayoAlalade please take the chance to review the PR. If you've already made your own changes, then feel free to make another PR and we can compare manually.

Overall, the PR was good, most comments were related to documentation

I also add @haily835 as a reviewer to take a look to keep him in the loop


[Spotify Player]...

Extension
https://raycast.com/mattisssa/spotify-player
Raycast Version
latest
macOS Version
No response
Description
Error:
Error: Player command failed: No active device found
    at yl (/Users/ammad/.config/raycast/extensions/320f40ef-a633-415a-ab0e-1e99515478f7/search.js:58:1232)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async o (/Users/ammad/.config/raycast/extensions/320f40ef-a633-415a-ab0e-1e99515478f7/search.js:60:469)

Steps To Reproduce
Just plan any song
Current Behaviour
No response
Expected Behaviour
No response

Could you share a recording? It'll make it easier to see what's happening.


Problem with setting mlint dir

Thanks for this package!
I am trying to set the mlint directory (on a mac) in the package settings but I keep getting an error:
Failed to run MATLAB linter - Directory containing mlint is set to '/Applications/MATLAB_R2016a.app/bin/maci64/mlint'
The directory is correct... any idea what could be the problem? (Would like to avoid adding it to PATH.)

I believe you need a trailing slash on the path (I am away from my computer at the moment and so can't check). Try setting it to /Applications/MATLAB_R2016a.app/bin/maci64/mlint/ and if that doesn't work let me know.

Doesn't work unfortunately. Same error:
Failed to run MATLAB linter - Directory containing mlint is set to '/Applications/MATLAB_R2016a.app/bin/maci64/mlint/'

I don't have MATLAB 2016a currently (only 2015a), but is mlint definitely in its own subdirectory in that version? For mine the correct path is /Applications/MATLAB_R2015a.app/bin/maci64/.

That's it - thanks a lot!


Option to export/import apps database

export the sqlite db to external file for backup/restore
exclude cookies or encrypt with password

Any progress with this feature? Just realized I'm going to have to manually add all my apps again :(

Just use normal backup tools

IF you are rooted, you can copy the database file from  /data/data/com.tobykurien.webapps/databases/webapps4.db (doing this from memory so it may not be exactly this) into the new device and it will work. However that's not ideal so I do plan to allow export of the db to sdcard at some point, the difficulty is to add some encryption otherwise anyone getting hold of that file has access to all your sessions.

You can also use adb from your PC (no root needed).
Connect old phone (with USB debugging enabled) to PC then execute:
adb pull /data/data/com.tobykurien.webapps/databases/webapps4.db
Disconnect old and connect new phone (with USB debugging enabled) to PC then execute:
adb push webapps4.db /data/data/com.tobykurien.webapps/databases/webapps4.db
Don't forget to delete the webapps4.db from your PC!

'Permission denied' when trying adb pull using adb on PC with Xiaomi Mi8...export option would be very handy if you can add it 👍

[…] I do plan to allow export of the db to sdcard at some point, the difficulty is to add some encryption otherwise anyone getting hold of that file has access to all your sessions.

Start without encryption? … and put big warning before allowsing to safe?
Then add SAF support, then people can directly safe into some protected area.
I haven't looked into it, but maybe the PGP tools allow direct encryption from one app to the sdcard?


fix: adding api resources to samples & region property update to outbound sample

Issue: adding in compute engine/cloud run api resource and region property to outbound and ingress samples to resolve below issue

@pattishin
Thanks for the PR! 🚀
Unfortunately it looks like some of our CI checks failed. See the Contributing Guide for details.

⚠check_terraform
Failed Terraform check. More details below.

Running terraform fmt
cloud_run_service_ingress/main.tf
--- old/cloud_run_service_ingress/main.tf
+++ new/cloud_run_service_ingress/main.tf
@@ -15,7 +15,7 @@
 */

provider "google-beta" {
-  region  = "us-central1"
+  region = "us-central1"
}

# Enable Cloud Run API
Error: terraform fmt failed with exit code 3
Check the output for diffs and correct using terraform fmt <dir>
cloud_run_service_static_outbound/main.tf
--- old/cloud_run_service_static_outbound/main.tf
+++ new/cloud_run_service_static_outbound/main.tf
@@ -15,7 +15,7 @@
 */

provider "google-beta" {
-  region  = "us-central1"
+  region = "us-central1"
}

# Enable Compute Engine API
Error: terraform fmt failed with exit code 3
Check the output for diffs and correct using terraform fmt <dir>

@pattishin
Thanks for the PR! 🚀✅ Lint checks have passed.


cmd/go: go mod tidy on a forked repo fails with unclear error message

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

$ go version
go version go1.13.4 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOOS="darwin"

What did you do?

This is I think more of an error message issue than anything else.
Say there is a project I like
https://github.com/manifoldco/promptui for example.
Say I forked it.
https://github.com/moficodes/promptui
And then changed the go.mod file to point to a new module name.
module github.com/moficodes/promptui instead of module github.com/manifoldco/promptui
Now once the code is pushed back to github. Then I want to use it in one my other module project.
What did you expect to see?
Go module to pull the code from the repo.
What did you see instead?
$ go mod tidy
go mod tidy
go: custom_select imports
	github.com/moficodes/promptui<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/manifoldco/promptui
	        but was required as: github.com/moficodes/promptui

Solution
This was happening because the original repo had releases and go mod was pulling that as the latest version although the go.mod in the actual repo was updated. So I just made a new release in my fork and it worked ok.
I think the error message can be little bit clearer if possible. I did not see it until I logged into github in my browser.
Or probably there is a more idiomatic way to solve this issue that I am not aware of. Either way more clarity on this would be useful.
😊

Thanks for reporting this, @moficodes. I took a look at https://github.com/moficodes/promptui, and noticed there was still many files that referenced the old name as an import path: github.com/manifoldco/promptui
	modified:   _examples/confirm/main.go
	modified:   _examples/custom_prompt/main.go
	modified:   _examples/custom_select/main.go
	modified:   _examples/prompt/main.go
	modified:   _examples/prompt_default/main.go
	modified:   _examples/prompt_password/main.go
	modified:   _examples/select/main.go
	modified:   _examples/select_add/main.go
	modified:   select_test.go

Changing those files for me locally resolved the issue. This seems to be in line with the error message you mentioned.
I'm going to /cc @bcmills to verify if there's a better way to clarify that, but I think it makes sense.

I dont think having those reference to the old repo was the problem.
I created a new folder with a new gomod file
go mod init select

added file main.go with code
package main

import (
	"fmt"

	"github.com/moficodes/promptui"
)

func main() {
	prompt := promptui.Prompt{
		Label:     "Delete Resource",
		IsConfirm: true,
	}

	result, err := prompt.Run()

	if err != nil {
		fmt.Printf("Prompt failed %v\n", err)
		return
	}

	fmt.Printf("You choose %q\n", result)
}

ran go mod tidy
got
go: select imports
        github.com/moficodes/promptui<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: github.com/manifoldco/promptui
                but was required as: github.com/moficodes/promptui

If I am the only one getting this error then I might have to check my go env setup.

Your forked repo still has references to the old package path. I still don't see a change in the current master. Please see @toothrot's message above.
I agree that this is the right error message. But perhaps @bcmills can weigh in if there is anything else that can be done.

@agnivade I got rid of all the reference to the old repo from this repo.
https://github.com/moficodes/promptui
https://github.com/moficodes/promptui/search?q=github.com%2Fmanifoldco%2Fpromptui&unscoped_q=github.com%2Fmanifoldco%2Fpromptui
Still on the new repo getting the same error message. Thought might module might have been cached. So deleted to go/pkg/mod folder just to be sure.
Still
go: finding github.com/moficodes/promptui v0.3.2
go: downloading github.com/moficodes/promptui v0.3.2
go: extracting github.com/moficodes/promptui v0.3.2
go<EMAIL_ADDRESS>parsing go.mod: unexpected module path "github.com/manifoldco/promptui"
go: error loading module requirements

Because it is taking the 0.3.2 release. Not master. Fetch the master branch or cut a new release and let us know if the problem persists.

@toothrot, note that files in the _examples subdirectory should have no effect. Per https://golang.org/cmd/go/#hdr-Package_lists_and_patterns:

Directory and file names that begin with "." or "_" are ignored by the go tool, as are directories named "testdata".

@moficodes, what about the error message is unclear?
The error messages refer to the specific path and version in use (v0.3.2), and the go.mod file in that version does indeed have a mismatched module path (https://github.com/moficodes/promptui/blob/20f2a94120aa14a334121a6de66616a7fa89a5cd/go.mod#L1).
You may also find the error messages a bit clearer using a go command built from head: we've adding a bit more logging when we resolve an import path to a module, which may help clarify why v0.3.2 is being added to your dependencies.


Implementation for DenseElementsAttr -> DenseResourceElementsAttr

Sai, are the pre-requisites met to rebase/merge this?

Yeah, should I bump iree in turbine first before we land this or either way is fine?


add 0x6bA6d37De43F3e0cD82E4cdE94be1BA724F3D90f.json/png

{
"symbol": "JB",
"address": "0x6bA6d37De43F3e0cD82E4cdE94be1BA724F3D90f"
}

Please provide more info about the token, and attach the token logo image.


Add pre-commit manifest to simplify installing the pre-commit hook (Issue #7)

Pre-commit simplifies installing and using one or more pre-commit hooks. The manifest, .pre-commit-hooks.yaml adds the feature. The README explains use.

Good day,
It looks like py27 is no longer supported by ubuntu-latest. Do you want to drop python 2.7 from CI?

@jfishe Looks good! Thanks so much for adding this - I apologize for the delay, I've been out on travel. I'll make a new release this evening.


NOW MY SEARCH IN NORMAL START MENU DOESNT WORK

NOW MY SEARCH IN NORMAL START MENU DOESNT WORK

skill issue :)

this repo deserves 10k stars

this repo deserves 10k stars

i ahree

"It just works, it just works
Little lies, stunning shows
People buy, money flows, it just works
It just works, it just works"


[test-util] - Consider a mockist approach to supportsTracing

Pulling this issue out from a recent PR review - today we setup a mock tracer, create it globally, track spans, etc. But from a package owner's perspective in most cases all they care about is that they called tracingClient#withSpan with the right arguments.
This issue tracks the investigation of a lighter approach to validation

What's the cost in terms of runtime of the current approach? How much of an improvement is a sinon mock based aproach?
What about package code that does not use withSpan? Do we need to provide a "supportsTracingSlow" and "supportsTracingFast" methods here? Name TBD of course

Really no rush here, especially if we never hit a wall with the current approach


Why is the file not showing up in request.files and in request.forms instead?

I'm am trying to send a file from React to flask, but when I send it, it doesn't come up in request.files where it's supposed to be, and instead in request.form. How do I fix this?
here is react code:
const data = new FormData();     data.append("file", file);     await fetch("/run", {       method: "POST",       body: data     }).then(res => res.json())     .then(data => console.log(data.file))     .catch(error => console.error(error))   }
here is the flask code:
@app.route('/run', methods=['GET', 'POST']) def upload_file():     if request.method == 'POST':         if 'file' not in request.files:             flash('No file part')             return {'file': "no file part"}         file = request.files['file']         if file.filename == '':             flash('No selected file')             return {'file': 'no selected file'}         if file and allowed_file(file.filename):             filename = secure_filename(file.filename)             file.save(os.path.join(app.config['UPLOAD_FOLDER'], filename))             return redirect(url_for('uploaded_file',                                     filename=filename))     else:         return {"file": "this a get"}

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.


Anchors limited to use on Single World only

Feature request
Feature description
I would like to be able to create an anchor that players can use on one world only.  In other words, I would like to create an anchor to Town on a HomeWorld that player can use to go to a Town while in the HomeWorld, but not allow them to use it while on a different world, such as a Survival World.
How the feature is useful
A common MV scenario:  A server allows players to port between worlds using only established portals (so allowing them world access but  denying them teleport permissions).  However, players can currently use an anchor to port between worlds bypassing denied teleport permissions, which allows them to cheat survival mode
In other words, if you give players permission to use an anchor to Town (anchor point) on a HomeWorld, they can use that anchor to port from a Survival World to the HomeWorld despite the fact they are denied teleport permission to port there.  This allows them to cheat survival mode by using the anchor when they get in trouble, such as lost.
Would you please consider adding permissions to anchors to allow use (1) in a single world only, or (2) multiple worlds.  Thank you very much in advance, a very big fan of your plugin.

If you don't have any anchors in your survival world, then is possible with LuckPerms since you can apply permissions uniquely per-world. For example, only give multiverse.teleport.self.a (the permission to teleport to anchors) on your HomeWorld.

Thank you, that is exactly what I did, and it still sets the anchor globally rather than per world.  I set one anchor on the HomeWorld, then gave users permission to use it (/lp group default permission set multiverse.teleport.self.a true).  With this permission, players can use the anchor from other worlds to port to the HomeWorld.  I didn't give them teleport permission (/mvtp Homeworld).  Instead they have to use established portals.  But, they are using the anchor to escape the survival world.
As a test, I removed the permission, and players can't use the anchor.  I added back the permission, and they can use the anchor to port from any world.  Albeit, a one-way trip, it's still being used as a survival world cheat more or less.
It appears to be an inconsistency between setting teleport permission per world and anchors only globally.  I think anchors just need to be given the option of setting them "per world" or "globally."
Thank you for the quick response.

I believe making anchors subordinate to teleport permissions might work.
If you denied teleport permissions to a world, then players couldn’t use anchors to bypass the permission setting.
If you gave teleport permission to another world, then there is no harm in letting them use either the teleport or anchor command to go there.

What I might with LuckPerms was to only give the the anchor permission on the world they are allowed to use it. For example, either of these commands might work:
/lp group default permission set multiverse.teleport.self.a true world=allowed_world
/lp group default permission set multiverse.teleport.self.a false world=survival_world

Thank you nicegamer7.  I tried what you suggested but the anchor command is still set globally, i.e., still allows players to teleport between worlds.  LuckPerms accepted your suggested permissions, but did not change anything.

What I did:
I removed MV anchor permission (/lp group default permission set multiverse.teleport.self.a).
Then I added what you suggested:
/lp group default permission set multiverse.teleport.self.a true world=allowed_world
/lp group default permission set multiverse.teleport.self.a false world=survival_world
The result:
No one could then use the anchor (/mvtp a:Town)
I added back MV anchor permission (step 1), and then did what you suggested (step 2):
But then players could use the anchor to port from SurvivalWorld to HomeWorld, again

I checked the Multiverse-Core or Multiverse-Portals server files, e.g., config, worlds, etc., but there isn't any files that I can edit to limit anchors for use on a a single world.  I don't believe the answer lies there.
This is a love/hate conundrum for me, I love solving puzzles like this, but then I hate that I'm driven to spend all of my time trying to solve them.
I do thank you very much for your helping in trying to solve this,  Malum_In_Se

OMG nicegamer7, it does work!  You're my new best friend.  I can't thank you enough so I'll just close with wishing you an awesome and healthy 2021.

No problem! And same wishes to you :)


Any update on iOS build issues?

With the newer SDK release, I was wondering if there was any progress made on the build issues trying to run the local build on Apple Silicon MacBooks iOS simulators? Or, is this something that is wholly dependent on Brother, and not really something we can address from the consumer side unless we add in our own drivers somehow?
Have you had any communication with Brother on this - have they expressed any indication or timeline for when this could be achieved? We have currently rolled our our printing solution to one customer, and plan to roll it out to another in a couple of months - would be nice to be able to fully test our build locally, without running on a physical device on a built ipa distribution.

Hi @mikhael28 ,
Thanks for reaching out about this. I'd love to add your app to the another_brother list of apps so other developers can get some inspiration so if you have the link handy it would be super.
With regards to the iOS simulator support,  on their recent release Brother seems to have added support for running on the iOS simulator. However, because I am not familiar with the configuration of the podspec, I have not been able to properly configure the pod to support both the device and simulator.
If you have more familiarity with it the project is available here https://github.com/CodeMinion/BRLMPrinterKit
Thanks again for reaching out,

Hey @CodeMinion It's been a while since we've updated, I think we locked at 0.32 - I will likely revisit the functionality in a few weeks, to see if I can somehow load the arm/rosetta build of the drivers conditionally in the pubspec. I think that's the issue? Would be nice to update the package then. My coworker is eager to resolve the issue of having to comment/uncomment the another_brother library while developing new features for.
Here is a link to the app - it's called ARMS Mobile, short for Automated Railroad Management System. It's an industrial piece of software, primarily serving the railroad industry and also transloading, intermodal & warehousing. The printing functionality is a very small part of it - a way for a facility to print backup QR codes for their inventory, in case the original gets damaged. Also, they can print QR codes to serve as labels for parts of their facilities. I originally became aquainted with your software during the Hackathon in 2022, and remembered it when I had to do this integration. I was a little salty when Linus asked me to ship the test printer back, but I did drop out of the hackathon towards the end so what can you do.
https://apps.apple.com/us/app/arms-mobile/id1615127530

Hi @mikhael28 ,
Thanks for the link. I have added it to the ReadMe and will be included in the next release of another_brother.

Sounds good! If anyone has any updates on how to make iOS build on an apple silicon simulator, please post here lol


My Snippets Page

I made the backend route named mysnippets/[id] to fetch the snippets submitted by the current user.
Along with the frontend with a useeffect to fetch current user and fetchUsersSnippets to fetch the snippets written by current user.

@notsoocool  This is in response to the issue #34

@inkerton, I appreciate your contribution, but we need to make a few adjustments:

Kindly pull the latest changes from the develop branch, as we've introduced a new My Profile section when you click on your avatar. This section now includes My Snippets, My Bookmarks, and My Contributions.
What I'd like you to do is revamp the design of My Snippets section and make any necessary updates. Also, please check if the My Snippets feature is functioning properly.

Thank you!

Also please raise a PR for develop branch only!

I don't see any  My Profile section . @notsoocool

I don't see any My Profile section . @notsoocool I made a new branch on your develop branch

you will see it if you fetch the develop branch with the latest updates


Simplify multi-arch toolchain config

To support remote cross-build use case, where a build could be run on a
MacOS arm64 host machine but target (and execute on) a remote Linux x86
executor pool, we want to rely on rctx.os as much as possible.
Simplify our ubuntu toolchain configuration to support both x86 and
arm64 import path. Drop the arch constraint value from the toolchain as
it should now support both architecture. Reformat toolchain's starlark
to improve readability.
Make sure that we always set Arch exec property on Linux platforms.
Simplify the templating for default platform.

There is a slight problem with the default container image as well: the older images don't support arm64. So we either need to (a) force the latest container image for the arm64 platform or (b) allow folks to use a different image for each platform.
Not sure what's the right UX here so I did not make that change.

I created an example usage here https://github.com/buildbuddy-io/buildbuddy/compare/master...sluongng/demo-multi-arch-toolchain
Tested with
# Use my local copy of the toolchain
> tail -1 user.bazelrc
common --override_repository=io_buildbuddy_buildbuddy_toolchain=/Users/sluongng/work/buildbuddy/buildbuddy-toolchain

> bazel test --config=remote //...

# some other changes are needed for a full repo build on arm64
> bazel build --config=remote-arm64 enterprise/server/cmd/executor


Several GPO options are wrong in the documentation

In the "Update endpoint protection configuration" section the "Enable sample collection on machines" policy option Is not located at "\Windows Components\Windows Defender SmartScreen*" but at "\Windows Components\Windows Defender ATP/".
Furthermore, the policy location " \Windows Components\Microsoft Defender AntivirusScan" does not exist an should probably be  \Windows Components\Microsoft Defender Antivirus\Scan

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 523651c3-a318-2369-6892-d13e04991596
Version Independent ID: 624c5e05-f87d-1bd4-bc87-bd55dcac1d86
Content: Onboard Windows 10 devices to Microsoft Defender for Endpoint via Group Policy
Content Source: microsoft-365/security/defender-endpoint/configure-endpoints-gp.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@jdcc2 Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5401
Thanks
Sri


raise_exception seems not to work properly with wait_for

Thank you and please read about what information I'll need to diagnose/fix.

Closing in favor of #20, which has more information. Thank you again!


Application gets stuck

I am trying to use Gemini for graph processing on a single server. It gives this warning:
" A high-performance Open MPI point-to-point messaging module
was unable to find any relevant network interfaces"
and then the program gets stuck (most likely outcome). Sometimes the program proceeds forward after this warning and execution finishes but that happens like 1 in 10 times. As I wish to run multiple experiments, I intend to use scripts and with the application getting stuck more often than not, it is difficult. Please let me know how to resolve this.

Hi. Can you run other simpler MPI programs (e.g. do some arithmetics and AllReduce the sum) correctly?

My simple MPI hello world program works. But the following run doesn't work. Is it because of MPI_THREAD_SERIALIZED?
$ mpirun -n 2 ./pagerank ../../inputs/soc-LiveJournal1.txt.bsnap 4847571 1
thread support level provided by MPI: MPI_THREAD_SERIALIZED
|V| = 4847571, |E| = 68475391
|V'_0| = 1261568 |E^dense_0| = 43518651
|V'_1| = 3586003 |E^dense_1| = 24956740
|V'_0_0| = 1261568 |E^dense_0_0| = 43518651
|V'_1_0| = 3586003 |E^dense_1_0| = 24956740
[yuxing-desk:01757] *** Process received signal ***
[yuxing-desk:01757] Signal: Segmentation fault (11)
[yuxing-desk:01757] Signal code: Address not mapped (1)
[yuxing-desk:01757] Failing at address: 0x14ec650
[yuxing-desk:01757] [ 0] /lib/x86_64-linux-gnu/libpthread.so.0(+0x11390)[0x7ff406b0d390]
[yuxing-desk:01757] [ 1] /usr/lib/openmpi/lib/openmpi/mca_pml_ob1.so(mca_pml_ob1_recv_request_progress_frag+0x59)[0x7ff3f98dbc19]
[yuxing-desk:01757] [ 2] /usr/lib/openmpi/lib/openmpi/mca_btl_vader.so(mca_btl_vader_poll_handle_frag+0x93)[0x7ff3fa344813]
[yuxing-desk:01757] [ 3] /usr/lib/openmpi/lib/openmpi/mca_btl_vader.so(+0x3abe)[0x7ff3fa344abe]
[yuxing-desk:01757] [ 4] /usr/lib/libopen-pal.so.13(opal_progress+0x4a)[0x7ff40602d1ea]
[yuxing-desk:01757] [ 5] /usr/lib/openmpi/lib/openmpi/mca_pml_ob1.so(mca_pml_ob1_send+0x4c5)[0x7ff3f98d5745]
[yuxing-desk:01757] [ 6] /usr/lib/libmpi.so.12(PMPI_Send+0x14b)[0x7ff40753ccdb]
[yuxing-desk:01757] [ 7] ./pagerank[0x4151dd]
[yuxing-desk:01757] [ 8] ./pagerank[0x409db3]
[yuxing-desk:01757] [ 9] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0)[0x7ff406752830]
[yuxing-desk:01757] [10] ./pagerank[0x409f79]
[yuxing-desk:01757] *** End of error message ***
--------------------------------------------------------------------------
mpirun noticed that process rank 1 with PID 1757 on node yuxing-desk exited on signal 11 (Segmentation fault).
--------------------------------------------------------------------------


Update CCG & GP Dashboard Text

Remove text on dashboards that says:
'Over the past 6 months, this organisation has prescribed worse than the median on 5 of 14 measures.
The measure with the biggest potential for improvement was Dosulepin (KTT8), where this CCG was at the 90th percentile on average across the past 6 months.'
Replace this with text:
'These measures show only a small subset of the data available on OpenPrescribing. Use our analyse page to ask your own questions of the data, and explore prescribing trends by BNF section or chemical. You can also compare CCG prescribing for all of our measures'

I'd like to propose we just remove the text for the time being, and then A/B test adding the text at some point soonish.
null hypothesis: adding any text there is just noise and won't lead to increased exploration of our data through other means.

Yes agree!


EXC_BAD_ACCESS in big painters on Mac OS (XCode)

Starting from an unknown size, painters start to crash with EXC_BAD_ACCESS.
For smaller sizes, crashes con't always occur and if they do, it can be in different places (constructor, destructor, in methods, ...)
For bigger sizes of the painter, the crash always occurs in the constructor and the stack trace ends in the AbstractProperty constructor.
Reproducible for me by adding int a[1024]; to the EmptyExample class and then executing the example.

Just tried building with Makefiles and can't reproduce the error.
Maybe it's a problem with how XCode configures the project?

Fixed by building all dependencies with XCode instead of using Makefiles.
Apparently, XCode uses a different compiler version than the one CMake detects.


Bring back - Wait for Static IP to be hooked up

Bring back changes from https://github.com/knative/net-kourier/commit/205bd4033e7ef70fb0abc64b5641f97536073b97 that very recently overridden.

/lgtm
/approve