Additional tag features

View additional tags implementation
To do

[x] show all feature
[ ] Align the tooltip (https://github.com/carbon-design-system/carbon/pull/7309)

Link to design
Maintainer: Marcie Coral
Catalog number: 1

@carrenelloyd Would you know if there's redlines or existing code available for this?

@marciecorral initial dev review.

In essence, the overflow part is a tooltip. The only difference is the change to Gray90
Gray90 is not a Carbon theme token and should NOT be used see https://www.carbondesignsystem.com/guidelines/color/usage
$interactive-02: Gray 80 is a better choice as the current tooltip background and the high contrast tooltip.
Hover is a poor interaction which does not work on well for interactive elements such as this one. Carbon defines the clickable interactive tooltip for this scenario.
What happens after the modal is closed?
4.1. A keyboard user may expect the focus to return to the +tag or list.
4.2. Is the list still open?
Should it support the tooltip positional settings?
The modal is ugly, would it not be better to present the tags as tags in this list. These could be presented many to a line.
If there are enough to cause the modal to scroll should it be filterable?
How narrow should it go? Should we assume it may not even fit the first tag https://codesandbox.io/s/brave-joliot-expkd
It is not true that accessibility is unchanged we need to ensure we deal with hidden information.
$link-01 is blue 60 and this color should be used for the link as it is the default carbon color.

Developer notes: The Carbon tabs now implements a feature where it assesses whether enough space is available. The same technique may be applicable here.

This is an item that was designed in 2019 by a designer who is no longer here at IBM. This was never implemented in CP4MCM. We can discuss details on the call tomorrow.

InVision prototype
https://ibm.invisionapp.com/share/ANO0H9RHEGK#/screens/319708255_AdditionalTagFeature_1

@Levinson1 the questions asked above need discussing before this is ready for production.

@lee-chase

Yes, this is essentially a tool tip. However my file is showing the tool tip to be Gray 80 (#393939).
and 3. I am not sure where the disconnect is happening, but my file for this component is showing the tool tip and the tag on hover as Gray 80 or #393939.
The original designer and myself thought that hover would be the best interaction for this situation. Why do you believe this interaction is not the best option?
Not sure what you're asking. Could you clarify? The modal will work like a typical modal.
When the modal is closed, the focus will return to the "View all labels" link within the tool tip.
When the modal is closed, the tool tip is still open and the focus returns to the last place of focus before the modal was opened. Which in this case, is the "View all labels" link.
Can you clarify?
The decision to have the tags in a list was because some labels can be too long for a tag and so we use an ellipses. With this list view there are no excessively long tags.
Do you mean, should we include a search bar in the modal? The tags in the modal are read only with no capabilities for selecting.
The Additional Tag should only be as narrow as the plus sign and number within (ex. +1). Any other tags should only be as long as the label within.
I addressed your feedback about accessibility in points 6 and 7. Is there another issue of accessibility you are referring to?
Since the link text is on Gray 80 (#393939), I used the link component from Gray 90 theme. The link text should be Blue 40 or #78A9FF in order to meet accessibility.

@CourtneyBanning

& 2. It may be the case that gray 90 was used at some point. The carbon themes for Gray 90 and Gray 100 do use Gray 90 but this is not currently used in White and Gray 10 themes.

Hover does not move focus, unlike click/touch. As a result, accessing the interactive part of the element via the keyboard is hard to achieve. Carbon uses focus for all overflow menu and interactive tooltip scenarios.
4., 5 & 6 we seem to be mixing numbers and you have answered 4 for me.

(My 5) Tooltips support positions: top, left, bottom, right and alignment of start, centre, end. Should component support these?

(My 6) Yes, but the modal shows plain text instead of tags which would be I think more aesthetically pleasing. I'm not a visual designer so feel free to shoot me down on this. Another option is to use the Carbon list component.

(My 7) Yes just thought based on the list appearing to be scrollable in the design (fades out towards bottom).

(My 8) Agreed but is there a minimum width for the set of tabs? If the first tag is 200px wide and we only have 200px for instance what doe we display?

(My 9) Yes, sorry for not being clear or polite ;-). The tooltip (non-interactive) provides the assistive text for screen readers, the interactive tooltip has a label. At the very least we need to have assistive text along the lines of "Additional tag list" or "Shows 5 additional tags"

You are correct this is represented by $inverse-link in white and gray 10 themes.

Bonus question. Tags can be different colors, the design only shows the high contrast which can be seen on https://react.carbondesignsystem.com/?path=/story/tag--default by changing the setting in the 'Knobs' tab. Is it OK to assume the additional tags tag uses high-contrast.

Bonus question 2. The standard Tag component has a filter property, is this permitted in sets of tags used in this way?

P.S. I believe we should try to stick to Carbon theme color tokens where every possible as opposed to the underlying Carbon colors. https://www.carbondesignsystem.com/guidelines/color/usage

@lee-chase
Sorry about the delay in response! You brought up some great points and I am re-evaluating aspects of this component. I will get back to you as soon as it is sorted.

Peter Mills (IBM) posted an existing implementation example on internal slack. He is investigating the possibility of open sourcing the component.
https://files.slack.com/files-pri/T27SFGS2W-F01DE57RS3H/action_tag_list.gif

NOTE:

Not yet added the show all feature.
Raised a PR to carbon regarding alignment of the tooltip https://github.com/carbon-design-system/carbon/pull/7309

Updates required by #59

Just updated the original issue with updates to the pattern.
@lee-chase many of your concerns are addressed with this updated version. thanks for the feedback!

@CourtneyBanning the current design extracts text from the tag, which is technically problematic due to the way React works and Carbon tags not requiring text.
The following is valid
    <Tag type="red" title="Clear Filter">
      <Warning16 />
      <Warning16 />
      <Warning16 />
      <Warning16 />
    </Tag>

The design also precludes the use of filter tags.
This implementation https://ibm-cloud-cognitive.netlify.app/?path=/story/experimental-tagset--many-tags produces the following

@CourtneyBanning  please review the current tags with overflow implementation. It currently does not display a simple text in the overflow pop up as this is not compatible with the tag implementation. We could use the contents of the tag but that can be anything, such as an icon.
Note 1: The next version of Carbon is required to align the popup, which should be in the next few days.
Note 2: There appears to be an issue in Carbons floating menu positioning. I will confirm on next release and raise a bug.
https://ibm-cloud-cognitive.netlify.app/?path=/story/experimental-tagset--many-tags

@CourtneyBanning has asked the overflow is changed to reflect the content of the tags, which will likely be text.
Noting that the design will need to document; Text only (Carbon allows anything), filter and disabled should not be used.

Shared by @CourtneyBanning in slack.

Requested clarification/noted in slack by @lee-chase
Standard Carbon modal widths are 24%, 36%, 48%, and 72%.
Do we really want a width of 640px when space is available? 640 is approx 38% width of a macbook pro screen or 36% which is the 'md' sized modal. It is worth noting that the Create Modal which uses size 36%.

md: 36% width agreed

@courtney-banning has given her approval of the implementation with PR #774
Note: Courtney has asked for the above github handle to be used.


Incorrect arch type does not clean up

If you neglect to change the qe.arch to the correct type (eg. x86), it will hang on the ssh auth and leave the qemu-system process in an uninterruptible state (does not respond to sighup).
Cleanup should look for and terminate residual processes on exit.

Thanks, you are right, this may need a better stop or destroy implementation.


Trying to use old nonexistent kernel

Bug: Jupyter tries to run with an old nonexistent kernel
Steps to cause the bug to occur
I'm not sure how to reproduce this setup, and if I knew I would probably be able to solve this myself.

Open any notebook
Set the environment to a pipenv virtualenv (3.8.3) with jupyter installed
Run the notebook

Actual behavior
The following error is received:

Note: This is probably a version that was installed, but I upgraded long ago. Also, Until recently every time I tried to run a notebook I received a message about it failing to use 3.6.1, but it prompted me to choose another kernel, and after that it worked.
Expected behavior
The notebook should run with the 3.8.3 kernel.
Your Jupyter and/or Python environment

Jupyter server running: Local
Extension version: 2020.8.101144
VS Code version: 1.48.0
Setting python.jediEnabled: Can't find this setting
Setting python.languageServer: Microsoft
Python and/or Anaconda version: python 3.8.3
OS: Windows 10
Virtual environment: pipenv

Developer Tools Console Output
-1597409255239.log
What I've Tried

Reinstalling the extension
Recreating the virtualenv (after removing Pipfile.lock)
Reopening VS code (multiple times)
Creating a new notebook
Removing paths to python3.6 from my PATH environment variable
Running jupyter outside vs code with the same notebooks (this works)

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer, @joyceerhl

This may be related to a bug I am having where the Jupyter kernel is not using my specified anaconda envirionment. Searching around this appears to be because of the "kernelspec" entry in the Jupyter notebook metadata, which I'm guessing was created when I was running the notebook previously. Manually opening the notebook and manually deleting the kernelspec json entry (located near the bottom of the notebook raw text) caused vscode to correctly open the notebook in the currently activated anaconda environment. Perhaps you could try that?

I tried deleting the entire Metadata section of an existing notebook as well as opening a new one and it didn't solve the problem.
Also, the notebook does run when using jupyter without vs code.

I've just noticed that jupyter lets me choose the old version for creating a new notebook:

Hi @dan1994,
Thanks for providing the logs and troubleshooting steps you've taken. Will have a dev take a look as soon as possible!

Thank you! Keep up the amazing work!


Sanitize field values for csv format

currently when outputting to csv some values might break csv because they have the delimiter character in them

do you have an example? I tried it with the project itself (node ./index.js --output=csv) and the output was fine.
But anyway: in case of problems you can change the delimiter anytime using the method described in the readme (see different output).

The most common delimiter which is , appeared for me once in the author field, as a result there was an extra column in that row. In my opinion it's always good to insure that protocols and formats are enforced... prevents a lot of nasty surprises

All very valid points!
We can start by throwing an error or just printing a warning that a delimiter character appears in a value and then maybe provide flags to do something with it (like escaping etc...)
wdyt?

good idea. Should I make a first draft of a solution as a pullrequest? Perhaps I find the time on the weekend?

That would be awesome.
On Wed, Oct 14, 2020, 07:31 BePo65<EMAIL_ADDRESS>wrote:

good idea. Should I make a first draft of a solution as a pullrequest?
Perhaps I find the time on the weekend?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ironSource/license-report/issues/31#issuecomment-708149595,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABD7J4R7656Q6L3COOGIWTSKUSRVANCNFSM4SKMAFKQ
.


【QUES】Padding ???

Obviously, only fixed-length sequences are used here, and there is no filling operation, which limits the scope of application of this model.

The "padding" is done while preprocessing the data. We explode the full ordered list of each user ratings into multiple subsequences. Let me illustrate:
# Example: Transform
# +---+------------+
# | id|  movie     |
# +---+------------+
# |  2|[1, 2, 3, 4]|
# +---+------------+
# Into
# +---+------+------+
# | id|  past|target|
# +---+------+------+
# |  2|[0, 0]|     1|
# |  2|[0, 1]|     2|
# |  2|[1, 2]|     3|
# |  2|[2, 3]|     4|
# +---+------+------+

We are using each movie on the sequence as a target with its subsequent past, padding with 0s if past is missing.
We reserve the latest window of the sequence 2, [2.3] ,4] as validation as is the latest known step of the user (closest to actual time).
There is no application limit of this model as you are not losing any datapoints, once you are using it for inference you can use any sequence length assuming your batch_size is 1.

Thanks a lot ! You used a third-party library in the model part to build the transformer module.. But will the transformer automatically ignore the loss value caused by padding?


Release 1.4.0

What it does

This pull request upgrades Refiner iOS and Android SDK versions to 1.4.0

How to test it

Run the example app
See the survey shows up successfully

Acceptance criteria

No issues appears during project build
Example app runs successfully

Reviewed & approved


Improve E2E testing

Current E2E testing is not good:

Tests are often broken in master since they are not run in CI
They are not run in CI to improve times and stability

Proposal:

Use a Docker image of Windshaft (the Windshaft team already has one) as a base image
Create a new docker image by combining that image with some fixed datasets for testing
Use that image to run a local Windshaft server and use that server as the target of E2E tests
Enable E2E tests on CI

cc: @jgoizueta @dgaubert @rochoa

Fixed by #744


Opening the dashboard of an event shows "0" tickets sold and attendees checked in event though they have been

While the app download data from the web the dashboard shows "0".
Expected: Show a loading icon instead of "0"

Can be closed


Hyperion-project.org website and wiki missing

[x] I confirm that this is an issue rather than a question.

Bug report
The contents of the hyperion-project.org website, the wiki and the forum seems to be missing.
Steps to reproduce
Open https://wiki.hyperion-project.org/ in a browser
What is expected?
The project wiki
What is actually happening?
It displays a "Plesk" default web site
System

https://hyperion-project.org
https://docs.hyperion-project.org/

Thanks! Good news that a new website is available. I appreciate the work.
The problem is that quite a lot of google search results and posts within the forum itself find links in the "old" format:
e.g.: https://hyperion-project.org/threads/mein-4k-setup-rp4-und-hyperion-ng.11142/page-8#post-34107
Whereas the "new" format looks like:
https://hyperion-project.org/forum/index.php?thread/11142-mein-4k-setup-rp4-und-hyperion-ng/&postID=34107
It would be great if a URL rewrite from the old format to the new could be implemented, so the old references still work because they redirect to the existing content in the new format.
I'd offer help in writing a block of nginx configuration according to http://nginx.org/en/docs/http/ngx_http_rewrite_module.html#rewrite if needed, but I'd need a bit of input.

Our webmaster @tpmodding  can give you more information about this.

added the wiki cname, thanks
will have a look for rewrite, @Paulchen-Panther did mentioned allready the problem with the old urls, it is on my list :-)


