[NIFI-987] Added Processor For Writing Events to Riemann

Introduced nifi-riemann-bundle for future Riemann backed monitoring

Added initial PutRiemann processor for writing events to Riemann
using the Riemann batch client.

Values for events are provided using the NiFi expression language
e.g. Metric -> ${latency.milliseconds:divide(1000)}

Ricky,
Sorry for the delay in providing feedback.  Can you look into why the branch has conflicts?  Perhaps I'm misunderstanding what I'm seeing from github.  Also, the PR doesn't appear to address the necessary changes to the nifi-assembly/NOTICE and provide one for the nar itself so that we ensure the EPL 1.0 dependency is properly annotated.  I haven't checked what dependencies it pulls in yet but wanted to put that out there first.
Thanks
Joe

Hey Joe -
Not a problem on the delay! I'll take a look when I get a chance. Thanks!
Thanks,
Ricky

The conflict was inside of the pom.xml inside the nifi-nar-bundles module. The new couchbase module was conflicting with the new riemann module inside the <modules> stanza. I made sure both modules are in there, and rebased the branch on the current upstream master.

Please edit commit message to start with JIRA key and brief summary. i.e.:
NIFI-987 Add Reimann monitoring

- Introduced nifi-riemann-bundle for future Riemann backed monitoring

- Added initial PutRiemann processor for writing events to Riemann
  using the Riemann batch client.

	- Values for events are provided using the NiFi expression language
		e.g. Metric -> ${latency.milliseconds:divide(1000)}

Why do this as a processor rather than as a reporting task?

just speculation, but if you have a bunch of metrics in a file from somewhere and want to post it to Riemann, a processor would be useful. Granted, when I think of people using Riemann, I think of a bunch of distributed processes all sending out live metrics/monitoring information.

@busbey good question. I envision that we can later extend the Riemann bundle to allow internal NiFi reporting to Riemann, similar to how Ambari was recently added. For now, there are plenty of situations where it would be useful to allow reporting of a flowfile's existence and/or attributes to Riemann for basic data flow reporting and monitoring.
Let me give you a few examples:

Send a notification to Riemann for each flowfile passed through a specific processor.  This not only allows for monitoring, but also alerting if desired.
Each node in the NiFi cluster can periodically send a message with a TTL that if expires would indicate that the node in the cluster is no longer running.
Immediate alerting when specific data flow scenarios are met (e.g. failed to extract archive, failed to hit an API).
To extend on what @trkurc said, you could very easily report metrics from files if desired. Or - you could use the ExecuteProcess to collect metrics using arbitrary commands.

I'm sure there are other use cases that can be applied.

that all sounds reasonable to me, thanks for the explanation!

@bbende I updated the review with a different approach to sending events. After I started doing a lot of random disconnect testing, it appeared that the RiemannBatchClient had some really odd behavior when it came to detecting a down connection. Instead of using that, I'm using the regular RiemannClient and batching them together myself. This way I can call the deref() on the IPromise in order to detect connection failures, or timeouts.
If the connection drops while sending events, the processor will yield() and attempt to re-connect on the next trigger until it can do so successfully. I also added a test (testFailedDeref) to verify that this works, as well as tested it internally with both TCP and UDP.

You are not including your nar in the nifi-assembly pom (and would want to include this in the root pom.xml as a managed dependency for versioning) and it was not getting included in the generated assemblies.
Additionally, it seems as though your nifi-riemann-processors does not have the requisite META-INF/services/org.apache.nifi.processor.Processor file for SPI.  Accordingly, it was not getting loaded on NiFi startup as an available processor.

Thanks for all of the comments, I'll address them as soon as I get a chance. As @apiri pointed out, If the connection fails, then we need to call client.close() (if not null) in order for the Riemann library to clean up its netty threads. Otherwise, it endlessly spawns threads which eventually causes the OOM exception.
I don't particularly like the way I'm backing off when failing to connect, as it seems to retry very quickly. This is why the OOM surfaces minutes after killing Riemann. What is the best practice here for yielding for a longer period of time?
    catch (IOException e) {
      if (client != null) {
        client.close();
      }
      context.yield();
      throw new ProcessException(String.format("Unable to connect to Riemann [%s:%d] (%s)\n%s", host, port, transport, e.getMessage()));
     }

That context.yield is what is controlled by the processor configuration so that the user can choose how often this retry should happen, which I think is the right fit.  However, it could be interesting to have different yield strategies as options; as an example, an exponential backoff among them.

@apiri let me know if there's anything else glaring I should address before we discuss committing it. If it helps, I've been running this NAR in production for about a month without any issues.

@rickysaltzer Sounds good, have this on my list to scope out soon as time permits.  Thanks!

@rickysaltzer Did another review on this and overall things look pretty good and the issues of bigger concern seem to be corrected.  I left an additional patch on the ticket with some stylistic items and version update (0.4.2-SNAPSHOT) changes I would like to apply on top of your patch.
My final comments are concerning some of the properties.  In the case of the metric property, this is something that you use as a float to provide in the transmission to the Riemann server.  It would be good to include this in the property descriptor to make this a little clearer.  This should also be applied to the TTL and time properties.  I did some digging to try to figure out how to flesh this out more, but wasn't quite sure how to best describe them.  If you can provide some additional detail on those and the anticipated input, it would certainly help.  Apologies if this is a misunderstanding on my behalf, still quite the Riemann novice.
Thanks again for the updates though and with some additional details on the properties I think we are good to go!  Thanks for all your efforts!

Thanks @apiri, I'll try and update a few of the descriptions to make things more clear and upload a new patch. Cheers

Awesome, thanks!

The latest patch should address most of the description concerns. I'm in the process of running all NiFi tests now and doing a final workflow test with a current master build. I will report results shortly. Thanks again.

Tests pass, and my few workflows I created behave as expected. If it's okay with you @apiri, I'd like to re-base my changes throughout this PR into one commit before I push upstream.

@rickysaltzer Sounds fair.  I had squashed/pushed to my personal fork at a35be5e358108a48b34de3fcf5edcb428a794d03 to make the review a little easier.  Additionally, there were a couple of style items I patched up as well as marking OnScheduled/OnStopped variables as volatile in 1d4fdfe57c9d966c0be6fd5cc537623854f4491c.  If they seem fair, I can merge them to accompany your contribution and push it all together.
Let me know what works and sounds good to you.  Thanks!

Oh nice, I'll see about merging your changes after I squash mine. Would it be okay if I push this patch once it's ready? I haven't yet verified I'm even able to push upstream, and now seems like a good time to try that out.

Yep, of course and please do!  Just was trying to save some unnecessary keystrokes if not needed.  Let me know if you hit any hurdles.  Presumably you can access https://github.com/apiri/incubator-nifi/commit/1d4fdfe57c9d966c0be6fd5cc537623854f4491c.patch to grab a patch for that commit and overlay it on yours.  I am sure there is a more elegant way, however I am also sure I don't know what that is off the top of my head ;)
Regardless, +1 with those changes and the work in your latest push to the PR.

@apiri awesome, your patch applied without issue!


Terms of Use, privacy policy and code formatting.

code formatting, cleanup, remove dead code, added terms of use, privacy policy, etc...

@triceam make sure you refresh the bluemix application

@germanattanasio literally refreshing as you typed that :)


fix(vue-app): add prefetched class to <nuxt-link> after chunk loaded

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)

Description

Add prefetched class only when the chunk is loaded.
Checklist:

[x] All new and existing tests are passing.

Codecov Report

Merging #6623 into dev will decrease coverage by 31.18%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              dev    #6623       +/-   ##
===========================================
- Coverage   95.83%   64.65%   -31.19%
===========================================
  Files          78       78
  Lines        2713     2713
  Branches      702      702
===========================================
- Hits         2600     1754      -846
- Misses         98      764      +666
- Partials       15      195      +180

Flag
Coverage Δ

#e2e
100% <ø> (ø)
:arrow_up:

#fixtures
64.65% <ø> (ø)
:arrow_up:

#unit
?

Impacted Files
Coverage Δ

packages/cli/src/options/server.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/utils/banner.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/run.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/commands/start.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/list.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/utils/memory.js
0% <0%> (-100%)
:arrow_down:

packages/server/src/listener.js
1.69% <0%> (-98.31%)
:arrow_down:

packages/server/src/middleware/error.js
4.16% <0%> (-95.84%)
:arrow_down:

packages/server/src/middleware/timing.js
4.54% <0%> (-95.46%)
:arrow_down:

packages/server/src/middleware/nuxt.js
5.26% <0%> (-92.11%)
:arrow_down:

... and 37 more

Continue to review full report at Codecov.

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


Select all checkboxes on page and all checkboxes in table

Is there a simple way to make a function or there is the function to select all checkboxes on page ( I want to use it on my own button not only with checkbox in TH).
I need also function to select all checkboxes on all pages in table is there a way for this?

            checkboxCheckList () {
                var vuetable = this.$refs.vuetable;
                vuetable.tableData.forEach(function(dataItem) {
                    vuetable.selectId(dataItem['id']);
                });
                this.selected = this.$refs.vuetable.selectedTo.length;
            },
            checkboxUncheckList () {
                var vuetable = this.$refs.vuetable;
                vuetable.tableData.forEach(function(dataItem) {
                    vuetable.unselectId(dataItem['id']);
                });
                this.selected = this.$refs.vuetable.selectedTo.length;
            }

Is this correct?

@lrembacz that could be an option. You could also directly call the method inside vuetable
http://jsfiddle.net/z11fe07p/2668/
selectAll(value){
     this.$refs.vuetable.toggleAllCheckboxes('__checkbox', {target: {checked: value}})
},

Nice :) And the second part of the question? Is there way to check all checkboxes on every page?

No that is not quite possible. If you use server side pagination then, you only get X items per page displayed and saved in your app. It's not physically possible to check all items if they do not exist yet in your app

I would need second query then? Just without pagination? For all of the Ids?

Well that's kind of the whole point of pagination - to not retrieve all the data.
Not sure what's your use case but usually such select-all functionality would affect/select only the items from the current page.

