mj-image, mj-divider rewrite style by class does not work

Hi.
I need divider or image with custom css but neither mj-divider or mj-img does rewrite style with my class.

 tag do it.
<mj-raw>
</mj-raw>
<mj-divider class="hr-type5"/>
<mj-image class="hr-type5"/>
.hr-type5{
    border: 0;
    height: 35px;
    text-align: center;
    background-color: #005151;
    background: url("{{ app.settings.baseUrl }}/media/img/hr-5.png") center no-repeat;
    background-size: contain;
  }

Can you add this feature.
Thanks.

Hello @vdeapps
You can't use any html attributes on mj-element. We're introducing css-class to address this in next 3.3.3 release. You can already test it with the beta release with npm install mjml@beta.
I'm  closing this issue because it's a dup of #480

Hello, I have the exact same problem, 5 years later :D
<mj-divider
      border-color="#CEE1FF"
      border-width="1px"
      css-class="divider"
    ></mj-divider>

.divider{display:block;margin: 20px 0px}
@media (min-width: 480px) {
  .divider{margin: 40px 0px}
}

This has no effect and the class isn't applied.
See:
Is there any ways to fix that?
Thanks.

Thanks for answering.
But yet doing a cmd + f of the class name yields no results? So I'm not sure what's wrong. Also I'm using:  "mjml": "^4.12.0"

Works just fine here : https://mjml.io/try-it-live/Hw-yDalBB

Thanks, so weird, I'm gonna give it an other go! :) :


TabError in ABC/workflow/rules/utils.smk

When I run "snakemake -j1 --use-conda", I get following errors:

TabError in file , line 6:
inconsistent use of tabs and spaces in indentation (, line 6)
File "/maps/projects/ralab_nnfc-AUDIT/people/lpm537/software/sc-E2G/workflow/Snakefile", line 28, in
File "/maps/projects/ralab_nnfc-AUDIT/people/lpm537/software/sc-E2G/workflow/../ABC/workflow/Snakefile", line 11, in
File "/maps/projects/ralab/people/lpm537/software/anaconda3/envs/scE2G_231121/lib/python3.12/tokenize.py", line 541, in _generate_tokens_from_c_tokenizer
File "/maps/projects/ralab/people/lpm537/software/anaconda3/envs/scE2G_231121/lib/python3.12/tokenize.py", line 537, in _generate_tokens_from_c_tokenizer

It seems that the line 5 of ABC/workflow/rules/utils.smk should use tab indentation instead of spaces indentation.
After fixing it, I get a SyntaxWarning:

snakemake -j1 --use-conda
/maps/projects/ralab_nnfc-AUDIT/people/lpm537/software/sc-E2G/ABC/workflow/rules/utils.smk:5: SyntaxWarning: invalid escape sequence '\d'
threshold="\d+.\d+",

Should the line 5 in ABC/workflow/rules/utils.smk be
threshold="\\d+\\.\\d+"

Fixed as part of https://github.com/broadinstitute/ABC-Enhancer-Gene-Prediction/pull/154


Dashboard: Add support for Private stories on Dashboard

Summary
Adds support for Private Stories in Dashboard
If there are private stories show the tab. If there are not, show nothing. Private stories can only be marked in the classic WP list view when you quick edit.

Relevant Technical Choices

To-do

User-facing changes
A new Private tab for private stories in the Dashboard My Stories view.
Testing Instructions

Create a new Story
Visit the Classic WordPress View and Quick Edit a story
Select the Private checkbox on the right.
Refresh the Dashboard and see the Private tab

Fixes #

If there are not, show nothing. Private stories can only be marked in the classic WP list view when you quick edit.

FWIW one can directly mark stories as private in the editor. It's just so that currently they won't be visible in the dashboard, but only the classic list view.

If there are not, show nothing. Private stories can only be marked in the classic WP list view when you quick edit.

FWIW one can directly mark stories as private in the editor. It's just so that currently they won't be visible in the dashboard, but only the classic list view.

I was able to see the private tab, including the count when logged as an author. You need to add a check for read_private_posts capability. This could look like this.
$post_type_object = get_post_type_object( Story_Post_Type::POST_TYPE_SLUG );
$read_private_posts = current_user_can( $post_type_obj->cap->read_private_posts );

We do something similar in get_editor_settings. You can copy and paste much of it.
I would add a similar test for user without that capability. Maybe an e2e test.

I was able to see the private tab, including the count when logged as an author. You need to add a check for read_private_posts capability. This could look like this.
$post_type_object = get_post_type_object( Story_Post_Type::POST_TYPE_SLUG );
$read_private_posts = current_user_can( $post_type_obj->cap->read_private_posts );

We do something similar in get_editor_settings. You can copy and paste much of it.
I would add a similar test for user without that capability. Maybe an e2e test.

@carlos-kelly let us know if you need help with any of the above, happy to assist!

@carlos-kelly let us know if you need help with any of the above, happy to assist!

@swissspidy If one of you wanted to add the PHP code that would be great, it's not my area of expertise and I can manage any front-end changes. As for private being part of all, the current definition is no, but I don't have strong feelings either way.

@swissspidy If one of you wanted to add the PHP code that would be great, it's not my area of expertise and I can manage any front-end changes. As for private being part of all, the current definition is no, but I don't have strong feelings either way.

LGTM! Thanks for the directions on how to make stories private.

I was actually confused by these instructions. What's wrong with just doing that in the editor?

So this:

Private stories can only be marked in the classic WP list view when you quick edit.

Is not really true

LGTM! Thanks for the directions on how to make stories private.

I was actually confused by these instructions. What's wrong with just doing that in the editor?

So this:

Private stories can only be marked in the classic WP list view when you quick edit.

Is not really true

@swissspidy That's fine! That was how it was mentioned in the ticket. I'm not sure everyone knows about that but I can update it!

@swissspidy That's fine! That was how it was mentioned in the ticket. I'm not sure everyone knows about that but I can update it!

@spacedmonkey Good call on the negative case for permissions. Added that test!

@spacedmonkey Good call on the negative case for permissions. Added that test!


[FEAT] Add possibility to customize Authenticating component for oidc-redux as in oidc-context

Issue and Steps to Reproduce
Currently react-oidc-redux only allow to customize 4 components:
notAuthenticated: PropTypes.elementType,
notAuthorized: PropTypes.elementType,
callbackComponentOverride: PropTypes.elementType,
sessionLostComponent: PropTypes.elementType,
and not Authenticating component as in oidc-context
Versions
Latest
Screenshots
Expected
Add authenticating props to Oidc or OidcSecure Component
Actual
Additional Details

Installed packages:

I know it's not recommended to use redux for new project but for old one with redux i thinks it's a good idea to have the same feature

The same happens to me, I need Authenticating component support to change the look and feel of that component in a Redux Project.

it's relatively simple to add a prop to OidcSecure component. If the owner is ok i think i can make a PR after my holidays if no one is available in the meantime :P

yes, go on :)

PR created

Thank you for merging my PR, however there's an issues regarding core package dependencies after your release:
npm ERR! code ETARGET npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting npm ERR! notarget a package version that doesn't exist. npm ERR! notarget  npm ERR! notarget It was specified as a dependency of '@axa-fr/react-oidc-redux'
Would you mind having a look at this issue because all my pipeline is blocked.
Thanks !

thank you @vivo87 for you awesome pull request! It is working now.


Docs for the core folder, and gulp task for jsdoc and docstrap

I've updated the jsdoc config to run on the entire source folder (using npm run docs) as well as fixed the jsdoc errors.


[bug] ClearlyDefinedTests are flaky

Describe the bug
Clearly defined tests are flaky
To Reproduce
See failing CIs: https://github.com/guacsec/guac/actions/runs/11797715031/job/32862288131
Expected behavior
Tests should not flake
Screenshots
If applicable, add screenshots to help explain your problem.
GUAC version
NA
Additional context
Add any other context about the problem here.

TestPurlsLicenseScan is also being skipped as CD fails to respond in time.


Display installed Northstar version in main menu screen

We should somewhat prominently display the current Northstar version in main menu screen. This should help with players that are unsure which version of Northstar they have installed.

Closed as completed with https://github.com/R2Northstar/NorthstarMods/pull/346


share aggregation limit on node

Changes how aggregation limits are shared. Previously they were per
request, now they are per node.
The counters are only updated after an allocation in tantivy, which
should keep the contention risk relatively low.
fix flaky tests

If I understand correctly, this PR makes it so that a new query will fail if running the query would cause the allocated memory over the whole node to exceed aggregation_memory_limit, correct?

Yes, except we need to execute the query and will abort it once it exceeds the memory, as we don't know up front how much memory it needs. (would sounds like we could do that up front)

Can we document that on the docstring of aggregation_memory_limit and in the documentation of Quickwit?

Updated the docs


More redirects are defined when using redirects:force (yml) than --redirects (CLI)

Summary
I see more binding redirects defined when I put the redirects: force flag in paket.dependencies than I do when I use the --redirects flag on the CLI install command. I'm not sure if this is a bug or expected behavior.
I've attached three app config files, once generated with the CLI install command without any flags (PlainInstall_app.config), one with the CLI --redirects flag (CLIRedirects_app.config), and one with the paket.dependencies redirects: force flag (Flag_app.config).
Description
I have a code base across multiple repos, but only want paket to set up binding redirects in one of them. I routinely rebuild paket.lock using the CLI install command, and often forget to use the --redirects flag in the repo that needs it, wasting some time.
I recently found out that I could add a flag that repo's paket.dependencies file that should produce the same behavior as using the --redirects flag on the CLI install command.
Using a merge tool, I see PlainInstall.app.config doesn't have binding redirects set up, as expected. The other two do, but, interestingly, the config file generated when I used the redirects: force flag in paket.dependencies has MORE assembly redirects defined than the one generated using the --redirects flag on the CLI install command.
Repro steps
I'm afraid I don't understand binding redirects, or how paket manages them, well enough to come up with a simple reproduction example, but I have attached three versions of a config file from one of my projects. FYI: I removed a token from them.
Expected behavior
I'd expect the resulting app.config to be identical between using paket install --redirects and  installing without --redirects where the redirects: force flag in paket.dependenncies is set.
Actual behavior
There are more binding redirects defined when I use redirects: force than when I don't have that flag in the yml, and use the --redirects flag on the install command.
Known workarounds
None. Not sure if this is a bug either.
Flag_app.config.txt
PlainInstall_app.config.txt
CLIRedirects_app.config.txt

paket add Microsoft.Toolkit.Uwp.DeveloperTools --version 4.0.0

In paket.dependencies, you can either specify on or force.
On tries to be smart and only adds required redirects.

I would assume that the cli flag is equivalent to redirects: on.
Can you try that?

Yup, looks adding redirects: on to paket.dependencies results in the same behavior as using the --redirects flag on the install command (except for an extra line in paket.lock that reads REDIRECTS: ON)
This is what I need, and the behavior's explained. Closing this issue, thanks!


Two polymer instances causes customElementsRegistry to fail