Evaluating a custom topic model

Is there any ways to use your package for evaluating a custom topic model? What outputs in what formats I should get from my model and use to compute the metrics?

Hello :)
yes, it is possible to evaluate  an external topic model using OCTIS metrics. You can refer to this issue for more details on how to do it and for the format to use: https://github.com/MIND-Lab/OCTIS/issues/61
Feel free to re-open the issue if you have further questions :)
Silvia


UX Research - Pilot Ministries Research with Coordinators

UX Research - Pilot Ministries Research with CoordinatorsTitle of ticket:
Description
UX Research with two Pilot Ministry Coordinators. To cover Processing Fees Design,  Questions about Desired Updates to Divisional Tracking, and Review Existing Process and Desire for Requests that have requested the same records
Research Plan and Study Guide

[x] Draft Research Plan
[x] Review and Finalize Research Plan
[x] Draft Study Guide
[x] Review and Finalize Study Guide

Book 1 hour sessions, set expectations that we may not use the full hour.

[x] Book sessions (Molly)
[x] Session with EDUC (Carrie)
[x] Session with ECON (Vicki)

Actual Sessions Completed

[ ] Session with EDUC (Carrie)
[ ] Session with ECON (Vicki)

Report Out on Sessions

[ ] Create Report Out
[ ] Present Report Out at Demo

To include in Research - Process of harms not being met by some divisions which is leading then do records review and harms at the same time and at the end which is leads to new redlines needing to be creating in IAO.
Build this into Ministry Reporting Research research

To include in Research - Research into better understanding what users needs are for tracking Requests (HTH brought up this issue)

STUDY GUIDE:  https://bcgov.sharepoint.com/:w:/t/03521/ES-HGDUthUpJnzGClceJs1kB7oq0OwmBj3CgD-jt2k7V0g?e=ksODZO

RESEARCH PLAN: https://bcgov.sharepoint.com/:w:/t/03521/EcfyCoq71ONMqrd6Es8WH7oBN3iMDj2rcPAgsCnE95u1wg?e=kSzehr

@m-prodan & @lmullane Here is a link to the new fess tab, and the undated OI section. Let me know if either of you have any questions?
https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/912622803
cc: @mpilchar @abin-aot

Thanks @JHarrietha-AOT - presented it to managers and got some good feedback which I will share with you, I expect we'll see a good turnout tomorrow so hopefully more good discussions at demo!


borderRadius and border not working for buttons variants

Hello,
I am attempting to create variant themed versions of my button, but it seems that the borderRadius and border properties in the theme.buttons object aren't being picked up by my <Button> elements.
Here is my button element:
const btn = ({ children, disabled, processing, onClick, variant, ...props }) => (
  <Button
    disabled={disabled || processing}
    onClick={onClick}
    variant={variant || 'primary'}
    {...props}
  >
    <Pica lineHeight='none'>{children}</Pica>
  </Button>
)

And my theme
// theme.js
...
 buttons: {
    primary: {
      color: colors.white,
      backgroundColor: colors.black,
      border: `1px solid ${colors.black}`,
      borderRadius: '0px',
      hover: colors.charcoal,
      '&:hover': {
        color: colors.white,
        backgroundColor: colors.charcoal,
        border: `1px solid ${colors.charcoal}`
      },
      '&:disabled': {
        backgroundColor: colors.smoke,
        border: `1px solid ${colors.smoke}`,
        color: colors.dawn
      }
    },
    secondary: {
      color: colors.black,
      backgroundColor: colors.white,
      border: `1px solid ${colors.black}`,
      '&:hover': {
        color: colors.black,
        backgroundColor: colors.charcoal,
        border: `1px solid ${colors.charcoal}`
      },
      '&:disabled': {
        backgroundColor: colors.smoke,
        border: `1px solid ${colors.smoke}`,
        color: colors.dawn
      }
    }
}

Is there something I'm missing here?

Because the Button uses the border and borderRadius props, and those were set as defaultProps, they will override the styles you have in your variants. Have a partial fix in #622 which should address the issue here


Kokkosify Shaman types and functions

Add Kokkos macros to make Shaman type portable accross Kokkos backends.
Mandatory Checks:

[ ] Compilation on CPU / gcc
[ ] Compilation on CPU / clang
[ ] Compilation on GPU / Nvidia
[ ] Compilation on GPU / AMD
[ ] Compatibility with Kokkos Views

Should be tested with at least examples from #1

Should require #11


Feature proposal: trim for the bar chart labels

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version 0.20.1
Sometimes users have labels that are quite long.
Would this kind of trim be a useful feature for superset?
I would start a PR, but just want to know your opinion first.
Expected results

Actual results

Maybe ut's easier to do that as sql expression, i think it would be quite hard to please everyone.

@xrmx it might be that after trim in SQL several different categories will end up in one.
e.g. abcde1 and abcde2 gonna be trimmed to abcde.
An alternative could be just to hide those labels, tooltips would be sufficient.

horizontal bar charts handle variable-length text much better than vertical bar charts.

@williaster can't find those. Where can I configure the layout of a barchart?

currently horizontal bars are not supported, I just meant as a solution horizontal bars will likely be better than other options with vertical bar labels because there are only so many options (truncate or rotate).
we are ramping up to do a big pass at visual design consistency across charts and will likely add this at that time.


Encode Buffers more efficiently

Currently buffers are being encoded by a simple JSON.stringify here.
This results in values being stored as arrays. Instead, I recommend base64 encoding buffers.
{"type":"Buffer","data":[248,120,149,32,219,219,219,44,189,35,183,131,116,30,141,127,207,81,228,89,180,151,228,166,184,96,248,94,128,154,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,160,86,232,31,23,27,204,85,166,255,131,69,230,146,192,248,110,91,72,224,27,153,108,173,192,1,98,47,181,227,99,180,33,160,197,210,70,1,134,247,35,60,146,126,125,178,220,199,3,192,229,0,182,83,202,130,39,59,123,250,216,4,93,133,164,112]}

{"type":"Buffer","data":"+HiVINvb2yy9I7eDdB6Nf89R5Fm0l+SmuGD4XoCaAQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAACgVugfFxvMVab/g0XmksD4bltI4BuZbK3AAWIvteNjtCGgxdJGAYb3IzySfn2y3McDwOUAtlPKgic7e/rYBF2FpHA="}

efficiently in what way though, this would make it shorter but it would be additional processing steps to base64 encode it, my vote would be for just utf8 string but I'd need to check how well the different back ends can support it.  Binary might be an option for postgres as well, though I don't know about mysql, you tend not to be able to have nice things with mysql

Here's some doc on msql blob types, don't know how prevalent this is.
http://dev.mysql.com/doc/refman/5.0/en/blob.html
And psql
http://www.postgresql.org/docs/7.4/static/jdbc-binary-data.html

small perf test shows utf8/base64 to be about equal for random binary data
encode hex:    4.0700430000000005e-8
decode hex:    0.0000019190725900000003
--------------
encode utf8:   1.70694e-8
decode utf8:   0.00000170014191
--------------
encode base64: 1.687998e-8
decode base64: 0.0000017771793000000002

