Add a config option to print DAG.

Add a config option "spark.rddDebug.enable" to check whether to print DAG info. When "spark.rddDebug.enable" is true, it will print information about DAG in the log.

Can one of the admins verify this patch?

You can print the same details by calling rdd.toDebugString in your program ?

15/01/26 22:38:37 INFO SparkContext: RDD.toDebugString:
(12) MappedRDD[13] at map at dagPrint.scala:26 []
|   MappedRDD[12] at map at dagPrint.scala:25 []
|   FlatMappedValuesRDD[11] at join at dagPrint.scala:24 []
|   MappedValuesRDD[10] at join at dagPrint.scala:24 []
|   CoGroupedRDD[9] at join at dagPrint.scala:24 []
|   ShuffledRDD[4] at reduceByKey at dagPrint.scala:17 []
+-(2) MappedRDD[3] at map at dagPrint.scala:16 []
|  MappedRDD[2] at map at dagPrint.scala:15 []
|  /data.txt MappedRDD[1] at textFile at dagPrint.scala:14 []
|  /data.txt HadoopRDD[0] at textFile at dagPrint.scala:14 []
+-(2) MappedRDD[8] at map at dagPrint.scala:22 []
|  MappedRDD[7] at map at dagPrint.scala:21 []
|  /data2.txt MappedRDD[6] at textFile at dagPrint.scala:20 []
|  /data2.txt HadoopRDD[5] at textFile at dagPrint.scala:20 []

I think @ScrapCodes's question is ... is this necessary, given users can easily print the dag themselves?

@rxin I have noticed that very few users know about toDebugString. Maybe we should open a JIRA to add better documentation for that function (i.e. discuss it in the programming guide). Overall, I agree with you and @ScrapCodes in that I'm not sure this particular flag is super useufl.

Can one of the admins verify this patch?

Actually now I think about it - users don't always control the RDDs. For example, if they are using the ML pipeline API or calling some other libraries. Maybe there is some merit in including this, especially if they are off by default?

Fair, but the issue is in some cases (e.g. GraphX) the printed representation of the DAG can be many hundreds of lines long. could that potentially explode the output?

I think, when running a application, users just need change a config option, not modify binary code. And the default value of  this config option is false. When users need get DAG info, users will go to modify this value. So i think there is a small chance of exploded output, or there is  Impossible of  exploded output.

@pwendell
For example, I had completed binary package, and put it on spark cluster. Now i want to get DAG info, i not need to change code and build again. I just change a config option. After i had got info, I change the config option to false. So I think there is impossible of exploded output. Thanks.

ok to test

Test build #27096 has started for   PR 4257 at commit adcb14f.

This patch merges cleanly.

Test build #27096 has finished for   PR 4257 at commit adcb14f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27096/
Test PASSed.

Maybe we can include this, provided that it is off by default.
That said, I think a better name for this option is "spark.logLineage".

Test build #27149 has started for   PR 4257 at commit 83c2b32.

This patch merges cleanly.

@rxin
OK. I had changed the option name to "spark.logLineage". Thanks.

Test build #27154 has started for   PR 4257 at commit c27ee76.

This patch merges cleanly.

Test build #27154 has finished for   PR 4257 at commit c27ee76.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27154/
Test FAILed.

Test build #27149 has finished for   PR 4257 at commit 83c2b32.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27149/
Test FAILed.

Test build #27166 has started for   PR 4257 at commit d9fe42e.

This patch merges cleanly.

Test build #27166 has finished for   PR 4257 at commit d9fe42e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27166/
Test PASSed.

Thanks. I've merged this.


Figure out which part of the data to use for the analyses

see PR: https://github.com/berkeley-stat159/project-lambda/pull/13


Addon does not show Warband gold or allow deposit or withdrawal

Addon Version
v0.1.80
World of Warcraft Version
Retail
Describe the bug
Addon removes Withdraw and Desposit for warbank and does not show gold.
Steps to reproduce
Open warbank.

This is being worked on today.

and released with v0.1.83
Enjoy!


Link highlighting done

Issue no. #217  Now as we scroll to different sections , the link associated with that section will get highlighted .

Can you please give level2 for this , as it required some coding

hey @Nimrat0199 your changes are not visible and your deploy preview has not been generated.

pass all the checks by Netlify first.


[TUBEMQ-412] tube manager start stop scrrpts

Codecov Report

Merging #317 (8395fca) into TUBEMQ-336 (4cf9e9a) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             TUBEMQ-336    #317   +/-   ##
============================================
  Coverage          8.03%   8.03%
  Complexity          468     468
============================================
  Files               245     245
  Lines             27131   27131
  Branches           4491    4491
============================================
  Hits               2179    2179
  Misses            24483   24483
  Partials            469     469

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 9dfa8ad...8395fca. Read the comment docs.


[Bug] Nothing happens when "docker run .." (trap invalid opcode)

OK, I found this great software today and tried it out right away on my laptop, awesome!
But now I was trying on my stationary pc (with same setup) and it doesn't run.
-> % pwd
# /home/alf/tmp
-> % ls -l .
# total 4
# drwxrwxr-x 2 alf alf 4096 Apr  4 21:03 output
-> % ls -l output
# total 2656
# -rw-rw-r-- 1 alf alf 2717920 Mar 17  2018 test.m4a
-> % docker run -v $(pwd)/output:/output researchdeezer/spleeter separate -i /output/test.m4a -o /output
# <nothing>
-> %

This is exactly what worked on my laptop. It downloaded the model an split the audio.
I found a difference in the kernel log though..
-> % tail  /var/log/kern.log
# ...
# Apr  4 21:03:45 tower kernel: [  552.800686] traps: spleeter[4636] trap invalid opcode ip:7f40e2481d99 sp:7fffea3652a0 error:0 in libtensorflow_framework.so.1[7f40e1d82000+1912000]
# ...

This was not present on my laptop when running the same command.
Some system info
-> % docker --version
# Docker version 19.03.8, build afacb8b7f0
-> % uname -a
# Linux tower 5.3.0-42-generic #34~18.04.1-Ubuntu SMP Fri Feb 28 13:42:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
-> % lsb_release -d
# Description:	Linux Mint 19.3 Tricia

The only difference here is that on my laptop I'm running docker version 19.03.2 (but I didn't test rolling back, cause downgrading docker is a hassle. Betting this isn't the problem).
So, any idea why it's working on my laptop but not my stationary pc?
 We'll update the docker images soon with a newer version of TF, hopefully it will solve this for you.

Not easy for us to help you on that :/ the kernel log suggest a rather low-level issue with tensorflow. We'll update the docker images soon with a newer version of TF, hopefully it will solve this for you.

Ok, thanks @mmoussallam. What about internal spleeter logs? Maybe you could add an --verbose for debug logging etc? For all I know it might not be the kernel, thats just one of the logs I could find a diff in

Hi @fractalf,
There is actually a --verbose option in spleeter that will output all tensorflow logs. It may help resoliving your issue.

Hi @fractalf,
I solved it by assigning more memory to docker container.

Same here - using the --verbose flag for spleeter and running docker stats in a separate terminal confirmed I was hitting 100% of available memory, causing the process to terminate early. For reference it took around 10Gb of memory on a 2019 MBP for an 8 minute wav file. YMMV

@jbibro Cheers man, will test this out. I have a total of 22GB mem on my pc, so probably docker isn't greedy enough


Update places to 1.2.0

Please update my adapter ioBroker.places to version 1.2.0.
This pull request was created by https://www.iobroker.dev c0726ff.

RE-CHECK!


Refactor VirtualReplicatorClusterMap to VirtualReplicatorCluster.

Introduce getAllDataNodeIds(), getDataNodeId() and getPartitionIds().

Codecov Report

Merging #1137 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #1137      +/-   ##
============================================
- Coverage     68.85%   68.83%   -0.02%
+ Complexity     5131     5129       -2
============================================
  Files           404      404
  Lines         31854    31853       -1
  Branches       4058     4058
============================================
- Hits          21934    21927       -7
- Misses         8850     8853       +3
- Partials       1070     1073       +3

Impacted Files
Coverage Δ
Complexity Δ

...ava/com.github.ambry.cloud/CloudBackupManager.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../main/java/com.github.ambry.router/DecryptJob.java
91.66% <0%> (-5.56%)
4% <0%> (-1%)

...ain/java/com.github.ambry.router/PutOperation.java
90.87% <0%> (-0.54%)
108% <0%> (-1%)

...src/main/java/com.github.ambry.commons/BlobId.java
93.18% <0%> (-0.36%)
71% <0%> (-1%)

...in/java/com.github.ambry.store/BlobStoreStats.java
71.9% <0%> (-0.21%)
106% <0%> (+1%)

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 ed297ba...e99bbf0. Read the comment docs.


StatefulDataloader is slower than Dataloader. Is there any best practice of StatefulDataloader?

📚 The doc issue
Hello,
Thank you for your awesome implementation of StatefulDataloader.
I use the compare the speed of Dataloader and StatefulDataloader, the StatefulDataloader is much slower than Dataloader. For example, Dataloader costs 10ms per iter, but StatefulDataloader costs about 2s per iter.
Is there any best practice of StatefulDataloader?
cc @andrewkho
Suggest a potential alternative/fix
No response