My case is that when Im using filters, Im getting like 200 records. So in pagination i got like 25-100 records per page and now I want to change all status or some other parameter for all 200 records at once (all records I got after filtering).


[Snyk] Security upgrade jsonwebtoken from 0.4.0 to 8.2.2

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Uninitialized Memory Exposure npm:base64url:20180511
  741

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


Make bot output prettier + Dockerify

By changing the nickname we can fit more into view at a glance without having to hover over the bot's username.

Didn't notice the new changes, let me rebase w/ those.

Thank you @JetFault :heart:


Add combinator for function/arity syntax

Example: -export([function_name/0])
Fully closes #12

[x] Have a combinator for function names function/0?

Before

After

Tests

Things done that are "unrelated" to this PR

Add of a changelog (since now we have a published package in hex)

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Selective Code Insertion in Chat Dialog

When suggesting source code in the chat dialog, I want to be able to mark only specific parts of the suggested code and insert them at the current cursor position in the source code. However, currently, the entire generated code block is being inserted.
Steps to Reproduce

Open the chat dialog in Visual Studio Code.
Request a code suggestion from GitHub Copilot.
Highlight a specific portion of the suggested code.
Click the button to insert the code at the current cursor position in the source file.

Expected Behavior
Only the highlighted portion of the suggested code should be inserted at the current cursor position in the source file.
Actual Behavior
The entire generated code block is inserted instead of just the highlighted portion.
Environment

Visual Studio Code Version: 1.90.1
GitHub Copilot Chat Version: v0.16.1
Operating System: Windows 11

Additional Information

It would be beneficial to have a feature that allows for selective insertion of code snippets.
This enhancement would improve workflow efficiency by reducing the need to manually copy and paste specific parts of the suggested code.

Screenshot / Video

Copy/pasting the selection should also work

It's about the flow of just being able to click on the button because the cursor is in exactly the right place.

You can reference the #selection
And we also plan to add other UI gestures from the editor to easily add the selection to the chat.


document required dependcies

Hi! Slightly confused by this PR as we don't use an of these modules? Was this an accidental PR or have I missed something?


Can not found LevelParsingFactorySample.java

I read about this toturial, https://github.com/AlmasB/FXGL/wiki/Building-Levels-Using-Tiled-Map-Editor-(FXGL-11)
and goto the link of the example code(in the end of Text Format section): LevelParsingFactorySample.java
which page is not found

Thanks for pointing this out. It has now been updated, please check it works.


Errors running simple_run.sh

I am running this repository with the following versions:
python 3.5.2
tensorflow 1.8
CUDA 7.5 (My GPU does not support more)
pytorch 0.2.0_3 (For cuda 7.5)
When I run the simple_run.sh it fives me the final values 0 0 0 0, apart from that it only gives warnings, and future warnings, and an index error.
The only changes I made to the code to been able to run ruin it without other errors were:
-change the library for the imread to matplotlib.pyplot, in this website tells that this is the workaround to the error scipy.misc not having that function.
-Ad the paths to PYTHONPATH to detect the functions inside the codes folder
The different parts of the simple run gives me the following results, the whole results are in the txt appended
results_PREMVOS.txt
:
##################### GENERATING COMBINED PROPOSALS
0.0001811981201171875
0.00019884109497070312
##################### GENERATING REFINED PROPOSALS
Several future warning similar to: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
I tensorflow/core/platform/cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
0 90 0 90
0 90 0 90 90 90
pre-prop-load
post-prop-load
0 90 0 90
0 90 0 90 90 90
creating testnet...
using bbox guidance
network:
testnet output : Using SEGMENTATION_LABELS_ORIGINAL_SIZE for calculating IoU
loading model from ../weights/PReMVOS_weights/refinement_net/specific_weights/refinement_specific_weights
Loading data from video
##################### GENERATING ReID PROPOSALS
Several future warnings similar to: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
{
images darta (Not included to make shorter the issue)
"network": {
network data (Not included to make shorter the issue)
}
Traceback (most recent call last):
File "./ReID_net/main.py", line 41, in
tf.app.run(main)
File "/home/user/environment/lib/python3.5/site-packages/tensorflow/python/platform/app.py", line 126, in run
_sys.exit(main(argv))
File "./ReID_net/main.py", line 32, in main
engine = Engine(config)
File "/home/user/git/PReMVOS/code/ReID_net/Engine.py", line 86, in init
self.valid_data = load_dataset(config, "valid", self.session, self.coordinator)
File "/home///git/PReMVOS/code/ReID_net/datasets/Loader.py", line 88, in load_dataset
return DAVISForwardSimilarityDataset(config, subset, coordinator)
File "/home/user/git/PReMVOS/code/ReID_net/datasets/Similarity/DAVIS_Forward_Similarity.py", line 47, in init
super(DAVISForwardSimilarityDataset, self).init(config, subset, coord, annotations, n_train_ids=1)
File "/home/user/git/PReMVOS/code/ReID_net/datasets/Similarity/Similarity.py", line 58, in init
has_tags = "tag" in annotations[0]
IndexError: list index out of range
################# Merging
Several future warnings similar to: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
creating testnet...
using bbox guidance
network:
testnet output : Using SEGMENTATION_LABELS_ORIGINAL_SIZE for calculating IoU
loading model from ../weights/PReMVOS_weights/refinement_net/specific_weights/refinement_specific_weights
../output/logs/ReID_net ReID_general_weights .log
number of parameters: 124,824,040
loading model from ../weights/PReMVOS_weights/ReID_net/general_weights/ReID_general_weights
0 0 0 0

Hey. Unfortunately I have only tested it on one computer setup, and this is the only setup I can guarantee it will work. This setup is (as described in read-me):
python 3.6
tensorflow 1.8
pytorch 0.2.0_3 (pip install https://download.pytorch.org/whl/cu80/torch-0.2.0.post3-cp36-cp36m-manylinux1_x86_64.whl) (this is a version for CUDA 8.0, but I always run this using CUDA 9.0 and it works)
CUDA 9.0
cudnn 7
COCO API for Python 3
It seems like you are using a different setup, I am not surprised that it doesn't just automatically work there. I'm sure there is someway to get it to work on your setup, but it could be arbitrarily complicated and difficult, I am not sure. But I am unable to offer support beyond the setup that I have built this on.


Prebuild stdlib [1.9.x]

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


figure out how to add volumes to user-volumes

not all apps work ok with overlay fs. I'm hoping the user-volumes can be modified from the cloud-init / cmdline, but needs test&doc

Related to https://github.com/rancher/os/issues/2188


[Question] Access to the SVG files?

Hey do you have access to the SVG files used to generate the font? And do you mind sharing them?

@CoderNamedHendrick Thank you but these ones brake when I convert them to a font

Can you point me to what section of icons you're referring to @WalidChebaro?

@CoderNamedHendrick Broken set


[WIP] (GH-40) Switch EditorConfigCommand to use InversifyJS

This is a first pass at fully implementing InversifyJS
Just getting the basics in place
Will need to implement the remaining commands
Putting this out for review

Description
Followed the instructions in this blog post:
https://rpeshkov.net/blog/vscode-extension-di/
Related Issue
Fixes #40
Motivation and Context
Dependency Injection is just good practice, and InversifyJS seems like a very good implementation
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

THis looks pretty good to me, other than the single suggestion I mentioned above

@admiringworm thanks for the review, and yes, I think you are right about that. I will have a play. Also, I think I need another common function for downloading and storing the file, and then simply call this from each command with the required arguments.
I may attempt that refactoring once I have converted all the commands.

Can you have a look at the latest commit when you have a minute?
I have created a new interface, and it seems to work, however, when the command is triggered, it doesn't seem to have anything "in it".  The this variable is undefined:

And as a result, I can't enumerate the commands to call execute on it.
Thanks!

@AdmiringWorm just tested the AllCommand, and it now seems to be working!  Thanks again for that!


Gpay Homepage clone

Hi @pranjay-poddar, I am a contributor in GSSOC'23.
I want to add a New project in React Js projects repository.
Project - I will create a  simple gmail sidebar clone, make It responsive.
please assign me this task under label gssoc'23

Hi sir, I want to work on this using HTML, and CSS. Kindly assign this to me so that I work on it

please assign me this issue under GSSoC'23.

i would like to work on this project. kindly assign me this project


✨ Allow easy overriding of PasswordResetConfirmSerializer user model lookup field

When resetting a password from an email link we may want the lookup field on the user to be something other than the primary key (ex. unique email address). To support easy overriding of the lookup field it is now set as an instance variable on the serializer class.

Alternatively I would suggest using UserModel.USERNAME_FIELD here.

Thanks for the PR. but the API already exposes PASSWORD_RESET_CONFIRM_SERIALIZER where folks can customize as they wish. I would prefer to support a custom serializer rather than making the default serializer customizable.


Update host pipe tests for FPGA emulator

Existing Sample Changes
Description
When a host pipe on FPGA emulator does not specify the depth, the depth will be 1 by default. For tests that write a number of data packets to a pipe, the pipe needs to specify its depth large enough to store data packets without any being read out.
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[X] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Command Line
[ ] oneapi-cli
[ ] Visual Studio
[ ] Eclipse IDE
[ ] VSCode
[ ] When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

Ping @whitepau for review.

Will this change be made unnecessary by fixing https://hsdes.intel.com/appstore/article/#/14019216094?

I have no access to https://hsdes.intel.com/appstore/article/#/14019216094.

@qichaogu @whitepau Let me know when I can merge this PR. Thanks.

@qichaogu @whitepau Let me know when I can merge this PR. Thanks.

I do not like the use model this proposes (namely; that users have to write different code for the emulator vs the simulator).
@qichaogu please consider @justin-rosner's suggestion.
Can you two please work together to ensure that this change is not necessary if the change documented in the HSD ES case is implemented? Since you don't have access to the case here is the summary:
It proposes to give emulation host pipes 'infinite capacity'. A host pipe really has two 'capacities': the depth of the FIFO in the device (which is set by the pipedepth property), and the size of the buffer in host memory. We can grow the host buffer as large as we want at runtime, since we never actually specify how big this buffer is.

Will this change be made unnecessary by fixing https://hsdes.intel.com/appstore/article/#/14019216094?
This PR makes changes that hurt readability of this tutorial without explaining them. Please make sure that code changes are minimal (the specific problem is in hostpipes.cpp) and that they are explained in the Readme.
If this is a general best practice for FPGA, we should update this tutorial as well: https://github.com/oneapi-src/oneAPI-samples/tree/master/DirectProgramming/C%2B%2BSYCL_FPGA/Tutorials/Features/hls_flow_interfaces/streaming_data_interfaces

Would changing the CL_CONFIG_CHANNEL_DEPTH_EMULATION_MODE environment variable also give us the desired result without having to make code sample changes?

Yes, I'm working on fix the mode control CL_CONFIG_CHANNEL_DEPTH_EMULATION_MODE.
So there is no need to modify the sample code here. I'll close the PR.
@justin-rosner @whitepau @jimmytwei Thanks for all your comments.


This is a masterpiece

I have been sufferred 1 month by the error when I updated my code from metis4 to metis5.  The interface you given in this hub is f**king good. I love you , bro.

I have been sufferred 1 month by the error when I updated my code from metis4 to metis5. The interface you given in this hub is f**king good. I love you , bro.

I think I should give my error here to make google retrieve this hub.

I have been sufferred 1 month by the error when I updated my code from metis4 to metis5. The interface you given in this hub is f**king good. I love you , bro.

Hi guys,
if you sufferred with metis5 with the following error:
Current memory used: 392 bytes
Maximum memory used: 392 bytes
***Memory allocation failed for SetupCtrl: ctrl->tpwgts. Requested size:<PHONE_NUMBER>9714178088 bytes
with IDXTYPEWIDTH 64 and REALTYPEWIDTH 64
please read this hub carefully, it is really helpful.

Glad it helped!
This reminds me I should post this to the METIS forum, so people can find it.


bash/sh checking fails

Running :Errors on bash and sh scripts fail with the following message.
Error detected while processing function <SNR>14_ShowLocList:
line    3:
E776: No location list

I cant reproduce this, Is this still occurring? if so, is it only with sh scripts? what options do you have set?
Cheers :)

I played around with it more today, and it looks like I only get this error when trying to bring up the error list when there are no errors.
Either that or something changed on my system between September and now fixed it.:/


Add/fix links and fix typos

#Partial fix for https://github.com/cucumber/docs.cucumber.io/issues/19
Add / fix all the links I could find a place to link to. Just leaves the following:

implementations/jvm.md:
[[Android Runner]] - 4x
[[Data Table]] - 2x
gherkin-reference.md
[[Multiline Step Arguments]] - 2x
cucumber/running-features.md
[[IronRuby]]
cucumber/state.md
[[glue path]]
[[Configuration Options]]
cucumber/step-definitions
[[Multiline Step Arguments]]
ruby/calling-steps-from-stepdefinitions
[[Multiline Step Arguments]]
ruby/ruby-on-rails [Spork and --drb]]
wiki/autotest-integration
[[Troubleshooting]]

