Not Running

I cant seem to get it to work,
ive followed the guide, also had to add the implementation lines in the build gradle before it would build.
finally got it bult and installed, but the canvas shows nothing but white ?
any ideas on how to get the camera preview to actual display something ?
i initially thought i was to do with opencv not init , but when looking at the logcat logs i can see the SUCCESS fro opencv debug line, then nothing after .
any help would be appreciated

@ramusbucket  @saturnim  didn't work for me!! Can you give me your project?

@JaydeeSale<EMAIL_ADDRESS>could you send ?
@SwagatRanjit84 i sent you my version

@saturnim sent you a drive link

@saturnim Did it work on your side?

@SwagatRanjit84 did not work even in version from @JaydeeSale . Only one issue might be i used not orginal build script for document scaner component. Or some enviroment issue...

@JaydeeSale  Is this library working on your side? Can you share the code?

https://drive.google.com/file/d/1mBx0zoWmZ_71ZO5W_dpGM_cG2upDURCP/view?usp=drivesdk
Here's a link to the file

@JaydeeSale<EMAIL_ADDRESS>could you send ?

Does the link not work?

NO, it was showing error.Build errors

Hey guys have you tried to add "flex: 1" to component style
<DocumentScanner
style={{ flex: 1 }}
onPictureTaken={data =>{
console.log(data.path)
}}
enableTorch={false}
detectionCountBeforeCapture={5}
/>

@andreluisjunqueira  Thanks mate, I have got it running and working properly.
@saturnim  @JaydeeSale  @ramusbucket  @thibbo  I have got it running, if you guys need help, ping me

Hello everyone,
First of all, I'm sorry to bother you with an issue like this one. I've been stuck for quite a long time and not finding the path to an error-free process. I'm still at the installation process.
Here what I'm doing so far for trying make it works:

react-native init --version="0.57.0" app_name
cd app_name
npm install --save react-native-documentscanner-android
edit files:

in Main.Application.java: import com.documentscanner.DocumentScannerPackage;
in the @Override part: new DocumentScannerPackage()
in AndroidManifest.xml:
in AndroidManifest.xml: <application tools:replace="android:allowBackup" ...>

react-native link react-native-documentscanner-android
react-native run-android

And look at what happens:
C:\Users\Utilisateur\Documents\ReactNative\docscan>react-native run-android
Starting JS server...
Building and installing the app on the device (cd android && gradlew.bat installDebug)...

Configure project :app
WARNING: Configuration 'compile' is obsolete and has been replaced with 'implementation' and 'api'.
It will be removed at the end of 2018. For more information see: http://d.android.com/r/tools/update-dependency-configurations.html

Configure project :openCVLibrary310
WARNING: The specified Android SDK Build Tools version (23.0.2) is ignored, as it is below the minimum supported version (27.0.3) for Android Gradle Plugin 3.1.4.
Android SDK Build Tools 27.0.3 will be used.
To suppress this warning, remove "buildToolsVersion '23.0.2'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

Configure project :react-native-documentscanner-android
WARNING: Configuration 'compile' is obsolete and has been replaced with 'implementation' and 'api'.
It will be removed at the end of 2018. For more information see: http://d.android.com/r/tools/update-dependency-configurations.html
WARNING: The specified Android SDK Build Tools version (23.0.1) is ignored, as it is below the minimum supported version (27.0.3) for Android Gradle Plugin 3.1.4.
Android SDK Build Tools 27.0.3 will be used.
To suppress this warning, remove "buildToolsVersion '23.0.1'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

Task :openCVLibrary310:compileDebugJavaWithJavac
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

Task :react-native-documentscanner-android:compileDebugJavaWithJavac
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

Task :app:compileDebugJavaWithJavac FAILED
C:\Users\Utilisateur\Documents\ReactNative\docscan\android\app\src\main\java\com\docscan\MainApplication.java:7: error: cannot find symbol
import org.opencv.DocumentScannerPackage;
^
symbol:   class DocumentScannerPackage
location: package org.opencv
1 error

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 15s
58 actionable tasks: 36 executed, 22 up-to-date
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/getting-started.html
I'm using Windows 10 and in Visual Studio, I've a "problem log" appearing: "Could not install Gradle distribution from 'https://services.gradle.org/distributions/gradle-4.4-all.zip'.
Could not create parent directory for lock file C:\ProgramData\chocolatey\lib\gradle\tools\gradle-5.2.1\wrapper\dists\gradle-4.4-all\9br9xq1tocpiv8o6njlyu5op1\gradle-4.4-all.zip.lck"
Do you guys know I'm doing wrong?
Thanks a lot!!

@audacesk whats ur problem mate?

@SwagatRanjit84  My problem is that I cannot build the project on my windows. I've followed the Readme's step, the building configs of @saturnim , and some other stuff that I found online, but I'm still having this error:
"FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
Get more help at https://help.gradle.org"

@audacesk I have got this document scanner running on git. Do you want it?

@SwagatRanjit84 Yes please, I would really apreciate :) Thanks!!
Here is my email is wanted<EMAIL_ADDRESS>

Hey, Tried every thing in above comments but still having white screen issue. Permission is set, Used style={{ flex: 1 }}. Not sure what to do also there is one more issue i have noticed in log cat
2019-03-04 13:44:04.170 6810-6810/com.documentscanner E/OpenCV/StaticHelper: OpenCV error: Cannot load info library for OpenCV

Hey, Tried every thing in above comments but still having white screen issue. Permission is set, Used style={{ flex: 1 }}. Not sure what to do also there is one more issue i have noticed in log cat
2019-03-04 13:44:04.170 6810-6810/com.documentscanner E/OpenCV/StaticHelper: OpenCV error: Cannot load info library for OpenCV