Oh I see, there's an arg snapshot_every_n_steps. The default val is 1


Cell width not resizing to menu width, can't see accessory on the right

I'm using an AMSlideMenu with Storyboards to show my left menu, but I couldn't see the accessory views.
Upon a bit of investigation it seems the cells aren't resizing to the width of the menu, so bits anchored the right side of the cell are hidden.
It's a bit hard to explain, so I've put some images in a cell (right aligned) to illustrate what I mean.

The Pin image on the right is anchored to the right hand side, as is the share icon which must be way off outside of the visible menu.  If the cells were resizing to the width of the menu, both would be visible.
I've tried using - (CGFloat)leftMenuWidth; which changes the size of the menu when it opens, but still doesn't change the width of the cells.
I'm sure I'm missing one little thing, can anyone point me in the right direction on where to set this?

Please, can you try that with an empty UITableViewController and see if it works without AMSlideMenu?
Just put a new UITableViewController in storyboard, set the checkbox "is Initial View Controller",  put an imageview into a cell and set right anchor.
I'm asking, because xcode has a bug with right anchoring in a tableviewcell.

I've tried that and it seems to be working:

New UITableView, 3 static cells with some left and right anchored labels/imageviews, set as initial view controller.

@batnom , Have you resolved your problem?
in the demo, I found some code fit your problem.
write these code in your customize AMSlideMenuLeftTableViewController,

(void)viewDidLoad {
[super viewDidLoad];
// Do any additional setup after loading the view.
[self fixUI];
}
#pragma mark - Method

(void)fixUI {
self.myTableView = self.tableView;
self.view = [[UIView alloc] initWithFrame:self.view.bounds];
self.view.backgroundColor = self.myTableView.backgroundColor;
CGRect fixFrame = self.myTableView.frame;
fixFrame.size.width = [UIScreen mainScreen].bounds.size.width - 70;
self.myTableView.frame = fixFrame;
[self.view addSubview:self.myTableView];
}

@andrew020 That's fantastic, have got it working thank you!
Here's my final fixUI function:
- (void)fixUI {

    self.myTableView = self.tableView;

    // Colour

    self.view = [[UIView alloc] initWithFrame:self.view.bounds];
    self.view.backgroundColor = self.myTableView.backgroundColor;

    // Width (for accessories)

    CGRect fixFrame = self.myTableView.frame;
    fixFrame.size.width = [UIScreen mainScreen].bounds.size.width - 70;
    self.myTableView.frame = fixFrame;

    // Status Bar

    UIView *statusBarView = [[UIView alloc] initWithFrame:CGRectMake(0, 0, MAX(self.view.frame.size.width,self.view.frame.size.height), 20)];
    statusBarView.backgroundColor = [UIColor blackColor];
    [self.view addSubview:statusBarView];

    // Done

    [self.view addSubview:self.myTableView];

}


my node crashed

Not sure why this happened or how to interpret the error message:
thread 'tokio-runtime-worker' panicked at 'Failed to read from main loop: channel closed', /home/alan/neptune-core/src/peer_loop.rs:973:35
2023-08-12T21:38:44.434407514Z ERROR ThreadId(02) neptune_core::connect_to_peers: Peer thread (incoming) for <IP_ADDRESS>:40412 panicked. Invoking close connection callback
Error: channel closed

stale.  Unclear how to reproduce, and the code has changed significantly since this was reported.   closing.


Sign-Up Page Redirects to Twitter (X) on Form Click

Description
When users click  to fill out the sign-up form , they are unexpectedly redirected to Twitter (X) instead of staying on the site to complete the registration. This issue disrupts the user experience by preventing access to the intended sign-up form, making it impossible to register on the site.
https://github.com/user-attachments/assets/0fc810b4-fedc-4845-996b-a628162f3125
Steps to Reproduce

Inspect the HTML and JavaScript Code: Check the href attribute in the HTML code for the "Register" button and ensure it is correctly linked to the registration page (e.g., /register or /signup) rather than any external site like Twitter.
Remove Unintended Links: If the button contains a JavaScript function or an event listener, confirm that no redirection to external URLs (like Twitter) is triggered within the function.
Update Link: Replace the incorrect link with the appropriate path to the sign-up page
Test: After updating, test the button to confirm that it correctly opens the sign-up form without any redirection to Twitter.

Expected Behavior
No response
Actual Behavior
No response
Screenshots
No response
Additional Information
No response
What browser are you seeing the problem on?

[X] Firefox
[X] Chrome
[X] Safari
[X] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

Kindly assign this issue @aditya-bhaumik , @Apoorva57

@shivhere007 sure go ahead

Make sure to join the discord server for further communication
Don't forget to star the repository
Complete the task in next 2 days and kindly mention your issue # in your pr like:
Closes #IssueNumber

Note: the level of the issue may change based on the quality of the PR code


Neo Blockchain Python Smart Contract compiler

Hi,
I have followed your tutorial on youtube : https://www.youtube.com/watch?v=oy6Z_zd42-4
The contract is at 1:22 and the command at 6:09
I have followed all the instructions but my whole neo-python hangs
neo> build ./contracts/WhileTest.py test '' 02 False False
Saved output to ./contracts/WhileTest.avm
creating instance!
created begustorage
cloining from live!
cloned db
self contract properties: 3
Thanks you very much for your help !

i think it should be build ./contracts/WhileTest.py test 05 02 False False

the WhileTest.py doesn't have any arguments for Main. I discussed this item yesterday on Discord and even tried using the void (0xFF) parameter and that also doesn't work. As you'll see in the video Tom also uses the commands above (with '' for no arguments). I do think this is a bug somewhere as my whole neo-python hangs with this test case. The video is quite old so there might have been some changes that breaks this.

I believe there at some point was a change to the compiler and it no longer works with the break statement in the while loop, so it is an unterminated loop.  We should perhaps remove that example for now in the compiler, or fix the issue with it.


Unpushed commits are in unpulled commits

I don't know what happend or what exactly is happening, but my unpushed commits are inside the unpulled changes section.
cc @RianFuro

Have I mixed those up while refactoring? I'll have a look

Yeah I totally messed that up. Sorry about that

Unmerged is somehow always empty for me. Does this also happen for you?

nope. I remember I had that problem while testing, but I believe I had fixed that. At least with my test repo and on HEAD this doesn't happen to me.
What does git log --oneline<EMAIL_ADDRESS>tell you?


Fixed #34

Fixes #34. I did not find a better solution to fix #34 than using setTimeout in 0ms.
After a setTimeout the text selection is updated correctly. The toolbox disappears
now the first time you click somewhere in the text.

Thanks!


AHHHH