test src:

console.log( 'encode hex:    '+testN(encodeTest, getRandomBuffer, 'hex', 100000) )
console.log( 'decode hex:    '+testN(decodeTest, getRandomData, 'hex', 100000) )
console.log( '--------------' )
console.log( 'encode utf8:   '+testN(encodeTest, getRandomBuffer, 'utf8', 100000) )
console.log( 'decode utf8:   '+testN(decodeTest, getRandomData, 'utf8', 100000) )
console.log( '--------------' )
console.log( 'encode base64: '+testN(encodeTest, getRandomBuffer, 'base64', 100000) )
console.log( 'decode base64: '+testN(decodeTest, getRandomData, 'base64', 100000) )

function testN(test, generateData, encoding, count){

  var testBuffer = generateData(32, encoding)

  var start = process.hrtime()
  for (var i = 0; i<count; i++) {
    test(encoding)
  }
  var duration = process.hrtime(start)

  var total = duration[0] + duration[1] * 1e-9
  var average = total / count

  return average

}

function encodeTest(testBuffer, encoding){

  var output = testBuffer.toString(encoding)
  return output

}

function decodeTest(testData, encoding){

  var output = Buffer(testData, encoding)
  return output

}

function getRandomBuffer (howMany) {
  var fs= require('fs');
  var bytes= new Buffer(howMany);
  var fd= fs.openSync('/dev/random', 'r');
  fs.readSync(fd, bytes, 0, howMany, null);
  fs.closeSync(fd)
  return bytes;
}

function getRandomData(howMany, encoding) {
  return getRandomBuffer(howMany).toString(encoding)
}

:warning:  wait the above perf tests are broken :warning:
fixing...

Fixed perf tests. Number is average time for single en/decode operation. src
encode hex:    0.0000004114
decode hex:    0.0000015450
--------------
encode utf8:   0.0000004258
decode utf8:   0.0000022681
--------------
encode base64: 0.0000003849
decode base64: 0.0000029669
--------------
encode json:   0.0000032028
decode json:   0.0000016718

JSON.stringify is 10x slower than buffer.toString('base64')

I remember now from a crypto project that binary will not correctly roundtrip as utf8.  Settled on using base64.

utf8 does not roundtrip
var buf = getRandomBuffer(33)
console.log(buf.length) //=> 33
console.log(Buffer(buf.toString('utf8'), 'utf8').length) //=> 61

var buf = getRandomBuffer(33)
console.log(buf.length) //=> 33
console.log(Buffer(buf.toString('base64'), 'base64').length) //=> 33

this has been done


Add Snake activation function

The Snake activation function is missed in KotlinDL.
Read more about Snake
The desired PR addressing this issue should include:

[ ]  Implementation of activation class named as SnakeActivation (you can take inspiration from the implementation of HardSigmoid  as reference) added to the Activations.kt file
[ ]  Documentation of activation function
[ ]  JUnit tests in api  module
[ ]  Support for export of activation function to JSON (see ModelSaver.kt)
[ ]  Support for import of activation function from JSON (see ModelLoader.kt)

P.S. If you want to take this ticket, please leave the comment below
P.P.S Read the Contributing Guidelines.
The reference implementation could be taken from tensorflow-addons

I would like to work on this issue.

Will wait for your PR @cagriyildirimR !


publish wheels

It is better for the ecosystem for package owners to publish wheels as well as source distributions - else everyone who installs your package must do it themselves.  That is slower, and might conceivably go wrong.  Better for the publisher to build the wheel once and for all.
I did not find a release pipeline in this repository or I would have submitted a pull request.
Since you are using poetry: I believe that poetry build and/or poetry publish will build and publish both wheels and sdists by default.

Thanks for the feedback! I'll take a look at adding that to the release process (and/or sorting a pipeline out).


Forced Mocha.js as direct script include

Apparently, the mocha.js script needs to be included as a separate resource.
See https://github.com/nathanboktae/mocha-phantomjs-core/blob/master/mocha-phantomjs-core.js#L123
I find this quite limiting since most of the times the JS libs are simply aggregated and served as a single .js file. Also I don't get the rationale.

Why would you need to bundle a test framework with your tests or product code? Some default webpack thing? I concat and minify my product code and run the tests on that, then the bundle as is is what is shipped out.
There are few settings like slow, timeout, etc that must be set before after Mocha loads but before the tests does, as they get replicated per test. This was a change in mocha and broke those settings for mocha-phantomjs 3.x and earlier as it would set them when mochaPhantomJS.run was called, but it was too late.
Currently I am only conditionally exposing initMochaPhantomJS, but I could always expose it and you'll have to explicitly call it before calling any mocha setup functions.

Hi @nathanboktae and thanks for your reply.
In my current setup (not sure it's the best possible, though), I run
mocha-phantomjs http://localhost/test.html

In test.html, I load the test-libs.js (which contains all the vendor libraries only used for testing, including mocha.js) and tests.js (the actual mocha tests code) files. As you see, it's a not so uncommon scenario.
Mocha-phantomjs 3.x works just fine while 4.x doesn't due to the mocha.js single include failure.

I load the test-libs.js (which contains all the vendor libraries only used for testing, including mocha.js)

Why? Most people use npm or bower and just include them. Your scenario is not common.
I explained problems that existed in 3.x and a workaround... Any responses to that?

Why? Most people use npm or bower and just include them. Your scenario is not common.
I also handle frontend libraries via bower but then use gulp to minify them (I have dozens) in order to speed up loading time. I don't see what's wrong with this approach.

Regarding the 3.x version usage, I simply have this
if (window.mochaPhantomJS) {
    // do the mocha setup load the tests and then
    mochaPhantomJS.run();
}

I use bower and gulp too, but bundling test dependiea with production dependencies make no sense. Tests run locally not over the internet.
You still didn't answer my first question and have repeated yourself thrice. I know how 3.x works.

Of course, I don't ship testing dependencies out but there are libraries, for instance Moment.js, used both for testing and production. So I end up with libs.js and test-libs.js with only the former being delivered.

So not only are you wasting your time bundling test dependicies, now your stack traces are total garbage as phantomjs doesn't support source maps. Bad idea.
Still I can always expose initMochaPhantomJS so you can call it from the appropriate time in your bundle

Thanks for the advice. Glad we come up with a solution in the end.


test: verify witnesses and multi-update proofs

I was surprised this worked without major debugging, but it works as intended.

#127  👈
master

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

Join @rphmeier and the rest of your teammates on  Graphite


bug: target file does not create a file when forcecreate: true

Describe the bug
When defining a target of type file with the specified not existing AND the attribute forcecreate specified to the value true,
updatecli fails with the following error:
ERROR: Something went wrong in target "xxxxxx" :
ERROR: Validation error: the provided manifest configuration had the following validation errors:
Invalid spec for yaml resource: the file "yyyyyyyy" does not exist.

Steps to Reproduce
Lucky us, there is already a failing example in the repo 🤗
go build -o ./dist/updatecli && ./dist/updatecli diff --config examples/updatecli.generic/file.yaml

Expected behavior

No error
File created (from empty content to source input value)

Screenshots
N.A.
Version

OS: all
Updatecli version: 0.17.2

Additional context
N.A.

Adding to 0.18.0 milestone as the fix is quick and easy (and PR already there)


avoidable typecasts

With the discussion on #465 about efficiency: there are a lot of cases in Mafia where something is taken, converted to an array, then iterated over. Would it be possible to avoid the array conversion in most cases (i.e. when not deleting from the iterated list)? For example, could
String[] keys = map.keySet().toArray(new String[0]);
for (String name : keys) {

be
for (String name : map.keySet()) {

I understand you've been trying for minimal changes to increase the chance of these being accepted, but might these perf improvements (if they are indeed improvements) be worth going for additionally?


Alex's Diverged work from master

This is the additional work from @GildedHonour on top of master, I am extracting it as a separate PR so once we restore the missing commits then we can update this code and merge as necessary.
Ref: https://github.com/riboseinc/asciidoctor-rfc/issues/6

@abunashir none of these was ready for a review -- see the "wip" in the commit message.

@GildedHonour: Yes, HoundCI will automatically do the style check on every PR and comment when necessary.


Code uses gethostbyname() which is deprecated

Packaging libe131 for openSUSE linux and this warning comes out:
The binary calls gethostbyname(). Please port the code to use getaddrinfo().

Here's a better explanation on why.

Apologies for the really late reply and thanks for bringing this up.
I now made a basic implementation in e8199d6.


improve logging for (premis) export in misconfigured project

Currently the export fails in case SDB and OBJECTS filegroups are configured. This is logged to a logfile/cloudwatch.
Additionally this should be sent to the process log to be easily visible for users.

:+1:


Args of hook methods reverted

In the documentation the arguments of the methods  infinityModelUpdated and infinityModelLoaded are reverted.

@jevanlingen Thanks for PR!

Thanks @jevanlingen  Looks like we might need to allow fail on canary for this to pass.  Let me get something up there after digging into the problem so we can get this merged in!

@jevanlingen Still interested in getting this one in?  Rebase with master and we can merge.

Fixed this in the README.  Thanks for pointing this out!


Adress Public API review comments

This addresses a bunch of comments that came out of the Public API review process

/azp run

/azp run


The paper is missing from https://paperswithcode.com/sota/code-completion-on-dotprompts

Also model names are duplicate as well :X

Still there yet...


[Bug] Can't access hidden channels after zoom in/out

When I zoom in/out I'm not able to access hidden channels. I need to logout + login to have this back.
Windows 10

@pavlovp
We are not able to repro this issue at our end. It works fine for us. Also we are responsible for teams development related queries. Adding the teams product tag.


TrackingEventProcessors shutdown is not working correctly in 4.5.13

Hi,
We switched from Axon 4.5.9 to 4.5.13 due to issue #1981. However, we noticed that the shutdown of the TrackingEventProcessors doesn't seem to work as expected anymore.
Basic information

Axon Framework version: 4.5.13
JDK version:  11

Steps to reproduce
I use almost the same demo program as for #1981:
@SpringBootApplication
public class DemoApplication {

	public static void main( final String[] args ) {
		final var applicationContext = new SpringApplicationBuilder( DemoApplication.class )
				.properties( "axon.axonserver.enabled=false" )
				.run( args );

		final var eventGateway = applicationContext.getBean( EventGateway.class );
		eventGateway.publish( 0 );

		await( ).until( ( ) -> MyEventHandler.elementProcessed.get( ) );

		applicationContext.close( );
	}

	@Autowired
	private void configure( final Configurer configurer ) {
		configurer.configureLifecyclePhaseTimeout( 30, TimeUnit.SECONDS );
	}

	@Autowired
	private void configure( final EventProcessingConfigurer configurer ) {
		configurer.registerTrackingEventProcessorConfiguration( MyEventHandler.PROCESSING_GROUP, c -> TrackingEventProcessorConfiguration
				.forParallelProcessing( 4 )
				.andEventAvailabilityTimeout( 20, TimeUnit.SECONDS ) );
	}

	@Component
	@ProcessingGroup( MyEventHandler.PROCESSING_GROUP )
	static class MyEventHandler {

		public static final String PROCESSING_GROUP = "MyEventHandler";

		private static AtomicBoolean elementProcessed = new AtomicBoolean( );

		@EventHandler
		public void on( final Integer i ) {
			elementProcessed.set( true );
		}

	}

}

Notice that I configured the lifecycle phase timeout. What happens is, that the timeout in DefaultConfigurer.invokeLifecycleHandlers is not exceeded (no TimeoutException occurs). But we still get a lot of exceptions afterwards in the log, as the event processor is still running after the database has been shut down. My workaround from #1981 won't work anymore as well, as it basically does the same thing as the modified code in DefaultConfigurer.invokeLifecycleHandlers.
We also checked the same code with 4.6.0-SNAPSHOT (the current master at f496fed2155d58362597a1f0f7ac33906432af73) - in this case the exceptions do not occur and the event processor is correctly stopped.
We are not entirely sure if this is an existing bug or maybe the code which was merged for #1981 relies on some other code on the master.
Expected behaviour
The event processor is no longer running after the shutdown lifecycle handlers have run.
Actual behaviour
The event processor is still running after the shutdown lifecycle handlers have run which leads to exceptions.

Hey Nils, curious, but did you test out whether the shutdown process worked with 4.6.0-SNAPSHOT?
In favor of getting 4.6.0 done (instead of again moving to patch releases) I'd like to rule out any effort of moving through commits from a half year ago to figure out where the discrepancy lies; if any at all.
If you could validate for me whether 4.6.0-SNAPSHOT does solve the predicament, that'll safe us a ton of time on our end.
Thanks!

Awesome, thanks for the confirmation, @nils-christian.

I have a hunch I found the predicament.
One of the commits that belonged to the TEP shutdown wasn't carrying the issues number.
Hence I missed it.
I believe this to be commit https://github.com/AxonFramework/AxonFramework/commit/696d8f5938bfbec2d7edba8fecbead0b8d4f5eb8, which I've just cherry-picked to axon-4.5.x.
The commit checks whether the TEP threads have stopped during shutdown and if that didn't happen, they're interrupted.
I'll start making a 4.5.14-SNAPSHOT soon.
Once it's present, would you be able to validate whether that solves the predicament, @nils-christian?

Hi @smcvb,
I just built f6e33b58fa36a0070af9d8271d8147c9fc4e44c2 locally and it looks much better. The shutdown is performed correctly. I now get something like the following:

2022-07-14 10:18:07.958  INFO 23468 --- [           main] o.a.e.TrackingEventProcessor             : Shutdown state set for Processor 'MyEventHandler'.
2022-07-14 10:18:07.958  INFO 23468 --- [           main] o.a.e.TrackingEventProcessor             : Processor 'MyEventHandler' awaiting termination...
2022-07-14 10:18:12.983  INFO 23468 --- [EventHandler]-3] o.a.e.TrackingEventProcessor             : Released claim
2022-07-14 10:18:12.983  INFO 23468 --- [EventHandler]-2] o.a.e.TrackingEventProcessor             : Released claim
2022-07-14 10:18:12.983  INFO 23468 --- [EventHandler]-0] o.a.e.TrackingEventProcessor             : Released claim
2022-07-14 10:18:12.983  INFO 23468 --- [EventHandler]-1] o.a.e.TrackingEventProcessor             : Released claim
2022-07-14 10:18:12.984  INFO 23468 --- [EventHandler]-0] o.a.e.TrackingEventProcessor             : Worker for segment Segment[3/3] stopped.
2022-07-14 10:18:12.984  INFO 23468 --- [EventHandler]-2] o.a.e.TrackingEventProcessor             : Worker for segment Segment[1/3] stopped.
2022-07-14 10:18:12.984  INFO 23468 --- [EventHandler]-1] o.a.e.TrackingEventProcessor             : Worker for segment Segment[0/3] stopped.
2022-07-14 10:18:12.984  INFO 23468 --- [EventHandler]-3] o.a.e.TrackingEventProcessor             : Worker for segment Segment[2/3] stopped.
2022-07-14 10:18:12.987  INFO 23468 --- [           main] j.LocalContainerEntityManagerFactoryBean : Closing JPA EntityManagerFactory for persistence unit 'default'
2022-07-14 10:18:12.987  INFO 23468 --- [           main] .SchemaDropperImpl$DelayedDropActionImpl : HHH000477: Starting delayed evictData of schema as part of SessionFactory shut-down'
2022-07-14 10:18:12.994  INFO 23468 --- [           main] com.zaxxer.hikari.HikariDataSource       : HikariPool-1 - Shutdown initiated...
2022-07-14 10:18:12.996  INFO 23468 --- [           main] com.zaxxer.hikari.HikariDataSource       : HikariPool-1 - Shutdown completed.