Steps to Reproduce
Create two polymer projects, one is a library, the other is a polymer app
During runtime polymer will define dom-module as custom element. When an external polymer asset is loaded, another instance of polymer (same version, different locations) will also try to define dom-module as a custom element, which will cause an exception.
Expected Results
Polymer should auto-detect whether another polymer instance was loaded, If so, should console-warn and abort mounting itself.
Actual Results
customElementsRegistry: "dom-module" already defined
Browsers Affected
Tested on crhome
Versions

Polymer: v3.0.2
webcomponents: none, using chrome

Now I See #5095

Yes this will be addressed by #5095. Closing this as duplicate of #4134


[DOCFIX] Update cn docs Presto.md and Spark.md

Cherry-pick and resolve conflicts based on https://github.com/Alluxio/alluxio/pull/8740 which has been merged to 1.8
Update the Chinese documents of Presto and Spark for Alluxio 1.8.
pr-link: Alluxio/alluxio#8740
change-id: cid-57ab23f705e3d29573202b6e4f74c783c5f230aa

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3255/
Test PASSed.

alluxio-bot, merge this please


Enhance our in-house data visualization material

Why
Students lose interest when we use the Kaggle resources to teach them the basics of data visualization
Engineering Spec
Create/enhance our data visualization material. The material should highlight various types of plots and charts to effectively communicate insights from data. It should use matplotlib and seaborn for static plots and plotly for interactive plots. The material should be topical, interactive, and beginner friendly. Use the synthetic dataset we generated for middle and high school students to develop challenges. The challenges need to encourage students to experiment, customize the visualizations, and interpret the results to gain a better understanding of the data.

Here is an example of tasks you might include in the data visualization material. Assuming a synthetic student performance dataset is available, here are several data visualization tasks that students can practice using the student performance dataset. Here are a few examples:

Line Plot:

Visualize the trend of scores over time for a specific subject (e.g., Math, English, Art).
Plot the average score for each subject over time.

Bar Chart:

Create a bar chart to compare the average scores across different subjects.
Generate a bar chart showing the top-performing students in each subject.

Histogram:

Create a histogram to visualize the distribution of scores for a specific subject.
Plot histograms for multiple subjects to compare their score distributions.

Scatter Plot:

Visualize the relationship between scores in different subjects (e.g., Math vs. English).
Create a scatter plot to explore the relationship between scores and exam dates.

Box Plot:

Generate a box plot to compare the score distributions across different subjects.
Create a box plot to identify any outliers in the scores for each subject.

Heatmap:

Create a heatmap to visualize the correlation between scores in different subjects.
Generate a heatmap to explore any patterns or trends in scores across different exam dates.

These tasks will help students practice using various types of plots and charts to effectively communicate insights from the data. They can use popular data visualization libraries like Matplotlib or Seaborn in Python to create static visualizations and plotly for interactive ones. Encourage them to experiment, customize the visualizations, and interpret the results to gain a better understanding of the data.


Fix typo in IP Address page

I guess the title of this PR is self-explanatory. 🙃
@vishr it seems you are a maintainer, but please tell me if someone else may review this PR.

Merged, Thanks


Check expected pex invocation failure reason in tests.

In some tests we just checked that expected invocation failures
happened. These tests continued to pass when the failure reason
changed from the one we expected, to an unrelated one having to
do with manylinux tag incompatibilities.
This change makes the tests check the invocation's stderr for the
expected failure output.

Note - the tests are now expected to fail. Will add a commit that fixes or bypasses them before sending for review.


Add Azure Application Insight Support

Description
Add's support for Azure Application Insights Monitoring.
Still requires a little bit of Documentation
https://docs.microsoft.com/en-us/azure/azure-monitor/learn/nodejs-quick-start#configure-app-insights-sdk
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Yes
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have updated changelog.md with changes made

I'm going to do a little further testing with one of my Prod instances before merging this one in ;-)

Seems to be working fine. Need to update documentation for this feature

Ooo nice - the latest change lets you view stats on your database as well 👍

Merging, i'll do any further tweaks or additions in another PR 👍


fix: F.vad returned is not correct when input is zeroed audio

fix #3668
Return empty result when not has_triggered.
Zeroed wav audio added to unit test.

Hi @mthrok , I have implemented the changes, please take a look😊

@mthrok It seems build conda failed. How can I fix this?

Hi, I think the current way to handle not has_triggered is wrong. It returns an empty tensor even if all the samples are non-zero. Here is a similar reproducible example to the one in https://github.com/pytorch/audio/issues/3668:
import torch
import torchaudio.functional as F
x = 10*torch.ones([16000])
y = F.vad(x, 16000)
print(f"input_size={x.size()}, ysize={y.size()}")

output:
input_size=torch.Size([16000]), ysize=torch.Size([0])


How to configure pug plugin in config file ?