[url=https://postimg.org/image/so75jy5c7/][img]https://s23.postimg.org/so75jy5c7/2017_02_01_09_58_49.png[/img][/url]
[url=https://postimg.org/image/i2na7xz0n/][img]https://s23.postimg.org/i2na7xz0n/2017_02_01_09_58_55.png[/img][/url][/img][/url]
RIP all my builds!

my bad, horrible links:
https://s23.postimg.org/kvghryzd7/2017_02_01_09_58_49.png
https://s23.postimg.org/4lqbp2op7/2017_02_01_09_58_55.png

RIP this issue that doesn't have an explanation.

However, your issue is your server software (most probably) and has nothing to do with Multiverse.

Hm, I had the same issue before. The fix is really simple. Just stand in the messed up chunk, and leave the game and then come back. Boom! The chunk fixes itself 😄


Fix "4.29kB gzipped"

I get more like 6.8K when I gzip myself the minified version (v2.0.6).
It may not be much but still 159% of the advertised size.

All the sizes are updated on the website directly from the minified files https://github.com/hammerjs/hammerjs.github.io/blob/master/scripts/generate-data.js


Add Cradles x Blast!

Song Name
Cradles x Blast!
Artist Name
Sub Urban x Big Baby Tape
Source
Youtube
Youtube Link
64ihJjHemn1mJ8mS
Direct File Link
No response
Song ID
10011223
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 106987309

reject
invalid YouTube link


Move gradients from stdlib to swift-apis

As discussed in https://github.com/tensorflow/swift-apis/issues/53, this PR will move operations from the standard library to "swift-apis" https://github.com/tensorflow/swift-apis/pull/63

Great. Please also make sure to move the op definition in Ops.swift and tests in TensorFlowRuntime/test/*.swift.

Yep - coming in another PR.

It needs to be in this PR because CI has to pass.

You can also copy things over to swift-apis and merge things there first. That makes sure we don't lose APIs or break things during the process.

I've already submitted that PR: https://github.com/tensorflow/swift-apis/pull/63

I'm closing this PR, I'll create a new one with other changes.


Unable to use babel stage presets in ignored transform patterns

Jest v18.1.0
I am importing private node_modules that need to transformed with babel and stage-3 by Jest.
in my package.json jest config:
    "transformIgnorePatterns": [
      "node_modules/(?!@sm)"
    ],

Jest is correctly using babel to transform the files.
However, it is not using any of the "stage-3" features that i am specifying in my .babelrc.
And throws unexpected token on spread operators and others.
My .babelrc:
  "presets": [
    "latest",
    "stage-3",
    "react"
  ],

I've also tried implementing the explicit babel plugins, with no change.
Is this a bug or am I missing a configuration option?

Pretty sure that's not a Jest issue. Can you provide a repository with a repro?

Thanks @cpojer .  its an enterprise instance.  but i can try and reproduce in a new repo.
If it makes any difference, jest compiles the files fine when they're not in the node_modules folder.
Does that help?

maybe something is wrong with the regex.

not really, because it is allowing babel to process the files.  (the ONLY part that is not transforming is the stage-3 features)


The Curiosity Module seems out of place

Isn't it better to have these items in The Mighty Architect mod?

Staff of Symmetry
Handheld Blockzapper
and maybe other decorative blocks

I'm not confident about the last one, the last one could be up to preference. Maybe if that one was a standalone mod more people would download it and people could download parts of this mod that they want?
The first two items really aren't in tune with cogwheels, water wheels and other stuff. All of the stuff in this mod are somehow tech and redstone related in a neat way, but the first two kind of ruins it and makes it a mod that enables people to cheat on servers.

The Staff of Symmetry actually used to be in TMA back in 1.12, and migrated to create when TMA became a client-only building tool.
I completely agree that these tools and the schematics have nothing to do with any of the stuff added in 0.1, however this will change as the mod grows. More content will come to flesh out the modules, and more connections and interactions between them with it.

I love this mod because at the core it adds nothing crazy to the game. All the new features are very smart, well thought-out and intuitive. It's easy to understand by just looking at how things work mechanically in-game and you don't need to go through long pages of wiki to do simple things. I wish those few elements were on a separate mod that still could be dependent on this mod (like 'TMA for Servers). Also please see my other suggestion here #28


When editor text wrap is enabled, extensions should never cause horizontal scrolling.

VSCode Version: Version 1.21.0-insider (1.21.0-insider) 9a4ba7d06157091c882a03f118793db5d4657a82
OS Version: 10.13.4 Beta (17E150g)

Steps to Reproduce:

Configure VSCode to wrap text (I believe this is the default).
Install gitlens extension.
Open a project that has a git history.
Place the cursor on a line that is almost the length of the editor window.

With the default settings, Gitlens draws text at the end of the current line showing git blame info.
This causes the line length to exceed that of the window, thereby enabling horizontal scrolling.
Note that horizontal scrolling is not possible, even with wrapping lines, until the first time gitlens takes this action and causes an overflow. The only recourse I see is to reload the editor window, but it's just a temporary fix until gitlens does it again.
I think this is a design problem with the core VSCode editor. I do not want the editor to scroll horizontally, ever, unless wrapping is disabled. That an extension can cause the core UI to switch into a fundamentally different (and annoying) mode seems to me a flaw in the editor itself. The alternative is to blame the extension; perhaps there is some way of configuring GitLens to prevent this but I have tried and haven't found it yet. To reiterate, it seems reasonable to ask that the editor not allow extensions to break the basic layout constraint that the editor does not scroll horizontally.
The reason this is so annoying is that once horizontal scroll is enabled, then when I scroll vertically with the trackpad I tend to shift the horizontal scroll slightly, thereby visually clipping the left side of the document by a character or two. Even after coping with this for months, I still get thrown off by it.
Does this issue occur when all extensions are disabled?: Not sure. Currently the repo depends on an extension, but it seems like something that the core should prevent.

Also filed with GitLens: https://github.com/eamodio/vscode-gitlens/issues/290

Unfortunately I don't think there is anything GitLens (or any extension that uses an after decoration) could do to prevent this. I played with changing the DecorationRangeBehavior but I didn't see any change.
I feel like this changed somewhat recently -- where after decoration now affect scrolling. FWIW, I prefer that they affect scrolling, but they should honor the editor wrapping if they do.
Also related if an after decoration causes a line to grow and cause the scrollbars to appear, when the decoration is removed, the scrollbars don't go back to the previous state.

Currently, inline decorations (such as CSS color boxes or git lens decorations) are implemented via CSS :before / :after selectors and they are unknown to the editor.

As a hack, when the editor is in word-wrap mode, maybe the :after can be position: absolute to pull it out flow so it won't scroll?

@eamodio That is a good thought, but I think that would not work for all kind of :after decorations (that can occur in the middle of the text, and which should push the text to the right, and which should result in an horizontal scrollbar).

Ah, right. I forget you can have them anywhere :)

Let's track in #32856


Chore: update dependencies.

Fix for #1

try v1.2.0 that I published. it has this and more. thank you very much for the PR!


use docker image from docker hub for quick start

use the docker image from docker hub in the sample docker-compose.yml file as the github hosted one seems to have an invalid manifest

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

What error you get with the one from ghcr?

@AlekSi sorry for the confusion but I'm unable to reproduce the error. You may close this pullrequest without merging it.
The error was something about the manifest but I don't remember the specific wording.


Duplicate identifier error in Visual Studio 2015 CTP

Hi
I create the new "typescript" project.
I generate the description file for the app.ts by tsp --target es5 -d app.ts
I immediately receive the duplicate identifier error.
I am not sure how to deal with this issue as I want to have both implementation and description files in my project.
Here are the files app.ts
class Greeter {
    element: HTMLElement;
    span: HTMLElement;
    timerToken: number;

    constructor(element: HTMLElement) {
        this.element = element;
        this.element.innerHTML += "The time is: ";
        this.span = document.createElement('span');
        this.element.appendChild(this.span);
        this.span.innerText = new Date().toUTCString();
    }

    start() {
        this.timerToken = setInterval(() => this.span.innerHTML = new Date().toUTCString(), 500);
    }

    stop() {
        clearTimeout(this.timerToken);
    }

}

window.onload = () => {
    var el = document.getElementById('content');
    var greeter = new Greeter(el);
    greeter.start();
};

and app.d.ts
declare class Greeter {
    element: HTMLElement;
    span: HTMLElement;
    timerToken: number;
    constructor(element: HTMLElement);
    start(): void;
    stop(): void;
}

Do you mean that you're receiving duplicate identifier errors in VS when you have app.ts and app.d.ts (or another file that refers to app.d.ts) open at the same time?
Also could you be more specific with which duplicate identifier errors you're getting from underscore.d.ts?

I have both app.ts and app.d.ts in my project and I am receiving duplicate identifier error. Removing app.d.ts from the project seemed to help in a specific case.

both app.ts and app.d.ts define the same objects. so you will get the duplicate identifier errors. you only need one of them, and not both. your .ts expresses API shape and implementation, where as your .d.ts expresses your API shape only. so if you are coding against it, you can use either, but not both.
I usually think of .js + .d.ts together as package (e.g. a .net dll) that you use .js portion for execution, and .d.ts portion for typechecking. but both are outputs.

@tomitrescak does this resolve your issues?

OK. The weird thing here is tha according to Typescript conventions I should always create d.t.s files and reserence them instead of the .ts files. This behaviour of Visual Studio makes this job a bit more difficult.

If you have a group of files part of a common project, I don't see any reason you shouldn't just reference the .ts file. The .d.ts is more useful to an external consumer.
If you're familiar with C/C++, think of a .d.ts as a header file that describes the interface to a linked library, which is the .js file here.

Thanks for your help.


Append to select

Is there an easy way to append label to form select helper?  The following does not seem to work...
<%
    options = [['first', 1]]
    (2..30).each {|day| options << [day.to_s, day]}
    options << ['last', 31]
%>
<%= f.select :due_day, options, append: 'day', label: 'Due Day' %>

Hi @carloslopes !
I use this regularly, and it would be great to get it into this gem.  Here is a working example for Bootstrap 4 alpha 6:

@

  <select class="form-control custom-select" id="work_project_id" name="work[project_id]">
    <option value="1">First</option>
    <option value="2">Second</option>
    <option value="3">Third</option>
  </select>

@

Hey @donv, I'm not actively working on this gem anymore, sorry :(
@mattbrictson can you take a look on this?

Sorry, this fell off my radar...
I think @donv's code is cool, it is not exactly what I was trying to do.  The issue is not appending groups but adding a label in a simple declaration.  The output would look more like this JSBin I just quickly made: https://output.jsbin.com/pelayi, or in HTML:
  <form role="form" class="form-horizontal" action="#" method="post">

        Text Label

          $
          <select class="form-control custom-select" id="my_input" name="my_input">
            <option value="1">First</option>
            <option value="2">Second</option>
            <option value="3">Third</option>
          </select>

  </form>

It looks like we support :append for simple fields but not for the select helper. Unfortunately I don't have time right now to dig into why we have this limitation or how to solve it. A naïve approach would be to modify the select helper to add support like this:
diff --git a/lib/bootstrap_form/form_builder.rb b/lib/bootstrap_form/form_builder.rb
index d89c966..e127c7b 100644
--- a/lib/bootstrap_form/form_builder.rb
+++ b/lib/bootstrap_form/form_builder.rb
@@ -71,13 +71,17 @@ module BootstrapForm
     if Gem::Version.new(Rails::VERSION::STRING) >= Gem::Version.new("4.1.0")
       def select_with_bootstrap(method, choices = nil, options = {}, html_options = {}, &block)
         form_group_builder(method, options, html_options) do
-          select_without_bootstrap(method, choices, options, html_options, &block)
+          prepend_and_append_input(options) do
+            select_without_bootstrap(method, choices, options, html_options, &block)
+          end
         end
       end
     else
       def select_with_bootstrap(method, choices, options = {}, html_options = {})
         form_group_builder(method, options, html_options) do
-          select_without_bootstrap(method, choices, options, html_options)
+          prepend_and_append_input(options) do
+            select_without_bootstrap(method, choices, options, html_options)
+          end
         end
       end
     end

Does anyone want to run with this to see if it works, and if so, open a PR?

@mattbrictson I'd be happy to do that.  My personal focus is on Bootstrap 4.  Which branch should I use as base for this?

@donv I would prefer to first try this against master (i.e. Bootstrap 3), since that will be more likely to get merged and released to rubygems quickly.

OK, I'll do master first.


TypeError: Queue is not a constructor

Hello!
When running version v1.2.0 of concaveman with Webpack (in a vue-cli environment), the following error occurs:
TypeError: Queue is not a constructor
    at findCandidate (index.js:95)
    at concaveman (index.js:65)

In Chrome Debugger, Queue is an object which holds a transpiled ES6 module, and has __esModule and default properties.
Is this a bug in concaveman or my Webpack configuration?
Thanks!

@mbullington, we have same issue after update to v1.2.0:
index.js:95 Uncaught TypeError: Queue is not a constructor
    at findCandidate (index.js:95)
    at concaveman (index.js:65)
    at convex (main.es.js:45)
    at centerOfMass (main.es.js:86)

Points to line var queue = new Queue([], compareDist):
function findCandidate(tree, a, b, c, d, maxDist, segTree) {
    var queue = new Queue([], compareDist);
    // ...
}

It's a known Webpack bug that triggers when a CommonJS package is require-ing a dependency that additionally exposes a module entry point and has a default export. I guess a fix would be to switch to ES bundling for this module too.

At least in the short-term, using webpack-chain we monkey patched tinyqueue like so:
// TODO: Track this issue
// https://github.com/mapbox/concaveman/issues/18
config.resolve.alias.set('tinyqueue', require.resolve('tinyqueue/tinyqueue.js'))

We've caught the issue too. Lost a day of debugging.

I guess a fix would be to switch to ES bundling for this module
Do you mean each client has to do it? Is it possible to fix it once and for everyone to avoid the multiplication of pain?

We manually apply @grassick's fix for now and it works locally but this bug still breaks our build pipelines.

We manually apply @grassick's fix for now and it works locally but this bug still breaks our build pipelines.

This bug seems to be breaking @turf/convex

@braco @sedot42 You can use https://www.npmjs.com/package/patch-package. Works like a charm.

@braco @sedot42 You can use https://www.npmjs.com/package/patch-package. Works like a charm.

Thanks for the suggestion @alebinson. I don't want to introduce another 124 dependencies to our project, so we resolve the dependency via a resolutions block in Yarn's package.json. Even better of course if you control the repo yourself.
 "resolutions": {
    "@turf/center-of-mass/concaveman": "https://github.com/grassick/concaveman#patch-1"
  },

Thanks for the suggestion @alebinson. I don't want to introduce another 124 dependencies to our project, so we resolve the dependency via a resolutions block in Yarn's package.json. Even better of course if you control the repo yourself.
 "resolutions": {
    "@turf/center-of-mass/concaveman": "https://github.com/grassick/concaveman#patch-1"
  },

Any update on this bug getting resolved? Really appreciate all the work MapBox does for the community.

Hi all, any update on the issue?

I am also facing the same issue, any update on the issue? Thanks.

You can get around this issue by installing tinyqueue as a dependency in your project:
npm install tinyqueue

And then configuring your webpack config with an alias within the resolve object like so:
 resolve: {
        alias: {
            tinyqueue$: require.resolve('tinyqueue/tinyqueue.js')
        }
  },

Hope this helps.

@mourner who at Mapbox is responsible for merging the PR fixing this issue? This issue is a year old, and is a big bug that's effecting a lot of users.

Sorry for the late reaction everyone, I was holding back on this because I really don't like the default hack and planned to fully convert this package to a pure ES module with type: "module" instead. Never got around to it but I will at some point.

You can get around this issue by installing tinyqueue as a dependency in your project:
npm install tinyqueue

And then configuring your webpack config with an alias within the resolve object like so:
 resolve: {
        alias: {
            tinyqueue$: require.resolve('tinyqueue/tinyqueue.js')
        }
  },

Hope this helps.

You are so great! It is effective,thanks!

You can get around this issue by installing tinyqueue as a dependency in your project:
npm install tinyqueue

And then configuring your webpack config with an alias within the resolve object like so:
 resolve: {
        alias: {
            tinyqueue$: require.resolve('tinyqueue/tinyqueue.js')
        }
  },

Hope this helps.

You are so great! It is effective,thanks! @JamesLMilner


Risks for configuration inconsistence between mysql_daemon and nextcloud_db_backend

hello @staticdev
Can you elaborate the use of mysql_daemon  (and  __mysql_daemon ) for managing the mysql/mariadb server ?
there could be cases when the user installed and used mysql instead of mariadb like in debian bullseye where it is still possible to choose beetween default-mysql-server and mariadb-server.
if one chooses mysql, the installation will still select the proper package (as defined here) but the following task will fail unless mysql_daemon is specified as var with the same value.
It feels a bit redundant. 😕
Have you met issues that are preventing to use nextcloud_db_backend instead ?
Regards
Aal.

I think there is a confusion between the names of installed packages such as default-mysql-server and mariadb-server and the name of the service (daemon) mysql_daemon. For some reason, different OSs such as Debian 11, even if you install mysql will create a service called mariadb.
I already introduced this variable since it will be used to replace the DB installation with the role as we planned in https://github.com/aalaesar/install_nextcloud/issues/139

Thanks for your answer @staticdev
It appears that debian11 use mariadb as mysql replacement. so calling the meta package default-mysql-server effectively install mariadb-server (source)
This breaks the user's hability to choose mysql or mariadb in the role for recent OS.
at least for debian 11 but I would like to test it for other OSs.
It adds another point to removing the DB installation from the role so distribution choices won't impact the role.
My concerns is  about having 2 variables for the same service but one exposed  for users while the other is not, or not documented.

I agree 100%, actually in last months I realized proper and secure instalation of Nextcloud is really complex, being db installation just another complex part.
It is a good idea to rely on separate popular roles to abstract a bit of this complexity. I will try next week to dedicate some time into adding this db role and make install_nextcloud more maintainable in this regard.


Keepinventory

Did i use an old version, or does this mod don't work with the vanilla function keepInventory=?

In version 1.3.3 (and later), I made it check the keepInventory game rule and the intended behavior is that if keepInventory=true, then no items are transferred from the player's inventory to the corpse. If that is not what you observe, then please provide a detailed description of what actually happens, what are the circumstances that you used to test this, whether keepInventory works when this mod is not installed, and whether the bug still persists when you remove all mods except Lootable Bodies. With that information, I should be able to track down any bug.
Also, make sure you are launching your server with just Forge and not a plugin-hosting launcher when you are testing for bugs. The plugin launchers tend to inject extra programming hooks which have been known to interfere with Lootable Bodies.

Are you still having issues with Lootable Bodies or can I close this issue thread?

Closed due to lack of response.


ENYO-944: Grab-bag of optimizations

showOnRender feature
enyo.LightPanels implementation
enyo.Loop rAF-based run-loop
enyo.BackgroundTaskManager implementation (and related kinds enyo.PriorityQueue, enyo.SystemMonitor, and enyo.TaskManagerSupport).

Enyo-DCO-1.1-Signed-off-by: Aaron Tam<EMAIL_ADDRESS>

Manually merging this work to coordinate some other merges prior to pushing to master.


Create: abstract-class for search paper

Created an abstract class PaperSearch for paper retrieval.

Modified both the SemanticScholarNode and OpenAlexNode to inherit from this new abstract class as well as the Node class.

Added test scripts for both the SemanticScholar and OpenAlex nodes. Be sure that the API calls have been mocked in this tests.

@genga6
If possible, I would also like you to add test execution using github actions.


dev to master merge

merged dev comit 17db0f5 to master.
Merged because some may still use master branch

Apperantly dev branch commits didnt merge. but it doesnt matter.
Here is a pull request without merge conflicts. tested and it worked

@Gary-BOT @douglascamata  how do you think about @jtdroste

Are you asking what ya'll think about me? :P
So if we accept this, we lose ALL master's history, and we will have problems later down the road merging dev and master.  tl;dr do not do this.
A contributor will have to do a merge request to master.  Closing this.


Fix sync-start bug + improve sync-start comments

This fixes the bug outlined here: https://github.com/ethereum-optimism/optimistic-specs/pull/353#discussion_r846131762, and adds a lot of comments + improves code structure to make the algorithm easier to understand.

Still needs a regression test.


JS ctags refinements

The following was adapted from https://gist.github.com/tim-smart/199338

Distinctly label variables, constants, and individual JavaScript types
Change getter/setter to own list type separate from function/class/method
Support one-character length identifiers
Avoid switch in apparent regex for inline functions (not so sure about the legitimacy of this entire line though: --regex-JavaScript=/(,|^|\*\/)[ \t]*(while|if|for|switch|function|([A-Za-z_$][A-Za-z0-9_$]*))[ \t]*\([^)]*\)[ \t]*\{/\3/,function/
For identifiers, avoid matching if begin with digit
Avoid duplication of function/method/class regexes given built-in parsing

Another useful addition would be:
--regex-JavaScript=/(,|(;|^)[ \t]*(var|let|([A-Za-z_$][A-Za-z0-9_$.]*\.)+))[ \t]*([A-Za-z_$][A-Za-z0-9_$.]*)[ \t]*=[ \t]*[^tfn0-9"'{\[]+([,;=]|$)/\5/,unknown/
It is a little imperfect, however, as I had to overcome the regex engine allowing negative lookahead, so I disallowed 't' (for true), 'f' (for false or function), and 'n' (for `null) to avoid displaying these. One can remove those characters to err on the side of caution, but there will be duplicates if one also has enabled the boolean, function, or null types.
These additions would also be useful:
--regex-JavaScript=/(,|(;|^)[ \t]*(var|let|const|([A-Za-z_$][A-Za-z0-9_$.]*\.)+))[ \t]*([A-Za-z_$][A-Za-z0-9_$.]*)[ \t]*=[ \t]*null/\5/,null/
--regex-JavaScript=/(,|(;|^)[ \t]*(var|let|const|([A-Za-z_$][A-Za-z0-9_$.]*\.)+))[ \t]*([A-Za-z_$][A-Za-z0-9_$.]*)[ \t]*=[ \t]*undefined/\5/,undefined/

...and changing the variable definition to undefined:
--regex-JavaScript=/(,|(;|^)[ \t](var|let|([A-Za-z_$][A-Za-z0-9_$.].)+))[ \t]([A-Za-z_$][A-Za-z0-9_$.])[ \t][ \t]([,;]|$)/\5/,undefined/
Note that the preceding does not distinguish declared variables in a list from function parameters, however.

And this expanded array to catch non-literal arrays as well:
--regex-JavaScript=/(,|(;|^)[ \t]*(var|let|const|([A-Za-z_$][A-Za-z0-9_$.]*\.)+))[ \t]*([A-Za-z_$][A-Za-z0-9_$.]*)[ \t]*=[ \t]*(\[|(new[ \t]+)?Array\()/\5/,array/

And to get more granularity in the built-in functions, one can instead use the following (being careful to deselect the existing function and method types to avoid duplicates):
--regex-JavaScript=/(,|(;|^)[ \t]*(var|let|([A-Za-z_$][A-Za-z0-9_$.]*\.)*))[ \t]*([A-Za-z0-9_$]+)[ \t]*=[ \t]*function[ \t]*\(/\5/,function-expression/
--regex-JavaScript=/(,|(;|^))[ \t]*(([A-Za-z_$][A-Za-z0-9_$.]*\.)+prototype\.)[ \t]*([A-Za-z0-9_$]+)[ \t]*=[ \t]T*function[ \t]*\(/\4\5/,prototype-method/
--regex-JavaScript=/(,|^|\*\/)[ \t]*([A-Za-z_$][A-Za-z0-9_$]+)[ \t]*:[ \t]*function[ \t]*\(/\2/,object-method/
--regex-JavaScript=/function[ \t]+([A-Za-z_$][A-Za-z0-9_$]*)[ \t]*\([^)]*\)/\1/,function-declaration/

@brettz9 Awesome!!!!! Thank you very much!

Glad you liked it! Would you like me to add the additionally discussed entries? They seem to be working nicely on my end. My PR was only for the items mentioned in the first post...


Check output format

Check format function added to main code.
Improved codes.
This commit is based on #55 and #57 's comments.

Thanks :)


Secret Manager Integration

Task Description
We'll be integrating GCP's secret manager to IKS & SS instance.
Integration points:

[x] DB creds
[x] Mnemonic(SS & IKS)
[x] RSA Pem

To explore how can we integrate with AWS's secret manager.

Closing this issue as discussed with dev.


Create Workflow Agent for gathering Workflow/C.Engine/C.Result information

Available as part of the litmus-portal.


Move closer alert is cut off

See attached screenshots in portrait and landscape

Confirm fixed as of 1.4


Integration with Pyomo (or similar)

Application
I want to solve my ILP/QP with multiple solvers, including (but not limited) to the QA. I'd much rather use a single piece of code for formulating the problem. For example, via a standard interface like Pyomo. However, I cannot do that for the QA - I need to reformulate the problem using dimod (or another library).
Proposed Solution
I'm suggesting to add an integration with Pyomo, or a similar library that supports solving non-linear integer optimization problems. I should be able to write code to formulate my optimization problem only once, and solve it with various common conventional solvers like Gurobi, CPLEX, XPRESS, etc., as well as with the QA.
Alternatives Considered
Using CQM directly - the existence of CQM makes this relatively easy. However, it introduces redundancy, which can be a source of errors, as well as requiring increased effort to test, validate, and maintain the codebase.

Agree that would be useful. In the mean time, we do support lp files which can serve as an intermediate representation.

Thanks, @arcondello I wasn't aware of this option, that should work nicely for now! But still it would be nice to have a more direct integration, to avoid the overhead of writing to a file and reading from it...

Very much agree with this request - would love to use an integrated approach with Pyomo and pass the problem to QA. CQM might enforce that no cubic or higher order terms be present in the formulation to start with. Might be generally be usable for MILP/ LPs/ QPs/ QCQPs solvable by CQM.


add pubmed update script and a functional test for testing first/corresponding author flag

When you are free, can you check out the code to see if there is any change I can make to improve the performance?
Also check to see if the functional test makes sense..
Thanks!

I've taken a quick look at the code regarding (i) module-level variable declaration and (ii) logs. The issue we had with module-level variables and object initialization outside functions was that whenever we were importing functions from modules with such declarations we were also initializing objects that should have been initialized only when executing the module as top level code (name == "main"). This means that it's ok to declare variables as module-level if they are not going to create issues or conflicts when the module or some of its functions are imported in other modules. For logs, I don't think we should focus on a single format as LogStash (or the exporters we'll use to send our logs there) probably supports multiple formats, even though using the standard logger with default formatting is probably the best solution at the moment.

Hi Juancarlos, thanks for looking over the code!! These are all good questions! I will try to explain what I think during my coding process.

line 303 in update_pubmed_papers.py, this is happening only when mod is passed in (see line 300).

In normal update case, if the old md5sum = new md5sum, it means no new update from pubmed. So we don't need to do anything (even though there is a first/corresponding author change for an author in the database).
In functional test, it won't check for old/new md5sum, so it will compare the data in the database and the data directly from pubmed to decide what to do next. Since in the functional test we have manually updated the data in the database before calling 'update_data' module so the program will decide if it needs to update the records. If there is a first/corresponding author flag, the script will only give a warning and will not update the authors for the given paper. So I think the functests should be able to catch this. Do you think so?

About md5sum, I am not sure if the xml files in s3 match the data in our prod database. If yes, we can upload the files to s3 when the script runs against rdsprod (I can update code to do this).

For this initial update, the script will download all xml files from pubmed and generate the json files for all of them, and compare the data from the database and the data from the json files to decide if there is a change (basically skip the md5sum checking since no previous versions of xml files/md5sum available). But after this initial run, all xml and json files and their md5sum will be in the system so later the monthly update will check the md5sum for each file in the system
(from the initial update) or in s3 (if we store them in s3) with the md5sum from newly downloaded/generated xml/json files and only proceed if there is a difference.

About downloading xml files in 150000 batch,  when the script was downloading the xml files for MGI, it stopped in the middle (multiple times).. so I break it into 150000 xml files at a time and add a long break in between (10min break) to make PubMed "not so mad". =)

Not sure this is due to my slow internet connection. Maybe it is ok if it is running in cloud.

About updating data by mod, just so we can have a clean report for each mod. Also there are so many papers so updating everything can take some time. Anyway, we can have a wrapper script to run it against each mod. Of course if we really want to run it against everything, we can add a 'no mod' option. I understand that some papers belong to multiple mods, but hopefully,  it shouldn't take too much extra time downloading them..

About logging, I agree. We should talk about this. The log_file that ends in .log with fw.write contains all the details about the update (old value, new value etc). The log.info part needs to be updated based on curators' specs.

About "I don't think you want to filter on ReferenceModel.mod_corpus_association because we want to update all the PubMed papers, even if they're not inside_corpus for any mod. I guess it makes sense that curators would only want to update papers if they're in-corpus, but I would have thought we'd want to update all. Maybe ask Ceri and Kimberly ? Related to that, get_pmid_to_reference_id should check that CrossReferenceModel.curie.like('PMID:%') is not obsolete, otherwise we'll be updating data to obsolete PMID connections. (So far this is the only thing I'm really sure should be different)."

I am not sure I understand this part, updating is based on mod so it will only retrieve the data that are needed and update the data associated with a given mod, right? Am I missing anything?
By the way, good catch about the obsolete PMIDs!! I will add a check for this.
ps. how many papers are not associated with a mod ( if any) ?

In the initial update against local database, even though the resource_id looks the same but it still got updated (only happened for a few papers)  so I was confused.. anyway, not a big issue.

"update_cross_reference_for_resource is written but not called by anything yet ? That's okay" good catch!!
good catch! forgot to remove (wrote too many functions lol). will remove it.

"I don't really know how sqlalchemy works, so I don't know db_session.add(x) works, but I assume you've queried x from the database (or created it from a model + data) and you're updating x, and doing .add(x) updates the database record by insert or update as appropriate ?"

It works. you are right! x is a record "handler" from database through model so we can add/update data through it.  I code this way for sgd all the time.

Hi Valerio, Thanks for checking the code. Do you mean that the functional test is importing those python modules inside of
"test_first_corresponding_author_flag' vs importing them in the beginning of the functional test script?  I was assuming these modules should not affect other tests.. ??
Yeah we need to come up with a std way to send reports. Right now, this script is calling the resource updating module, xml downloading, and json generating modules so we need to think about what to send to logger from each program to give curators a good summary of everything.
Thank you so much for spending time to review the code @azurebrd @valearna

Hi @sweng66 ,
Thanks for breaking things down and numbering them :)
1 - Ah, okay.  So for now the old vs new md5sum check is only happen when given a list of PMIDs.  And when doing based on MOD, it's going to always update, and that's because it needs to update everything because of category and language, but in the future you will update it to also do that check ?
Ah, got it, the functional test assumes that the data has changed, so it's not testing the old vs new d5sum.  That makes sense.
2 - The files in s3 should match the data in our prod database.  For some reason I can't access the s3 files with my account right now, so I can't doublecheck.  The original files should go into a directory that means original/ and the latest files should go into a directory that means latest/ .  We should also upload them as Glacier IR storage type, and I think the upload xml to s3 function already does this.  Sounds great about current run vs future runs.  Did what I write about sort_dqm_json_reference_updates.py merge_md5dict[mod] = {**old_md5dict[mod], **new_md5dict[mod]} vs merge_md5sum makes sense ?  It's okay if not, specially if you're not going to deal with it until the future update.
3 - Oh, good to know about the xml files failing.  I hadn't had that problem, but sounds good, as it's not really time-sensitive.
4 - Yeah, makes sense to keep things clean and simple to understand instead of adding extra code to check for which mod something belongs to and copying the output to several reports.  You're right that it shouldn't take long to re-process multi-mod files.
5 - Ah, gotcha.  I don't think the curators want the output of log.info because that has a bunch of timestamps and which script/function generated the line.  But they still haven't told us what they want, so who knows.  I thought the log.info would be what would get added to logstash, but I really don't know how that works (defer to Valerio, and we can do it on the other ticket)
6 - I thought we wanted to process all of our papers with non-obsolete PMIDs, instead of those belonging to mods.  When papers get created via PMID, they don't get a mod_corpus_association value (it's not a required field).  Same if a paper gets created manually and a PMID gets added to it later.  Possibly curators could assign papers to other MODs as needs_review and never check it, or even be out_of_corpus for all mods.  I suppose it'd be good
to check with Ceri and Kimberly whether they want all non-obsolete PMIDs updated, even if they're out_of_corpus or needs_review.  I agree that this makes the logs / reports much simpler.
About how many papers are not associated with a mod, some PMIDs got created through reference_comments_and_corrections and I think (but not sure) that they don't get attached to any mods, just to the paper that was its parent (or whatever connection).
I tried these queries on pgadmin, but it kept spinning and never returned anything.
SELECT * FROM public.reference WHERE reference_id NOT IN
( SELECT reference_id FROM public.mod_corpus_association )
SELECT * FROM public.reference WHERE reference_id IN
( SELECT reference_id FROM public.cross_reference WHERE curie ~ 'PMID' )
AND reference_id NOT IN
( SELECT reference_id FROM public.mod_corpus_association )
7 - yeah, weird.
8 - cool :)
9 - cool, thanks !

Hi @azurebrd

Yes, when given a MOD, for current run, it will always update everything because of category and language like you said (also because there is no previous version of xml files available in the local system). For future runs (all category and language columns have been updated), the program will check new md5sum vs old one before updating (since all "old" version of xml files will be available in the local system in that time).

When given a list of PMIDs, the program will skip md5sum checking step and will always compare the data in the db vs data in json before updating. Assume the pmid list is not huge so comparing data in db vs in json won't cost much.

It is great to know that the data in db matches the data in s3. So instead of	checking old md5sum in the local system, the program will be updated to check the md5sum from s3 for the future runs. Thanks for the pointer, I will check out sort_dqm_json_reference_updates.py.

Agree.

The program is currently skipping the Obsolete papers and only updating the MOD associated ones, but I will think	about how to deal with the ones that are not associated with a MOD.
I also tried those SQL queries and they never return anything.

@sweng66
1 - Ah, I'm not sure we can rely on the old xml and json files being in the system.  I originally wrote it similarly, but then the plan was to run it inside docker, and I wasn't sure the files would always be there.  At the time I didn't consider that we could have a docker volume from outside.  But that would mean it would always have to run from the same computer.  I'm not sure we need to compare the existence of files, since we can compare the md5sums, which will always be quickly downloadable from s3.
Sounds good for PMID list always updating without md5sum check.
2 - Oh !  Great, thanks :)
4 - Cool !
I guess one could sqlalchemy the separate parts of the queries and do the joins in python.

@sweng66 Got access to the account.  There's develop/ and prod/  , there's xml/ vs json_from_xml/ , and there's original/ vs latest/
Here's a URI where all the xml files should be.  Replacing the pairs of values above you can get the other versions.
s3://agr-literature/prod/reference/metadata/pubmed/xml/original/
I think the plan is to use develop just for testing a few at a time, so no need to upload to all the files every time a run happens.  But if you do, that's probably fine.  prod should always get all the files for live runs.
xml holds xml files and json_from_xml gets the files generated by xml_to_json from the xml files.
original gets the file the first time the PMID comes into the system.  latest gets the file every time a file gets generated after the first time.  The idea is that pubmed could delete the papers and then we lose the source of truth, so we want to keep a copy of  the most recent file, but also the original because sometimes authors won't get updated, so the latest file won't necessarily be the source of truth.
Thanks !

In RDSDEV:
total pubmed papers:				                         689,012
Total pubmed papers associated with mod: 	                 683320
Total pubmed papers not associated with a mod:	    5,692
Will update these pubmed papers that are not associated with a mod.

cool cool !  I guess we should check with Ceri and Kimberly about who should get notifications for those, but I imagine no one really wants them, and they don't matter too much.

yeah.

Just saw your comments about xml/json files in s3. Will look into it soon. Thanks @azurebrd

"merge_md5dict[mod] = {**old_md5dict[mod], **new_md5dict[mod]}
merges any data in the old md5sum that isn't in the new one into the new one,
and that's what a lot of merge_md5sum does "
YES,  lol @azurebrd

@azurebrd Just looked at the code in sort_dqm_json_reference_updates.py for md5sum checking and saving..
Seems like the md5sum files are generated from dqm json files and they are MOD based,  one file for each mod.
Does that mean same paper from different mods may have different md5sum?
The md5sum generated through 'generate_json' function will be different from the one from mod dqm file, I think, at least the allianceCategory and language fields are different. right?
Do you think we should upload a new md5sum file to s3 just for updating script?  This file will contain the md5sum for all pubmed papers in the prod database and it is generated through 'generate_json' function. We can start uploading this md5sum file along with updated xml files to s3 after we are done updating the rdsprod database for current run.
And for all future updates, the updating script will download this md5sum file from s3 and compare with newly generated md5sum to check if a paper gets updated. In the end, the script will upload the newly generated md5sum file to s3 (only for prod update) and also upload the updated xml files to s3
The pubmed search program can  be fixed to add the md5sum for each new paper to this file as well.
Does this sound reasonable?  Or am I missing anything?

Hi @sweng66 .  Yeah, there's multiple source of truth for each paper.  If the paper's at PubMed, most of the source of truth is the PubMed xml, but some stuff can only come from each MOD, like the mod_reference_type (I can't think of anything else right now, but I'm not looking at the code ;  for a while the allianceCategory was also coming from there), and for papers that are not at PubMed, then the mods are the whole source of truth, which yes, has conflicts when different mods have the same paper, but so far we only know of one paper created separately by two mods, and we only know they're the same paper because they have the same DOI (since many papers can have the same title, or be the same paper but have the authors in a different format, and so on, so it's really hard to know whether any given two papers in our database are actually different papers).
For each paper that comes from each source (a mod or pubmed), there's an md5sum file https://s3.console.aws.amazon.com/s3/buckets/agr-literature?prefix=develop%2Freference%2Fmetadata%2Fmd5sum%2F&region=us-east-1&showversions=false# (and also a prod version for the prod files, which is what should really be used).  When doing any updates from any source, the idea is to download its md5sum file, then for each new paper generate its new md5sum, and only if it's different from the s3 file md5sum, do we update the data in ABC, then update the md5sum file and upload it back to the s3 location.  I think this is basically what you said, but when you said "Do you think we should upload a new md5sum file to s3 just for updating script?" I wasn't sure if you meant that we needed to create a whole new type of file for this purpose.  I think we want to use the existing file, then update it, and then yes, upload this new file to s3 for future updates.  Please let me know if this isn't clear.  Thanks !

@azurebrd Oh cool. I didn't know the file is already in s3.  I don't have access to the alliance AWS account so I am not sure what is in the s3. Sorry for the confusion. I will check things out and fix the program to use this file. Thanks!!
About AWS access,  who should I talk to? Adam Wright?

@azurebrd Adam just setup the passwd for me to access aws from console so I am able to see what is in there now =)
https://s3.console.aws.amazon.com/s3/buckets/agr-literature?prefix=develop%2Freference%2Fmetadata%2Fmd5sum%2F&region=us-east-1&showversions=false#
I saw the file PMID_md5sum
Assume this is the file the updating script should use. Correct?

@azurebrd Just checked xml_to_json.py. It is awesome to know that this program is updating the PMID_md5sum every time it is generating the json files so  the PMID_md5sum in s3 is always up to date as long as the PubMed papers are loaded/updated through this pipeline.

@sweng66 Oh, cool that Adam got you set up !  In theory Valerio or I could also set you up, but I don't know how, so we would have had to ask Valerio.
Yeah, that would be the file to update when doing testing in develop against rdsdev.  And then you'd change the 'develop' to 'prod' for live changes.
Oh !  I'd forgotten that xml_to_json does that update !  That's great.  It should always be correct for that to update the md5sum file.  The uploading of the xml and json files should still happen separately to the correct s3 locations based on develop/prod original/latest (first time vs update) and xml/json .  Please let me know if that doesn't make sense.  I don't think I wrote code to already handle all that (certainly not for updates, since that wasn't happening yet, but there's probably code for develop/prod and xml/json)

@azurebrd Yes, we still need to upload the xml and json files and I will let you know if I need any help. Thanks


Dragon's Dogma 2 Character Creator - Menu not showing

Describe the bug
The menu is not rendering. It sometimes flickers quickly at startup but then vanishes. Pressing the menu key does bring up the cursor but still no menu.
Upload logs and any crash dumps
re2_framework_log.txt
Identify the REFramework version
Github developer build e1a13a6 (Nightly 629)
To Reproduce
Steps to reproduce the behavior:

Open the game.
Press the menu key.
Notice the cursor appearing, but no menu.

Expected behavior
The menu showing up.
Desktop and VR (please complete the following information):

OS:   Win 10
GPU: RTX 4080
CPU: i5 12600kf

Additional context
I've tried reinstalling, disabling every type of other overlay (Nvidia, Steam), clearing shader cache, but nothing helped.
These are the latest errors the log gives:
[error] [D3D12] Failed to get back buffer for rtv.
[error] Present failed: 87a0001

Try the newest nightly and report back: https://github.com/praydog/REFramework-nightly/releases/

Sadly no improvement, behaves pretty much exactly like before.
New logs:
re2_framework_log.txt
excerpt of other errors
I swear the log looks different every time. Sometimes it only includes the "present failed: 87a0001" error, sometimes no errors at all and sometimes a 60MB monstrosity I can't even upload.
short video of some flickering (game always starts in this window for me and then switches to fullscreen)
Not sure what else I can do to help with this.

Download nightly again and upload logs https://github.com/praydog/REFramework-nightly/releases/
It won't fix anything but should give me more context and if I should rule out third party software causing conflicts.

Literally just did (Nightly 641). I don't see any obvious errors but I've had that before as well.
re2_framework_log.txt
The menu flickers and disappears when the game fullscreens. For context I have the Nvidia overlay and Steam overlay disabled. No Afterburner running either.

OH. I know what it is. It's DLSS3... I think. It's inserting fake present calls, confusing REFramework. I have Hardware Accelerated GPU Scheduling turned off, so DLSS3 was not auto-activating for me so I couldn't see the issue.

I'm getting mixed feelings here now, I just had it work once by randomly changing things but can't reproduce.
Shouldn't deleting the framegen DLL stop it from engaging? Because that doesn't change a thing.

 Restarting without GPU Scheduling and the menu renders flawlessly.
So weird because you can't change any settings in this demo.
Good luck fixing it though ^^

Fixed in e206942355177d0c7a6ca0f294acd55c8e26169e


(,,new video Leaked,,!!) *HMP Wandsworth Prison Officer Video Leaks on X Twitter and Facebook

HMP Wandsworth Prison Officer Video Leaks on X Twitter and Facebook
Leaked Viral Video Sparks *** and Outrage Across Social Media Platforms
In a shocking turn of events, a leaked viral video has taken the internet by storm, garnering millions of views and sparking intense debates among netizens. The video, which surfaced online yesterday, shows footage of a high-profile public figure engaging in questionable behavior, leaving viewers stunned and divided. As the video continues to circulate rapidly across various social media platforms, it has ignited a firestorm of *** and outrage, with many calling for accountability and transparency.
⬇👇⬇👇⬇👇⬇👇
Watch 🟢 ➤ ➤ ➤ 🌐Click Here To link (Full Viral Video Link)⬇
🟢 ➤ ➤ ➤ 🌐🔴 ➤►DOWNLOAD (Full Viral Video Link)⬇
⬇👇⬇👇⬇👇⬇👇
The leaked viral video has quickly become a trending topic on Twitter, Facebook, and Instagram, with users expressing shock, anger, and disbelief at the contents of the footage. The incident has also prompted discussions about privacy rights, ethical standards for public figures, and the power dynamics at play in today's digital age. As the story unfolds and more details emerge about the origins of the video and its implications, one thing is certain: this unprecedented leak has captured the attention of people worldwide and raised important questions about accountability in the age of social media.
TAG :
HMP Wandsworth Prison Officer viral mp4
HMP Wandsworth Prison Officer di lejja
HMP Wandsworth Prison Officer viral di lejja
HMP Wandsworth Prison Officer
HMP Wandsworth Prison Officer viral
viral HMP Wandsworth Prison Officer di lejja
HMP Wandsworth Prison Officer video viral
video viral HMP Wandsworth Prison Officer di lejja
A woman has been arrested over a social media video allegedly showing a member of prison staff having *** with an inmate in a jail cell. Scotland Yard said it launched an investigation on Friday The police are investigating after a video surfaced online appearing to show a female prison officer having *** with an inmate in a cell in Wandsworth Prison. It is not known when the footage A woman has been arrested after a video of a prison officer apparently having *** with an inmate went viral on social media.. The video, supposedly filmed recently inside a jail cell at HMP A woman has been arrested after a video emerged appearing to show a prison officer having *** with an inmate at HMP Wandsworth.. Police had previously said they were investigating the footage Later in the video he adds: “This is how we roll in Wandsworth.” The video was filmed in Wandsworth Prison in south west London. Picture: Getty On Saturday, the Metropolitan Police confirmed the arrest saying: “A police investigation was launched on 28 June after officers were made aware of a video allegedly filmed inside HMP Wandsworth. A FEMALE prison officer is being investigated by police over a shocking *** tape filmed in a cell at a scandal-hit jail. She was seen performing a lewd act and having *** with a burly lag as her diâ¦ The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. Date/Time Date(s) – June 29, 2024 – July 1, 2024 All Day. Categories No Categories . 13 sec ago-Scandal Erupts: HMP Wandsworth Prison Officer Video Leaks on Twitter Scandal Erupts: HMP Wandsworth June 29, 2024 – July 1, 2024 @ All Day – 3 sec ago-HMP Wandsworth Prison Officer New collections of HMP Wandsworth Prison Officer now being a creator on Fanfix uploading adult contents. Social The HMP
Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. June 29, 2024 – July 3, 2024 @ All Day – HMP Wandsworth Prison Officer New collections of HMP Wandsworth Prison Officer now being a creator on Fanfix uploading adult contents. Social media star 11 sec ago-HMP Wandsworth Prison Officer New collections of HMP Wandsworth Prison Officer now being a creator on Fanfix uploading adult contents. Social media star HMP Wandsworth Prison Officer been posting short videos and naughty pics on Tiktok pla Female Prison Officer filmed having s_x with inmate in HMP Wandsworth All mod cons and amenities. julia kim Â· Original audio ((LEAKED@VIDEO)) HMP Wandsworth Prison Officer Video Leaks on Twitter by Hmp-wandsworth-prison-officer, released 01 July 2001 A woman has been arrested after the emergence of a social media video apparently showing a prison officer having *** with an inmate in a cell. The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. June
29, 2024 – July 3, 2024 @ 12:00 am – Watch ð¢ ð Click Here To link (Full Viral Video Link) ð´ DOWNLOADðð (Full Viral Video Link) 13 sec ago-HMP Wandsworth Prison Officer New The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen. The video, supposedly filmed at HMP Wandsworth, showed a woman wearing a uniform performing a *** act on a man before having *** with him. The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen. The video, supposedly filmed at HMP Wandsworth, showed a woman wearing a uniform performing a *** act on a man before having *** with him. Date/Time Date(s) – June 29, 2024 – June 30, 2024 All Day. Categories No Categories . 13 sec ago-Scandal Erupts: HMP Wandsworth Prison Officer Video Leaks on Twitter Scandal Erupts: HMP Wandsworth June 29, 2024 – July 4, 2024 @ All Day – 3 sec ago-HMP Wandsworth Prison Officer New collections of HMP Wandsworth Prison Officer now being a creator on Fanfix uploading adult contents. Social .. Twitter
Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. June 29, 2024 – July 3, 2024 @ All Day – HMP Wandsworth Prison Officer New collections of HMP Wandsworth Prison Officer now being a creator on Fanfix uploading adult contents. Social media star 11 sec ago-HMP Wandsworth Prison Officer New collections of HMP Wandsworth Prison Officer now being a creator on Fanfix uploading adult contents. Social media star HMP Wandsworth Prison Officer been posting short videos and naughty pics on Tiktok pla Female Prison Officer filmed having s_x with inmate in HMP Wandsworth All mod cons and amenities. julia kim Â· Original audio ((LEAKED@VIDEO)) HMP Wandsworth Prison Officer Video Leaks on Twitter by Hmp-wandsworth-prison-officer, released 01 July 2001 A woman has been arrested after the emergence of a social media video apparently showing a prison officer having *** with an inmate in a cell. The HMP Wandsworth prison officer *** unfolds with a leaked video from inside Wandsworth prison, where a female prison officer is seen in the footage. This prison video, shared on Twitter, captures the Wandsworth prison guard engaging in actions that have ignited a viral response. June

🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)
🔴✅👉Watch 🟢 ➤ ➤ ➤ ✅👉 Click Here To link (Full Viral Video Link)
✅🔴 ➤►WATCH or DOWNLOAD🔴👉👉(Full Viral Video Link)


bug validate command fails on server

chris@clegoues-ubuntu-vm1:~/bugzoo$ bugzoo bug validate manybugs:lighttpd:1913-1914
validating bug: manybugs:lighttpd:1913-1914
Compiling...                                                                [OK]
Running test: p10...                                          [UNEXPECTED: FAIL]

    OCI runtime exec failed: exec failed: cannot exec a container that has stopped: unknown

Traceback (most recent call last):
  File "/home/chris/.local/lib/python3.5/site-packages/docker/api/client.py", line 223, in _raise_for_status
    response.raise_for_status()
  File "/home/chris/.local/lib/python3.5/site-packages/requests/models.py", line 935, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 409 Client Error: Conflict for url: http+docker://localunixsocket/v1.35/containers/8ee2552e8bb5417c1ea5b33ecf39d846b609675fe334071367225f8cbf744288/exec

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/chris/.local/bin/bugzoo", line 9, in <module>
    load_entry_point('bugzoo==0.0.1', 'console_scripts', 'bugzoo')()
  File "/home/chris/.local/lib/python3.5/site-packages/bugzoo/cli.py", line 441, in main
    args.func(args)
  File "/home/chris/.local/lib/python3.5/site-packages/bugzoo/cli.py", line 388, in <lambda>
    cmd.set_defaults(func=lambda args: validate_bug(rbox, args.bug, args.verbose))
  File "/home/chris/.local/lib/python3.5/site-packages/bugzoo/cli.py", line 83, in validate_bug
    if bug.validate(verbose=verbose):
  File "/home/chris/.local/lib/python3.5/site-packages/bugzoo/bug.py", line 276, in validate
    outcome = c.execute(t)
  File "/home/chris/.local/lib/python3.5/site-packages/bugzoo/container.py", line 306, in execute
    response = self.command(cmd, ctx, stderr=True)
  File "/home/chris/.local/lib/python3.5/site-packages/bugzoo/container.py", line 234, in command
    response = client.api.exec_create(self.__container.id, cmd, stdout=stdout, stderr=stderr)
  File "/home/chris/.local/lib/python3.5/site-packages/docker/utils/decorators.py", line 19, in wrapped
    return f(self, resource_id, *args, **kwargs)
  File "/home/chris/.local/lib/python3.5/site-packages/docker/api/exec_api.py", line 80, in exec_create
    return self._result(res, True)
  File "/home/chris/.local/lib/python3.5/site-packages/docker/api/client.py", line 229, in _result
    self._raise_for_status(response)
  File "/home/chris/.local/lib/python3.5/site-packages/docker/api/client.py", line 225, in _raise_for_status
    raise create_api_error_from_http_exception(e)
  File "/home/chris/.local/lib/python3.5/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception
    raise cls(e, response=response, explanation=explanation)
docker.errors.APIError: 409 Client Error: Conflict ("Container 8ee2552e8bb5417c1ea5b33ecf39d846b609675fe334071367225f8cbf744288 is not running")

Caused by using an outdated version of BugZoo.


fix mysql connection

Codecov Report

Merging #2 (9ea5054) into main (4e0b79d) will decrease coverage by 46.83%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main       #2       +/-   ##
===========================================
- Coverage   77.27%   30.43%   -46.84%
===========================================
  Files           3        3
  Lines          22       23        +1
===========================================
- Hits           17        7       -10
- Misses          5       16       +11

Flag
Coverage Δ

unittests
30.43% <0.00%> (-46.84%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

database/mysql/mysql.go
0.00% <0.00%> (-66.67%)
: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 4e0b79d...9ea5054. Read the comment docs.


.props() don't receive data in when child component is extended and stubbed

Version
1.0.0-beta.19
Reproduction link
https://github.com/Loremaster/vue-test-utils-issues
Steps to reproduce

Download source code of the example
Install packages $ yarn
Run tests $ yarn test:unit

What is expected?
All tests should pass
What is actually happening?
Test is failing and child component doesn't receive prop (as it does when it's a non typescript component):
  ● List.vue › ListItem receives text as a prop

    expect(received).toEqual(expected)

    Expected value to equal:
      "Learn Vue"
    Received:
      undefined

    Difference:

      Comparing two different types of values. Expected string but received undefined.

      15 |     const wrapper = shallowMount(List);
      16 |     const listItem = wrapper.find(ListItem);
    > 17 |     expect(listItem.props().text).toEqual("Learn Vue");
         |                                   ^
      18 |   });
      19 | });

Thank you @eddyerburgh :) Can't wait for new release!

@eddyerburgh is it possible to create new release soon? We really need this fix in our project.


Corrupted XLSX output due to invalid sheet name

So I noticed that while some softwares could open generated XLSX files using exceljs (like Numbers on MacOS and Google Sheets), Microsoft Excel (or Excel Online) was unable to open the files, giving me an error indicating that the file is corrupted. After simplifying the code and isolating the issue, I realized that for some reason MS Excel assumes a maximum length of 31 characters for Worksheet titles and as a result was deeming the generated file corrupt.
So basically, this snippet works fine:
const excel = require('exceljs');

let book = new excel.Workbook();
let sheet = book.addWorksheet('random sheet name that is valid');

sheet.columns = [
  {header: 'First Name', key: 'firstName'},
  {header: 'Last Name', key: 'lastName'}
];

sheet.addRow({ firstName: 'John', lastName: 'Wick' });
sheet.addRow({ firstName: 'Elliot', lastName: 'Alderson'});

book.xlsx.writeFile('test.xlsx');

While this one doesn't:
const excel = require('exceljs');

let book = new excel.Workbook();
let sheet = book.addWorksheet('random sheet name that is not valid');

sheet.columns = [
  {header: 'First Name', key: 'firstName'},
  {header: 'Last Name', key: 'lastName'}
];

sheet.addRow({ firstName: 'John', lastName: 'Wick' });
sheet.addRow({ firstName: 'Elliot', lastName: 'Alderson'});

book.xlsx.writeFile('test.xlsx');

Since MS Excel is what most end-users will use to open automatically generated files by a web-service, I would highly recommend adding an error/warning message for when this case occurs, a toggle-able validation that is on by default, or at least a hint to the documentation.

@Siemienik Any ideas how to validate correctness of excel? maybe I will try to create pull request.

@omurbekjk somethink like that works for me:
       name = name.replace(/[\\\/*\[\]?]/g, '.');  //replace forbidden chars
        if (!scope.isUnique(name))) { //name has to be unique
            name = scope.next(name);
        }
        name = name.length > 31 ? name.substr(name.length - 31) : name; //max 31 letters


⚠ Bofrost.fr has degraded performance

In b4c45fd, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1247 ms

Resolved: Bofrost.fr performance has improved in d2eaa88 after 6 minutes.


[FEATURE REQ] Update enum type PublicAccessType

Is your feature request related to a problem? Please describe.
A cx reported issue.
When "Allow Blob public access" of storage account is set to "Disabled",
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/storage/azure-storage-blob/src/main/java/com/azure/storage/blob/BlobContainerClient.java
The API call to set access policy will fail due to an error "Public Access is not permitted on this storage account".
    BlobContainerClient blobContainerClient = storageClient.getBlobContainerClient(containerName);
    blobContainerClient.setAccessPolicy(PublicAccessType.CONTAINER, Collections.singletonList(identifier));

The workaround is to set the first parameter as null:
blobContainerClient.setAccessPolicy(null, Collections.singletonList(identifier));
Describe the solution you'd like
Can we update the enum type PublicAccessType by adding a value "None"
Or can we have a overload method which only accepts one parameter

@yangchengs Thanks for the good feedback. Unfortunately we don't think this can be integrated in a way that avoids both breaking changes and introducing more ambiguity to the API. We'll track this work for a potential future breaking release.