No exceptions occur afterwards.

That's great news.
That means we likely have caught the issue.
Honestly, I'm happy we found it as quick as we did :sweat_smile:.

So, how do you feel, @nils-christian.
Shall I close this issue as resolved because of cherry picking https://github.com/AxonFramework/AxonFramework/commit/696d8f5938bfbec2d7edba8fecbead0b8d4f5eb8?

I am happy too. As you know we are looking very forward to the fix for #1981 :-)
Yes, we can close this issue.

So, 4.5.14 has been released. Be sure to let me know if any other issues pop up, @nils-christian!


[Bug] ImageStatus and ListImages CRI return an error image info

Ⅰ. Issue Description
When defines a container runtime user name in image, e.g.

Kubelet will read and parse it from CRI ImageStatus interface , and pass runtime user to createContainer CRI.
In kubelet logic , first choose use uid if it is not nil, then we choose user name if it is not empty,

But in pouch CRI logic, ImageStatus and ListImages interface call imageToCriImage, which it will initialize Uid as runtime.Int64Value{}, so in any case, the Uid will not be nil

So if user define user in image, it can't work as user expect.
Ⅲ. Describe what you expected to happen
Return real uid and username in image info
Ⅵ. Envronment:

pouch version (use pouch version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

@changyaowei Look forward your PR 😆


Adding instruction for use with React projects.

In the Index.js file.
import * as smoothscroll from 'smoothscroll-polyfill';
smoothscroll.polyfill();
Thanks for this, works great and it was exactly what I needed for a project.

it's already in the readme

Having trouble getting it to work with React + Typescript. I have it in my index.tsx but it does not seem to work when I run my app in the iOS simulator.

Having trouble getting it to work with React + Typescript. I have it in my index.tsx but it does not seem to work when I run my app in the iOS simulator.

This doesn't work for me... :(
`import React from 'react';
import ReactDOM from 'react-dom';
import './index.css';
import App from './App';
import reportWebVitals from './reportWebVitals';
import * as smoothscroll from 'smoothscroll-polyfill';
smoothscroll.polyfill();
ReactDOM.render(
<React.StrictMode>

</React.StrictMode>,
document.getElementById('root')
);
reportWebVitals();`
This is in the index.js file.
Am I doing anything wrong?!

@florian-wilisch try importing it before React, e.g.
// Polyfills
import * as smoothscroll from 'smoothscroll-polyfill';
// The rest :)
import React from 'react';
import ReactDOM from 'react-dom';
import './index.css';
import App from './App';
import reportWebVitals from './reportWebVitals';

smoothscroll.polyfill();

ReactDOM.render(
  <React.StrictMode>
    <App />
  </React.StrictMode>,
  document.getElementById('root')
);

@Dakkers , unfortunately that didn't work. :-/

@fredspivock Solution for Typescript, add in an index.tsx hook for useEffect. It needs to load first as soon as the page is rendered.
Example:
useEffect(() => {     smoothscroll.polyfill()   }, [])


Unable to change Virtual Display size. (similar to Issue #34)

I just enabled enhanced mode on my Kubuntu 18.04 VM, and it looks like it's working fine. However I selected a screen-size that's a bit too big - it cuts off the bottom part of the VM display. I looked at issue #34 which looks the same, however I can't get the "Edit Session" menu. I have tried all the options in that issue and several not in that issue, and the menu item just doesn't show up - if I could get it, that would probably solve the problem.

@sjbanner - To get the enhanced size dialog box you need to close and re-open VMConnect. This display is only shown the first time an enhanced session is made in any given VMConnect window.

Then the question would be how does one do that? I am going to guess that's shutting down the VM and closing the window (though I thought I tried that, so I will let you confirm). But from your reply, I did a bit of searching about and found an article with the powershell command 'vmconnect localhost "" /edit' which did the trick, so I am happy.
This might be a good thing to add to the FAQ.


Workflow using pyansys actions

Closes #339

Codecov Report

Merging #363 (d771928) into main (60497a5) will decrease coverage by 13.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #363       +/-   ##
===========================================
- Coverage   62.72%   49.68%   -13.05%
===========================================
  Files          15       94       +79
  Lines        1132    16602    +15470
===========================================
+ Hits          710     8248     +7538
- Misses        422     8354     +7932

Impacted Files
Coverage Δ

src/ansys/meshing/prime/internals/utils.py
35.52% <0.00%> (ø)

src/ansys/meshing/prime/autogen/deletetool.py
38.46% <0.00%> (ø)

src/ansys/meshing/prime/autogen/volumecontrol.py
64.15% <0.00%> (ø)

...c/ansys/meshing/prime/autogen/scaffolderstructs.py
48.46% <0.00%> (ø)

src/ansys/meshing/prime/autogen/igastructs.py
40.06% <0.00%> (ø)

.../ansys/meshing/prime/autogen/sizecontrolstructs.py
53.93% <0.00%> (ø)

src/ansys/meshing/prime/autogen/volumemeshtool.py
72.72% <0.00%> (ø)

...rc/ansys/meshing/prime/autogen/splittoolstructs.py
40.38% <0.00%> (ø)

src/ansys/meshing/prime/autogen/fileio.py
37.27% <0.00%> (ø)

src/ansys/meshing/prime/autogen/wrapper.py
77.50% <0.00%> (ø)

... and 69 more

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

Hi @ninad-kamat - got the workflow working. This will publish dev docs to the gh-pages branch and will start the transition to the multidocs as well. Still needs some fine tuning probably regarding coverage flags but overall the repo behaved well to the actions =) Let me know what you think, doubts etc.

Presenting workflow graph:

See also https://github.com/pyansys/pyprimemesh/actions/runs/4086124385

Hi @ninad-kamat, warnings are raised when building the docs. Eventually these should be solved. But in the meantime I am just going to ignore them since they are not blocking/critical and they already existed. Feel free to merge this PR if you are fine with it. I'm OoO on Monday. I'll get back to the multi-version documentation on Tuesday (it will be ready before 0.3.0 release, you can count on it).

@RobPasMue Looks good to me.


Publish the "Learning Paths" blogpost (Issue #2323).

DESCRIPTION
This PR fixes issue #2323
A new blog post, with the title "Layer5 Learning Paths" (written by a community member) has been added to the layer5.io website.

### Notes for Reviewers
The blogpost can be found on this path => https://layer5.io/blog/layer5-learning-paths/
Thank you!
Signed commits

[x] Yes, I signed my commits.

Thanks. Although, this blog post is to be published on layer5.io/blog.
Also, a fair warning - this draft has not been reviewed for grammar and content, and so, it has yet to be approved as ready for publishing. // @Anita-ihuman

Thanks. Although, this blog post is to be published on layer5.io/blog.
Also, a fair warning - this draft has not been reviewed for grammar and content, and so, it has yet to be approved as ready for publishing. // @Anita-ihuman

Okay,
I left dropped the draft for review, I did not see any changes, that's why I went ahead with the issue.
@NPDebs Please pause this PR until we have completely reviewed the draft.

Hi Lee. Thanks for responding so promptly!
My sincere apologies for this terrible oversight. I'm dying of mortification over here!
A new PR will be ready at the end of the day (on the right repo 😅).
Do let me know if I can contribute in any way to draft review process. Thanks! @Anita-ihuman

@Anita-ihuman Oh. That's alright.
I'm ready when you are...

@Anita-ihuman @NPDebs Closing this as per the comment by @leecalcote
@NPDebs Could you please create a PR in this https://github.com/layer5io/layer5 repo for the blog?

Sure @adithyaakrishna. 👍🏻

Awesome, thanks and looking forward to the PR 🚀 :)


Updated path for manifest file

Since the mix-manifest.json file is placed in the theme directory, this will correctly load the asset mappings.

Thanks for your contribution 👍
I'm just wondering what your project structure is? Have you moved Statamic above the webroot, as described in the docs here?
https://docs.statamic.com/knowledge-base/running-above-webroot
This is the setup I use, so my mix-manifest.json is located in /public/mix-manifest.json. It sounds like yours is located in /site/themes/theme-name/mix-manifest.json?
I think that if I merged this in then it'd break my websites, but if I leave it unchanged then it doesn't work for you! Perhaps I should change this plugin so that there are some configuration options which let you specify where your manifest file is?

Yes, Statamic is running above webroot, with my theme located in /public/themes/theme-name, which is also the public path specified in your docs.
My package.json and webpack.mix.js is located in project root, like new Laravel 5.4 sites, and my asset files are located in /site/resources/js and /site/resources/sass.
Using the latest version of Laravel Mix, the mix-manifest.json will be placed in the public path specified in webpack.mix.js.
Can you try and test it with a fresh install?
I'll have a look at some config options when my current project is done, which should be around next week.

Hi guys!  I followed the docs and this thread with the pull request but can't seem to make this work. It doesn't output the path to the assets properly. This is the href I get: "/public/site/themes/league/". Can you please give me a quick rundown on how exactly you setup things? My theme is located in /public/site/themes/theme-name. I would like to serve the assets either from within /site/themes/my-theme or /site/resources/. Thank you!


Path Complex

This pull request contains source code to implement the Path Complex based on simple paths, which is inspired from the original Path Complex.

Please add the following methods :
(1) to_other_complex()
where other_complex is any more general of path complex. e.g colored HG and combinatorial complex.
(2) restrict_to_nodes
(3) restrict to paths
(4) set attrs and get attrs (similar to https://github.com/pyt-team/TopoNetX/blob/da155dfbb46a77a9034a1d5707b573f16b93cce9/toponetx/classes/cell_complex.py#L957)
(5) add hodge_laplacian_spectrum methods in the file  :
https://github.com/pyt-team/TopoNetX/blob/main/toponetx/algorithms/spectrum.py
(6) add tutorial on path complex demonstrating the main methods.

Finally for adj/incidence/laplacian matrices please add weight parameter to make the signature consistent across the package #242.

Please add the following methods :
(1) to_other_complex()
where other_complex is any more general of path complex. e.g colored HG and combinatorial complex.
(2) restrict_to_nodes
(3) restrict to paths
(4) set attrs and get attrs (similar to
https://github.com/pyt-team/TopoNetX/blob/da155dfbb46a77a9034a1d5707b573f16b93cce9/toponetx/classes/cell_complex.py#L957
)
(5) add hodge_laplacian_spectrum methods in the file :
https://github.com/pyt-team/TopoNetX/blob/main/toponetx/algorithms/spectrum.py
(6) add tutorial on path complex demonstrating the main methods.

Regarding (1), the current implementation of CombinatorialComplex prevents me from generalizing PathComplex, because paths take into account order sequence, while CombinatorialComplex converts all inputs to frozenset.
Besides that, all of the above methods are implemented. I am working on (6), which will take a while. \

@ffl096 can you please take a look at this one when you get a chance, give your comments ? Thanks!

@ninamiolane any high level comments?

@quang-truong we are aiming to publish the package soon and the publisher requires almost 100% code coverage, can you try to increase the code coverage of path complex to its maximal ? thank you


[docs] Usage example missing for torchvision.utils.draw_bounding_boxes

https://pytorch.org/vision/master/utils.html?highlight=draw_bounding_boxes#torchvision.utils.draw_bounding_boxes
Return type is not described.
Also, it says:  If filled, Resulting Tensor should be saved as PNG image.. What does word filled refer to? Where would it be saved? To some file path? Or encoded into the output tensor?

Nice points @vadimkantorov . We recently added these utils and they are bit on the rough edge.
We have these TODO points in #3364 . I wanted to add this PR after #3330 . @datumbox  Since it will take time to add segmentation mask utility. Should I quickly patch #3364 ?

@oke-aditya Apologies for the delay. I'm currently OOO...
@fmassa Thoughts on how we could unblock Oke on his open PR or how to proceed?


wsn transport is reacting incorrectly to case where maximum ws message size is exceeded

As @Anisotrop remarked in comments in https://github.com/kaazing/gateway/pull/445, the server (wsn transport) continues to read incoming data after detecting that the incoming WebSocket frame size exceeds the configured limit. This results in further protocol decoder errors and is also in violation of RFC 6455, which states (in section 7.1.7 Fail the WebSocket connection
An endpoint MUST NOT continue to attempt to
   process data (including a responding Close frame) from the remote
   endpoint after being instructed to _Fail the WebSocket Connection_.

and section 10.4.  Implementation-Specific Limits implies that when the limit is exceeded the server should fail the WebSocket connection.
To reproduce:

Alter test LimitsMaxSizeDefaultIT by adding a throw instruction as follows:

    public void shouldRefuseBinaryFrameWithPayloadLengthExceeding128KiB() throws Exception {
        k3po.finish();
        throw new Exception("TEMP");
    }

Run the test from Eclipse, or the command line:
mvn verify -DskipTests  -Dit.test=LimitsMaxSizeDefaultIT
The throw causes trace level of messages to be printed out (thanks to the MethodExecutionTrace rule). These show numerous protocol decoder exceptions after the detection of the message size exceeded exception, indicating that the server is continuing to read incoming data in violation of the spec.

@cmebarrow It seems like you already have the fix for this bug based on your comment. You can post the PR here and close this issue, when you are back :)

Fix is in my branch gateway_issue_463

This was fixed by PR https://github.com/kaazing/gateway/pull/466 which was merged on 28-Mar-16


Don't claim to support ESM

Clipboard.js doesn't support ESM, its package.json shouldn't claim it does.
Reference: https://github.com/rollup/rollup/wiki/pkg.module

Thanks @realityking!


Building Library\Bee\artifacts\Android\d8kzr\libil2cpp.so failed

Yesterday I was able to build the APK normally, but today I cannot. Even if I rolled back the code to yesterday, it will not help. How can I solve this problem？？？

have you solve it, bud

have you solve it, bud

yes, try this
remove project Library folder and open unity again


NIFI-9336 - Show icon for property values with leading or trailing whitespace

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Jira: https://issues.apache.org/jira/browse/NIFI-9336
When configuring processors and controller services in the UI, if a user enters a property value that has leading or trailing whitespace, it's not always obvious and may not be desired. To indicate when a value contains leading/trailing whitespace, an icon with a tooltip will show next to the value in processor and controller services configurations.
To test:

go to configurations in a processor or controller service
enter a property value with leading and/or trailing whitespace
you should see an info icon and tooltip show to the right of the value

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[x] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks for the PR @mtien-apache! Will review...

@mcgilman Per our offline discussion, I've pushed a commit that fixes a bug to clean up tooltips. With the addition of the whitespace icon, when editing a value cell, it was re-rendering the property table, causing property tooltips to become orphaned on the DOM. This no longer occurs. Can you check again? Thanks!


Ubuntu/OrangePiZero "Could not initialize SDL video"

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Bionic with Linux 5.4.27-sunxi (Ubuntu / OrangePi Zero)
scrcpy version: 2.0
installation method: manual / apt
device model: OrangePi Zero LTS SSH through Windows 10 Laptop

I was under the impression that SCRCPY would work for Raspberry Pi and similar Pi devices. I am unable to stream the video after calling SCRCPY. Is there any fix to this? The Pi has a Mali 400MP2 GPU at 600MHz
root@orangepizerolts:~# adb devices
List of devices attached
TC202204009503  device

root@orangepizerolts:~# scrcpy -m800 -b2M --no-audio
scrcpy 2.0 <https://github.com/Genymobile/scrcpy>
[100%] /data/local/tmp/scrcpy-server.jar
ERROR: Could not initialize SDL video: No available video device
ERROR: "adb reverse" exited unexpectedly
WARN: 'adb reverse' failed, fallback to 'adb forward'

I think it's because your OrangePi don't have display devices. You can try enable v4l2 option and use ffmpeg to proxy virtual camera device.


Logging in to semi-authenticated registry

I am 100% sure this won't work with everything, but I only have artifctory to test against, and it works with that :) doesn't work (in the original form) with Docker Trusted Registry v2.5.3 FYI.
Make this better and/or make it configurable.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Noah Nordrum seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

You're right. I just tried all permutations and passing in an empty basic auth to those methods works. With #120 in place, this would be a LOT easier to figure out...

I see. We can surely improve it. I thought empty basic auth is something only DockerHub did...


Email notification sent even with no changes

Cannot reproduce. Maybe due to Maghrib change?

That was because of new masjids appearing after manually deleting the old data with existing subscribers. This should not normally happen in production.


Fix Linux CI

Let's fix the Linux CI.

Hello, can you please provide some more information?

Yes @bulat-gab.
The Linux CI, triggered by https://ci.appveyor.com/project/kblok/puppeteer-sharp-esipd/builds/35593283,  started failing a few months ago. See https://github.com/hardkoded/puppeteer-sharp/blob/master/appveyor-linux.yml
The job would be trying to see what needs to be changed in that yml to bring that back to life.

hi, how do i provide the chromium installed path link in .net core.

@RahulSapkota LaunchAsync has an executablePath attribute in the LaunchOptions. You can also use the environment variable PUPPETEER_EXECUTABLE_PATH from outside the code.

i am using chromium.exe within the folder inside project itself. but when i use the same in linux i think .exe is not working.  can you please help on this like how to run this in linux. please refer to below screen.i am alreadyng using the attribute you mentioned.

i have fixed it and get it working on linux as well but not sure why it's very very slow :(

Closed due to inactivity. Feel free to reopen it if needed.


Add cloudflare KV support

https://developers.cloudflare.com/api/operations/workers-kv-namespace-read-key-value-pair
We can implement cloudflare kv support via our KV adaptor

I‘d like to have a try.

Thanks a lot! Have fun.

It seems that official client is still in wip status and don't provides read key etc. for some reason issue 26. So I'd like use httpclient to handle requests.
Besides, there are some operations related to namespaces rather than kv， like list namespaces. What should we do with them?

Besides, there are some operations related to namespaces rather than kv， like list namespaces. What should we do with them?

We just need this.

We just need this. Namespace should be a parameter for building this service. 🤔

After reading some codes, I think opendal foucs on data access like read & write rather than data management like catalog. Is it right?
And, should account_identifier be display in Debug?

After reading some codes, I think opendal foucs on data access like read & write rather than data management like catalog. Is it right?

Yep.

And, should account_identifier be display in Debug?

LGTM.

I have opened the pr #3348.
But I don't know how to test it for I don't have a cloudflare account😥.
By the way, cloudflare kv provides batch write and batch delete. Does our KV adaptor need to support such operations ?

By the way, cloudflare kv provides batch write and batch delete. Does our KV adaptor need to support such operations ?

No need for now.


Font/h2 elements getting doubled/tripled in output

i have been struggling to figure out whats going on here.

this is the only dom->data uri package i can find that works well with safari
im trying to force my component to render before calling html2canvas
the final output always has the h2 element doubled/trippled
i am doing a bunch of rendering updates to the h2, so possibly its snapshotting while its still rendering?
not entirely sure, but seems like a lot of the time this only happens on safari (both web and ios)

it is happening in chrome too actually


"baud detect failed"

I get this error,I want to know how to detect if I enter the bootloader mode? I erase the flash using SmartRF Flash Programmer2,but I still get the same return! could u please tell me how to debug the device?BTW, my device is cc2640r2f
thx

Have you enabled and configured the bootloader settings in the .ccfg file?

Somehow , I use another device and it work. Thanks


[CSidebarNavItem] Icon props are not passed to CIcon

I am trying to adjust the size of icon in the CSidebarNavItem by passing icon as an object, such as
<CSidebarNavItem
  name="Dashboard"
   to="/dashboard"
   :icon="{name: 'cil-chart', size: '4xl'}"
/>

however the size prop is not passed to CIcon.
However, creating a CIcon with the size prop works just fine
<CIcon name="cil-chart" size="4xl"/>

After searching through the code I saw that CIconRaw is imported instead of CIcon
https://github.com/coreui/coreui-vue/blob/ad9f36c5bff5f9466bf1ebc8653228d1af659eef/src/components/template/CSidebarNavDropdown.vue#L19
Could it be that importing CIconRaw instead of CIcon is to blame here?

Hi, thank you for the report!
in fact, it doesn't work because 'customClasses' is set to 'c-sidebar-nav-icon' inside CSidebarNavItem overriding size prop in CIcon.
Hotfix:
<CSidebarNavItem
  name="Dashboard"
   to="/dashboard"
   :icon="{name: 'cil-chart', customClasses: 'c-sidebar-nav-icon c-icon c-icon-4xl'}"
/>

We will think of a better solution, and fixing documentation.

@woothu, thank you for the hotfix!


Async data and Flux stores

I am using the Flux architecture. Each time I visit a page, the page requests data from a Store, but as the action is created (eg. REQUEST_TODO), there is no explicit callback triggered, but another action will be  created once the request is completed (eg. REQUEST_TODO_COMPLETED).
I am trying to implement something similar to (https://github.com/rackt/react-router/blob/master/docs/api/run.md or https://github.com/rackt/react-router/blob/master/examples/async-data/app.js), I get the principles, but I struggle adapting with the concept of actions. Should I consider my actions differently, by adding some sort of callback, but this would make the dataflow more complex?
I am just starting, so this is likely I have missed something.
Thanks

@gaearon thanks for this technique.
I have a login with an async call to fetch user data with a token (which might be stored locally). I'm using the observeStore in a willTransitionTo for a page that requires authentication as follows.
 static willTransitionTo(transition, params, query, callback) {
      if (!LoginStore.isLoggedIn()) {
        //store next path in RouterStore for redirecting after authentication
        let transitionPath = transition.path;
        RouterActionCreators.storeRouterTransitionPath(transitionPath);

        //check if still waiting for a user data request
        observeStore(LoginStore, s => !s.isRequestingUserData())
        .then(() => {
          if (!LoginStore.isLoggedIn()) {
            transition.redirect('/login');
          }
        })
        .finally(callback);
      }else{
        callback();
      }
    }


[WIP] Update to latest alcotest, conduit, cohttp, github-hooks and session

This needs a bunch of not yet released packages:

github-hooks.0.2.0 and github-hooks-unix.0.2.0: https://github.com/dsheets/ocaml-github-hooks/pull/12 /cc @dsheets
session-redis-lwt.0.4.0 and session-webmachine.0.4.0: https://github.com/inhabitedtype/ocaml-session/pull/16 /cc @seliopou

I still need to update the Dockerfiles and CI scripts to pins these versions.

github-hooks 0.2.0 and github-hooks-unix 0.2.0 are now released.

all the related libraries are now released -- can we get a release of datakit-ci in opam @samoht?  The reason is that the current stable release is pinned to an older jbuilder...

Superseded by #612


template syntax error : v-model does not support dynamic input types. Use v-if branches instead.

Vue.js version
2.0.0-rc.1
Reproduction Link

Steps to reproduce
What is Expected?
What is actually happening?

As the error suggested, v-model does not support dynamic input types, so don't use :type="type"

i am stuck in this error, someone can help me out ... thx

ok ... thank you ... i should change the implement ...

@yyx990803 Is this limitation going to be fixed? Or is there a workaround other than painful/unmaintenable v-if branches?

@Akryum This is a feature introduced in this commit: https://github.com/vuejs/vue/commit/f35f7e35cd48efc21e759752058c07a1ff32cfb6. It seems that this check is skipped in production.
Perhaps you can bypass this check by defining :value and @input separately instead of v-model, though it may be discouraged: https://vuejs.org/v2/guide/components.html#Form-Input-Components-using-Custom-Events

A better implementation would be to create your dynamic input type as a component. In Vue 2.x, you can attach v-model to custom components to get two-way data binding.

@sirlancelot But in that component you would still need to dynamically bind the type, no? Or could the limitation be circumvented by using a custom model?
If not, it would mean you have duplicated code for every type you want to cover. That can get quite messy, especially as those generic components change quite often until you get a more or less stable version.

@gwildu Yes, I encountered the same issue while trying to use an input component in (the development version of) Keen UI: https://josephuspaye.github.io/Keen-UI/#/ui-textbox-docs. After the recent update of Vue.js, now I cannot make it work. A workaround would be to always set the input type to text and validate its value, but it will require work on every similar component.

@sirlancelot I'm not sure if this is possible but couldn't the old behavior be hidden by an attribute flag like 'generic-type' or 'dynamic-type'? My guess is, that this new behavior has a positive performance impact but  that might not be an issue for all projects.

How about implementing a render function in JavaScript? It does not look elegant but probably powerful enough to do almost anything. Actually, it is the way some components are implemented. https://vuejs.org/v2/guide/render-function.html

Thanks for sharing that  @akirak, I hadn't personally read that far in the docs yet but it outlines exactly what I had in mind for OP's dynamic input component which I suggested earlier: Create a component which has as many hard-coded inputs as desired and wrap them in a v-if. Abstracting this functionality away in to a component allows re-usability and you don't have to think about its implementation after it's written.

Render functions don't support v-model, so you'd have to re-implement v-model essentially, and that doesn't look that simple: https://github.com/vuejs/vue/blob/dev/src/platforms/web/compiler/directives/model.js

The error doesn't seem to appear if you use <component is="input" v-model="m" :type="t" /> and I believe you still get all the functionality.
Is this intentional?

You probably don't get all the functionality of v-model.
Unless I'm mistaken, what's going to happen is that you'll get genDefaultModel here: https://github.com/vuejs/vue/blob/dev/src/platforms/web/compiler/directives/model.js#L27
So it won't work properly with select, checkbox and radio.

@nkovacs The point for my use case is creating a generic text input component, part of a UI toolkit. It won't be used as a select/checkbox/radio (there will be specific components for that), but, it could be used as a text/email/password/phone/etc. text input.

maybe I'm missing something but this works for me
dynamic-input.vue
export default {
  props: {
    tag: {
      type: String,
      required: true
    }
  },
  render: function(createElement) {
    var component = this
    return createElement('input',
      {
      domProps: {
        type: this.tag,
        value: component.value
      },
      on: {
        input: function(event) {
          component.value = event.target.value
          component.$emit('input', event.target.value)
        }
      }
    })
  }
}

and then
<dynamic-input :tag="dynamicTag" v-model="value" />

It does work in some cases, but not in others, e.g. if type is radio. v-model, when used on native tags (input, select) in vue template files, is compiled into different render functions depending on the tag and the type attribute. You can see that code here: https://github.com/vuejs/vue/blob/dev/src/platforms/web/compiler/directives/model.js
That's why type can't be dynamic.
Your code doesn't work on radio or checkboxes, for example, because it only listens to input events. v-model checks the type and listens to the correct event.

ah okay, got it thanks :-)
for me, alt least for now, it's exactly what i needed :D

I worked around this by generating all the different input types I needed and then using v-if to activate only the one that matches the dynamic type (as suggested by the error message). I used pug to avoid a bunch of duplication:
<template lang="pug">
.form-group
    label.label(:for="id", :class="offsetClass", v-if="label") {{ label }}
    div(:class="widthClass")
        each type in ["text", "password", "search", "email", "date"]
            input.form-control(
                :id="id",
                v-if="type === '" + type + "'",
                v-model="val",
                type=type,
                :placeholder="placeholder"
            )
        slot(name="help")
</template>

This way I can use the built-in v-model directive. This can be extended to radio and checkboxes as well.

interesting, thanks for sharing =)