Suggestions to fix remaining:

Add info on Android runner (but not by me)
Add file on Multiline Step Arguments, including Data Tables (unless this was intentionally left out / removed?)
Remove links to IronRuby, glue path, Configuration Options, Spork and --drd, Troubleshooting (unless someone can tell me what they should point to)

I didn't know this but IronRuby is a .NET Ruby release like JRuby for Java. Link: http://ironruby.net/
I'm not sure how crucial this is right now (it's certainly out of scope for this link fixing PR anyway) but I noticed the tags.md file uses the old tag syntax of --tags ~@important which will throw warnings in 3.x

Past that, LGTM! Nice catch on some of the grammar stuff outside the links.

@js Thanks! What's the new syntax on the tags? Let's make it up to date!
@olleolleolle Thanks!
Will process comments hopefully this evening

@mlvandijk - not @tag basically. Here's more: https://github.com/cucumber/cucumber/tree/master/tag-expressions

Also added a link to Spork and removed tags for 'glue path' 'Cucumber options' and 'Troubleshooting'.
This leaves 'Data Tables' / 'Multiline step arguments' which I think could be covered by adding the part about StepArguments (DocSring/DataTabels) as it is here: https://cucumber.io/docs/reference
And it leaves 'Android Runner'.

Sorry @js - missed you comment about tags. Will update later.


Indicator setup: UI improvements to disaggregation selection menu

Note: This ticket is dependent on #1849 being completed.
Stories

I can review and opt out of global disaggregations when they do not make sense for a particular indicator. (Right now, global disaggs are not even displayed in the disaggregations menu, and are automatically selected for every indicator, regardless of relevance.)

While browsing disaggregation options, I can easily see what the categories are for any given disaggregation type.

Affects: Indicator setup form > Performance tab > Disaggregation multi-select. Here is how it looks today -- a flat list of all Custom disaggregations. Global disaggregations such as SADD are not displayed.

Acceptance criteria
The proposed design replaces the multi-select with a simple list of checkboxes, showing all disaggregations available to this indicator:

Acceptance criteria

[x] 1. Global disaggregations come first, followed by Country disaggregations.

[x] 1.1. All global disaggregations are checked by default, but may be unchecked.

