Execution failed for task ':react-native-community_art:compileDebugJavaWithJavac' when to try build android after update to 51 sdk

Build/Submit details page URL
https://expo.dev/accounts/jackcat1/projects/cckzMApp/builds/303734f0-fccd-429c-93e2-1f784b3f1630
Summary
After updating the SDK to version 51, it is not possible to build for development
Managed or bare?
Managed
Environment
expo-env-info 1.2.0 environment info:
System:
OS: Windows 10 10.0.19045
Binaries:
Node: 20.15.0 - C:\Program Files\nodejs\node.EXE
npm: 10.7.0 - C:\Program Files\nodejs\npm.CMD
IDEs:
Android Studio: AI-241.18034.62.2411.12071903
npmPackages:
babel-preset-expo: ~11.0.0 => 11.0.12
expo: ~51.0.21 => 51.0.21
react: 18.2.0 => 18.2.0
react-dom: 18.2.0 => 18.2.0
react-native: 0.74.3 => 0.74.3
react-native-web: ~0.19.6 => 0.19.12
Expo Workflow: managed
WARNING: We recommend using PowerShell or Bash via WSL 2 for development with Expo CLI on Windows. You may encounter issues using cmd.exe.
✔ Check Expo config for common issues
✔ Check package.json for common issues
✔ Check native tooling versions
✔ Check dependencies for packages that should not be installed directly
✔ Check for common project setup issues
✔ Check npm/ yarn versions
✔ Check for issues with metro config
✔ Check Expo config (app.json/ app.config.js) schema
✔ Check for legacy global CLI installed locally
✔ Check that native modules do not use incompatible support packages
✔ Check that packages match versions required by installed Expo SDK
✔ Check that native modules use compatible support package versions for installed Expo SDK
Didn't find any issues with the project!
Error output