Hi! create simple component for error solution: https://github.com/nikitamarcius/vue-form-input

Here is y working solution in a component bootstrap based for this... Enjoy!
//Vue component definition
Vue.component("boost-input", {
    props: ["gridLayout", "txtIdentifier", "txtType", "txtLabel", "txtVal", "txtValidation",  "txtPlaceholder",  "helpBlockText", "maxLength", "parentModel"],
    model: {
        prop: 'parent-model',
        event: 'update:parentModel'
    },
    data: function() {
        return {
            templateRender: null,
            modelValue: (typeof this.txtVal === "undefined") ? "" : this.txtVal
        }
    },
    render(h) {
        return h('div', [
            (this.templateRender ?
                this.templateRender() :
                '')
        ]);
    },
    mounted: function ()
    {
        var compiledTemplate = Vue.compile('\

\
            \
            <input v-model="modelValue" type="' + this.txtType + '" :maxlength="maxLength" v-on:change="change($event.target.value)" v-validate.initial="' + this.txtValidation + '" data-vv-as="' + this.txtPlaceholder + '" :id="txtIdentifier" :name="txtIdentifier" :placeholder="txtPlaceholder" class="form-control">\
            {{ errors.first(txtIdentifier) }}\
            \

');
        this.templateRender = compiledTemplate.render;
        this.inputSetup(this.txtIdentifier);
    },
    methods: {
        change: function (newValue) {
            this.$emit('update:parentModel', newValue)
        }
    }
});

//Using the component in html
<boost-input v-model="Username"
                             grid-layout="col-sm-6 col-md-4 col-lg-3"
                             txt-identifier="Username"
                             :txt-val="Username" //The parent model
                             txt-type="text"
                             txt-label="Username"
                             txt-validation="'required'"
                             txt-placeholder="Username"
                             max-length="60"
                             help-block-text="">

Another alternative and the way this framework push you to do it could be something like this...
<input v-if="type == 'text'" type="text" :name="name" :id="name" v-model="inputModel">
<input v-if="type == 'number'" type="number" :name="name" :id="name" v-model="inputModel">

I suggest you create a simple project that returns the input type, something like this.

<input ref="input" v-model="modelValue">

props: {
  type: {
    type: String,
    default: 'text'
  }
},
mounted () {
  this.$refs.input.type = this.type
}

interesting :D 👍

@meteorlxy Your solution will break when you use radio or checkbox.
Vue 2.5.0 supports dynamic types by converting it to a v-if/v-else: https://github.com/vuejs/vue/commit/f3fe012d5499f607656b152ce5fcb506c641f9f4#diff-6eaa1698ef4f51c9112e2e5dd84fcde8R4, which looks similar to my workaround.
It's buggy in 2.5.0, you have to call the property type: https://github.com/vuejs/vue/issues/6800

@nkovacs Yeah, I only use it for those "text-like" types. Happy to see that dynamic types supported in new release.


Bandainamco scraper is inactive on scraper.peviitor.ro

ENVIRONMENT: Windows, Mozilla Firefox
URL: https://scrapers.peviitor.ro/
STEPS TO REPRODUCE:
Open URL.
Type Bandainamco in the search bar.
Click on Bandainamco
Run Scraper function.
Expected results: We should have a result with company jobs
Actual results: Scraper is not loading . Status inactive

company page is inactive.


TransportOptionsValidator doesn't validate properly if AllowUnsecuredTransport=true

Is there an existing issue for this?

[x] I have searched the existing issues

Describe the bug
If AllowUsecuredTransport=true then TransportOptionsValidator stops validating the address. For example, there is no validation that no value is specified if AllowUsecuredTransport=true.
https://github.com/dotnet/aspire/blob/08dd9928cdf483947fd01d35ea77e8cd5d1294a2/src/Aspire.Hosting/Dashboard/TransportOptionsValidator.cs#L14-L16
Expected Behavior
AllowUsecuredTransport=true only impacts validating whether the transport is secure or not.
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version info
No response
Anything else?
No response

TransportOptionsValidator is doing more than validating TransportOptions.


/mkpasswd - Unknown command: mkpasswd

This might just be a question rather than an issue. Once I run this image in a container and connect to the server the following command is not recognized:
command:  /mkpasswd hmac-sha256 somePass
output: Irssi: Unknown command: mkpasswd
I see that the appropriate modules appear to be loaded correctly.
irc_1  |
irc_1  | InspIRCd Process ID: 1
irc_1  |
irc_1  | Loading core modules ....................
irc_1  | [*] Loading module:    m_md5.so
irc_1  | [*] Loading module:    m_sha256.so

Am I missing something? Thanks! Sweet Image.

You need to load the password_hash module.

Isn't it loaded by default? Is  m_password_hash.so not the correct module?
irc_1  | [*] Loading module:    m_password_hash.so <-- this is loaded.

Try /quote mkpasswd instead?

I'm going too assume this is solved.
If you have any more issues then please ask in irc.inspircd.org #inspircd. Thanks.


is there any way to add disqus as a default using remote-theme? Not to add each.

Originally posted by @ohoroyoi in https://github.com/mmistakes/minimal-mistakes/issue_comments#issuecomment-432094838

find answer
https://github.com/mmistakes/minimal-mistakes/issues/1562#issuecomment-408714491


Modification suggestion with PropertyPanel

~/Scripts/Serenity/Serenity.Script.UI.js
line number:9722
loadInitialEntity: function() {
if (ss.isValue(this.propertyGrid)) {
this.propertyGrid.load(this.get_entity() || new Object());
}
}
English is not good, please forgive me

You can override loadInitialEntity and load entity you like, e.g.
protected loadInitialEntity() {
this.propertyGrid.load(someEntity);
}

Do not know why not get “propertyGrid”
Property 'propertyGrid' does not exist on type 'UserInformationPanel'.

If you are using C# its
protected override void LoadInitialEntity

It's been working, but it shows the error.
Thank you for your answer

It is a .d.ts error, for now write (this as any).propertyGrid

Thank you very much for your patience.

You're welcome. Intellisense should be fixed in latest.


Now server expects listen options (host, port, cert, key) in constructor

During configs merge (as we discussed at #24) I've changed options priority of method loadOptions: options.host = options.host || argv.h || env.LOGUX_HOST. Previously it was really default options for parser function. Now it become a wrapper for all options and you want them to be provided as is. It will be magic if some of them will change during operations.
You will be very surprised if this server will start at port 2000 because of env params:
const app = new Server(Server.loadOptions(process, {
  subprotocol: '1.0.0',
  supports: '1.x',
  root: __dirname,
  port: 11337
}))

I think it will be messy behavior. If you want to use env params then you shouldn't provide option at all.

Oops, I stuill doesn’t accept your previous PR :). Hope they will be mergable.

@ai Not a problem. I'll merge them. Just choose one that you want to merge earlier. And I'll prepare another :)