[x] 1.2. Whenever additional global disaggregation types are introduced, they should be listed in alphabetical order. (To test, we'll need to add a fake second global disagg type in Django admin on the QA server.)

[ ] 1.3. We should have only one global disaggregations -- SADD -- that should be fully spelled out as illustrated in the mockup.

[ ] 1.3.1 There should not be a "---" or a "By Municipality" disaggregation. (If there are any results linked to the latter disaggregation options, we should make that a country-level disaggregation and reassign results accordingly.)

[ ] 1.3.2. All sections should be OPEN by default, with the caret pointed downwards. (Carets should always point downward when the item is open.) I neglected to specify that -- sorry. Unlike the results form, where they're closed by default, here they should be open. (We're limiting the vertical height of the field, so we don't need to worry about it stretching out the form to a crazy length... And we don't want this option to be easily missed, which could happen if it's closed by default.)

[x] 3. Country disaggregation groupings are displayed as "[CountryName] disaggregations". Note: Because the vast majority of programs are based in a single country, I see no need to over-complicate the UI with nested collapso-expandos. As shown in the mockup, it's fine to display in a single list: Global, Country 1, Country 2

[x] 3.1. When there are multiple country disaggregations, the countries are listed in alphabetical order. To emphasize, Global always comes first, followed by the alphabetized list of country disaggs.

[ ] 4. If there are a more than [n] available disaggregations (what is [n]?), they could be grouped in a scrolling div to prevent the form from growing very long.

[x] 5. There should be no "Select all" option. More is not better in this case, so we shouldn't make it easy to select all of them. (M&E advisors often have to restrain program teams from getting over-ambitious with disaggregated data, committing to more granular data collection than their M&E budget can support.)

[x] 6. A “Preview” of the labels (options) for a disaggregation appear in a tooltip when the user clicks on an info icon.

[x] 7. When the following criteria are met, the checkbox is checked but inactive: There is at least one result submitted for that particular indicator that includes at least one disaggregated value from that specific disaggregation.
Right now, best I can tell, unchecking a disaggregation box has no effect on saved results, so we're doing the right thing on the back end in not permitting you to delete data by unchecking a disaggregation box. It's still problematic to allow that deselection after results are added, because you can easily end up with a situation where results reported for the same indicator have varied categories -- with the indicator form not accurately reflecting what's available. It also seems problematic for IPTT queries that need to figure out which categories to display on a per indicator basis, not a per result basis.

[x] 7.1. When a disaggregation is inactive, include this sentence after the preview list: This disaggregation cannot be unselected, because it was already used in submitted program results.

[x] 7.2. In this scenario, if I go ahead and deselect a disaggregation for an indicator, it no longer appears in any result forms -- new or saved. (Right now, it doesn't appear in new results for the indicator, but continues to appear in saved results.)

[x] 7.3. Conversely, if I add a new disaggregation to an indicator, it should appear in both new and saved results. (Right now, this is the case UNLESS a disaggregated value of any kind has been entered under Custom disaggregations. So if a program forgets to check a disaggregation and starts reporting results, there is no way for them to update existing results later on, with disaggregated data. Note: I filed a separate bug covering this: #2054. It's possible that was overkill and it's a moot point with the way we're restructuring everything)

Random UI thing while you're in the area

[x] 1. Please update the label to sentence case for "Rationale or justification for indicator", immediately above the disaggregation section. (It's currently title case on Prod.)

Housekeeping

[ ]  Unit test/s written wherever applicable
[ ]  i18n tags included wherever applicable
[ ]  Functionality is accessible to people using screen readers

Either we convert the indicator modal to React, which makes this ticket easier (but still not trivial). Or we leave it as non-React, in which case this is even more work.  Either way a high level of effort.

@sanjuroj Assuming you closed the ticket by accident. :-)

Gah, yes, sorry. Second time I've done that.

Moving to acceptance testing despite one unclear spec (number 4) @jennyhavah  - wanna field test and see if you want to pick an "n"?

@camgnostic I added a section 1.3 to address some issues I just noticed just now. The item about Global having some unwanted disagg types was based on seeing this.

I also noticed that the caret is not responding when I open sections -- also captured in 1.3.

Moving to acceptance testing despite one unclear spec (number 4) @jennyhavah - wanna field test and see if you want to pick an "n"?

I can't see a list with lots of options at the moment. For starters, let's go with what's in the mockup -- 8 options visible, any more are enclosed in the enclosing box thingy.
Please note that the font size for the options are smaller than the form field labels. (A little hard to see in the mockup, maybe.)

Requested UI refinement:
Please remove white fill you currently see inside the selection box when interacting with the collapso-expando. Here's how it looks when I click it open:

And when I close it again:

For an example of how I'd like it to look, check out the lack of fill on the program page, like so:

I'd actually prefer there not be a blue selection box either, but only if it doesn't require weird hax or workarounds to accomplish. (Not sure if Paul already attempted this or not.)

Reminder to self: Retest "Can I still view selection as a Low/Med" user after refinements are done.

Some additional UI refinements:

Add space between global and country level disaggregation "clusters".

Add space between Disaggregation and Type fields.

Add border and scrollbar to disaggregation list, so it's more obvious that it's a scrollable list. The original mockup has the thing partially obscured by the tooltip. Here's a mockup without the tooltip so you can see everything more clearly:

Passing to @axoplasm for the last round of UI tweaks

@axoplasm I created a new ticket just for the remaining UI tweaks so none of us have to look at this ticket everrrr again. See #2091.
Moving this one to Passed.


fix(spinlock): comparison of integer expressions of different signedness (IDFGH-13581)

developers who do not use idf.py but prefer classic CMake currently receive a lot of warnings because a comparison in the header file spinlock.h is not correct.
comparison of integer expressions of different signedness: 'esp_cpu_cycle_count_t' {aka 'long unsigned int'} and 'int32_t' {aka 'long int'} [-Wsign-compare]
since it is a header, the gcc option -Wall, which is relevant for the application code, unfortunately also applies to all includes.
this fix casts the signed timeout to the corresponding unsigned type esp_cpu_cycle_count_t for a correct comparison.

sha=681b37266861b2f3d35e50d43b146691cdfe7281


Could not find unf_ext-0.0.8 in any of the sources (but bundle info unf_ext can find it)

Not sure if the problem happened when I finally switched to Xcode 13.0 or a fastlane update, but now I get
Could not find unf_ext-0.0.8 in any of the sources
Run `bundle install` to install missing gems.
[15:44:39]: Exit status: 7

when trying to build for TestFlight. I do get an archive (which I can manually upload).
bundle install doesn’t change anything. As near as I can tell, it’s installed:
  * unf_ext (0.0.8)
	Summary: Unicode Normalization Form support library for CRuby
	Homepage: https://github.com/knu/ruby-unf_ext
	Path: /Users/david/.gem/gems/unf_ext-0.0.8

This is pretty similar to an issue I had a few months ago; not sure how I solved it (likely with a fastlane update).
I’ve attached bundle exec fastlane env output.
env.txt

Through magical means (i.e. I have no idea how, I think I had to reinstall fastlane twice) this is no longer an issue.


Can't fully reproduce with Firefox 85

I can't reproduce my IDs with Firefox 85 (Linux). I don't know if it is related to this, they only mention cross-site supercookie protection.
Chromium 88 seems to be still vulnerable.
Tabs underlined in blue are part of the same container (Firefox extension), others (left to right) are vanilla Firefox, private Firefox and vanilla Chromium.
First try:

After restarting my browsers:

Can't totaly reproduce on Firefox 85.
Works when I check my id a second time. Failed to retrieve it if I close Firefox.
I got the option "Delete cookies and site data when Firefox is closed".

Firefox 85 provides supercookies protections. You have to close and restart the browser.

I can't reproduce my IDs with Firefox 85 (Linux). I don't know if it is related to this, they only mention cross-site supercookie protection.

Seems like the latest version of Firefox is no longer vulnerable to this attack scenario. Good to know, thank you!

I can't reproduce for Firefox 85 (Windows), the demo gives me different id's

Please read here.


Large file conflict in Upload File URL

Hello Mr. Kunal Varma,
The API is working properly but I tried to upload a 68+mb file using Upload file URL, and nothing happens. I can only upload files with a size of maybe 5mb and below.
Is the Upload file by URL where
$asyncJobID = $dropbox->saveUrl("/my-logo.png", 'http://mywebsite.com/wp-content/uploads/2016/06/logo.png');
have file size limit? I mean can we upload like chunked files using this method?
kind regards,

Are you using the checkJobStatus() method, which gives more information on the job?
See Upload File By URL (Check Job Status)

checkJobStatus() returns in_progress but the file is not uploaded to my Dropbox

@briandelacruz @tristanlesaca0 Is it still not getting uploaded to your Dropbox? I'm unable to reproduce this issue. I tried a 300mb+ file and it was uploaded within seconds (after calling the saveUrl() method). This does not seem to be an issue with the package/SDK.

@kunalvarma05  I am uploading large file size image its not uploading .. Whenver i tried with small size its getting uploaded to dropbox
My code is below:-
                     $file = $_FILES['file'];
		    // File Path
		    $fileName = $file['name'];
		    $filePath = $file['tmp_name'];

		    try {
		        // Create Dropbox File from Path
		        $dropboxFile = new DropboxFile($filePath);

		        // Upload the file to Dropbox
		        //$dropbox->createFolder("/My Folder");
		        $uploadedFile = $dropbox->upload($dropboxFile, "/" . $fileName, ['autorename' => true]);

		        // File Uploaded
		        echo $uploadedFile->getPathDisplay();
		    } catch (DropboxClientException $e) {
		        echo $e->getMessage();
		    }

I am alos using Chunked File Upload but that is alos not working. Can you help me


Make first contribution

Add name to contributors
Add your name to the contributors file.
File
CONTRIBUTORS.md
This is beginner friendly issue to get you started in contributing to this repository and your open source journey.
Follow the format of the file to add a new name.

can you please assign this to me under swoc tag

@adi271001 this is just a beginner issue to make your first contribution. You can directly make the changes and make a PR. SWOC tags will be added on Jan 1.

Give me this issue @yagyesh-bobde

@Nitinshukla88 go ahead and create a PR for the same. There is no need for assigning in this issue

Hi. I would like to help with this issue. This is my first attempt.

Hi, I am willing to contribute something here. This is my first time if I contribute to an open source project.


fix(rules): Fixed an issue where 'Plex - Present in amount of other c…

…ollections' wouldn't work as expected

:tada: This PR is included in version 1.7.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Label placement algorithm

I'm trying to understand the label placement algorithm for roads (and want to see if it can be tweaked to produce fewer labels).
Looking at osm2vectortiles.tm2source/data.yml tells me that the final road_label layer is stitched together from SQL views with names like road_label_z8toz10. src/import-sql/layers/road_label.sql then shows that these are generated from the osm_road_geometry table.  But I can't find where/how osm_road_geometry is generated...
https://github.com/osm2vectortiles/osm2vectortiles/search?utf8=✓&q=osm_road_geometry
only shows usages, not creation.  But Google + github search suggests osm_road_geometry  is specific to osm2vectortiles, not imposm3.  I'd be very grateful if anyone could tell me where the osm_road_geometry table comes from, so I can track back the source of the label data.

I'd be very grateful if anyone could tell me where the osm_road_geometry table comes from, so I can >track back the source of the label data.

imposm3 default table name prefix is osm_  see https://imposm.org/docs/imposm3/latest/tutorial.html#writing
so just look for road_geometry

https://github.com/osm2vectortiles/osm2vectortiles/blob/master/src/import-osm/mapping.yml#L513

Thanks!


In IB Controller, Segments disappear under a Translucent Navigationbar

If using Interface Builder for the SJSegmentedViewController, embedded in a translucent NavigationBar, then the segments will scroll under the navigationBar to the top of the screen, when they should stop at the bottom of that translucent navigation bar. If you use an opaque NavigationBar, the segments will show correctly as the main view is not extended under the navbar.
So it seems the control doesn't handle the case of a translucent NavigationBar that extends the view to always make segments visible when scrolling

I tried to look in the code and it seems linked to SplitViewController that I use in my app. I use your control in the SplitViewController, detail view controller embedded inside a NavBarController.
In SJUtil.swift, there is this code in getTopSpacing method :
 if let _ = viewController.splitViewController {
     return 0
 }

If I comment those line, everything works fine and the segments stops scrolling just under the translucent navigationbar.
But those line must exist for a good reason. Could you tell me which case is handled by them ?


MapBox is not loading on Tag: 2.1.0

On this version of superset (2.1.0) MapBox is not loading at all for map charts even if MapBox api key has been set.
But when I tried latest version it stuck on login page but MapBox worked correctly. not only login but also logout is not just working without any error (I have logged in with older version then update to latest to check map box and it worked but thats not the solution)
so is there any solution for that?

I'm unable to validate this as MapBox is working correctly on my install with version 2.1.0.  I'm guessing this is a problem with your MapBox setup (maybe your account, limit, config with Superset).  Especially as you say MapBox worked with older versions of Superset.
There does seem to be a problem with the latest version of Superset, where people are unable to login.  I would suggest not running that version in production as may be unstable, only official Apache releases are meant for production.  There is already an issue for this bug: https://github.com/apache/superset/issues/24579  Maybe something in that thread will help you.


Error with YouTube search/playback using IPV6 Rotation

Description
I have enabled IPV6 rotation using the following config options:
ratelimit:
      ipBlocks: ["2001:41d0:a:7b33:3460:6b77:c818:0/123"]
      #excludedIps: ["..."]
      strategy: "RotateOnBan"
      searchTriggersFail: true
      retryLimit: 0

but YouTube playback is erroring. The same error occurs when performing a search. In fact, I never have a case where Lavalink plays a track without searching first so it is likely just search.
YouTube playback and search works fine without IPV6 rotation enabled. Soundcloud playback works both with IPV6 rotation enabled and with IPV6 rotation disabled.
Logs (Also attached as a file)
spring.log
Version info
Client used: Victoria 5.1.6 (C#)
Output of java -version:
openjdk 11.0.7 2020-04-14
OpenJDK Runtime Environment (build 11.0.7+10-post-Debian-3deb10u1)
OpenJDK 64-Bit Server VM (build 11.0.7+10-post-Debian-3deb10u1, mixed mode, sharing)

Output of java -jar Lavalink.jar --version:
Version:        606fbf3ca475d39e15183fe91ab0149fa17d0fe9-SNAPSHOT
Build:          1061
Build time:     29.05.2020 10:23:19 UTC
Branch          master
Commit:         606fbf3
Commit time:    29.05.2020 10:19:10 UTC
JVM:            11.0.7
Lavaplayer      1.3.49

Soundcloud playback works both with IPV6 rotation enabled and with IPV6 rotation disabled.

We only balance IPs for YouTube.
RotateOnBan doesn't work with a bitmask greater than 63. You're using a bitmask of 123 which is probably too small to be of any help.
First, please re-read the documentation:
https://github.com/Frederikam/Lavalink/blob/master/ROUTEPLANNERS.md#rotateonban-strategy
You may have more luck with the one of the nano strategies if you use a /64, if one is assigned to your OVH machine. Though I have heard that OVH's IPv6 is banned by YouTube anyways.
Closing this as it is not a Lavalink issue. Please seek help in the Lavalink discord channel instead of here.


[demographics-banner] Demographics banner assistive technology a11y

Summary

What was changed:

Patient/person name header level is now defaulted to H2 because H1 likely does not follow the correct order if a title is present. Also added optional prop to update the level as needed called personNameLevel (to match existing personName prop).
DOB and few other abbreviations have long form text for their abbreviations that are read by screen readers. The identifiers prop can consist of additional abbreviations (like MRN) that do not have the ability to add long form text so a new optional prop was added called identifiersLongForm for consumers to add.
Patient "nickname" in parenthesis is created using before and after CSS which could lead screen reader to not fully understand visual presentation, the parenthesis is now part of the "nickname" string when it is present

Why it was changed:
Updates to demographics banner for assistive technologies
Testing

This change was tested using:
Main page examples updated with new prop for identifiers and header levels for patient name are now H2.
Tabbing through the name now reads 'heading level 2  left parens ,nickname> right parens' and MRN reads 'Medical Record Number'

[ ] WDIO
[X] Jest
[X] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[X] UX review
[X] Accessibility review
[ ] Functional review

Additional Details

This PR resolves:
UXPLATFORM-8742

Thank you for contributing to Terra.
@cerner/terra

I validated the Demo Banner with Jaws and VoiceOver. I used the H key (JAWS) and VO+Command+H (VoiceOver) to navigate to the Demo Banner, Patient Name element and heard the screen reader announce the "Doe, Johnathan [Patient Name], Heading Level 2"
I used the down arrow key to navigate to the identifiers and heard the screen readers announce the long form of the identifier, "Medical Record Number"

Demo banner works perfectly with JAWS. When I tested with VoiceOver, it did not read the data in the demo banner when I pressed the down arrow.

Demo banner works perfectly with JAWS. When I tested with VoiceOver, it did not read the data in the demo banner when I pressed the down arrow.

@mjpalazzo We were wondering about the down arrow you have mentioned with VO. Is that something that a VO user would use to go through page content (like inside the demo banner) one element at a time? Is that enabled by  VO+Command+H? Also, would the equivalent in Jaws be CTRL + arrow keys? @smason0 has navigated the banner using Shift to toggle "quick nav" and use left/right arrow keys to read out each group of text which reads like the following
"Johnathon Doe(John)" -> "St. Johns..." -> "25 YearsMaleDate of BirthDOB:May 9, 1993" -> "MRN:12343REA:3JSDA" (using long forms)

Demo banner works perfectly with JAWS. When I tested with VoiceOver, it did not read the data in the demo banner when I pressed the down arrow.

@mjpalazzo We were wondering about the down arrow you have mentioned with VO. Is that something that a VO user would use to go through page content (like inside the demo banner) one element at a time? Is that enabled by VO+Command+H? Also, would the equivalent in Jaws be CTRL + arrow keys? @smason0 has navigated the banner using Shift to toggle "quick nav" and use left/right arrow keys to read out each group of text which reads like the following
"Johnathon Doe(John)" -> "St. Johns..." -> "25 YearsMaleDate of BirthDOB:May 9, 1993" -> "MRN:12343REA:3JSDA" (using long forms)

One small correction, I learned it was was actually "Left Arrow+Right Arrow" that started Quick Nav, after reading the VO control docs. I also have a similar question to Andrew's, specifically if navigating with VO using the left and right arrows is insufficient.

I tried VO+right arrow and VO read all text. Thanks for setting me straight! I am updating the labels to indicate this is approved from my perspective.


tensorflow (cpu only) with mkl is much slower than without mkl!

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Red Hat 4.8.5-4
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 1.8
Python version: 2.7
Bazel version (if compiling from source): 0.10.0
GCC/Compiler version (if compiling from source): 4.8.5
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
I compile tensorflow 1.8 with mkl, and I use c api session to predict wide and deep model.
The Parallel Performance is very slow, avg reponse time is 40 microsecond in 60 qps, cpu load is only 200%
export KMP_BLOCKTIME=0
export KMP_AFFINITY=granularity=fine,verbose,compact,1,0
export OMP_NUM_THREADS=64
export KMP_SETTINGS=1
export MKLDNN_VERBOSE=1
in the opposite， without mkl, The Parallel Performance is ok, avg reponse time is 9 microsecond in 60 qps. I don't know why.Maybe I use wrong!
Describe the expected behavior
Code to reproduce the issue
SessionOptions session_options;
ConfigProto& cp = session_options.config;
cp.set_intra_op_parallelism_threads(64);

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@goldbalance is there as a reason you are using a much older version of Tensorflow? We have most of our optimizations for W&D in 1.14

Also looping in @wei-v-wang and @agramesh1 for settings.

Thanks @penpornk
@goldbalance  As a quick suggestion, is it possible to try setting  export OMP_NUM_THREADS=1 and check the performance? Wide & Deep according to my understand, will benefit more from inter_op setting (graph parallelism) than from intra_op (data parallelism).
We are still identifying the gap with Eigen though.

@goldbalance is there as a reason you are using a much older version of Tensorflow? We have most of our optimizations for W&D in 1.14

thank you for your advise, i will try later. I have to consider upgrade our framework software in order to use 1.14

Also looping in @wei-v-wang and @agramesh1 for settings.

Hi, I ever knew from your reply about using mkl may lead to much more threads running. As decresing inter threads pool size,  My App runtime cannot use more cpu

Thanks @penpornk
@goldbalance As a quick suggestion, is it possible to try setting export OMP_NUM_THREADS=1 and check the performance? Wide & Deep according to my understand, will benefit more from inter_op setting (graph parallelism) than from intra_op (data parallelism).
We are still identifying the gap with Eigen though.

Hi, I ever knew about using mkl may lead to use much more threads running. In 1 qps, mkl is slower then without mkl.  In 60qps, when decreasing inter threads pool size,  My App runtime cannot use more cpu. When adding the size, there are too many threads to running. I try serveral times from 1 to 64, finally, for the best performance, I set just 1 thread pool size  . Therefore, i give up to mkl tf 1.8

Thanks @penpornk
@goldbalance As a quick suggestion, is it possible to try setting export OMP_NUM_THREADS=1 and check the performance? Wide & Deep according to my understand, will benefit more from inter_op setting (graph parallelism) than from intra_op (data parallelism).
We are still identifying the gap with Eigen though.

I wanna to know when mkl-dnn can adapt for eigen thread pool

Thanks @penpornk
@goldbalance As a quick suggestion, is it possible to try setting export OMP_NUM_THREADS=1 and check the performance? Wide & Deep according to my understand, will benefit more from inter_op setting (graph parallelism) than from intra_op (data parallelism).
We are still identifying the gap with Eigen though.

OP_NUM_THREAD=1
SessionOptions session_options;
ConfigProto& cp = session_options.config;
cp.set_inter_op_parallelism_threads(64);
cp.set_intra_op_parallelism_threads(64);
sorry, i met the worst performance using above settting, cpu load also is low

@goldbalance Sorry to hear that. The above setting would generate 64 OMP threads though, eventually. Can you set inter op to e.g. 8 or 16, and then intra_op set to 64 - 8 (or 16)?
Hopefully this does not give you worse performance than worst :)

@goldbalance Sorry to hear that. The above setting would generate 64 OMP threads though, eventually. Can you set inter op to e.g. 8 or 16, and then intra_op set to 64 - 8 (or 16)?
Hopefully this does not give you worse performance than worst :)

export KMP_BLOCKTIME=0
export KMP_AFFINITY=granularity=fine,verbose,compact,1,0
export OMP_NUM_THREADS=1
export KMP_SETTINGS=1
export MKLDNN_VERBOSE=1
export OMP_STACKSIZE=1G
SessionOptions session_options;
ConfigProto& cp = session_options.config;
cp.set_inter_op_parallelism_threads(8);
cp.set_intra_op_parallelism_threads(64);
Sorry, the performance is also low, 100ms
best performance is 25ms, when inter op is 1 and omp thread is setted 64.
without mkl is only 10ms.
I guess that it is blocked, because cpu is low with mkl.

Ok, thank you for trying. Yes, it seems there are fundamental issues that needs to be resolved on our side to bring up the performance.

Hi I am also experiencing slowness with 1.14 with mkl. It is slower than 1.14 without MKL.
I'm using the intelaipg/intel-optimized-tensorflow:1.14.0-mkl-py3 docker image rather than building it myself, running darknet based yolov3.
Any updates regarding these fundamental issues?

@pigubaoza  We are testing and will be rolling out some new features. Regarding your Darknet based YoloV3, have you tuned OMP_NUM_THREADS, inter_op_parallelism_threads and intra_op_parallelism threads, KMP_BLOCKTIME setting?  Also, what CPU are you testing with?
As a last resort, can you try "export TF_DISALBE_MKL=1" and rerun your model and see if it recovers the performance similar to 1.14 w/o MKL?

@pigubaoza We are testing and will be rolling out some new features. Regarding your Darknet based YoloV3, have you tuned OMP_NUM_THREADS, inter_op_parallelism_threads and intra_op_parallelism threads, KMP_BLOCKTIME setting? Also, what CPU are you testing with?
As a last resort, can you try "export TF_DISALBE_MKL=1" and rerun your model and see if it recovers the performance similar to 1.14 w/o MKL?

Thanks wei-v-wang. After some testing I found out OMP_NUM_THREADS and KMP_BLOCKTIME should be as recommended. OMP_NUM_THREADS=4 (I am running an i7 6700K), and KMP_BLOCKTIME=0 (Running CNNs), intra_op_parallelism_threads=4, inter_op_parallelism_threads=2. I previously did not use these values.
Running with TF_DISABLE_MKL=1, my average latency is insignificantly different. ~0.01s
Could it be that because I am running a 6700K, the benefits are not as obvious as running Xeon, or multiple Xeons?
Also, what are these new features that would be rolled out and where can I follow them?


mirb - memory leak in mrb_default_allocf - state.c:60

Hello,
ASAN detects a memory leak in mirb. Testcase:
0if()&.s=()&.g=0

ASAN:
==4271==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0x7f673ac52090 in realloc (/usr/lib/x86_64-linux-gnu/libasan.so.3+0xc2090)
    #1 0x4a4155 in mrb_default_allocf /tmp/mruby/src/state.c:60
    #2 0x4cbe86 in mrb_realloc_simple /tmp/mruby/src/gc.c:203
    #3 0x4cc541 in mrb_realloc /tmp/mruby/src/gc.c:217
    #4 0x40524c in parse_args /tmp/mruby/mrbgems/mruby-bin-mirb/tools/mirb/mirb.c:279
    #5 0x402f59 in main /tmp/mruby/mrbgems/mruby-bin-mirb/tools/mirb/mirb.c:390
    #6 0x7f6739e742b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)

SUMMARY: AddressSanitizer: 8 byte(s) leaked in 1 allocation(s).

I think that occurs mainly because state:60 does not check the realloc's return value and/or because the execution terminate without properly freeing allocated memory by realloc.
void*
mrb_default_allocf(mrb_state *mrb, void *p, size_t size, void *ud)
{
  if (size == 0) {
    free(p);
    return NULL;
  }
  else {
    // If realloc() fails the original block is left untouched; it is not freed or moved.
    return realloc(p, size);
  }
}

Compiled with:
LDFLAGS="-lasan" CC=clang CFLAGS="-fPIC -static -g -ggdb -O0 -fsanitize=address" ./minirake --rakefile=Rakefile default

Test platform:
Linux 4.3.0-1-amd64 #1 SMP Debian 4.3.3-7 (2016-01-19) x86_64 GNU/Linux

GIT SHA:
82bc036a5947a9b672568c664fd0a37071243c91

@matz
Hm, doesn't this fix mean that something like
ary = []
ary << "hallo" # this wants to increase the size of the array, if it fails the array gets freed(?)

can now cause a use-after-free?

@Asmod4n It doesn't happen on my box.

HM, after closer looking, this might look like a false positive.
mrb_realloc tries to alloc more memory than it can get and throws a exception, there should be no leak.


[Error] Unknown tag

Hello,
I tried to use this gem but I got this error:
❯ bundle exec jekyll serve --trace Configuration file: /Users/X/_config.yml             Source: /Users/X        Destination: /Users/X/_site  Incremental build: disabled. Enable with --incremental       Generating...    Liquid Exception: Liquid syntax error (line 3): Unknown tag 'google_photos' in /_layouts/post.html bundler: failed to load command: jekyll (/Users/X/.rvm/gems/ruby-2.6.3/bin/jekyll) Liquid::SyntaxError: Liquid syntax error (line 3): Unknown tag 'google_photos'   /Users/X/.rvm/gems/ruby-2.6.3/gems/liquid-4.0.0/lib/liquid/document.rb:23:in unknown_tag'
/Users/X/.rvm/gems/ruby-2.6.3/gems/liquid-4.0.0/lib/liquid/document.rb:11:in block in parse'   /Users/X/.rvm/gems/ruby-2.6.3/gems/liquid-4.0.0/lib/liquid/block_body.rb:33:in parse'
/Users/X/.rvm/gems/ruby-2.6.3/gems/liquid-4.0.0/lib/liquid/document.rb:10:in parse'   /Users/X/.rvm/gems/ruby-2.6.3/gems/liquid-4.0.0/lib/liquid/document.rb:5:in parse'
/Users/X/.rvm/gems/ruby-2.6.3/gems/liquid-4.0.0/lib/liquid/template.rb:132:in parse'   /Users/X/.rvm/gems/ruby-2.6.3/gems/liquid-4.0.0/lib/liquid/template.rb:116:in parse'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/liquid_renderer/file.rb:13:in block in parse'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/liquid_renderer/file.rb:49:in measure_time'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/liquid_renderer/file.rb:12:in parse'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/renderer.rb:121:in render_liquid'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/renderer.rb:195:in render_layout'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/renderer.rb:158:in place_in_layouts'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/renderer.rb:88:in render_document'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/renderer.rb:62:in run'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:479:in render_regenerated'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:464:in block (2 levels) in render_docs'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:463:in each'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:463:in block in render_docs'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:462:in each_value'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:462:in render_docs'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:191:in render'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/site.rb:71:in process'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/command.rb:28:in process_site'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/commands/build.rb:65:in build'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/commands/build.rb:36:in process'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/commands/serve.rb:93:in block in start'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/commands/serve.rb:93:in each'   /Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/commands/serve.rb:93:in start'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/lib/jekyll/commands/serve.rb:75:in block (2 levels) in init_with_program'   /Users/X/.rvm/gems/ruby-2.6.3/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in block in execute'
/Users/X/.rvm/gems/ruby-2.6.3/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in each'   /Users/X/.rvm/gems/ruby-2.6.3/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in execute'
/Users/X/.rvm/gems/ruby-2.6.3/gems/mercenary-0.3.6/lib/mercenary/program.rb:42:in go'   /Users/X/.rvm/gems/ruby-2.6.3/gems/mercenary-0.3.6/lib/mercenary.rb:19:in program'
/Users/X/.rvm/gems/ruby-2.6.3/gems/jekyll-3.8.5/exe/jekyll:15:in <top (required)>'   /Users/X/.rvm/gems/ruby-2.6.3/bin/jekyll:23:in load'
/Users/X/.rvm/gems/ruby-2.6.3/bin/jekyll:23:in <top (required)>'

Did you install the gem correctly? And added it to _config.yml?

Same issue here... I am trying to use it on github-pages and the error is:
The tag google_photos on line 10 in blah blah page is not a recognized Liquid tag

I included on the _config.yml and also added as a folder on the _plugins :(

I think this is the problem you guys are facing:

https://stackoverflow.com/questions/54499478/my-liquid-tag-is-not-recognized-by-github-pages

I would recommend you to use Gitlab pages. It's simply better :)

I think this is the problem you guys are facing:

https://stackoverflow.com/questions/54499478/my-liquid-tag-is-not-recognized-by-github-pages

Hi @heychirag,
Thanks for your answer. It is not clear how you related both issues.
I mean, how should we solve the issue? I can not find an example in that link.
Thanks in advance.

Hi @sergiotocalini. I am talking about this:

Since GitHub Pages supports only a limited set of Ruby Gems (https://pages.github.com/versions/), the recommended way is to build locally and push the contents of your destination folder (./_site by default) to your remote for hosting.

If the site build fine locally, then the problem lies with Github pages as it doesn't support it. I would strongly recommend you to use Gitlab pages. In my personal opinion it's far superior.
Give me follow up if you were able to get this problem solved.


Massive slow downs with increased CPU usage every HA update?

Been using your beta versions for the last couple of months without any issues, until Home Assistant updates the core, like a couple of days ago. Since then using the Dreame integration beta is impossible, the CPU usage is increased and Home Assistant is unusable until the integration is disabled.
Same thing happened with the last core update.
Is there some breaking change from the core causing this?
Something I can do to mitigate it?

https://community.home-assistant.io/t/psa-2024-5-upgrade-failure-and-dreame-vacuum-custom-integration/724612

Yes, that thread references the issue with 2024.5 which I mentioned. The same thing happens with 2024.7 which released a couple of days ago.
Still no solution for that?


Update putty derivation

Use source code tarball instead of svn.
Add meta information on package.

Rebased to latest code.
Building is running locally to ensure everything is still ok.

Here we go:
# tony at lenovo in ~/repo/perso/nixpkgs on git:update-putty-derivation o [16:20:20]
$ nix-build -A putty
these derivations will be built:
/nix/store/71zjyy5lfbd8mnpxcv2z90qn3ykxs46m-putty-0.64.drv
building path (s) ‘/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64’
unpacking sources
unpacking source archive /nix/store/jmnnxrw3yknx6mvka9ycxbxi4pfc3ixp-putty-0.64.tar.gz
source root is putty-0.64
patching sources
...
glibPreFixupPhase
post-installation fixup
patching ELF executables and libraries in /nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin/puttytel
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin/puttygen
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin/putty
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin/pterm
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin/psftp
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin/pscp
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin/plink
gzipping man pages in /nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64
stripping (with flags -S) in /nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/lib  /nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/bin
strip:/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64/lib/gdk-pixbuf-loaders-2.0/putty-0.64/loaders.cache: File format not recognized
patching script interpreter paths in /nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64
/nix/store/1ckx7qv4vgrysqvyib07izk8s1wabvsf-putty-0.64

A small screenshot:

Built locally, thanks.

A nitpick: when building from a tarball, autotools typically aren't needed.

A nitpick: when building from a tarball, autotools typically aren't needed.

There is no nitpick, there is only the Knowledge :D
I did not know that otherwise I would have removed it.
Thanks for sharing.

Actually, if the code needs to change configure.ac as it does, it will need autotools to regenerate stuff from it :-) I just don't know why the changes were there etc. (and whether you/someone want to spend more time with this expression).


Very large text files shouldn't be just displayed - this kills the browser

This is an example: https://github.com/mattn/go-sqlite3/blob/master/code/sqlite3-binding.c
Instead of just truncating, you can load the file in portions when the user scrolls. This would be better than GitHub.

In my opinion the behaviour of github is not bad because you are doing weird things when you want to view large (source) files. This introduces more complexity in the backend and UI to have chunk file viewing which can break.

I think this issue is solved.


Add installer file associations for Notebook .ipynb files.

Fixes https://github.com/microsoft/azuredatastudio/issues/6680

@abist Did we get rid of the icon for the SQL files types? Not seeing a sql.ico in the install anywhere.

@corivera Not sure, but you're right, I don't see it either.

Should the icon for these file types default to the ADS icon?


ShowDefaultCloseButton doesn't work

ShowDefaultCloseButton doesn't work for me, my Code:
<?xml version="1.0" encoding="utf-8"?>
<Window x:Class="Webstart.Main"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:tabDrag="http://dragablz.net/winfx/xaml/dragablz"
        Title="Webstart" Height="300" Width="600" WindowState="Maximized">
	<DockPanel>
		<Menu DockPanel.Dock="Top">
			<MenuItem Header="Menu">
				<MenuItem Header="Settings" />
			</MenuItem>
		</Menu>

		<tabDrag:TabablzControl x:Name="tabControl" DockPanel.Dock="Bottom" ShowDefaultCloseButton="True">
			<TabItem Header="Test 1" />
			<TabItem Header="Test 2" />
			<TabItem Header="Test 3" />
			<TabItem Header="Test 4" />
		</tabDrag:TabablzControl>
	</DockPanel>
</Window>

Ha, it's in all the styles, except the default style.  Are you intending to use the default style, or one of the others?

I think the MahApps style would be good, how can I use this style (I'm new to WPF)?

Takie a look in this demo solution, there is a MahApps app:
https://github.com/ButchersBoy/DragablzSamplez


Azure SQL Database support

When it'll be supported by Azure SQL Database?

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

ID: e0840e04-b52d-2c5f-197d-26bec998fcaa
Version Independent ID: 40ca3965-4f80-cbb3-839c-3244a536675e
Content: Graph edge constraints - SQL Server
Content Source: docs/relational-databases/tables/graph-edge-constraints.md
Product: sql
Technology: table-view-index
GitHub Login: @shkale-msft
Microsoft Alias: shkale

@shkale-msft
Hello Shreya,
In new GitHub Issue 2353, a customer is asking if/when graph edge constraints are supported on Azure SQL Database (single database - elastic pools) service.
Should this documentation article be updated, to say Yes for Az SQL DB?:

https://docs.microsoft.com/en-us/sql/relational-databases/tables/graph-edge-constraints?view=sql-server-2017

https://github.com/MicrosoftDocs/sql-docs/issues/2353

Currently the above article claims that graph edge constrains are NOT supported on Az SQL DB.    But I did a quickie simplistic run of the example T-SQL, and the example ran complete without errors.
Also, assuming the following article is closely related, it is worth noting that the following article does say Yes for Az SQL DB:

https://docs.microsoft.com/en-us/sql/relational-databases/graphs/sql-graph-overview?view=sql-server-2017

So, are graph edge constraints already supported on Az SQL DB?
If yes, I (or another Writer) will update the documentation accordingly.    I will update the article to choose a different AppliesTo includes/ file, and I will update its OPS Versioning monikerization.
I gather that graph edge constraints are Not supported in either Az SQL DW or ASP-PDW.    But if I am wrong about that, please let me know.
Thank you.
(GeneMi)
CC: @msebolt

Fixed by PR 10743.  I am therefore now Closing this

https://github.com/MicrosoftDocs/sql-docs-pr/pull/10743

Per confirmation email I (GeneMi) received by Shreya Verma:

From: Shreya Verma
Sent: Friday, June 21, 2019 2:51 PM
To: Gene Milener
Subject: RE: [MicrosoftDocs/sql-docs] Azure SQL Database support (#2353)
Hi Gene,
Yes. The feature is now supported on Azure SQL DB. Please feel free to update the documentation.
Graph edge constraints are not supported in Azure SQL DW or PDW.
Thanks,
shreya

CC: @shkale-msft


[Language Server] Incorrect suggestion for completion

Description:
Refer the below recording for your reference.
https://user-images.githubusercontent.com/7523274/117953094-6f7b7d00-b333-11eb-8a03-3b49857efbe0.mp4
Steps to reproduce:
Below is the code snippet. Here, there are suggestions such as "error constructor" and "object constructor" which are invalid.
import ballerina/io;

type InvalidAccountIDError error;

type AccountNotFoundError error;

const ACCOUNT_NOT_FOUND = "AccountNotFound";

const INVALID_ACCOUNT_ID = "InvalidAccountID";

public function main() {
   int negResult = getAccountBalance(-1);
   int invalidResult = getAccountBalance(200);

    transaction {
        int parsedNum = check parse("12");

        // Parsing a random string will return an error. This will
        // cause control to be transferred to the `on fail` clause.
        int parsedStr = check parse("invalid");

        error? res = commit;
    } on fail error e {
        io:println("Error occurred during parsing: ", e.message());
    }
}

function parse(string num) returns int|error {
    return int:fromString(num);
}

function getAccountBalance(int accountID) returns int {

    if accountID >

}

There are use cases where error constructor and object constructor can be used to generate valid expressions in this context.
eg:
function myFunc() {
    int abc = 12;
    var x = object {};
    // x
    if abc > (object {int aa = 12;}).aa {

    }
}

function myFunc2() {
  if error("Custom Error") === error("Custom Error")  {

  }
}

@malinthar Agreed. What we can do from our side is sorting the suggestions to match the LHS of the condition. Let's try and do that.

This completion comes to the top since we do not have a filter text for the snippets and VSCode matches to a space in the label.

This completion comes to the top since we do not have a filter text for the snippets and VSCode matches to a space in the label.

resolved by https://github.com/ballerina-platform/ballerina-lang/pull/31124

Closing based on the previous comment


Removed unneeded sort from SpriteFramesEditor plugin

The anim_names list was just created and is empty.

Cherry picked into 3.0.3


add not exits rule

NotExists method comment should be like Exists comment. e.g: "NotExists checks if given value  doesn't exist in desired table or not."
NotExistsMsg has wrong comment, should be: "NotExistsMsg is default error message format for records that don't exist."

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@edfe7c2). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##             main       #24   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         8
  Lines           ?        71
  Branches        ?         0
========================================
  Hits            ?        71
  Misses          ?         0
  Partials        ?         0

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


Chapter 3: Rerunning the notebook

For some reason when I download and rerun the Chapter 3 notebook locally the results of the SGD classifier are different. I have not modified any of the code. For example, I get different outputs for the decision function and my predictions for some_digit for multiclass classification return 3 not 5. I also see a different PR curve. See the attached image.
I was under the impression that if we used the same random seed the SGD classifier results would be reproducible? Does anyone know why I might be getting different results?

The same is happening to me and I also thought that using the random seed would mitigate this issue. Hopefully someone can elaborate.

Take a look at oreilly.com/catalog/errata.csp?isbn=0636920142874

Take a look at http://oreilly.com/catalog/errata.csp?isbn=0636920142874

I have also come across this problem and cannot see any refference to it in the errata table in the provided link.

If you search for this "Ch 3 (code)" in that link it should take you to an orange segment that has the explanation of this issue. It is essentially to do with newer versions of Sklearn's packages altering the results.

If you search for this "Ch 3 (code)" in that link it should take you to an orange segment that has the explanation of this issue. It is essentially to do with newer versions of Sklearn's packages altering the results.

I see, thanks Dseal95 :)

Hi,
Also see my comment(https://github.com/ageron/handson-ml2/issues/197#issuecomment-653839332)
:)


Device info r2

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: anand-ky:white_check_mark: steveford-obs:white_check_mark: karuna2git:white_check_mark: MFransen69:white_check_mark: cmurug728:white_check_mark: maheshkumar4:x: liginPunooseYou have signed the CLA already but the status is still pending? Let us recheck it.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 20 files pending identification.

Protex Server Path: /home/blackduck/github/ThunderInterfaces/300/rdkcentral/ThunderInterfaces

Commit: dc5536df3f2aa9631e82cf7c12fa2866af077029

Report detail: gist

@liginPunoose: If you are merging to master, you will need Metrological copyrights in headers instead of RDK please.

Martin Hughes, RDK CMF Compliance Team


Style Guides?

Do we have style guides for our repos or ones we prefer?
Google has just shared their style guide for documentation:
https://developers.google.com/style/

We have a TypeScript Style Guide in JupyterLab that we link to in our CONTRIBUTING.md.


WIP: Use Issuer interface to allow for custom issuers

This is a high-level WIP PR to show the general plumbing we'll need to do to start using this interface. I've broken it up into a couple smaller initial PRs for review that we'll need:

[ ] https://github.com/sigstore/fulcio/pull/1005
[ ] #1007

Right now I've only implemented the interface for the Email and Github issuers, I'll have to do all the other ones too. But this should generally demonstrate how it'll all work together.

LGTM with the concept, as we chatted about, server implementers who want to use private implementations of an issuer will instantiate the gRPC server with their own issuer pool.

Will take a look Monday!

@haydentherapper just so you know it's still a WIP, have a couple more things to add!

@haydentherapper this should be RFAL! Thanks :)

@haydentherapper i was planning on deleting everything in a follow-up!


Fix disk metrics on Windows

Performance Counder doesn't provide counter for collecting IOPS per disks.
This change collect metrics using WMI.

LGTM!!!


scribe integration?

Support logging messages directly to scribe.

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.


Update Elrond integration

Update Elrond integration to work with the latest version of Elrond Proxy - slight adjustments with respect to the new format of the API responses.

@maxUo
Could you take a look what's wrong with  Login to DockerHub Registry step?

@maxUo
Could you take a look what's wrong with  Login to DockerHub Registry step?

I'll take a look at this issue soon.

@maxUo please, help us with the pipeline today

@maxUo please, help us with the pipeline today

Check #1173

@miiu96 pls checkout master branch

@miiu96 can you please checkout to the latest master branch, we cannot change your pr. Sorry for some problems on our side before

@miiu96 can you please checkout to the latest master branch, we cannot change your pr. Sorry for some problems on our side before

Hello. I updated branch but i still have some problems... Maybe you can give me more indications please.

@miiu96 can you please checkout to the latest master branch, we cannot change your pr. Sorry for some problems on our side before

Hello. I updated branch but i still have some problems... Maybe you can give me more indications please. Build still failing.

@miiu96 Please update the branch again.

Can you also take a look at GetBlockByNumber()
If you will try to run it - you will constantly have errors.
Fast way to run it:
You need to change config to elrond, and you need to have a docker installed

make start-docker-services
cd cmd/worker
go run main.go

Can you also take a look at GetBlockByNumber()
If you will try to run it - you will constantly have errors.
Fast way to run it:
You need to change config to elrond, and you need to have a docker installed
1. make start-docker-services

2. cd cmd/worker

3. go run main.go

Hello I successfully reproduced this error .... There are to many requests to our database. I'm not sure what is the best fix for this. If you want to work correctly fetchBlocksInterval should be increased.


Feature/linear reorder for residual

Depends on : #112  (first 2 commits)
Add checks in reorder transforms so they only apply changes in linear segments of graphs.
The same checks is used in all the transforms. Check that:

neither of the nodes to swap were merging to branches
there was no fork between those nodes

For this the helper functions is_fork_node and is_join_node were added to ModelWrapper.
Test cases added.

Thanks Tobi, looks good. One thing that came to mind reviewing this was how the is_join_node is currently implemented. In ONNX there are two ways to specify constants, one with initializers (which FINN adopts) and one using Constant nodes to actually produce the constant. The current way of checking the direct predecessors will work when using the initializer method we adopt, but it will fail for any graphs that use Constant nodes. I'll open a separate issue about checking for Constant nodes.


Enhancing FlowCV Page site

Describe the Feature you want to end
would like make the login page responsive and create account page

Expected behavior
it helps in login to site only after giving credentials

Screenshots (optional)

Additional context (optional )
As GSSOC'23 contributor, would like to contribute for this project. Kindly, assign me to this

@MohitGupta121 kindly, assign this to me.


fix: fix common test module package name

Signed-off-by: Omar Al Bastami<EMAIL_ADDRESS>PR summary

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

:tada: This PR is included in version 0.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix lint errors.

Description
There are two lint errors when running foodcritic.  The scheduled_task resource does not specify a default action, and the metadata uses a suggests keyword (FC016 and FC052 respectively).
You can see these errors by running the following command from the cookbook directory:
foodcritic -f any -t any .
Issues Resolved
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Closing this one out since we have the same rules ignored in the .foodcritic file


Crash after few seconds launched the game

Mod Version:
1.12.2
Forge Version:
<IP_ADDRESS>05
Link to crash log (if applicable):
PasteBin
Description:
I don't if is this the right mod but I really don't know how to read crash reports I do some researchment and I think this mod causes the crash.

The crash happens a few seconds later launching the game

Please help me.

This is a weird one.  Shouldn't be happening.  Is it possible to update your Java to the latest 1.8 version?

If it is true, yes

You need to tweak your Minecraft instance config in your launcher to use it.  The pastebin log indicated you are using an older version: Java Version: 1.8.0_51, Oracle Corporation

I'm searching to how to do that but can u tell me how to if u are available ?

it didn't help :/

Can you post the updated log using the update version of Java?

Sorry I didn't literally understand you (My bad english :/ But if u were talking about crash log here is the link:
Minecraft Crash Log PasteBin

Thanks!  According to the log it is still using version 51 (Java Version: 1.8.0_51, Oracle Corporation).

How can I update that ?

Ok, I'm sure about I did it but any changes about crashing

Cool.  The Java version updated and you still have the crash.  I have some ideas as to what it may be.  I am going to make some tweaks.  I should have a new version posted to CurseForge in a day or 2.

So I'll leave up to you thanks for your helps ! ^^

Hey! I have something about crash I thought you should know:
I disabled the mod and entered the game. After that Trollager mob from Primitive mobs mod, Attacked me with throwing cobblestone entity. I think this is the reason of crash

Just like that

I don't think its any other mod causing it.  If anything the mod changes the timing during startup which triggers a bug in DS.  I did find an issue in my code that could lead to this type of exception.  I am making a fix for it.  Unfortunately it may be 2 weeks for me to update the JAR.  A family emergency has come up and I don't want to publish something and walk away for 2 weeks.

Oh I'm sorry about that. Hope nothing is bad. Anyway that's your mod after all. You can update it anytime you want. Thanks for all of your helps.

Pushed v<IP_ADDRESS> to CurseForge with a fix.

Hello again it didn't fixed :/
I use the latest version of forge and the latest version of mod but I get the same issue
https://pastebin.com/xL2jmJvZ

If I pull together a debug JAR would you be willing to test it for me?  This problem defies explanation beyond the fix I made.  (There was a real problem there which would have resulted in a concurrency exception.)

I would but you need to lead me because I don't know the things that much :/

It wouldn't be something to difficult.  It's a matter of downloading a JAR and putting it in your mods folder.  I will see about making one available tomorrow sometime.

ok then no problem with that ^^


Dragging folders to iCloud results in strangeness

I just configured 6.0b2 to add an iCloud account. I then selected all of my folders of subscriptions and dragged them from On My Mac in the sidebar to iCloud in the sidebar.
I was prompted with a large number of "XYZ can not be copied because it no longer exists" dialog boxes with varying numbers of subscriptions in each, which, fair, I haven't cleaned up my subscriptions in literal years. I clicked to acknowledge them all.
The result was only one folder (named after one of my folders), which seems to contain no subscriptions, yet has unread articles in it.

I've expanded the 45 Mine folder in the On My Mac section to show that it does contain discrete feeds there, but the expanded folder in iCloud does not. When I originally dragged the subscription folders, it was collapsed.
The app badge for NNW shows 3754 unread, which is much larger than the sum of what's shown in my sidebar. I haven't quit and restarted the app yet.

Did it ever finish for you?

Closing — we will address this in #2888

@vincode-io It didn't resolve by itself, but after quitting and relaunching, I see the expected folders, with the (very large) unread counts that I would expect if nothing had been read.


Add Glue JobRunSucceeded waiter

This adds a JobRunSucceeded waiter to Glue. Polling delay and max attempts are borrowed from the EMR StepComplete waiter.

Codecov Report

Merging #1801 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1801   +/-   ##
========================================
  Coverage    92.51%   92.51%
========================================
  Files           53       53
  Lines         9958     9958
========================================
  Hits          9213     9213
  Misses         745      745

Continue to review full report at Codecov.

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

@kyleknap @JordonPhillips I noticed you've reviewed some of the more recent PRs that add waiters. Is there anything else you'd like to see here before this is ready to merge?

Hi @jmsanders
Thanks for the request. We are now submitting requests for paginators and waiters upstream so that all AWS SDKs benefit from a consistent implementation. I will pass on the request for this waiter to the service team. We will also update the guidance in the contributing guide to better reflect this policy. I've added an issue to track this here:
aws/aws-sdk#82
Thanks again for bringing it to our attention!


(x+1)^2(x+2-1)^2 cannot be expanded

Seems to be a fault of a low MaxExpansionTermCount default at 50.

Each term of the expanded result should be simplified to avoid this.

We cannot afford calling .Simplify in Expand. We might need to work on some kind of fast Simplify that unites terms by their common multiplier.

Also should fix ((x-1)*(x-2))^3 not expanding.

@Happypig375 is it still an issue?


fix flux_update service

Description:
the flux_update service was not updating the current time properly.

Thanks @balloob! I should've gone with your suggestion right away; it gets the original tests working and fixes the flux_update service.  :tada:

💃


Create new image helper

Image fallthrough code is scattered around the code and it's making the code a bit too complex. Extract that code into a helper.
This would require images to be declared a bit different. Follow the snippet below:
[asset]
  image = "image-path"
  width = ""
  height = ""
  text = ""

Or use assets where there are more than one image of the same type.

I assume weight is only defined for [[assets]].

Also it probably should be:
[asset]
  image = ""
  text = "" <- recommended
  width = "" <- optional
  height = "" <- optional
  url = "" <- optional
  weight = "" <- optional - mostly used for [[assets]]

And
[[assets]]
  image = ""
  text = "" <- recommended
  width = "" <- optional
  height = "" <- optional
  url = "" <- optional
  weight = "" <- optional - mostly used for [[assets]]

Also would we actually use width and height for images within member etc. or would we overwrite that? The only places where we currently allow setting width and height is hero and 404. The rest is usually done responsively without specific settings.

Also we should add documentation on recommended image sizes or at least ratios to the fragment documentation.


Week 6 lecture notes link is dead

This link on README doesn't work https://nbviewer.org/github/Imperial-MATH50003/MATH50003NumericalAnalysis/blob/main/notebooks/DifferentialEquations.ipynb
because that file doesn't exist. The JMD file is there, I could weave it to a notebook and fix this but I don't know if you want the lecture notes to be uploaded yet so I'll just post this as an issue for now.
Nevermind the file does exist - not sure why that link doesn't work then?

Viewing it on github works...  https://github.com/Imperial-MATH50003/MATH50003NumericalAnalysis/blob/main/notebooks/DifferentialEquations.ipynb

seems fixed


Conditional protocol compilation

It would be nice to be able to use feature flags to conditionally compile only a specific set of protocols (and/or games).

This is something that @cainthebest addressed over on the Discord server (invite link here), this could totally be done, can't really say an exact date when this will be added, but for sure after the Q2 project goals are met.

This is something that @cainthebest addressed over on the Discord server (invite link here), this could totally be done, can't really say an exact date when this will be added, but for sure after the Q2 project goals are met.

Whoops, I had already written an implementation before you posted this. If it's too early to merge or isn't what you had in mind that's my bad.

@Douile I haven't yet got a imputation on it as ive been working on a primarily on longer term concept of this lib. feel free to open PRs as they will most likely be discussed before impl.

Took me quite a while to come up with a good and practical way of doing this, i will close this as completed for now as it has been implemented in #227 and will be expanded as more in the beta pre-release milestone.
Example:
https://github.com/gamedig/rust-gamedig/blob/74f8b8bfe42d8719a2361da4b206fbe38bace608/crates/lib/Cargo.toml#L46-L191


Unable to fix all the badges in the board

The board size is less such that we cannot have all the badges visible.
Sticking badges on one another does look appealing.
Appealing for a design change! :)

😂
Will do something for it. To select specific sizes or allowing the user to resize the canvas.


Simplify range over runes.

There's no behavior change. This is equivalent code, but simpler and more efficient.

Thanks for reviewing @rtfb!
(I know it's a trivial change, but I still wanted to go through the review process since this package is a still a dependency of blackfriday v2 for now.)