SparkML sample

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[ ] ~Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.~
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[x] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

Meta comment - before you do final submission - look for all instances of "citi" throughout the notebook and decide on a consistent spelling for Citi Bike vs citibike or whatever (I think if we're being proper it's "Citi Bike" though we all colloquially say "citibike") - I just want it to be spelled the same way every time 😁


Migrate reconcilers to use eventing.* v1

Proposed Changes

Migrate reconcilers (Broker and Trigger) to use eventing.* v1

/lgtm
/approve


Install and Set Up kubectl on Windows

Hello, on the current date (13 Dec 2023) there is a bug on the page https://kubernetes.io/docs/tasks/tools/install-kubectl-windows/#install-kubectl-binary-with-curl-on-windows
The bug is for example here: Download the latest 1.29 patch release: kubectl %!f(string=1.).0.
%!f(string=1.).0 is in all links - so it's not possible to download kubectl.exe. Please fix.
Thank you.

Page reported in issue: https://kubernetes.io/docs/tasks/tools/install-kubectl-windows/#install-kubectl-binary-with-curl-on-windows
/language en
/sig docs

It appears that there may be an issue with the shortcode responsible for generating the version on our page. While this issue has been reported for a single page, it is a site-wide issue affecting multiple pages that use the shortcode.
/kind bug
/triage accepted

I'll try to investigate this issue and see if I can offer assistance.
/assign

/retitle Broken info in “Install and Set Up kubectl on Windows”


Adjust weights at the end.

Solution to #72
Note: Never tried it, maybe somebody can try and tell if it is working better(because I’m not at home).

Just tried it, looks better.

Running examples:
XOR: http://z3r0.site/Toy-Neural-Network-JS/examples/xor/
MNIST: http://z3r0.site/Toy-Neural-Network-JS/examples/mnist/
You may need to increase the learning rate.


Back to top button is not working in Edit page

Bug Report

PHP Version: 7.0.18
MySQL Version: 15.1

Expected Behavior
Scroll to top
Actual Behavior
Not doing anything

Hi @OHopiak, Thanks for reporting this. To be honest, The back to top button should not even display on the edit page as you shouldn't be able to scroll the main content far enough for it to show. Could you let me know what browser and OS you're using?

I can confirm that the Back to Top Pop-Up does come up in the edit window, and does not function.  When you copy in paste in a large amount of text you get to the bottom very quickly.
Bookstack Version 0.16
Firefox 54.0.1 (64-Bit)

Can anyone provide a screenshot? Do you have to hover over the correct area to show or does it show after scrolling or does it show all the time?

As of next release, THe back-to-top button will be fully hidden on pages such as the edit page.
Change in 93147f434065e474a27b31a396caa332de8ee45d


Figure.Tile() throws TypeError: unsupported operand type(s) for /: 'Unit' and 'int'

See Stack Overflow post by user Allan for a complete description: https://stackoverflow.com/questions/45850144/is-there-a-bug-in-svgutils-compose-module-regarding-the-arrangement-of-figures/45863869#45863869
Here is a one-liner that reproduces the problem:
from svgutils import *
Figure( "21cm", "29.7cm",
    Text("A"),
    Text("B"),
).tile(1,2)

I propose the following fix in function svgutils.compose.Figure.tile():
dx = self.width.to('px').value/ncols
dy = self.height.to('px').value/nrows

The problem was specific to Python 3. Unit class already implemented a division operator (div), but it was not triggered in Python 3, which uses a different special function (truediv). This should be now solved in master. Thanks for reporting.


media=print for pdf creation

page.emulateMedia('print') for PDF creation to make sure all assets that are only visible in print-css are loaded
See https://github.com/puppeteer/puppeteer/issues/436#issuecomment-564008025 for details

@chaelli emulateMedia options is great, but modified Dockerfile is not match with this topic.

@zenato sorry, should not have used master for the pull request (I pushed the other changes later) - created a clean PR: https://github.com/zenato/puppeteer-renderer/pull/21


ElapsedTime when elapsed time values greater than the largest time value that can be represented in the Elapsed Time option.

Hello,
When elapsed time values greater than the largest time value that can be represented in the Elapsed Time option, the client uses the value 0xffff.
Plz this commit
https://github.com/jerome-laforge/dhcp6/commit/924665b1b681f36187f78da14256467fc0fd8fa2
If it is suitable for you, I can submit a PR

Looks good! Please do submit a PR.


Added a script to find/add missing Gateways in the vendor/omnipay/common/composer.json file

(not sure if this should be sent to omnipay-common or omnipay-example, so sending to both)
It was written while playing with omnipay-example and expects to be in some sort of dir/omnipay/common type directory with other omnipay gateways
installed in dir// for example   dir/omnipay/paypal.
It parses the various vendor///src/*Gateway.php files looking at the namespace and class name to determin the proper gateway name and searchs
if its already added to the common/composer.json file
Most people should be fine just running the script without worrying about changing anything
Calling it by itself will display a list of gateways missing (if any)

example #1:   ./updategateways.sh
Calling it by first setting the environment variable FIX=1 will insert the missing gateways into the common/composer.json file
example #2:   export FIX=1; ./updategateways.sh
example #3:   FIX=1 ./updategateways.sh

Runing the script found these to add:

AuthorizeNet_DPM
Buckaroo_CreditCard
Eway_Direct
Eway_RapidDirect
Eway_RapidShared
PayPal_Rest

Closed as duplicate of https://github.com/thephpleague/omnipay-common/pull/49


Use rackup as default command

Using rackup as default command would cover more ruby-based web frameworks. The change should not break compatibility with existing setups.

@patricklee2 any chance this change could get merged? might be added as a default for new ruby versions https://github.com/Azure-App-Service/ruby/pull/42 🤔

rackup was extracted to a separate gem https://github.com/rack/rack/blob/3d0b9f377a4753d1fca40fca011c2c0e9282ab50/UPGRADE-GUIDE.md#binrackup-rackserver-rackhandlerand--racklobster-were-moved-to-a-separate-gem


ODC-7484,ODC-7485,ODC-7486: Added Task, TaskRun and ClusterTask list pages to dynamic plugin

This PR is considering support of Pipelines dynamic plugin from Console 4.15 onwards
Story:
https://issues.redhat.com/browse/ODC-7484
https://issues.redhat.com/browse/ODC-7485
https://issues.redhat.com/browse/ODC-7486
Description:
Added Task, TaskRun and ClusterTask list pages to dynamic plugin

/cc @jerolimov


Missing font oversampling support for custom scaling implementations.

Godot version
3.3 stable and 3.x
System information
Windows 10
Issue description
Issue:
Missing font oversampling support for custom scaling implementations.
What I expect:
A way to set the font oversampling value in STRETCH_MODE_DISABLED, as this mode objective is custom scaling implementations.
More details about the issue:
I required a pixel perfect solution for my game, where I can only scale the game in perfect multiples of the base size (1x,2x,3x,4x, etc) and celeste style HD font rendering (font oversampling).
Reading the documentation, there is a recommendation for the integer resolution handler plugin:

Godot currently doesn't have a way to enforce integer scaling when using the 2d or viewport stretch mode, which means pixel art may look bad if the final window size is not a multiple of the base window size. To fix this, use an add-on such as the Integer Resolution Handler.