Rebased. Please rerun the build. Created an issue #26 for this unstable test.

I cleaned up previous PR, so you will need rebase :).
git checkout master
git pull git@github.com:logux/logux-server.git master
git checkout merging-configs
git rebase master
git push origin merging-configs --force

Rebased :)

@artmnv Thanks. Now we need final step — add options for logger type and test it.


Update guava

Fixes failing build

+1

@cgruber Any news on this PR?

Sorry, have been overloaded this quarter.  I'll try to get this done in the next few days.  I need to take this inside and then push it out and I'm OOO tomorrow.

Context question... do you mean 22.0-SNAPSHOT, or do you mean 22?  Do you need unreleased functionality, or does 22.0 resolve the cygwin issue?

I don't remember why I used 22.0-SNAPSHOT, but I'm sure there has been a reason. I will find out again why I didn't use 22.0 and let you know.

It's possible that when you filed the pull request, 22.0 was not yet
released.  But let me know if you need post-22 functionality.
On Tue, 27 Jun 2017 at 18:49 Yannic Bonenberger<EMAIL_ADDRESS>wrote:

I don't remember why I used 22.0-SNAPSHOT, but I'm sure there has been a
reason. I will find out again why I didn't use 22.0 and let you know.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/MOE/pull/39#issuecomment-311534064, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAUN4oo9pBENjjKkE6elW9Zf3xgqBAVgks5sIbE4gaJpZM4M6jbE
.

