Revive chain not working in some cases

When running docker network all the validators go offline (inx-validator), and the validator nodes are restarted.
After the first validator goes back online with IgnoreBootstrapFlag, the node it's connected to correctly revives the chain, but the other nodes don't handle that situation correctly and don't process blocks for some reason with the following log:
Once I left the network without acceptance for longer (30 minutes) and the above scenario worked.
docker-network-node-4-1            | 2023-10-26T13:56:27Z	DEBUG	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:98	BlockReceived, blockID: BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61), peerID: 12D3KooWRVt4Engu27jHnF2RjfX48EqiAqJbgLfFdHNt3Vn6BtJK
docker-network-node-4-1            | 2023-10-26T13:56:27Z	ERROR	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:115	ProtocolError, error: failed to dispatch block BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61): slot commitment CommitmentID(0xfbc2d398eb74464828eba6a08642df953037cc390f51e94c0562b21b6d641ea736000000:54) is not solid
docker-network-node-2-validator-1  | 2023-10-26T13:56:27Z	DEBUG	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:98	BlockReceived, blockID: BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61), peerID: 12D3KooWRVt4Engu27jHnF2RjfX48EqiAqJbgLfFdHNt3Vn6BtJK
docker-network-node-3-validator-1  | 2023-10-26T13:56:27Z	DEBUG	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:98	BlockReceived, blockID: BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61), peerID: 12D3KooWRVt4Engu27jHnF2RjfX48EqiAqJbgLfFdHNt3Vn6BtJK
docker-network-node-2-validator-1  | 2023-10-26T13:56:27Z	ERROR	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:115	ProtocolError, error: failed to dispatch block BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61): slot commitment CommitmentID(0xfbc2d398eb74464828eba6a08642df953037cc390f51e94c0562b21b6d641ea736000000:54) is not solid
docker-network-node-5-1            | 2023-10-26T13:56:27Z	DEBUG	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:98	BlockReceived, blockID: BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61), peerID: 12D3KooWRVt4Engu27jHnF2RjfX48EqiAqJbgLfFdHNt3Vn6BtJK
docker-network-node-3-validator-1  | 2023-10-26T13:56:27Z	ERROR	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:115	ProtocolError, error: failed to dispatch block BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61): slot commitment CommitmentID(0xfbc2d398eb74464828eba6a08642df953037cc390f51e94c0562b21b6d641ea736000000:54) is not solid
docker-network-node-5-1            | 2023-10-26T13:56:27Z	ERROR	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:115	ProtocolError, error: failed to dispatch block BlockID(0x371f26ae8e562f2159a0e11c9aab0b4063da8f609cdbe17e5429f9bb2dedb78b3d000000:61): slot commitment CommitmentID(0xfbc2d398eb74464828eba6a08642df953037cc390f51e94c0562b21b6d641ea736000000:54) is not solid
docker-network-node-1-validator-1  | 2023-10-26T13:56:27Z	DEBUG	Protocol	app@v0.0.0-20231020115340-13da292c580b/component.go:98	BlockPreAllowed, blockID: BlockID(0x86ac12a4398b4a68d70b8cc5ae5fb447cf55311129d0cc2bd6f169c8567548843d000000:61)

Turned out to be a shutdown problem in the Metrics module.


New common keyword 'Run And Watch Command'

Add a new keyword in Common.robot to run any command and watch its output.
This is extremely useful for those RF keywords that calls a sub-process
which runs more than few seconds, like calling an external shell script.
The 'Run And Watch Command' keyword arguments are:

Full command including args (single String).
Optional: Timeout in minutes: 10 by default.
Optional: An expected text: Verify a search phrase to exist in command output.

Here's a test where this keyword was used:
https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/cluster-operations/23/console
You can see that instead of just printing:
Uninstalling RHODS operator in selfmanaged
[Waiting for ever without any indcation what's going on]
Now the output, which is printed in realtime,
without the need to wait for the long process of RHODS uninstall to finish:
==============================================================================
Rhods Olm :: Perform and verify RHODS OLM tasks
==============================================================================
Can Uninstall RHODS Operator                                          Cluster type: selfmanaged
Uninstalling RHODS operator in selfmanaged
Watching command output: cd /home/jenkins/workspace/cluster-operations/23/ods-ci/rhodsolm && ./cleanup.sh -t operator
Shell process started in the background
.Removing RHODS... Be patient...
.No resources found
kfdef.kfdef.apps.kubeflow.org "modelmesh-monitoring" deleted
kfdef.kfdef.apps.kubeflow.org "monitoring" deleted
............................................Unable to connect to the server: EOF
| FAIL |
Error occured while running: cd /home/jenkins/workspace/cluster-operations/23/ods-ci/rhodsolm && ./cleanup.sh -t operator: 1 != 0
------------------------------------------------------------------------------
Rhods Olm :: Perform and verify RHODS OLM tasks                       | FAIL |
1 task, 0 passed, 1 failed
==============================================================================


DXCAM does not respond

i was testing dxcam with cv2 with this code
import dxcam
import cv2
camera = dxcam.create()

camera.start(target_fps=175)
for i in range(100):
    image = camera.get_latest_frame()  # Will block until new frame available

    # Display the image
    cv2.imshow("Image", image)
camera.stop()

and when i the screen pops up its grey and tells me that its not responding
it tells me that the fps was 40

Please try https://github.com/lovettchris/wincam.  If you have a good system that can capture in under 4ms you should be able to reach 250fps, but perhaps only on a smaller cropped area of the screen, not full screen.


Linear gradient: support all angles

Relates to #11
I hope this is ok, but is there a reason to start in (0.5, 0)? If not to simplyfy math, I'm starting in (0,0).
Also for this logic to work i assume that 0deg is equal "to right". I can rotate everything by 90deg so default 0deg would be "to bottom" like in original file.
Tested it in console and should work fine I think. I did not run tests by npm test nor in sketch (working on windows).
@kdzwinel Is precision of .toFixed(2) ok?

Hey! Thanks for taking time to work on this. @matzimowski and @coderitual are now running this project so I'll leave it to them to do the review, but yeah, no idea why this 0.5 was hardcoded there.

Thanks @kdzwinel ! @matzimowski @coderitual Please let me know where to push this feature.

thanks @tlenex for the PR :) I I need to prepare some simple examples to test it out before we merge. Our e2e tests are still a bit cryptic to me and I am not entirely sure if new values are still correct. Or maybe you have some files on which you have already tested these changes?

Nope, I don't have Skech files to test, nor Sketch on Windows. Is't just math challange that I wanted to solve.

oh I see, no worries I will prepare something but it will take some time. If the result will be fine we gonna change those e2e expected values.

@coderitual Already did :)

@tlenex Awesome, let me do checking this week and we're good :)

hi @tlenex . This is how visual tests look so far:

is simple one without any angle defined (works fine on your branch and master one)

Uses turn instead of degrees. Stops are not translated properly here (not sure if they worked before) and gradient position is off.

Uses 2 colors and custom 336deg angle. Colors looks fine but position of gradient is off.

Just to have more advanced example with 3 gradients. This didn't work previously and still is an issue (we don't need to resolve it imho)

Would be nice if we can resolve 2 and 3. At leas gradient positioning. I will test multiple stops separately. Wdyt?

Hmm... what is x and y then in from and to? I've assumed that it is position starting in top-left corner used as center of x-y axis and retrieve the angle from it.
But it seems like they are positions from the bitmap, maybe in percents? Additionally x and y seems to be inverted somehow.
1) convertAngleToFromAndTo('180deg') = {from: {x: 0, y: 0}, to: {x: 0, y: 1}}
2) convertAngleToFromAndTo('0.4turn') = {from: {x: 0, y: 0}, to: {x: -0.41, y: -0.91}}
3) convertAngleToFromAndTo('336deg') = {from: {x: 0, y: 0}, to: {x: 0.59, y: 0.81}}

Case 4 touches multiple background support.
@coderitual I need to investigate it. It would be helpful if you could show me screenshots of gradient positions images in sketch for 0deg, 90deg and 270deg, so I can make predictions what to correct in my math.
// cases to fix probably
convertAngleToFromAndTo('0deg') = {from: {x: 0, y: 0}, to: {x: 0, y: -1}}
convertAngleToFromAndTo('90deg') = {from: {x: 0, y: 0}, to: {x: 1, y: 0}}
convertAngleToFromAndTo('270deg') = {from: {x: 0, y: 0}, to: {x: -1, y: 0}}

hey @tlenex . The whole algorithm is "a bit" more complicated. First we need to take a look on how css gradients interpolation works.
I prepared rectangles with big aspect ration to better illustrate this.

as you can see even with a big aspect ratio, we have gradient centered and interpolated in expected way.
if we put gradient line in sketch in a naive way we would lost the angle:

to keep angle correct we need to move points of gradient section to the center. But again we cannot do it in naive way:

To make it work correctly we need to calculate in the same way browser does. There are 2 algorithm. The current one and already adopted is the following:

Let's apply that into sketch:

After exporting this from sketch we can see how from and to were defined:

So the findings are as follows:

to calculate correctly we need to consider polygon and position of its points
if we have different shape than rect we need to calculate virtual bounding box around the shape (to be confirmed but it looks like that)
sketch coordinate system is between 0 and 1 (so again we need to normalize from shape coordinate system to 0 - 1)

I think that it's gonna be a lot of work, but I am happy to help you further or we can limit the scope of this PR to only support perpendicular gradient lines + different units.
Also I think It would me much easier for you if you have any access to sketch (maybe hackintosh on vm?)
Reference:
https://www.w3.org/TR/css-images-3/

Thanks it seems to be clear for me now. I'll resume my work on this in next week I think.

@coderitual please check this code. Im not sure about width and height - did I get correct values?

hey @tlenex, thanks for fixes and sorry for late response.
I prepared codesandbox where a build from your pr is used so we can easily see what's working and what's not.
It looks that for degrees it is working properly 👏

Let me check if other units are also handled in a right way.

@tlenex, for other units also works correctly. Nice work!

What is not working: more than one stop but I think this can be handled on different pr :)

Thanks :)
I'm fine with another PR for steps support. Just to keep a reminder with all my findings I'll repost link to this comment:
https://github.com/html-sketchapp/html-sketchapp/pull/208#issuecomment-704610035
Meanwhile I'll try to look at radial-gradient.