Task :app:processDebugMainManifest
/home/expo/workingdir/build/android/app/src/debug/AndroidManifest.xml:6:5-162 Warning:
application@android:usesCleartextTraffic was tagged at AndroidManifest.xml:6 to replace other declarations but no other declaration present
/home/expo/workingdir/build/android/app/src/debug/AndroidManifest.xml Warning:
provider#expo.modules.filesystem.FileSystemFileProvider@android:authorities was tagged at AndroidManifest.xml:0 to replace other declarations but no other declaration present
Task :app:processDebugManifest
Task :app:processDebugManifestForPackage
Task :react-native-community_art:compileDebugJavaWithJavac
/home/expo/workingdir/build/node_modules/@react-native-community/art/android/src/main/java/com/reactnativecommunity/art/ARTShapeShadowNode.java:25: error: cannot find symbol
import static com.facebook.react.common.ArrayUtils.copyArray;
^
symbol:   class ArrayUtils
location: package com.facebook.react.common
/home/expo/workingdir/build/node_modules/@react-native-community/art/android/src/main/java/com/reactnativecommunity/art/ARTShapeShadowNode.java:25: error: static import only from classes and interfaces
import static com.facebook.react.common.ArrayUtils.copyArray;
^
Note<EMAIL_ADDRESS>uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
2 errors
/home/expo/workingdir/build/node_modules/@react-native-async-storage/async-storage/android/src/main/java/com/reactnativecommunity/asyncstorage/AsyncStorageModule.java:84: warning: [removal] onCatalystInstanceDestroy() in NativeModule has been deprecated and marked for removal
public void onCatalystInstanceDestroy() {
^
Task :react-native-community_art:compileDebugJavaWithJavac FAILED
Task :react-native-async-storage_async-storage:compileDebugJavaWithJavac
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note<EMAIL_ADDRESS>uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 warning
Task :app:processDebugResources
Task :expo-modules-core:compileDebugKotlin
w: file:///home/expo/workingdir/build/node_modules/expo-modules-core/android/src/main/java/expo/modules/adapters/react/permissions/PermissionsService.kt:20:26 'Promise' is deprecated. AsyncFunction will crash when called. Use expo.modules.kotlin.Promise instead
w: file:///home/expo/workingdir/build/node_modules/expo-modules-core/android/src/main/java/expo/modules/adapters/react/permissions/PermissionsService.kt:63:51 'Promise' is deprecated. AsyncFunction will crash when called. Use expo.modules.kotlin.Promise instead
w: file:///home/expo/workingdir/build/node_modules/expo-modules-core/android/src/main/java/expo/modules/adapters/react/permissions/PermissionsService.kt:90:54 'Promise' is deprecated. AsyncFunction will crash when called. Use expo.modules.kotlin.Promise instead
w: file:///home/expo/workingdir/build/node_modules/expo-modules-core/android/src/main/java/expo/modules/kotlin/defaultmodules/CoreModule.kt:6:34 'ReactFeatureFlags' is deprecated. Deprecated in Java
w: file:///home/expo/workingdir/build/node_modules/expo-modules-core/android/src/main/java/expo/modules/kotlin/defaultmodules/CoreModule.kt:71:12 'ReactFeatureFlags' is deprecated. Deprecated in Java
w: file:///home/expo/workingdir/build/node_modules/expo-modules-core/android/src/main/java/expo/modules/kotlin/events/KModuleEventEmitterWrapper.kt:97:7 'constructor Event<T : Event<(raw) Event<*>>!>(Int)' is deprecated. Deprecated in Java
w: file:///home/expo/workingdir/build/node_modules/expo-modules-core/android/src/main/java/expo/modules/kotlin/types/JSTypeConverterHelper.kt:44:17 'get(String!): Any?' is deprecated. Deprecated in Java
FAILURE:
Build failed with an exception.

What went wrong:
Execution failed for task ':react-native-community_art:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Hello I can't see that the issue is closed, how did you maneged to fix the issue as I have the same ?

I removed "react-native-community_art" lib.

If someone is still facing this, just comment out the line
import static com.facebook.react.common.ArrayUtils.copyArray;
in the directory
<EMAIL_ADDRESS>

Hey, I'm facing the exact same error.
How did you manage to fix it?


show-open-prs-of-forks: UI changed, feature missing

Description
The UI was updated, so it's likely that this feature is now broken
How to replicate the issue + URL

Visit https://github.com/pulls?q=is%3Apr+is%3Aopen+author%3A%40me+archived%3Afalse+-user%3A%40me
Find a PR made from a fork
In it, open your own fork

Extension version
23.9.21
Browser(s) used
Safari

I see it


[6.0][CSDiagnostics] Add a few tailored diagnostics for opaque return type requirements mismatches

Cherry-pick of https://github.com/apple/swift/pull/72493, https://github.com/apple/swift/pull/72558

Explanation:

Add a tailored diagnostics for same-type and superclass requirement failures related to opaque return types.
(All of the failures were covered by "conformance" diagnostic).

Scope: Mismatches between opaque return type requirements and type of return expressions

Main Branch PRs: https://github.com/apple/swift/pull/72493, https://github.com/apple/swift/pull/72558

Risk: Very Low

Reviewed By: @slavapestov

Testing: Added test-cases to the test suite.

@swift-ci please test

@swift-ci please test


Autoplay not working in modal screen with wix/react-native-navigation

Description
We recently updated react-native-lottie from v4.0.2 to v5.1.3.
After this update we started encountering an issue only on iOS with LottieView in modal screen managed via wix/react-native-navigation . The auotplay does not start.
I do not know if this issue can be reproduced with other navigation SDKs.
In order to to fix this issue at the moment I used a workaround with useRef, useEffect and a timeout:
  useEffect(() => {     setTimeout(() => animationRef.current?.play(), 100);   }, []);
In this way I managed to start the animation.
Steps to Reproduce

Integrate LottieView in a modal screen with autoplay
Open modal screen

Expected behavior: Lottie animation should play automatically
Actual behavior: The Lottie animation does not start
Versions
lottie-ios: "^3.2.3",
lottie-react-native: "^5.1.3",

Hi there, I have exactly the same Issue.

same issue here .. it doesn't work at all!!

Please try upgrading to the latest version with lottie-ios:3.4.0 and lottie-react-native:5.1.4, and let us know if it still occurs.

I upgraded it like you recommended but the issues still occur.
Do you have another idea how to fix this properly?

having same issue, did anyone find a way to fix this ?

Happening in my expo project as well, using the same workaround which op has mentioned

Same problem here, does not work in modal.

I guess we have to use the "trick". Even said in Issue #870
Doesn't seem like the maintainers care about this issue.

@matinzd There have been many issues reported with this same issue that have all gone stale with no replies from maintainers except telling us to do the useEffect "hack"

don't know why to use effects, I think it would be better:
function onLayout() {
  requestAnimationFrame(() => ref.current?.play());
}

but, don't know why use tricks at all...

We still have this issue on react-native version 0.70.9. If there is no intention to fix this issue – maybe we should at least document the workaround?

We still have this issue on react-native version 0.70.9. If there is no intention to fix this issue – maybe we should at least document the workaround?

Hi Fotos,
Sorry for the inconvenience this has caused to anyone. There are some other issues which is priotised to be done. This is on my radar and I am the only one maintaining this library actively.
I appreciate your patience and we will try to get to this issue as soon as possible.

Thank you @matinzd for working on this library. 🙏
No worries – the workaround works reliably so far. I just wanted to make sure we document this (README?) so at least people can find this more easily.
There is also PR #1021 open – although to be honest I'm not sure whether that's the best approach, since it just moves the timeout workaround from the application to the library.

Can anyone test the latest RC version and let me know if this still exists there? @fotos @PaoloSalta

Closing this due to inactivity.

Here is a simple hack, use onLayout prop and autoPlay prop both of LottieView and play it like this,
const animationRef = useRef(null);

<LottieView ref={animationRef} onLayout={_ => animationRef.current?.play()} source={YourAnimation} autoPlay loop style={{ ..yourStyles }} />

Here is a simple hack, use onLayout prop and autoPlay prop both of LottieView and play it like this,
const animationRef = useRef(null);

<LottieView ref={animationRef} onLayout={_ => animationRef.current?.play()} source={YourAnimation} autoPlay loop style={{ ..yourStyles }} />

You are a lifesaver I was struggling from last 24 hours Thank you


Access groups

Update the installation process to use the revised acp scripts.

https://github.com/ibm-garage-cloud/ibm-garage-developer-guide/pull/100/commits/1dfe5d2f6829cbf363c0f08ac6ff029a6106c529: Updates to the overall installation plan.

https://github.com/ibm-garage-cloud/ibm-garage-developer-guide/pull/100/commits/43e8af9fd16a774c2af8dc23fca4c84bc833b38a: Updates how to create the access groups.


[FEA] mvn test phase should not require prior install

I wish would make it possible to use Maven test phase without a prior install despite using aggregator modules as a dependency.
New developers routinely run into
Couldn't download artifact: Failure to find com.nvidia:rapids-4-spark-aggregator_2.12:jar:spark311:22.12.0-SNAPSHOT

when executing mvn test
Current procedure for running a single test:
$ mvn clean install -pl tests -am -DskipTests
$ mvn test -pl tests -Dsuites='com.nvidia.spark.rapids.CastOpSuite @ cast string to decimal (include NaN/INF/-INF)'

Resolved via #9698


does not work with hyper's with_proxy_config

This code fails to compile:
extern crate hyper;
extern crate hyper_native_tls;

use hyper::net::HttpsConnector;
use hyper::client::{Client, ProxyConfig};
use hyper_native_tls::NativeTlsClient;

fn main() {
    let ssl = NativeTlsClient::new().unwrap();
    let connector = HttpsConnector::new(ssl);

    let client = Client::with_proxy_config(
        ProxyConfig::new(
            "http", "localhost", 3128, connector, ssl
        )
    );

    let response = client.get("https://httpbin.org").send().unwrap();
    println!("{}", response.headers);
}

What follows is part of the error output:
error[E0277]: the trait bound `hyper_native_tls::TlsStream<hyper::net::HttpStream>: std::fmt::Debug` is not satisfied
  --> src/main.rs:13:9
   |
13 |         ProxyConfig::new(
   |         ^^^^^^^^^^^^^^^^ the trait `std::fmt::Debug` is not implemented for `hyper_native_tls::TlsStream<hyper::net::HttpStream>`
   |
   = note: `hyper_native_tls::TlsStream<hyper::net::HttpStream>` cannot be formatted using `:?`; if it is defined in your crate, add `#[derive(Debug)]` or manually implement it
   = note: required because of the requirements on the impl of `std::fmt::Debug` for `hyper::net::HttpsStream<hyper_native_tls::TlsStream<hyper::net::HttpStream>>`
   = note: required because of the requirements on the impl of `hyper::net::SslClient<hyper::net::HttpsStream<hyper_native_tls::TlsStream<hyper::net::HttpStream>>>` for `hyper_native_tls::NativeTlsClient`
   = note: required by `<hyper::client::ProxyConfig<C, S>>::new`

I asked first on Stack Overflow.

Fixed in 0.2.1!

works now... you are too quick


Live Property Explorer doesn't show ActualOffset value (computed values)

Describe the bug
The Live Property Explorer doesn't show the ActualOffset value. I was thinking it was because it is a Vector3, but DesideredSize is correctly shown even if it isn't a scalar value.
Steps to reproduce the bug
Just create a Winui3 project from scratch and check the ActualOffset of the button.
Expected behavior
No response
Screenshots

NuGet package version
WinUI 3 - Windows App SDK 1.5.6: 1.5.240802000
Windows version
Windows 10 (21H2): Build 19044
Additional context
Visual Studio 17.11.2

ActualOffset & ActualSize are composition properties and so only exist if you request a Composition visual for the property (or use a composition based value on the FrameworkElement)

May you be more clear? I don't know about Composition Properties.
I only know that if I set a Margin of a FrameworkElement, I can observe that ActualOffset change as expected, but in Live Property Explorer is always 0.

ActualOffset and a few other properties on UIElement are what we call "simple properties", which are lighter weight properties not backed by a full DependencyProperty. Live Property Explorer should be made aware of these simple properties.

Got it. Do you think it is very hard to implement my request given the actual state of things?
I was developing a UI component (with subcomponents which are going to move around), and to debug it would be very useful having such feature.


ngModel Controller

I think my biggest concern with this library is it's lack of integration with angular controllers.
When an editable-{{ type }} directive is specified it doesn't (at least internally) use the ng-model controller - this has some pretty ugly side effects when trying to transclude ng-pattern, ng-required, and any custom directives that use ng-model
Has anyone been able to get a workaround for this? Or maybe identified the different roadblocks that the library may have moving to a better integrated directive(s)?

1

Pull request are welcome :)


Server cannot be started in a non-blocking manner

Eel version
0.14.0
Describe the bug
When starting an Eel server using threading, multiprocessing or block=False it does not start
To Reproduce
Steps to reproduce the behavior:

Start the server in any of the non-blocking manners mentioned.
Visit http://localhost:8000/, only to see the "connection refused" tab.

Expected behavior
Server should start in a non-blocking manner.
Desktop (please complete the following information):

OS: Ubuntu 21.04
Browser: any

Additional context
When starting the server in a blocking manner, that is using server.start("page.html"), the server works as intended. I don't know if there's a problem with it being started like I mentioned, but this is a major roadblock if I want to, say, display the GUI using a module like pywebview.

Nevermind, using a function to call eel.start("page.html") breaks it, while using
server = Process(target=eel.start, args=["page.html"], ...); server.start() doesn't.


fix for https://github.com/pvlib/pvlib-python/issues/102

inserted link to variables

Thanks. Perhaps I wasn't clear enough in #102... Since you seem particularly interested in improving the documentation, I recommend that you set up readthedocs.org to render your fork of pvlib. This will make it easier to review this PR and future PRs. If you're just making small edits to the docs then I don't think it's needed, but here it will be helpful to see the rendered version since you're making new files and tables.
 See here:
http://docs.readthedocs.org/en/latest/getting_started.html#import-your-docs
Then post a comment here with a link to your version of the new docs.

See this:
http://pvlib-python-dacoex.readthedocs.org/en/latest/

Thanks, that is helpful. A few comments:

The page name is "Variables and style rules" but it currently only has a table of variables. We need to either just have the table of variables and change the name or we need to include some text about style rules and keep the name as is. I don't have an opinion on what's best.
I'd replace "rules" with "conventions".
Do we need to keep the "comments" column? I'd say no.
Do we want to keep the wiki page after merging this? I'd say delete it so that we don't have to make these changes in two places.

OK. 1 question to my earlier comment :+1:

Additionally, the csv values can be reused, e.g. to assign variables or translate variables into >descriptions.

We could add the table to the folder
https://github.com/pvlib/pvlib-python/tree/master/pvlib/data
Then, we could read the variables in and make descriptive lables for graphs etc from it:
long_name = variable_csv_file['dni']

We could add the table to the folder
https://github.com/pvlib/pvlib-python/tree/master/pvlib/data Then, we could read the variables in and make descriptive lables for graphs etc from it: long_name = variable_csv_file['dni']

Maybe. I doubt that many people would take advantage of something like that, but it probably doesn't hurt anything. We'd probably want to change the multiple-valued cells if we did this.

@wholmgren I will move the table
https://github.com/pvlib/pvlib-python/tree/master/pvlib/data
this makes renaming to the agreed variables easier.

@dacoex you should rebase this and rebuild your docs now that #108 has been merged.

@dacoex thanks for the updates. Please see the line notes above.

all proposals where I did not add a counter comment above are noew implemented.
Probably left with the table wrap issue.

Doesn't look like it's rendering right now.
http://pvlib-python-dacoex.readthedocs.org/en/latest/variables_style_rules.html

looks good now, I hope. CU tomorrow. I am in another tz (UTC+1)

Looks good, thanks.

Thanks for the patience. I can only learn from this peer review. And it increases my confidence in the code.
Next time tell me and I will stash the trial and error together.


Option to set UE node title to type on connection

Maybe with "?" if UE?, and auto collapse

I recently ran into something similar and fixed it using the Anything Everywhere? node, here is a very basic example, maybe it will help with what you are doing?

anything_everywhere_fix.json

Personally I like naming them after the actual input_regex sent and keep them collapsed, while only naming them after the amount of signals when grouped


Please support for laravel 5.7

Thanks for your awesome package. Please support for laravel: ^5.7.*

#518 is already open. Today I've holiday so I can't release. @dimsav could or we wait until tomorrow. 😉
My opinion: don't use a super fresh release in production.

v9.2.0 is released now.


Failed to resolve: moe.feng:MaterialStepperView:latest-version

I've configured the gradle  as in description, by the way I have this message
Error:(30, 13) Failed to resolve: moe.feng:MaterialStepperView:latest-version
How it is possible?

latest-version is a placeholder. Please replace it with a actual version code such as 0.2.3.

sorry
too many hours in front of the pc, Thanks


A couple of little comment fixes

I'm not sure if any description is needed here. :p

@Iniquitatis thanks!


[docs] Better API spread section

Closes #10393
-Any other properties supplied will be [spread to the root element](/guides/api#spread).
+Any other properties supplied will be [spread to](/guides/api#spread) the root element ([Paper](/api/paper)).

Good idea, but could we perhaps just link from the word "spread" to match the API guide section name?

Good idea


Fix scheduling option

URL to the relevant documentation
the scheduling option on this page: https://docs.quantum.ibm.com/transpile/defaults-and-configuration-options, `
scheduling_method="as_late_as_possible"
is not a valid option
Select all that apply

[ ] typo
[X] code bug
[X] out-of-date content
[ ] broken link
[ ] other

Describe the fix.
Verify that all options are valid

I'm following up with this, as the API seems to list as_late_as_possible as a valid option: https://docs.quantum.ibm.com/api/qiskit/transpiler_preset#generate_scheduling

Closing - the documentation is OK, but there's a bug:  https://github.com/Qiskit/qiskit/issues/12464


ISPN-10863 java.lang.IllegalAccessException logged by PropertyFormatter

should not attempt to access static or non-public methods and should
mark them accessible to prevent eventual cross-module accessibility
checks from failing

https://issues.jboss.org/browse/ISPN-10863

CI looks good :)

Thanks @anistor. Do we need this backported to 10.0.x?

Opps, did not see this message. I'll make a pr just in case. I'm sure there will be more micros in 10.x stream.


[BUG] Empty item at start of the Rows while reading excel file

testexceljs.xlsx
🐛 Bug Report
I am trying ti read a excel(.xlsx) file
so i raid it, while getting values of rows there is a  in start of each row.
Lib version: latest
Steps To Reproduce

const readWorkbook = new ExcelJS.Workbook();
await readWorkbook.xlsx.readFile('testexceljs.xlsx')
const readWorksheet = readWorkbook.getWorksheet('TestWorksheet');
expect(readWorksheet.getRow(2).values).to.be.have([1, 'John', 'Doe'])
        ```

## The expected behaviour:

i was expecting the row should be [1, 'John', 'Doe'] but the actual value is [<empty item>, 1, 'John', 'Doe']

Duplicate of https://github.com/exceljs/exceljs/issues/1436. I found what the issue is and commented in there.

You can use row._cells
to get the parse row array without any undefined element


728 indiv names on sighting encounters

Moves some schemas (schemae?) around so that individual names are included on a sighting's encounters, with each name's context and value listed.
Adds a NamedIndividualSchema to individuals/schemas.py which is just the base schema plus names. This is then included in the encounter AugmentedEdmSchema which is used on a sighting GET.
test_create_sighting.py::test_edm_and_houston_encounter_data_within_sightings tests that we're getting the contexts and values for each name.
A side-effect here you'll see in the integration test is that if there is no individual on an encounter, that entry will be individual: None on the sighting encounter, conforming with other None-able fields like submitter. Previously, the no-individual value was {}.

leaving this as a draft until tests are complete since I just rebased.

/rebase

\rebase

/rebase


It will keep overheating in summer in world_nether

This issue has been fixed in 1.0.5: 34f6451
However, you can play with heat-effect-threshold in the config if you are still not happy with the results.


wifi task: abort in ppRxProtoProc() (IDFGH-677)

I've got the abort() in the 'wifi' task periodically. Since the library is closed-sourced, I can't find any fixes or workarounds.
Environment

Development Kit:      [none]
Module or chip used:  [ESP32]
IDF version: v3.3-beta1-328-gabea9e4c0
Build System:         [Make]
Compiler version: crosstool-NG crosstool-ng-1.22.0-80-g6c4433a
Operating System:     [Linux]
Power Supply:         [USB]

Problem Description
abort() was called from a closed-source IDF code
Expected Behavior
System libraries must never call abort()
Actual Behavior
Thread #6<PHONE_NUMBER> (wifi : Running @CPU0) (Suspended : Breakpoint)
abort() at panic.c:160 0x4008ce4c
ppRxProtoProc() at 0x4013749c
ppRxPkt() at 0x40137b88
ppTask() at 0x40088121
vPortTaskWrapper() at port.c:143 0x4008f3b8
Steps to repropduce

init wifi
spend some time in debugger (stop program execution at breakpoints, step program etc.
abort() occurred in the 'wifi' task.

The rest of the message is the log. The last log record seems being related to the problem.
I (551) phy: phy_version: 4100, 6fa5e27, Jan 25 2019, 17:02:06, 0, 0
I (551) wifi: mode : sta (d8:a0:1d:62:d1:68)
D (551) event: SYSTEM_EVENT_STA_START
I (1761) wifi: new:<11,0>, old:<1,0>, ap:<255,255>, sta:<11,0>, prof:1
I (2751) wifi: state: init -> auth (b0)
I (2751) wifi: state: auth -> assoc (0)
I (2761) wifi: state: assoc -> run (10)
I (2781) wifi: connected with monster1, channel 11, bssid = 38:4c:4f:12:28:84
I (2781) wifi: pm start, type: 1
D (2781) event: SYSTEM_EVENT_STA_CONNECTED, ssid:monster1, ssid_len:8, bssid:38:4c:4f:12:28:84, channel:11, authmode:3
D (2791) tcpip_adapter: dhcp client init ip/mask/gw to all-0
D (2791) tcpip_adapter: if0 start ip lost tmr: enter
D (2801) tcpip_adapter: if0 start ip lost tmr: no need start because netif=0x3ffcd0e0 interval=120 ip=0
D (2811) tcpip_adapter: dhcp client start successfully
D (9271) tcpip_adapter: if0 dhcpc cb
D (9271) tcpip_adapter: if0 ip changed=1
D (9271) event: SYSTEM_EVENT_STA_GOT_IP, ip:<IP_ADDRESS>, mask:<IP_ADDRESS>, gw:<IP_ADDRESS>
I (9271) event: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
D (9281) sdspi_host: sdspi_host_init_slot: SPI2 miso=18 mosi=19 sck=21 cs=22 cd=23 wp=-1, dma_ch=2
D (9291) spi: SPI2 use gpio matrix.
D (9291) intr_alloc: Connected src 30 to int 13 (cpu 0)
D (9301) spi_master: eff: 400, limit: 26666k(/2), 0 dummy, -1 delay
D (9301) spi_master: SPI2: New device added to CS0, effective clock: 400kHz
I (9311) gpio: GPIO[22]| InputEn: 0| OutputEn: 1| OpenDrain: 0| Pullup: 0| Pulldown: 0| Intr:0
I (9321) gpio: GPIO[23]| InputEn: 1| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:0
D (9331) spi_master: SPI device changed from 3 to 0
D (9331) sdspi_host: start_command_default: poll_cmd_response returned 0x107
D (9341) sdspi_host: sdspi_host_start_command: cmd=52 error=0x107
D (9351) sdmmc_cmd: cmd=52, sdmmc_req_run returned 0x107
D (9391) sdmmc_sd: SDHC/SDXC card
D (9391) sdspi_transaction: cmd=5, R1 response: command not supported
D (9391) sdmmc_cmd: cmd=5, sdmmc_req_run returned 0x106
D (9391) sdmmc_io: sdmmc_init_io: io_send_op_cond (1) returned 0x106; not IO card
D (9401) sdspi_host: data CRC set=1
D (9571) sdmmc_common: host_ocr=0x40ff8000 card_ocr=0xc0ff8000
D (9571) sdmmc_common: sdmmc_card_init: host_ocr=40ff8000, card_ocr=c0ff8000
D (9571) sdmmc_init: sdmmc_card_init: card type is SD
D (9581) sdmmc_common: sdmmc_init_host_frequency: using 20000 kHz bus frequency
D (9581) sdspi_host: Setting card clock to 20000 kHz
D (9591) spi_master: eff: 20000, limit: 26666k(/2), 0 dummy, 0 delay
D (9591) spi_master: SPI2: New device added to CS0, effective clock: 20000kHz
D (9601) spi_master: SPI device changed from 3 to 0
I (9611) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE
I (26541) wifi: bcn_timout,ap_probe_send_start
I (45451) wifi: bcn_timout,ap_probe_send_start
I (54231) wifi: bcn_timout,ap_probe_send_start
E (98211) wifi: l:209138774, o:411, n:219374678, now:286578322

spend some time in debugger (stop program execution at breakpoints, step program etc.

How long, approximately, do you keep the program halted in the debugger?
The thing is, Wi-Fi hardware can keep working for a short while without interrupts being serviced, but not indefinitely. At some point, because the CPU is halted and can not process interrupts, Wi-Fi hardware will come to a state from which the driver fails to recover, indicating it with an abort() call.
If you are debugging something that is not Wi-Fi related, it might make sense to build the application in a configuration where it will not initialize Wi-Fi, or mock that initialization somehow.

Hi Ivan,
05.03.2019 18:12, Ivan Grokhotkov пишет:

spend some time in debugger (stop program execution at
breakpoints, step program etc.

How long, approximately, do you keep the program halted in the debugger?

It depends. For about a minute, totally.

The thing is, Wi-Fi hardware can keep working for a short while
without interrupts being serviced, but not indefinitely. At some
point, because the CPU is halted and can not process interrupts, Wi-Fi
hardware will come to a state from which the driver fails to recover,
indicating it with an abort() call.
If you are debugging something that is not Wi-Fi related, it might
make sense to build the application in a configuration where it will
not initialize Wi-Fi, or mock that initialization somehow.

the thing is I develop a Wi-Fi-based application. It would be nice, if I
could set some timeout value in the library for the period while I'm
debugging.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp-idf/issues/3126#issuecomment-469716358,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOQDgJB04FOsg_3Bxmxs7v3ejdSsjyBCks5vTolFgaJpZM4beatn.

Regards,
Vladimir
PS: when do you plans to open-source the library code?

@tsichevski  we have no plan to fully open-source WiFi library code in short-time.

Hi @tsichevski, the abort() happens in following scenario:

The WiFi power save is enabled (it's enabled by default)
The STA receives a beacon0 in time T0 (T0 here means the local time is T0), the timestamp in beacon0 is T0 (or about T0), the CPU is halted before the power save module processes beacon0.
The CPU is stopped for about 1 minutes, about 600 beacons are lost (the beacon interval of most of the AP is 100ms).
The CPU is resume, power save module processes beacon0. It detects that the next TBTT time is T0 + 1003 = T0 + 300 (let's assume that the DTIM counter is 3),  but the current time is T0 + 600100 = T0 + 60000ms. Power module thinks it's abnormal and abort the system.

Briefly speaking, the WiFi power save module will abort() the system if the CPU halts for more than about 30000ms.
How to avoid abort():

Stop power save by calling esp_wifi_set_ps(WIFI_PS_NONE) or

Don't halt CPU too long, e.g. just halts CPU for 10 seconds etc.

@tsichevski, we can remove this abort(). Before that, please do not halt CPU too long when WiFi is working.

Thank you!

How to avoid abort():

Stop power save by calling esp_wifi_set_ps(WIFI_PS_NONE) or
I will try this approach and report the results.

Don't halt CPU too long, e.g. just halts CPU for 10 seconds etc.
It is hard to achieve while I am debugging :(

Regards,
Vladimir

@tsichevski, we can remove this abort().

Please, do. The abort() is for detecting programming errors, and shall never be called in the production environment.

Stop power save by calling esp_wifi_set_ps(WIFI_PS_NONE)

Should I call this before or after the call to esp_wifi_set_config()?
Should I call this before or after the call to esp_wifi_start()?
Do you have any examples of using any esp_wifi_set_XXX() functions?
Can I use the examples/wifi/power_save/main/power_save.c as the esp_wifi_set_ps() usage example?

Regards,
Vladimir

Hi,

Stop power save by calling esp_wifi_set_ps(WIFI_PS_NONE)
Unfortunately, this does not help :( I still got abort() call from time to time.

Hi @tsichevski this issue has been fixed, please check.

@tsichevski Thanks for reporting. Feel free to reopen if the issue still happens. Thanks.


Create TypeScript composite action

Pull Request
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

#75  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JackPlowman and the rest of your teammates on  Graphite

Merge activity

Oct 21, 6:59 AM EDT: A user merged this pull request with Graphite.


Adapter Model: cached adapter.NewAspect()

Update adapter model so that results of adapter.NewAspect() can be independently cached by Mixer.
This ensures that every AspectWrapper does not open a new connection to the backend when they share the same AdapterConfig.
Investigate how

Mixer can directly call adapter.NewAspect()
AspectManager.Execute will get a reference to the object created in (1), which should be used as a pooled connection.

This will involve several internal API changes.
Details:
In mixer we have Adapters
  - name: default
    kind: metrics
    impl: statsd
    params:
        host: statsd.svc.cluster.local

And Aspects
  - kind: metrics
    params:
      metrics:
      - descriptor_name: request_count
        value: "1"
        labels:
          source: source.name | "unknown"
          target: target.name | "unknown"
          service: api.name | "unknown"
          method: api.method | "unknown"
          response_code: response.http.code | 200

At present NewAspect  --> aspect.Wrapper, this aspect Wrapper contains the  object produced by the Adapter with a call like this
asp, err := b.NewMetricsAspect(env, c.Builder.Params.(adapter.AspectConfig), defs)
Mixer cache manager caches aspectWrapper. Doing so means that multiple AspectConfigs sharing the same AdapterConfig will not share the  same object.

Affects #272 is handled.

No longer relevant with new adapter model.


nginx, postgres and redis have moved

Right now you get ERRORs in the logs if you run docker-stress because of this.

Thanks @andrewmichaelsmith


Allow opening console.vv in virt-viewer when clicking on VM console in Firefox

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1638217
Do not prompt for how to open the console.vv file if the user selects Remote or virt-viewer as a default application in Firefox.
Browsers act differently when setting the download attribute of the HTML <a> element. For FF, we don't want to set this attribute, but in Chrome/Chromium we do want - to achieve the expected behavior.
Some more info about the download attribute: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a
Before:

After:

Note:
Don't forget to set the Remote or virt-viewer as a default app in Firefox general preferences, to allow opening console.vv file automatically:

Looks good.  With this approach, we can't control the name of the file that is downloaded, the browser generates one based on the file type.  I don't think that is a problem.
If we do need to control the file name, I think the only workable way forward is to use the reflect servlet in webadmin.
I'm going to verify functionality on FF on windows and Safari on MacOS to see how the file gets downloaded.

For firefox, reading the docs on the download attribute is interesting (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a#Attributes):

Prompts the user to save the linked URL instead of navigating to it.

So one reading of that doc has FF doing exactly what it was asked to do before ... prompt the download!

LGTM


Serialization

I added save and load method to serialize/deserialize a dataset object in a folder.
It moves the arrow files there (or write them if the tables were in memory), and saves the pickle state in a json file state.json, except the info that are in a separate file dataset_info.json.
Example:
import nlp

squad = nlp.load_dataset("squad", split="train")
squad.save("tmp/squad")
squad = nlp.Dataset.load("tmp/squad")

ls tmp/squad
dataset_info.json squad-train.arrow state.json

cat tmp/squad/state.json
{
  "_data": null,
  "_data_files": [
    {
      "filename": "squad-train.arrow",
      "skip": 0,
      "take": 87599
    }
  ],
  "_fingerprint": "61f452797a686bc1",
  "_format_columns": null,
  "_format_kwargs": {},
  "_format_type": null,
  "_indexes": {},
  "_indices": null,
  "_indices_data_files": [],
  "_inplace_history": [
    {
      "transforms": []
    }
  ],
  "_output_all_columns": false,
  "_split": "train"
}

cat tmp/squad/dataset_info.json
{
  "builder_name": "squad",
  "citation": "@article{2016arXiv160605250R,\n       author = {{Rajpurkar}, Pranav and {Zhang}, Jian and {Lopyrev},\n                 Konstantin and {Liang}, Percy},\n        title = \"{SQuAD: 100,000+ Questions for Machine Comprehension of Text}\",\n      journal = {arXiv e-prints},\n         year = 2016,\n          eid = {arXiv:1606.05250},\n        pages = {arXiv:1606.05250},\narchivePrefix = {arXiv},\n       eprint = {1606.05250},\n}\n",
  "config_name": "plain_text",
  "dataset_size": 89789763,
  "description": "Stanford Question Answering Dataset (SQuAD) is a reading comprehension dataset, consisting of questions posed by crowdworkers on a set of Wikipedia articles, where the answer to every question is a segment of text, or span, from the corresponding reading passage, or the question might be unanswerable.\n",
  "download_checksums": {
    "https://rajpurkar.github.io/SQuAD-explorer/dataset/dev-v1.1.json": {
      "checksum": "95aa6a52d5d6a735563366753ca50492a658031da74f301ac5238b03966972c9",
      "num_bytes": 4854279
    },
    "https://rajpurkar.github.io/SQuAD-explorer/dataset/train-v1.1.json": {
      "checksum": "3527663986b8295af4f7fcdff1ba1ff3f72d07d61a20f487cb238a6ef92fd955",
      "num_bytes": 30288272
    }
  },
  "download_size": 35142551,
  "features": {
    "answers": {
      "_type": "Sequence",
      "feature": {
        "answer_start": {
          "_type": "Value",
          "dtype": "int32",
          "id": null
        },
        "text": {
          "_type": "Value",
          "dtype": "string",
          "id": null
        }
      },
      "id": null,
      "length": -1
    },
    "context": {
      "_type": "Value",
      "dtype": "string",
      "id": null
    },
    "id": {
      "_type": "Value",
      "dtype": "string",
      "id": null
    },
    "question": {
      "_type": "Value",
      "dtype": "string",
      "id": null
    },
    "title": {
      "_type": "Value",
      "dtype": "string",
      "id": null
    }
  },
  "homepage": "https://rajpurkar.github.io/SQuAD-explorer/",
  "license": "",
  "post_processed": {
    "features": null,
    "resources_checksums": {
      "train": {},
      "train[:10%]": {}
    }
  },
  "post_processing_size": 0,
  "size_in_bytes": 124932314,
  "splits": {
    "train": {
      "dataset_name": "squad",
      "name": "train",
      "num_bytes": 79317110,
      "num_examples": 87599
    },
    "validation": {
      "dataset_name": "squad",
      "name": "validation",
      "num_bytes": 10472653,
      "num_examples": 10570
    }
  },
  "supervised_keys": null,
  "version": {
    "description": "New split API (https://tensorflow.org/datasets/splits)",
    "major": 1,
    "minor": 0,
    "nlp_version_to_prepare": null,
    "patch": 0,
    "version_str": "1.0.0"
  }
}

I've added save/load for dataset dicts.
I agree that in the future we should also have a way to save indexes too, and also the in-place history of transforms.
Also I understand that it would be cool to have the load function directly at the root of the library, but I'm not sure this should be inside load_dataset that loads dataset scripts and data from the dataset repository. Maybe something like load_from_disk ?

Yes load_from_disk and save_to_disk  could work as well.

I renamed save/load to save_to_dick/load_from_disk, and I added nlp.load_from_disk
nlp.load_from_disk can load either a Dataset or a DatasetDict.

Awesome! Let's add them to the doc and we're good to go!


Help???

I copied the CSS over the appearance tab nothing happened does it take time to apply the changes?

Yes, there's like 10-15 minutes cache in Mastodon.


[Enhance] Show instance statistics before and after through pipeline

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation

Show all statistics of the dataset: the length of the dataset and the number of instances per category in the dataset.
In order to debug in KITTI training, show the variance of the number of instances before and after through the pipeline by modifying the config. Here is an example:

train_dataloader = dict(
    ...,
    dataset=dict(
        type='KittiDataset',
        ...,
        show_ins_num_var=True))

The output infomation:
The number of instances per category after and before through pipeline:
car             4/4
Cyclist         5/6
Pedestrian      10/13

Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

Can add an example print in the PR message?

Suggest rebasing the code to the most recent dev.


Remove "bin/" from search ignore and just rank low unless typed

Fixes https://github.com/sourcegraph/cody/issues/2401 by removing bin/ from the ignore list and instead just penalizing it unless you have typed bin in the query specifically.
https://github.com/sourcegraph/cody/assets/1078012/2ad0ef3b-b2f9-45bd-bfc0-54fd77811511
Test plan

Run pnpm test:unit and ensure the new tests pass
Open a project with a file in a folder named bin and ensure the file shows up in search results, near the bottom unless you've explicitly typed "bin"

@abeatrix I've pushed a test and updated the change log, let me know if this still looks good to you.


Constrained Naked Functions

Rendered
Summary
This is a quick attempt at improving the usefulness of naked functions by constraining their use.
Tasks

[x] Resolve disabling unused arguments lint
[ ] Finalize the RFC
[ ] Implement:

[x] warn if calling convention is unspecified - WARN
[x] warn if calling convention is extern "Rust" - WARN
[ ] warn if arguments contain FFI-unsafe arguments
[ ] warn if return types contain FFI-unsafe arguments
[ ] error if #[inline(never)] is missing - PR
[ ] error if #[inline] or #[inline(always)] is present - PR
[x] error if wrong number of asm blocks - WARN
[x] error if asm contains operands other than const or sym - WARN
[x] error if asm is missing options(noreturn) - WARN
[x] error if options except att_syntax are used - WARN and WARN
[ ] don't lint for unused function arguments
[x] don't emit instructions in the function body before asm!()

CC
https://github.com/rust-lang/rust/issues/32408
https://github.com/rust-lang/rust/issues/72016
https://github.com/rust-lang/rfcs/pull/2774
@amluto @comex @programmerjake @Amanieu @haraldh @bjorn3 @joshtriplett

I’m trying to understand how one might create a naked extern “C” in a way that is correct and also accomplishes something useful.

https://github.com/enarx/frenetic/blob/master/src/arch/x86_64/sysv.rs#L30-L50
While it might be true that the uses which strictly speaking require naked are not super common, the number of cases where naked can either (1) improve efficiency or (2) offload concerns like stack alignment to the compiler is much larger.
One important point is that I'm working on this because I need naked and I want to ensure that there is a clear path to stable for the benefit of my users. I deferred the question of extern "custom" because I don't want the perfect to become the enemy of the good.

#[used] only applies to globals such as functions and statics. In the case of naked functions all we need to do is suppress the unused argument lint.

Resolved in the latest commit.

I’m trying to understand how one might create a naked extern “C” in a way that is correct and also accomplishes something useful.

#[naked]
pub unsafe fn switch_to_usermode(instruction_pointer: u64, stack_pointer: u64) -> ! {
    asm!("
…
        iretq
          ",
    in(reg) sp,
    in(reg) ip,
    options(noreturn, nomem)
    );
}

I would like to use it for the syscall kernel function.
https://github.com/enarx/enarx/blob/master/enarx-keep/shims/shim-sev/src/syscall.rs#L26-L86

In my case, I'd like to use it for a low-level context switching function (saving and restoring register states) in my kernel. My current implementation doesn't use naked function (because naked functions have historically been too buggy to use), and takes a slight performance penalty as a result (the compiler inserts a useless prelude). I have no need for extern "custom", as my function is meant to be C ABI compliant.
It's generally possible to replace naked functions with an extern block and a global_asm!()/external asm file, but that comes at the cost of creating a global symbol, potentially leading to symbol conflicts if the same name is used twice.

I’m trying to understand how one might create a naked extern “C” in a way that is correct and also accomplishes something useful.

https://github.com/enarx/frenetic/blob/master/src/arch/x86_64/sysv.rs#L30-L50
While it might be true that the uses which strictly speaking require naked are not super common, the number of cases where naked can either (1) improve efficiency or (2) offload concerns like stack alignment to the compiler is much larger.

I can see how this type of context switching would make sense in a naked, extern "C" context, but I have a question about your implementation.  As far as I can see, your save function can return more than once.  This seems like something that rustc would not expect, and I'm surprised to see that functionality show up in a non-unsafe function.

I would like to use it for the syscall kernel function.
https://github.com/enarx/enarx/blob/master/enarx-keep/shims/shim-sev/src/syscall.rs#L26-L86

This would need extern "custom" to really be correct, right?
Also, I find it surprising that you are swapgs-ing back to user GSBASE before running Rust code.  You also seem to be leaking whatever Rust sticks in rcx and r11 to userspace.  And you're returning with iret, which is painfully slow but at least avoids the return-to-noncanonical-address SYSRET issue on Intel CPUs.

As far as I can see, your save function can return more than once. This seems like something that rustc would not expect, and I'm surprised to see that functionality show up in a non-unsafe function.

load is unsafe though, and is ultimately what triggers the "second return". But, yes, this pattern is unlikely to be sound, just like how setjmp/longjmp is currently unsound

I would like to use it for the syscall kernel function.
https://github.com/enarx/enarx/blob/master/enarx-keep/shims/shim-sev/src/syscall.rs#L26-L86

This would need extern "custom" to really be correct, right?

Yes, I guess so.

Also, I find it surprising that you are swapgs-ing back to user GSBASE before running Rust code.

yeah, maybe I'll change that.

You also seem to be leaking whatever Rust sticks in rcx and r11 to userspace.

Thanks for the heads-up.

And you're returning with iret, which is painfully slow but at least avoids the return-to-noncanonical-address SYSRET issue on Intel CPUs.

Yeah, that was the intention. When we will do performance optimizations, we will turn on sysret for AMD.

Could we extend this to allow multiple asm! blocks, to support #[cfg(...)]? Right now I have code that looks like:
#[naked]
pub unsafe extern "C" fn start64() {
    #[cfg(feature = "init_data")]
    asm!(
        "movabs  rsi, offset __init_data",
        "movabs  rdi, offset __start_data",
        "movabs  rcx, offset __stop_data",
        "sub     rcx, rdi",
        "rep movsb [rdi], [rsi]",
    );
    #[cfg(feature = "zero_bss")]
    asm!(
        "movabs  rdi, offset __start_bss",
        "movabs  rcx, offset __stop_bss",
        "sub     rcx, rdi",
        "xor     eax, eax",
        "rep stosb [rdi], al",
    );
    asm!(
        "movabs  rsp, offset __rust_stack_top",
        "movabs  rax, offset __rust_start",
        "jmp     rax",
        options(noreturn)
    )
}

which seems to meet all the requirements we want to have (only asm!, ends with noreturn). But this code will currently trigger the lint despite being "safe".

Could we extend this to allow multiple asm! blocks, to support #[cfg(...)]? Right now I have code that looks like:
#[naked]
pub unsafe extern "C" fn start64() {
    #[cfg(feature = "init_data")]
    asm!(
        "movabs  rsi, offset __init_data",
        "movabs  rdi, offset __start_data",
        "movabs  rcx, offset __stop_data",
        "sub     rcx, rdi",
        "rep movsb [rdi], [rsi]",
    );
    #[cfg(feature = "zero_bss")]
    asm!(
        "movabs  rdi, offset __start_bss",
        "movabs  rcx, offset __stop_bss",
        "sub     rcx, rdi",
        "xor     eax, eax",
        "rep stosb [rdi], al",
    );
    asm!(
        "movabs  rsp, offset __rust_stack_top",
        "movabs  rax, offset __rust_start",
        "jmp     rax",
        options(noreturn)
    )
}

which seems to meet all the requirements we want to have (only asm!, ends with noreturn). But this code will currently trigger the lint despite being "safe".

@josephlr this code block doesn't meet this requirement:

must have a body which contains only a single asm!() statement

As if one of the feature is enabled, there are multiple asm statements.

@josephlr this code block doesn't meet this requirement:

must have a body which contains only a single asm!() statement

As if one of the feature is enabled, there are multiple asm statements.

As if one of the feature is enabled, there are multiple asm statements.

Exactly. That's why I was wondering about the feasibility of (now or as a future extension) allowing multiple asm! blocks in a naked function.
Otherwise, you would need weird macros to get the above code to compile without warnings.

As if one of the feature is enabled, there are multiple asm statements.

Exactly. That's why I was wondering about the feasibility of (now or as a future extension) allowing multiple asm! blocks in a naked function.
Otherwise, you would need weird macros to get the above code to compile without warnings.

I've been using macro_rules which expand to string literals in my code. For example you can do this:
#[cfg(feature = "zero_bss")]
macro_rules! zero_bss {
    () => {
        r#"
            movabs  rdi, offset __start_bss
            movabs  rcx, offset __stop_bss
            sub     rcx, rdi
            xor     eax, eax
            rep stosb [rdi], al
        "#
    }
}
#[cfg(not(feature = "zero_bss"))]
macro_rules! zero_bss {
    () => { "" }
}

asm!(
    zero_bss!(),
    "movabs  rsp, offset __rust_stack_top",
    "movabs  rax, offset __rust_start",
    "jmp     rax",
    options(noreturn)
);

I've been using macro_rules which expand to string literals in my code. For example you can do this:
#[cfg(feature = "zero_bss")]
macro_rules! zero_bss {
    () => {
        r#"
            movabs  rdi, offset __start_bss
            movabs  rcx, offset __stop_bss
            sub     rcx, rdi
            xor     eax, eax
            rep stosb [rdi], al
        "#
    }
}
#[cfg(not(feature = "zero_bss"))]
macro_rules! zero_bss {
    () => { "" }
}

asm!(
    zero_bss!(),
    "movabs  rsp, offset __rust_stack_top",
    "movabs  rax, offset __rust_start",
    "jmp     rax",
    options(noreturn)
);

I'd like to propose one small modification to this RFC for consideration. Currently, the RFC defines that the constrained naked functions "must not specify the #[inline] or #[inline(always)] attribute" and in turn the complier will emit the equivalent of #[inline(never)]. This means, however, that we can never change this behavior in the future.
I'd like to change the wording instead to define that constrained naked functions "must additionally define the #[inline(never)] attribute. This has several benefits. First, the complier behavior is explicit rather than implicit. Second, if we desire to change this behavior in the future we can loosen the requirement. Third, the code becomes self-documenting.
Thoughts?

@npmccallum That sounds reasonable. In general, I think "requires specifying" seems preferable to "implies" unless we're absolutely certain that it'll never ever make sense to do otherwise.
And I can imagine wanting to inline a naked function that follows the calling convention.

I just did an inventory of the current state of the naked functions. Thanks to @Amanieu we're actually closing in on the finish line for implementing this RFC!
The remaining items basically fall into two categories:

Implementing new lints
Addressing ERROR lints that are currently WARNINGs.

Does anyone have opinions on the second item? Should we change the lints to errors? Or should we update the RFC to match the existing behavior?

I don't think that allowing naked functions to be inlined ever makes sense. If you want your code to be inlinable, just use asm! in a non-naked function. I would prefer if #[naked] continued to implicitly add #[inline(never)] and we just had a lint to deny all inline attributes on naked functions.

I don't think that allowing naked functions to be inlined ever makes sense.

I agree that inlined naked functions are probably a rare use-case. However, I really like @npmccallum's approach for requiring the attribute, as it makes much clearer to a reader of the code what is going on.

FWIW Clang will implicitly add noinline when a function is declared with __attribute__((naked)).

@Amanieu Clang's precedent notwithstanding, I still prefer my proposal because the implications of naked functions are more obvious to:

developers writing the code.
auditors or debuggers reading the code.

The cost for this is a slight bit of increased verbosity. OTOH, naked functions are:

rare
likely to be a source of difficult to debug issues

Therefore, I prefer the clarity over the increased verbosity.

@Amanieu @josephlr @joshtriplett FYI, I have an open PR implementing lints according to my proposal. This puts us within striking distance of fully implementing this RFC. Please review the PR if you are inclined.

The fact that LLVM requires a noinline attribute on naked functions is an implementation detail because LLVM's inliner doesn't recognize the naked attribute. I would rather have a lint that entirely disallows all #[inline(*)] attributes which simply states that naked functions can't be inlined.

@Amanieu I have updated my PR to match your preference. The PR now explicitly rejects all usage of the inlining attribute.

Status update: there are now PRs for all open tasks in this RFC.
It looks to me like the only things we need are:

Some (but not all) of our warning lints need to be converted to errors.
We need a final revision of the RFC fixing some minor wording errors.
Final comment period?

This seems reasonable, it doesn't have any apparent outstanding issues, and it's actually implemented in nightly.
Thus, I would propose we accept this RFC
@rfcbot merge

Sorry, I'm a little late to the party, but there's a concern that I just raised in the Tracking issue: currently, naked functions have an extra ud2 instruction inserted after the asm block due by the TrapUnreachable LLVM option. According to Amanieu:

I did a bit of searching in LLVM's code and it seems that TrapUnreachable is actually required on some targets because the unwinder requires that call return addresses point within the bounds of the caller function. This requires adding an ud2 (or equivalent) at the end of a function to ensure the byte after a noreturn call is still inside the function.
In conclusion I don't think that we can actually guarantee the lack of any instructions in the epilogue with the current implementation in LLVM.

See this comment.
To workaround this, we may want to ensure naked functions are additionally marked nounwind. I don't think I've ever seen a naked function that called an unwinding function, and if it did, I'd expect it to deal with the unwinding itself, through assembler directives (that's rather the point of naked functions - they are supposed to deal with everything).
I'm not sure if this would be enough to allow naked functions to guarantee to only generated the exact asm block (and have no suffix instructions) though.

If @roblabla 's suggestion actually allows us to discard the trailing ud2, I'm in favor of the suggestion and would be willing to delay the RFC to implement the requirement if necessary. However, it may not be necessary to block the RFC on this, pending the input of others.
I propose that we accept the RFC as it stands and explore this option as an additional requirement after the RFC lands. If we do this, we could simply enforce this with a test. This would delay the stabilization of naked functions, but not necessarily this RFC.
Thoughts?

Huzzah! The @rust-lang/lang team has decided to accept this RFC.
Tracking issue: https://github.com/rust-lang/rust/issues/90957
If you'd like to keep following the development of this feature, please subscribe to that issue, thanks! :)


Repository Configuration: Build Scripts

Build Scripts
Possibly migrate to Gradle

Gradle build scripts were created to compiling, packaging, and analyzing the projects in the repo: https://github.com/opendcs/rest_api/pull/46 and replaced the original maven and ant build scripts.


Make switching units on tract sizers preserve the relative size of the tract

Right now if you switch a tract from something like 1fr to px you get a super disorienting 1px wide tract. What should happen is that the tract stays the same size and just sets the pixel value to whatever it needs to be to be the same size as it was before

Fixed by #110


Option in parameter or configuration to set value in metadata > authors in SBOM (CycloneDX)

What would you like to be added:
Parameter or configuration to add and set metadata > authors > name / email in SBOM in CycloneDX format.
I was actually looking for this ability in the parameter and configuration options but did not find any.  Please let me know if I am wrong.
Why is this needed:
To automatically include required metadata in CycloneDX format when generating SBOM.
Additional context:

Based on CycloneDX schema, it should look like this:
"metadata": {
"authors":{
"name": "Name"
"email"<EMAIL_ADDRESS>}
}

Hi @haiazuki, thanks for the report! Based on our read of the CycloneDX specification, those fields aren't required, so we don't include them. It's definitely something we will consider adding. In the meantime, a workaround might be to add a step after your Syft run to modify the output using jq or similar tool and add those fields to the document.

We have had some other requests for ways to add additional information to the SBOM documents. It would be nice to spend some time designing a generalized approach for this.

Hi @tgerla ,
Thank you for considering this enhancement.  Yes, for now we are manually adding the metadata/authors element on our sboms as required by our organization.


TS-534: Switch to localtunnel from tunnelmole.

I'm not exactly sure why tunnelmole doesn't work anymore, but I couldn't get it to run on the Orange Pi in my HIL setup, and there were literally no logs that I could use to debug. This commit swaps out tunnelmole for localtunnel. I tested it, and it works.

Ugh, don't review this yet. There are some kinks that need to be worked out.

Doesn't imply all tests should pass now?


Please add Retail Price to the GetReceivableCharges API

We need to pull the Retail Price, not just the Cost Price so we can sync prices to our billing system.
Please add this to the GetReceivableCharges API.
Thank you,

Hi Joel! We have just added this information inside the subscription! You may take a look at the pre-release changelog v1.0-beta06 here
Here is also the direct link to the documentation for the endpoint that now exposes the subscription fees.
I will close this issue, but don't hesitate to ask here or create a support request (the link is at the bottom of the main page of the documentation) if you have any questions.
Have a great day 🌞

Awesome thank you, this will really help us with our billing integration!
Sent from my iPhone
On Oct 16, 2023, at 6:29 AM, Raphaël Cayouette @.***> wrote:

Hi Joel! We have just added this information inside the subscription! You may take a look at the pre-release changelog v1.0-beta06 herehttps://github.com/sherweb/Public-Apis/releases/tag/v1.0-beta06
Here is also the direct linkhttps://developers.sherweb.com/api-details#api=service-provider-api&operation=GetCustomerSubscriptions to the documentation for the endpoint that now exposes the subscription fees.
I will close this issue, but don't hesitate to ask here or create a support request (the link is at the bottom of the main page of the documentation) if you have any questions.
Have a great day 🌞
—
Reply to this email directly, view it on GitHubhttps://github.com/sherweb/Public-Apis/issues/32#issuecomment-1764489783, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH5HFKQOFZS3VOCG2M6NFCLX7UZFBAVCNFSM6AAAAAAW5YZVHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRUGQ4DSNZYGM.
You are receiving this because you authored the thread.Message ID: @.***>


[Crash] React Native crashes when rendering some images

Sometimes I'm getting this error when trying to upload an image, and the app crashes. It only happens when I'm not connected to debugger (both for dev and production builds).
10-15 17:58:50.124  15806-16396/io.scrollback.neighborhoods A/fbassert﹕ Assert (/Users/mkonicek/code/react-native/ReactAndroid/src/main/jni//first-party/jni/LocalString.cpp:36): 4 byte utf-8 encodings only valid for up to 21 bits
--------- beginning of crash
10-15 17:58:50.124  15806-16396/io.scrollback.neighborhoods A/libc﹕ Fatal signal 11 (SIGSEGV), code 1, fault addr 0xdeadb00c in tid 16396 (b/1/480x960.jpg)
cc @mkonicek

If you have those images, can you include them in a zip somewhere? Zip as a container so the images aren't altered so someone can test them as/is.

@jaygarcia Sorry for late reply. Was sick. Here is a link - https://hey-neighbor.s3.amazonaws.com/generated/content/satya164/3e8c1c34c0d9c771e159c993b43878e6/1/480x960.jpg

Ok, so this doesn't happen when rendering the image, but happens after uploading the image is complete. I'm not familiar with the C++ code, so cannot debug what's happening.

I'm gonna close this and file a new issue since it seems to be a string issue rather than image


Failed to assemble APK     program not found

cargo android run
[2023-05-05T01:14:58Z DEBUG tauri_mobile::util::cli::interface] raw args: [
        "C:\\Users\\dhorner\\.cargo\\bin\\cargo-android.exe",
        "run",
    ]
[2023-05-05T01:14:58Z INFO  tauri_mobile::config::raw] looking for config file at "\\\\?\\C:\\working\\whosball\\mobile.toml"
[2023-05-05T01:14:58Z INFO  tauri_mobile::config::raw] found config file at "\\\\?\\C:\\working\\whosball\\mobile.toml"
[2023-05-05T01:14:58Z INFO  tauri_mobile::templating] checking for template pack "wry" at "C:\\Users\\dhorner\\.cargo\\.tauri-mobile\\templates/apps\\wry.toml"
[2023-05-05T01:14:58Z INFO  tauri_mobile::templating] checking for template pack "wry" at "C:\\Users\\dhorner\\.cargo\\.tauri-mobile\\templates/apps\\wry"
[2023-05-05T01:14:58Z INFO  tauri_mobile::templating] found template pack "wry" at "C:\\Users\\dhorner\\.cargo\\.tauri-mobile\\templates/apps\\wry"
 (moto g stylus 5G (2022)) with target "aarch64-linux-android"
[2023-05-05T01:14:59Z INFO  tauri_mobile::util::cli::interface] exiting with RunFailed(
        ApkError(
            AssembleFailed(
                Error {
                    kind: NotFound,
                    message: "program not found",
                },
            ),
        ),
    )
error: Failed to assemble APK
    program not found

rustup shows the toolchain installed.
installed targets for active toolchain
--------------------------------------

aarch64-linux-android
armv7-linux-androideabi
i686-linux-android
x86_64-linux-android
x86_64-pc-windows-msvc

I know it's not flutter development but doctor has useful info.
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.3.10, on Microsoft Windows [Version 10.0.19044.2846], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.4.3)
[√] Android Studio (version 2021.3)
[√] VS Code, 64-bit edition (version 1.74.2)
[√] Connected device (4 available)
[√] HTTP Host Availability

• No issues found!

any help would be greatly appreciated.

Could you check if <project-dir>/gen/android/<app-name>/gradlew.cmd exists?

Not sure what caused it to start working (i updated my jdk) but it is there now.
I am linking and getting the following:
= note: ld: error: unable to find library -lgcc

I'm running on windows.

I tried setting this up on another machine and found the same issue:
error: Failed to assemble APK
    program not found

I ran cargo mobile init a second time.  It then works.
On this machine (another windows machine), I had no issue linking and actually ran the app on my phone.
nice work!  thanks.

Not sure what caused it to start working (i updated my jdk) but it is there now. I am linking and getting the following:
= note: ld: error: unable to find library -lgcc

I'm running on windows.

You're probably using an older NDK or rustc version

I ran cargo mobile init a second time. It then works.

yeah you need to call cargo mobile init at least one time on the project (so it generates gen folder) before you can use other commands

I also encountered gcc linking erro. It's because of we removed the linking workaround in https://github.com/tauri-apps/tauri-mobile/pull/125 since newer version of rustic can link NDK's GCC correctly now.
If you also see error like this, try rustup update to newer version of rustc.


Cant jit the function using soft_sort.quantile() and jnp.where()

Discussed in https://github.com/ott-jax/ott/discussions/406

Originally posted by yyang97 August  1, 2023
import jax
import ott
import jax
import jax.random
import jax.numpy as jnp

N = 200
x = jax.random.normal(key = jax.random.PRNGKey(0), shape=(N,))
a = 0.5
quant = jnp.array([.2,.8])
def quantile_cal(x,quant,a):
   return( jax.vmap(lambda quant: ott.tools.soft_sort.quantile(jnp.where(x < a, x, a),
                                                                    q=quant))(quant))

quantile_cal(x,quant,a)

# Array([[-0.81860286],
#        [ 0.49906105]], dtype=float32)

If I don't jit the function, it looks fine.
But if I jit it, it fails:
@jax.jit
def quantile_cal(x,quant,a):
   return( jax.vmap(lambda quant: ott.tools.soft_sort.quantile(jnp.where(x < a, x, a),
                                                                    q=quant))(quant))

quantile_cal(x,quant,a)

# NonConcreteBooleanIndexError: Array boolean indices must be concrete; got ShapedArray(bool[3])

# See https://jax.readthedocs.io/en/latest/errors.html#jax.errors.NonConcreteBooleanIndexError

However, I don't have any array boolean things...
Further, it seems that ott package about the quantile function is updated... The same code works before...

Hi @yyang97, will be fixed in #408, just tests need to pass; after you should install ott-jax from main.
Furthermore, after #382 introduced by @marcocuturi , you don't need to (and shouldn't) vmap over the quantiles; the below code will just work:
import jax
import jax.numpy as jnp
from ott.tools import soft_sort

N = 2000
x = jax.random.normal(key = jax.random.PRNGKey(0), shape=(N,))
a = 0.5
quant = jnp.array([.2,.8])

@jax.jit
def quantile_cal(x, quant, a):
   return soft_sort.quantile(jnp.where(x < a, x, a), q=quant)

quantile_cal(x, quant, a)

Thank you very much!!! This function is just very important to me :)

I also noticed that the vmapped version is computing wrong values, at the moment, am not sure why... just running:
import numpy as np
np.quantile(np.where(x < a, x, a), q=quant)

# array([-0.87661878,  0.5       ])

whereas the vmapped function returned
# Array([[-0.81860286],
#        [ 0.49906105]], dtype=float32)

When passing q=jnp.array([.2,.8]), I get Array([-0.87872696,  0.4870388 ], dtype=float32), which seems correct to the actual "hard" quantiles. Will investigate in another issue.


CMakeLists.txt not found and hence unable to flash the code

1- The following is the error when I try to flash the board with MinGW64.
C:\FreeRTOS\vendors\espressif\esp-idf\tools>idf.py flash -B C:\FreeRTOS\Builds
Note: You are using Python 3.8.2. Python 3 support is new, please report any problems you encounter. Search for 'Setting the Python Interpreter' in the ESP-IDF docs if you want to use Python 2.7.
Setting IDF_PATH environment variable: C:\FreeRTOS\vendors\espressif\esp-idf
Checking Python dependencies...
Python requirements from C:\FreeRTOS\vendors\espressif\esp-idf\requirements.txt are satisfied.
CMakeLists.txt not found in project directory C:\FreeRTOS\vendors\espressif\esp-idf\tools
2- I am getting a CMakeLists.txt error because apparently there is no CMakeLists.txt in the folder C:\FreeRTOS\vendors\espressif\esp-idf\tools
I have downloaded the ESP32 DevKit C from the official AWS SDK resources. The CMakeLists.txt is available in FreeRTOS folder but it is not present in the ...\esp-idf\tools folder. Due to this reason, I am not able to flash the code.
3- However the ESP flashing part: ./vendors/espressif/esp-idf/tools/idf.py erase_flash -B build-directory works and the ESP is able to reset successfully.
4- Also the monitoring part: ./vendors/espressif/esp-idf/tools/idf.py monitor -p /dev/ttyUSB1 -B build-directory does work partially too, It does not ask for a CMakeLists.
A Solution to this issue would be highly appreciated.

Hi @NikhilPatale,
From where did you ran the idf.py flash command? The IDF tool has to be run from the your project root. (As said here https://docs.aws.amazon.com/freertos/latest/userguide/getting_started_espressif.html)
There was a simialr issue asked under esp-idf repo: https://github.com/espressif/esp-idf/issues/6007
Can you verify and see if that works for you? If you still having the issue, please let me know.
Thank you!
Ming

Hi @mingyue86010
The path which I have used is C:\FreeRTOS\vendors\espressif\esp-idf\tools
In this case, I think this itself is the root folder right? Since idf.py is present in the tools directory. Your help on this issue will be highly appreciated since I am stuck on this error for weeks and even though the error seems dumb, I am unable to find a solution for this

Hi @NikhilPatale,
The root shoud be the FreeRTOS proejct. I checked on my local repo it works. It I run the idf.py I can see the same error you saw.
So please go back to your FreeRTOS proejct folder by
cd C:\FreeRTOS
Then run:
./vendors/espressif/esp-idf/tools/idf.py flash -B Builds
For more instruction, you can find on https://docs.aws.amazon.com/freertos/latest/userguide/getting_started_espressif.html
Thanks,
Ming

Hi @mingyue86010
Thank you for your help. However, I seem to be stuck on this error when I try to run it. Could you kindly let me know why am I not being able to execute the code? pasting the same from my Run terminal.
C:>cd freertos
C:\FreeRTOS>./vendors/espressif/esp-idf/tools/idf.py flash -B C:/Builds
'.' is not recognized as an internal or external command,
operable program or batch file.
Thanks,
Nikhil

Hi @mingyue86010,
Thanks for the solution! I am able to flash it successfully. Thank you for your help!!


Reverse a Linked List in groups of given size

Is your feature request related to a problem? Please describe.
Reverse a Linked List in groups of given size
Describe the solution you'd like
Implementation of reversing every k nodes in linked list using C++
Example:
Input: 1->2->3->4->5->6->7->8->NULL, K = 3
Output: 3->2->1->6->5->4->8->7->NULL
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

@Amit366 @Ayush7614 @ravikr126 @Saurabh4626 , Kindly assign me this issue using C++ under LGMSOC

Already raised issue @Palak2801


Reflect method aliases in "constructors"?

Unlike #42 this not supposed to be a major change, but as noted there the Either monad does somewhat double as a Result monad. So given that right? and left? are already aliased to success? and failure?, would it be reasonable to also alias the "constructors" defined in the mixin?
Essentially just this:
index 083c4f0..f48b32e 100644
--- a/lib/dry/monads/either.rb
+++ b/lib/dry/monads/either.rb
@@ -210,2 +210,3 @@ module Dry
         end
+        alias Success Right

@@ -216,2 +217,3 @@ module Dry
         end
+        alias Failure Left
       end

For reference, Rust originally had both, but then went with Result for the standard library. Relevant quote:

[W]e actually used to have both Either and Result, until one day we went through and realized that no code in existence was using Either and decided to go all-in on Result instead.

On a side-note, I find it would make dry-transaction code much more readable to the average user™:
def validate(input)
  if input[:email].nil?
    Failure(:not_valid)
  else
    Success(input)
  end
end

I think this is a great idea. It'd make teaching these gems more straightforward, especially since by our conventions so far, we're mostly using Either as a result object.
I'd be happy for both the Right/Success and Left/Failure type "constructors" to remain here in the long term, since this is a general purpose library after all, but it might be worth having a discussion about which of these name pairs becomes the "primary" name, or whether we can create full class aliases for each of these (i.e. it might be disconcerting for a user creating Success("hi") to see it become Right("hi") immediately afterwards).

@flash-gordon what do you think?

@timriley @citizen428 yeah, this is a good suggestion, although you @timriley made an excellent point about unexpected naming confusion, I’d try to have full class aliases but this will require setting up a custom equalizer. Overall, having weird names from the “world of monads” scares people even the concepts are not that complex. I think, that’s one of the reasons Rust came up with “normal” names in the end and I guess we should make a similar move so I’ll give it a shot

@flash-gordon I have no problem with monadic vocabulary, I just think that so far Either has mostly been used in the context of what Rust calls Result (so does MonadEx for Elixir btw) and making things accessible/less "scary" seems like a good move.
I'll play around with over the weekend.

OK, we renamed Either to Result in 0.4.0


Wrong/Outdated instructions

"In Fusion360, go to MODIFY ..."  there is no such thing.
The correct instructions are:

Click the UTILITIES Tab,
then the "UTILITY" drop-down menu,
then the first option "Manage Materials",
then click to expand the bottom-left icon of a folder
and pick the "Open Existing Library" option.

Thank you for making these up-to-date instructions! I will update the current usage instructions in the README.


patch groovy 3.0.17

Description
quick security patch for groovy
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[X] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[X] I signed an Apache CLA.
[X] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5418 (567e931) into master (37605b4) will decrease coverage by 72.15%.
The diff coverage is n/a.

:exclamation: Current head 567e931 differs from pull request most recent head 3fc4b1d. Consider uploading reports for the commit 3fc4b1d to get more accurate results

@@            Coverage Diff             @@
##           master   #5418       +/-   ##
==========================================
- Coverage   76.65%   4.51%   -72.15%
==========================================
  Files         241     241
  Lines       14630   14630
  Branches      616     616
==========================================
- Hits        11215     660    -10555
- Misses       3415   13970    +10555

see 205 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Release Pytorch Lightning as a conda package

🚀 Feature

Please make the pytorch-lightning package available from the conda package manager. This would probably be done through conda-forge: conda install pytorch-lightning -c conda-forge
Motivation

The default way of installing Pytorch is via their conda channel, so (probably) most users of Lightning already use the conda package manager.
Conda packages provide automatic updates (conda update --all) and better package dependencies management, which could make the user's life a lot easier.
Pitch

I'd like to install pytorch-lightning and its dependencies via this command:
conda install pytorch-lightning -c conda-forge

Ideally, this wouldn't force reinstall the pytorch package from its channel to conda-forge.
Alternatives

My personal way of installing pytorch-lightning is running:
pip install --no-deps pytorch-lightning test-tube
And then conda install the missing dependencies that show up on pip check.
On version 0.5.3.x this was mandatory, since the required pytorch-lightning packages would mess up numpy and other conda packages.

@dangpzanco agreed. Is this something you can take care of? at least the config. then we can set up what needs to happen for conda release.

@williamFalcon I'm not sure if I'll be able to take care of this (I don't have much spare time). But if nothing comes up in a couple of weeks, I'll probably look into it.

@Borda want to look at this for this release?

Depends how much time will leave, I don't have any experience with conda packaging yet...

According to @hadim it is not possible to release as conda package:
https://github.com/PyTorchLightning/pytorch-lightning/issues/199#issuecomment-588269459

Apparently it's possible to add pytorch packages to conda-forge following this example: https://github.com/conda-forge/staged-recipes/pull/11762
The pytorch version available on conda-forge is 1.1 CPU so tests won't pass but tests are already performed on this repo anyway.

See https://github.com/conda-forge/staged-recipes/pull/11778

would be great to have it in conda! Thanks hadim

You can close here.

we should probably add instructions in the readme.
And it does not seem to work on windows but that could be because... Windows.


Update electrum to 3.0.1

After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

- [ ] `brew cask audit --download {{cask_file}}` is error-free.
- [ ] `brew cask style --fix {{cask_file}}` reports no offenses.

These steps are completed automatically by cask-repair

Thanks for the PR!


DMP is more Scrutinous, removes Pyo3

Removes python requirement and python code entirely.
Adjusts the similarity threshold of dmp for patching so that it does not override changes between patches and fails more often.

This closes #6 and closes #7 although it is much more aggressive on what it determines to be an actual patching conflict. This sensitivity will need to be reduced without causing obviously wrong results, later.


Percentage value of number of death and recovered.

Is your feature request related to a problem? Please describe.
As a user I would like to know what percentage of people have recovered and percentage of the total cases led to death.
Describe the solution you'd like
A very simple percentage value.
For example RecoveryRatePercentage = TotalRecovery * 100/TotalCases
TotalDeathsPercentage = TotalDeaths * 100/TotalCaese
Describe alternatives you've considered
NA
Additional context
It is good metric to understand how are we processing with the COVID-19 fight.
People are able to relate with percentage more easily than the big numbers.
We can also add a graph of percentage value, how it changed with time.

I'm new to react and a first-time contributor. But I would like to work on this.
/claim

Can I work on this?

Will we be working on this?

Hi team, tried to take it up.

Compiled and running as expected on my local machine. thanks 😄

What should be the next step here ?

@shuklaayush , can you assign this issue to someone?(I would be happy to be help), there was already a PR created #2160 , or reopen the old the PR?
I think, this will be a useful feature.


Union webpack.js.org repo and voting-app repo

Feature to document
Move voting-app repo https://github.com/webpack-contrib/voting-app inside that repo
Author(s)
Don't known
Additional information
Pros:

Easy to maintenance (all in one place)
Easy to update
Less repositories
When a feature was released it will easy to update status for voting app

[ ] I am willing to work on this issue and submit a pull request.

inside that repo

Which repo?

inside that repo

Which repo?

I guess he meant in this repo

If that is the case, why?
This repo should only host documentation, any external app, plugin, etc should be a dependency.
Initial discussion: https://github.com/webpack/webpack.js.org/issues/301
Initial implementation: https://github.com/webpack/webpack.js.org/pull/1717

@montogeek
Pros:

Easy to maintenance (all in one place)
Easy to update
Less repositories
When a feature was released it will easy to update status for voting app
Sync with partial implemented features

It is not high priority

I'll pick this up 👍

I dont think this is necessary, its already easy to update: #3617

@EugeneHlushko we want to move all repos from webpack-contrib in long term, some of repos will be migrated to monorepo webpack/webpack-official, some of repos will be deprecated, some of them move out from official support, so we start it process, we need move voiting-app to here or maybe webpack/voting-app, I prefer first

we dont want any technical complexity in documentation repository, all modules that are abstractable should live in separate repositories. cc @skipjack

What about move it to webpack/voting-app?

Definitely, that it better :)
On Tue 17. Mar 2020 at 14:33, Evilebot Tnawi<EMAIL_ADDRESS>wrote:

What about move it to webpack/voting-app?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webpack/webpack.js.org/issues/3614#issuecomment-600073745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHUXXOWWAUK6BTTOW5SHGLRH533RANCNFSM4LF5GFJQ
.

--
Fernando Montoya
montogeek.com

@montogeek @EugeneHlushko moved to webpack org https://github.com/webpack/voting-app, need update https://github.com/webpack/webpack.js.org/blob/master/package.json#L170

I'll do it 👍

@anshumanv dont forget to do it via yarn on node 10, ty in advance!

its merged, ty all


ImportError: No module named pkg_resources

I am getting this error when running the script manually or through sickbeard.  When I roll back to the version right before the sub downloading was added I get no errors.
Traceback (most recent call last):
File "C:\apps\sickbeard_mp4_automator\sabToSickBeardwithConverter.py", line 7, in
from mkvtomp4 import MkvtoMp4
File "C:\apps\sickbeard_mp4_automator\mkvtomp4.py", line 10, in
from babelfish import Language
File "C:\apps\sickbeard_mp4_automator\babelfish__init__.py", line 13, in
from .converters import (LanguageConverter, LanguageReverseConverter, LanguageEquivalenceConverter, CountryConverter,
File "C:\apps\sickbeard_mp4_automator\babelfish\converters__init__.py", line 6, in
from pkg_resources import iter_entry_points, EntryPoint
ImportError: No module named pkg_resources

ImportError: No module named pkg_resources: the solution is to reinstall python pip using the following  Command are under.
Step: 1 Login in root user.
sudo su root

Step: 2 Uninstall python-pip package if existing.
apt-get purge -y python-pip

Step: 3 Download files using wget command(File download in pwd )
wget https://bootstrap.pypa.io/get-pip.py

Step: 4 Run python file.
python ./get-pip.py

Step: 5 Finaly exicute installation  command.
apt-get install python-pip

Note: User must be root.


[REQ] Inline CNAME compatibility

Just a quick request!
Express.js has an extension that allows it to use CNAMEs...

Is this related to DoneJS? How?

@AltEDU2 Can you give more information for what you are requesting?


Attempting Patch

Soln proposed by @sum1190
Issue #88
cc @Maziar-Fotouhi
@GillesC  can you check all test cases.

@alokrajiv @GillesC Thanks for your efforts.
Is there going to be an official release with this fix?

@Maziar-Fotouhi I think soon we will have to push a new release.


Not working in Xcode 6.1.1

This plugin doesn't appear to work any more. The menu item shows up, but it's not adding a build phase. I'm using a workspace with CocoaPods, if that makes a difference.

Hello, thanks for reporting this. It looks like this was issue with workspaces in general, not 6.1.1.
I have pushed new version which should now work with default Cocoapods configuration. Please check when you have time and let me know does it work for you.

It's not actually finding unused imports as far as I can tell, but it now correctly modifies the project to add a build phase.

Can you please try to run command line version(https://github.com/dblock/fui/) to see if it finds anything?

It's working. I misunderstood what fui does. Am I right that it finds classes that are never used, rather than redundant or unused #import statements?

Great. Yeah, this issue is about that https://github.com/dblock/fui/issues/13


Estimate size and timing information for migrating PeMS clearinghouse

Most of the raw historical data is available in the PeMS Clearinghouse, and this may be the best way for us to access it. However, the web server is an old system, and we can't hit it with too many concurrent requests.
We should estimate:

How much data we would need to download
How long it would take with low concurrency

Quick update here based on some work in this branch:

Approximate size of 1 year of raw data (mostly gzipped CSVs): 100 GiB
Approximate time to download 1 year using two concurrent Airflow workers in us-west-2: 4-5 hours.

So, for 20 years of data we are looking at ~2 TiB (more when uncompressed, but the web server doesn't care), and ~4-5 days of consistent downloading. Not too bad!

Closing as complete!


toggleAction

I wanted to ask, what's the reason that you decided to make clicking an action toggle the action, rather than rewind time to that action as it was when you first demoed the early version of the devtools. It seems like this version is more fragile.
For example, if we look at the counter example from this repository. If we first take the following sequence of actions:
Increment, Increment, Decrement, Increment if Odd, Increment if Odd, Increment
(Increment, Increment, Decrement, Increment,               , Increment)
We end up at 3. If we now toggle the second increment action, we end up at
(Increment, Increment, Decrement, Increment,            , Increment)
This gives us a total of 2. However, if we try and replay the sequence of actions, we would get
Increment, Increment, Decrement, Increment if Odd, Increment if Odd, Increment
(Increment, Increment, Decrement,                    ,                      , Increment)
Giving us 1.
This desync doesn't occur if we simply rewind time to any point, because we can't obtain an invalid history by going to a previous point in time.
Could we get back the functionality from the first demo, where toggling rewound time?

I wanted to ask, what's the reason that you decided to make clicking an action toggle the action, rather than rewind time to that action as it was when you first demoed the early version of the devtools.

Actually when demoing I had a super secret shortcut where shift+click would work as “toggle all actions before” (or after, depending on where you do it) and just click worked as “toggle action”.
I removed it because it was done in a hacky way, and I think toggling and jumping require different UIs. (See also https://github.com/gaearon/redux-devtools/pull/5).
I think there is a misunderstanding here: Increment if Odd will actually never get into the action history. Only the “raw” actions get into the history so they can never “disappear” later because some condition has changed. The total count of actions is always the same. Only the ones you toggled explicitly will be skipped.
I find toggling immensely useful for debugging. You can filter out actions unrelated to the scenario you're working on. Moreover, you can quickly check different action orders just by toggling them and seeing whether a crash occurs (which will be displayed).
That said, I totally understand the the toggling UI doesn't make sense for debugging some kinds of applications or problems (e.g. debugging animation). This is why you can use JUMP_TO_STATE added in #5 to implement an alternative monitor that lets you just go back and forward, whether with a slider or an action log.
Does this help?

I foresee a UI with several buttons by each action, e.g. rollback (jump?) to here, make this action never have happened (toggle?), etc. Eventually the filtering demoed at ReactEurope will be available as a text box in the devtools sidebar. All easy stuff not worthy of @gaearon's time.

Closing as inactive. Let's track the "powerful UI" in #74.


Separe Location and Idiom

I'm from Brazil, so I'd like my streak stats based on brazilian's timezone. At the same time, I want my card written in English.
There's no way to set my timezone without changing the card's language.

The timezone can't be changed. What is displayed is whatever the GitHub API returns which should be UTC time but sometimes it's unreliable.

I thought there was a relation with the timezone because when I change the location, it also changes my streak.

It is not related to the locale. If the image is cached, changing the url parameters can get it to generate the data again which can make it show different data.


[BEAM-3131] Updates Python datastore wordcount example to take a dataset parameter.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @aaltay

Do we need to update the parameters in https://github.com/apache/beam/blob/239319b2a093cdb5c63375eddb5a91ac70172641/sdks/python/apache_beam/io/gcp/datastore/v1/datastoreio.py#L335 ?

I don't think we need to update datastoreio module. project==dataset currently and we are consistent across Java and Python. This change simply allows this example to be used for cases where project where the job is executed is different from the project where the dataset is stored.

The change LGTM.
Do we at least want to document that project argument can take a dataset? Or is it obvious for Datastore users that these two things are interchangeable?

Thanks. Updated the datastoreio documentation to clarify that the project is the project to read from/write to.


Delete command with RETURN COUNT statement throws Error parsing query

OrientDB Version:  v2.2.24 (using Docker image orientdb)
Java Version: openjdk version "1.8.0_131"
OS: Alpine Linux v3.6.2
Expected behavior
delete from Contact return COUNT where name = 'Bob' should return number of deleted records (command without return COUNT works)
Actual behavior
Returns error Error parsing query: delete from Contact return COUNT where name = 'Bob' ^ Encountered " <COUNT> "COUNT "" at line 1, column 28. Was expecting: <BEFORE> ... DB name="amc"
Steps to reproduce
delete from Contact return COUNT where name = 'Bob'
Not sure if documantation is outdated or it is a bug

@luigidellaquila it looks like you forgot to add COUNT in the new parser.

Yep, it seems so, I'll fix it asap
Thanks
Luigi

Fixed on 2.2.x branch, the fix will be released with 2.2.26
Thanks
Luigi


Can't run custom commit-msg hook from overcommit

I am trying to run a custom commit-msg hook, CommonCommit for my project, but with no luck.
My .overcommit.yml file looks like this:
PreCommit:
  Rubocop:
    enabled: false
CommitMsg:
  CommonCommit:
    enabled: true
    quiet: false
    description: 'Checks if the commit message follows the common commit specifications'
    required_executable: 'yarn'
    command: ['yarn', 'run', 'commitlint']
    flags: ['-e']
  TextWidth:
    enabled: false

I am running overcommit not from a globally installed gem but as a dev dependency inside my Gemfile. When I run `bundle exec overcommit -l', I will see CommonCommit listed and enabled
CommitMsg:
  GerritChangeId: disabled
  HardTabs: enabled
  RussianNovel: enabled
  SingleLineSubject: enabled
  TextWidth: disabled
  TrailingPeriod: enabled
  CommonCommit: enabled

And when I run yarn run commitlint -e, it will check my last commit message and fail, because it doesn't follow the specifications.
However, when I run git commit --amend, it will show me this:
Running commit-msg hooks
Checking for hard tabs.....................................[HardTabs] OK
Checking subject line.............................[SingleLineSubject] OK
Checking for trailing periods in subject.............[TrailingPeriod] OK

And CommonCommit is nowhere to be found. No error message, as if it was never there. Could this be a bug, or do you think I'm just not doing something properly?

Not sure what the issue is, @camiloforero. I can't reproduce what you're seeing. This suggests to me that potentially the globally-installed overcommit gem you have is a very old version, or you are doing something else incorrectly.
When I have the following .overcommit.yml:
CommitMsg:
  CommonCommit:
    enabled: true
    quiet: false
    description: 'Checks if the commit message follows the common commit specifications'
    command: ['echo', 'hello']

...and I git add ... it and attempt to commit, I correctly see the output:
Running pre-commit hooks
(...omitted...)

✓ All pre-commit hooks passed

Running commit-msg hooks
Check subject capitalization.....................[CapitalizedSubject] OK
Check subject line................................[SingleLineSubject] OK
Check for trailing periods in subject................[TrailingPeriod] OK
Check text width..........................................[TextWidth] OK
Checks if the commit message follows the common commit specifications[CommonCommit] OK

⚠ All commit-msg hooks passed, but with warnings

I'm not sure what else to offer—this is a pretty straightforward example. Let us know what you discover.

Oh that did it!
I ran bundle exec overcommit -v and got I was using version 0.22.0. I have no idea how that happened as I installed it by adding it to my gemfile and running bundle install, but I just removed it and reinstalled it again, and I have v 0.49.0 now, which works great.
Thank you


Long running process

I have a javascript method which invokes a php function which takes around 4 seconds, well the bar starts loading just when the ajax call returns the value from the PHP function. I'd like the bar to start loading just when the user clicks the JS function call. How can this be achieved.
function jsfunction(){
     simplebar.go(10);
     /*ajax call PHP takes around 4 seconds*/
    simplebar.go(100);
}

well I managed to do it. My ajax call sync = true I set it to false and got it working!