I'm using the code
module.exports = { ... templates: { joinTo: 'javascripts/app.js' }, },
plugins: { pugjs: { staticBasedir: 'app/static_pug', staticPretty: true, compileDebug: true, }, ....
But I don't get anything as result. I was trying to change joinTo param, but it didn't work for me. Do you have any thinking?

@sera2007 not sure what you're talking about. Try this:
module.exports = {
  files: {
    templates: { joinTo: 'javascripts/app.js' },
  },
  plugins: {
    pugjs: {
      // you options here
    }
  }
}

@sera2007 , for pugjs-brunch the key is pug.
staticBasedir is the "assets" directory of Brunch (any with "assets/" by default). Pug templates here will be converted to static html files in the "public" directory, so don't use staticBasedir unless you have special needs.
For precompilation you must to use 2 options: preCompile and preCompilePattern.
Example:
module.exports = {
  files: {
    templates: { joinTo: 'javascripts/app.js' },
  },
  plugins: {
    pug: {
      preCompile: true,
      preCompilePattern: /\.html\.pug$/
    }
  }
}

The above settings with generate functions that returns plain HTML strings for all files ending with ".html.pug" wherever they are --inside the "app" directory, except in "assets/". Example:
const tmpl = require('./views/content.html.pug')
$('#container').html(tmpl)


Bug report link error

Listed as https://github.com/InvenTree/issues in version info but should be https://github.com/inventree/InvenTree/issues

Thanks for pointing this out!


make undistort_image more convenient to use during pano stitching

I will suggest documenting the extra options at https://github.com/nasa/astrobee/blob/develop/localization/camera/readme.md. It would be nice to also add a sample invocation which makes use of these new options, and likely saying a sentence about the motivation, so that in the future a reader could reproduce your use case.
BTW, the doc for this tool does not seem accessible from the auto-generated documentation at https://nasa.github.io/astrobee/html/index.html. I don't know how to fix that, and doing this is beyond the scope of your work now, but at some point we need to ensure all readme files are reachable from the table of contents.

this? https://nasa.github.io/astrobee/html/camera.html

Marina, you are right. I found this myself as well.
This tool's documentation is accessible from https://nasa.github.io/astrobee/html/camera.html. I will now make a very quick edit to that doc page (in the dev branch)  to make it show up prettier in the html.

As before, I suggest documenting these new options. If that is done, this can be merged.

Thanks for the feedback! I updated the documentation.
I very much appreciate your attention to this, since your mapping tools are necessarily quite complex and therefore are not always easy for others to operate. But as a general point, I think for readme files, we're probably better off putting more tutorial-style information and tips on when to use the options vs. in some cases we are essentially copy-pasting the output of cmd --help... with the latter I worry that it is redundant and can easily get out of sync. (Perhaps there is an auto-doc tool that could magically address this.)

vs. in some cases we are essentially copy-pasting the output of cmd --help

We need all of that. Good documentation has examples of usage. It also has a discussion for things where user judgement is needed, where sample invocations are not enough. And we need a full list of options.
I approve the merge. You are welcome to pull it in.


Update Makefile, service definition, and README to point to container and correct instructions

The current files were copied from Home Assistant's example repo.  Please replace the functionality that points to the home assistant container, replace any parameters, and ensure that both the instructions and the makefile run all tasks properly.

Need to re-test the instructions and confirm installation works in the community lab instance.


25 update sample project

Updated the sample project so that it works on more recent versions of Django.

There may still be some deprecations in the sample project.  I mostly focused on getting it to work again.

:closed_umbrella:


ProcParts inside a fairing register with a min 1m diameter

Seen with v6.0_RC1 (#fb0326f), don't know how long it's been there
https://discord.com/channels/319857228905447436/620690446540341261/845710042883948574
Place an interstage base with 1 side part, give the base a top & bottom diameter < 1m (say 0.5m). Then place a procparts tank on the base's middle node, and change its diameter.
Expected: with a tank diameter <0.5m, the fairing is a straight cylinder. Tank diameters > 0.5m make the fairing swell out
Actual: fairing diameter swells out to ~1m diameter even with a smaller tank. >1m tanks inflate the fairing as expected
Extra note: the fairing doesn't act as if the full height of the tank is 1m, only its middle(ish) section

The surface attachment collider on the ProcPart is, for whatever reason, always positioned at z=0.5 and in the middle of the part.  Presumably this is where it starts on an unmodified ProcPart cylinder.  So when PF tries to scan everything and determine what is inside, this is identified as something to be kept within the fairing.  Thus the minimum radius is found to be 0.5, and it's "correctly" placed at the center.


Implementing a label system

We are going to start using GitHub labels to more effectively maintain this repository. It is important that the labels:

are maximally useful
are Easily understood without reference
clearly say where it is and what it needs

Here are my proposals for the labels:

stage: idea

needs assignments

stage: skeleton
stage: research
stage: writing

needs rewrite

stage: review

needs content review
needs tech review
needs cohesion review
needs mech review
needs sign off

stage: ready

The stages will also conform to the Waffle board at https://waffle.io/ipfs/blog.
I propose that all 'needs review' are shades of blue, all 'stages' are shades of yellow, except for 'stage: ready' (green) and 'stage: idea' (pink).
If this sounds good, I'll go ahead and create them and put them on the posts currently in the system. These are:

Static Web Publishing #31 ----- "stage: writing" "needs rewrite" owner: @RichardLitt
ipfs.pics blog post #15 ----- "stage: review" "needs sign off" reviewer: @jbenet
quick install of ipfs #8 ----- "stage: writing" "needs rewrite" owner: @RichardLitt

These sound good to me!

I think the natural flow is for once an idea has been assigned it moves from "needs assigned" to  "stage: skeleton"
@aistich Can you go through the open issues and start tagging appropriately?
Let's also get the blog ideas that have already been assigned written out as issues and get the tagging going so that we can see what's on the board.

@em_ly I can start tagging on Tuesday-
However, I want to make sure there is a pick-up indicator.
There is a definite stage between the idea being assigned and the skeleton
being actually worked on.
Having that tag makes it easier to see if someone has let an article drop
in the early stages.
It may also be a good idea to create a tag for people who need help on
their post like an "help needed" label for use if someone no longer feels
able to write their piece or gets stuck at a stage before they are ready
for technical review.
On Thu, Jun 30, 2016 at 12:31 AM, em-ly<EMAIL_ADDRESS>wrote:

I think the natural flow is for once an idea has been assigned it moves
from "needs assigned" to "stage: skeleton"
@aistich https://github.com/aistich Can you go through the open issues
and start tagging appropriately?
Let's also get the blog ideas that have already been assigned written out
as issues and get the tagging going so that we can see what's on the board.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ipfs/blog/issues/57#issuecomment-229581776, or mute
the thread
https://github.com/notifications/unsubscribe/ASpYnsI-B0AbQsPPrVLMO0NrrUNaaCvkks5qQ3DRgaJpZM4JBIe1
.

--
Ariel Stich
Operations Manager
Protocol Labs

Maybe add one that indicated the idea has been assigned, but the owner has not begun work?
There is a definite stage between the idea being assigned and the skeleton
being actually worked on.
Having that tag makes it easier to see if someone has let an article drop
in the early stages.

I'm not sure there needs to be this one; you can assign people directly, and from then on, that is their responsibility. How they juggle that isn't something we should label, I don't think? Just like writing and  research, this is going to be a process. If someone hasn't committed anything to the blog within a few days, it would be good to ping them - you can tell this from the history in the issue // PR, I think.

@aistich Can you go through the open issues and start tagging appropriately?

None of the open issues are actually topic related. We should have a topic label, maybe, to show that it goes with these. The other issues are currently all meta or bugs in the actual blog mechanics.

Let's also get the blog ideas that have already been assigned written out as issues and get the tagging going so that we can see what's on the board.

This sounds good to me.

It may also be a good idea to create a tag for people who need help on
their post like an "help needed" label for use if someone no longer feels
able to write their piece or gets stuck at a stage before they are ready
for technical review.

Like all of the IPFS repositories, this one has a help wanted tag. This should be used in those cases. The difficulty: easy|medium|hard options are also here.

I've created all of the labels, and implemented it on Waffle.
@aistich - when an article is started as a PR, it will move automatically to Skeleton. You should be able to use this as an indicator that the work has begun or not.
Regarding opening issues for the current blog posts: I don't think this is unnecessary. A blog post will remain as an open PR until it is published - because of this, might be a good idea to talk about the entire process in the PR. I think having an issue and a PR for each post might get a bit confusing; the initial issue should perhaps be closed when a PR is opened?

So, the agreed upon flow was that each post should start as an "idea" issue, to allow discussion before it becomes a blog post. As it moves from IDEA to SKELETON, the issue will be closed, linking to a correlating PR opened for that blog post.

Also, I would say this was successfully completed. 😃 Thanks, @RichardLitt! Feel free to close.

Reopened because I think this should be closed when the workflow in this issue is written in a Markdown.  Good information here.

Workflow is in https://github.com/ipfs/blog/blob/master/PIPELINE.md.


R119 affinity

openmp-related perf tweaks

Refer to this link for build results (access rights to CI server needed):
/job/nd4j_PR/115/Build result: ABORTED[...truncated 2365 lines...][INFO] [INFO] --- javacpp:1.2.3:build (process-classes) @ nd4j-native ---[INFO] Detected platform "linux-x86_64"[INFO] Building for platform "linux-x86_64"[INFO] Generating /jenkins/workspace/nd4j_PR/nd4j-backends/nd4j-backend-impls/nd4j-native/target/classes/org/nd4j/nativeblas/linux-x86_64/jnind4j.cpp[INFO] Compiling /jenkins/workspace/nd4j_PR/nd4j-backends/nd4j-backend-impls/nd4j-native/target/classes/org/nd4j/nativeblas/linux-x86_64/libjnind4j.so[INFO] g++ -I/jenkins/workspace/libnd4j//blas -I/jenkins/workspace/libnd4j//include -I/usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>-3.b13.el7_2.x86_64/include -I/usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>-3.b13.el7_2.x86_64/include/linux /jenkins/workspace/nd4j_PR/nd4j-backends/nd4j-backend-impls/nd4j-native/target/classes/org/nd4j/nativeblas/linux-x86_64/jnind4j.cpp -std=c++11 -std=c++11 -Wl,-rpath,$ORIGIN/ -Wl,-z,noexecstack -Wl,-Bsymbolic -march=x86-64 -m64 -Wall -O3 -fPIC -shared -s -o libjnind4j.so -L/jenkins/workspace/libnd4j//blasbuild/cpu/blas -Wl,-rpath,/jenkins/workspace/libnd4j//blasbuild/cpu/blas -lnd4j In file included from /jenkins/workspace/libnd4j//blas/NativeBlas.h:20:0,                 from /jenkins/workspace/nd4j_PR/nd4j-backends/nd4j-backend-impls/nd4j-native/target/classes/org/nd4j/nativeblas/linux-x86_64/jnind4j.cpp:98:/jenkins/workspace/libnd4j//include/types/float16.h: In function 'float cpu_half2float(half)':/jenkins/workspace/libnd4j//include/types/float16.h:64:36: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]     return ((float)((void*)&temp));                                    ^/jenkins/workspace/libnd4j//include/types/float16.h: In function 'half cpu_float2half_rn(float)':/jenkins/workspace/libnd4j//include/types/float16.h:72:37: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]     unsigned x = ((int)(void*)(&f));                                     ^/jenkins/workspace/nd4j_PR/nd4j-backends/nd4j-backend-impls/nd4j-native/target/classes/org/nd4j/nativeblas/linux-x86_64/jnind4j.cpp: In function 'void Java_org_nd4j_nativeblas_Nd4jBlas_setMaxThreads(JNIEnv*, jobject, jint)':/jenkins/workspace/nd4j_PR/nd4j-backends/nd4j-backend-impls/nd4j-native/target/classes/org/nd4j/nativeblas/linux-x86_64/jnind4j.cpp:1481:14: error: 'class Nd4jBlas' has no member named 'setMaxThreads'         ptr->setMaxThreads(arg0);              ^ERROR: Maven JVM terminated unexpectedly with exit code 1Adding one-line test results to commit status...Setting status of 04be39020e8432f509ff50c993a1d650c7aa39bc to FAILURE with url http://ec2-54-191-187-162.us-west-2.compute.amazonaws.com:8080/job/nd4j_PR/115/ and message: 'Build finished. 1 tests run, 1 skipped, 0 failed.'


Add metadata to created namespace

Currently createNamespace doesn't set any metadata. Please add the following:
apiVersion: v1
kind: namespace
metadata:
  name: foo
  labels:
    app: HERE
    chart: HERE
    heritage: Tiller
    release: HERE

To do this, would it be ok to pass the entire Release object instead of just the Release.Namespace at
//  cmd/rudder/rudder.go:94
err := kubeClient.Create(in.Release.Namespace, b, 500, false)

We can propogate this down to the ensureNamespace() method.
// pkg/kube/client.go:95
if err := ensureNamespace(client, namespace); err != nil {

And then createNamespace can use the Release information to create the label map and create the appropriate ObjectMeta.
Is the approach fine?

I wish to add custom labels to namespaces, or maybe the default best practices one will suffice as well. Currently don't know of a way to do it with namespace being created by helm automatically i.e. using --namespace. Nor I believe I can add a namespace manifest like here to helm charts. My use-case is this: https://github.com/coreos/prometheus-operator/issues/1331#issuecomment-388961725. Whereby namespaces are being selected using LabelSelector and we are unable to because of missing labels on namespaces.

+1 to this. I want to use a ValidatingWebhookConfiguration and need to exclude validation on the namespace the release is deployed into for bootstrapping purposes.
The namespaceSelector only supports selecting based on labels, so I need to somehow label the release namespace.

Custom labels will be very helpful for labelling namespaces for istio sidecar injection using the mutating webhook admission controller as well.

I had the same issue, I was able to get things to work using a chart pre-install hook e.g. https://gist.github.com/kvudata/12ba57ae1e7f01799aaa7f36350a9b2e.

@kvudata Does that work if the namespace is being created by helm, or only if the namespace already exists when you run the helm install ?

@sudermanjr, when I tested it worked in both scenarios (whether the namespace exists already or not).

Is there a way to add your own labels to a namespace helm is creating for you?

@jordan-da See the gist from kvudata above.

@sudermanjr yea, I saw it (and commented on it).. I am just so surprised! Seems like a worthy one off thing to have.
I ended up creating my namespace before running helm, with the labels I required. Speaking of one offs.

It's a workaround, but using helmfile allows you to define hooks that are executed before a chart is deployed, see for example roboll/helmfile#538 for the use case defined here…

Another workaround would be to use Reckoner hooks as well.
https://github.com/reactiveops/reckoner#hooks
(Full disclosure, I work for the company that wrote Reckoner)

I'm currently trying to work around this issue by deploying Namespaces via magic namespace chart (with Tiller and Service Accounts disabled).

Closing as a wontfix. The namespace creation code was removed from Helm 3 to align with Kubernetes best practices. See #5753 for a more detailed explanation. Thanks!

Hi all! Please reopen due to #7648

We re-introduced —create-namespace with the caveat that we will not be planning to support this use case due to the complexity of manipulating the namespace.
If you want to apply custom labels or annotations to the namespace, please use kubectl create namespace instead.


Support reading/writing 16-bit images

Problem: Stitching 16-bit TIFF images results in an 8-bit output image.
If stitching is the last step of an editing workflow before saving a file for web/print, then that's OK because 8-bit is enough for web/print. But it means that stitched images lose a lot of the data that is useful during editing. If stitching supported exporting 16-bit images, and maintained the color data of the input images (to the best of its ability), then it could be used as the first step of an editing workflow. As you can imagine, editing a whole stitched image is easier than editing individual pieces of it. :-)
Currently, stitching uses OpenCV's imread() to read input image files.

By default cv2.imread will convert a 16 bit, three channel image in a.tif to 8 bit as shown in the question.

Source: https://stackoverflow.com/a/18461475 (I can't find this officially documented in https://docs.opencv.org/4.10.0/d4/da8/group__imgcodecs.html, but it appears to be true.)
This means that all processing is performed on 8-bit image data, which is then passed to imwrite(), so the output image is 8-bit.
The imread() function accepts an IMREAD_UNCHANGED flag, which ensures that it is not converted to 8-bit: https://docs.opencv.org/4.10.0/d8/d6a/group__imgcodecs__flags.html#gga61d9b0126a3e57d9277ac48327799c80aeddd67043ed0df14f9d9a4e66d2b0708
However, simply adding that flag to stitching's imread() call (by changing line 113 of images.py from img = cv.imread(img_name) to img = cv.imread(img_name, cv.IMREAD_UNCHANGED)`) causes an error.
The error depends on the --detector option that is used:
orb:
stitching.stitching_error.StitchingError: No match exceeds the given confidence threshold.
                                  If yes, you might want to lower the
                                 'confidence_threshold' or try another
                                 'detector'.

sift:
cv2.error: OpenCV(4.10.0) /io/opencv/modules/features2d/src/sift.dispatch.cpp:512: error: (-5:Bad argument) image is empty or has incorrect depth (!=CV_8U) in function 'detectAndCompute'

brisk:
cv2.error: OpenCV(4.10.0) /io/opencv/modules/imgproc/src/color.simd_helpers.hpp:92: error: (-2:Unspecified error) in function 'cv::impl::{anonymous}::CvtHelper<VScn, VDcn, VDepth, sizePolicy>::CvtHelper(cv::InputArray, cv::OutputArray, int) [with VScn = cv::impl::{anonymous}::Set<3, 4>; VDcn = cv::impl::{anonymous}::Set<1>; VDepth = cv::impl::{anonymous}::Set<0, 2, 5>; cv::impl::{anonymous}::SizePolicy sizePolicy = cv::impl::<unnamed>::NONE; cv::InputArray = const cv::_InputArray&; cv::OutputArray = const cv::_OutputArray&]'
> Invalid number of channels in input image:
>     'VScn::contains(scn)'
> where
>     'scn' is 1

akaze:
cv2.error: OpenCV(4.10.0) /io/opencv/modules/stitching/src/exposure_compensate.cpp:564: error: (-215:Assertion failed) _image.type() == CV_8UC3 in function 'apply'

(Originally discovered in this discussion: https://github.com/OpenStitching/stitching/discussions/224)

This provides a good explanation why 16-bit is important for editing: https://photographylife.com/8-bit-vs-16-bit-images

Even though 16 bits may sound like overkill, we saw here that it finds its use in post-processing images. But 8-bit per channel images take up much less file space, so it’s worth exporting your images, especially for the web, to 8-bit per channel to save space.

Thanks for reporting! This might be a wontfix since I'm not sure if this can be solved within stitching alone but would need changes in the stitching module of opencv as well..


Add a bypass level for the moderator filters

Add the ability to manually set a bypass level for pajbot. Eg I set supibot to lvl 250 and I set the bypass level for moderator modules to 250 (set on a per module basis). So in this case, supibot would bypass multiple filters (depending how it's setup per module):

Ascii
banphrase
link checker
max message length
case checker
paid timeout
repetitive spam
mass ping protection

So you want to be able to set them all at once?

@RAnders00 No, set them on a per module basis

From what I see these modules already have a configurable bypass level:

ascii
maximum message length
case checker
paid timeout
repetetive spam
mass ping protection

and these modules are lacking them:

banphrase
link checker

Do i understand that correctly?

Yeah looks like it. I personally won't use the banphrase level bypass (I don't think anyone should), but if it's easy enough to integrate, then feel free to


Set Working Directory does not work

Intermittently, setting the working directory does not work (neither via keyboard shortcut or using the menu item).
Error message in console:
no command for selector: noop:

What keyboard shortcut are you using? There is default keybind for setting working directory at all. By the way, did you install SendCode?

{         "keys": ["super+shift+h"], "command": "send_code",         "args": {"cmd": "setwd(\"$file_path\")"},         "context": [             { "key": "selector", "operator": "equal", "operand": "source.r" }         ]     },
However, it does not depend on the shortcut (using the menu entry produces the same results)
Latest version of SendCode is installed.
Note that the problem is temporary, it is usually fixed by restarting ST.

It is weird..

By the way, are you using R-Box or R-IDE? (I could see that LSP is running)

R-Box.
(Happy to try R-IDE)


[Kafka] Add dashboard url feature and fix minor styling issues

Hey, I just made a Pull Request!

Added a link to a dashboard, which can be defined over an annotation or config
Fixed style for multiple clusters. There was no spacing between tables.
Potential fix for https://github.com/backstage/backstage/issues/11077

:heavy_check_mark: Checklist

[X] A changeset describing the change and affected packages. (more info)
[X] Added or updated documentation
[X] Tests for new functionality and regression tests for bug fixes
[X] Screenshots attached (for UI changes)
[X] All your commits have a Signed-off-by line in the message. (more info)

Will pick up work after my vacation 😉

To be clear, where is the link to the dashboard in the screen at the top? Is it clicking on the cluster name?

Yes exactly.

Ah right! I think that makes sense for the actual cluster dashboards. As far as configuring the dashboard for consumer entities, we have a single dashboard that is filterable by consumer group, topic etc. I can see myself providing those pre-configured URLs rather than a generic cluster wide dashboard. For that use case, I don't imagine it would be clear what that link does compared to say; an explicit "dashboard" link 🙂

Aside from that, this kind of nicely solves an issue we had with the Datadog plugin where it's really annoying to provide pre-filtered queries but we only really want to do that for Kafka consumers so being able to provide a dashboard URL on a per-entity/consumer basis is neat

@marcus-crane nice to hear that this will improve things on your end.
Would an annotation like cluster-name/consumer-group-name/dashboard-url help?
This way you can provide a specific dashboard url per consumer group like cluster1/consumer-group-1/https://dashboard.com?consumergroup=consumer-group-1 and therefore reuse the dashboard with filters per consumer group.

@marcus-crane nice to hear that this will improve things on your end.
Would an annotation like cluster-name/consumer-group-name/dashboard-url help? This way you can provide a specific dashboard url per consumer group like cluster1/consumer-group-1/https://dashboard.com?consumergroup=consumer-group-1 and therefore reuse the dashboard with filters per consumer group.

@marcus-crane nice to hear that this will improve things on your end.
Would an annotation like cluster-name/consumer-group-name/dashboard-url help? This way you can provide a specific dashboard url per consumer group like cluster1/consumer-group-1/https://dashboard.com?consumergroup=consumer-group-1 and therefore reuse the dashboard with filters per consumer group.

Sure, that sounds like a reasonable implementation 🙂 We have a few services that read from multiple clusters at the same time so being able to attach multiple dashboards (from arbitrary sources) sounds great
Just to double check, is your example intended to be namespaced ie;
metadata:
  annotations:
    kafka.io/cluster1/consumer-group-1/dashboard-url: https://dashboard.com?consumergroup=consumer-group-1

or just a standalone annotation like
metadata:
  annotations:
    cluster1/consumer-group-1/dashboard-url: https://dashboard.com?consumergroup=consumer-group-1

Not that it would happen for any use case I have, but I can imagine someone unfamiliar with Kafka may be uncertain what cluster refers to example since it could be a Kubernetes cluster, Kafka cluster, ECS cluster etc etc.
Consumer group is a big hint of course but I suppose not everyone touching a catalog would necessarily recognise that.

The annotation would look like this
metadata:
  annotations:
    kafka.apache.org/dashboard-urls: cluster1/consumer-group-1/https://dashboard.com?consumergroup=consumer-group-1

This would be more aligned with the current annotation kafka.apache.org/consumer-groups: cluster1/consumer-group-1

@dweber019 Ah, gotcha! I mean, if I'm honest, it looks a little uglyto read but given the limitations that annotations can't be a map, it seems perfectly reasonble. It's designed for a machine to parse after all 😉

Thanks for your efforts with this PR and I'm looking forward to taking it for a spin 🙂

Just updated PR to support consumer groups.
@jhaals could you have a look again, thx.

OK let's get this in. I'm wondering if there's a nicer way to do this and move some of this logic to a backend plugin or something. But I think we can iterate on that later down the line 👍


Performance-Customized ETW With Array Parameters

Aside: I'm really looking for how to deal with this for EventProvider+manifests, but it looks like EventSource and EventProvider work the same way in this specific scenario.
I can't find documentation anywhere (GitHub, MSDN, the works) on this. How would you approach implementing this method?
[Event]
public void Foo(string[] vals) { ... }

Or this one?
[Event]
public void Foo(int[] vals) { ... }

The ETW examples don't demonstrate this scenario. The string list necessitates the following "classic" manifest, for example:
<template tid="t23EFDE6D13D62A45">
    <data inType="win:UInt16" name="vals.Count" />
    <struct name="vals" count="vals.Count">
        <data inType="win:UInt16" name="vals.Length" />
        <data inType="win:UnicodeString" length="vals.Length" name="vals" />
    </struct>
</template>

This means that the buffer has to effectively follow that schema. My best guess is the following:
// 0: vals.Count
// 1: vals
EventData* descr = stackalloc EventData[2];

var valsLen = Math.Min(vals.Length, 20);
EventData* valsDescr = stackalloc EventData[valsLen * 2];

descr[0].DataPointer = new IntPtr(checked((ushort)vals.Length));
descr[0].Size = sizeof(ushort);

try
{
    // n + 0: val.Length
    // n + 1: val
    var valsLength = 0;
    for (var i = 0; i < vals.Length; i++)
    {
        var val = vals[i];
        if (val != null)
        {
            var ii = i * 2;
            var ij = ii + 1;

            valsDescr[ij].DataPointer = Marshal.StringToHGlobalUni(val);
            valsDescr[ij].Size = (val.Length + 1) * sizeof(char);

            valsDescr[ii].DataPointer = new IntPtr(checked((ushort)valsDescr[ij].Size));
            valsDescr[ii].Size = sizeof(ushort);

            valsLength += valsDescr[ij].Size;
        }
    }

    descr[1].DataPointer = new IntPtr(valsDescr);
    descr[1].Size = valsLength;

    // Write Event
}
finally
{
    for (var i = 1; i < vals.Length; i++)
    {
        var ptr = valsDescr[1 + i * 2].DataPointer;
        if (ptr != IntPtr.Zero) Marshal.FreeHGlobal(ptr);
    }
}

Any chance we could have the examples expanded to include int[] (fixed-length item arrays) and string[] (variable-length item arrays)?

Unrelated to your question, but pinning the strings instead of copying them might be more performant.

@weltkante I tried, but the problem is that you can't used fixed with ref types - the automagic string-to-char* seems to break as soon as you involve arrays.

@jcdickinson You can pin manually using GCHandle.Alloc and GCHandle.AddrOfPinnedObject on each string instead of the fixed language statement. Its a struct so no actual allocations happen and you can optimize away the array-of-GCHandles too:
private unsafe void Foo(string[] vals)
{
    var handles = stackalloc GCHandle[vals.Length];
    try
    {
        for (int i = 0; i < vals.Length; i++)
        {
            handles[i] = GCHandle.Alloc(vals[i], GCHandleType.Pinned);
            var pointerToFirstCharOfString = handles[i].AddrOfPinnedObject();
            // ...
        }
    }
    finally
    {
        for (int i = 0; i < vals.Length; i++)
            if (handles[i].IsAllocated) // if we are leaving the try-block via exception some handles may not have been assigned yet and calling Free() on them would mask the original exception by throwing a different one
                handles[i].Free();
    }
}

Sorry for leading the question off-topic.


3D hand normalization with empty frames

I am trying the code from https://github.com/sign-language-processing/pose/pull/36.
When I input a .pose file where the first few frames are empty (=no right hand detected and all values are 0s), normalization output becomes all nans, where 0s are expected.
Since the pose data is a masked array, I use pose.body.data.toflex() to inspect the arrays and now I have to use np.nan_to_num(pose.body.data) as a workaround to convert the nans back to 0s.
@AmitMY maybe worth checking the 0-handing logic in the code?

I'll need to see a reproduction.
In any case, the correct way to get the value from a masked array is pose.body.data.filled(0)

https://colab.research.google.com/drive/1XyfAoKeTDRakiCHYEFgsEoceciaIlsar


OnBlur not called in 2.11.0

Describe the bug
The OnBlur callback does not appear to be being called in 2.11.0
To Reproduce
Visit https://reactdatepicker.com/#example-onblur-callbacks-in-console and open the developer console

Interact with the picker and select a date.
No events are logged in the console.
Click the input again (see the cursor is inside the input)
Press tab. Blur event logged.

Expected behavior
Blur callback to be called one the input has left focus.
Desktop (please complete the following information):

OS: OSX Catalina
Browser Chrome
Version 79.0.3945.117

Additional context
The picker would blur when closed in 1.9.0

Anyone found a work around?
onBlur not working for me
OS:  OSX Catalina
Browser: Chrome
react-datepicker: 2.14.1

The opposite event onFocus is not working anymore neither https://github.com/Hacker0x01/react-datepicker/issues/1601

OnBlur is not even working when selecting a date on the Demo site ...
https://reactdatepicker.com/

This is really annoying if you want to validate on blur, It does blur but only with keyboard enter, very bizarre.

A work around I discovered (if you are trying to use it for an onChange  function) is onChangeRaw and then you declare your event based function for this attribute.

Another work around is to listen to onCalendarClose() and trigger validation manually.  I would still be grateful for a fix. 🙏

any proper fix

This desperately needs resolution. I'm not able to validate the datepicker in Formik after it gets blurred. Any progress?

FYI, this solution in Formik worked for me. I used React-Datepicker's onCalendarClose together with Formik's setFieldTouched(..). This fixed the immediate blur-validation problem for me.
        onCalendarClose={val => {
            // In Formik, manually do setTouched (with re-validation)
            setFieldTouched(field.name, true, true);
        }}

Still appears to be an issue and onCalendarClose() is not a suitable alternative as it does not pass event as onBlur() would, they aren't interchangeable and cause issues when integrating with existing libraries such as Formik, it's also still not working on the demo site "onBlur callbacks in console".

Hello,
We are still waiting for this fix. Is it possible for an estimation time for this fix?
Thanks!

Same issue. No effect onBlur
But I did solve my issue with a wrapper

const handleChange = (date) => {
  setStartDate(date)
  setOpen(true)
}

<div onBlur={() => setOpen(false}>
  <DatePicker  onChange={(date) => handleChange(date)} />
</div>

Bumping, we're also waiting for this fix

onBlur is working, but only after second click on input after selecting. This is showing to me that onChange isn't really firing onBlur and only after you manually clicking on input fires onBlur. This is difficult when you want to do validation on onBlur.
My workaround when working with react-hook-formm was to run trigger() after setting the field value.

onChangeRaw can fire onBlur event but It's NOT work when user just focus then blur out without any change.

The issue with onFocus and onBlur makes this component basically unusable. It's too bad.

still happening...

OnBlur isn't called if DatePicker is used with Portals and you tab out of the input:
https://github.com/Hacker0x01/react-datepicker/issues/3522#issuecomment-1149954270

I was able to check validation on closing the calendar: comment  #2068

I'm also interested in solving this. I can submit a PR, but I'm not sure what the best solution is.
Expected behavior

User types a date into the input and presses tab, or user clicks a date on the calendar popup.
The user-supplied onBlur callback prop runs.

Actual behavior
onBlur callback does not run.
Potential Messy Workaround

Duplicate onBlur functionality in the onSelect prop. For my use case, this handled the user clicking on a date from the date picker.
Also duplicate onBlur functionality in the onCalendarClose prop. This partially resolved my requirements for when the user enters dates manually from the keyboard. However, this callback does not receive an event argument (as noted by others above). This workaround also does not fire when the user types a date and presses tab. It only works when the user types a date and then presses Enter or clicks outside the datepicker to close the calendar.
Also duplicate onBlur functionality in the onKeyDown callback if event.key is "Tab". This works for shift-Tab as well because there is no need to check if the shift key is also pressed.

My cumbersome workaround now looks like the below, but I'm not sure how many edge cases I'm missing. I can also understand this may not work when integrating with another library that expects onBlur to work.
<DatePicker
        selected={date}
        onChange={(d) => {
          setDate(d);
        }}
        onSelect={handleBlur}
        onBlur={handleBlur}
        onCalendarClose={handleBlur}
        onKeyDown={(e) => {
          if (e.key === "Tab") {
            handleBlur();
          }
        }}
></DatePicker>

Potential [untested] Solution
Change the DatePicker internal handleBlur function to propagate blur events to the user even when the calendar is still open. Currently, the handle blur function ignores blur events if the calendar is open (see below). There is probably a reason for ignoring these events, but it isn't immediately obvious to me what the reason is. This solution would simply be to delete the first conditional check !this.state.open ||.
handleBlur = (event) => {
    if (!this.state.open || this.props.withPortal || this.props.showTimeInput) {
      this.props.onBlur(event);
    }

    this.setState({ focused: false });
  };

I'm also interested in solving this. I can submit a PR, but I'm not sure what the best solution is.
Expected behavior

User types a date into the input and presses tab, or user clicks a date on the calendar popup.
The user-supplied onBlur callback prop runs.

Actual behavior
onBlur callback does not run.
Potential Messy Workaround

Duplicate onBlur functionality in the onSelect prop. For my use case, this handled the user clicking on a date from the date picker.
Also duplicate onBlur functionality in the onCalendarClose prop. This partially resolved my requirements for when the user enters dates manually from the keyboard. However, this callback does not receive an event argument (as noted by others above). This workaround also does not fire when the user types a date and presses tab. It only works when the user types a date and then presses Enter or clicks outside the datepicker to close the calendar.
Also duplicate onBlur functionality in the onKeyDown callback if event.key is "Tab". This works for shift-Tab as well because there is no need to check if the shift key is also pressed.

My cumbersome workaround now looks like the below, but I'm not sure how many edge cases I'm missing. I can also understand this may not work when integrating with another library that expects onBlur to work.
<DatePicker
        selected={date}
        onChange={(d) => {
          setDate(d);
        }}
        onSelect={handleBlur}
        onBlur={handleBlur}
        onCalendarClose={handleBlur}
        onKeyDown={(e) => {
          if (e.key === "Tab") {
            handleBlur();
          }
        }}
></DatePicker>

Potential [untested] Solution
Change the DatePicker internal handleBlur function to propagate blur events to the user even when the calendar is still open. Currently, the handle blur function ignores blur events if the calendar is open (see below). There is probably a reason for ignoring these events, but it isn't immediately obvious to me what the reason is. This solution would simply be to delete the first conditional check !this.state.open ||.
handleBlur = (event) => {
    if (!this.state.open || this.props.withPortal || this.props.showTimeInput) {
      this.props.onBlur(event);
    }

    this.setState({ focused: false });
  };

Can you submit PR?) Please

onClickOutside={handleClickOutside} and then close the calendar changing date picker open attribute

Bump, this is still happening I'm afraid
Type "31-2-2024" in the input field, press tab, now the input field automatically fixes the date.  ✅
Type "31-2-2024" in the input field, then click outside the input field. Nothing happens.  ❌


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

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

Comparison is base (d59b1ba) 90.76% compared to head (dd61960) 90.76%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                  Coverage Diff                   @@
##           ci/dependency-updates     #141   +/-   ##
======================================================
  Coverage                  90.76%   90.76%
======================================================
  Files                         26       26
  Lines                        433      433
======================================================
  Hits                         393      393
  Misses                        40       40

Flag
Coverage Δ

linux
90.76% <ø> (ø)

windows
90.07% <ø> (ø)

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

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


Graphical expression of the result is necessary

[x]  Graphical expression of the result is necessary. Add it.

Done, working well!


Set "Active" as product default?

Hi,
Is it possible to set products to be active when loaded rather than inactive?
Thanks
Dave

Actually don't worry. I got around this by adding a simple bit of jquery:
//Set Product to be Active by default:
$('#pActive').find('option').eq(0).prop('selected', true);
Also, in catalog mode it still insists on a price, so with this simple jquery it'll automatically set a minimal price:
//Set Price to be £1 by default:
if($('#pPrice').val() === '') {
$('#pPrice').val('1');
}

Nice one.  The only other thing I would be suggesting is using the on_community_store_product_add event, to change the product after saving, but that's probably more work than your jquery here.
It might be nice to have these defaults adjustable

Yes I have used that add-on for sending emails when order status is changed, very useful.
Adjustable defaults would be great too.


一加3 root启动黑域失败

WARNING: run as root is experimental!!!
checking for server..fail
please report bug to https://github.com/brevent/Brevent/issues with log below
--- crash start ---

logcat -b crash -t '07-23 14:27:08.000' -d
--- crash end ---
--- brevent start ---
logcat -b main -t '07-23 14:27:08.000' -d -s BreventLoader BreventServer
07-23 14:27:09.441  9874  9874 D BreventLoader: loader path: /data/app/me.piebridge.brevent-1/base.apk
07-23 14:27:09.885  9893  9893 D BreventServer: get service: package
07-23 14:27:09.885  9893  9893 D BreventServer: args: [10]
07-23 14:27:09.886  9893  9893 I BreventServer: signal: 10
07-23 14:27:09.893  9893  9893 I BreventServer: starting Brevent Server 2.0.4b
07-23 14:27:09.901  9893  9893 D BreventServer: get service: package
07-23 14:27:09.903  9893  9893 I BreventServer: brevent user: 0, uid: 0
07-23 14:27:09.904  9893  9893 I BreventServer: brevent data directory: /data/user_de/0/com.android.shell
07-23 14:27:09.916  9893  9893 I BreventServer: loading brevent list
07-23 14:27:09.916  9893  9893 I BreventServer: loading brevent conf
07-23 14:27:09.918  9893  9893 D BreventServer: get service: appwidget
07-23 14:27:09.918  9893  9893 E BreventServer: Cannot dump appwidget(remote dead)
07-23 14:27:09.937  9874  9874 E BreventLoader: worker 9893 exited with status 1
07-23 14:27:09.937  9874  9874 D BreventLoader: signal arrived, update: 0
07-23 14:27:09.939  9870  9870 D BreventLoader: worker 9874 exited with status 0
--- brevent end ---
please report bug to https://github.com/brevent/Brevent/issues with log above

Cannot dump appwidget(remote dead)

请禁用 selinux，或者参考 #32


Enable sfdx force:source:deploy to return code coverage

Is your feature request related to a problem? Please describe.
I was wondering if it is possible to add a new flag to sfdx force:source:deploy -c -l RunLocalTests to return the current code coverage after the deployment validation.
This article shows how to do it accurately, but it would be nice to have SFDX doing that for us.
What are you trying to do
Return codeCoverage when running sfdx force:source:deploy
Describe the solution you'd like
Add a new flag to sfdx force:source:deploy to output the code coverage
Describe alternatives you've considered
I was thinking about automating the steps from this article https://help.salesforce.com/s/articleView?id=000335222&type=1
Additional context
N/A

this can already be done. my bad


add tags to generated language tests

I also didn't realize npm run cucumber_base -- "$@" tests/features/*.feature could run tests that don't have tags. Very thoughtful of the developers.

Rather, npm run langs_setup && npm run cucumber -- $@

Will close #54, research tags

Will also address #19

@plocket What does the comment about "isn't ignoring reports" mean?  What reports?

We have files that get saved now that are called reports. They have _report_ in the name. They give the tester some extra information about what happened during the tests.
I thought that we added _report_ to the script that gives a repository its .gitignore, but something must have gotten lost along the way.

We have files that get saved now that are called reports. They have _report_ in the name. They give the tester some extra information about what happened during the tests.
I thought that we added _report_ to the script that gives a repository its .gitignore, but something must have gotten lost along the way.


fix: Obtain version of this extension from the manifest

Fix for #18.

Looking good - lgtm!

:tada: This PR is included in version 2.1.2 :tada:
The release is available on:

Chrome Web Store
GitHub release

Your semantic-release bot :package::rocket:


Check current zone on quest objectives

in addition to the quest header do a check on objectives for what zone they are in?
also check the header group of the quest for zone name? i.e. the pvp quests are in the "Valdrakken" group - how can i use that?
(just an idea)

V1.3.5 added the ability to check the quest header group name.  V1.4.0 further optimized this check.
Quest objectives do not have zone attachments so can't be used.


How can I finally solve the problem of low PED performance in reproducible results?

Hi, @aidiaoyudemao
Training the model with the latest code does not result in low performance for pedestrian.
For more details, kindly refer to the response in #3 . Thanks!


[MINOR] Fix comment about usage of build.sh

Proposed changes
Current the usage of build.sh in the comment is misleading, this patch try to fix that.
Python 2.7.5
Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
Maven home: /usr/share/maven
Java version: 1.8.0_202, vendor: Oracle Corporation, runtime: /usr/java/jdk/jre
Default locale: en_US, platform encoding: ANSI_X3.4-1968
OS name: "linux", version: "4.19.76-linuxkit", arch: "amd64", family: "unix"
cmake version 3.12.3

CMake suite maintained and supported by Kitware (kitware.com/cmake).
build.sh: invalid option -- 'c'
build.sh: invalid option -- 'l'
build.sh: invalid option -- 'e'
build.sh: invalid option -- 'a'
build.sh: invalid option -- 'n'```

## Types of changes

What types of changes does your code introduce to Doris?
_Put an `x` in the boxes that apply_

- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] Documentation Update (if none of the other choices apply)
- [ ] Code refactor (Modify the code structure, format the code, etc...)
- [ ] Optimization. Including functional usability improvements and performance improvements.
- [ ] Dependency. Such as changes related to third-party components.
- [x] Other.

## Checklist

_Put an `x` in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code._

- [ ] I have created an issue on (Fix #ISSUE) and described the bug/feature there in detail
- [ ] Compiling and unit tests pass locally with my changes
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] If these changes need document changes, I have updated the document
- [ ] Any dependent changes have been merged

## Further comments

cc @caiconghui

Thanks for your time @morningman PTAL


Error Loading Quests, ScriptEntryData

What is your Quests version / build number (do not say "latest")?
Quests-v3.7.7-b12
Denizen-1.06-b1676-REL
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
Quests won't load
https://pastebin.com/d405EnkU
If applicable, can you provide an example quest from quests.yml using Pastebin.com?

Use https://ci.codemc.org/job/PikaMug/job/Quests/13/


Screen tracking and analytics for the first page

I need to track every screen view.
I already checked out https://reactnavigation.org/docs/guides/screen-tracking which works fine except for the very first screen.
The reason is because there is no action being fired when loading the first screen.
I can't be the first person who has this problem right?
I want to know how people get around solving this, hopefully without having to hard code within first screen component.
Thank you.

You can use componentDidMount of a container, that renders e.g. your stack navigator. Since you know what will the initialRoute be, it's easy.

This still doesn't work well with apps that persist user state when the user returns. Then the initialRoute can be different.


Steps to include custom dependency in sample

In docs there is description how to add own samples.
However I struggle to add a sample that uses my domain code from other assembly (and I want to keep it there)
If I put "open mydomain.subdomain" in my sample in REPL it cannot be compiled (the namespace or module is not defined). I added my dll (with .txt extension as well) to metadata folder.
I tried by adding my domain code via paket or nuget and I can see it is compiled to js but I still cannot compile the samples in Repl.
Is there additional file to configure the mapping or any additional step?
I tried to look at similar sample with Recharts but I do not understand the workflow of adding such a dependencies.

Well, unfortunately this is a somewhat complicated process. Dependencies in the REPL are precompiled for better performance, but this is causes problems with Fable (inlined code, namespace resolution) so we used several hacks to make it work for this particular case.
Again, this was implemented in a quick and dirty way and it was not designed to be used general. But I think you can add extra dependencies following these steps:

Bring in the dependency sources with Paket. The dependency shouldn't have public inlined functions.
Add references to the dependency sources in this project file.
Add an entry to this JSON file to help Fable with the namespace resolution. An entry like this: "Elmish.Program": ["Elmish", "src/program"] means "to get a reference to a member of the Elmish.Program namespace, remove the Elmish- part and go to the src/program file (in the lib output folder)".
Build the project to rebuild the library. You will likely have to check the public/js/repl/lib to see the exact path of the newly compiled sources.

Thanks Alfonso.
So I was almost there but even with your suggestion I still cannot see my domain.
In paket I specify my github repository, my 2 files are downloaded and after adding to Repl Lib project they are successfully compiled and the lib folder is as follow:

wrometr.domain has two js files: Address.js and District.js
Repl-lib-map after modification contains following entry:
"wrometr.domain.addresses":["wrometr.domain.addresses", "wrometr.domain/Address"]
I'm compailing the repl.lib project and copying the dll (dll.txt) to metadata folder
Still cannot find the domain in Repl:

well, it worked with yet another building, strange but maybe common.
I'm playing now with typical bindings like simple-statistics, turfjs, D3js.
If no further problems I will close the issue and maybe do the PR, do you welcome such a samples?


Single line json output

Hi,
I'm using goss in a systemd timer and JournalBeat to ship the logs to ElasticSearch.
At the moment I'm using the nagios_verbose format and parsing it in logstash but if the json output was a single line json message I could use that directly with no need for parsing.
Can you make the json output format single line and maybe introduce a json_pretty that is multi line?
Thanks.

I would probably add a json_oneline to preserve backwards compatibility.

That would work as well.

Sorry, forgot about this when creating today's release. It'll be in the next version of goss.

Released in v0.3.4


WormJam v0.1.0 - Prerelease before curation round #3

This PR begins the official versioning of the WormJam model. It contains the work done over the past 9 months.
After this merge, ownership of the repository will be transferred to the WormJam Consortium organisation.
A release will be created and a DOI assigned to it.

Great work! Let's make an archived version and move this version over to the WormJam consortium


[Footer] Incorrect alignment of footer on screen resolution 1024px, 768px, 576px, 320px.

Environment: macOS High Sierra 10.13, Google Chrome 81.0.4044.129 (64-bit)
Reproducable: always
Build found: commit a101c84
Steps to reproduce

Open Eco news page https://ita-social-projects.github.io/GreenCityClient/#/news;
Scroll to the footer of the page;
Click F12;
Change screen resolution on 1024px, 768px, 576px, 320px.

Actual result:

Incorrect distance of elements and alignment of text "© Copyright 2020. Green City" on screen resolution 1024px
Incorrect distance of elements and alignment of text "© Copyright 2020. Green City" on screen resolution 768px
Incorrect alignment of text "© Copyright 2020. Green City" on screen resolution 576px
Incorrect distance between elements on screen resolution 320px

Expected result:

Distance and alignment on screen resolution 1024px is according to mockup
Distance and alignment on screen resolution 768px is according to mockup
Alignment on screen resolution 576px is according to mockup
Distance between elements on screen resolution 320px is according to mockup

User story: #384

Menu links and Social network links don't stand on the one line.
Actual result:

Expected result:


Improve highlighting for Elixir.

Specifically:

Add missing keywords 'require', 'import', and 'with'.
Correctly (not) highlight the '::' operator, instead of treating as symbol.
Allow for compound names (for modules, but affects other definitions too).

Fixes #1824.

Looks like the shell detection test is failing, perhaps because I added the with keyword and the test fixture has the line Your branch is up-to-date with 'origin/master'..  Is there an easy way to exclude a keyword from contributing to language detection?  I see some language definitions with relevance: 0 on rules but not on keywords.

Never mind, it was indeed the with keyword and I found the docs on how to zero out its relevance.  All tests passing now.

Oh, great work @pkaminski! Thanks for figuring it out!


missing table is not created?

I ran the import process against baglung and it failed with over 1900 messages which said "Error encountered!".
I guessed that might correspond to one error per object and added the table into the database by hand. The second run finished without error.
The unit test code calls the table creation when needed. I'll need to look more closely at the integration code to see if it is tested there.

This issue has just been confirmed on Travis.
Running "mochaTest:integration" (mochaTest) task

  Integration

Error encountered!
  1) "before all" hook
Error encountered!

  0 passing (2s)
  1 failing

  1) Integration "before all" hook:
     Error: relation "couchdata" does not exist
      at Error (native)
      at wrapError (common.js:7:11)
      at common.js:19:19
      at lib$rsvp$$internal$$tryCatch (node_modules/rsvp/dist/rsvp.js:489:16)
      at lib$rsvp$$internal$$invokeCallback (node_modules/rsvp/dist/rsvp.js:501:17)
      at lib$rsvp$$internal$$publish (node_modules/rsvp/dist/rsvp.js:472:11)
      at lib$rsvp$$internal$$publishRejection (node_modules/rsvp/dist/rsvp.js:415:7)
      at lib$rsvp$asap$$flush (node_modules/rsvp/dist/rsvp.js:1290:9)

On my local machine, I can delete the given database and the integration test still succeeds (it creates the relation and populates it). The checkout commit is the same for travis and my local system. Time to play a rousing game of "what's different?"
I'm wondering if the postgres setup is not being performed as expected on Travis or if somehow I'm missing a step there.

On my local env, I ran the following two commands as postgres user (which appear to work fine on Travis based on output):
psql -c "CREATE ROLE medictest WITH LOGIN PASSWORD 'password';" -U postgres
psql -c "CREATE DATABASE medicdata WITH OWNER medictest;" -U postgres

Then I created a file with env vars to source based on .travis.yml settings:
#travis
export POSTGRESQL_URL="postgres://medictest:password@localhost:5432/medicdata"
export POSTGRESQL_TABLE="couchdata"
export POSTGRESQL_COLUMN="json"
#export COUCHDB_URL="http://localhost:5984/medic/_all_docs?include_docs=true"

#from local VM
export COUCHDB_URL='http://bryan:password@<IP_ADDRESS>/medic/_all_docs?include_docs=true'

I ran grunt ci. Interestingly, I get no errors, but the integration test fails. I went to check the database to see what was in it and quickly found the error on my local system:
$ psql -U medictest medicdata
psql: FATAL:  Peer authentication failed for user "medictest"

Of course on my local postgres, I've disabled password auth and only trust unix domain auth. This makes me wonder if the Travis postgres settings preclude certain forms of access. I still can't replicate the error I see on Travis.

Local system now works totally fine with .travis.env settings except for local postgres user issue. Still, Travis claims that the relation couchdata is not found as though the table is not made.
I'm not sure what else to do here besides use the postgres user outright, since I know for sure that works without error to create databases and roles and it is the only thing documented for accessing postgres in Travis docs.

NOPE.
I got nothing. You win Travis. Your mysteries are too mysterious for me.

Travis does not create the table because JSONB is not a valid type! Is this actually running 9.4 as requested?
Some debug which explains a lot:
$ psql -d ${POSTGRESQL_URL} -c "CREATE TABLE couchdata (jsondata JSONB);"

ERROR:  type "jsonb" does not exist

LINE 1: CREATE TABLE couchdata (jsondata JSONB);

Travis cannot build a table with a jsonb column because it is running 9.1:
0.06s$ psql --version

psql (PostgreSQL) 9.4.0

$ psql -d ${POSTGRESQL_URL} -c "SELECT version();"

                                                   version

-------------------------------------------------------------------------------------------------------------

 PostgreSQL 9.1.14 on x86_64-unknown-linux-gnu, compiled by gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3, 64-bit

(1 row)

I think this issue comment addresses the problem. Apparently many people make the mistake in YAML of adding a hyphen where there should be none.
https://github.com/travis-ci/travis-ci/issues/2983#issuecomment-75420000
Sure enough I have a hyphen, and sure enough the travis CI docs exclude a hyphen. Apparently this is the difference between an object and an array in YAML:
addons:
  postgresql: "9.3"

addons:
  - postgresql: "9.3"

Still getting red on Travis, but the integration failure clearly indicates the table was created.
Both local and Travis can create table. This ticket is now lies.


How to config the hint by spring namespace int the sharding-jdbc 3.1 version

How to config the hint by spring namespace int the sharding-jdbc 3.1 version
My applicationContext.xml config is useless
as follow:
<bean id="tableShardingAlgorithmByMonth" class="com.xxx.TableShardingAlgorithmByMonth"></bean>
<bean id="hintTestBean" class="com.xxx.ModuloHintShardingAlgorithm"></bean>
    <sharding:hint-strategy id="hintStrategy"  algorithm-ref="hintTestBean" ></sharding:hint-strategy>
  	<sharding:standard-strategy id="payOrderPreciseAlgorithm" sharding-column="create_date" precise-algorithm-ref="tableShardingAlgorithmByMonth"></sharding:standard-strategy>
    <sharding:data-source id="shardingDataSource">
        <sharding:sharding-rule data-source-names="dataSource">
            <sharding:table-rules>
            	<sharding:table-rule logic-table="pay_order" table-strategy-ref="payOrderPreciseAlgorithm"  />
            </sharding:table-rules>
            <sharding:binding-table-rules>
                <sharding:binding-table-rule logic-tables="pay_order"/>
            </sharding:binding-table-rules>
        </sharding:sharding-rule>
        <sharding:props>
            <prop key="sql.show">false</prop>
        </sharding:props>
    </sharding:data-source>

The ModuloHintShardingAlgorithm.java as follow:
import io.shardingsphere.api.algorithm.sharding.ShardingValue;
import io.shardingsphere.api.algorithm.sharding.hint.HintShardingAlgorithm;

import java.util.Collection;

public class ModuloHintShardingAlgorithm implements HintShardingAlgorithm {

	@Override
	public Collection<String> doSharding(Collection<String> availableTargetNames, ShardingValue shardingValue) {
		System.out.println(shardingValue.getLogicTableName());
		for (String each : availableTargetNames) {
			System.out.println(each);
		}
		return null;
	}

}

But when querying, this function will not be called.
Is my configuration right?
I could`t find the configuration of hint spring namespace in https://github.com/apache/incubator-shardingsphere-example

@xiaotianlongxing, Hint is sharding strategy, it should be injected into table-strategy-ref or database-strategy-ref
In your example, you injected sharding:standard-strategy not sharding:hint-strategy, so this function will not be called.
you can change like following and retry
<bean id="tableShardingAlgorithmByMonth" class="com.xxx.TableShardingAlgorithmByMonth"></bean>
<bean id="hintTestBean" class="com.xxx.ModuloHintShardingAlgorithm"></bean>
<sharding:hint-strategy id="hintStrategy"  algorithm-ref="hintTestBean" ></sharding:hint-strategy>
<sharding:standard-strategy id="payOrderPreciseAlgorithm" sharding-column="create_date" precise-algorithm-ref="tableShardingAlgorithmByMonth"></sharding:standard-strategy>
<sharding:data-source id="shardingDataSource">
    <sharding:sharding-rule data-source-names="dataSource">
        <sharding:table-rules>
            <sharding:table-rule logic-table="pay_order" table-strategy-ref="hintStrategy"  />
        </sharding:table-rules>
        <sharding:binding-table-rules>
            <sharding:binding-table-rule logic-tables="pay_order"/>
        </sharding:binding-table-rules>
    </sharding:sharding-rule>
    <sharding:props>
        <prop key="sql.show">false</prop>
    </sharding:props>
</sharding:data-source>

Of course you can inject sharding:hint-strategy as a default sharding strategy like
<bean id="tableShardingAlgorithmByMonth" class="com.xxx.TableShardingAlgorithmByMonth"></bean>
<bean id="hintTestBean" class="com.xxx.ModuloHintShardingAlgorithm"></bean>
<sharding:hint-strategy id="hintStrategy"  algorithm-ref="hintTestBean" ></sharding:hint-strategy>
<sharding:standard-strategy id="payOrderPreciseAlgorithm" sharding-column="create_date" precise-algorithm-ref="tableShardingAlgorithmByMonth"></sharding:standard-strategy>
<sharding:data-source id="shardingDataSource">
    <sharding:sharding-rule data-source-names="dataSource" default-table-strategy-ref="hintTestBean">
        <sharding:table-rules>
            <sharding:table-rule logic-table="pay_order" table-strategy-ref="payOrderPreciseAlgorithm"  />
        </sharding:table-rules>
        <sharding:binding-table-rules>
            <sharding:binding-table-rule logic-tables="pay_order"/>
        </sharding:binding-table-rules>
    </sharding:sharding-rule>
    <sharding:props>
        <prop key="sql.show">false</prop>
    </sharding:props>
</sharding:data-source>

@KomachiSion  I see ,think you very much

If no more questions, I will close issue.


Error after normalization with scran

Hej
I have been looking at the single-cell-tutorial repository and tried the scran normalization with R.
After calculating the size factors in scran, I use them to normalize cell-wise my data matrix following the tutorial commands:
adata.X /= adata.obs['size_factors'].values[:,None]
sc.pp.log1p(adata)

When I look for highly expressed genes with
sc.preprocessing.highly_variable_genes(adata, n_top_genes=5000)
I get this error (the error does not show up when I use the normalization command from scanpy instead of the normalization with size factors from scran):
LinAlgError                               Traceback (most recent call last)
<ipython-input-97-96c692867dde> in <module>
----> 1 sc.preprocessing.highly_variable_genes(adultAll, n_top_genes=10000, flavor='cellranger')

~/miniconda3/envs/scRNA/lib/python3.6/site-packages/scanpy/preprocessing/_highly_variable_genes.py in highly_variable_genes(adata, min_disp, max_disp, min_mean, max_mean, n_top_genes, n_bins, flavor, subset, inplace)
    101     X = np.expm1(adata.X) if flavor == 'seurat' else adata.X
    102
--> 103     mean, var = materialize_as_ndarray(_get_mean_var(X))
    104     # now actually compute the dispersion
    105     mean[mean == 0] = 1e-12  # set entries equal to zero to small value

~/miniconda3/envs/scRNA/lib/python3.6/site-packages/scanpy/preprocessing/_utils.py in _get_mean_var(X)
     16             mean_sq = np.multiply(X, X).mean(axis=0)
     17         # enforece R convention (unbiased estimator) for variance
---> 18         var = (mean_sq - mean**2) * (X.shape[0]/(X.shape[0]-1))
     19     else:
     20         from sklearn.preprocessing import StandardScaler

~/miniconda3/envs/scRNA/lib/python3.6/site-packages/numpy/matrixlib/defmatrix.py in __pow__(self, other)
    226
    227     def __pow__(self, other):
--> 228         return matrix_power(self, other)
    229
    230     def __ipow__(self, other):

~/miniconda3/envs/scRNA/lib/python3.6/site-packages/numpy/linalg/linalg.py in matrix_power(a, n)
    600     a = asanyarray(a)
    601     _assertRankAtLeast2(a)
--> 602     _assertNdSquareness(a)
    603
    604     try:

~/miniconda3/envs/scRNA/lib/python3.6/site-packages/numpy/linalg/linalg.py in _assertNdSquareness(*arrays)
    213         m, n = a.shape[-2:]
    214         if m != n:
--> 215             raise LinAlgError('Last 2 dimensions of the array must be square')
    216
    217 def _assertFinite(*arrays):

LinAlgError: Last 2 dimensions of the array must be square

Do you have any hints? I am trying to find the error but so far I have been unsuccessful.

Hi,
This bug report may be better placed in the single-cell-tutorial repo as you're having issues with how the tools are chained.
A few things to check:

Are you using sparse matrices in your data?
are the size factors you get all real numbers (not NaN)?

I've seen this error before... just can't remember what the issue was. I think it was sparse matrices.

Hi,
I just made a test and it seems that it is indeed a problem of non-sparse matrix. How can I render the matrix sparse again from one that is dense? Is there something similar to the adata.X.todense() command?

Hi again,
after normalizing with an external tool I resparsified the matrix as it follows:
adata.X = scipy.sparse.csr_matrix(adata.X)
and everything works.
Thanks for help. Cheers.

So for future reference. scran and other R tools don't work on sparse matrices. Whenever you need to do batch correction, that doesn't matter anyway, as you'll probably have to densify your matrix for batch correction anyway (or it will be dense after batch correction).

scran works on sparse stuff, maybe not all its tools yet

The above comment was referring to the old rpy2/anndata2ri implementation of the single-cell-tutorial notebook. The newer version of the notebook (Version 1906) indeed does allow for sparse matrix conversion into SingleCellExperiment objects.


autoluv errors ruby 2.7.0 / Ubuntu 20.04

I have a fresh install, implemented step by step.
invoking autoluv fails, with several warnings and an error.  hoping for some ideas on what's not set up properly

xxxx@instance-2:~$ sudo autoluv schedule AAAAAA Fake Name<EMAIL_ADDRESS>/var/lib/gems/2.7.0/gems/net-protocol-0.2.1/lib/net/protocol.rb:68: warning: already initialized constant Net::ProtocRetryError
/usr/lib/ruby/2.7.0/net/protocol.rb:66: warning: previous definition of ProtocRetryError was here
/var/lib/gems/2.7.0/gems/net-protocol-0.2.1/lib/net/protocol.rb:214: warning: already initialized constant Net::BufferedIO::BUFSIZE
/usr/lib/ruby/2.7.0/net/protocol.rb:206: warning: previous definition of BUFSIZE was here
/var/lib/gems/2.7.0/gems/net-protocol-0.2.1/lib/net/protocol.rb:541: warning: already initialized constant Net::NetPrivate::Socket
/usr/lib/ruby/2.7.0/net/protocol.rb:503: warning: previous definition of Socket was here
Please create a valid Southwest header file before continuing. Learn more: https://github.com/byalextran/southwest-headers

LUV_HEADERS_FILE = /home/xxxx/southwest-headers/southwest_headers.json
LUV_FROM_EMAIL  = xxxx
LUV_USER_NAME   = xxxx
LUV_PASSWORD    = xxxx
LUV_SMTP_SERVER = smtp.gmail.com
LUV_PORT        = 587

xxxx@instance-2:~$ cat /home/xxxx/southwest-headers/southwest_headers.json
{"content-type": "application/json", "x-channel-id": "MWEB", "ee30zvqlwf-a": "78uwUrKpupN3bGLq-=3lDKyqHrs=I-Paxy73AqPeRXgY0DiG_iGPEi==UvsSUDRn2NTidAwR8y99KYNfdSZopR6kix8rxFmf=AcYbIFkCinOP8tFrKfiZCMyoL1tHyZzg66XdWf3KS0D2pTWh05pdA_oYbKw041nCtaJObn83x6BbifJgNiA6tZ1oGuI=enJ8D8yLicLZOPxZJ9Pqb4r=8WPsktZIxnlK6JXJXC9ExyWl_aOgMA=Gb-OWgEc9C=yYq85upxJqTUno6SPPpWZthix0sL9_x=78MFZ=X6d4w-XYiEBGwWF40C4HC46YqYmL-TU=IK5wTG3WeS5x4h8A_JeJY0VasaEXDdaF91Lu-g6p5....... etc etc

based on what you're showing, it seems the headers file is where it should be. what are the chances the XXXX in the path have spaces in it? if so, try wrapping the full path in quotes.
also, if everything is installed properly you shouldn't need to run the command with sudo'. what happens if you try that?

No spaces in the path anywhere.  Running without sudo yields this
xxxx@instance-2:~$ autoluv schedule AAAAAA Fake Name<EMAIL_ADDRESS>/var/lib/gems/2.7.0/gems/net-protocol-0.2.1/lib/net/protocol.rb:68: warning: already initialized constant Net::ProtocRetryError
/usr/lib/ruby/2.7.0/net/protocol.rb:66: warning: previous definition of ProtocRetryError was here
/var/lib/gems/2.7.0/gems/net-protocol-0.2.1/lib/net/protocol.rb:214: warning: already initialized constant Net::BufferedIO::BUFSIZE
/usr/lib/ruby/2.7.0/net/protocol.rb:206: warning: previous definition of BUFSIZE was here
/var/lib/gems/2.7.0/gems/net-protocol-0.2.1/lib/net/protocol.rb:541: warning: already initialized constant Net::NetPrivate::Socket
/usr/lib/ruby/2.7.0/net/protocol.rb:503: warning: previous definition of Socket was here
Traceback (most recent call last):
        14: from /usr/local/bin/autoluv:23:in `<main>'
        13: from /usr/local/bin/autoluv:23:in `load'
        12: from /var/lib/gems/2.7.0/gems/autoluv-0.3.1/bin/autoluv:3:in `<top (required)>'
        11: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
        10: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         9: from /var/lib/gems/2.7.0/gems/autoluv-0.3.1/lib/autoluv.rb:3:in `<top (required)>'
         8: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         7: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         6: from /var/lib/gems/2.7.0/gems/pony-1.13.1/lib/pony.rb:1:in `<top (required)>'
         5: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         4: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         3: from /var/lib/gems/2.7.0/gems/mail-2.8.0/lib/mail.rb:3:in `<top (required)>'
         2: from /var/lib/gems/2.7.0/gems/mail-2.8.0/lib/mail.rb:14:in `<module:Mail>'
         1: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
/usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require': cannot load such file -- mail/indifferent_hash (LoadError)
        15: from /usr/local/bin/autoluv:23:in `<main>'
        14: from /usr/local/bin/autoluv:23:in `load'
        13: from /var/lib/gems/2.7.0/gems/autoluv-0.3.1/bin/autoluv:3:in `<top (required)>'
        12: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
        11: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
        10: from /var/lib/gems/2.7.0/gems/autoluv-0.3.1/lib/autoluv.rb:3:in `<top (required)>'
         9: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         8: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         7: from /var/lib/gems/2.7.0/gems/pony-1.13.1/lib/pony.rb:1:in `<top (required)>'
         6: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         5: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
         4: from /var/lib/gems/2.7.0/gems/mail-2.8.0/lib/mail.rb:3:in `<top (required)>'
         3: from /var/lib/gems/2.7.0/gems/mail-2.8.0/lib/mail.rb:14:in `<module:Mail>'
         2: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:156:in `require'
         1: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:168:in `rescue in require'
/usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:168:in `require': cannot load such file -- mail/indifferent_hash (LoadError)

I might try fresh on a new vm image with Debian

because this is a Google Cloud base image it is probably hardened, and the .autoluv.env is not working


fix(discv4): ignore self-entry on neighbours response

Closes #636
ignore the SelfEntry variant when handling node records received from other nodes. this will prevent us from sending packets to our own node.

Codecov Report

Merging #664 (f1eb86e) into main (24d5612) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #664      +/-   ##
==========================================
- Coverage   73.02%   73.02%   -0.01%
==========================================
  Files         260      260
  Lines       26665    26668       +3
==========================================
+ Hits        19471    19473       +2
- Misses       7194     7195       +1

Flag
Coverage Δ

unit-tests
73.02% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

crates/net/discv4/src/lib.rs
50.95% <0.00%> (-0.14%)
:arrow_down:

crates/stages/src/stages/bodies.rs
93.76% <0.00%> (+0.37%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[PE-D][Tester A] delete command does not update the top right panel in the gui even if the person is deleted.

When I entered delete 2, I deleted Bernice successfully, but her name still appeared in the top right panel of the gui.
It only changes the display if I I manually click on the contact in the contact list on the left

Labels: type.FunctionalityBug severity.Low
original: ITZXITZX/ped#6

There must have been some steps that he missed that caused that bug. All 5 of us tried to reproduce this bug and it doesn't happen

planned enhancement: more detailPanel corrections i.e. deleting until last person leaves detail still on detailpanel


target option is missing

There is a --target option in the README, but it does not seem to be implemented.
https://github.com/egoist/package-size/blob/master/README.md#target

Is the option already lost for --target? Or will it be implemented from now on?

That's available in the API but not in the CLI, you can add it here:
https://github.com/egoist/package-size/blob/d7804a972bc5ec68167585eed8036610eb691b8c/cli.js#L126
PR welcome.

I see, I'm sorry!
I looked over the documentation again, but as you said it was an API only option.
I read source well again, and I will try to put out PR if necessary!


add eval around gerp score retrieval

Requirements
None
Description
We started having problems retrieving files from the FTP site. This is still a problem for staging. The error message doesn't appear anymore on the page after applying the PR [my sandbox].
For the release 99 page we changed a die to a warning message [merged PR]. We are planning to revert the API update and also provide a web code update. It is likely that users will run an analysis with the API code and only after will notice the warning message about the FTP site problem.
Views affected
Variant summary page
Possible complications
FTP site errors will be more difficult to spot now
Merge conflicts
None
Related JIRA Issues (EBI developers only)
https://www.ebi.ac.uk/panda/jira/browse/ENSVAR-3021

Should we catch any errors and warn/log them?

I checked other places in the webcode where eval is used and none of them catch and print any messages. What would you suggest? Do you have a way of catching anything that is reported from the website? Otherwise it would only spam the log files.

@at7 We really only have the log files. Most of the time it's sufficient to do:
unless ($@) { ...next step in process...}
to prevent failed API calls from bringing down the Apache process. We generally only log the errors when debugging, otherwise as you say it fills up the error log unnecessarily.

Thanks @ens-ap5
I'm happy with my PR then. There is a check after the eval to see if the GERP score has been assigned which is safe for the display purpose.

Are you OK to merge? It would be helpful for us if this got be updated on staging. At the moment the error prevents loading the data on the variation summary page. Thank you

Hi Anja
Sure, I can do that. Sorry, yesterday was full of meetings and fixing problems caused by the DC migration.

Thank you Anne!


preview not showing

Describe the bug
try to view the preview and nothing is not coming up, pls help me
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-8xes3m
Steps to reproduce
just preview it and you will see it
Expected behavior
show preview
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 109307858,
  "usedJSHeapSize": 77809978,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = Gdataly
Chat ID = 3a41c0f57203

Additional context
No response

Sorry to hear this! This is a known issue and is being tracked here: #1791.
Make sure you don't have any browser extensions running that might interfere with Bolt such as an add blocker.
Hope this helps!


[WIP] [infra-proxy-service] Add infra-proxy-service & manage servers & orgs APIs

:nut_and_bolt: Description: What code changed, and why?
Implement a new service that is managing the chef servers ends points and teach how to fetch data of the desired state.
As of now, we are dealing with current state data of chef node which might fails and loss of data if the particular node/server fails during convergence.
In order to make a single source of truth for chef infra servers data here, we have added infra-proxy-service service.
In this PR following points I have covered:

Connect infra-proxy-service to deployment-service.
Provide gPRC and HTTP endpoints to manage chef infra servers & orgs.
CRUD for infra servers.
CRUD for infra server orgs.

Pending tasks:

Add IAM policies for admin users only.
Manage infra server user's KEYS using secrets-service.
Add Unit & integration test cases.
Documentations of infra-proxy-service

As I am a newbie to Automate and I have tried to cover as much as possible in order to get the understating of Automate code flow.
Please reviews and do provide the feedback.
Also, I am very thankful to @stevendanna  who guided me with proceedings.
Thanks

:chains: Related Resources
https://github.com/chef/automate/issues/1544
:+1: Definition of Done
Added infra-proxy-service a separate service in order to manage chef infra servers endpoints.
:athletic_shoe: How to Build and Test the Change

build components/infra-proxy-service
rebuild components/automate-gateway
rebuild components/automate-deployment && start_all_services
run chef-automate dev grpcurl automate-gateway list chef.automate.api.infra_proxy.InfraProxy in order to get list of implemeted gRPC APIs.

:white_check_mark: Checklist

[ ] Tests added/updated?
[ ] Docs added/updated?

Attached DB structure for reference:

https://github.com/chef/automate/pull/2189