@tlenex definitely. I am still happy that the tool is able to cover more use cases now. Even though internally we don’t use gradients.
Regarding general structure of the project and possible refactoring I think the best moment to do that is when new features come. I have quite limited capacity atm and different things to implement in the first place (new features in sketch related to design systems) but I am happy to help you with any further work.
Thanks!


It takes 24 hours to train 10 epoch, about 246,080 pictures, is there something wrong, or it does so?

parameter of the computer:
GPU:1*1080Ti,  12GB
parameter of the train code:
crop_size:128*128
batch_size:16

Hi,
What dataset do you train?
Actually, I trained with a single GPU (2080TI), it takes about 20s for each epoch in the Part A dataset. Hence, I think it is impossible to only train 10 epochs with 24 hours, even in 1080TI.

The dataset names DroneCrowd, which contain 24,600 pictures in the train dataset. I think it's difficult to train this datasets with HRNet, it takes me a long time!
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2021年6月23日(星期三) 下午5:10
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [dk-liang/FIDTM] It takes 24 hours to train 10 epoch, about 246,080 pictures, is there something wrong, or it does so? (#17)
Hi,
What dataset do you train?
Actually, I trained with a single GPU (2080TI), it takes about 20s for each epoch in the Part A dataset. Hence, I think it is impossible to only train 10 epochs with 24 hours, even in 1080TI.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This is a huge dataset. I think you can try to utilize the VGG16 as the backbone.


https://master.tus.io/files/ returning 500

Describe the bug
Some time ago I was using Uppy with https://master.tus.io/files/ as Tus endpoint for university project and it was quite ok. For now it returns 500 with message saying there is some problem with s3 storage:

You can also reproduce it on your demo page: https://tus.io/demo.html. Just try to upload something.
To Reproduce
Steps to reproduce the behavior:

Make POST request to https://master.tus.io/files/
See error

Setup details
Please provide following details, if applicable to your situation:

Used tusd data storage: AWS S3 on https://master.tus.io/files/
Used tus client library: Uppy.js

Cool, thank you :)

Works for now, closing

Thanks for reporting. We fixed our deployment in the meantime, as you noticed :)


Can't add custom command path

Sorry for my English...
I can call addCommandsPath() in the Controller, but there is bug in Longman\TelegramBot\Telegram.php method getCommandObject() at line 243:
$command_namespace = __NAMESPACE__ . '\\Commands\\' . $auth . 'Commands\\' . $this->ucfirstUnicode($command) . 'Command';
NAMESPACE return only "Longman\TelegramBot".
NAMESPACE  never return my commands path.
So new Commands can be executions only they are located in a system folder like Longman\TelegramBot\Commands\SystemCommands.

All commands need to belong to the correct namespace.
Your custom commands classes need to extend the type of command they are.
Look at the examples/commands folder for examples.
e.g. MarkdownCommand.php

Thanks a lot, it's work!
But it is not logical. MarkdownCommand.php should have namespace examples\commands, not Longman\TelegramBot\Commands\UserCommands.
I thought we adhere to the standard PSR-0

The example commands are intended for copy-pasting, so that the namespaces are already set correctly 👍


Fix typo in docs

@arcz please sign the CLA

I happened to have fixed this typo in a separate PR so we can close this out. Thank you for bringing it to our attention.


Specify repository for specific packages.

I have a weird issue with proof general where I need to install it from melpa but not melpa-stable (because that version is 4 years out of date). It would be nice if we could override our package priorities for specific packages.

I second this idea, it would help quite a lot. (One anecdotal example: lsp-mode has been broken for a long time since flycheck hasn't released a stable version for nearly three years, but melpa (vs. melpa-stable) is working fine.)
One idea for how to implement this is to adjust the form of kotct/dependency-list a bit. Now, instead of just a list of symbols, kotct/dependency-list could be a list of both symbols and structs (?)—something like (just an idea):
'(
  linum-off ; just follow the defaults; prefer the default repository and latest version
  ; ...
  (flycheck :repository "melpa") ; configure the repository
  ; ...
)

(This structure would lend itself to extension with more keywords like :version later on.)


[BUG]: Improve Search-Algorithm

Expected Behavior
Searching for "John Doe" or "Doe John" should return the same result.
Actual Behavior
The algorithm distinguishes the order of words.
Steps to Reproduce the Problem

Create new User "John Doe"
Search for "John Doe"
Search for "Doe John"

Critical Bug?

[ ] Yes
[X] No

More Information
N/A

Closed: https://github.com/BerkeAras/WorkGroup-API/pull/7


Cache repeated COLR subtables (2)

Some improvements to #3215 to use pickle (faster than json), sha256 for the digest to mitigate collisions, and a LRU cache to avoid it growing unbounded.
I'm still unsure regarding the cache max size (I used 128 like lru_cache but don't know what a good number is in general for this task).
I'm also hesitant to enable this unconditionally and would rather it be opt in, as it's not guaranteed to lead to faster builds given the overhead of pickling+hashing objects mutliple times (the TableBuilder.build is called recursively for each struct fields in a table tree).

Using paintcompiler on Bitcount:
FontTools main: 2:19.07 (peak memory usage 4.8G)
cache-color-builds: 54.015 (peak memory usage 236M)
cache-color-builds-2: 2:41.66 (peak memory usage 5G)
Probably better to do nothing than to do this.

well.. that's strange, how come your branch where we don't manage the cache at all and let it grow unbounded actually reports using less memory? How did you measure?

also note that #3215 has a bug which I fixed in 259e013, the cache key must also contain the class itself otherwise two objects with distinct classes but same data can be confused

Don't know. The TTX is different between the master build and my build, but visually it's the same. I'm measuring the memory footprint with "top", which I know isn't completely accurate, but it's not going to be an order of magnitude wrong either.

if the ttx is different it must be a bug, because this optimization is only meant to speed things up but the end result ought to be identical


[type: refactor] Minor code optimizations

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3086 (b183bb9) into master (c6c623a) will decrease coverage by 0.43%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #3086      +/-   ##
============================================
- Coverage     58.98%   58.55%   -0.44%
+ Complexity     4396     4358      -38
============================================
  Files           694      694
  Lines         19933    19931       -2
  Branches       1800     1799       -1
============================================
- Hits          11758    11671      -87
- Misses         7175     7262      +87
+ Partials       1000      998       -2

Impacted Files
Coverage Δ

...plugin/httpclient/config/HttpClientProperties.java
0.00% <ø> (ø)

...che/shenyu/plugin/metrics/constant/LabelNames.java
0.00% <ø> (ø)

...n/metrics/prometheus/PrometheusMetricsService.java
53.84% <100.00%> (+3.84%)
:arrow_up:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