That plugin replicates the godot internal scaling found in gdscript scene_tree with the exception of the "_update_font_oversampling" line because the method isn't available in GDScript. This plugin uses "STRETCH_MODE_DISABLED" + "STRETCH_ASPECT_IGNORE" flags scaling the base resolution in floored screen values (1x,2x,3x, etc).
Because of this engine limitation, that plugin misses the font oversampling support, only available for "STRETCH_MODE_2D" using an internal scaling formula not compatible with all custom implementations.
I did a pull request fixing this issue (https://github.com/godotengine/godot/pull/52881)
With this commit the user can call in that plugin:
get_tree().update_font_oversampling(scale)
And have full font oversampling support in any stretch mode or any custom scaling implementation.
As note I did the pull request at 3.x branch, but I need it for 3.3.3.stable as it is the branch We use in our project and I don't know what I need to do for this to be cherry picked for the future 3.3.4.stable, both branches are affected by this issue.
Steps to reproduce

Create a project with some labels and a pixel art background.
Download and install this plugin: https://github.com/Yukitty/godot-addon-integer_resolution_handler
Enable font oversampling.
Set the project size to the same of the pixel art background.
The plugin ensures no distortion in the pixel art background, but the labels ignores the oversampling flag.

Minimal reproduction project
No response

As note I did the pull request at 3.x branch, but I need it for 3.3.3.stable as it is the branch We use in our project and I don't know what I need to do for this to be cherry picked for the future 3.3.4.stable, both branches are affected by this issue.

To ensure stability, new features are only added to the 3.x branch;


When printing with Chrome the signature disappears.

You can see the signature on the page when loaded on Chrome. But when you go to print the same page in the Print preview window is missing the signature, and print to PDF or printing to paper does not yield a signature.

I have also files a chromium issue here: https://code.google.com/p/chromium/issues/detail?id=500022

Seems like a bug that won’t print the <canvas>.


"to be ran" --> "to be run"

Small grammar correction.

@rcabr : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rcabr  - thanks for the catch!


Documentation on port mapping under boot2docker needs extra step

The documentation is pretty clear about how to get the instance up and running, but if you're using boot2docker on OS X, then you need to execute:
boot2docker ip

in order to get the host machine IP address of boot2docker, then, you would use that IP address in the browser instead of <IP_ADDRESS> or localhost to access the UI.

I think that's more a Docker general question than any image

closing as this is a docker question, not the image


Brand/messaging roll out

User story
As a user, I want to clearly understand ReportStream and how it meets my needs. As an internal team member, I want to know how I should communicate about ReportStream.
Background & context
After finalizing the messaging guidelines, we need to socialize the guide throughout the ReportStream team for alignment and application. The training and alignment meeting is the primary launch of this and that is tentatively slotted for All Team Sync 6/28.
Open questions

Confirmation that 6/28 will work for the meeting.

Working links
Messaging deck
Brand roll out plans
##Acceptance criteria

[x] Agenda and materials created
[x] Meeting hosted

Training deck


How can the five mentioned test sets in the article be obtained?

Hello! As it is mentioned in paper(https://www.sciencedirect.com/science/article/pii/S2666675823001157) that "To assess the performance of MutFormer against existing methods of deleteriousness prediction, a total of five [testing datasets] were used." They were showed in "Table 2. Details for each testing dataset". However, it appears that I couldn't find these five test sets in the supplementary materials of the article or on GitHub. I would be very grateful if you could provide them to me!

Hi! You are correct that the testing datasets mentioned in the paper are not publicly available. This is because I think some of the datasets may require licenses and prohibit redistribution (though I am not positive about this). I can however point you to exactly where to download each dataset (note that during MutFormer's experiments, we filtered all testing datasets by removing data points present in any pretraining or finetuning data; if you are interested in the filtered dataset versions, I can take a closer look at the licensing situation again and if possible provide you with these sets):

Datasets 1, 2, and 3 correspond to testing datasets I, II, and III in supplementary table S8 from this paper
Datasets 4 and 5 are both from supplementary table S2 from this paper (all lines in the table with first column = PPARG were used for dataset 4 and all lines in the table with first column = TP3 were used for dataset 5).

Hope this helps!
-Theo

Thank you for your patient reply! I am indeed very interested in your filtered dataset versions. I would be extremely grateful if it were convenient for you to provide me with these filtered datasets!

Hello! I sincerely apologize for reaching out again. I would like to inquire if you could provide me with the filtered dataset regarding the above-mentioned issue. This is crucial for our research, and I would greatly appreciate it if you could respond at your earliest convenience. Thank you very much.

Hi! So sorry for the late reply; it does appear that since all the data sources are open-source, it should be okay. Here are the filtered datasets (please let me know if you have any more questions!): test_data.zip

Hi! I sincerely appreciate your patience and professionalism. The data you provided will be immensely helpful to us! I have downloaded and reviewed the test dataset, and I noticed that there are 5 columns. According to my understanding, these columns should be labeled as follows: label, reference sequence, mutant sequence, external data, and mutation position. This aligns perfectly with the format required for MutFormer input files.
However, I am eager to know whether the mutations include an identifier similar to 'Q96NU1_P10S,' encompassing the UniProt canonical accession, reference residue, mutation position, and mutation residue information. This unique ID would be instrumental for our cross-referencing efforts. Your assistance in providing this information would be greatly appreciated!
Thank you once again for your invaluable support.

Furthermore, I checked the variant counts, and they seem not to correspond entirely between the description of test sets in paper and the downloaded TSV files. Only the variant counts in test_set1.tsv and test_set3.tsv match up. Additionally, is there any relationship between the numbers of variants in the validation set and the test set?
My data summary table is as follows:

filename
dataset_name
independent   validation set in paper
testing   sets in paper
find   in download TSV file

negative example
positive examples
benign examples
pathogenic examples
label 0

test_set1.tsv
Meta_SVM_LR_set_1
56
35
35

test_set2.tsv
Meta_SVM_LR_set_2
5,866
4,115
5,884

test_set3.tsv
Meta_SVM_LR_set_3
2,422
0
2,422

test_set4.tsv
Varibench_PPARG
4,671
3,428
2,207

test_set5.tsv
Varibench_TP53
3,444
4,505
531

Furthermore, I checked the variant counts, and they seem not to correspond entirely between the description of test sets in paper and the downloaded TSV files. Only the variant counts in test_set1.tsv and test_set3.tsv match up. Additionally, is there any relationship between the numbers of variants in the validation set and the test set?
My data summary table is as follows:

However, I am eager to know whether the mutations include an identifier similar to 'Q96NU1_P10S,' encompassing the UniProt canonical accession, reference residue, mutation position, and mutation residue information. This unique ID would be instrumental for our cross-referencing efforts. Your assistance in providing this information would be greatly appreciated! Thank you once again for your invaluable support.

Hi! That is a really good question; an accession would indeed be really helpful, though I think the source of these datasets also unfortunately did not provide any accession IDs. I really apologize for the inconvenience on this end. I believe it definitely should be possible to backtrack using the reference sequence, though that definitely is by no means simple. I can spend some time looking into this and will let you know if I do end up finding some accession ID information.
-Theo

Furthermore, I checked the variant counts, and they seem not to correspond entirely between the description of test sets in paper and the downloaded TSV files. Only the variant counts in test_set1.tsv and test_set3.tsv match up. Additionally, is there any relationship between the numbers of variants in the validation set and the test set?

That is a great catch! Upon looking into this more, I believe the discrepancy between the numbers in the two sets is actually a versioning mistake; it seems to me that the numbers in the paper don't reflect the most updated values. As for the actual test set, this should be the most updated version and should be the data we used to obtain the results in the ROC/PRG curves, so you should be able to go ahead and use it; the discrepancy is still due to the versioning error in the numbers. So sorry for the confusion and please let me know if you have any more questions!
-Theo

However, I am eager to know whether the mutations include an identifier similar to 'Q96NU1_P10S,' encompassing the UniProt canonical accession, reference residue, mutation position, and mutation residue information. This unique ID would be instrumental for our cross-referencing efforts. Your assistance in providing this information would be greatly appreciated! Thank you once again for your invaluable support.

Hi! That is a really good question; an accession would indeed be really helpful, though I think the source of these datasets also unfortunately did not provide any accession IDs. I really apologize for the inconvenience on this end. I believe it definitely should be possible to backtrack using the reference sequence, though that definitely is by no means simple. I can spend some time looking into this and will let you know if I do end up finding some accession ID information.
-Theo

Thank you for your reply! I have considered using BLAST to obtain the corresponding accession for the reference sequences. However, my concern is that, due to the sequence length limitation of 1024, the reference sequences provided in the file should ideally not exceed 512 in length. This is also a challenge, as per my understanding.

Furthermore, I checked the variant counts, and they seem not to correspond entirely between the description of test sets in paper and the downloaded TSV files. Only the variant counts in test_set1.tsv and test_set3.tsv match up. Additionally, is there any relationship between the numbers of variants in the validation set and the test set?

That is a great catch! Upon looking into this more, I believe the discrepancy between the numbers in the two sets is actually a versioning mistake; it seems to me that the numbers in the paper don't reflect the most updated values. As for the actual test set, this should be the most updated version and should be the data we used to obtain the results in the ROC/PRG curves, so you should be able to go ahead and use it; the discrepancy is still due to the versioning error in the numbers. So sorry for the confusion and please let me know if you have any more questions!
-Theo

Thank you once again for your patient guidance and explanations. In terms of the quantity, I currently have no further questions.

HI! I've been trying to access the file named "http://www.openbioinformatics.org/mutformer/hg19_MutFormer.zip" and the 'basic_example.zip
i tried to open this link on different pc's but its not working on either of them.can you help me with that?

Hello,
I am able to use my browser or use the command wget http://www.openbioinformatics.org/mutformer/hg19_MutFormer.zip  to download the file.

yes. i still cant access the files

can it be because of the region??
can you provide me with the files here or perhaps email me??

The file is too large and cannot be attached here.
Please let me know if you can download from here:
http://<IP_ADDRESS>/0c10d412d553db42102cec1fb51e668a/hg19_MutFormer.zip

this link isn't loading either!

The server (http://<IP_ADDRESS>/ ) has been working for years and being accessed by people from all over the world.
Please try to ping <IP_ADDRESS> and see what you get

yes, PPAR gene database is there.

does your browser/institute block HTTP downloading because it is not HTTPS?

no, there's nothing like that.

try to open this link  http://<IP_ADDRESS>/0c10d412d553db42102cec1fb51e668a

and then click the file to download

yes, it's working now. i need the link for test zip too please.

It is uploaded to the same place. please open http://<IP_ADDRESS>/0c10d412d553db42102cec1fb51e668a/  and download

Thank you so much sir for your help.i really appreciate it :)

i need the file for analysis because it appears that the data i'm using is from hg38.
can you please provide the link?

Hello,
Sorry for the late reply, these files have been uploaded to the same place. Please open http://<IP_ADDRESS>/0c10d412d553db42102cec1fb51e668a/ and download.


Add note for publishing UI themes (mine got published in "packages")

Hey,
yesterday I published my first UI theme and followed all the steps in Creating an Interface Theme. That means I forked atom-dark-ui and when I tried publishing my theme finally it was listed as package because some weird things happened during older commits of the forked repo.
I would like to help others and add a note that if they have the same problems and their theme appears as package they should create a new repo and everything will work fine then.
I hope this is the right place for telling you guys that or is there a way to contribute to the documentation directly? I found something but had no idea if it would have worked.

Please, please, please... Fix this one.
Had the problem with my first theme.
With my first ui theme right after that.
Look at the mess created in "packages" :   https://atom.io/users/melinadonati
And I can't even unpublish any of those "ghost" packages...
It's ugly and furthermore, even renaming the theme + theme repo + theme name in files didn't work after 3 tries (precision: working directory fully clean, an untouched clone of distant repo).
Help accepted on the unpublish problem in the waiting time...

@melinadonati I deleted the two packages that are not backed by repositories. You should be able to unpublish the other two with the apm unpublish command since the repositories for them still exist. If not, let me know.

Thanks a lot lee-dohm !
I will try again to unpublish the other two, and will hopefully be successful this time.

The apm unpublish is getting me the following error :
Error: EINVAL, invalid argument     at new Socket (net.js:157:18)     at process.stdin (node.js:693:19)     at Unpublish.module.exports.Unpublish.promptForConfirmation (C:\Users\Melina\AppData\Local\atom\app-1.7.3\resources\app\apm\lib\unpublish.js:87:48)     at Unpublish.module.exports.Unpublish.run (C:\Users\Melina\AppData\Local\atom\app-1.7.3\resources\app\apm\lib\unpublish.js:126:21)     at Object.module.exports.run (C:\Users\Melina\AppData\Local\atom\app-1.7.3\resources\app\apm\lib\apm-cli.js:226:32)     at Object.<anonymous> (C:\Users\Melina\AppData\Local\atom\app-1.7.3\resources\app\apm\lib\cli.js:8:7)     at Object.<anonymous> (C:\Users\Melina\AppData\Local\atom\app-1.7.3\resources\app\apm\lib\cli.js:19:4)     at Module._compile (module.js:456:26)     at Object.Module._extensions..js (module.js:474:10)     at Module.load (module.js:356:32)


Added timestamp to log messages

Small PR to add timestamps to log messages.

@jworcestBAH done reviewing. From the perspective of someone who manually opens and reads local the error log, this looks fine. A question for the folks using LogStash....I'm not sure if there's some kind of front-end or utility that parses the log messages. If so, would we want the timestamp to be in the JSON, along with error_log_message, error_log_type, etc?

It's easy enough to have it be in the json as "error_timestamp", and that does seem better either way, so I'll go ahead and change that.

Okay pushed that change

@jworcestBAH thanks, looks good!


Warning: You have opted-out of Automatic Static Optimization due to getInitialProps in pages/_app. This does not opt-out pages with getStaticProps

b/c getInitialProps in _app.tsx

SSGしたい

closed by 4ccf963878a250812043082a477d9c23687b0870


How does the explorer determine which properties to show?

I've got datacontext objects where the explorer always leaves out several properties - usually the one(s) I'm interested in - instead showing a few that have no apparent difference from the ones it did not show. Could the documentation explain what critieria it uses to not show certain properties?

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

ID: fd2baf39-d3c3-5d5a-d93c-f4040430c3dd
Version Independent ID: c443e681-03f0-ea47-a09d-718693448b3d
Content: Inspect XAML properties while debugging - Visual Studio
Content Source: docs/debugger/inspect-xaml-properties-while-debugging.md
Service: unspecified
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@rkpatrick Thanks for the feedback. Can you take a look at this article?
https://visualstudiomagazine.com/articles/2015/08/01/debugging-xaml.aspx
It discusses limits on what is shown in the visual tree, and talks about how to view the default values that aren't specified in XAML.

@rkpatrick I took another look at this issue. If you click "Show properties", do the properties you are interested in show up in the Live Property Explorer? (It looks like properties that aren't directly specified in XAML can be found in the Live Property Explorer.) Also, if the DataContext object contains a large number of elements, maybe they are only added to the object as required, similar to a virtualized list for a ListView object? (as noted in the article)


install: Disable kube-proxy-replacement by default

Related: https://github.com/cilium/cilium/pull/15422
Signed-off-by: Thomas Graf<EMAIL_ADDRESS>

I rebased the PR as it was quite old and closed/reopen to trigger the tests (I think something is broken in the workflow triggers, I'll have a look). Let's see if the e2e tests still pass.

Tests are currently failing because KPR is disabled but the bandwidth manager is enabled. Thus, devices are autodetected and we end up in https://github.com/cilium/cilium/issues/12205's case for the native routing setups. I'll disable the bandwidth manager.

The only test failing, External Workloads, is failing with a different error than on master. That's because this PR is editing external-workloads-install.sh but the workflow runs the code from master. I've opened https://github.com/cilium/cilium-cli/pull/291 to run the PR version of the workflow and show that this workflow fails with the same error as on master. That error is tracked at https://github.com/cilium/cilium-cli/issues/224.
Only waiting for a review from @nebril now :rocket:


[badgerdb] vlog growing unbounded - consider adding GC and exposing options

I noticed that BadgerDB storage started growing unbounded (it was many files and several gigabytes quite quickly). Looks like the Badgerhold/Badger Storage example doesn't provide a way for the user to utilize GC correctly per Badgers docs. https://github.com/dgraph-io/badger/blob/main/db.go#L1225
Fairly sure the option tuning is probably useful to expose as well somehow -- for example ValueLogFileSize has a pretty large default
I didn't plumb up a full fix, just copied the default and started experimenting, but this showed that something like option tuning and GC can resolve the infinite growth modified a copy of the default to include some signalling and a regular GC call.
I would recommend at least adding the GC as a default. I would also suggest lowering some of the default or exposing them.
        go func() {
                ticker := time.NewTicker(5 * time.Minute)
                defer ticker.Stop()
                for {
                        select {
                                case <- ticker.C:
                                        _ = h.db.Badger().RunValueLogGC(0.7)
                                        continue
                                case <- h.doneCh:
                                        return
                        }
                }
        }()

I've checked the official documentation of Badger regarding garbage collection. @woodzy , you're right. Since neither of us has experience using BadgerDB in a production environment, if you could, it would be great if you could submit a PR. This would be very helpful for most users who use BadgerDB as a persistent database.

@woodzy Can this #371 fix the issue?

Thanks! These look like better implementations of my local workarounds during experimentation.

This should be fixed in v2.6.0 - please let us know!

It not fix, still growing on mem

@woodzy @OpenJarvisAI Could this issue be closed?

The issue has been resolved, and due to the lack of feedback over an extended period, we are closing this issue. If there are any further problems, please reopen it.


Js and css paths and maybe add pipes?

Hi,
love your lib! But there's a couple of things i think, would be nice:

Being able to set different output paths for js and css
Being able to add a gulp pipe like sass so i could reference .scss files

I'm sorry if this is possible and I just could'nt find out how :p

Thanks for your ideas. I'll add support for different output paths, hopefully today or tomorrow.

Being able to add a gulp pipe like sass so i could reference .scss files

You might be better off just using this package to get the file list with a custom gulp task:
var Bundle = require('laravel-elixir-bowerbundle/src/Bundle'); // grab the Bundle class
var customBundle = new Bundle(['jquery']); // instantiate with the packages you want
customBundle.files(); // returns [ 'jquery/dist/jquery.js' ]

The gulpfile.js might look something like:
// Use elixir as normal
var elixir = require('laravel-elixir');
require('laravel-elixir-bowerbundle');

elixir(function (mix) {
    mix.task('myBower');
});

// Define your custom task
var gulp   = require('gulp');
var filter = require('gulp-filter');
var sass   = require('gulp-sass');
var Bundle = require('laravel-elixir-bowerbundle/src/Bundle');

gulp.task('myBower', function () {
    return gulp.src(new Bundle(['jquery']).files())
        .pipe(filter('*.scss'))
        .pipe(sass())
        // ...
});

It's working!
But a lot of packages use .scss files so 'out-of-the-box' support would be nice, or at least clean up the gulpfile a lot ;)
Thanks for the help!

Thanks for confirming that works - I'll improve the API to expose the Bundle class directly, so you won't need to reference a specific file (i.e. the require('laravel-elixir-bowerbundle/src/Bundle') bit). And I'll add an example to the readme.

But a lot of packages use .scss files ...

If it's a common use case then I can certainly add support for sass, but it seems a bit strange. Are there really many Bower packages that only include source sass files and not the compiled css?
Otherwise, if the package includes both formats, surely the only benefit of consuming the sass over the css is if you want to change some variables used by the sass... and if you do that, you'll need a custom build task anyway, no?
Being able to tap into the gulp pipeline in a generic way, as you originally suggested, could be nice but probably beyond the scope of a small plugin like this. (But if a feature like that gets added to the core Elixir repo then I'd make sure this plugin supports it!)

You're probably right about the sass integration, i've checked and only come across it once and got it working using a sepperate gulp task compiling that file.
Thanks for the help!

No problem!
v2.1.0 adds support for multiple output directories - the output path can now be given as an object like { css: 'public/css', js: 'public/js', misc: 'public/assets' } instead of a single string.
The readme has also been updated, including a section of using this package with a custom build task.

Sorry  to bother you again :p
Another thing that could be usefull (so the compiled css could be used with the @import in other scss files) is to be able to override the file extension to .scss.
Now you have to use mix.copy(), it wouldn't do the file any harm ;)

It sounds like it's already possible to import plain css from a scss file: https://github.com/sass/libsass/pull/754
(Alternatively, your sass could always import the relevant source sass from bower_components and bypass this plugin completely.)


Needs a license

What license is this code released with?

Ah, right - thanks for the heads up.  I will add one asap.

Thanks! I'd love to be able to use this at work. I'd have submitted a pull request but I don't know what license you'd want :-)

Sorry about that - running a little behind. I just added a license. I'll push a new gem tomorrow morning.

No problem. Thanks for making such a fun gem!

Just release 0.0.4 gem.  Give it a shot and let me know if you run into any problems.


Update free-programming-books-el.md

I added some books about JAVA and SQL that I found very informing. Also I changed "Index" to "Περιεχόμενα" (Table of Contents in greek)
What does this PR do?
Add resource(s)

Sorry for the failed checks I thought I added pdf indications to everything but apparently I did not. Everything is ok now.

Thanks, and welcome to free-programming-books!


docker.github.io/machine/reference/scp.md is very sparse

Problem description
This page could use a few more examples. My specific issue is here:
http://stackoverflow.com/questions/41651212/how-to-move-a-local-volume-onto-a-remote-docker-machine
And the docs don't answer my question, which I feel like they should, since it's a narrow question about docker-machine scp and a common use case I suspect.
Problem location

I couldn't find the information I wanted. I expected to find it near the following URL: https://docs.docker.com/machine/reference/scp/

Suggestions for a fix
I think a couple more examples could go a long way to improving this page. I'm particularly interested in my use case but I imagine there are other common ones as well.

@fresnizky thanks for your feedback on this.
I just added some detail to docker-machine scp reference around your example. I know it's an issue specific to your use case, but I think the understanding the idea of copying to remote systems for deployment is useful in a more general sense. I contextualized your use case as a sort of second example, beyond the use of scp and more about file paths on remote systems. We are somewhat de-emphasizing docker-machine going forward but if you have ideas for more pointers that can help those docs, please do send, and we'll have a look at them.

@londoncalling no worries I solved these issues long long ago. =) Thanks for patching up the docs though.


doc need: creating a new reference project and deploying it locally

We're missing a doc in the repo (ideally at references/README.md) that details how to:

Create a new reference project by copying the hello-world reference project
Get the dev CLI command running against a local instance of Trigger.dev (e.g. the webapp)
Get the deploy CLI command running against a local instance of Trigger.dev by setting up and running the webapp alongside the coordinator and docker-provider:

pnpm run dev --filter webapp --filter coordinator --filter docker-provider

Then deploying it like so:
pnpm exec trigger deploy --self-hosted --load-image

This README should also make sure the CONTRIBUTING.md guidelines are enough to get the webapp running locally.
This new README should be sufficient for a new contributor to come in from scratch and create a new reference project and both run it in dev and deploy it.

/bounty $100

@ericallam
Wouldn't it be better to use the CLI's init command instead of copying over the hello-world project? as it creates the same files

@varshith257 I can't remember how to assign things with Algora or if that's even possible, but here is a gif instead:

@ericallam I would like to work on it, Can I get it assigned?

I've been informed that GitHub has issue assignment built in, and gifs don't count. So I've assigned @varshith257 (and myself), to this issue. @Kritik-J thanks for the offer, but since @varshith257 came first I'll give them a crack at it 🫡

@varshith257 any updates?

@ericallam Will draft PR soon

/attempt #1362

/attempt #1362

He is taking too much time

@Wackyator how are things going with this issue? Anything I can help with?

@ericallam I've mentioned something in the discord thread that I require some help on, I've created a draft PR for the time being.


Fix monitoring test

CHANGE DESCRIPTION
Problem:
Short explanation of the problem.
Cause:
Short explanation of the root cause of the issue if applicable.
Solution:
Short explanation of the solution we are providing with this PR.
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PG version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

scheduled-backup
passed

init-deploy
passed

demand-backup
passed

upgrade
passed

version-service
passed

scaling
passed

smart-update
passed

recreate
passed

affinity
failed

monitoring
passed

self-healing
passed

operator-self-healing
passed

clone-cluster
passed

tls-check
passed

users
passed

ns-mode
passed

We run 16 out of 16

commit: https://github.com/percona/percona-postgresql-operator/pull/579/commits/d8fb60432b81fe14216c2e7aea7012a44f32bb90
image: perconalab/percona-postgresql-operator:PR-579-d8fb60432-pgo-apiserver
image: perconalab/percona-postgresql-operator:PR-579-d8fb60432-pgo-event
image: perconalab/percona-postgresql-operator:PR-579-d8fb60432-pgo-rmdata
image: perconalab/percona-postgresql-operator:PR-579-d8fb60432-pgo-scheduler
image: perconalab/percona-postgresql-operator:PR-579-d8fb60432-postgres-operator
image: perconalab/percona-postgresql-operator:PR-579-d8fb60432-pgo-deployer


Setting breakpoints does not work with source-map

Which OS: MacOs
Which VSCode version:  Version 1.31.0-insider (1.31.0-insider)
Which extension version:  1.2.0
Which adapter type: classic
Which LLDB version: lldb-1<IP_ADDRESS>
What is the problem and how did you get there:
For my project I recently had to specify a sourceMap. Ever since then the break points can not be added anymore via the GUI. They appear greyed out and do not trigger.
Adding them via the debug console (breakpoint set --file file.cpp --line 1379) still works fine.
I have debugged this issue to https://github.com/vadimcn/vscode-lldb/blob/master/adapter/debugsession.py#L430 where file_path is the path to the file on my local machine. If I modify the file path and apply the inverse source map before calling BreakpointCreateByLocation everything starts to work again.

Sorry, missed you report back when.   Are you still having this issue?

would this be fixed?

yes, this is fixed


BibDocFile: doctype not correctly set/retrieved?

Originally on 2013-07-22
The doctype of a BibDocFile does not seem to be set up correctly any more when using BibRecDocs.add_new_file(…):
$ echo 'test' > /tmp/test
$ python
>>> from invenio.dbquery import run_sql
>>> from invenio.bibdocfile import BibRecDocs
>>> bibarchive = BibRecDocs(21)
>>> bibarchive.list_bibdocs()
[]
>>> bibarchive.add_new_file('/tmp/test', doctype="Foo")
BibDoc(119L, 'Foo', False)
>>> bibarchive.list_bibdocs()
[BibDoc(119, 'Foo', False)]
>>> bibarchive.list_bibdocs(doctype="Foo")
[]
>>> bibarchive.list_bibdocs(doctype="Main")
[BibDoc(119, 'Foo', False)]
>>> bibarchive.list_latest_files(doctype='Foo')
[]
>>> bibarchive.list_latest_files(doctype='Main')
[BibDocFile('/opt/invenio/var/data/files/g0/119/content;1',  1, 'test', *, 21, 119, *, 'd8e8fca2dc0f896fd7cb4cb0031ba249', BibDocMoreInfo(119, "ccopy_reg\n_reconstructor\np1\n(cinvenio.bibdocfile\nBibDocMoreInfo\np2\nc__builtin__\nobject\np3\nNtRp4\n(dp5\nS'cache_reads'\np6\nI01\nsS'docid'\np7\nI119\nsS'format'\np8\nNsS'cache'\np9\n(dp10\nS''\n(dp11\nS'flags'\np12\n(dp13\nsS'descriptions'\np14\n(dp15\nsS'comments'\np16\n(dp17\nsssS'cache_only'\np18\nI00\nsS'version'\np19\nNsS'relation'\np20\nNsS'dirty'\np21\n(dp22\nsb."), False)]
>>>
>>> foo_file = bibarchive.list_latest_files()[0]
>>> foo_file.get_type()
'Main'
>>> run_sql("SELECT id, status, docname, doctype FROM bibdoc WHERE id=%s", (119,))
((119, '', None, 'Foo'),)
>>> run_sql("SELECT id_bibrec, type, docname FROM bibrec_bibdoc WHERE id_bibdoc=%s", (119,))
((21, 'Main', 'test'),)

This is tracing how "Foo" and "Main" get inserted into bibdoc and bibrec_bibdoc:
bibarchive.add_new_file(…, doctype, …)
 -> BibRecDocs.add_bibdoc(…, doctype, …)
   -> BibDoc.create_instance(recid=self.id, doctype=doctype, a_type = 'Main')
      # where a_type is left to default value 'Main'
      -> BibDoc.create_new_document(doctype, rec_links)
         # where rec_links has been built using a_type, because no docid was provided
         -> BibDoc.create_new_document(doc_type=doctype, rec_links)
            # where doc_type (Foo) is inserted as `doctype` into `bibdoc table`
            -> doc.attach_to_record(rec_links[i]["rec_id"], a_type=rec_links[i]["a_type"], …)
              # where a_type (Main) is inserted as `type` into `bibrec_bibdoc`

while list_biboc(…) is retrieving the doctype on the basis of bibrec_bibdoc:
    def list_bibdocs(self, doctype=None):
        [...]
            return [bibdoc for (bibdoc, attype) in self.bibdocs.values() if doctype == attype]

(and self.bibdocs is initialized in BibRecDocs.__init__(…) with BibRecDocs.build_bibdoc_list(…)):
    def build_bibdoc_list(self):
        [...]
            res = run_sql("""SELECT brbd.id_bibdoc, brbd.docname, brbd.type FROM bibrec_bibdoc as brbd JOIN
                         bibdoc as bd ON bd.id=brbd.id_bibdoc WHERE brbd.id_bibrec=%s AND
                         bd.status<>'DELETED' ORDER BY brbd.docname ASC""", (self.id,))
        for row in res:
            cur_doc = BibDoc.create_instance(docid=row[0], recid=self.id,
                                             human_readable=self.human_readable)
            self.bibdocs[row[1]] = (cur_doc, row[2])

Is BibRecDocs.add_new_file(…) the proper way to add a file?
Should Foo have been inserted into bibrec_bibdoc too?
Should BibRecDocs.build_bibdoc_list(…) builds its list on the basis of table bibdoc, not bibrec_bibdoc?

Given the general legacy code freeze and new implementation in master I am closing this issue.


When exporting canvas, hover preview of linked note not supported

In the Obsidian app canvas view, if a card has a link to a note, hovering over the link will present a tooltip preview of the note. This is very useful, but is not exported with the plugin in its current version.
I would like to have the same functionality in the HTML export plugin.
https://github.com/user-attachments/assets/e1806260-b38b-4070-b391-5381f7dffcc9

This is actually the case with any link to a markdown note anywhere in obsidian, and there is already an issue tracking this feature request. I do plan to implement this feature soon though, so keep an eye out :)
Since this is a duplicate I will go ahead and close it

Thanks!


Support multi-lingual content folders in the CMS

Feature Request
Is your feature request related to a problem? Please describe.
Right now Wowchemy CMS does not work with multi-lingual sites having different folders for the different languages.
Describe the solution you'd like
To be able to create a file such as this one inside a language folder.
Describe alternatives you've considered
Hugo's alternative multi-lingual approach may be a workaround.
Additional context
Empty Wowchemy CMS, even though the site has plenty of content (live here)

Blocked by https://github.com/netlify/netlify-cms/issues/4416 - feel free to upvote/contribute

Based on what is described in https://github.com/netlify/netlify-cms/issues/4416, and from my testing of the code in the gist linked there, the solution that is being worked on in that issue won't match the multilingual folder structure in wowchemy. It is aiming for a structure like this: <locale>/<folder>/<slug>.<extension>. With the <folder> in wowchemy being content/<content_type>, the resulting structure when using this proposed config of the cms with wowchemy is:
<locale>/content/<content_type>/<slug>.<extension>
whereas the structure required by wowchemy is like this:
content/<locale>/<content_type>/<slug>.<extension>
I'm currently looking for any  workaround for using netlify-cms multilingual editing for a wowchemy multilingual site. I've forked netlify-cms and implemented a variation on the proposal from https://github.com/netlify/netlify-cms/issues/4416 which looks like this: https://github.com/cleeyv/netlify-cms/commit/c49e136d4d57befed7d0dc7dc5ee2458042f4fb6 (this is the same as the gist linked above, except for edits to lines 81 and 97). It works well for creating new content, but still fails to view any of the content. I have confirmed that this is because for viewing content, the cms is still looking in content/<content_type>/<slug>.<extension> so it is not taking into account the locale folder at all.
I'm going to keep trying to find a solution by modifying netlify-cms, but I'm not a javascript/react developer so its not moving quickly. In the meantime I'd gladly try any other workarounds that might get this functional, if anyone has any suggestions.

A generous friend who knows javascript/react, @elhil (thank you!), was able to help out with this and get netlify-cms multilingual fully working with the folder structure of wowchemy: https://github.com/cleeyv/netlify-cms/tree/wowchemy-i18n
For anyone who wants to use this, the instructions for building netlify-cms are here: https://github.com/netlify/netlify-cms/blob/master/CONTRIBUTING.md (after installing dependencies, run yarn build). The relevant output file from the build is at packages/netlify-cms/dist/netlify-cms.js (this file is already minified even though it doesn't have .min in the filename).
The final steps are hosting the modified build of netlify-cms.js somewhere, and then changing the src URL in https://github.com/wowchemy/wowchemy-hugo-modules/blob/main/wowchemy-cms/templates/single.html to load it. I modified this file by creating a fork of wowchemy-hugo-modules and then configuring it in the go.mod of my site, though there may be other ways to do this as well.

@cleeyv thanks for sharing. I assume you've submitted your implementation as a PR to be reviewed by the Netlify CMS team? If so, please can you link to the PR and let us know if you received any feedback from the team yet on when it might be merged?
Note that there is also a fork of netlify-cms at https://github.com/wowchemy/netlify-cms where I implemented support for multilingual Hugo sites after this issue was initially opened by Rodri. It's fully working with the exception of a similar issue which you reported on 28th April where only newly created files were showing in the lists of content. The Netlify CMS codebase was lacking in documentation when I having a go at it and my fork was in need of someone more familiar with the codebase to help resolve the issue with content lists.

My primary goal was more to find a functional workaround for a wowchemy site, and less to make a contribution to upstream. However, I fully agree that it would be better to get this into upstream, and opening a PR would get the ball rolling in that direction. I'm still testing that the workaround actually works on my site. If I don't run into any serious bugs I could submit a PR, and then l would definitely post an update about it here.


base64 encoded scripts on Windows VM

Hi,
Is it possible to use base64 encoded scripts directly inputted into the ARM Template ?
On Linux VMs this is possible through the 'script' setting set under 'protectedSettings'.
If not, do you think that it is possible on Windows VMs to have this CustomScriptExt without an actual script download from Azure Storage / GitHub / etc. ? (I was thinking about multiple 'commandToExecute' under 'protectedSettings'.)
Thank you for any input you offer!
Kindest regards!

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

ID: fbccf725-84f1-a5c0-2ff5-49dace353416
Version Independent ID: f2d1b170-702d-9c2a-4787-ca75dea9ce0c
Content: Azure Custom Script Extension for Windows
Content Source: articles/virtual-machines/windows/extensions-customscript.md
Service: virtual-machines-windows
GitHub Login: @danielsollondon
Microsoft Alias: danis

@carct  Thanks for the feedback. We are actively investigating and will get back to you soon.

@carct great questions!
@danielsollondon is this something you might be able to shed some light on?

found this ref regarding 'customData' property under the VM \ osProfile which sounds that it can accomplish what i would want:
https://mitra.computa.asia/articles/msdn-passing-custom-data-your-azure-vms-metadata-service-workaround
will give it a try in a couple of minutes

so i managed to use the customData field as per directions from the above link, just that when passing a base64 encoded string as parameter (generated by a different script and dynamically overriding the parameter from the template) it didn't recognize it as base64... therefore i have basically double-encoded the original script (not an issue in the end as it got even more small) and then used a CustomScriptExtension to further decode and execute the script on the VM.
PS: concatenated the multiple powershell cmdlets into a variable and passed it on the 'commandToExecute" under protectedSettings of CustomScriptExtension
a bit of a hassle, to be honest, but in the end i managed to obtain the desired result
if i may, i would want to propose the same capabilities as with the new v2.0.x CustomScript on the Linux VMs (it's funny that the extension for Linux VMs is actually a bit more evolved... )
thanks to anyone getting involded!

Thank you for taking the time to share your product and documentation feedback with us. Your input is valued because it helps us create the right documentation for our customers. Due to the volume of issues in our queue, we are closing open issues older than 90 days. We hope to continue hearing from you. Thank you.


Error handler

From @kptdobe

The pipeline should have a global error handler, like .error(error, status) which allows you to centrally manage error cases. The pipeline is responsible to call the handler if any of the step returns an error.
What do we do if the pre.js fires an error ? You must extend the pipeline and add a when right after to handle the case the pre function has a error case… ? That does not make sense.

I would probably keep the error signature identical to all other signatures, so make it error(context, action).
@kptdobe I assume that whatever error function you specify would unset the context.error property when it's done, correct?
If you force doing this manually, you could have multiple error functions, maybe even in combination with when, so that you can escalate fixes.

If there is an error that is not treated / caught in any of the action, this is a critical state and you must stop the pipeline execution and end up into the error handler.
I don't really know if error function would unset the context.error but at least if should properly log the error and make sure the proper response type is set correctly. Some use cases:

With https://github.com/adobe/helix-cli/issues/302, it is clear that the developer forgot to catch an error, things should end up with a 500 and the appropriate log message.
If fetch fails, there could be multiple reasons: md does not exist (404), wrong branch name (500 but user error), github might be unreachable (500 but unexpected error)...

Some of those errors are the responsibility of the fetch method like non existing content (404), some falls into the category "user configuration error", some falls into "unexpected things happened" and ends up with a 500 that needs to be debugged.
The simplest thing for now is to route everything to a generic error function that logs the error and sets a 500 status if no status has been set before.
In dev, you only need the most accurate error message to debug.
In production, you may anyway only have 2 interesting cases: "not found" and "oops, something went wrong and there is nothing you can do about it" which are never cached.

suggest:

if a pipeline step throws an error, it sets the payload.error
a step is ignored, if payload.error is set
pipeline.error() is executed, if payload.error is set
pipeline.error() usually sets the response.status if not already defined.
pipeline.error() can clear the payload.error if needed (i.e. to implement a try/catch like semantics around some steps)

further, for the html pipeline:

fetch should set the response.status to 404 AND the the payload.error
we add error(set-status) as last step.

I'll address this together with #81 in #82

@tripodsan I agree with your proposal, but not this part:

a step is ignored, if payload.error is set

There are two ways to read this: "the next step is ignored" or "all steps are ignored as long as payload.error is set". In both cases there is too much magic for me, and it would be more transparent to simply add .when(okay) to the steps that should be skipped in case of errors.

There are two ways to read this: "the next step is ignored" or "all steps are ignored as long as payload.error is set".

I don't think this is ambiguous: if error then skip ...

In both cases there is too much magic for me, and it would be more transparent to simply add .when(okay) to the steps that should be skipped in case of errors.

I disagree. Actually, I think error is error and the entire pipeline should just stop and throw an exception. The solution above is a tradeoff of this radical approach and allows 'some' error handling.

As I wrote in my review in #82, I would prefer an approach that encourages error remediation, i.e. trying to fix things instead of just making the error prettier (i.e. setting correct status, or presenting error messages in HTML).

I would prefer an approach that encourages error remediation

yes. this can be done with the current proposal. just add an error() step when you want to handle the error.

or presenting error messages in HTML).

which is a very bad behaviour as it reveals too much information.


Fix #83 (scripts).  Run date just once in production scripts.

Waiting for these three to all run at least once before merging.


[3.2] Backport hex_to_int/bin_to_int zero check and C# changes

See #43978. This backports the zero check, and also backports the C# changes including adding BinToInt.
Crashing with a zero length string can't be done from GDScript in Godot 3.2, but could potentially be done elsewhere.

Thanks!

Thanks!


scully serve not working inside windows 10

I have run ng build & npm run scully
🐞 npm run scully:serve not working or start the server
When I run npm run scully:serve, it shows starting static server..., but then it stops automatically.
Description
🔬 npm run scully:serve
💻Your Environment
Angular Version:

** Scully Version:**
0.0.8

Hello @mbaljeetsingh im working on windows 10!
For scully you need an angular 9 project!
Can you tell me what steps you try?
This is one example in windows 10

Hi @jorgeucano
Yes, I'm using it inside the Angular 9 project.
Steps I have taken are as follows:

npx @angular/cli new scully-project --routing --style=scss
ng add @scullyio/init
ng build
npm run scully
npm run scully:serve

From the screenshots displaying your version, you using v8, not v9

From the screenshots displaying your version, you using v8, not v9

Oh yeah, This is because, I haven't used the global cli for the scully demo project, I just npx to create create project.
Here's the package.json screenshot,

@mbaljeetsingh npm run scully just show this and never finish?

npm run scully runs successfully, but getting some weird error at the end.

ERROR: The process "6208" not found.

ohhh ok, is not an error really @mbaljeetsingh , we are killing the process and for this show this error, now we are in alpha and for this you see this error.
We are working and focus in other place now, after finish some task, we will work in this error!
So if this the unique "error" you have, please tell me and close this issue :)
thanks!

Alright, Yes the build is successful with npm run scully.
Actually, this issue is related to npm run scully:serve. Can you share how it works?

Scully serve is for create 2 servers
the first server is a http server with the scully compile version.
the second server is a http server with the angular build version.
We found a bug when try to run the serve and you have a gost process...
So please, Can you try to start the npm run scully:serve and if dont run anything check in your task manager if you have any node process running, if you have, please kill the process and try again to run npm run scully:serve and check if work!
thanks

@mbaljeetsingh As @jorgeucano explained, we are still in alpha.
There are a number of issues with scully serve which we are already addressing.
You issue is one of them. We expect to have this fixed in one or two days.
Scully serve in meant to be a developer tool so you can try the results of scully without any 3rth party tool.
Also, we are using it to 'serve' your distribution files, so we can run our generator against it. That is the reason  it opens up 2 ports (when all fixes are in)
I assume I answered all your questions, and will close the issue now.
When you want to know more, feel free to add in more comments on the closed issue, and I will make sure I'll answer those.


Constant Crash on Ubuntu, reproducable

Update: reproduction steps described below. problem appears related to the hover bubble over files  in the file tree.
After some time of running atom --safe (1.10.2) it crashes. The nature of the crash is that the window remains open but when I resize the window the canvas does not update. I am no longer able to scroll in the tabs open or do anything with atom other than forcefully quit it. The system I am running on in Ubuntu 14.04.4LTS with Gnome 3.12.2.
I've had this issue consistently through various atom versions. Because atom does an okay job in recovering the previous open state, and because most of my projects involve git which means I'm saving a lot, it doesn't bother me too much. But often I'll get frustrated and update my atom (compiling from latest stable) in hopes it is fixed.
I would love to figure out how to reproduce this but I haven't found it yet. It seems to happen only when I'm working on a tree of files, and have a few tabs open at least. Then after about 10 or 15 minutes I will be scrolling and it will lock up.

In the future, please completely fill out the provided issue template - it's there for a reason.
Some questions for you:

What is the output of atom --version?
Would you say that the "crash" is more of a "Atom stops responding" issue?
Has the freezeup only happened while you were scrolling, or has it happened at other times as well?

Thanks for the response
Output of atom --version
Atom: 1.10.2
Electron: 0.37.8
Chrome: 49.0.2623.75
Node: 5.10.0

Today while keeping a close eye on it, it appears to only happen while scrolling. Just now I was able to crash by scrolling the file tree up and down, then right clicking on a file, then scrolling up and down again. It then crashed. Not sure if this is reproducable. I suspect the application needs to be open for a while for this to happen. Will report back again

actually i restarted with --safe now and it happened again. Perhaps one clue is that when it crashes I see a odd change in the file tree part... I suspect what you see here is the leftovers of a lost tool tip or file information tip that shows up on mouse over of file names.

Another clue: Normally on mouse over of a file name on my gnome setup the background of the is brown. When it locks up the background is black, as shown in the image. I'm now almost certain that the file info hover has something to do with the crash. This could be an issue in gnome? though this doesn't happen in other applications. Yet, who knows if I use many/any applications in gnome that also have a file list as that used in atom.

I can now reproduce this consistently.

open a directory in atom that has enough files or files in directories to require scrolling in the file tree
open as many of the sub directories up so their tries are in view
move with the mouse over some of the file names so that the name appears in a hover bubble for each as you move the mouse over them
with one file name still having it's name shown in a hover bubble over it, then scrill the whole tree

this consistently causes atom --safe to crash on my system

could someone remove the "needs-reproduction" tag now that I've detailed how to reproduce? Or, does removing the tag require a core member check this? If the latter then ignore this request

Or, does removing the tag require a core member check this?

Correct.  I haven't been able to reproduce this on Windows.

Can you post your system specifications? What is the RAM consumption by the system? It may be due to a slow system or little available RAM.

It is definitely something with the Gnome window manager. (version 3.12) I have since switched to using a different window manager for a while (awseomwm) and the bug with atom io has not appeared.

I was unable to reproduce the issue. I however have a bit different versions gnome - 3.18.5 Atom 1.11.2. Rest are same.

Did you try updating to newer versions of gnome-shell and atom? Does the problem exist with the latest version also?

I'm on latest atom. The problem has gotten worse with each new atom release over the years. Typically when I would get tired of the bug I would update atom only to find it is worse.
There is no newer version of gnome-shell for ubuntu 14.04lts. I started down the road of attempting to compile latest gnome for this distro but now two days later, it's clear if gnome is the issue it will be easier to backup and move over to a different distro with later gnome.

This bug from Chrome relating to gnome 3.12+ubuntu 14.04 seems pretty much the same: https://bugs.chromium.org/p/chromium/issues/detail?id=381732#c14

@Ceda-EI upgraded the host to 16.04. Indeed the bug does not occur there (with gnome 3.18). Wouldn't recommend this solution to anyone though. 14.04lts to 16.04lts has issues.
@50Wliu thanks

Since the issue is solved, it should be closed.

It's not solved yet per-say as other users on Ubuntu 14.04 will still run into this issue.  This issue will be closed when the aforementioned Electron PR is merged.


Understand connected TestConcept

[x] get familiar with the test concept
-> https://developer.android.com/training/testing
[x] add unit tests
[x] automate connected tests in CI, too
[x] monitor coverage using sonarcloud

Fix Test for API 33


documentation | ws client example

The example WebsocketClient Methods doesn't include self.channels so copy-pasting the example doesn't work out of the box.
self.channels = ['ticker']

I think you meant to have that set in the on_open

This also applies to the main code at the tail of websocket_client.py

This also applies to the main code at the tail of websocket_client.py


Add no_std support

This PR enables no_std support for crossbeam-epoch. It depends on crossbeam-rs/crossbeam-utils#3.
The only API-level change is that a default collector is only provided if the "std" feature is enabled (it is enabled by default).
Note that no_std is only properly supported if the "std" feature is disabled and the "nightly" feature is enabled. This is because this crate uses Box and Arc which depend on liballoc.

Looks good to me.
I've never worked much with no_std, so pardon my ignorance for asking:

Would it make sense for crossbeam-epoch to allow no_std support on stable Rust as well?
What's the stabilization plan for the liballoc? Is liballoc going to be forever nightly-only? I'm aware that things like Alloc trait and Heap are going to be stabilized soon, but are they going to be simply reexported into libstd, or will liballoc be available on stable Rust as well?

I don't think no_std is possible with only stable rust since the code relies on Arc and Box, which are only available through libstd on stable.

My understanding is that liballoc will eventually be stabilized, but I'm not sure if there is a timeline for this.

In any case, most people currently working with no_std environments use nightly compilers, so I don't think it should be too much of an issue.

I'm happy to merge this now.  @stjepang please have another look and merge!


Feature/home page text

change homesite

 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.:x: tklawun sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


When updating datasource via service call, it may throw a db-locked error

Describe the bug
setup a gtfs with a short refresh interval
add a service call that updates the same source
as the short interval wil kick in before the service has updated it, the db-locked will occure

Resolved in dev release 0.1.6c1


REST 200 response with empty body

It seems that the Angular issue with a REST 200 response with empty body has been fixed with the latest version upgrade (Angular 5.2.1).
This cause a bug to fix on the application.
Indeed, unlike before that the angular bug was resolved, now response with empty body are parsed with this error generated :
TypeError: Cannot convert undefined or null to object
    at Function.keys (<anonymous>)
    at DSOResponseParsingService.BaseResponseParsingService.flattenSingleKeyObject (base-response-parsing.service.ts:129)
    at DSOResponseParsingService.parse (dso-response-parsing.service.ts:30)
    at MapSubscriber.project (request.effects.ts:40)
    at MapSubscriber._next (map.js:79)
    at MapSubscriber.Subscriber.next (Subscriber.js:90)
    at CatchSubscriber.Subscriber._next (Subscriber.js:126)
    at CatchSubscriber.Subscriber.next (Subscriber.js:90)
    at MapSubscriber._next (map.js:85)
    at MapSubscriber.Subscriber.next (Subscriber.js:90)

I think however it should be more appropriate that the status code when there is no content were 204 (https://httpstatuses.com/204)

Note: this issue and #223 may be related, as the error message looks similar.


Please support vector 0.11.x

+1

I'm trying to make sparse compiles with vector-0.11.* in my forked branch.
It seems that at least it type-checks, but finally GHC panics with following error messages (both in GHC 7.10.3 and 8.0.1):
ghc: panic! (the 'impossible' happened)
  (GHC version 7.10.3 for x86_64-apple-darwin):
	Template variable unbound in rewrite rule
  Variable: sc_s1Uto
  Rule "SC:$j0"
  Rule bndrs: [sc_s1Utn, sc_s1Uto, sc_s1Utp, sc_s1Utq, sc_s1Utr,
               sg_s1Uts]
  LHS args: [sc_s1Utn,
             (MV_Complex
                @ (PrimState (ST s2_isBP))
                @ (Complex Double)
                @ Double
                @~ <Complex Double>_N
                sc_s1Utp
                sc_s1Utq
                sc_s1Utr)
             `cast` (sg_s1Uts
                     :: MV_Complex (PrimState (ST s2_isBP)) (Complex Double)
                        ~R# Mutable V_Complex (PrimState (ST s2_isBP)) (Complex Double))]
  Actual args: [sc_s1Uth,
                wild_iuz6
                `cast` (Sub (Sym TFCo:R:MutableV_Complex[0]) <PrimState
                                                                (ST s2_isBP)>_N <Complex Double>_N
                        :: MV_Complex (PrimState (ST s2_isBP)) (Complex Double)
                           ~R# Mutable V_Complex (PrimState (ST s2_isBP)) (Complex Double))]

Please report this as a GHC bug:  http://www.haskell.org/ghc/reportabug

Any thoughts on this?

@konn: You ran into GHC Trac #13410, which will be fixed in GHC 8.2.1. (See also https://github.com/ekmett/hybrid-vectors/issues/7)


Property wrapper type 'Locatable' does not contain a non-static property named 'wrappedValue'

Fails to compile with Swift 5:

I've created a pull request here which fixes it. The fix is to rename the value property to wrappedValue.


[WIP] Introduce support for blue/green deploys.

With this PR, we can execute zero downtime deployments with Marathon as described here: https://github.com/mesosphere/marathon-lb/tree/blue-green-deployment#zero-downtime-deployments

I ran a test based on tests/run-benchmark.rb to validate the correctness of the code. The test involved deploying continuously for ~70 hours, and the results are here: https://docs.google.com/spreadsheets/d/1n7qa6j6wGyEA4LN2gHoGKExOLVTDGiAnJkEZ3FV5BuQ/edit?usp=sharing

@brndnmtthws What exactly is fail_on_length? Most of the other fields are quite obvious.

@SEJeff I think it means the connection was closed before all the data was received, but I'm not sure. The docs are here: https://httpd.apache.org/docs/2.2/programs/ab.html

This looks really good, but I do wish marathon supported it natively, making it easier for an app like this. I'll likely use this when it is merged however so thanks for working on it


Way to get notifications from tray when app is opened?

RN 0.48 RN-FCM 9.7.2
Device: Android Emulator and Android 5 Mobile
App running in background

[Question]
If I have a notification in the tray and I open the app without clicking on the notification - is there a way to read the notifications in the tray? Right now the notification stays in the tray.

Have you try call removeAllDeliveredNotifications in FCM.on(FCMEvent.Notification

I will try that - but I need to process those notifications (read their data) before dismissing those.

dup of #306

remove notification section  and add   "content_available": true....it will work
I have  attached my code
{
 "to":"yyyyyyyyyyyyyyyy",

"content_available": true,
"data":{
"badge":3,
"custom_notification": {
"body": "test",
"sound": "default",
"priority": "high",
"show_in_foreground": true
},
"title":"Alert"
}
}


[Type checker] Don't create new generic signature builders for known signatures

Stop creating new GSBs when the generic signature is always known a priori, i.e.,

When we have a non-generic member of a generic type
We have an unconstrained extension of a generic type or protocol

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge


Repeated rerenders on iOS Safari scroll

On iOS Safari with URL bar at the bottom of the screen (>15.0), the window resizes every time the scroll direction changes, as the URL bar is being minimized or expanded.
This triggers a re render of the SplitText component and the animation is triggered every single time.
Is there a way to prevent those undesired rerenders by filtering vertical resize listening events on iOS devices maybe?

Any updates on this? @Cerom have you managed to find a workaround?

I'm clone this repo, and edit SplitText.ts
useDebounce hook
`
import { useState, useEffect } from 'react';
export function useDebounce<T = any>(value: T, delay: number) {
const [debouncedValue, setDebouncedValue] = useState(value);
useEffect(
    () => {
        const handler = setTimeout(() => {
            setDebouncedValue(value);
        }, delay);

        return () => {
            clearTimeout(handler);
        };
    },
    [value, delay],
);

return debouncedValue;

}
`
and SplitText.ts
`
export const SplitText: FC = forwardRef(function SplitText({ children, ...props }, ref) {
const [key, setKey] = useState(0);
const [lastWidth, setLastWidth] = useState();
const onResize = useDebounce(() => {
    if (viewport.width !== lastWidth) {
        setLastWidth(viewport.width);
        setKey((v) => v + 1);
    }
}, 300);

useLayoutEffect(() => {
    setLastWidth(viewport.width);
}, []);

useIsomorphicLayoutEffect(() => {
    window.addEventListener('resize', onResize);
    return () => window.removeEventListener('resize', onResize);
}, []);

return (
    <SplitTextInner key={key} {...props} ref={ref}>
        {children}
    </SplitTextInner>
);

});
`
also I'm use our project dependencies and add some types, anyone can use it, or I try to find time to fork it

Also facing this issue... any plans to fix?

Yes I ended up splitting the text myself

I ended up splitting the text myself

I ended up splitting the text myself

Damn. Should have tested on safari myself.  I was able to solve the issue without having to split the text myself and I suggest you give it a try...
I just forked the repo and added it directly to my project. Then I simply removed the resize event handler in SplitText.tsx... and instead added a resize handler in my component that re-renders the text only if the window width has changed (instead of on width OR height change) - the re-render happens because I'm storing the window width as a state variable. I hope this helps someone!

My component:
  const [currentWidth, setCurrentWidth] = useState(
    typeof window !== 'undefined' ? window.outerWidth : 0, // next.js builds fail without this window undefined check
  );

  const onResize = (e: any) => {
    const newWidth = e.target.outerWidth;
    if (newWidth !== currentWidth) {
      setCurrentWidth(newWidth);
    }
  };

  useEffect(() => {
    if (typeof window !== 'undefined') {
      setCurrentWidth(window.outerWidth);
      window.addEventListener('resize', onResize);
      return () => window.removeEventListener('resize', onResize);
    }
  }, []);

Cool thanks for sharing!
Should we open a PR for this?


Chinese[zh] Translation

Some minor terminology correction:
On the first page of Supervised Learning——

“Model Type”

would better be translated to

“模型类型”

instead of

"型号类型"

Thank you for your suggestion @THUrachel. The change will be visible on the next iteration of the website.


[Feature Request] The automatic downloads of Boxart and Icon appear to be inaccurate

When I add 465607F0 and 465607E7, the manager confused the two.
I got boxart for 465607F0 and title name for 465607F0, whether it's 465607F0 or 465607F0
It is recommended to learn AuroraAssetEditor to get assets from xbox.com and specify the store area to get them from.
https://github.com/XboxUnity/AuroraAssetEditor

Please post logs from Xenia Manager. Use a service like Pastebin or Github Gists

Log-20241020_002.txt
I tested other games and it seemed like a big problem, 4B5607D4 was NINJA GAIDEN 3, and the manager mistakenly identified it as NINJA GAIDEN 3 Razor's Edge, but boxart was correct.

I think I have fixed this now in the latest dev build. Let me know if it works.

@shazzaam7
I have updated to 1.15.2
465607E7 is still incorrectly identified as 465607F0. And the 465607F0 icon is missing. This icon appears to exist only in the Japanese Marketplace.


Adds scroll to selected group in left nav

Fixes #565

If you do not understand what I mean, I can do it myself (it is quite clear for me as I implemented this left menu)


small images less than 10Kb

it's no triangulate. check an image from attach

thanks for cool app!!


Option to already pre-provide new column names in 'Split into several columns' dialog

When doing the quite simple operation of splitting columns via the Split into several columns... menu item, OpenRefine will simply add a number to each splitted column (eg. when I split a column 'Address' into three columns, I may end up with Address 1 / Address 1 1 / Address 1 2, depending on how often I split again). I am literally never happy with these column names and always rename them afterwards (e.g Street / Number / Postal code / Municipality).
Renaming columns after the fact is a bit of a PITA and a lot of extra menu fiddling and clicking, especially when you are splitting a lot (I tend to split a lot). When splitting, in most cases I actually already know what I want the new column names to be, so:
Proposed solution
I suggest to add an option / field to the Split into several columns dialog window that allows the user to already type the column names as they would like them.

I am aware that it's sometimes not known how many columns the user will end up with, especially when they don't enter a number at 'Split into ... columns at most'. Perhaps the field can be pre-filled with Column 1, Column 2, ... (of course using the original column name) if the user doesn't enter a number there, but as soon as they enter one, they can actually adjust the names themselves, or something? I'd be happy to hear other suggestions, no strong feelings here.

This sounds reasonable, and the input box could operate similar to the CSV/TSV importer that has a fairly new feature option of renaming the columns.  So just providing a comma separated list of column names into the input box.  In your proposal above, that similar input box should be greyed out and not able to accept input until the user enters a number in the Split into [ __ ] columns at most


Multibyte characters are removed

The process of synchronizing with mms removes multibyte characters from mms.
I think it might be due to the replace processing of the fixString function in DocGenUtils.java.
https://github.com/Open-MBEE/mdk/blob/develop/src/main/java/gov/nasa/jpl/mbee/mdk/docgen/DocGenUtils.java#L178

when you say synchronizing, did you mean just the pushing of a project to mms causes those characters to not be on mms, or is it limited to use cases involving ve/view generation?

As far as I know, it occurs only in the latter case. When I tried to use multibyte characters as a column name in the TableStructure, they were not printed in mms after the ve/view generation process.

Thanks for your response.


Add labels to Dockerfile of cluster-setup and gateway-deployment

It looks good to me. Let's first agree on #129 to make sure things are aligned.


Dice Roller

Is it possible to insert a dice roller to avoid having to open the imperial cards?
Thank you

Hello!  There is a dice roller in the app already:
https://github.com/GlowPuff/ImperialCommander2/wiki/4.-Imperial-Activations#the-dice-roller


Improve name resolution process

Right now the umbra-js library has two different codepaths for name resolution—one for ENS and one for CNS. The resolution library introduced in #38 supports resolution of both ENS and CNS names.
The scope of this issue involves

First examining the resolution library to confirm covers all the required ENS functionality, and to ensure there's no concerns with integrating it
Assuming it looks good, we can use that in place of the current process

Name resolution process is sufficient for now, so closing this


Viewport size

For the sidebar, on smaller displays life Amazon Fire Tablets, the sidebar doesn't appear.  Its because its screen size is width is 1024 and you have it set statically to that.  Maybe you can modify the code so the user can specify at what screen size to hide the side bar?

In the new release v1.1.0 this possibility is added. You can now use the config-key sidebar.screenMinWidth to set the minimum screen width at which the sidebar will be shown.
But if your screen is 1024px width, I don't think that you have to change the default setting, because I also corrected a bug in screen width calculation.


Get get_client() from ocp_utilities

!DO_NOT_MERGE

!DO-NOT-MERGE

depends on:
https://github.com/RedHatQE/openshift-python-utilities/pull/132
https://github.com/RedHatQE/openshift-python-utilities/pull/130


i18n BZs: 1284792 1284799 1284816 12848818

Way to go Eder! +1 to merge.

@manstis btw on 1284799 the screen title will remain the same
on 1284816 the place name will remain in english because is a uberfire concept.
is that right?

"Place Names" as in PlaceManager.goto("placeName") don't need to be i18n'ed as they are internal logical identifiers for physical screens and are not (should not) be exposed to Users. If we are exposing them to Users we should probably change to use a "regular" GWT PropertyBundle constant (ie like all other i18n).. Have I understood correctly?

I mean, On the edit screen popup there is a label called Place Name and a list with all the available screens/editors.
I don't think that translates this is a good idea because that is related with uberfire screen/editor identifier concept.

I've changed my mind! I'll i18n this! :)

master: 5f9e274b7c6f016e75eaf480d7d18c8efa3f56af


Consider adding deprecated_consistency to recommended

It's frustrating to remove something that you think was properly deprecated only to discover that many users of it never saw the deprecation warning. Enabled in Flutter.

This seems like a tooling issue: Any use of a member or constructor of a class that is marked deprecated should be flagged.

@mit-mit to file tools issue

@goderbauer do you have an example of a case where you are not seeing a warning?
I tried to create a class marked deprecated, and I get usage warnings on both static method calls and on constructor calls.

@Piinks Do you remember where we saw the inconsistencies around deprecation that let us to adopting the deprecated_consistency lint?

cc @pq, is that fixable?

Just to clarify, I think what you're asking is whether, at least for the purposes of deprecation warnings, we can consider fields and field initializing parameters to be equivalent. That is, if either is deprecated then we consider both to be deprecated.
Personally, I don't have any problem with that, but we might want to update the documentation for deprecated to make this association clear. @lrhn

This has been approved for the recommended set.

An update - this is still approved, but we'll delay adding this to the lint set until after we have a fix for https://github.com/dart-lang/linter/issues/4752#issuecomment-1728213638 (and that fix has shipped in an sdk).

@pq @lrhn - from the discussion here: https://github.com/dart-lang/linter/issues/4752#issuecomment-1729426585, are we considering this lint not necessary - that we should bake a version of deprecated_consistency into the warnings that we normally issues for deprecated members?

We should make sure to warn about unnecessary //ignores!

I have a CL that introduces such a check (first uploaded in March of 2021), that has not yet been able to land for various reasons that I won't go into here. Suffice it to say that warning about unnecessary ignores presents some social challenges. (Happy to talk about it more, but maybe in a different thread.)

This is a lint about two (or more) places in the code, so we should be clear on how they interact, including where // ignores can go.
...
Then we should make sure that the // ignores work as intended.

I'm not saying that we couldn't enhance the current implementation to support what you're proposing, but the implementation we have today requires that the ignore comment be on either the same line as the beginning of the diagnostic's highlight region, or by itself on the line before the beginning of the highlight region.

..., that has not yet been able to land for various reasons that I won't go into here.

Let's hope we can do something. :)

...  the implementation we have today requires that the ignore comment be on either the same line as the beginning of the diagnostic's highlight region, or by itself on the line before the beginning of the highlight region.

So commenting the @override is out. (I'm assuming it's not part of the region. If it was, things would get much weirder than they are today.)
This diagnostic relates to two places in the code, the variable declaration and the initializing formal, being out of sync. I'm assuming we only make one diagnostic for it (or if we make two, we'll need two ignores).
It's probably better to always report it at the initializing formal, because there can be more than one of those,
which means you can, and must, ignore each one of those independently. Annoying, but doable. The assumption is that's it's going to be rare to need an ignore, and in those cases, the first few steps of the necessary refactoring will likely remove the warning, leaving only a deprecated getter backed by something else. So not a big issue.
An initializing formal is usually not that long, and easy to break onto a line of its own, so it should be easy to add a comment to that line, even if it might mean reformatting parameter list to give it a new line.
So the analysis server can insert a suitable comment, which won't be moved by dartfmt. That's probably good enough.
SGTM.
Aka:

If an initializing formal is deprecated, and the instance variable it's initializing is not, or vice versa,
the deprecation is considered inconsistent, and a warning is emitted for the initializing formal.
If the design is deliberate, and one of the two will be removed while the other stays in the API,
an // ignore: deprecated_consistency comment should be added to the initializing formal,
either at the end of the line where the initializing format starts, or alone on the line before.
Using an // ignore: deprecated_consistency comment is the recommended way to signal
that the initializing formal and instance variable are not both going away.
The comment will become unnecessary as soon as the deprecated member has been removed,
and possibly earlier if the class is refactored to prepare for removing the deprecated declaration.
Example:
class Example {
  @deprecated
  final int goingAway;

  final int hereToStay;

  Example.v1(
      // ignore: deprecated_consistency
      this.goingAway,
      @deprecated
      // ignore: deprecated_consistency
      this.hereToStay);

  Example.v2(
      this.goingAway, // ignore: deprecated_consistency
      @deprecated this.hereToStay // ignore: deprecated_consistency
      );

  // No warnings, everything is consistent.
  Example.v3(@deprecated this.goingAway, this.hereToStay);
}

This should be enough. Again, it's not going to be used often, it's likely that the comment goes away before the deprecated declaration, if any refactoring is done in anticipation.
All in all, a short-lived // ignore to opt out of a warning for something which otherwise has a good chance of being a mistake, and it's easy to remove the need for the ignore (just change the initializing formal to a normal parameter, and initialize in the initializer list, if it's that important).

I'm assuming we only make one diagnostic for it (or if we make two, we'll need two ignores).

Correct. Given
class B {
  @Deprecated('soon')
  Object? field;

  B({this.field});
}

there is a single diagnostic and the highlight covers this.field.

It's probably better to always report it at the initializing formal, because there can be more than one of those ...

That was our reasoning too.

An initializing formal is usually not that long, and easy to break onto a line of its own, so it should be easy to add a comment to that line, even if it might mean reformatting parameter list to give it a new line.

While that's possible, it isn't necessary. Either of the following solutions will also work:
class B {
  @Deprecated('soon')
  Object? field;

  // ignore: deprecated_consistency
  B({this.field});
}

or
class B {
  @Deprecated('soon')
  Object? field;

  B({this.field}); // ignore: deprecated_consistency
}

Given the relative value we get from this lint, and the work necessary to fix it, we're unlikely to add this to our recommended lint set. Closing.


Add "--arch-filter" flag that mimics "--skip-constraints"

… but without printed warnings and without applying Constraints

$ bashbrew --arch amd64 list --uniq clefos
clefos:7
$ bashbrew --arch amd64 list --uniq --arch-filter clefos
$

$ diff -u <(bashbrew --arch s390x list --uniq debian) <(bashbrew --arch s390x list --uniq --arch-filter debian)
--- /dev/fd/63	2021-08-02 15:31:44.164225854 -0700
+++ /dev/fd/62	2021-08-02 15:31:44.164225854 -0700
@@ -5,18 +5,12 @@
 debian:buster-backports
 debian:buster-slim
 debian:experimental
-debian:oldstable
-debian:oldstable-backports
-debian:oldstable-slim
 debian:rc-buggy
 debian:sid
 debian:sid-slim
 debian:stable
 debian:stable-backports
 debian:stable-slim
-debian:stretch
-debian:stretch-backports
-debian:stretch-slim
 debian:testing
 debian:testing-backports
 debian:testing-slim


Reformat the CNODE structure

This branch removes a lot of overhead from the CNODE struct.
This is a major change from the main branch, but the performance gains justify it completely.
Basically this branch shows that pre and pos force operations are not needed.
At the moment, only the Burgers examples have been updated.

LGTM,
A question:
Should we fix the NeuralClosure dependency first (i.e. bringing the code that we use form NeuralClosure to this repo)?
and a request:
Can we squash and merge? I see a lot of commits, let's try to summarize in the merge commit. (I offer to do that myself)


YARN-11075. Explicitly declare serialVersionUID in LogMutation class

Description of PR
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 38s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m 18s

trunk passed

+1 :green_heart:
compile
1m  6s

trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
1m  2s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
0m 45s

trunk passed

+1 :green_heart:
mvnsite
1m  2s

trunk passed

+1 :green_heart:
javadoc
0m 49s

trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 43s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
1m 55s

trunk passed

+1 :green_heart:
shadedclient
21m  1s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 57s

the patch passed

+1 :green_heart:
compile
1m  5s

the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
1m  5s

the patch passed

+1 :green_heart:
compile
0m 54s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
0m 54s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 36s

the patch passed

+1 :green_heart:
mvnsite
0m 55s

the patch passed

+1 :green_heart:
javadoc
0m 40s

the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 36s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
1m 57s

the patch passed

+1 :green_heart:
shadedclient
20m 43s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
96m 46s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 35s

The patch does not generate ASF License warnings.

186m 36s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3986/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3986

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 3bb1e8bff1fb 4.15.0-156-generic #163-Ubuntu SMP Thu Aug 19 23:31:58 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk /<PHONE_NUMBER>a2be87ebb4cc35c08d20bdd7559014

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3986/1/testReport/

Max. process+thread count
959 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3986/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Hi @brumi1024,
Thanks for working on this.
Patch LGTM, committed to trunk.
Thanks @tomicooler for the review.