Worked by changing style to backgroundColor: 'transparent'

@omerabbas01 Whats your problem? it is built sucessfully or not?

@SwagatRanjit84 Please, can you share your Git with me ? I only resolve a problem to find another one... #help<EMAIL_ADDRESS>@omerabbas01 I have the same error : Cannot load info library for OpenCV... Where did you write backgroundColor: 'transparent' ?
<DocumentScanner
...
style={{
flex: 1,
backgroundColor: 'transparent'
}}
/>

@SwagatRanjit84 share to me his git project and it resolves the OpenCV library loading problem. (thanks) The error "Cannot load info library for OpenCV" is still here but the library is loaded just after so it should not be a real problem.
But the detection of rectangular paper doesn't work !
The camera is on and I can see it on the screen but the document is not detected.
I try it on 3 devices (Nokia, Samsung and Zebra (Android device for logistic)) and I get the same problem.
While trying to detect, there is a loop of logs like these :
03-12 09:52:24.076 17254-17254/com.billscanner D/JavaCameraView: sending message to ImageProcessor: previewFrame - com.documentscanner.helpers.PreviewFrame@18410f0
03-12 09:52:24.076 17254-17330/com.billscanner D/ImageProcessor: Message Received: previewFrame - com.documentscanner.helpers.PreviewFrame@18410f0
03-12 09:52:24.114 17254-17330/com.billscanner I/DESENHAR: Quad----->null
The "Quad----->null" seems to say "no rectangle detected" ???
But I have a document : sheet of white paper on dark background.
Any idea ???

Ok by approaching the camera very close to the document (80% of the screen), the detection worked.
I was using it like with other scan library I try (not OSS) OpenCV seems less permissive (or we can parameter this ?)...

@SwagatRanjit84 Think you could share the secrets to getting this library to work with me as well? I'm currently stuck on the black screen issue, and I've tried all of the "fixes" mentioned in all of the issues in this repo. :sweat_smile:

It worked on react native version 0.57.8, after adding
  implementation 'com.android.support:support-v4:27.1.1'     implementation 'com.android.support:appcompat-v7:27.0.2'     implementation 'com.android.support:design:27.1.0'
in android/app/build.gradle

It worked on react native version 0.57.8, after adding
 implementation 'com.android.support:support-v4:27.1.1' implementation 'com.android.support:appcompat-v7:27.0.2' implementation 'com.android.support:design:27.1.0'
in android/app/build.gradle

@SwagatRanjit84 Thanks for this.
Also it worked on react native version 0.59.8.

Also it worked on react native version 0.59.8.

@anuj002 can you share please your configurations for android/app/build.gradle and android/build.gradle, for RN 59.8?
i´m still can´t run the app after added those lines
thanks

Hi,
It's working for a seconds only, but right now its Showing only black
screen with error "OpenCV error: Cannot load info library for OpenCV".
If you find any solution please let me know.
--
Thanks & Regard
Anuj Agrawal
React Native & Android Developer
Handheld<EMAIL_ADDRESS><EMAIL_ADDRESS>+919462797441
On Fri, Jun 7, 2019 at 3:03 AM enrique26<EMAIL_ADDRESS>wrote:

Also it worked on react native version 0.59.8.
@anuj002 https://github.com/anuj002 can you share please your
configurations for android/app/build.gradle and android/build.gradle, for
RN 59.8?
i´m still can´t run the app after added those lines
thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/andreluisjunqueira/react-native-documentscanner-android/issues/15?email_source=notifications&email_token=AHAROHKG7MFHOF6SSZVOXCTPZF7BDA5CNFSM4GPSMROKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXEHGAQ#issuecomment-499675906,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHAROHIGVBBI3ETAPZXIRHDPZF7BDANCNFSM4GPSMROA
.

@anuj002 you can see how to fix issue Black screen at here: https://github.com/andreluisjunqueira/react-native-documentscanner-android/pull/30#issue-295843425

Hello,
is it working with RN 0.60.5 ?
I'm stuck with this error on Build :
error Failed to install the app. Make sure you have the Android development environment set up: https://facebook.github.io/react-native/docs/getting-started.html#android-development-environment. Run CLI with --verbose flag for more details.
Error: Command failed: gradlew.bat app:installDebug -PreactNativeDevServerPort=8081
C:...\android\app\build\generated\rncli\src\main\java\com\facebook\react\PackageList.java:21: error: cannot find symbol
import org.opencv.DocumentScannerPackage;
^
symbol:   class DocumentScannerPackage
location: package org.opencv
C:...\android\app\build\generated\rncli\src\main\java\com\facebook\react\PackageList.java:61: error: cannot find symbol
new DocumentScannerPackage(),
^
symbol:   class DocumentScannerPackage
location: class PackageList
2 errors
FAILURE: Build failed with an exception.
Any idea ?


[bug] fix the address issue for OIDC

See: https://github.com/Timshel/vaultwarden/issues/65#issuecomment-2156223591

When searching for the issue I found this previous issue with additional information: https://github.com/casdoor/casdoor/issues/1184.
There is some mention of previously reverted code due to conflict and a hint on a possible way to fix it.
I won't have time to look at it so Good luck :).

And exactly the same problem with proxmox authorization

Commit was reverted so probably make sense to reopen the issue.
Tested the 1.632.0 release, and it was failing too.
When manually exchanging the code for token I could see:
  "address": {
    "formatted": "",
    "street_address": "",
    "locality": "",
    "region": "",
    "postal_code": "",
    "country": ""
  },