...admin/controller/ShenyuHttpRegistryController.java
77.77% <0.00%> (-22.23%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
69.10% <0.00%> (-14.64%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
42.85% <0.00%> (-12.70%)
:arrow_down:

...in/disruptor/RegisterServerDisruptorPublisher.java
52.94% <0.00%> (-11.77%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.12% <0.00%> (-11.71%)
:arrow_down:

... and 29 more

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 c6c623a...b183bb9. Read the comment docs.


Inject Source does not work

Steps

Create new Xcode master-detail application
Add main.m, add it to target
Run the app on iPhone 7 Plus simulator
Open Injection, select macOS Project -> Patch, it generates something in the main.m file

// From here to end of file added by Injection Plugin //

#ifdef DEBUG
#define INJECTION_PORT 31452
static char _inMainFilePath[] = __FILE__;
static const char *_inIPAddresses[] = {"<IP_ADDRESS>", "<IP_ADDRESS>", 0};

#define INJECTION_ENABLED
#import "/tmp/injectionforxcode/BundleInjection.h"
#endif

Run the app again. In the console says

2017-01-13 10:46:44.027 Test Injection[22173:214386] +[BundleInjection load] <IP_ADDRESS> (see project's main.(m|mm)
2017-01-13 10:46:44.030 Test Injection[22173:214424] Injection attempting connection to: <IP_ADDRESS>:31452
2017-01-13 10:46:44.034 Test Injection[22173:214424] Connected to "Injection" plugin, ready to load x86_64 code.

Change some code in detail view controller

override func viewDidLoad() {
    super.viewDidLoad()

    view.backgroundColor = UIColor.blue // try changing to red, green, blue colors, ...
  }

Select Injection -> Injection Source
Nothing happens 😢

It should show something like
Compiling ...
2 injections performed so far

Please help if I miss something. Thanks

Did you reload the view controller so the new code is executed?

@johnno1962 as you know in the Xcode template master detail app, I have the detail controller that I can push and pop. So when I change its code, I click back and click on a cell to go to it again

It should say more in the console so something is not working. If you want to TeamView send your co-ordinated to injectionapp at johnholdsworth.com


Feature request: Option to use individual password to protect the app

I suggest to include the option to define an individual password/PIN to protect the app. Currently, (v0.2.4 F-Droid) only the phone credentials can be used.

Agreed the use of the system password negates any benefits.  If the idea is to protect the tokens from an attacker, if they already know the system password they can open andOTP
The GuardianProject's various libraries would be perfect use for this.   SQLCipher and IOCipher would provide secure and password protected storage.   Using PanicKit would allow for destruction of the tokens / container in extreme situations
https://guardianproject.info/code/

Just to clarify: the system password is in no way used to encrypt the data of andOTP.
It is only a check before starting the app to add a little protection from someone who just picks up your unlocked phone.
I am not gonna change the way the data is stored, which is in the Android KeyStore.
However I'm gonna add the option to unlock the app with a custom password instead of the device credentials.
If you want your data to be encrypted with a custom password there is already a fork of OTP Authenticator out there which does that (OneTimePad).
Sent from my Google Nexus 5X using FastHub

6 digit pin would be awesome. Anyway this has to be implemented so as to avoid someone that has your phone to access your OTPs. You can also export these unencrypted which makes it a necessity.

Done, in the next release you will be able to set an individual password or PIN to unlock the app.


PieChart plot alignement

I am rendering multiple PieChart into a pdf document, and I am trying to have them aligned :
Graph background is in gray, plot and legend background in white.

Is there a way to specify a Plot width, a margin left, or to change the default alignement ?

@zorator
Set legend's position:
chart.getStyler().setLegendPosition(LegendPosition.InsideNE);

Set the size of the drawing area to avoid overlapping with legend:
chart.getStyler().setPlotContentSize(0.8);

Hi, thanks for the tip, but the plot is now too small...
I ended-up printing the chart in two images (one without legend, and one without plot and title) and then joining them in the pdf file.


Custom Dialog ignoring Accent color

Custom Dialog ignoring Accent color.
Steps to reproduce the behavior:

Go to 'Dialog'
Check "Use accent"
Open Customer Dialog
See error

Custom dialog should use accent color.

MahApps.Metro version [v2.0.0]

@Tim Okay, might be true.
Thanks again
KKauK

@KKauK this has now been fixed, and it will be released in the next version of MahApps.Metro v2.0


Add hyperterm-chester-dark theme

Checklist:

[x] The title for my package or theme uses its npm title (hyperterm-plugin-example)
[x] The link for my package or theme uses the npmjs.com link (https://npmjs.com/package/hyperterm-plugin-example)
[x] There is a visual representation of what my plugin does in the repo. (Plugin: screenshot or gif || Theme: screenshot of the colors within HyperTerm)
[x] VERY IMPORTANT: I've written a short (one sentence) description for my package or theme of why it's awesome and deserves to be on the list.

Could you please add a slightly more descriptive description of why you think hyperterm-chester-dark is an awesome theme?
Just asking, as awesome-hyperterm is intended to be a curated list of the best HyperTerm packages, themes, and resources available, so we want to make sure the package, theme, and resource list items say why we think they're the most awesome!

Hey @evgenTraytyak, could you please add the theme to the bottom of the list? Thanks!

Hey @evgenTraytyak - closing this for now. If you're able to go through and resolve the aforementioned requests, feel free to update the README.md and re-open. Thanks!


Handle vector-image-converted text in PDFs

Requested feature

Our users encounter from time to time documents that instead of text have vector path's representing themselves as text.
Because of the vector nature of it, we do not automatically OCR such pages, and because there is no actual text in such PDF pages, conversion output is usually empty.
Alternatives

As a solution we need to reliably detect such cases, and render such pages into raster images, then running them through standard OCR pipeline. We can use layout model output to detect such cases, if layout model predicts text blocks that don't have programmatic text associated with them, this could be a good indication.

Hello,
I am facing this exact challenge, I believe.
I have a pdf file with some titles in some sort of "image" format that does not seem to be captured by the OCR, and is therefore not presented in the markdown text output.
I'll try to find the file and post it here with the extraction output.

Hi @maxmnemonic I am facing a similar issue but the behavior is different for markdown export and text export, while converting the following document...
Could you please explain what might be happening for the inconsistent results for different file types.

@deborah-drongoai, I believe this could be a different issue, but we can check it up, are you converting it from PDF, any chance you could share it with us?

@maxmnemonic Thank you for your quick response, sure I will attach the pdf file that I used to work with the module.
rajesh_1026319_20240917055115_stationerypdf_oh_merged.pdf
It would be of great help if I could get some insights with this issue. Thanks of lot.


NimLime creates a whole menu just for itself

It is present even when I am not writing Nim, and it doesn't contain anything but a link to this Github page. Is there a way to turn it off?

Yeah, this should not be there. I think we should get rid of it.

I had some thoughts that other things could be added to it, but of it bothers you that much, I guess the entry could be moved to preferences.
I'd just prefer that a link to the issues page was displayed somewhere, to try and encourage users to actually report problems.

It makes sense! Just, a top level menu visible for all types of files seems like overkill.

@mcclure I've placed the menu in the preferences section for now. (although I wonder if I could create a top-level menu conditionally).


Fix [GitHub Issue #1451] Canabalt [ARM-x86]: blocked at splash screen

The issue here is that when a sublayer is removed from super, we need call setNeedsLayout which will flush out corresponding transactions to update the view hierarchy.
This is based on the observation from reference platform when removing layer. The following call sequence is happening.
[subLayer removeFromSuper] triggers the following call sequence
[CALayer setNeedsLayout] -> [CALayer drawInContext] ->[UIView drawRect]

This change is

CALayer* pSuper = (CALayer*)priv->superlayer;

nit: can we remove the hungarian while we're here?

Refers to: Frameworks/QuartzCore/CALayer.mm:809 in 8cdcd27. [](commit_id = 8cdcd27fe340aa879f54db9b7e36d67169927cb2, deletion_comment = False)

:shipit:


[$250]  LHN - RBR stays for paid report

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.27-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/cases/view/3266257
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Precondition: control ws exist with enabled categories, Tags and Taxes. Enable toggle 'Members must categorize/tag all expenses'.

Employee: submit an expense with violation (without category and tag) in ws chat.
Admin: approve or pay the report with violation.

Expected Result:
Employee: report doesn't have a RBR in the LHN and isn't pinned.
Actual Result:
Employee: report still have a RBR in the LHN and isn't pinned.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/4ed69491-f663-4e83-b488-26731680c9d5

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021832065922256660213
Upwork Job ID:<PHONE_NUMBER>256660213
Last Price Increase: 2024-09-13

Issue OwnerCurrent Issue Owner: @mkhutornyi

We think this issue might be related to the #wave-control

This should be fixed in BE. If you clear cache the RBR disappears so the BE should properly clear violation data on pay. Although we could do it optimistically for payer side the same clearing should be applied for other users too so it should be done in BE altogether.

Proposal
Please re-state the problem that we are trying to solve in this issue.
LHN - RBR stays for paid report
What is the root cause of that problem?
In all MoneyRequestPreview, MoneyReportView and LHN RBR we are showing RBR whenever there is a report or transaction violations
https://github.com/Expensify/App/blob/e60b8210a1f5a2471d8aa9d84d0e461774b791d6/src/components/ReportActionItem/ReportPreview.tsx#L168-L170
https://github.com/Expensify/App/blob/e60b8210a1f5a2471d8aa9d84d0e461774b791d6/src/components/ReportActionItem/MoneyRequestPreview/MoneyRequestPreviewContent.tsx#L112
https://github.com/Expensify/App/blob/e60b8210a1f5a2471d8aa9d84d0e461774b791d6/src/components/LHNOptionsList/OptionRowLHNData.tsx#L40
What changes do you think we should make in order to solve the problem?

We should only display RBR when violations exist if the iouReport is not settled (or not approved if we choose to do so).
Therefore, in all the cases listed above we need to add a condition of !ReportUtils.isSettled(. We can centralize the code by creating new functions to add these conditions or we can update all TransactionUtils.hasViolation, ReportUtils.hasReportViolations and ReportUtils.hasViolations to check for isSettled of the linked iou report.
What alternative solutions did you explore? (Optional)

Hm, I'm curious for where this regressed as I'm sure it has been discussed before. CC: @JmillsExpensify @cead22
Moving to #wave-control project anyway, as a violations bug.

Looping in another BZ member as I am going OOO

Here is a test branch @mkhutornyi

@FitseTLT from what I understand, the functions you mentioned in the proposal have never had code to check if a report is settled, so that check has to exist somewhere ele, and I think it works for some cases. Can you share where that check is, and why it works for some cases and not others?

Can you share where that check is, and why it works for some cases and not others?

@cead22 the only FE check we are including is for transaction thread violations here
https://github.com/Expensify/App/blob/fc33ac1ac35248ecf888bb8d4e8e04b5e72cc56e/src/libs/ReportUtils.ts#L5897-L5899
But the inconsistency for other cases comes from the BE: sometimes the BE clears all the violations (transactionViolations and reportViolations) on payment and sometimes it doesn't.

If the backend doesn't clear the violations, and they show when the report is settled, then we're going through a flow that doesn't go through the code you linked. What flow is that?
Can you share the flows for which the backend isn't updating violations correctly?

If the backend doesn't clear the violations, and they show when the report is settled, then we're going through a flow that doesn't go through the code you linked. What flow is that?
Can you share the flows for which the backend isn't updating violations correctly?

Ok @cead22 the flow is here in report preview
https://github.com/Expensify/App/blob/e60b8210a1f5a2471d8aa9d84d0e461774b791d6/src/components/ReportActionItem/ReportPreview.tsx#L168-L170
and here in money request preview
https://github.com/Expensify/App/blob/e60b8210a1f5a2471d8aa9d84d0e461774b791d6/src/components/ReportActionItem/MoneyRequestPreview/MoneyRequestPreviewContent.tsx#L112
Here is a demo vid: after paying the RBR remains and when clearing cache the RBR will disappear from report preview but after opening the transactions the RBR reappears because the transactionViolations are fetched by then.
Uploading 2024-09-17 19-11-14.mp4…

From @nkdengineer in https://github.com/Expensify/App/issues/48397#issuecomment-2362825011

@cead22 I think we should clear the hold violation in optimistic data if we paid full instead of adding !isSettled condition.

@FitseTLT thanks for that

The red circle on showing on the report preview is working as intended, because the report has violations and we show that no matter what the state of the report is.
The green circle on the LHN showing for that report might be working as intended, assuming we decided the green circle that is shown on reports with pending actions -- in this case paying, should take precedence over the red circle in the LHN that we show on reports with violations. Can you confirm @JmillsExpensify?
After paying, the red circle should remain in the report preview, for the same reason as in the first bullet
After clearing the cache, the report preview isn't showing the red circle, and after clicking on the preview, the transaction previews aren't showing the red circle. These are bugs because they should be showing for the same reason as in the first bullet

@JmillsExpensify did I get this right or has this changed from what I remember?

From this comment, seems part of the behavior is expected. Closing it out for now, but we can re-open if it's reported again as an issue


API_ENDPOINT is a misnomer

In workshop2 utilities.js, there's a constant const API_ENDPOINT = "http://catbook-workshop2.herokuapp.com";
I think this is an incorrect/misleading name, since the catbook website itself is not an API endpoint.
This is especially confusing when there's expressions like API_ENDPOINT + endpoint
Something like BASE_URL might be more accurate

donezo


source: csv: Enable setting repo src_url using CSVSourceConfig

This is a fix for issue #48. Here I've included CSVSourceConfig separately so that FileSourceConfig remains undisturbed with respect to addition of key parameter.

Codecov Report

Merging #102 into master will increase coverage by 0.17%.
The diff coverage is 98.14%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   92.64%   92.81%   +0.17%
==========================================
  Files          49       49
  Lines        3223     3272      +49
  Branches      373      375       +2
==========================================
+ Hits         2986     3037      +51
+ Misses        183      182       -1
+ Partials       54       53       -1

Impacted Files
Coverage Δ

dffml/source/csv.py
97.26% <100%> (+6.69%)
:arrow_up:

tests/source/test_csv.py
97.36% <96.77%> (-2.64%)
:arrow_down:

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 e479a4e...9166b62. Read the comment docs.


Vulkan: Non-directional lights with size > 0 shining on material with zero roughness results in broken specular lobes

Godot version
v4.0.alpha16.official [86dd3f312]
System information
Arch Linux, Vulkan Clustered, Nvidia 1660(Nvidia open driver 515.65.01-11)
Issue description

Note: issue is more visible when the camera is close to the specular lobes.
Steps to reproduce

Add a mesh and assign it a material with zero roughness. Currently only tested with a sphere.
Add an omnilight with size > 0 and shine it on the mesh

Minimal reproduction project
Lighting problem.zip

There's likely a NaN or infinite value being generated as a result of a division by zero or something similar in the shader.


How To Specify Ordering of Stages / Jobs

When multiple stages / jobs are configured to run in parallel (dependsOn: []), their ordering in the ADO UI is indeterministic.  Is there a way to order stages / jobs?  neither ordering them in the YAML template nor alphabetizing seems to work.
When there are later stages dependent on these jobs, the lines end up getting crossed and it becomes difficult to determine what is dependent on what in the UI

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

ID: d322215c-8025-4f21-0700-7dfa7dc5c46e
Version Independent ID: 141fcdbb-8394-525b-bb29-eff9a693a9c4
Content: Stages in Azure Pipelines - Azure Pipelines
Content Source: docs/pipelines/process/stages.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the  documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps Support Bot
Azure DevOps Developer Community.


Custom Pangenome Allele Database

Nice tool!!! I also have a similar idea for developing such a tool to genotype all genes witch cnv for large cohorts. The problem is you used HPRC etc. to construct a database, how can I construct a new one with my own assemblies (i.e. my own pangenome) to genotype for myself. Thank you very much!

Hi Yeeus,
Thanks so much for reaching out and for the kind words.
For the reproducibility, please check my other repository here: PATs repository. It includes the codes and tools I used to build the database.
However, given the code complexity and workload involved, constructing a portable compilation pipeline for other pangenomes would require significant time and effort, which could impact my progress on other ongoing projects. As such, it would require approval from my PI<EMAIL_ADDRESS>Please email him and provide more details about your custom pangenome, your study objectives, and what exactly you need from us.
In the future projects, we may consider collaborating with other teams to develop an efficient and portable soltware for constructing databases for any pangenomes, but keep in touch.
Best,
Walfred

Thank you I will have a try. I do like to work with someones who have the same interest with me and as you said keep in touch please.
Another question, as I know, assemblies from CPC are unavailable to get, so how do you get them?

Thank you I will have a try. I do like to work with someones who have the same interest with me and as you said keep in touch please.
Another question, as I know, assemblies from CPC are unavailable to get, so how do you get them?

Hi Yeeus,
Please read their paper, and I believe they do have a public availability download links.
Best,


Allow package maintainer register an external URL instead of upload real wheel file

What's the problem this feature will solve?

Nowadays, lots of packages contain large GPU code binaries that may take hundreds of megabytes. As CUDA11 has released recently, we found our package size increased from 850MB to 2.3GB, we have to request a larger limit again and again
(see https://github.com/pypa/pypi-support/issues/289 https://github.com/pypa/pypi-support/issues/767 ). This lead to 2 problem

spent a great amount of bandwidth and storage from PyPI
have to delete all old version for more space, this means no history version on PyPI

Describe the solution you'd like

Package maintainer should serve the wheel file on external service (CDN or S3)
PyPI should allow the package maintainer to register an external URL & checksum, which will be looked like a normal release for users
pip should follow the external URL and crawl real wheel package from it; validate the checksum for security reason

Additionally, to solve the history version problem:

PyPI should allow the maintainer to update an existing URL in order to fix a dead link
PyPI should allow the maintainer to convert a normal file into an external URL (for saving space)
Maintainer must not change the checksum for an existing item

Additional context

This solution provides a better user experience comparing to ask users to add -f https://.... after pip install
Another potential way to solve this problem is addressed in  https://github.com/pypa/warehouse/issues/7852

Thanks for raising the issue.
This used to be a feature of PyPI, and was removed. The reasons why are documented in https://www.python.org/dev/peps/pep-0470/ and still stand today -- ultimately, while it may be easier for maintainers in the short term, it's not very user-friendly in the long term.
We're generally able to raise limits for projects like megengine and are happy to help explore other options for reducing distribution size, but we're not likely to reverse PEP 470 anytime soon.


Feat: Added structs and classes!

Users can now define structs (data only) and classes (data + functions) and use them in code.
Everything is stack based, and the location of the class variable is always set as the first argument in the functions.
Two examples added to demonstrate
Struct Test and Class Test

Ok, Github doesn't reference the files correctly. The file paths are
Struct Test: Examples/Structs/struct-test.tree
Class Test: Examples/Classes/class-test.tree


improment: ZENKO-749 removed stretch from aws-node-sdk object test

Remove stretch from aws-sdk-node object test
This needs to be done so that we can add E2E tests for zenko. Currently we do not support stretch envrionment.

Hello anurag4DSB,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get
information on this process.
Status report is not available.

Integration data created
I have created the integration data for the additional destination branches.

this pull request will merge improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests into
development/8.0
w/8.1/improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests will be merged into development/8.1 (pull request #1381)

The following branches will NOT be impacted:

development/6.4
development/7.4

Follow integration pull requests if you would like to be notified of
build statuses by email.

Waiting for approval
The following approvals are needed before I can proceed with the merge:

2 peers

Peer approvals must include at least 2 approvals from the following list:

@RachedBenMustapha

@rahulreddy

@dora-korpar

History mismatch
Merge commit #7198651bbd74c95285b971d3b9bb92061e34e0f1 on the integration branch
w/8.1/improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests is merging a branch which is neither the current
branch improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests nor the development branch
development/8.1.
It is likely due to a rebase of the branch improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests and the
merge is not possible until all related w/* branches are deleted or updated.
Please use the reset command to have me reinitialize these branches.

@bert-e reset

Reset complete
I have successfully deleted this pull request's integration branches.

Integration data created
I have created the integration data for the additional destination branches.

this pull request will merge improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests into
development/8.0
w/8.1/improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests will be merged into development/8.1 (pull request #1383)

The following branches will NOT be impacted:

development/6.4
development/7.4

Follow integration pull requests if you would like to be notified of
build statuses by email.

Waiting for approval
The following approvals are needed before I can proceed with the merge:

2 peers

Peer approvals must include at least 2 approvals from the following list:

@RachedBenMustapha

@rahulreddy

@dora-korpar

History mismatch
Merge commit #4b795c8cd4680be0ad72689ece419f579e29efae on the integration branch
w/8.1/improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests is merging a branch which is neither the current
branch improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests nor the development branch
development/8.1.
It is likely due to a rebase of the branch improvement/ZENKO-749-remove-stretch-from-aws-node-sdk-tests and the
merge is not possible until all related w/* branches are deleted or updated.
Please use the reset command to have me reinitialize these branches.


Timesheet - 26/04/2021 - TristanS

Task 1
Updated Audit workflow
Proof of Work
Change .zip to .drawio to verify. Open file on this site: https://app.diagrams.net/

Bounty Account Workflow - April 26 Update.zip
Time Sheet Workflow - April 26 Update.zip

Time Spent
0.5 hours

Task 2
Worked on crypto 101 blog.
Proof of Work
https://docs.google.com/document/d/1ANY7zuvKkeZGNfUF5KLZ0YqSIp39p-rHlIhnlZU0XK0/edit
Time Spent
1 hour

Task 3
Attended Marketing Meeting
Proof of Work
https://docs.google.com/document/d/1RpCYhe6gcoDb1h7Wp3xZqYvcbdJ49VwXGKvp8hDjGqs/edit
Time Spent
0.5 hours

Total Time Spent
2 hours
Account Number
6d4e7ec412472b5cfabb234555bc37f9a5a1a75a4926f9fee2a54d3e070f4b3f

Approved


Patch to add class .toc-h1, .toc-h2, etc to TOC

The following patch adds a class to the TOC "a" tags so we can use CSS to control indent.
This is against the original, not "-plus"
--- vue-markdown.js.orig	2020-07-14 15:28:15.698495629 -0700
+++ vue-markdown.js	2020-07-14 22:32:31.651135683 -0700
@@ -10704,9 +10704,16 @@
tocMarkdown = generateTocMarkdownFromArray(tocArray, options);
    tocTokens = markdownItSecondInstance.parse(tocMarkdown, {});

  // Adding tocClassName to 'ul' element
  if (_typeof(tocTokens[0]) === "object" && tocTokens[0].type === "bullet_list_open") {

    for (var i = 0; i< tocTokens.length; i++) {

       if (tocTokens[i].type === 'list_item_open') {

           var attrs = tocTokens[i].attrs = tocTokens[i].attrs || [];

           var lev = tocTokens[i].level;

           if (lev>1) lev = Math.floor(1+(lev/2));

           attrs.push(['class', 'toc-h'+lev]);

       }

     }
    var attrs = tocTokens[0].attrs = tocTokens[0].attrs || [];
    attrs.push(["class", options.tocClassName]);
  }

FYI, given the limitations of TOC this patch is probably not really all that useful.

Here's a quick suggestion for you to keep in mind next time you write a patch. It will be nice if you create a pull request rather than posting code on an issue. If we decide to incorporate your code, it would be less work for both the contributor and the reviewer.

Thanks for the advice.  As you can probably tell I'm new to git and github.
Anyways, as I mentioned this patch was against "vue-markdown" because the "vue-markdown-plus/dist/" is minified.  And I am using the browser directly, not node or php.
I pasted the patch as I saw no "attach file" button, and so followed what it said at the bottom of the comment textarea:
"Attach files by dragging & dropping, selecting or pasting them"

Of course as it turns out to be the "attach button" in camouflage.


Ability to trigger a recipe cleanup programatively

There is currently no implementation allowing a manual cleanup of a recipe execution, like for example
jbaker.cleanUp(processId)
or
jbaker.delete(processId)
or
jbaker.end(processId)
The possibility is to apply on the recipe
.withRetentionPeriod(Duration.ofDays(n))
In our use case of Baker, we would like to trigger ourselves the cleanup of the recipe executions that reached a  state that we consider as "final", at a given time scheduled in a batch cleanup mechanism and not based on an expiration period.
As discussed in a question with Merlijn, this could be done without too many issues as

This functionality is already there since it is used internally in the form of 'retention period'.

Yes, it's a nice feature that I guess is sort of expected for a process orchestration a library.
One consideration is running interactions.
Maybe optionally stop them gracefully? For the retention period we currently force stop them.


Marker label position

Description
This pull request allows the users to choose where the marker label should be rendered inside the Chart.
Why?
Sometimes it's a requirement from the design team to position the label of the marked in a specific place inside the chart. Mine was to show the labels close to the marker indicator.
How?
First, I created a sealed interface with two implementations to represent where we want to render the label. The Top implementation represents the previous behaviour of showing the labels always on top. The AboveIndicator, as the name suggests, represents the behaviour of rendering the label above the marker's indicator. I've added a spacingDp argument to add a bit more space between the indicator and the label. That would be useful when the user has a custom label layout or/and custom indicators and wants to fine-adjust the spacing between them.
Then, I updated the MarkerComponent to receive the label position. I extracted the current implementation of the Y position of the label and put it alongside the new position calculation separated by a when structure. This way, if someone decides to implement another position, they only need to implement a new branch.
And finally, I updated the sample application by using the new label position on the Chart3
What does that look like?
https://github.com/patrykandpatrick/vico/assets/10220064/05765188-ea90-4a78-8190-d2792b928f00

@patrickmichalik Is there any other thing here that needs some change?

This is included in Vico 2.0.0 Alpha 7. I wanted to follow up and briefly explain the reason for one of the changes we made, as it may not be immediately obvious. You’ll notice that AboveIndicator#spacingDp has been removed. This is because TextComponent supports margins and padding, making the parameter in question redundant. Setting spacingDp to 4 would be equivalent to giving the label a bottom margin of 4 dp. We overlooked this while reviewing the PR. Cheers for contributing!


fix: api benchmark uses mock node tip

I have mocked the currentNodeTip for the API benchmark not to crash.
Issue Number
No issue, quickfix.

Merging before the next nightly build starts;


Fix 2 problems in Dockerfile

CI succeeds in https://github.com/yzs-lab/habitat/runs/4312102030
closes #1
closes #3

2382fac follows the original solution from upstream in https://github.com/tianon/gosu/issues/39

Thanks for the fixes!


Minimist dependency

Hey! This project seems very awesome, it's exactly what I need. Except it seems to replace & use minimist. Have you thought about just providing a way to make a string into exactly what node does to args, then just passing that to minimist?

There's a ton of these packages, I bet I can find one that does exactly what I need, if this doesn't do the trick. Thanks for your work on this! Sorry for the issue!


[opencv] Update to OpenCV 3.4.4

Update OpenCV port to OpenCV 3.4.4.
Test: .\vcpkg install opencv[core,dnn,ffmepeg,ipp,contrib] (major options with high demand)

x64-windows
x86-windows
x64-windows-static
x86-windows-static

Fixed Known Issues:

[x] dnn #4861
[x] gdcm #4880
[x] openexr #4820 (wait for merge)

@cenit @twMr7 Sorry, this pull request was re-sent, because my mistake git operation.
First, Merge this pull request. And, If this port have other problems yet, let's fix those problems based on OpenCV 3.4.4. Can you agree with this proposal? I will suggest to merge to the administrator if you agree.

Just as to not loose track with issues reported in the other PR, what about the problems with opencv[ovis] and opencv[sfm] (the last one is broken because the dependency walker - for static build at least - goes to gflags-static.lib also for debug builds, instead of gflags-staticd.lib)? Have them been fixed in the meanwhile or not?

I propose to add them to the issue list as a future work.
Known Issues:

[ ] sfm
[ ] ovis

I am building the latest master to verify the status of this port, but all I can do is x64-windows only, no enough spare time for x86 and static build. I'll post the result later. Meanwhile, @UnaNancyOwen , I don't see any configuration file contains tiny_dnn downloading in official opencv v3.4.3 source branch, but the following instruction exists in portfile.cmake twice.
vcpkg_download_distfile(TINYDNN_ARCHIVE
    URLS "https://github.com/tiny-dnn/tiny-dnn/archive/v1.0.0a3.tar.gz"
    FILENAME "opencv-cache/tiny_dnn/adb1c512e09ca2c7a6faef36f9c53e59-v1.0.0a3.tar.gz"
    SHA512 5f2c1a161771efa67e85b1fea395953b7744e29f61187ac5a6c54c912fb195b3aef9a5827135c3668bd0eeea5ae04a33cc433e1f6683e2b7955010a2632d168b
  )

I think this downloading is confusing and unnecessary.

I built opencv:x64-windows with every features, found something's wrong from config-x64-windows-out.log:

Module opencv_dnn_objdetect disabled because opencv_dnn dependency can't be resolved!
Module opencv_text disabled because opencv_dnn dependency can't be resolved!

I am checking what package or setting I missed.

data: Download: face_landmark_model.dat
xfeatures2d/boostdesc: Download: boostdesc_bgm.i
...
config-x64-windows-out.log
vcpkg_installation.log

I though all download should carry out before configuration?
All features built fine according to console messages, but I suspect that some module is missing in bin and lib, e.g. opencv_dnn.

@twMr7 i just opened this PR #4963 to fix that problem, which must be incorporated also in this PR before merging (if it works!)


How to run any of the websharper templates?

I have a compiled websharper template (using websharperspa just for testing)
How do I see the result in my browser? In full visual studio I can press the green play button

It's probably better to use the Stack Overflow question you asked for this kind of question, and leave the Github issue tracker for tracking bugs and feature requests. I'd recommend closing this Github issue (and first editing it to include the URL of your Stack Overflow question) so there's only one single, obvious place for people to answer your question.


[Settings][OOBE] Update RemoveInstallerHashesRegex to match new installers list

Summary of the Pull Request

PR Checklist

[x] Closes: #25252
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

you're gonna hate me,
i kinda want to swap it now to a more table format.
https://github.com/microsoft/PowerToys/issues/22277
can we maybe shift it to ignore everything from
Installer Hashes to Highlight?

could have the script basically generate these lines to and have direct links

Description
Filename
sha256 hash

Per user - x64
PowerToysUserSetup-0.69.0-x64.exe
D05FC31F137718516C1D792765AAFEC51551FC172064BC3EE170911E455053AC

Per user - ARM64
PowerToysUserSetup-0.69.0-arm64.exe
0B113EAF17EEEA86299DEF7F73E7AF15A9CBFB8033C7F9218E33BDA276F37727

Machine wide - x64
PowerToysSetup-0.69.0-x64.exe
ADE1E56DF8CC52839FBD9DB6A0BCF34E38B88AAA52544EEAD3C1BB023FDAF29A

Machine wide - ARM64
PowerToysSetup-0.69.0-arm64.exe
CF13F75F93462BE68CA3E4C90B23F3828BBCAA44E7AA987503E73652DCB9FDF8

you're gonna hate me,
i kinda want to swap it now to a more table format. #22277
can we maybe shift it to ignore everything from Installer Hashes to Highlight?

Updated to match everything between ## Installer Hashes and ## Highlights


Update README.md

for issue #254 i edited the codenewbie podcasts
In this pull request

[ ]  I am adding a new resource.
[ ]  I am adding a new step
[x ]  I am providing an edit
[ ] Removing a resource

Adding new resource

The resource I am adding is _ _ _ _
I am adding this resource to _ _ _ _ step.
I think this resource deserves to be in this list because _ _ _ _

Do not add more than one during hacktoberfest
Add new step

The step I am adding is _ _ _ _
The reason why I think this step should be there is because _ _ _ _

[ ] I have added at least one resource to this step

Do not add more than one during hacktoberfest
Providing an Edit

The edit I am providing is to add newbiecoding to podcast section

Do not add more than one during hacktoberfest
Remove a resource

The resource I am removing is _ _ _ _
I am removing this resource from _ _ _ _ step.
I think this resource should not be in this list because _ _ _ _

Don't ask to remove stuff - unless you believe the content violates the code of conduct

Thanks for the heads up, I was in a hurry when I was re-editing.  Sent from Mail for Windows 10 From: Michelle Mei-Ling SandfordSent: Saturday, October 24, 2020 12:53 PMTo: msandfor/10-Easy-StepsCc: Dada abdulrasheed; MentionSubject: Re: [msandfor/10-Easy-Steps] Update README.md (#292) @msandfor requested changes on this pull request.I'm sorry @iamhabee what you are changing the entry to, is not correct. As it is now, it is correct. This is what is looks like in the README.MD - which is already correct:When you click on the link from the page it correctly takes you to the Codenewbie podcast page:and the Description matches the information on the site. As it should be. Please stop attempting to change the CodeNewbie entry - and pick another issue. Like add the Developer Tea Podcast to line 177 as I have suggested:#225—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.


Adding the --aws-profile command line option to the creational commands.

Referring to #440
I preferred to use a different approach than setting the environment variable as proposed in the issue since the Profile option was already there in the Sdk.
I felt free to add a new parameter  to cutil.GetReconciler so that each adapter
can pass its own specific parameters needed at runtime.
In my opinion this is ideal since there's a whole kind of parameters that does not necessarily fits into the state, like in this case with the need to actually choose a profile from the ~/.aws/credentials which is not related to the actual state but to the way the user want to run the whole thing at a specific moment.
Closes #440
Example:
  kubicorn apply --aws-profile default mycluster

Signed-off-by: Lorenzo Fontana<EMAIL_ADDRESS>

This can lead to a BC for whom uses cutil.GetReconciler

Hi there, welcome to kubicorn project. This looks great, thank you very much for the contribution!
Auto-generated file (bootstrap.go) is not go formatted, so tests are failing. Can you please run make gofmt to format all Go files?

@xmudrii formatted, committed and squashed

@fntlnz Just to check a thing -- how this will work with existing environmental variables (AWS_*)? Will they continue to work or no?

@xmudrii yeah there’s no change on that side, I just removed the tests that were checking for error if the env where not defined since it’s not the case anymore.

One could even set the AWS_PROFILE and completely ignore this flag. This is worth mentioning in the docs too.
@xmudrii also to reply your last question, The Profile in the session strict is populated via the AwsProfile, it that is an empty string environment variables are used

@fntlnz Thank you very much for the explanation! I can't merge this due to conflicts, so can you please resolve them and make sure to re-run make gofmt so we prevent tests problems?

rebased


MudTooltip: Add Disabled parameter

Description
Resolves #8842
How Has This Been Tested?
Added 3 unit tests
Added 1 visual test
Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[X] I've added relevant tests.

The Visible property should never be set to true while disabled and that should be tested as well.

@danielchalmers Do you mean I need to modify/add the logic in the Visibility setter that prevents setting when disabled
or only ensure (test) that Visibility never gets changed when toggling the tooltip disabled state?

Do you mean I need to modify/add the logic in the Visibility setter that prevents setting when disabled or only ensure (test) that Visibility never gets changed when toggling the tooltip disabled state?

I was thinking that the Visible property shouldn't be set to true (inside HandleMouseEnter, HandleFocusIn, HandleMouseUp) if Disabled == true to avoid a misleading state since it's not actually visually rendered.
I'm not sure what should happen if the user uses the Visible setter to make it explicitly true while Disabled == true 🤔

I was thinking that the Visible property shouldn't be set to true (inside HandleMouseEnter, HandleFocusIn, HandleMouseUp) if Disabled == true to avoid a misleading state since it's not actually visually rendered.

All 5 HandleXXX methods set the Visible state.
However, the entire MudPopover is skipped from rendering when Disabled == true, so I don't see an issue here.
(Of course I might miss something)

I'm not sure what should happen if the user uses the Visible setter to make it explicitly true while Disabled == true 🤔

I believe that nothing should happen when Disabled == true and other properties are set.
I'm assuming this behavior also applies to all other MudBlazor components.

My point is from a public API perspective where someone might rely on the Visible property to know whether or not the tooltip is actually open on the screen

Do you think you could take a look at #4466 as well? Seems like a simple fix.

My point is from a public API perspective where someone might rely on the Visible property to know whether or not the tooltip is actually open on the screen

By default Disabled == false, so no harm done.
Once people are aware of this new parameter they'll adjust their logic?

In all other components Disabled prevents user interaction via mouse / keyboard. So this implementation is ok, we don't have to concern ourselves with the fact that the library consumer can force it open via the Visible parameter, that may be exactly what they want and lies entirely in their own responsibility.

Thanks Yomodo! FYI @ScarletKuro


Migration Enhancements

After Migrating the novel it doesn't goes back to migrated novel page and it doesn't even migrated properly it adds the migrated novel as a new unread novel
Here is the YouTube video link : https://youtu.be/WjF-AB_SZDg

Currently this is the expected behavior as this was just a primary implementation of the feature and I didn't work more on it to bring it on parity with Tachiyomi's migrate feature.

Are you the only one working on this app or you have a team ??

It's mostly me working on this app in my free time. The project also gets contributions sometimes as it’s open sourced, here's the list of contributors.


test copying uploaded media to new QA machine

we'll need an automated procedure for syncing uploaded files and media to the new QA machine that we can run once at the start of the migration process, and at least one final time at the end before we make the switch.

according to its man page rsync doesn't support copying between two remote hosts (one must be localhost), so I ended up doing the media transfer with scp because it has an option to move files between two hosts using localhost as a middleman. this also means there's no need to mess with proxying options, because scp just reads your ssh config file, so it will access the hosts in the same way you would via ssh.
to transfer the media folder to QA, for example, I did:
$ scp -rpv3 cdhweb-old:/srv/www/media/ cdhweb-qa:/srv/www/

where:

cdhweb-old is an alias for cdh.princeton.edu which proxies thru libservdhc7 and logs in as deploy
cdhweb-qa is an alias for cdh-test-web1.princeton.edu which logs in as pulsys
-r copies the directory recursively
-v displays extra stats in case it fails
-p preserves permissions on files (they'll still be owned by pulsys on the target machine)
-3 proxies the transfer through my local machine, which makes the ssh aliases above work correctly

note also the lack of /media on the second path, otherwise we'll get /media/media.
this command takes quite awhile to run to completion. unfortunately scp isn't as smart as rsync when it comes to copying, so we'll just have to clobber everything in the destination each time we run it.

@thatbudakguy I had imagined sshing into the new server and running the rsync command there to transfer content directly from the old server to the new one. Wouldn't that be more efficient / effective?

to do that, the new servers need to know how to proxy through libservdhc7 to get to the old production instance. it seemed simpler to just run it on my machine, which has that already set up in my ssh config.
i don't think conan has permission to write to /srv/www/media, so it needs to be pulsys to do the transfer (but we can chmod it afterward).

@thatbudakguy why is the proxy needed? We can request RC allow access from the new IP if that's the problem (likely a trivial change for them). I think the efficiency of using rsync and keeping the transfer within the same network is worth it!
Let's make sure we figure out media directory ownership as part of testing the deploy. Would be nice to have it set properly at the outset.

Testing using rsync from production old cdh web to new pul test server.

manually created new ssh key for conan user on cdh-test-web1
manually copied new public key to deploy user authorized keys on cdh-web
created new test directory for media in tmp just to check the process
ran rsync with these options:
rsync -az --stats deploy@cdh-web.princeton.edu:/srv/www/media/uploads media

Output is pretty verbose with those options, but nice to see what's happening.
Stats reported:
Number of files: 3,180 (reg: 2,203, dir: 977)
Number of created files: 2,295 (reg: 1,859, dir: 436)
Number of deleted files: 0
Number of regular files transferred: 1,859
Total file size: 1,402,599,027 bytes
Total transferred file size: 952,300,431 bytes
Literal data: 952,300,431 bytes
Matched data: 0 bytes
File list size: 139,569
File list generation time: 0.001 seconds
File list transfer time: 0.000 seconds
Total bytes sent: 39,345
Total bytes received: 916,075,327

sent 39,345 bytes  received 916,075,327 bytes  17,123,638.73 bytes/sec
total size is 1,402,599,027  speedup is 1.53

I also ran it with time to see how long syncing everything took:
real	0m52.887s
user	0m15.999s
sys	0m7.951s

Tested and documented.


Added check for requests that would cause payments to crash

Codecov Report
Attention: Patch coverage is 72.00000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 92.59%. Comparing base (b94ef22) to head (de81777).
Report is 11 commits behind head on main.

Files with missing lines
Patch %
Lines

utils/api-error.js
60.00%
5 Missing and 1 partial :warning:

routes/payments.js
90.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   92.53%   92.59%   +0.06%
==========================================
  Files          24       24
  Lines         241      270      +29
  Branches       21       27       +6
==========================================
+ Hits          223      250      +27
- Misses         16       18       +2
  Partials        2        2

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


Proposal: Support including extra files with final bundle

Decided to mock it up in actual code to demonstrate how it could work.  Basically I need to add some supporting files with my bundles (native libraries for example) for my function to work properly so I thought SST could support an includes parameter.
Usage:
    const api = new sst.Api(this, "Api", {
      defaultFunctionProps: {
        include: [
          {from: "node_modules/.prisma/client/prisma.schema", to: "prisma.schema"},
        ],
      }
    })

Thanks @thdxr this is great! I'm going to take a look at it tomorrow.

Thought about this more and another approach would be exposing a similar API to aws-lambda-nodejs https://github.com/ryands17/prisma-lambda/blob/main/lib/prisma-lambda-stack.ts#L24

@thdxr your fork has been merged in to #225 and is work in progress


Uncaught TypeError when fold section button clicked in .coffee file

I've seen a issue which has similar stack trace, but this one can be different.
Step to reproduce:

open a .coffee file
click one of the fold button (on the left of edit panel, right to line number, only visible when mouse hover). This may cause the bug.
if bug didn't appear, try another fold button.

The bug dosn't appear every time, event when I click the same button.
If you click one fold button for many times repidly, the bug appears to.
Atom Version: 1.0.19
System: Ubuntu 14.04.3
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'onDidDestroy' of undefined
At /usr/share/atom/resources/app.asar/src/decoration.js:62

TypeError: Cannot read property 'onDidDestroy' of undefined
  at new Decoration (/usr/share/atom/resources/app.asar/src/decoration.js:62:49)
  at DisplayBuffer.module.exports.DisplayBuffer.decorateMarker (/usr/share/atom/resources/app.asar/src/display-buffer.js:1319:20)
  at DisplayBuffer.module.exports.DisplayBuffer.decorateFold (/usr/share/atom/resources/app.asar/src/display-buffer.js:1720:19)
  at DisplayBuffer.module.exports.DisplayBuffer.handleBufferMarkerCreated (/usr/share/atom/resources/app.asar/src/display-buffer.js:1709:14)
  at /usr/share/atom/resources/app.asar/src/display-buffer.js:5:61
  at Emitter.module.exports.Emitter.emit (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:82:11)
  at TextBuffer.module.exports.TextBuffer.markerCreated (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:1354:20)
  at MarkerStore.module.exports.MarkerStore.createMarker (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/marker-store.js:332:21)
  at MarkerStore.module.exports.MarkerStore.markRange (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/marker-store.js:149:19)
  at TextBuffer.module.exports.TextBuffer.markRange (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:695:31)
  at DisplayBuffer.module.exports.DisplayBuffer.createFold (/usr/share/atom/resources/app.asar/src/display-buffer.js:843:95)
  at TextEditor.module.exports.TextEditor.createFold (/usr/share/atom/resources/app.asar/src/text-editor.js:2571:33)
  at LanguageMode.module.exports.LanguageMode.foldBufferRow (/usr/share/atom/resources/app.asar/src/language-mode.js:148:30)
  at TextEditor.module.exports.TextEditor.foldBufferRow (/usr/share/atom/resources/app.asar/src/text-editor.js:2511:32)
  at LineNumberGutterComponent.module.exports.LineNumberGutterComponent.onClick (/usr/share/atom/resources/app.asar/src/line-number-gutter-component.js:137:30)
  at HTMLDivElement.<anonymous> (/usr/share/atom/resources/app.asar/src/line-number-gutter-component.js:3:61)

Commands
     -1:28.8.0 editor:move-to-beginning-of-line (atom-text-editor.editor.emacs-plus.is-focused)
     -1:28.7.0 atomic-emacs:next-line (atom-text-editor.editor.emacs-plus.is-focused)
  2x -1:28.5.0 atomic-emacs:previous-line (atom-text-editor.editor.emacs-plus.is-focused)
     -1:28.2.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.emacs-plus.is-focused)
  2x -1:28.2.0 atomic-emacs:previous-line (atom-text-editor.editor.emacs-plus.is-focused)
     -1:27.8.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.emacs-plus.is-focused)
     -1:27.7.0 atomic-emacs:backward-char (atom-text-editor.editor.emacs-plus.is-focused)
     -1:27.4.0 atomic-emacs:next-line (atom-text-editor.editor.emacs-plus.is-focused)
     -1:27.2.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.emacs-plus.is-focused)
  2x -1:27.1.0 atomic-emacs:backward-char (atom-text-editor.editor.emacs-plus.is-focused)
  4x -1:25.7.0 editor:select-word (atom-text-editor.editor.emacs-plus.is-focused)
     -1:24.6.0 editor:consolidate-selections (atom-text-editor.editor.emacs-plus.is-focused.mark-mode)
     -1:24.6.0 core:cancel (atom-text-editor.editor.emacs-plus.is-focused.mark-mode)
     -1:24.4.0 editor:move-to-beginning-of-line (atom-text-editor.editor.emacs-plus.is-focused)
  3x -1:24.4.0 atomic-emacs:next-line (atom-text-editor.editor.emacs-plus.is-focused)
     -1:23.3.0 atomic-emacs:previous-line (atom-text-editor.editor.emacs-plus.is-focused)

Config
{
  "core": {
    "packagesWithKeymapsDisabled": [
      "remote-edit"
    ],
    "autoHideMenuBar": true,
    "disabledPackages": [
      "Remote-FTP"
    ]
  },
  "editor": {
    "invisibles": {},
    "backUpBeforeSaving": true
  }
}

Installed Packages
# User
atomic-emacs, v0.7.4
coffee-compile, v0.19.0
emacs-plus, v0.9.8
less-autocompile, v1.0.0
minimap, v4.13.4
minimap-find-and-replace, v4.3.0
minimap-git-diff, v4.1.8
minimap-hide, v0.3.0
minimap-linter, v1.0.0
remote-edit, v1.8.21
symbol-gen, v1.0.0
sync-settings, v0.6.0

# Dev
No dev packages

Let me know if there's more information I can provide :smile:

Thanks for the report, @giyyapan, but I'm having trouble reproducing this. Have you been able to reproduce this in safe mode?
Still, seems like something Atom should be handling better. /cc @nathansobo @maxbrunsfeld

No, I can't reproduce this in safe mode.

@izuzak I've been tested this for a while, and my current conclusion is this bug only appear when both package atomic/emacs and aki77/atom-emacs-plus enabled.
Well, to be honest, I'm not 100% sure about this, for this bug is really hard to trigger sometime. but I did my best to find some pattern out there, and I'm going to double check this conclusion.
Hope this helps :smiley_cat:

Ahh!
Update: aki77/atom-emacs-plus is the key. The bug only appear when it's enabled.

Thanks for helping investigate this, @giyyapan. :sparkle: I'm going to keep this open so that @nathansobo and @maxbrunsfeld can take a look and see if there's anything we should do about this in Atom core.

I guess I should close this for it's not appearing (for a long time!)


Fix for Java12+ compatibility

This commit fixes issues #1342, #1335, #1301, #1257, and #1249
Prior to this change, static analysis of code containing a break statement would fail when running on Java version 12+, with an error similar to the following:
BugPattern: Finally
Stack Trace:
java.lang.NoSuchFieldError: com/sun/tools/javac/tree/JCTree$JCBreak.label

This was due to a change in the class com.sun.tools.javac.tree.JCTree$JCBreak. In Java 12+, label is no longer a field, and is instead a calculated value. This change simply changes com.google.errorprone.bugpatterns.Finally to call JCBreak.getLabel() instead of JCBreak.label.
This change maintains compatibility with prior versions of Java.
Changes to be committed:
modified:   core/src/main/java/com/google/errorprone/bugpatterns/Finally.java

@googlebot I fixed it.


Added extra logs

Added extra logs to look into prow CI error for the e2e test case, will revert it once issue get solved.

/hold

/unhold

\hold

/lgtm


Handling invalidated tokens and 401 statuses

I'm trying to figure how how invalidated tokens (on the server side) should be handled.
Once a user logs in successfully the token is stored locally and on each page reload the token is retrieved and the profile is being requested.
I assumed that if the profile returns a 401 HTTP code it would remove the locally stored token and require the user to login again.
What would be the "correct" or suggested way to do this?
Additionally how would an $http response interceptor force the user to re-login when an 401 HTTP status is received?
This would be useful for custom made requests using the token.
Thank you and I do apologise if this is common knowledge around here. :)

Hi @geoah, you can find some discussions about it in issue #17.


Error on building js file, need linux requirements packages

Hi, so i just want to try build (just build hello.js --clean --static) with my linux machine (ElementaryOS Hera) and got error like this:
...
deps/v8/include/v8-platform.h
deps/v8/include/v8-value-serializer-version.h
deps/v8/include/OWNERS
deps/v8/include/js_protocol.pdl
deps/v8/include/v8-wasm-trap-handler-posix.h
rm -f v8lib-0.0.22.tar.gz
g++ -c -fno-exceptions -ffunction-sections -fdata-sections  -DJUST_VERSION='"0.0.22"' -std=c++17 -DV8_COMPRESS_POINTERS -I. -I./deps/v8/include -O3 -march=native -mtune=native -Wpedantic -Wall -Wextra -flto -Wno-unused-parameter just.cc
g++ -c -fno-exceptions -ffunction-sections -fdata-sections  -std=c++17 -DV8_COMPRESS_POINTERS -I. -I./deps/v8/include -O3 -march=native -mtune=native -Wpedantic -Wall -Wextra -flto -Wno-unused-parameter main.cc
g++ -s -static -flto -pthread -m64 -Wl,--start-group deps/v8/libv8_monolith.a main.o just.o builtins.o modules/sys/sys.o modules/fs/fs.o modules/net/net.o modules/vm/vm.o modules/epoll/epoll.o -Wl,--end-group -Wl,--gc-sections  -ldl -lrt -o hello
g++: error: modules/sys/sys.o: No such file or directory
g++: error: modules/fs/fs.o: No such file or directory
g++: error: modules/net/net.o: No such file or directory
g++: error: modules/vm/vm.o: No such file or directory
g++: error: modules/epoll/epoll.o: No such file or directory
Makefile:85: recipe for target 'main-static' failed
make: *** [main-static] Error 1
failed in 319.41 sec

I really dont have experience with c/c++ or something related but i tought the problem is some lib/package are missing & not installed on my machine. Thanks.

hi. the error seems to be happening earlier - module library files are missing so linker is failing.
can you try
just build hello.js --clean --cleanall

and if that works then try
just build hello.js --clean --cleanall --static

the compiler is very much a PoC at the moment. lots of bugs on no tests!

Thanks for the solution and sorry for late reply, the both commands was successfully build with no error, but the executable produced from hello.js return undefined in here, but simple program like print string is perfectly work fine.

closing this as these issues should be resolved in upcoming 0.1.2 release


On the road to adoption

For ReFormNext next versions we'll be focusing on bringing the API as close as possible to Formik's while also showing Reason's power.
Todo

[ ] Field component, for fastness and context
[ ] Remove Validation out of ReFormNext

Let's do a todo so we know all helpers that we could bring from Formik's API design


refactor: move from io/ioutil to io and os packages

Summary
The io/ioutil package has been deprecated in Go 1.16 (See https://golang.org/doc/go1.16#ioutil). This PR replaces the existing io/ioutil functions with their new definitions in io and os packages.
Ticket Link
N/A
Release Note
NONE

Thanks!


Fix casing of module name in webpack

https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/1118

Coverage remained the same at 73.76% when pulling 3b62bb1de6ce18c53ffb28cc962474341fa06e76 on RMacfarlane:rmacfarlane/casing into c55a9563e189c8bfa99ea599f86aa9641f635a6f on AzureAD:dev.

@RMacfarlane Thanks for the submission. This is a breaking change for people currently using the library as "Msal", so we will try to incorporate this in a future version of msal. Thanks for your patience.

Closing, as this will be addressed in v2.


Spatial zones for selections

Allowing the select atoms inside a sphere/cube/..., or within a given distance of another atom.
What is needed:

Decide on syntax;
Decide on implementation strategy inside the current implementation;

I think this will be better resolved by #83: instead of having to add specific case for every kind of spatial zone, one can just use mathematical expressions for the spatial zone.
For example, intead of
# sphere(x, y, z, radius)
atoms: in sphere(0, 0, 0, 15) and name Zn
# sphere(x, y, z, radius, axis_x, axis_y, axis_z)
atoms: in cylinder(0, 0, 0, 15, 0, 0, 1) and name Zn

we can have
atoms: math(x*x + y*y + z*z < 15*15) and name Zn
atoms: math(x*x + y*y < 15*15) and name Zn

It is a bit less readable, but much more usable: no need to remember which argument does what, and no need to implement every kind of geometry.


Add podspec for cocoapods installation

If you try to install this library with a project that uses cocoapods you will get the following error:

[!] use_native_modules! skipped the react-native dependency 'react-native-render-html'. No podspec file was found.
- Check to see if there is an updated version that contains the necessary podspec file
- Contact the library maintainers or send them a PR to add a podspec. The react-native-webview podspec is a good example of a package.json driven
podspec. See https://github.com/react-native-community/react-native-webview/blob/master/react-native-webview.podspec
- If necessary, you can disable autolinking for the dependency and link it manually. See
https://github.com/react-native-community/cli/blob/master/docs/autolinking.md#how-can-i-disable-autolinking-for-unsupported-library

@systemride Thank you for your contribution! It's a good idea. Would you mind rebasing on master? There seems to be conflicts with package.json.

@systemride Can you provide a link to a repository reproducing the error you where facing? As far as I know, RN does uses Cocoapods by default, and I have not seen any ticket related to the issue you are referring.

I'm going to close now, because I don't really understand the need yet. I other library consumers encounter issues with cocoapod, I'm open to reopening.

We also get this warning, although the library still works. It'd be nice to have this merged so it goes away. I can't share the repo, but we're on RN 0.62.2.

@jsamr thank you!


Use test-and-test-and-set for scheduler scaling logic

This commit switches from test-and-set to test-and-test-and-set
approach for the scheduler_count_changing atomic bool when
relying on signals for dynamic scheduler scaling because
test-and-test-and-set reduces cache line invalidations and scales
better. See the following link for more details:
https://geidav.wordpress.com/2016/03/23/test-and-set-spinlocks/

As far as I can tell, this only changes the spinlock implementation when putting scheduler threads to sleep and waking them up. If I have that right, then I think this is a good change, although the performance impact will be small since these are uncommon operations.

Thanks @dipinhora

@sylvanc correct, this is only in relation to the schedulers sleeping and waking. agreed that the change is limited in scope and benefit. it should allow for the scheduler scaling logic to work better on systems with larger numbers of cores (hundreds maybe?).


URLs in markdown are not auto linked

Gitea version (or commit ref): Gitea Version: 1.3.2
Git version: git version <IP_ADDRESS> (server)
Operating system: ( centos7)
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL) https://try.gitea.io/33485938475983475/DeezLoader-Reborn/issues/2
[ ] No
[ ] Not relevant

Log gist:

Description
in issue comments or markdown editor
enter a link
http://example.com

expect it auto linked
expect to no need to enter the following
[http://example.com](http://example.com)

Auto-linking is a feature of GitHub formatted markdown, and not standard markdown. You should be using <http://example.com> instead.

That way is lot easier than the default one, it helps a lot for me, thanks.  though, I think the Auto-linking  feature should still be added.

Solved by #3354 (merged)


Add user search

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#41  👈
#40
#39
main

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

Join @techwithanirudh and the rest of your teammates on  Graphite


Fix different dtype input case in histogram

cub.device_histogram assumes following dtype combination.

same float dptye x and bin_edge
x is integer dtype, binedge is double dtype

cub.device_histogram assumes following dtype combination.

same float dptye x and bin_edge
x is integer dtype, binedge is double dtype

It's not CUB's requirement, but actually NumPy's:
https://github.com/numpy/numpy/blob/2f3b82638a24204a13c9b16a3a36a59f199df41d/numpy/lib/histograms.py#L442-L443
So I simply used SFINAE to specialize kernels that satisfy this requirement:
https://github.com/cupy/cupy/blob/0af4d978374e53fc9bd4568084a0386c12a45264/cupy/cuda/cupy_cub.cu#L585-L589

pfnCI, test this please.

Jenkins CI test (for commit 4ea497f3f881bc02baffdee97c189c7c0402e3f7, target branch master) failed with status FAILURE.

Cub histogram also gets wrong with float32?

The CI also gets wrong with float32?

We have to use double type bins, so bins = bins.astype('d') is correct. @takagi

In @takagi'a snippet, x is fp32, so bins should also be fp32.

Blocked by #3617

As #3617 is merged to the master now, would you resolve the conflict?

I fixed conflicts. So, this PR only adds testcase

pfnCI, test this please.

Jenkins CI test (for commit 229e52ea88b79e69a054e88d8e889ba600433eb4, target branch master) failed with status FAILURE.

pfnCI, test this please.

Jenkins CI test (for commit 229e52ea88b79e69a054e88d8e889ba600433eb4, target branch master) failed with status FAILURE.

pfnCI, test this please.

Jenkins CI test (for commit 229e52ea88b79e69a054e88d8e889ba600433eb4, target branch master) failed with status FAILURE.

Waiting for CI problem fixed.

pfnCI, test this please.

Jenkins CI test (for commit 229e52ea88b79e69a054e88d8e889ba600433eb4, target branch master) failed with status FAILURE.

pfnCI, test this please.

Jenkins CI test (for commit 306ed6feca731211582a54480d6d8ec498248ccd, target branch master) succeeded!

LGTM!


Core: Bump to 0.8.0

Description
Bump to 0.8.0
Issue ticket number
closes #xx
Additional Context

Codecov Report
Patch coverage has no change and project coverage change: +0.03% :tada:

Comparison is base (e60accc) 90.19% compared to head (3e525f8) 90.22%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #272      +/-   ##
===========================================
+ Coverage    90.19%   90.22%   +0.03%
===========================================
  Files          350      350
  Lines         5964     5964
  Branches       803      803
===========================================
+ Hits          5379     5381       +2
+ Misses         425      423       -2
  Partials       160      160

Flag
Coverage Δ

build
89.57% <ø> (ø)

catalog-server
100.00% <ø> (ø)

cli
75.85% <ø> (ø)

core
94.18% <ø> (ø)

extension-authenticator-canner
78.37% <ø> (ø)

extension-dbt
97.43% <ø> (ø)

extension-debug-tools
98.11% <ø> (ø)

extension-driver-bq
84.93% <ø> (ø)

extension-driver-canner
84.65% <ø> (ø)

extension-driver-clickhouse
88.09% <ø> (ø)

extension-driver-duckdb
92.94% <ø> (ø)

extension-driver-ksqldb
90.33% <ø> (+0.84%)
:arrow_up:

extension-driver-pg
96.11% <ø> (ø)

extension-driver-snowflake
96.26% <ø> (ø)

extension-huggingface
86.25% <ø> (ø)

extension-store-canner
98.31% <ø> (ø)

integration-testing
90.27% <ø> (ø)

serve
87.17% <ø> (ø)

test-utility
∅ <ø> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

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


Remove assignment and declarations in if statements

After discussing with @reduz and @akien-mga it was decided that we do
not allow assignments or declarations in if statements. This PR removes
the instances of this I could find by automated means.

Even for guaranteed initialization purposes?

I think that because declarations in if conditions are syntactically and semantically distinct from simply using an assignment expression in the condition, they should be considered separately.

100% agree with this pr. Thank you! :)

@raymoo I made the exact same argument but @reduz and @akien-mga didn't want either case. I'll update the PR's title.

Ok, I guess it's their call in the end.

Another option that avoids a null dereference later on is to put an extra pair of braces around the whole thing. For example:
if (Banana* banana = GetABanana()) {
  // banana is not null in here
}

// becomes
{
  Banana* banana = GetABanana()
  if (banana) {
    // banana is not null
  }
}


Proper tail calls

Original author<EMAIL_ADDRESS>(March 04, 2013 06:24:43)
The following gives a "too much recursion" error:
function sumTo(n, accumulator = 0) {
return n === 0 ? accumulator : sumTo(n - 1, accumulator);
}
sumTo(123456);
Original issue: http://code.google.com/p/traceur-compiler/issues/detail?id=215

Traceur has a CPS transformer. Together with a trampoline, proper tail calls do not seem out of reach.
However, this will slow down execution by some factor. Nevertheless, I think it is important because proper tail calls will probably the feature of ES6 that is implemented last in browser engines.

You all are wizards. PTC always seemed like something that just wasn't possible to polyfill. I am interested to read how it happens here.

Not impossible, but hard to emulate because it needs a global transformation of the program.

Or at least manually handling all function calls.

The PTC pass could be the very first pass. Every call in tail position (including setters and getters) as in
function f() {
    k();
    return b ? g(x) : h(y);
}

is transformed into
function f() {
    k();
    return b ? $cont(g, x) : $cont(h, y);
}

where
$cont = (g, ...args) => [CONT_SENTINEL, g, ...args]

and CONT_SENTINEL is a non-forgeable, unique key.
When the final output is generated every call (including setters and getters) as in
f(x)

is then transformed into something like
$call(f, x)

where (again in ES6 syntax)
$calll = (f, ...args) => {
    var c, s;
    while ((c = f(...args))[0] === CONT_SENTINEL) {
        [s, f, ...args] = c;
    }
    return c;
}

Started work along these ideas here: https://github.com/mnieper/traceur-compiler/tree/proper-tail-calls
Feel free to post suggestions.

The code in the branch above now handles general tail calls properly and allows interfacing with code that does not know about this transpilation. The flag is --proper-tail-calls.
It's not yet finished, though, as not all expression types in possible tail position are handled yet (due to lack of time today).
@arv: As Function.prototype.call and Function.prototype.apply in tail position are not allowed to grow the stack, I have to patch them in the runtime. Is there a way for the runtime to detect whether it needs to support a particular compilation switch or not?

Is there a way for the runtime to detect whether it needs to support a particular compilation switch or not?

That is a slippery slope. We are trying to remove the need of doing that. One way to get around that is to inject an init call at the top of the program.

Another question: Can I add a file to the runtime that is not being transpiled during the build process? My bootstrap machinery for proper tail calls cannot be compiled with the option enabled.

Can you give a bit more context? Code that cannot be compiled by traceur but can be compiled by node seems odd.

My PTC transformer (when enabled) transforms every function that has at least one tail recursive return. The transformation needs a bit of runtime. When I compile traceur with PTC enabled, it tries to transform the runtime functions it needs for the transform.
I can manually remove all tail calls from the function in my runtime part so that they are not transformed, but isn't there a better way to disable transformations for bootstrap scripts?


FEATURE: TableGrid能否加上拖拽行（Dragable功能）

Version
Vben Admin V5
Description
场景：表单录入时，经常会用到排序这个字段，手动录入比较麻烦，容易出错
Proposed Solution
加上row-dragable功能，提升友好性
Alternatives Considered
No response
Additional Context
No response
Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.

目前v5版本还没有 TableGrid 组件吧，你应该是看错了。


Org Level Jobs

I would like to have hook listen for org-level events and be able to create prow jobs for entire organizations.
Reference: https://github.com/kubernetes/test-infra/issues/16572

/kind feature


One-click to copy JSON to clipboard

The JSON output by Vulnogram is typically copy and pasted elsewhere such as a text editor, email client, etc. It would be nice if there was a single button to click in order to copy the JSON into the clipboard rather than having to highlight the JSON before copying it.

That is a good idea. Thank you!
Would a download link, that could save the JSON as a text file to filesystem be more useful? and reduce clicks/keystrokes needed to get work done? perhaps I can add icon buttons for both.
If the purpose of copying is to eventually commit it to a git repo for submitting a pull request to cvelist repo, perhaps a button that says "Sync to my git repo" or "Submit to CVE list" would be desirable.
Git/github integration is in the roadmap.

Interesting idea. I'm not sure which one I'd use more between the "copy to clipboard" and "save" buttons. I think that either would improve the current workflow. Both could be nice as long as it doesn't complicate the graphical interface too much.

JSON tab now shows Copy and Download buttons.


Double counting of kinetic ionization

When using kinetic ionization with multiple ion species, the kinetic ionization contribution to the ionization rate is counted multiple times.
The IonKineticIonizationTerm is implemented such that it evaluates the fast electron ionization contribution for all species. However, in src/Settings/Equations/ions.cpp, it is added once for each ion species in the plasma. Because of this, if N ion species are added to the simulation, the kinetic ionization contribution will be amplified by a factor N (this does not, however, generally lead to a factor N difference in the charge state distributions).

It seems this is actually not the case... Something may still be wrong, but it could also be in my benchmark code, and the factor of two I found could have been a coincident...