The reason I uses 22.0-SNAPSHOT was that 22.0 has not been released in April. I updated this PR to use 22.0

Other snapshot dependencies were removed/updated in https://github.com/google/MOE/pull/41 because it led to runtime failures

Hello


fix: support fallback for prefixxor

Main Changes:
to remove the dependent of palm instruction, so we support fallback logic in prefixxor function.

Codecov Report

Merging #59 (7a22458) into master (50514ec) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   95.88%   95.88%
=======================================
  Files          21       21
  Lines        2431     2431
=======================================
  Hits         2331     2331
  Misses        100      100

Impacted Files
Coverage Δ

include/sonic/internal/arch/avx2/base.h
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Fallback non-simd codes is better if target machine doesn't have pclmul instruction.


Gluon RNN memory leaks with extra variables

Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to most of the technical issues and bug reports. For non-technical issues and feature requests, feel free to present the information in what you believe is the best form.
For Q & A and discussion, please start a discussion thread at https://discuss.mxnet.io
Description
Gluon allows one to define extra variables that may not lead to model outcome. However, having them may cause memory leak.
Environment info (Required)
----------Python Info----------
Version      : 3.6.5
Compiler     : GCC 7.2.0
Build        : ('default', 'Apr 29 2018 16:14:56')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 10.0.1
Directory    : /home/ec2-user/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/pip
----------MXNet Info-----------
Version      : 1.3.1
Directory    : /home/ec2-user/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/mxnet
Commit Hash   : 19c501680183237d52a862e6ae1dc4ddc296305b
----------System Info----------
Platform     : Linux-4.14.77-70.82.amzn1.x86_64-x86_64-with-glibc2.9
system       : Linux
node         : ip-172-16-95-144
release      : 4.14.77-70.82.amzn1.x86_64
version      : #1 SMP Mon Dec 3 20:01:27 UTC 2018
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                8
On-line CPU(s) list:   0-7
Thread(s) per core:    2
Core(s) per socket:    4
Socket(s):             1
NUMA node(s):          1
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 79
Model name:            Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz
Stepping:              1
CPU MHz:               2706.669
BogoMIPS:              4600.11
Hypervisor vendor:     Xen
Virtualization type:   full
L1d cache:             32K
L1i cache:             32K
L2 cache:              256K
L3 cache:              46080K
NUMA node0 CPU(s):     0-7
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0020 sec, LOAD: 1.0198 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.0912 sec, LOAD: 0.1530 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 0.5845 sec, LOAD: 0.1434sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0089 sec, LOAD: 0.1170 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0100 sec, LOAD: 0.3888 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0104 sec, LOAD: 0.0782 sec.```

Package used (Python/R/Scala/Julia): Python
Error Message:
If you run watch -n0.1 nvidia-smi, you may observe memory growth every by 2MB every few seconds.
Minimum reproducible example
See mxnet-memory-leak.tar.gz
The main differences between the attachment and examples/gluon/language_model/ are to add extra on Line 56 in model.py add to add mx.nd.array([], ctx=context) on Line 166 and 183 in train.py
Steps to reproduce
(Paste the commands you ran that produced the error.)
1. python train.py --cuda --tied --nhid 200 --emsize 200 --epochs 20 --dropout 0.2 &
2. watch -n0.1 nvidia-smi

What have you tried to solve it?

Add a dummy link between all inputs and outputs. However, this may not always be possible / convenient / readable.
I previously suggested a feature request to allow None input types in the gluon models. Communicated with @szha that this would not be fundamentally challenging. However, this has not been acted upon and may be a low-hanging fruit alongside the memory fix leak.

Related: https://github.com/apache/incubator-mxnet/issues/13247

@apeforest Why is this not a bug?

@yifeim Sorry, got too busy and haven't got chance to dive deep into this. Yes, I think it's a bug. @mxnet-label-bot add [Bug]

The memory leak is related to the extra unused variable you passed into your RNN model but it is NOT specific to RNN. In your repro script, you created a size-zero ndarray in each loop which caused the memory leak.
for epoch in range(args.epochs):
    ...
    for i, (data, target) in enumerate(train_data):
        ...
        with autograd.record():
            ....
            output, hidden = model(data, hidden, mx.nd.array([], ctx=context))

However, since the size-zero ndarray is unused anywhere, it is a better code practice to create once outside the loop and use it throughout your training. The same change applies to the eval() function in your repro script.
extra = mx.nd.array([], ctx=context)
for epoch in range(args.epochs):
    ...
    for i, (data, target) in enumerate(train_data):
        ...
        with autograd.record():
            ....
            output, hidden = model(data, hidden, extra)

With this change, I ran your repro script for 10 epochs with mxnet_cu90mkl 1.3.1 and 1.4.0 packages and did not see memory leak.

@yifeim After a little bit more digging, I think the issue is specifically related the usage of size-zero ndarray for your extra variable. If you just use mx.nd.array([1], ctx=context) as the extra variable in the loop of your repro script, you will not observe any memory leak. The true problem is creating size-zero ndarray in a loop.

Very interesting. Thanks a lot for the insights!

Thanks for handling @yuxihu!

@anirudh2290 Could you please reopen this? The original fix has been reverted due to test flakiness. I am working on alternative fix.


fix token secret en-/de-coding

for #25
I found two points that are problematic:

I could get the same arbiter token as this by base64 encoding the file content. Just wondering if this is the intended behaviour, as I could see in CM that the token is encoded before writing to the file, maybe what we want here is base64 decoding? If not, just ignore this, I'll keep the encoding part in this PR. For what it worth, the node js module doc seems not accurate enough about fs.readFileSync's behaviour with regards to encoding argument(to encode or to decode?), but in the source code, buffer.toString(encoding) is called before returning, so I guess the argument is used to indicate encoding scheme.

As pointed out by @yousefamar, the returned secret should be used directly in macaroon verification, so removed the decoding step.

Haven't merged yet so @yousefamar and @Toshbrown can check. Hope moving on from node will help here too


Optipng

This PR adds support for an OptiPNG based PNG compressor.
Things that need to be done after this lands:

Return a proper version from the optipng module
Try to refactor the C code to not rely on Emscripten’s emulated filesystem (use fmemopen if at all possible)
Add zopflipng

I ran a few images through this, then imageoptim to see what extra things it could do:
Paletted 7k to 6k (Zopfli got the best number, but advpng also improved upon the 7k).
Paletted 47k to 44k (Advpng+Zopfli got the best number).
Non-paletted 148k to 142k (Advpng+Zopfli got the best number).
It's notable how fast advpng is and appears to beat the numbers optipng gives us. Zopfli isn't fast, but it gets the numbers.
That said, optipng is pretty close in all cases, so I'm happy for this to land. For "v2", I'd like to explore how much work it would be to port advpng's algorithms to something that works on data in memory. But maybe that's a world of pain.

Wasn't a criticism, just wanted to document the kind of numbers we're dealing with. Optipng is absolutely good enough for v1.

@jakearchibald PTAL

Yay! Great to see this land given how much of a PITA it was.


Cant have translation keys which are Rust keywords

For example, mod.
--> /home/felix/workspace/lemmy/target/debug/build/lemmy_websocket-5c9ec7d2402ea26c/out/rosetta_output.rs:1:35802
|
1 | ...[allow (clippy :: match_single_binding)] pub fn mod (& self) -> & 'static str { match self { _ => "mod" } } # [allow (clippy :: match_...
|                                                    ^^^ expected identifier, found keyword
|
help: you can escape reserved keywords to use them as identifiers

Would be good if identifiers could be escaped, as suggested by the help message. Syntax is like let r#mod = 123;.

Thanks for reporting, I'll fix that as soon as possible!


Forum or wiki?

I've been very impressed so far with what I've seen of Fable and its viability from a technical point of view. But unlike other projects, there doesn't appear to be any forum or wiki for users to post their experiences / silly questions / discussion items. So raising an issue here seems to be the only way to interact, and while I've been impressed with the responsiveness to my issues, I don't think it's the right place for everything.
Have I missed something?

That's a very nice idea. Only my problem is that I'm not very good at keeping an eye to notifications from many different sites, so I'm trying to focus on Github and Twitter at the moment. Opening a forum would be great but it'd have to be mostly community-maintained. I'm fine with using Github issues for comments and suggestions, as it makes it easier to look up for them later. Also, right now the compiler is shaping up with community's feedback before releasing v1.0.0 so any comment is valuable and can open a discussion for new features :)

Maybe just Gitter? It's working pretty well for most F# projects.

Yeah, that should work (again I cannot promise I'll be checking it constantly). Probably the best timing to create the channel will be when the logo and the web are ready. Then we can also open a Twitter account :+1:

Reopening as a reminder to create the Gitter channel when the logo is ready.

Merge with #183


Readme is out of date for deploy_method

the method is now method. The console print outs have it correct but it would still be nice to fix the readme up.

Ping!
Come on guys, the BASIC usage of the extension can't be outdated in the readme!

+1
Lost at least two hours because of this. There are two pull requests for this. Just merge them please.

I think the problem is that middleman-deploy is still sitting on the fence between middleman 3 and 4. I think for now the README should clarify that this method has changed and you have to use the prerelease version to use it with middleman 4.

Then maybe it will be better to use a non-default branch (like develop) for the development of the version 4 compatible and keep master with the latest stable release code (and readme).


Allignement with changes from grafana-data-source-backup

Refactor export script to reduce terraform output calls.
Extended substituted characters list in dashboard names.
Check if counter require .value.title to calculate length.
Bump Terraform version.

Done, except reducing terraform output. Looks like it's not possible at the moment.


Suspend : facade suspend method to Promise

Currently handled like a normal method, suspend is simply ignored.
A naive/first version could be to wrap it in a Promise with GlobalScope.promise {...}

Delivered in v0.3.0


Add info about logging into shell before removing /mnt/data

details

rm -rf should be run after logging into the node shell

/check-cla

/lgtm
/approve


Forgoten password > sent email

As a user I would like to have an email sent to my inbox with a link to reset my forgotten password.

duplicate, see #363


SchemaStore and Kafka topic name update

2 changes:

Make mysql-schema-store optional in config. Schema store is not mandatory to run SpinalTap but current config doesn't allow empty schema store config.
Update kafka topic name, use source name instead of hostname, as hostname can change in db migration.

@mangobatao

addressing issue #18


Gotify support for android users

## Feature Request
I'm not an android user anymore but this would be a huge value add for those who are.

what means gotify ?

https://gotify.net/


Added open graph meta tags

I need the image link. I will update that too. couldn't find it actually.

og:image here ya go:

you need to re-add the og:image tag and add the image to the repo :)
btw. here's the figma file: OG Figma File

Awesome! Yes, add the image and I'll merge.

which image tag to keep? figma or normal one?

everything is fine the way you did it.
you might want to change the url to: https://microsoft.github.io/join-dev-design/microsoft-og-image.png
since this is the place your image resides once the pull request got merged.

👌 thank you! Very good work.