So a bit unsure why it would fail, but I would recommend to just not include the field when there is no data.

can you tell us what tool / platform you are using that cause this problem ?

Commit was reverted so probably make sense to reopen the issue. Tested the 1.632.0 release, and it was failing too.
When manually exchanging the code for token I could see:
  "address": {
    "formatted": "",
    "street_address": "",
    "locality": "",
    "region": "",
    "postal_code": "",
    "country": ""
  },

So a bit unsure why it would fail, but I would recommend to just not include the field when there is no data.

can you tell us what tool / platform you are using that cause this problem ?

I'm using https://github.com/ramosbugs/openidconnect-rs which in general follow the spec quite strictly.
Since the change in 1.632.0 was reverted I did not spend more time trying to understand why it failed even with the new format.

I'm using https://github.com/ramosbugs/openidconnect-rs which in general follow the spec quite strictly.
Since the change in 1.632.0 was reverted I did not spend more time trying to understand why it failed even with the new format.

thanks for reply, we will try to fix it soon.

I'm using https://github.com/ramosbugs/openidconnect-rs which in general follow the spec quite strictly.
Since the change in 1.632.0 was reverted I did not spend more time trying to understand why it failed even with the new format.

I guess you are using the gitlab example of the openidcinnect-rs. however, the gitlab exapmle example add some external fields to the UserInfoClaim, you can remove the relevant these field to fix the problem, like below.
struct GitLabClaims {
    // Deprecated and thus optional as it might be removed in the futre
    // sub_legacy: Option<String>,
    // groups: Vec<String>,
}

**Note: you need to use v1.632.0 **

Just to clarify I'm not really using Casdoor, I'm maintaining a PR to add OpenId Connect to Vaultwarden and someone had issues with it.
As for your suggestion since the Address is part of the default CoreIdTokenClaims it's not easy to ignore.


Add collaborators to this repo

Comment below to be added as a collaborator
We will all be using this repository today. Even though it's public right now, you will not be able to make changes until you're given the correct permissions. We've automated this process via the GitHub API. Once you comment, we will add you as a collaborator.
You'll start to receive a lot of emails. (❗) You can slow these down immediately by clicking the Unwatch button at the top of the page and selecting either the Participating and @mentions or Ignore option.

hello there!

Howdy

Hello

Hello everyone!

Hello

hey there!

We made it!

Hello

Hello


fix 404 link

Some link addresses have expired. I replace them with new ones.

Coverage remained the same at 71.087% when pulling 0a6d8a27b554eee9b469d9c76edb9cae39dfe3f2 on ReedSun:dev into 04293873eb97d8cccac6396e420df85f12bb904f on dvajs:master.


Cannot put debug points to query in or out inside a partition

Description:
Cannot put debug points to query in or out inside a partition
Affected Product Version:
WSO2 Stream Processor 4.0.0 Milestone 13 Release
OS, DB, other environment details and versions:
Ubuntu 16.04
Steps to reproduce:
try to add debug points to follwing siddhi app.
-------------SIddhi app------>
@App:name('TestSiddhiApp')
define stream sensorStream (sensorId string, temperature double);
partition with (sensorId of sensorStream)
begin
from sensorStream#window.length(1)
select max(temperature) as maxTemp
insert into DeviceTempStream
end;
----------End of siddhi app---->

This issue is not fixed yet. (The commit referred to here is a mistake)

Seems like this a limitation in Siddhi. @suhothayan Please confirm.

Yes its a limitation in Siddhi we'll be fixing this in the future


🧑💻(frontend) configure site proxy

Purpose
Allow locale configuration to proxy backend (and later be able to deploy a backend on aws for dev purposes)
Proposal
Use the configuration of the backend adress to configure the proxy.
You can override this config in your local environnement in the file .env.local and target the backend you wish.

Shouldn't you provide a default value for REACT_APP_BACKEND_API_BASE_URL somewhere? So we can still run it like now or there is something magic I can't see?

Shouldn't you provide a default value for REACT_APP_BACKEND_API_BASE_URL somewhere? So we can still run it like now or there is something magic I can't see?

It is already defined in .env.development

Shouldn't you provide a default value for REACT_APP_BACKEND_API_BASE_URL somewhere? So we can still run it like now or there is something magic I can't see?

It is already defined in .env.development

:+1: I was looking in the env.d folder of the project...


add fetch model to Makefile

Fetch model already exists on the main branch (hence the merge conflicts). Is there a reason you added it again? Was the original fetch-model not working for you?

fetch-model already exists and is functioning


Add button to gracefully close a form

Currently I just kill a form when I am done with it, might be nicer to have a propper way to close, via a button

What would you want it to do?


Suggestion: AOTHelper methods with type parameter

https://github.com/jilleJr/Newtonsoft.Json-for-Unity/wiki/Reference-Newtonsoft.Json.Utilities.AotHelper
It would be nice to have AOTHelper methods with Type parameter in addition to the generic version. My use case is I have a collection of types, and I want to ensure lists exist for every one of them, so calling EnsureList(Type type) instead of EnsureList() would be better in this situation.
Nice work, by the way! I'm glad to have found this.

I'm sorry to say that this cannot be done :(
The AotHelper works just because it's generic. It's magic comes from tricking the compiler on generating the types specified from the generic type parameter. Something that cannot be done with regular type parameters.
You will have to specify all the types that will be used explicitly via the AotHelper or via link.xml.


Update data-factory-build-your-first-pipeline-using-vs.md

Updated StorageLinkedService to AzureStorageLinkedService1

Hi @adfteam, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@spelluru
Please review and approve/disapprove this PR here, thanks.

I have made exact same changes as part of another PR today. Please close this. I have already communicated this to Wee Hyong. Thanks.

Here is the duplicate PR: https://github.com/Azure/azure-content-pr/pull/14065.


pgsql generateScript & pgsql executeScript

Is it not implemented?

If I remember, generator produces universal script good for most DBMS, maybe a few minor differences in data types.


Block commenting out a Blazor attribute issues a compile error.

I tried to comment out the attribute using the razor comment syntax:
<svg id="template" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns="http://www.w3.org/2000/svg"
     viewBox="0 0 651 442" @*@onmousedown="@OnDesignPanelClicked"*@>

Upon execution, it interpreted the commented area as regular Blazor code:
warn: Microsoft.AspNetCore.Components.Server.Circuits.RemoteRenderer[100]
      Unhandled exception rendering component: InvalidCharacterError: Failed to execute 'setAttribute' on 'Element': '@*@onmousedown="@OnDesignPanelClicked"*@' is not a valid attribute name.
System.InvalidOperationException: InvalidCharacterError: Failed to execute 'setAttribute' on 'Element': '@*@onmousedown="@OnDesignPanelClicked"*@' is not a valid attribute name.
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.InvokeRenderCompletedCallsAfterUpdateDisplayTask(Task updateDisplayTask, Int32[] updatedComponents)
fail: Microsoft.AspNetCore.Components.Server.Circuits.CircuitHost[111]
      Unhandled exception in circuit 'juq2BjFD4RL46PNRh3SetydfRBC3OCZTupfYCpRWu2U'.
System.AggregateException: One or more errors occurred. (InvalidCharacterError: Failed to execute 'setAttribute' on 'Element': '@*@onmousedown="@OnDesignPanelClicked"*@' is not a valid attribute name.)
 ---> System.InvalidOperationException: InvalidCharacterError: Failed to execute 'setAttribute' on 'Element': '@*@onmousedown="@OnDesignPanelClicked"*@' is not a valid attribute name.
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.InvokeRenderCompletedCallsAfterUpdateDisplayTask(Task updateDisplayTask, Int32[] updatedComponents)
   --- End of inner exception stack trace ---
info: Microsoft.AspNetCore.Routing.EndpointMiddleware[1]
      Executed endpoint '/_blazor'
info: Microsoft.AspNetCore.Hosting.Diagnostics[2]
      Request finished in 417.2702ms 101

@333fred It looks like we're incorrectly parsing here, we find the comment, but wrap it in an attribute:

That then means when we emit, we're emitting it as an attribute to the component.

we find the comment, but wrap it in an attribute

This doesn't surprise me, given that the parser doesn't use leading or trailing trivia; I'm guessing this won't be easy to fix without doing that.

5 years later this is still an issue, and i wasted few hours on it, learning Blazor. And must be very common issue, as this is how visual studio comments in razor by default... Cmon people


🐛 BUG: wrangler dev --remote does not work if a custom domain doesn't already exist

Which Cloudflare product(s) does this pertain to?
Wrangler core
What version(s) of the tool(s) are you using?
3.7.0
What version of Node are you using?
18.17.0
What operating system are you using?
Windows
Describe the Bug
wrangler dev --remote with a custom domain that doesn't yet exist throws an unhelpful ENOTFOUND error.

Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

This now shows a more helpful error message as of #5107


merge gdnsuppress output from pipeline

Which issue this PR addresses:

Fixes (hopefully) gdn issues
What this PR does / why we need it:

Test plan for issue:
Run the pipeline

Is there any documentation that needs to be updated for this PR?

closed in favor of #3032 3032


Add PropertiesProvider as a utility class

Separate the property loading functionality as a utility class so it can be easily reused in google-cloud-java.
More context: https://github.com/GoogleCloudPlatform/google-cloud-java/pull/1790/files

Codecov Report

Merging #250 into master will decrease coverage by 0.2%.
The diff coverage is 64.7%.

@@             Coverage Diff              @@
##             master     #250      +/-   ##
============================================
- Coverage     70.98%   70.78%   -0.21%
- Complexity      526      527       +1
============================================
  Files            75       76       +1
  Lines          2592     2601       +9
  Branches        268      268
============================================
+ Hits           1840     1841       +1
- Misses          648      656       +8
  Partials        104      104

Impacted Files
Coverage Δ
Complexity Δ

...gle/api/gax/grpc/InstantiatingChannelProvider.java
49.42% <100%> (-4.43%)
10 <0> (-4)

...a/com/google/api/gax/utils/PropertiesProvider.java
53.84% <53.84%> (ø)
5 <5> (?)

Continue to review full report at Codecov.

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

PTAL @garrettjonesgoogle

Commit msg updated. PTAL

LGTM after tests pass.


Tuple should allow negative subscripts

...to be consistent with the list and string classes both allowing negative subscripts.

Occasionally I think back on this. Implementing it would involve patching a couple places in emitter, so it's not that it's difficult to do. In fact, the vm probably wouldn't even notice a difference. It's more that I question why someone would want to do this. I'm normally okay with adding seemingly odd things in (like piping to variants), but this one I struggle to find any use at all for.
My mind isn't completely closed to this. If there's a compelling argument for it that isn't "consistency", then I'll think it over. For now, no.


fix: call model.quantizer with latents in PretrainedDACPretransform

If quantize_on_decode = False this code will break since z is not defined. We want to quantize the latents to get z.

Good catch, I guess I never tested that path after adding the quantize_on_decode functionality


How to add state for 2players-Atari

Issue summary
When we want to define new another 2players game such as Boxing, we need to create a new state file for player2. How do I create that file?
[Put a detailed description of the issue here.]
System information

[Operating system]
[Python version]
[Gym Retro version]

Since that's an atari game, maybe try out https://www.pettingzoo.ml/# first?


chore: option to op-out of informer

The Resourcebuilder:view starts an informer on all curl cmds, that's a problem when getting a resource that doesn't support the watch feature such as metrics.
Add an option such as:
      if opts.cmd == "curl" and opts.with_informer then
        informer.start(builder)
      end

And clean up the metrics check in informer.lua

https://github.com/Ramilito/kubectl.nvim/pull/275


Size of sample is invalid since max_positions=(1024, 1024)

Hi @wasiahmad ,
I trained PLBART for JAVA -> PYTHON translation. But while testing, I was getting below error -
2021-07-21 05:31:11 | INFO | train | {"epoch": 30, "train_loss": "2.69", "train_nll_loss": "0.723", "train_ppl": "1.65", "train_wps": "7795.4", "train_ups": "0.35", "train_wpb": "22402", "train_bsz": "58.2", "train_num_updates": "240", "train_lr": "1.2e-05", "train_gnorm": "0.607", "train_train_wall": "5", "train_wall": "638"}
2021-07-21 05:31:11 | INFO | fairseq_cli.train | done training in 637.0 seconds
Traceback (most recent call last):
  File "/home/jovyan/.local/bin/fairseq-generate", line 8, in <module>
    sys.exit(cli_main())
  File "/home/jovyan/.local/lib/python3.8/site-packages/fairseq_cli/generate.py", line 379, in cli_main
    main(args)
  File "/home/jovyan/.local/lib/python3.8/site-packages/fairseq_cli/generate.py", line 41, in main
    return _main(args, sys.stdout)
  File "/home/jovyan/.local/lib/python3.8/site-packages/fairseq_cli/generate.py", line 132, in _main
    itr = task.get_batch_iterator(
  File "/home/jovyan/.local/lib/python3.8/site-packages/fairseq/tasks/fairseq_task.py", line 227, in get_batch_iterator
    indices = self.filter_indices_by_size(
  File "/home/jovyan/.local/lib/python3.8/site-packages/fairseq/tasks/fairseq_task.py", line 137, in filter_indices_by_size
    raise Exception(
Exception: Size of sample #81 is invalid (=(1024, 1045)) since max_positions=(1024, 1024), skip this example with --skip-invalid-size-inputs-valid-test

I didn't understand what (1024, 1045) and (1024, 1024) mean. I'm using default 510 for training and 9999 for testing as below -
if [[ $SPLIT == 'test' ]]; then
        MAX_LEN=9999 # we do not truncate test sequences
    else
        MAX_LEN=510

Could you plz suggest how to proceed further..?

Which fairseq version are you using? I am not sure but it seems like fairseq is complaining about target length. My guess is (=(1024, 1045)) means the source length is 1024 and the target length is 1045. Can you try truncating the target?  But I am not sure because, during preparing, we truncate the source to confine it in 512 lengths, in that case, fairseq cannot get a source of length 1024. Did you make any changes in our script?

Hi @wasiahmad ,

Which fairseq version are you using?

I'm using 0.10.2
I think it has something to do with max_positions of BART. This suggests that maximum 1024 can be used for inference. For longer sequences (>1024) we have to modify embed_positions and finetune to adjust weights.
I also saw below info from logs when I just cloned and ran the PLBART scripts without any changes -
1)
2021-07-23 06:47:29 | INFO | fairseq_cli.train | Namespace(activation_fn='gelu', adam_betas='(0.9, 0.98)'........ max_epoch=30, max_source_positions=1024, max_target_positions=1024,........
and 2)
2021-07-23 06:47:32 | INFO | fairseq_cli.train | BARTModel(
  (encoder): TransformerEncoder(
    (dropout_module): FairseqDropout()
    (embed_tokens): Embedding(50005, 768, padding_idx=1)
    (embed_positions): LearnedPositionalEmbedding(1026, 768, padding_idx=1)

After these observations, below changes worked for 'test'-
if [[ $SPLIT == 'test' ]]; then
        MAX_LEN=510

and
if [[ $SPLIT == 'test' ]]; then
        MAX_LEN=1022

So I think max cannot be more that 1024 for 'test' but I'm not sure about this. May be you can give your thoughts and confirm.

One thing that we can do is, during preprocessing, we can use --only-source to only process the source file. In that case, during generation, fairseq won't load target data. We actually do not require processed test data.

Closing this, feel free to open it if it is not solved yet.


Here are some items of improvement:

[x] Default to "Redirect to current tab"
[ ] Auto-save feature, remove the "Save" button
[x] Add "Enable" checkbox
[x] New theme/visual effect
[x] Re-design buttons
[x] Re-design "Rules"

@lzb any opinion?

Trigger of Auto-save is rather complex, we need to figure out the timing.
@lzb


Use CUDA_VISIBLE_DEVICES in "lmql serve" layout calculation

Fixes #333, a bug in which lmql serve ignored CUDA_VISIBLE_DEVICES when figuring out what to do with its --layout argument. I've tested this in a scenario where I had devices 0 and 2 on a given machine; without this fix, setting --layout 2x1 resulted in the lmql server worker processes being assigned to devices 0 and 1, which caused problems since somebody else was using device 1. With this fix, the right GPUs are used.

Thanks, this looks good. I was just able to test it on a multi GPU system.


Boston data set used by axiline

This dataset is deprecated. Actually it isn't even used.
ImportError:
load_boston has been removed from scikit-learn since version 1.2.
 Furthermore the goal of the
research that led to the creation of this dataset was to study the
impact of air quality but it did not give adequate demonstration of the
validity of this assumption.
The scikit-learn maintainers therefore strongly discourage the use of
this dataset unless the purpose of the code is to study and educate
about ethical issues in data science and machine learning.

This dataset was used in previous tests with decision tree generation. Thanks for the reminder, it was removed in a recent update.


Mimetypes to render to conform with notebook

[ ] application/javascript
[X] text/html - Note: needs to handle scripts, comes from transformime
[X] text/markdown - Commonmark implementation
[ ] text/latex - Separate from transformime
[ ] image/svg+xml
[X] image/png - in transformime
[X] image/jpeg - in transformime
[ ] application/pdf- current pdf link form can be here, may want to try PDF.js
[X] text/plain - in transformime

Should we migrate ConsoleTextRenderer from transformime-jupyter-transformers to transformime (?)
Actually, shouldn't the text/plain type handle ANSI codes too anyway?

I think text/plain =/= text/plain+ansi .

We have these types of plainish output in Jupyter right?

stdout, stderr (should absolutely respect emoji, character codes, color codes, etc.)
text/plain (default output, strings, etc.)

All of these should end up in pres I assume. What does our current text/plain do?

I guess with your above suggestion, stdout/stderr are text/plain+ansi

Yeah that's what I'm thinking.  I don't recall if that's a real or valid mimetype though...  I should probably review how to use the +.

I think conformance with the notebook is #1 priority here.

\#1 ;)

I feel silly, didn't realize the was the "jupyter" specific repo.  Ok, yes text/plain is a-okay to me.

Since you're grabbing JS and SVG Jon, I'll tackle the current version of the PDF renderer. Oh but guh... It does linking and I'm not sure what that's like for Atom or Electron. Only makes sense in the server side notebook. :unamused:

Finished off JS and SVG in #9

The remaining one is proper script running for text/html.


build: add build to ci checks

I created a lint rule so we are good. Also forgot once again that we do not have .env file for the prod build in the repo.


To Scan or not to Scan ... and Reconnect without scan?

Thanks for a great lib. Stackoverflow channel for questions? Two things confuse me in the doc and may be the reason one of my devices does not reconnect after power-cycle (others do).

You say "DO NOT scan with the adapter while you have an open GATT connection". Well isn't a scan required to find other devices or reconnect? If I turn off the scan, then no new devices or power-cycled devices will be found. What am I missing here?

Reconnect: if I don't do this via the scan procedure, then how does it work. Pointer?

Thanks!
Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[] iOS
[x] Android
[] tvOS
[] macOS
[x] .NET Standard
x UWP IS NOT SUPPORTED DURING THE BETA PERIOD - ONLY PRs ARE WELCOME FOR NOW

Version of OS(s) listed above with issue
4.3+
Version of Library
Expected Behaviour
Some sensor devices do not reconnect after power cycle.

Thanks, especially for answer 2 which points to a place I hadn't looked yet. The sources are extensive, I have looked but no conclusives to my situation found.


Notifications and privacy

Not all of these user stories have to be in the MVP, but we should be thinking about enabling these in the future as we build the system.
As a USER I would like to be able to subscribe multiple devices for notifications on my account.
As a USER I would like to be able to subscribe devices for notifications on my account, even if I am logged off. This will help me when I am sharing my phone with my other family members.
As a USER I would like to be able to unsubscribe from individual notifications (for example "new survey notifications for health workers"), and also unsubscribe other devices than the one I am using from notifications.
As a USER I would like my notifications to not be sent at the exact same time as all other users. If I get a notification at the exact same time as another IoGT user, they can guess that we have requested the same notifications which may be sensitive.
As a USER I would like to receive a default message "A message has arrived" notification (rather than the exact notification text) if there is a chance I don't have privacy, or if I request it. Some indicators that I may not have privacy are

I have not visited IoGT in eg 30 days
I have subscribed this device to receive notifications while I am logged out
I have indicated that I want default messages in my notifications when I subscribed this device

As a USER, when I receive a default message notification, I would like the text to help ME understand that it's meant for ME, but I don't necessarily want the notification to reveal my username or display name.
As an ADMIN I want to be able to change the number of days of non-visiting before a user starts receiving default message notifications.
As a USER, if I unsubscribe a device from notifications, and there are queued notifications that should be sent to that device, I want those queued notifications to be deleted.
Same as above with possibly more detail:

[ ] When a user who receives notifications logs in to a new device, ask them if they want to sign this new device up to get notifications (this will also work for clearing cache/cookies)
[ ] Need some way for users to unsubscribe a particular device from notifications while keeping others
[ ] Need two options for sending notifications - normal or urgent. Urgent sends all notifications immediately. Normal staggers the sends across a time period (several hours?), randomly by useraccount x device. This ensures that two users on a single device don't receive a notification simultaneously, which would give User 1 insight into what User 2 has subscribed to
[ ] When a user is logged out the notification reads "A new message has arrived for [useraccount or display name?]. To login and view the notification click here. To stop receiving notifications for [useraccount/displayname] on this device, click here."
[ ] Option in Admin Panel - when a user hasn't accessed the site in X days, the notification will appear as if the user were logged out
[ ] Option for users - "I share this device other other people, please don't show the content of my messages when I receive notifications."
[ ] If there are queued unsent notifications for X device and Y account, and user Y unsubscribes device X from notifications, those unsent notifications should be deleted

As a USER I would like to be able to unsubscribe from individual notifications (for example "new survey notifications for health workers"), and also unsubscribe other devices than the one I am using from notifications. - This must be considered for first itteration


Refactor our S3 integration tests

Our S3 integration tests consists of one massive test that tries to test most of our S3 functionality. This is difficult to read and failing tests can be hard to track down.
We should refactor it so it's easier to read: a test per behavior is a good goal.
The Drop pattern from our Elastictranscoder tests can be used to ensure if a test fails, it cleans up after itself by deleting the test bucket.

Starting to work on these :man_dancing:


Update config for Markdown links check

Closes #296

Generally I don't like the idea of treating 4xx codes as success codes. I guess 403 could mean that CI just doesn't have access to some resource but it could also mean that resource is no longer available and we should fix the link.
I intentionally didn't mark Format & lint / markdown check as a requirement for pull requests because I expected some fails which may not have anything with the specific pull request.
Where is that failed check that you mentioned in the https://github.com/mlcommons/mobile_app_open/issues/296?
Is it permanent, or is it happening only sometimes?
If this specific link often (or always) fails to load on CI, I think it would be better to make an exception for this specific link.
Previously I excluded some links in the datasets directory from the check because they are located in the private repository which is not accessible from the CI.
Se here: datasets/util/ade20k/README.md
# ADE20K
<!-- markdown-link-check-disable-next-line -->
ade_20k_calibration_files.txt source is <https://github.com/mlcommons/mobile/blob/master/calibration/ADE20K/ADE20k_calibration.txt>

The mentioned link is here:
https://github.com/mlcommons/mobile_app_open/blob/903bd43c8d2faf5eff0e9104e00a5531f1c11336/CONTRIBUTING.md?plain=1#L14
403 means forbidden and requires authorization. I remember there are some other links to SDK which also have this error 403 and were excluded from check, but the browser can redirect to login page.
If the resource is not available, we will often get a 404 instead.
So I think it's quite safe to treat 403 as a successful "live link" check.

The mentioned link is here

I mean, in which pull requests or commits did it fail?

This one: https://github.com/mlcommons/mobile_app_open/pull/292
I rerun the check several times but it always failed.

403 means forbidden and requires authorization

Resources don't tend to suddenly require authorization.
If something is accessible now, I expect this link to remain accessible in the foreseeable future.
If some link is known to require authorization, we can mark it.
If some resource was available freely but started requiring authorization, we may need to audit it and replace the link with something else, or add a note how to get authorized.

Ok. I will add the exclusion for the mentioned link.


create help topics infrastructure

[ ] test with https://docs.rs/trycmd/latest/trycmd/ ?
[ ] visible from wavrw help [topic]
[ ] also as web pages? (mdbook?)

https://docs.rs/snapbox/latest/snapbox/ is another CLI focused snapshot testing tool.


Open app/bookmark in the same tab, but per item setting

In 1.5 you introduced the global option to force apps&bookmarks to open in the same or a new tab.
What I meant in my previous ticket was to make this setting per item setting (both for bookmark and app).
Doable?

Added separate options for apps, bookmarks and search.

Thanks for adding this, but what I really meant was to add that option to each individual item. For instance I could decide that app-x opens in a new tab, but app-y opens in the same one.
Similarly for each bookmark.
So there could be those global options as introduced in 1.5.2, but also some kind of override per each app/bookmark (maybe a checkbox or add some optional prefix in the URL?).
Apologies if it was not clear from the get-go,


Type conflict when calling a function module.

Hi Eric,
Thanks for you development,
today,  when i tried to do, some errors have caused.
its the code what i had written
REPORT ZTA_TEMP2.
DATA: log TYPE REF TO zcl_logger.
BREAK-POINT.
CREATE OBJECT log.
log = zcl_logger=>new( object = 'ZTAMER'
subobject = 'ACCOUNTING'
desc = 'Stuff imported from legacy systems' ).
log->e( 'You see, what had happened was...' ).
BREAK-POINT.
LOG->POPUP( ).
and error is -
Error in the ABAP Application Program
The current ABAP program "ZCL_LOGGER====================CP" had to be
terminated because it has
come across a statement that unfortunately cannot be executed.
A function module was called incorrectly.
1 method POPUP.
2 * See SBAL_DEMO_04_POPUP for ideas
3
4   DATA: profile TYPE bal_s_prof.
5   CALL FUNCTION 'BAL_DSP_PROFILE_POPUP_GET'
6     IMPORTING
7       e_s_display_profile = profile.
8

CALL FUNCTION 'BAL_DSP_LOG_DISPLAY'
10     EXPORTING
11       i_s_display_profile    = profile
12       i_t_log_handle         = me->handle.
13
14 endmethod.

thank you for reply

The popup method has an error.
You can change its code to
method POPUP.

See SBAL_DEMO_04_POPUP for ideas
DATA: profile TYPE bal_s_prof.
CALL FUNCTION 'BAL_DSP_PROFILE_POPUP_GET'
IMPORTING
e_s_display_profile = profile.
data lt_handles type BAL_T_LOGH.
append me->handle to lt_handles.
CALL FUNCTION 'BAL_DSP_LOG_DISPLAY'
EXPORTING
i_s_display_profile    = profile
i_t_log_handle         = lt_handles.
endmethod.

With this change the method works for me.

Thank you for your comment,
İ had solved it before :)


While doing the assignment, I noticed that my train_dataset was getting trampled

My laptop loads the original data very slowly

Can one of the admins verify this patch?


fix: recv_fifo for asyncfl

Description
recv_fifo() takes first_k as one of its arguments to determine the first_k number of messages are received and returned.
To implement the functionality, asynchronous call
_streamer_for_recv_fifo() was implemented. However the function has a bug that is triggered in asyncfl. The function adds the first_k messages into an rx queue and re-adds the remaining messages to the head of each corresponding end id's rx queue. The remaining messages then processed in the next recv_fifo call. However, since _streamer_for_recv_fifo() is an asynchronous call, there is no guarantee that the messages saved in the previous call will be processed first. This causes some trainers blocked and made them fail to participate future rounds.
The issue is fixed as follows. Since recv_fifo() always returns first_k number of messages from the rx queue, there is no harm even if the remaining messages are added to the queue. By removing all the code to handle the remaining code, the code becomes now clean and easy to understand.
Type of Change

[x] Bug Fix
[ ] New Feature
[ ] Breaking Change
[x] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[ ] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ ] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[ ] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

Codecov Report

Merging #449 (fa494d7) into main (e175d42) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main     #449   +/-   ##
=======================================
  Coverage   14.82%   14.82%
=======================================
  Files          48       48
  Lines        2827     2827
=======================================
  Hits          419      419
  Misses       2379     2379
  Partials       29       29


golang.org/x/crypto version in go.mod has critical CVE

What is the issue?
Using grype on Ollama's binary (latest version) yields CVE-2024-45337: x/crypto/ssh: misuse of ServerConfig.PublicKeyCallback may cause authorization bypass
go-module: golang.org/x/crypto (v0.23.0) identified via go-module-buildinfo-entry
Whilst this is actually not regarded as an issue in the crypto library by the golang community , it still appears on widely used scanners as such and it would be nice to get fixed.
Mitigation is available in crypto library version 0.31.0
OS
No response
GPU
No response
CPU
No response
Ollama version
0.5.3

It will be helpful to get this fix, as the CVE vulnerability is being flagged in security scans.

Fixed in #8172


Terminal bell not availble in SSH mode.

VSCode Version: 1.43.0
OS Version: Windows10
Remote SSH OS: Ubuntu linux 18.04

Steps to Reproduce:

Open remote ssh.
echo -e "\a" # or echo -e "\07"
Should hear bell sound.

Does this issue occur when all extensions are disabled?: No(we use ssh mode)

/duplicate https://github.com/microsoft/vscode/issues/47711

@Tyriar No it's not the same issue.
I can hear the bell in none ssh mode, but not if I connected using Remote-SSH.
If a native shell, the bell is available.
If using a vscode shell, connect remote via ssh command, the bell is available.
Bell is not available only when connected using Remote-SSH.


Dynamic host name for different TLDs (i.e. ".io", ".com")

Question!
I am using the latest version of springfox (these days is 2.9.0). In my system, we have the backend services sitting behind a reverse proxy (pretty common) but the website is served through 2 TLDs, ".io" and ".com" (also sometimes common). Is there any way to dinamically associate the host name? Or, at least, the TLD? Otherwise, we will have to bind the doc to one specific TLD, which we can do, but we do not necessarily like :)
TIA

@bbossola you need to have your reverse proxies set the X-FORWARD headers, something like this
X-Forwarded-For: xxx.xxx.xxx.xxx #upstream ip
X-Forwarded-Host: something.io
X-Forwarded-Port: 4433
X-Forwarded-Proto: https


No node features

Hi -- I'm interested in using the GCN in a situation where the node's don't have any features associated with them (besides the label).  It seems like the thing to do is set featureless=True in the first layer in models.py, and then change the line in train.py to
model = model_func(placeholders, input_dim=adj.shape[0], logging=True)

This runs, but the performance isn't great.   Any thoughts?  I would expect this to perform somewhere in the neighborhood of deepwalk, but I may be misunderstanding the algorithm
Thanks
Ben

My bad -- I confused planetoid and deepwalk when I was reading the paper -- the results w/ featureless are actually roughly equivalent to the cited deepwalk numbers.  Regardless, I would expect GCN to do better than deepwalk, since the labels are available as the node embeddings are learned.  Perhaps the structure of these graphs are modular enough that that doesn't actually matter.

Thanks for your question. Note that by setting featureless=True, dropout will be ignored on this layer (as the input to the layer is bypassed altogether). This can significantly hurt generalization performance. You can instead simply pass an identity matrix as input if you want to train without features and use dropout at the same time.
I haven't run experiments without input features for this particular model, so I'm not quite sure what the optimal hyperparameter setting will be in this case and what kind of improvements you can expect over baselines such as DeepWalk.

Could you give a sample of trained data? Where do the features come from?

Suppose that I have:

an N by N adjacency matrix (N is the number of nodes),
an N by D feature matrix (D is the number of features per node), and
an N by E binary label matrix (E is the number of classes).

in text files.
How can I feed them into gcn?

The code in its current form expects the adjacency and feature matrix to be a sparse CSR matrix (scipy.sparse.csr_matrix) and the label matrix as a numpy matrix.


Improve the exception error message to better capture underlying issues.

Why:
Existing exception error messages often don't include any udnerlying issues, and they are not surfaced to user. For instance, some users and customers experienced error of "There was issue communicating with Salesforce" without any more details provided.

Fixed. Imported and use cask style

https://github.com/data-integrations/salesforce/pull/166
PR already merged.


Propagate SIGTERM and SIGINT to child processes

Looks good to me, anything to consider before releasing?

Hey,
Honestly I think this would just work :)

any update ?

Could you test if this has been fixed by installing 3.0.0-dev version: npm i -g<EMAIL_ADDRESS>?