implement "implements" in type annotations

This is a:

[ x ] Feature Request

Which concerns:

[ x ] flow-runtime
[ x ] babel-plugin-flow-runtime

What is the current behaviour?
The interfaces a class implements are missing from its type annotation.E.g.:
interface Serializable {
  serialize(): string;
}

class Foo implements Serializable {
  serialize() { return '[Foo]'; } // Works!
}

transforms to
@t.annotate(t.class('Foo', t.method('serialize')))
class Foo {
  constructor() {
    Serializable.assert(this);
  }

  serialize() {
    return '[Foo]';
  } // Works!
}

What is the expected behaviour?
The interfaces declared by "implements" would be included in the type annotation.
Which package versions are you using?
Example is from whatever is running here: https://codemix.github.io/flow-runtime/#/try

I was also experiencing this behavior and was working around this issue, by defining an IntersectionType, containing class members as ObjectType and all interface references, and calling unwrap() on it in place. This is definitely not ideal and it may be of advantage to first investigate #123 and then allowing to pass a merged ObjectType, containing members of the class and its implemented interface.

For my needs passing in the referenced interface types is enough, for my use case I just need to know which interfaces the class explicitly declares that it implements. Sounds like your use case is more complicated.

You're totally right, it just needs to verify the signatures of the types, which it obviously doesn't.
In any case, I think you would need to annotate the class method as well.


Product detail page breaks if variants get removed

Description:
Product detail page breaks if variants get removed and a main variant was configured.
Environment:
Empty Shopware <IP_ADDRESS> setup
Steps to reproduce:

In Admin: Create product in admin, make it visible in storefront category & sales channel
In Admin: Add 2 variants to it
In Admin: Select 1 of the variants as main variant
In Admin: Delete all variants
In Storefront: Try to click on product in category view.
In Storefront: 404 error because the configured variant_listing_config still has the mainVariantId configured that no longer exists.

Expected result:
Page is accessible
Actual result:
404 error
SQL to find all broken products:
SELECT
	DISTINCT id,
	product_number,
	variant_listing_config ->> "$.mainVariantId" AS missing_product_id
FROM
	product p
WHERE
	UNHEX(variant_listing_config ->> "$.mainVariantId") NOT IN (
	SELECT
		DISTINCT p2.id
	FROM
		product p2) AND
		variant_listing_config ->> "$.mainVariantId" != "null"

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-36800.
[created from NEXT-36800, comment 486652]


Video rotate

I use my GN9005 to record a video, and then use mediaplayer-demo to play it.
I found the video been rotated,because I recorded the video vertically,but it shows horizontally.
Then I use annother phone to play this video, it shows normal(vertically).This stituation happens in some kinds of phone.
My English is poor so I make a photo to show it.
In GN9005:

This is the video i use:
video.zip
Hope for your help.

Duplicate of #6. Rotation works from API 21 (Android 5.0), there is no nice solution for older Android versions.


Get sensor simulation code working

In the current version attempting to enable sensor simulation by defining the SENSOR_SIMULATE preprocessor symbol results in a variety of compilation errors and other complaints.  I don't know the history of that code, and it may be dependent on other configuration parameters, but it would be good to have it fully operational.

HARDWARE_SIMULATE code tested and integrated.


Remove environment annotation from examples

What does this PR do?

[ ] Adds new functionality
[ ] Alters existing functionality
[ ] Fixes a bug
[ ] Improves documentation or testing

Please briefly describe your changes as well as the motivation behind them:

Removes all of the safemode environment annotations from the examples
Removes the requirement to use the safemode annotations in lima
We've had many, many users who, quite naturally, build their disruptions by looking at examples, rather than reading any of the docs. They end up not knowing what this env annotation is for, and set it incorrectly. In environments where it is actually mandatory, the webhook rejection message explains how to use it, so we don't need to explicitly include it in the examples like this. It's only been doing more UX harm than good

Code Quality Checklist

[ ] The documentation is up to date.
[ ] My code is sufficiently commented and passes continuous integration checks.
[ ] I have signed my commit (see Contributing Docs).

Testing

[ ] I leveraged continuous integration testing

[ ] by depending on existing unit tests or end-to-end tests.
[ ] by adding new unit tests or end-to-end tests.

[ ] I manually tested the following steps:

x
[ ] locally.
[ ] as a canary deployment to a cluster.

should we add just one example called "with_environment_annotation" (or something similar)?

I'll put the annotation back into complete.yaml, for completeness


Implement clear signing

@maxtaco, did you finish the implementation? Are there any docs on how to use this?

Should indeed work
{clearsign} = require 'kbpgp'

await clearsign { msg : "hi", signing_key }, defer err, msg


ScopedBlock hook with subscriber.SyncWithHook

Adds the ability to specify a block hook in a Sync call, and have that block hook be called for blocks associated with that sync. Could help clean up logic in indexer and usages of block hook in general.
For example instead of having to keep track of a mapping of entriesCid -> AdCid like we currently do, we could rely on closured variables to get that information. For example:
// in syncAdEntries in linksystem.go in indexer

	_, err = ing.sub.Sync(ctx, from, entriesCid, ing.entriesSel, nil, func(_ peer.ID, c cid.Cid) {
		node, err := decodeIPLDNode(c.Prefix().Codec, bytes.NewBuffer(val))
	        if err != nil {
		        log.Errorw("Error decoding ipldNode", "err", err)
		        return
	        }
	    	err = ing.indexContentBlock(adCid, from, node)
	          if err != nil {
		          log.Errorw("Error processing entries for advertisement", "err", err)
	          } else {
		          log.Info("Done indexing content in entry block")
		          ing.signalMetricsUpdate()
	          }
       })

Codecov Report

Merging #79 (6746583) into main (f627b45) will increase coverage by 0.98%.
The diff coverage is 90.62%.

@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
+ Coverage   67.62%   68.61%   +0.98%
==========================================
  Files          19       19
  Lines        1662     1692      +30
==========================================
+ Hits         1124     1161      +37
+ Misses        400      392       -8
- Partials      138      139       +1

Impacted Files
Coverage Δ

subscriber.go
80.05% <90.62%> (+0.83%)
:arrow_up:

gpubsub/trace.go
43.47% <0.00%> (+21.73%)
:arrow_up:

Continue to review full report at Codecov.

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

Can this be closed because #89 includes it?

Closing because #89 brings it in


[BugFix] Fix left null aware anti join short circuit bug

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #6704
Problem Summary(Required) ：

like sql:
MySQL w2> SELECT subt0.c_0_0 FROM  t0 subt0
WHERE (subt0.c_0_0) not IN (
    (
     SELECT t1_44.c_1_0 FROM t1 AS t1_44
     WHERE (subt0.c_0_1) IN ('1969-12-10')
    )
)
+-------+
| c_0_0 |
+-------+
+-------+
0 rows in set
Time: 0.098s

reason:

t1.c_1_0 contains null value
left null aware anti join will check the result of right child isn't contains null and do short circuit
but need consider join may have other on-clause predicate, like (subt0.c_0_1) IN ('1969-12-10'), the means only check contains null on the data which can be join with (subt0.c_0_1) IN ('1969-12-10'), not check all data contains null

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

left null aware anti join will check the result of right child isn't contains null and do short circuit

should be left null aware anti join will check the result of right child contains null and do short circuit


A background to disable any action in the controller

@jose920405 Do you want to disable only current controller, or a window entirely (if you use tab bar, for example, results will be different)? Or both?

OK. Makes sense to disable the entire window.
Thanks!.

@jose920405 Check v0.14 https://github.com/peterprokop/SwiftOverlays/tree/0.14


Componentize pay and payment reducers

Description:
Refactors the pay and payment reducers following latest convention.
Motivation and Context:
Continue with reducer refactoring to clean up code and make it easier to read and use.
How Has This Been Tested?
test suite
Types of changes:
Refactor

Coverage decreased (-0.04%) to 22.331% when pulling 6fe052bd7cdf80a0eaefb34e633a31629b38c388 on mrfelton:refactor/pay-payment-reducers into 5ed7dda5903afa41e895b482ba5e2d842c1ecb3b on LN-Zap:next.


Unit tests

The Pretty Diff unit tests will make for a good start.  These tests are written to compare beautification results, which silently implies the parsing step.  The code samples need to converted to parse tables.
https://github.com/prettydiff/prettydiff/tree/master/test/samples_correct
The generated code samples can be compared to the stored code units using Pretty Diff's code comparison.  This issue is blocked by #4 and should be completed before anything else.

I am working this now in branch tests.
Current status:

[x] code validation
[x] code unit test runner
[ ] good diff of parse tables
[ ] test framework independent of running the lexers or code samples

The test runner seems solid.  Just need to populate more test samples for the lexers.


Text in UI is misleading at "Verification"

Summary
During the Verification phase of process for generating key, the UI uses text which is inconsistent, and can easily be misinterpreted.
Steps to reproduce

Run the eth2deposit-cli

Unexpected behaviour: during Verification, the UI prints

Verifying your deposits.
This might lead a user to believe that deposits are being made.
Suggested alternative
At step 2. during Verification, the UI could print:
Verifying your depositdata
This would maintain consistency with terminology used above:
Creating your depositdata
Screenshot

@chrishobcroft thanks for catching it!


Graceful Shutdown

Hi,
I'm using klaemo/couchdb:latest via docker-compose but it's not stopping with a clean zero exit code:
$docker-compose up couchdb
$docker-compose stop -t 50

Starting couchdb
Attaching to couchdb
couchdb    | -name couchdb@mynode
couchdb    | grep: /opt/couchdb/etc/local.d/*.ini: No such file or directory
couchdb    | ****************************************************
couchdb    | WARNING: CouchDB is running in Admin Party mode.
couchdb    |          This will allow anyone with access to the
couchdb    |          CouchDB port to access your database. In
couchdb    |          Docker's default configuration, this is
couchdb    |          effectively any other container on the same
couchdb    |          system.
couchdb    |          Use "-e COUCHDB_USER=admin -e COUCHDB_PASSWORD=password"
couchdb    |          to set it in "docker run".
couchdb    | ****************************************************
couchdb    | [info] 2016-11-17T15:01:19.152277Z couchdb@mynode <0.7.0> -------- Application couch_log started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.159696Z couchdb@mynode <0.7.0> -------- Application folsom started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.206457Z couchdb@mynode <0.7.0> -------- Application couch_stats started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.206857Z couchdb@mynode <0.7.0> -------- Application khash started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.218711Z couchdb@mynode <0.7.0> -------- Application couch_event started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.227111Z couchdb@mynode <0.7.0> -------- Application ibrowse started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.233544Z couchdb@mynode <0.7.0> -------- Application ioq started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.233877Z couchdb@mynode <0.7.0> -------- Application mochiweb started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.234354Z couchdb@mynode <0.7.0> -------- Application oauth started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.243788Z couchdb@mynode <0.204.0> -------- Apache CouchDB 2.0.0 is starting.
couchdb    |
couchdb    | [info] 2016-11-17T15:01:19.243966Z couchdb@mynode <0.205.0> -------- Starting couch_sup
couchdb    | [info] 2016-11-17T15:01:19.386974Z couchdb@mynode <0.204.0> -------- Apache CouchDB has started. Time to relax.
couchdb    |
couchdb    | [info] 2016-11-17T15:01:19.387390Z couchdb@mynode <0.204.0> -------- Apache CouchDB has started on http://<IP_ADDRESS>:5986/
couchdb    | [info] 2016-11-17T15:01:19.387914Z couchdb@mynode <0.7.0> -------- Application couch started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.388671Z couchdb@mynode <0.7.0> -------- Application ets_lru started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.471168Z couchdb@mynode <0.7.0> -------- Application rexi started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.628794Z couchdb@mynode <0.7.0> -------- Application mem3 started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.629043Z couchdb@mynode <0.7.0> -------- Application fabric started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.874313Z couchdb@mynode <0.7.0> -------- Application chttpd started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.939799Z couchdb@mynode <0.7.0> -------- Application couch_index started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.939845Z couchdb@mynode <0.7.0> -------- Application couch_mrview started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.939874Z couchdb@mynode <0.7.0> -------- Application couch_plugins started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.058859Z couchdb@mynode <0.7.0> -------- Application couch_replicator started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.058913Z couchdb@mynode <0.7.0> -------- Application couch_peruser started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.162602Z couchdb@mynode <0.7.0> -------- Application ddoc_cache started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.275910Z couchdb@mynode <0.7.0> -------- Application global_changes started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.275959Z couchdb@mynode <0.7.0> -------- Application jiffy started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.332753Z couchdb@mynode <0.7.0> -------- Application mango started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.361387Z couchdb@mynode <0.7.0> -------- Application setup started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.361780Z couchdb@mynode <0.7.0> -------- Application snappy started on node couchdb@mynode
couchdb    | [os_mon] cpu supervisor port (cpu_sup): Erlang has closed
couchdb    | [os_mon] memory supervisor port (memsup): Erlang has closed
**couchdb exited with code 143**

Could you share your compose file?
All misspelling thanks to my iPhone.
On Thu, Nov 17, 2016 at 5:04 PM +0200, "simon-temple"<EMAIL_ADDRESS>wrote:
Hi,
I'm using klaemo/couchdb:latest via docker-compose but it's not stopping with a clean zero exit code:
$docker-compose up couchdb
$docker-compose stop -t 50
Starting couchdb
Attaching to couchdb
couchdb    | -name couchdb@mynode
couchdb    | grep: /opt/couchdb/etc/local.d/*.ini: No such file or directory
couchdb    | ****************************************************
couchdb    | WARNING: CouchDB is running in Admin Party mode.
couchdb    |          This will allow anyone with access to the
couchdb    |          CouchDB port to access your database. In
couchdb    |          Docker's default configuration, this is
couchdb    |          effectively any other container on the same
couchdb    |          system.
couchdb    |          Use "-e COUCHDB_USER=admin -e COUCHDB_PASSWORD=password"
couchdb    |          to set it in "docker run".
couchdb    | ****************************************************
couchdb    | [info] 2016-11-17T15:01:19.152277Z couchdb@mynode <0.7.0> -------- Application couch_log started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.159696Z couchdb@mynode <0.7.0> -------- Application folsom started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.206457Z couchdb@mynode <0.7.0> -------- Application couch_stats started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.206857Z couchdb@mynode <0.7.0> -------- Application khash started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.218711Z couchdb@mynode <0.7.0> -------- Application couch_event started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.227111Z couchdb@mynode <0.7.0> -------- Application ibrowse started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.233544Z couchdb@mynode <0.7.0> -------- Application ioq started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.233877Z couchdb@mynode <0.7.0> -------- Application mochiweb started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.234354Z couchdb@mynode <0.7.0> -------- Application oauth started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.243788Z couchdb@mynode <0.204.0> -------- Apache CouchDB 2.0.0 is starting.
couchdb    |
couchdb    | [info] 2016-11-17T15:01:19.243966Z couchdb@mynode <0.205.0> -------- Starting couch_sup
couchdb    | [info] 2016-11-17T15:01:19.386974Z couchdb@mynode <0.204.0> -------- Apache CouchDB has started. Time to relax.
couchdb    |
couchdb    | [info] 2016-11-17T15:01:19.387390Z couchdb@mynode <0.204.0> -------- Apache CouchDB has started on http://<IP_ADDRESS>:5986/
couchdb    | [info] 2016-11-17T15:01:19.387914Z couchdb@mynode <0.7.0> -------- Application couch started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.388671Z couchdb@mynode <0.7.0> -------- Application ets_lru started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.471168Z couchdb@mynode <0.7.0> -------- Application rexi started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.628794Z couchdb@mynode <0.7.0> -------- Application mem3 started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.629043Z couchdb@mynode <0.7.0> -------- Application fabric started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.874313Z couchdb@mynode <0.7.0> -------- Application chttpd started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.939799Z couchdb@mynode <0.7.0> -------- Application couch_index started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.939845Z couchdb@mynode <0.7.0> -------- Application couch_mrview started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:19.939874Z couchdb@mynode <0.7.0> -------- Application couch_plugins started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.058859Z couchdb@mynode <0.7.0> -------- Application couch_replicator started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.058913Z couchdb@mynode <0.7.0> -------- Application couch_peruser started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.162602Z couchdb@mynode <0.7.0> -------- Application ddoc_cache started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.275910Z couchdb@mynode <0.7.0> -------- Application global_changes started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.275959Z couchdb@mynode <0.7.0> -------- Application jiffy started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.332753Z couchdb@mynode <0.7.0> -------- Application mango started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.361387Z couchdb@mynode <0.7.0> -------- Application setup started on node couchdb@mynode
couchdb    | [info] 2016-11-17T15:01:20.361780Z couchdb@mynode <0.7.0> -------- Application snappy started on node couchdb@mynode
couchdb    | [os_mon] cpu supervisor port (cpu_sup): Erlang has closed
couchdb    | [os_mon] memory supervisor port (memsup): Erlang has closed
couchdb exited with code 143
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Here you go...
version: '2'
networks:
main:
driver: bridge
services:
couchdb:
container_name: couchdb
image: klaemo/couchdb:latest
volumes:
- /opt/couchdb.data:/opt/couchdb/data
environment:
- COUCHDB_USER=user
- COUCHDB_PASSWORD=pwd
ports:
- "5984:5984"
networks:
- main

Kamino closed and cloned this issue to apache/couchdb-docker

Moved to https://github.com/apache/couchdb-docker/issues/6


Pulls/4.0/layout annoIEnces

fix layout issues in IE & Edge

Changes LGTM but I'd like to test this in other browsers before merging

Tests were run in chrome, safari, ie11, edge... and I think firefox (but not by me - @raissanorth I think you did though right?)

Here is a picture of:
Edge | Chrome
---------------
IE11 | Firefox

Previously the plus icon was below the bar on edge, and the line was higher than the edges of two elements, and overlays the top one (and the plus icon is a bit out):

Seems like you guys have the bases covered...


Allow the MDR scoring method to use arbitrary sklearn scoring metrics

e.g. metrics from http://scikit-learn.org/stable/modules/classes.html#classification-metrics

Rename add_score to scoring_function

Allow the MDR.score function to take parameters for the custom scoring function. e.g.,

def score(self, features, classes, scoring_function=None, **scoring_function_kwargs):
    ...
    else:
            return scoring_function(classes, new_feature, **scoring_function_kwargs)


Upgrading solace-services-info to 0.2.0

This will work for internal releases only. Expected to fail on travis CI for now.


Doesn't Work in Windows....

I think is not compatible with Windows 10 ?

Looks like there is a problem with Windows file paths. Latest commit should resolve this but I will leave it until I do some further testing on Windows machines.

I think this may help.............


Fixed #24112 -- Fixed counting of html elements if needle has no root element

The parse_html method (at the bottom of test/html.py) creates a instance of Parser and feeds in the html. The parser starts with a RootElement and adds to it as it parses, meaning that what parse_html gets back always has an additional RootElement above the rest of the HTML. It then removes this if it can, ie, if the RootElement only has one child (ie, the given HTML had a root element of its own).
This means that if the html passed to parse_html has no root node, it will retain its RootElement from the parser.
We can then check this when counting to see if the needle HTML has multiple top level elements. If it does, then counting becomes much more fiddly, so my current patch just raises a ValueError.
There aren't any tests as I couldn't find any existing ones for this module. Should things within the test module get tests?

Yes, tests should be added in tests/test_utils/tests.py. Even if assertInHTML itself is not in those tests, you'll find related tests (testing parse_html and count).

OK, thanks for that. I'll add relevant tests ASAP.

The PR currently contains two commits. (I know I'll want to squash these before a merge) The first (1a56cea) adds a failing test that demonstrates what I believe is incorrect behaviour.
Specifically:
FAIL: test_html_contain (test_utils.tests.HTMLEqualTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File ".../django/tests/test_utils/tests.py", line 579, in test_html_contain
    self.assertIn(dom1, dom2)
AssertionError:

foo

bar

 not found in

foo

bar

which seems wrong to be saying "not found in".
The second commit (430531c) rectifies this by throwing a ValueError if the needle HTML has no root element, unless if the two elements are equal, which preserves existing functionality.
I believe that the net effect of my change is that if where before you got AssertionError("... not found in ...") and it made no sense, you now get ValueError('Needle HTML does not have a root element').

Please send a new PR if you can reply and/or update per my comments, thanks!


Adding 'partial' prefix for class / interface generation

Hello, I had to modify the generator to let all classes / interfaces have a 'partial' declaration - this way I can re-generate my whole repository and retain all the changes too - in separate files.

Hi @kbalint, yea that's a good idea. if you'd like you can submit a pull request with the partial modifier.
related to #4


404 at /engine/installation/)

I found a broken link : https://docs.docker.com/engine/installation/)

Hi @9491241722, thanks for sharing the issue. Where did you find this link?
It looks like there's an extra ) at the end of the link.
The correct link is https://docs.docker.com/engine/installation/, which redirects to https://docs.docker.com/engine/install/

Closing as stale. If the source link is inside docs.docker.com, please let us know.


optional field not optional

The following line
Nov 22 18:57:55 SERVER postfix/qmgr[1671]: QID: from=<>, size=3618, nrcpt=1 (queue active), QID: from=<>, size=3618, nrcpt=1 (queue active)
is parsed such that the field
postfix_from
contains
>
=> postfix_from should be optional

You input is correctly parsed into the postfix_keyvalue_data field. The kv plugin should process the data correctly using the trim_value parameter as set at https://github.com/whyscream/postfix-grok-patterns/blob/44d8da8e3772c53e12d472b04e17a678057257da/50-filter-postfix.conf#L174, but maybe you founda bug in the kv plugin in your version of logstash?
In newer versions of logstash you could try to adapt the kv filter setting to use include_brackets or remove_char_value. I'm reluctant to add that to the repositoy when the current method is supported by all versions of logstash and the other options are only available in newer releases..

Ok. Unfortunately, I don't have the data anymore, I had it only for test purposes. I'll inverstigate further if it occurs again and close the ticket for now.
Nice work by the way :)


Unable to create a character or login with an account that has a character already created.

Attached is the error I receive after logging in and either creating a character or, logging in then arriving at the character selection screen.
Note:this is a fresh build after pulling from here, and freshly importing the project from the asset store - the asset store server works as intended but this server nets me this error.
ArgumentException: Argument_InvalidOffLen
System.ArraySegment1[T]..ctor (System.Byte[] array, System.Int32 offset, System.Int32 count) (at <27124aa0e30a41659b903b822b959bc7>:0) LiteNetLib.Utils.NetDataReader.GetBytesSegment (System.Int32 count) (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Plugins/LiteNetLib/Utils/NetDataReader.cs:323) LiteNetLib.Utils.NetDataReader.GetString () (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Plugins/LiteNetLib/Utils/NetDataReader.cs:316) MultiplayerARPG.CharacterItem.Deserialize (LiteNetLib.Utils.NetDataReader reader) (at Assets/UnityMultiplayerARPG/Core/SharedData/Scripts/CharacterData/RelatesData/CharacterItem.cs:191) LiteNetLib.Utils.NetDataReader.Get[T] (System.Func1[TResult] constructor) (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Plugins/LiteNetLib/Utils/NetDataReader.cs:345)
MultiplayerARPG.PlayerCharacterDataExtensions.DeserializeCharacterData[T] (T characterData, LiteNetLib.Utils.NetDataReader reader, System.Boolean withTransforms, System.Boolean withEquipWeapons, System.Boolean withAttributes, System.Boolean withSkills, System.Boolean withSkillUsages, System.Boolean withBuffs, System.Boolean withEquipItems, System.Boolean withNonEquipItems, System.Boolean withSummons, System.Boolean withHotkeys, System.Boolean withQuests, System.Boolean withCurrencies, System.Boolean withServerCustomData, System.Boolean withPrivateCustomData, System.Boolean withPublicCustomData) (at Assets/UnityMultiplayerARPG/Core/SharedData/Scripts/CharacterData/PlayerCharacterDataExtensions.cs:471)
MultiplayerARPG.MMO.ResponseCharactersMessage.Deserialize (LiteNetLib.Utils.NetDataReader reader) (at Assets/UnityMultiplayerARPG/MMO/Scripts/MMOGame/Src/Central/Messages/Character/ResponseCharactersMessage.cs:20)
LiteNetLibManager.LiteNetLibResponseHandler2[TRequest,TResponse].InvokeResponse (LiteNetLibManager.ResponseHandlerData responseHandlerData, LiteNetLibManager.AckResponseCode responseCode, LiteNetLibManager.ResponseDelegate1[TResponse] anotherResponseHandler) (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Scripts/RequestResponse/LiteNetLibResponseHandler.cs:30)
LiteNetLibManager.LiteNetLibRequestCallback.Response (System.Int64 connectionId, LiteNetLib.Utils.NetDataReader reader, LiteNetLibManager.AckResponseCode responseCode) (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Scripts/RequestResponse/LiteNetLibRequestCallback.cs:31)
LiteNetLibManager.TransportHandler.ProceedResponse (System.Int64 connectionId, LiteNetLib.Utils.NetDataReader reader) (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Scripts/TransportHandler.cs:235)
LiteNetLibManager.TransportHandler.ReadPacket (System.Int64 connectionId, LiteNetLib.Utils.NetDataReader reader) (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Scripts/TransportHandler.cs:73)
LiteNetLibManager.LiteNetLibClient.OnClientReceive (LiteNetLibManager.TransportEventData eventData) (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Scripts/LiteNetLibClient.cs:77)
LiteNetLibManager.LiteNetLibClient.Update () (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Scripts/LiteNetLibClient.cs:36)
LiteNetLibManager.LiteNetLibManager.Update () (at Assets/UnityMultiplayerARPG/Core/LiteNetLibManager/Scripts/LiteNetLibManager.cs:150)
MultiplayerARPG.MMO.CentralNetworkManager.Update () (at Assets/UnityMultiplayerARPG/MMO/Scripts/MMOGame/Src/Central/CentralNetworkManager.cs:137)
I've gone through a few times trying to figure it out but ultimately I fail 🤣 any help would be appreciated

Hello, try to export data button from game instance component

To export it to this project -> Configs folder

Yes, it can be related

@insthync any ideas on a fix?

May check out on the version which match to your game or just download and use latest version which was uploaded yesterday.


[DEV] Evolve Captured

Expected Behavior
Catch a Pokémon
Check if it can be evolved, if so evolve it
Check release parameters and see if there are Pokémon, especially the evolved form, to release
Actual Behavior
Catches Pokémon
Checks if it can be evolved, if so evolves it
Tries to release the unevolved form leave the evolved Pokémon in bag to take up space.
Steps to Reproduce
Turn the setting to evolve when caught
Other Information
OS:  Windows 10
Git Commit: 70e8d50d8ad3ed59dbf7b718d4cc98b168e957d6

@Foulwerp Can you please be more specific?

I think this sentence is more accurate as a title :

Tries to release the unevolved form leave the evolved Pokémon in bag to take up space.

Okay. Bot catches a Pokémon, for example Pidgey, if it can evolve it does so right away. Then instantly tries to exchange Pidgey for candy not Pidgeotto . Pidgeotto then stays in your inventory taking up space.
I would think that it would attempt to exchange the evolved form not the unevolved form after evolving, or at run through the Pokémon to see if any can be released based on configs.


New global property alpha-background

Emacs 29 has introduced alpha-background frame parameter, which I am currently using for transparency. As a result, writeroom-mode no longer toggles transparency for me. I have added the following lines to my configuration to restore the previous toggle behavior
(define-writeroom-global-effect alpha-background 100)
(add-to-list 'writeroom-global-effects 'writeroom-set-alpha-background)

I don't know if it is backwards compatible to add these to the mainwriteroom-modesource, but I think this is the correct behavior in the newer Emacs versions.

I think it should be safe to add this, because Emacs should ignore frame parameters that it doesn't know about. I'll have to test it, though.
Thanks for the suggestion.


BMC Remedy Integration Pack

This is a first version for BMC Remedy Integration Pack

Very nice!
I see a few linting issues (super small... like # of spaces between imports and class declaration in a few of the actions). but those were not exposed to you automatically because I disabled the Travis CI check on PR. Sorry about that!
Anyway, instead of enumerating through them, would you either...

Run the tests locally make and see what pops up, or...
Push another commit to re-trigger Travis

I don't see a re-trigger option with the new GitHub -> Travis integration, so apologies on asking you to do this. However, that should at least get those squared away that we can merge this in.
The rest looks nice! I especially like the sensor contribution! :heart_eyes:
Thanks so much for the contribution! :bow:

https://stackstorm.slack.com/archives/stackstorm/p1449719068011672

Sorry about the errors. I'll recheck entire pack.

@renancaldeira Would you mind addressing the comments and taking the PR to completion? Seems like this could be useful to people :).

Closing this in favour of #604


Add code highlighting

Added code highlighting to storsimple-data-manager-change-default-blob-path.md

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

@alkohli
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

The changes look good. #sign-off


Is there a way to store the map?

Hello. I didn't find the interface to save the map. Meanwhile, the visualization is different from that in your RSS paper. I wonder if there is a way to output the map of yours and Octomap into .bt format for further evaluation and visualization?

Hi Zike! The only interface to access the data for all the active bricks from an MRFMap object is to get tuples of occupied voxels via
https://github.com/mrfmap/mrfmap/blob/9ccbca8454ebc3cec825d00d7acaf4830720068d/include/mrfmap/GVDBMRFMap.h#L59
Here's an instance of the python script accessing the voxel data and saving it as a binary numpy file to disk
https://github.com/mrfmap/mrfmap_ros/blob/8c1e108860ff297f39591d97f8f8ce2937b29a51/nodes/dataset_evaluation/rss_dataset_eval.py#L115
(Which is internally calling this method:
https://github.com/mrfmap/mrfmap_ros/blob/8c1e108860ff297f39591d97f8f8ce2937b29a51/src/mrfmap_ros/MRFMapGenerator.py#L38
)
To save it in a custom file format you'll have to write a wrapper to do so with the voxel location and occupancy data. Hope that's helpful!
Regarding visualisation, could you clarify which visualisation are you talking about?

Hi Zike! The only interface to access the data for all the active bricks from an MRFMap object is to get tuples of occupied voxels via
https://github.com/mrfmap/mrfmap/blob/9ccbca8454ebc3cec825d00d7acaf4830720068d/include/mrfmap/GVDBMRFMap.h#L59
Here's an instance of the python script accessing the voxel data and saving it as a binary numpy file to disk
https://github.com/mrfmap/mrfmap_ros/blob/8c1e108860ff297f39591d97f8f8ce2937b29a51/nodes/dataset_evaluation/rss_dataset_eval.py#L115
(Which is internally calling this method:
https://github.com/mrfmap/mrfmap_ros/blob/8c1e108860ff297f39591d97f8f8ce2937b29a51/src/mrfmap_ros/MRFMapGenerator.py#L38
)
To save it in a custom file format you'll have to write a wrapper to do so with the voxel location and occupancy data. Hope that's helpful!
Regarding visualisation, could you clarify which visualisation are you talking about?

The Pangolin visualization is different from Fig. 8 of your paper(attached as follows).
Another issue is about the intrinsic matrix. Can mrfmap accept negative focal length (Original ICL-NUIM)? I generate a new bag file from original ICL-NUIM livingroom kt-1, but the projected point cloud is not correct. Bag file from Burghers dataset is working fine.

Hi! Fig 8 was the Mayavi generated rendering of the inferred volume of the ICL NUIM bag - did you forget to upload a screenshot of that? Irrespective, if you can tell me how it is different it would be useful - there might be some config file parameters that I didn't change.
Well, I've designed it for 'normal' camera intrinsics. The negative focal length is a really annoying feature of the ICL NUIM dataset because of how they generated it (using POVRay) and I don't think a simple swap of one of the focal lengths will work for MRFMaps because the camera intrinsics are used for a lot of internal operations that would probably fail. I suggest reprojecting the dataset and generating a 'fixed' bag file for using the other datasets.
Another thing to note that I vaguely recall is that you might have to rotate the camera odometry to the ROS Z up coordinate frame. I think there's a special pre-rotation flag I have included for the ICL datasets to take that into account
e.g.
https://github.com/mrfmap/mrfmap_ros/blob/8c1e108860ff297f39591d97f8f8ce2937b29a51/config/ICL.yaml#L6
which is incorporated at
https://github.com/mrfmap/mrfmap_ros/blob/d4f1bfab2b2ce19586ef14bcf5a5727f8e655027/src/GVDBBatchMapCreator.cpp#L87
A sanity check is to make sure that the camera frustum isn't backprojecting the pointcloud beyond the volume boundaries in the Visualiser

Hi! Fig 8 was the Mayavi generated rendering of the inferred volume of the ICL NUIM bag - did you forget to upload a screenshot of that? Irrespective, if you can tell me how it is different it would be useful - there might be some config file parameters that I didn't change.
Well, I've designed it for 'normal' camera intrinsics. The negative focal length is a really annoying feature of the ICL NUIM dataset because of how they generated it (using POVRay) and I don't think a simple swap of one of the focal lengths will work for MRFMaps because the camera intrinsics are used for a lot of internal operations that would probably fail. I suggest reprojecting the dataset and generating a 'fixed' bag file for using the other datasets.
Another thing to note that I vaguely recall is that you might have to rotate the camera odometry to the ROS Z up coordinate frame. I think there's a special pre-rotation flag I have included for the ICL datasets to take that into account
e.g.
https://github.com/mrfmap/mrfmap_ros/blob/8c1e108860ff297f39591d97f8f8ce2937b29a51/config/ICL.yaml#L6
https://github.com/mrfmap/mrfmap_ros/blob/8c1e108860ff297f39591d97f8f8ce2937b29a51/config/ICL.yaml#L10
which is incorporated at
https://github.com/mrfmap/mrfmap_ros/blob/d4f1bfab2b2ce19586ef14bcf5a5727f8e655027/src/GVDBBatchMapCreator.cpp#L87
A sanity check is to make sure that the camera frustum isn't backprojecting the pointcloud beyond the volume boundaries in the Visualiser

Hi, I managed to run it on ICL dataset by adjusting the projection and the corresponding volume boundary. Really appreciate your kindness for such thourough suggestions. It is a little bit late for me to find this repo and I am in a hurry. Nice work by the way!


Question about degenerated image quality

Hello Xuyang,
Thanks for your great work and open-source code.
About the experiments on degenerated image quality, TransFusion work well. I'm confused and have two questions.
(1) What about other value(instead of 0) padding,  does it make greater effect on TransFusion performance ?
(2) How is TransFusion affected when the image quality is very bad (without any useful information in case of light breakdown at night) ? What attention map (Figure 3) should be ?
Looking forward to your reply.

Hi, Thanks for your interest.

We have tried zero-padding or random-padding, in both cases TransFusion works well.
Note that the network indeed does not have any information about the image quality, but relies on the attention to dynamically determine whether the image information is useful or not. The zero-padding (to simulate the missing camera) and random-padding experiments are good examples of that situations. And the attention map is expected to be sparse and have a low value for all the image pixels.

If you are interested in the robustness of image quality, here is a related paper: "Benchmarking the Robustness of LiDAR-Camera Fusion for 3D Object Detection" which proposes another noisy camera data case (camera lens occlusions)

Thanks for your patient reply which helps me figure out the results.


Resize the music box

🐛 Bug Report
The music box page seems pretty cool and well done. But it seems to require scrolling on my viewport. So i would like to resize it to avoid the scrolling in that page
Add relevant screenshot or video (if any)
(Add here)

Please kindly look into it assign it to me

Sure @ankh77sb , assigned you the issue. Kindly align the back button to the top left corner also.

I have made a pull request. Please suggest changes if any

Sure @ankh77sb


[Modulated DeformableConv] fix a bug for deformable_group>1

Fix a bug for deformable_group>1.
The performance of Faster-RCNN R-50-FPN  is increased by 0.2 (40.2 -> 40.4)

btw, how to upload the model ?

Thanks for the fix! You could provide a google drive or dropbox link and we can upload it to AWS.
It is better to add another config file instead of modifying the existing configs, since deformable_groups=1 is the default setting provided by the authors to reproduce DCNv2.

@hellock    Here is the LINK for the model and log. I also create a new script (configs/dcn/faster_rcnn_mdconv_c3-c5_group4_r50_fpn_1x.py) for this exp. Thanks


Timeout isn't working for the url (onion)

Hello,
Using the following code with pylacus
uuid = lacus.enqueue(url=host['slug'], general_timeout_in_sec=90)
The timeout is not respected with Lacus (results by running tools/monitor.py)
{
  "max_concurrent_captures": 150,
  "max_capture_time": 3600,
  "ongoing_captures": 3,
  "captures_time": {
    "f7b3fc60-9dea-474b-bf8e-fe63e0605377": 228.637663,
    "be37d5c2-494a-4184-9da5-6474df9c29f0": 230.759614,
    "4ab61217-f770-4df0-842c-640c7c439df9": 266.525206
  },
  "enqueued_captures": 0
}

Some captures get stuck and the playwright timeouts aren't respected (that's the 90sec you pass as parameter), but the capture (often) finishes, at some point. If it doesn't, the max_capture_time it the one that kicks in and it will kill the capture regardless of its status.
I'll review the script again to make sure I'm not forgetting a timeout setting somewhere, but in your case, I'd recommend to reduce the max_capture_time to something like 600s or so, to make sure a stuck capture is killed somewhat faster.
general_timeout_in_sec evolved into a best effort setting.

Seems to be solved.


Use the nativescript-floatingactionbutton module

Sorry for the polluted PR, I cloned the repo with Visual Studio and it added some junk and I didn't realize it. I only added the module to the package.json file, added the ic_history.png file for the respective android app_resources folders and then updated the list.xml to use the Fab component.
I understand if you think this might be too much to include in the sample, no hard feelings if you think it's best to not include it. I just prefer the native 'ripple' effect when pressing and getting rid of that fab.png image. Ping me if you want to discuss.

Thanks @bradmartin! No worries about the extraneous files; I can always clean those up later. I’ll look at this over later today.

Merged. Thanks! 🎉
Now I need help with some checkboxes 😀


Specifying testFileRegex does not work

I'm trying to use the feature added in #33 but I always get an error:
Chrome 45.0.2454 (Mac OS X 10.10.4) ERROR
  TypeError: testFileRegexp.test is not a function

It seems that the value is not properly converted to a RegExp object when passed between the index.js code, running in node, and the adapter.js code, running in the browser.
I have tried both passing the value as a string (in which case it ends up as a string in browser land, but is never converted to a RegExp object), or as a RegExp in the Karma config file (in which case it gets passed as an opaque Object in browser land, which still does not have a test method).
I'd be happy to open a PR, but I'm not sure what approach you would want: should the value be specified as a string in the config file, and converted to a new RegExp(config.testFileRegex)? Or should it work to specify the regex as a RegExp object in the Karma config file? (I haven't looked at the runner/browser IPC to see how/if this is possible).
Here are the two different ways I've tried specifying the config:
    systemjs: {
      configFile: 'system.config.js',
      testFileRegex: /test_.*\.js/
    },
    // ...

and
    systemjs: {
      configFile: 'system.config.js',
      testFileRegex: 'test_.*\.js'
    },
    // ...

Hi @spiralman
Well spotted.  The adapter.js code definitely expects a RegExp, but the unit tests show strings.  I'm pretty sure you can only pass strings from the framework down to the adapter.
However in the 0.8 branch (which I'm about to merge) the testFileRegex and testFileSuffix properties have been deprecated in favour of simply importing everything that has {included: true} (see #38), which should simplify things a lot for users.


gnu.mapping.SimpleSymbol cannot be cast to java.util.List

What and Where?
I encountered this problem after verifying that I was doing right according to the source code.
https://github.com/vknow360/CustomWebView/blob/d0db408b4cb5f0256083eb23de6898e50287c1f0/com/sunny/CustomWebView/CustomWebView.java#L835
My blocks:

Error: in title.

Can you check this aix?
com.sunny.CustomWebView.zip

I have this issue too, I'll try that aix.

I have this issue too, I'll try that aix.

Try using v12beta.

Alright, and just to add the aix you sent here didn't work.

@UCYT5040 Can you check this aix?
com.sunny.CustomWebView.zip
Now it uses this method:
public void LoadWithHeaders(String url, YailList headerList) {
        if (headerList.size() != 0) {
            java.util.Map<String, String> header = new HashMap<String, String>();
            String[] str = headerList.toStringArray();
            for (String s : str) {
                String[] s2 = s.split(",");
                header.put(s2[0], s2[1]);
            }
            webView.loadUrl(url, header);
        } else {
            GoToUrl(url);
        }
}

Ok I will try it.

It shows this error
length=1; index=1
I used both a list and a dictionary for the headers.

@vknow360
Pinging in case you didn't get a notification

It shows this error length=1; index=1 I used both a list and a dictionary for the headers.

Try passing a list of CSV headers.
For example:

I did them like this neither worked
https://img.ucyt5040.repl.co/img.png

@vknow360

You have to pass headers in this way:

Closing due to no response

@vknow360

I have it just like in the picture you showed and the companion just crashes.

In last push, the method signature has been changed. It now accepts a dictionary as headers.

tried beta extension and your updated too, same error.


RAMIP: new variables

These are RAMIP variables copied over from the RAMIPday and RAMIPmon mip tables here
Modifications to the RAMIP tables:

Daily ozone variable on 3 levels changed to have modeling_realm aerosol for consistency with other ozone variables
CF Standard names:

truncated several standard names which had _at_the_surface appended -- these are not in the latest (v86) release of the CF standard names and I don't see a proposal for adding them.
removed standard names which have not yet completed their registration, i.e. are not part of the latest CF standard names release -- I believe these are not far from being included, but this is not available yet.

@ramipaerosols, please check that you are happy with these changes

@wolfiex, let me know what needs to be done with the JSONLD side of things


ModuleNotFoundError: No module named 'grpc_reflection.v1alpha.proto_reflection_descriptor_database'

What version of gRPC and what language are you using?
language: python
grpcio: 1.45.0
grpcio-reflection: 1.45.0
What operating system (Linux, Windows,...) and version?
Windows
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 3.7.4
What did you do?
I follow the gRPC Python Server Reflection, and when I import the ProtoReflectionDescriptorDatabase then it raise a error
import grpc
from grpc_reflection.v1alpha.proto_reflection_descriptor_database import ProtoReflectionDescriptorDatabase

ModuleNotFoundError: No module named 'grpc_reflection.v1alpha.proto_reflection_descriptor_database'

What did you expect to see?
proto_reflection_descriptor_database can be built into grpc_reflection?
What did you see instead?
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

@Su-yj We haven't released the reflection client feature for gRPC Python yet (not in https://github.com/grpc/grpc/releases). It was targeting v1.45.x, but got reverted due to a protobuf compatibility issue. It will be released in v1.46.x in a few weeks.

Sounds like this was merged in #29085


bug: Treesitter error with nvim-cmp docs

Did you check docs and existing issues?

[X] I have read all the noice.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of noice.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev-881+g865ba42e0
Operating system/version
Linux Mint 21.3
Describe the bug
When auto-completing in Python files and the nvim-cmp completion menu pops up, when traversing through the available entries the following error pops up
   Error  07:17:11 PM msg_show.lua_error Error executing vim.schedule lua callback: ...y/share/nvim/runtime/lua/vim/treesitter/languagetree.lua:111: No parser for language "text"
stack traceback:
	[C]: in function 'assert'
	...y/share/nvim/runtime/lua/vim/treesitter/languagetree.lua:111: in function 'new'
	.../.repro/plugins/noice.nvim/lua/noice/text/treesitter.lua:39: in function 'highlight'
	....00.1x/.repro/plugins/noice.nvim/lua/noice/text/init.lua:68: in function 'highlight'
	...Tx 6.00.1x/.repro/plugins/nui.nvim/lua/nui/line/init.lua:58: in function 'highlight'
	...Tx 6.00.1x/.repro/plugins/nui.nvim/lua/nui/line/init.lua:73: in function 'render'
	...00.1x/.repro/plugins/noice.nvim/lua/noice/text/block.lua:98: in function 'render'
	....1x/.repro/plugins/noice.nvim/lua/noice/lsp/override.lua:52: in function 'stylize_markdown'
	...00.1x/.repro/plugins/nvim-cmp/lua/cmp/view/docs_view.lua:64: in function 'open'
	...on/MITx 6.00.1x/.repro/plugins/nvim-cmp/lua/cmp/view.lua:299: in function 'callback'
	... 6.00.1x/.repro/plugins/nvim-cmp/lua/cmp/utils/async.lua:138: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

I don't see this error in Lua files and since I don't do anything else except Python and Lua, I'm not sure if this also occurs with other languages as well.
Steps To Reproduce
I'm using LazyVim distro but was also able to reproduce this without LazyVim with this minimal repro.

Open a Python file
Start typing print and go through the available entries
The error should pop up

Expected Behavior
No error should pop up. I'm guessing this has something to do with the latest Treesitter changes on Neovim nightly, because it's not reproducible with Neovim stable 0.10.1.
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "folke/noice.nvim", opts = {} },
    -- add any other plugins here
  },
})

I think I know what's causing this. There's an upstream recent change where get_parser no longer errors when no parser is available and returns nil instead. Will look into it tomorrow. not home right now

No problem. Thanks a lot :smile:

Should be fixed now. Let me know if the problem would persist

All good now. If I find something else over the next couple of days I'll comment here.
Tnx again :smiley:


Add event which would emit info about check-in and check-out date

Is your feature request related to a problem? Please describe.
the onHover event is not sufficient. If I want to change data based on first and second day selected, for example outside datepicker and can't do that with "attributes" passed in function(date, attributes) on hover event.
On mobile it is even worse because when you click on day item, attributes only give "day-item" without "is-start/end-date"
Describe the solution you'd like
I want new event onClick: function(date, oldAttributes, newAttributes). I want new and old attributes because when second date is clicked on mobile it's day-itemand after click it's alsois-end-date`
Describe alternatives you've considered
I tried using existing onHover event, but encountered many bugs in this hacky way and codes becomes bloated with validations and skip variables. OnClick event should really be there
Additional context
after first click on day item I want to add text under datepicker dateOne - check-out-date and after second day clicked dateOne - dateTwo

+1 would be great!

https://wakirin.github.io/Litepicker/docs/events#preselect

https://wakirin.github.io/Litepicker/docs/events#preselect


Add test for list joining

The input rules for list joining are currently not tested, because without keymaps (#61), there is no way to "exit" a list (Enter is provided by the keymap)

https://github.com/marekdedic/prosemirror-remark/blob/c5b10c2e3e58ad5bb0dc9254f6c7796dec06f0a6/__tests__/syntax-extensions/OrderedListExtension.test.ts#L345-L365
https://github.com/marekdedic/prosemirror-remark/blob/c5b10c2e3e58ad5bb0dc9254f6c7796dec06f0a6/__tests__/syntax-extensions/UnorderedListExtension.test.ts#L310-L330


Have script pull thumbnail from repo automatically

Once https://github.com/manubot/manubot.org/issues/24 and https://github.com/manubot/rootstock/issues/267 are resolved, this can be done. Hopefully it's simple to do if pulling a known filename from a known location.
Related: https://github.com/manubot/catalog/pull/9

As per https://github.com/manubot/rootstock/issues/267#issuecomment-529611462, would it be best to extract the thumbnail URL from the HTML metadata rather than the source repository?
I imagine we still want to keep thumbnail_url as part of the catalog.yml to override author-set thumbnails or for setting thumbnails that are not set upstream.

I think that is over complicating it. I think the catalog should have a nice image, and if the one upstream isn't good enough, it's on the author to incorporate the feedback from the catalog maintainers. And if there is no option for a thumbnail, then the manubot logo gets used, as it should.
I think it should be simple. Just one image: thumbnail.png.  We can look into how the typical thumbnail looks at favicon scale, but I think it will be okay.
And if it's not, we can do this: have the users also specify thumbnail-small.png, which would be a simplified version of the figure, if they want to provide one. If they don't provide the small version, then thumbnail.png is used in places where the favicon is larger (android and ios app/bookmark icons, windows 10 start menu tiles, social media share thumbnail) and the manubot favicon is used in places where the favicon is very small (just browser bookmarks, really). If neither is provided, then the manubot logo is used for all of it.

I think the catalog should have a nice image, and if the one upstream isn't good enough, it's on the author to incorporate the feedback from the catalog maintainers.

If there is no way to set/override the image from the catalog.yml, that presents problems IMO. Since we don't control manuscript repositories we cannot actually require users to set thumbnails that meet our guidelines. We could not add manuscripts to the catalog unless they have a proper thumbnail, but I think we need to err on having the catalog be more complete than incomplete. Furthermore, there are manuscripts like the one added in https://github.com/manubot/catalog/pull/8 that don't follow the standard manubot workflow. In this cases, the easiest solution was to upload the thumbnail to a github comment and use that URL. It's not much added implementation complexity to support thumbnail_url overrides, but may come in handy in several places.
thumbnail-small.png for favicons and thumbnail.png for catalog and large previews makes sense to me.

I'd recommend just reinforcing the rules by not accepting manuscripts with bad thumbnails, but that is ultimately your call. As for the workflow though, that person would just have to put the thumbnail.png in that repo in the right location, as we'd just be scraping it from that... I don't see why that would be a problem with someone not using the manubot workflow.
Also it's probably better to have the image in the repo itself than in a github comment, imo.

As for the workflow though, that person would just have to put the thumbnail.png in that repo in the right location

Repositories like sbonaretti/FAIR_metrics don't follow the standard directory layout.

I'd recommend just reinforcing the rules by not accepting manuscripts with bad thumbnails

We will encourage proper thumbnails but if we're unsuccessful for a certain manuscript, I still want the ability to add it with either the bad thumbnail suppressed or set to a third-party location (like a GitHub Issue comment). Not ideal, but I'm hoping the incentive to have a nice thumbnail displayed when a manuscript is shared will help incentivize users towards the desired behavior.

thumbnail.png in that repo in the right location, as we'd just be scraping it from that

Were there downsides to scraping from the HTML metadata that we'll be setting? The benefits of using the HTML metadata rather than a file location is that it's more flexible by supporting manuscript-external thumbnails specified by URL, possibly works better for versioned manuscript links, doesn't require inspecting file contents of repositories (which can be more complex). The Manubot software would still look to a default file location to set the HTML metadata, but the end users would just read the URL path from the HTML.


Cannot see source code to add breakpoints [gcc]

Hi,
I have a simple C program that I'm compiling with gcc. It generates the .pdb file but when I open the executable with raddebugger I cannot see the source code to add breakpoints. It generates a raddbg file but the debugger just shows a disassembled code when I run it. Am I doing something wrong?
#include <stdio.h>

int main(int argc, char *args[])
{
    int n = 1;
    printf("%d", n);
    return 0;
}

I'm using this command to compile:
gcc main.c -g -ggdb -gcodeview

Thanks.

As far as I know codeview support in gcc is in very early stage. It is pretty incomplete. I tried it last year and if I remember correctly they only had line number info, no local symbols/types, and sometimes pdb file was not loading properly in Visual Studio at all for me. Not sure how much it has changed now. I think they plan to officially release it in gcc 14, but no idea about how complete it will be.
It should be no problem to support what gcc generates, but they should first implement more complete pdb generation, it does not make much sense to spend resources on this in debugger at this moment. At least until they'll include support for codeview output in official release.

Thanks!


bump to latest importlib-metadata, only use with Python < 3.8

Relying on an older importlib-metadata is a problem when other Python packages are installed that require a newer version (see https://github.com/EESSI/software-layer/issues/81 for example).
In addition, importlib-metadata is part of the Python standard library since 3.8, so then you don't need a separate installation at all (cfr. https://github.com/python/importlib_metadata#readme).

Can I test this patch?

@boegel can you make a PR on my branch where I bump all the package versions? Or should I just add the python condition: https://github.com/eth-cscs/reframe/pull/1905

@boegel Thanks for the PR. Since @teojgo was faster with his #1905 PR, I will make this a duplicate. @teojgo Please update your PR with the python version.

@teojgo See https://github.com/eth-cscs/reframe/pull/1905/files#r606134681


feat(handle, makeObservable): new function with observable's on()

makeObservable, the new function with observable's on()

maybe not need


Umbraco 8.6.3 > Doesn't Work

I've installed the package by Our.Umbraco, GitHub and tried Local install, and it doesn't do anything. I don't see a "sitemap.xml" file, nor does going to the url "/sitemap.xml" show anything (it's 404 error).  There isn't much instruction on what else to do, if I'm supposed to.  I've tried publishing a page, building the project, ensuring IISExpress is stopped and re-running the project, etc.

Hey @deladroid!
Thanks for trying Friendly Sitemap! Sorry to hear you've had issues getting things up and running...
The sitemap is a virtual sitemap and therefore you won't see a physical sitemap.xml file on disk, but you should be able to browse to /sitemap.xml and see it.
In order to bind /sitemap.xml to a site the root node needs to have a domain assigned. Can you confirm you have a hostname assigned to your site root under the "Culture & Hostnames" menu?
Cheers,
Callum

Hey @deladroid,
I've had a brainwave over the weekend about what this issue might be!
The sitemap needs to be enabled via an app setting in your project's web.config file in order to work.
<add key="Umbraco.Sitemap.EnableSitemap" value="true" />

When installing via NuGet this is done automatically for you, but if you have installed via Our Umbraco or the backoffice you need to do this manually yourself.
If you can, I suggest installing the package via NuGet. If not, you should be able to add the above line to your web.config file.
Please let me know if this resolves your issue! 😊
Cheers,
Callum

Thanks, I'll try this soon, and report back.

I'm facing the same issue, running Umbraco 8.5.4 and version 1.1.1 of your package. Unfortunately adding the key to the project's web.config file did not resolve the issue. /sitemap.xml returns a 404.

I have this package installed and working on Umbraco 8.6.4. Make sure you have a domain set in Culture and Hostnames
for the start node

Tried with 8.6.4 just now, ensured domain is set on the root node and added the key Umbraco.Sitemap.EnableSitemap to the project's web.config file (set to true). Still no luck (404)... :(

Hey @rongrisnich @john-mckillip!
Thanks for trying out Friendly Sitemap – really sorry you're having issues.
Can you both confirm for me how the package was installed? Was this via NuGet? Or Our Umbraco / the Umbraco backoffice?
Can you both further confirm if there is a DLL / reference in your projects for Our.Umbraco.Extensions.Routing?
Cheers,
Callum

I installed the package using the Umbraco backoffice.
Our.Umbraco.Extensions.Routing.dll is not there. Our.Umbraco.FriendlySitemap.dll is added to the root of the web application.

Aha! Thanks @rongrisnich!
Looks like this is the issue... For some reason the Umbraco package zip does not include that DLL, which is required to configure the routing. I will fix this and release a patch shortly.
In the mean time you can download the DLL here and manually add it to your project to hopefully resolve your issue.
Cheers,
Callum

Downloaded Our.Umbraco.Extensions.Routing.dll and added it to the bin folder, but unfortunately no luck yet...

Just to confirm @rongrisnich, the bin folder should now have both Our.Umbraco.FriendlySitemap and Our.Umbraco.Extensions.Routing... Do you still have the same error?

Strangly, the file Our.Umbraco.FriendlySitemap.dll was placed in the root folder (not bin). Moving this file into the bin folder resolves the issue!

Fantastic, glad we found a way to get it to work.
I know what the issue is now and will fix accordingly 😊

I just came to this today, with the same issue. I have to note that indeed the dll's were in the root of the site and not the bin folder and the routing dll was missing.
I would like to add that this also happens to the friendly robot package. Don't forget to update that one.
For the record, I installed from the package dashboard in umbraco. 8.6.3. Once copied everything worked.

Same issue for me after installing via packages section on dashboard, but resolved after doing 4 things mentioned above (not sure if all steps were needed or not)

 in web.config
Moved Our.Umbraco.FriendlySitemap.dll from root folder into bin
Downloaded the Our.Umbraco.Extensions.Routing.dll linked to above, copy to bin folder
Added my domain to 'Culture and Hostnames' via 'root node' > 'actions' > 'culture and hostnames' on the umbraco dashboard.

not sure what i am doing wrong did above 4 steps still no luck

Sorry for such a delay in me replying.  I put a domain in as stated above, and it's working (installed via Umbraco "Packages").

Confirm that Our.Umbraco.FriendlySitemap.DLL is in bin
Confirm  is in web.config
Confirm Domain added at "Root > Culture and Hostnames" (Ie., local dev "https://localhost:port")

Hello everyone!
Thanks for your patience.
For anyone who was still running into issues with the Our Umbraco version of the package, the latest v1.2.0 is now released and should install without any issues like before!
Cheers,
Callum

Hey @deladroid!
Thanks for trying Friendly Sitemap! Sorry to hear you've had issues getting things up and running...
The sitemap is a virtual sitemap and therefore you won't see a physical sitemap.xml file on disk, but you should be able to browse to /sitemap.xml and see it.
In order to bind /sitemap.xml to a site the root node needs to have a domain assigned. Can you confirm you have a hostname assigned to your site root under the "Culture & Hostnames" menu?
Cheers,
Callum

Can you add this to your documentation? Your docs just say to enable through the web config. It would have been super helpful if this was in the docs.

Hey @Bleeus,
Thanks for using Friendly Sitemap!
The usage docs were recently updated to cover what is required to setup the sitemap.
As of the latest release v1.2 a domain is not required in order to run. If you have Friendly Sitemap installed already, update the Our.Umbraco.Extensions.Routing dependency to the latest version and you should be golden.
Cheers,
Callum

Just updating you with the steps we took in case they help you or anyone else.
We did a fresh Umbraco install (Umbraco version 8.14.1) using the Zip (Not NuGet) and installed FriendlySitemap from the Umbraco Packages section. The package manager installed version 1.2.1 of FriendlySitemap.
We added the  <add key="Umbraco.Sitemap.Enable" value="true" /> to the web.config.
The sitemap would not show up until we added the domain to the Culture & Hostnames section.


Open-CE Builder 9.0

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Updated version in conda package
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

Codecov Report

Merging #112 (c9a03c7) into main (cb5f739) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   96.20%   96.20%
=======================================
  Files          23       23
  Lines        2105     2105
=======================================
  Hits         2025     2025
  Misses         80       80

Flag
Coverage Δ

unittests
96.20% <ø> (ø)

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

Continue to review full report at Codecov.

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

Codecov Report

Merging #112 (c9a03c7) into main (cb5f739) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   96.20%   96.20%
=======================================
  Files          23       23
  Lines        2105     2105
=======================================
  Hits         2025     2025
  Misses         80       80

Flag
Coverage Δ

unittests
96.20% <ø> (ø)

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

Continue to review full report at Codecov.

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

Codecov Report

Merging #112 (c9a03c7) into main (cb5f739) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   96.20%   96.20%
=======================================
  Files          23       23
  Lines        2105     2105
=======================================
  Hits         2025     2025
  Misses         80       80

Flag
Coverage Δ

unittests
96.20% <ø> (ø)

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

Continue to review full report at Codecov.

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


Bump compiler to go 1.15

May I help with this, i'm doing hacktoberfest :)

Wasn't this merged already from PR #350? But the changes have not shown up in the current version of the master branch.

Sure, go for it!


Dot in directory (flow.js) breaks grunt plugins

The issue is outlined in detail here: flowjs/ng-flow/issues/110

My quick solution(bower register flow-js https://github.com/flowjs/flow.js):
 ~ $ bower install flow-js
bower flow-js#*                 cached git://github.com/flowjs/flow.js.git#2.8.0
bower flow-js#*               validate 2.8.0 against git://github.com/flowjs/flow.js.git#*
bower flow-js#~2.8.0           install flow-js#2.8.0
flow-js#2.8.0 bower_components/flow-js

This should solve your problem.

Didn't know you could do that. So for ng-flow to reference this package flow-js, I'd need to follow the same idea and create another package, and set its dependency to flow-js rather than flow.js?

No, just leave  flow.js  dependency for ng-flow library and require flow-js for your own package. Flow.js will be downloaded two times in separate directories.


[BUG][DEPLOY] quay.io/stolostron/kube-rbac-proxy-mce: unauthorized: access to the requested resource is not authorized

When deploying the upstream (mid-stream?) stolostron operator on OKD 4.12, everything deploys as expected, except for the hypershift-addon-agent pod.  I get this pull failure for the kube-rbac-proxy-mce container.
Failed to pull image "quay.io/stolostron/kube-rbac-proxy-mce@sha256:d0cd6580603bcdf2ebef1f9a836f91c024ce8113dca480962b515bc0efbda523": rpc error: code = Unknown desc = quay.io/stolostron/kube-rbac-proxy-mce: unauthorized: access to the requested resource is not authorized
Note, the other container in the pod deploys successfully (hypershift-addon-agent)

this has been remedied with the new release channel 'community-0.2' version 0.2.0.  Closing...


added custom keyword search  feature

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


Failed test:  1) "before each" hook for "should emit deployed event"

1 test failed after running yarn test.
how to fix this? what is the issue?
kind regards,

  1 failing

  1) Contract: BondingCurveFactory
       Deploy BancorCurveLogic + ERC20 Collateral
         "before each" hook for "should emit deployed event":
     Error: Returned error: VM Exception while processing transaction: revert
      at Object.ErrorResponse (node_modules/@openzeppelin/upgrades/node_modules/web3-core-helpers/src/errors.js:29:16)
      at /Users/alex/Documents/OpenRaise/node_modules/@openzeppelin/upgrades/node_modules/web3-core-requestmanager/src/index.js:140:36
      at /Users/alex/Documents/OpenRaise/node_modules/truffle/build/webpack:/packages/provider/wrapper.js:107:1
      at XMLHttpRequest.request.onreadystatechange (node_modules/truffle/build/webpack:/node_modules/web3/node_modules/web3-providers-http/lib/index.js:98:1)
      at XMLHttpRequestEventTarget.dispatchEvent (node_modules/truffle/build/webpack:/node_modules/xhr2-cookies/dist/xml-http-request-event-target.js:34:1)
      at XMLHttpRequest.exports.modules.996763.XMLHttpRequest._setReadyState (node_modules/truffle/build/webpack:/node_modules/xhr2-cookies/dist/xml-http-request.js:208:1)
      at XMLHttpRequest.exports.modules.996763.XMLHttpRequest._onHttpResponseEnd (node_modules/truffle/build/webpack:/node_modules/xhr2-cookies/dist/xml-http-request.js:318:1)
      at IncomingMessage.<anonymous> (node_modules/truffle/build/webpack:/node_modules/xhr2-cookies/dist/xml-http-request.js:289:47)
      at endReadableNT (_stream_readable.js:1145:12)
      at process._tickCallback (internal/process/next_tick.js:63:19)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
alex@HL072 OpenRaise %

Hey @alexrmacleod, this repo has just been archived (should have been sooner). The contracts were never fully finished or audited. Please feel free to fork and continue as you see fit!


ADD Usama to Rookies list

Checklist:

[x] My name is alphabetically ordered by first name in the README
[x] My profile picture is added to the /images folder of either the Heroes/ or the Rookies/ directory
[x] All the fields are filled out for my profile (see the other profiles for examples)

Welcome, @Ali-Usama! 🎉


Write the latexify object directly to file

Hi,
I have a question. When printing the function with the latexify decorator, is there any way that I can use the format string to print the "function" string directly. As I have 10 functions name from "p1" to "p10". Is it possible to write a for loop to print all the functions?
Also, is it possible to directly convert the function to string and write it to a file? Thanks!
Thanks.

Do you want to print the rendered latex or the raw (unrendered latex)?
Rendered LaTeX
Assuming you already decorated the functions:
from IPython.display import display

list_of_funcs = [p1, p2, p3, p4, p5, p6, p7, p8, p9, p10]

for f in list_of_funcs:
    display(f)

If not:
from IPython.display import display
from latexify import get_latex

list_of_funcs = [p1, p2, p3, p4, p5, p6, p7, p8, p9, p10]

for f in list_of_funcs:
    display(get_latex(f))

Unrendered latex
Assuming functions were not decorated:
list_of_funcs = [p1, p2, p3, p4, p5, p6, p7, p8, p9, p10]

for f in list_of_funcs:
    print(get_latex(f))

get_latex is the function that directly returns the LaTeX form without modifying the original function. You can also use str(your_fn) if you already wrapped your function by with_latex.

Thanks for the clarification!


Exception in shim delegate doesn't respect try/catch in isolated method

Here is some code where MethodA calls MethodB - if there is no exception, MethodA returns "0" otherwise it returns "1".  However, if I make a shim for MethodB that calls an exception, Pose execution stops and the exception isn't caught in MethodA.
Is there a way to throw exceptions in shims that are caught in the isolated method?
        [Fact]
        public void Test()
        {
            int i = MethodA();
            Assert.Equal(0, i);

            PoseContext.Isolate(() =>
            {
                i = MethodA();
            }, Shim.Replace(() => MethodB()).With(() => { throw new Exception("test"); }));
            Assert.Equal(1, i);
        }

        public static int MethodA()
        {
            try
            {
                MethodB();
            }
            catch
            {
                return 1;
            }
            return 0;
        }

        public static void MethodB()
        {
        }

Hi @dtkujawski, Pose currently doesn't seem to handle try...catch blocks quite well. Will look into it

Hi @dtkujawski #31 currently respects exception handling in methods. Will push out a new release tomorrow

Thank you for the fix!  This works great.


:pointerover logic does not respond to content changing or moving without mouse movement

Currently IsPointerOver and :pointerover are only updated on mouse movement. Changing content under a stationary mouse has no effect. wpf does respond to content changing.

In trying to fix this I discovered another related issue. In control catalog if you click a tab heading then select different tabs with the arrow keys whilst moving the mouse rapidly over the app you can trigger a hit test exception. It always seems to be a Rectangle that is not part of the visual tree.  The same thing happens frequently in my attempts to update pointerover on layout change.

An extra check for IsAttachedToVisualTree in Scene.HitTest prevents the hit test exception but I don't know if this is just hiding an error.


cleaned up to some of the pep8 standard, but not all.  Of note, there…

… is a for loop that does not use the i variable (line 130 in this code), wonder if there is a way to do it otherwise?

You are right about line 130, I think juggling C++ and Python might sometimes make me confused. I have been looking into different solutions. I will change this

So- great feedback, thank you!! I just used the line in my IDE where I thought it was about 79/80 char's.  I will have to recheck it and fix that.  Obviously it wasn't that, so my bad.  As to the rest of your questions, I am unsure- PEP8 can be confusing at times and I haven't read it in about a year so I will go through it again.

Awesome. I'll try and take a look at PEP8 too (seems like something that is good to work with) and see if I can find something about keeping the lines together.

PEP8 is really short, there isn't much there, but it is good.  Keeping the lines together is about using either a \ or making sure the lines line up with the leading (, not much else.

It is a little unclear to me from the PEP8 whether the \ char is for breaks in comment lines, or only in lines of actual code. It is not important to me how we do it, as long as we have a consistent approach

Hey @elheffe80 I haven't merged this yet, cause I wanted us the solve the continuing line issue, and the max number of chars pr. comment line. Do you wanna take another stab at it at some point, or have you abandoned this issue?
I will fix the summary lines in the docstrings this weekend, so that they are all short descriptions, that can fit one line.

@lineaba Sure thing!  So, according to the pep8, its 80 chars/line regardless of who/what/where/when/why.  There is no convention for comments and indentation of them for continuing lines when you have a list in the comments section, so I just made it up and did two spaces per your request.  As for code shortening, you are supposed to line up the code with the preceding paranthesis, or if there is none then you use a \ and line it up according to some mythological beasts determination.  I have a busy weekend (BSidesCharm!! WOOT!), but should be able to take a crack at it again Sunday evening if not Saturday night.  What I will do is just do a completely new pull and start fresh from the code scraping the work done here because there has been a lot of great work done by everyone.

That's awesome, have fun at BsidesCharm. No stress on finishing it up, just wanted to hear where we were standing.
I am wondering if we are looking at two different versions of the PEP8 guide? This is the one I have been reading: https://www.python.org/dev/peps/pep-0008/
It says:
"Limit all lines to a maximum of 79 characters.
For flowing long blocks of text with fewer structural restrictions (docstrings or comments), the line length should be limited to 72 characters."
I don't have a strong preference for it to be one way or another, but I would like to add a link to the PEP8 guide for future contributors, so if I have found the wrong version, could you send me the link you are using for reference?

AFAIK, the 79 character limit per line is kind of useless nowadays given most of us have big 16:9 or 16:10 monitors. They mention it fruther down the link you shared.

Some teams strongly prefer a longer line length. For code maintained exclusively or primarily by a team that can reach agreement on this issue, it is okay to increase the nominal line length from 80 to 100 characters (effectively increasing the maximum length to 99 characters), provided that comments and docstrings are still wrapped at 72 characters.

even with 100 character per line I still have plenty of space before a string wrap. I'm open to have a bigger limit than 79, what do you guys think?

I have no personal preference, other than consistency, so we just need to settle on a max and stick to it. We can set out own max, or depend on some max determined in a styleguide, either is fine by me. If someone else have strong opinions about it, they should definitely chime in :)
I would like to point out that using "you guys" can be perceived as exclusive language. I hope for this community to feel welcoming to everyone, so I would like to encourage contributors to avoid gendered language when addressing other contributors. Maybe consider "y'all", "the rest of you", "others", etc. in the future. Our code of conduct states that we want to be using welcoming and inclusive language.

You guys being perceived as exclusive language must be a regional thing, but okay... as for a max, I’m for whatever. Being a Linux terminal geek, I have a love for 80, but I know a lot of you luckier users have wide screens, so we can bump it up to 100 so we can keep the comments inline as you originally wanted @lineaba. Let me know what standard and I should be able to knock it out in an hour or two tonight.
Sent with GitHawk

@elheffe80 Hey, sorry, I've lost track of this PR. Is it ready to be merged?

@elheffe80 Let me know when you've had time to look at this :)

I swear I am working on it. My life is upside down and inside out at the moment. This week I am in class 8-5 and then my nights are spent taking care of kids and wife. I should be able rob scroll through it today though if I screw off in class. 😂
v/r
Ben Heffron
Forgive brevity, this was sent from a mobile device.

On May 15, 2018, at 1:39 AM, Linea<EMAIL_ADDRESS>wrote:
@elheffe80 Let me know when you've had time to look at this :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@elheffe80 Hey Ben, no worries, I was mainly just checking in to make sure that this wasn't stale, cause then I would just close it out and take another contribution to the issue.
Good luck with classes!


Temporarily import loop_in_thread fixture from distributed

[x] Closes https://github.com/dask/distributed/issues/6842
[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

This is a quickfix similar to https://github.com/dask/dask/pull/9337. Note there's a conversation happening over in https://github.com/dask/distributed/issues/6806 around what testing utilities in distributed should be considered public.

It's not immediately clear to me why the docs build is failing. Has that been happening elsewhere?

It's not immediately clear to me why the docs build is failing. Has that been happening elsewhere?

we turn sphinx warnings into errors when building docs on RTD. It appears the SSL certificate for numba.pydata.org has expired and sphinx is throwing a warning which is then turned into an error after docs build.
loading intersphinx inventory from https://rasterio.readthedocs.io/en/latest/objects.inv...
loading intersphinx inventory from https://sparse.pydata.org/en/latest/objects.inv...
WARNING: failed to reach any of the inventories with the following issues:
intersphinx inventory 'https://numba.pydata.org/numba-doc/latest/objects.inv' not fetchable due to <class 'requests.exceptions.SSLError'>: HTTPSConnectionPool(host='numba.pydata.org', port=443): Max retries exceeded with url: /numba-doc/latest/objects.inv (Caused by SSLError(CertificateError("hostname 'numba.pydata.org' doesn't match either of '*.github.com', 'www.github.com', 'github.io', 'github.com', '*.github.io', 'githubusercontent.com', '*.githubusercontent.com'")))

I opened an issue just in case this is something that can be fixed upstream.

https://github.com/numba/numba.github.com/issues/10

Thank you for this fix, @jrbourbeau!

Thanks @jrbourbeau!


show user created roles in Add Cluster/Add Project Member Role drop-down

Rancher versions: 2.0 master 2/27
Steps to Reproduce:

Create a role that is specific to projects

Go to Add Project and Add User

Click on Role drop-down

Do above steps but with cluster specific role

Results: Have the user created role show up in the member Role drop-down.
builtin, external & hidden = false

Version - 2.0 master 3/26
Verified fixed


Update shelley-genesis.md

--shelley-mode parameter added twice to the transaction submit command

Thanks for this PR.  The changes have been integrated into https://github.com/input-output-hk/cardano-node/pull/2598


bpf: xdp: replace inline assembly in csum helpers

Now that we have native support for bpf_xdp_load_bytes(), wouldn't it be nice if we also replaced the inline assembly in https://github.com/cilium/cilium/blob/82dbff8e5a5f7ced99e96cd85997fae90e035aac/bpf/include/bpf/ctx/xdp.h#L167 and https://github.com/cilium/cilium/blob/82dbff8e5a5f7ced99e96cd85997fae90e035aac/bpf/include/bpf/ctx/xdp.h#L204.

please assign me

@julianwiedmann
Can you confirm if bpf_xdp_load_bytes can be replaced?
Those functions are directly changing the packet, so it doesn't seem possible to change the packet with bpf_xdp_load_bytes.
What do you think?
ps.
I am a noob in eBPF. Please let me know if I'm doing something wrong


[Feature Request] Emmet support

When i type hello and press Tab key, it will auto complete with <hello></hello>
emmet official website: https://emmet.io/

It has been already requested previously. I might give it more priority if there's demand, emmet is really cool. I would love to get some collaborators for this kind of functionality.

There seems to be possible to add emmet support by simply integrating the emmet language server. I'll try it soon, if this works I'll set the emmet LSP as the default LSP for HTML files.

This has been added via LSP in the latest version.


[SPARK-22883] ML test for StructuredStreaming: spark.ml.feature, I-M

What changes were proposed in this pull request?
Adds structured streaming tests using testTransformer for these suites:

IDF
Imputer
Interaction
MaxAbsScaler
MinHashLSH
MinMaxScaler
NGram

How was this patch tested?

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1913/
Test FAILed.

LGTM. 👍

Thanks!  I'll rerun tests since they are stale and merge after they pass.

Test build #4151 has started for PR 20964 at commit 6dff94e.

Test build #4151 has finished for PR 20964 at commit 6dff94e.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

I rebased off of master because of the merge warning in the last tests.
I did not have to resolve any conflicts.
I'll merge this once tests pass.

Test build #89174 has started for PR 20964 at commit 592670c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2194/
Test PASSed.

Test build #89174 has finished for PR 20964 at commit 592670c.

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

Merged build finished. Test PASSed.

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

Merging with master


Is it possible to train the detectors and CLOCs end-to-end together?

Yes, it is possible, but I guess the performance may not be good. I have not tested everything together end-to-end, but I have tested to training CLOCs and SECOND RPN together (frozen other layers), but the performance was not as expected, the final results are worse than original SECOND.


Extension Throws Exception at Activation in New Component Files

Vue - Official extension or vue-tsc version
2.0.21
VSCode version
1.90.0
Vue version
3.4.21
TypeScript version
5.4.0
System Info
No response
Steps to reproduce

Create a new project, such as those generated by npm create vue@latest.
Add a devcontainer.json file in a .devcontainer in the root of the project, configured to provide a Node.js and Typescript environment and install the Vue - Official extension.
Reopen the project in the DevContainer.
Install dependencies specified in package.json.
The installed Vue - Official extension appears to work, with IntelliSense running and utilities like "go to definition" working as expected.
Attempt to create a new single file component (e.g., MyTest.vue).
The Vue - Official extension appears to no longer work completely as expected in the new component file (e.g., if importing an existing component, there is no auto-import and "go to definition" does not work).
Observe that the extension now reports uncaught errors during activation.

Link to minimal reproduction
https://github.com/CristopherH95/Vue-Extension-Issue
Any additional comments?
This issue was originally encountered while using Podman with a DevContainer environment for a Vuetify project. This also seemed to occur for me the same in a "vanilla" Vue project such as one in the provided GitHub repo. Upon a quick test, it looks like I also run into this outside of the DevContainer as well.
Examples of errors seen:

The issue of creating a new file seems to be related to #4424.
The source code of the thrown error (TypeError: Cannot read properties of undefined (reading 'match')) is https://github.com/volarjs/volar.js/blob/a59765125bb02f1ea62bedd8e3db2a88433538ba/packages/vscode/lib/features/autoInsertion.ts#L63, which is related to auto insertion, I think it's a different issue.

The issue of creating a new file seems to be related to #4424. The source code of the thrown error (TypeError: Cannot read properties of undefined (reading 'match')) is https://github.com/volarjs/volar.js/blob/a59765125bb02f1ea62bedd8e3db2a88433538ba/packages/vscode/lib/features/autoInsertion.ts#L63, which is related to auto insertion, I think it's a different issue.

so how to avoid the error about  Cannot read properties of undefined (reading 'match')

我也遇到了相同的问题，当出现该问题时，插件 ts-plugin 的纠错功能将失效，并且会卡住一些任务的线程。
I encountered the same issue. When this problem occurs, the error correction function of the plugin ts-plugin will fail, and it will hang some threads of tasks.
例如格式化功能会需要长达数秒的时间才能完成，git无法检测文件变更等...
For example, the formatting function takes several seconds to complete, and Git cannot detect file changes, etc...


All mounted devices capability is pushed into both ODL streams (config +oper) when user perform edit on ODL attributes

Steps followed:

ODL Controller is up and running and ODL streaming is not subscribed
Notification proxy is deployed and running
Invoked API: /v1/listen-to-controllers
Performed PUT request on any ODL attribute for only one node(exampl: NODEA) and observed events for all mounted devices from the controller are pushed into both ODL streams (config + oper)

Expected:
Only NODEA related events which is associated for PUT operation should get streamed

The url which used for ODL subscription is incomplete.
Actual url:
/rests/data/ietf-restconf-monitoring:restconf-state/streams/stream/data-change-event-subscription/network-topology:network-topology/datastore=CONFIGURATION/scope=SUBTREE/JSON
Expected url:
/rests/data/ietf-restconf-monitoring:restconf-state/streams/stream/data-change-event-subscription/network-topology:network-topology/datastore=CONFIGURATION/scope=SUBTREE/JSON?changed-leaf-nodes-only=true
Because of this output notification is not correct


Next release date?

I'd like to use the frozen string PR stuff. Is there a plan for when the next release is going out? Are we waiting to get socket-level pings working for all concurrency libraries?

I released 0.11.2, #202 is the only code change.


Allow markText with "bookmark" type.

setBookmark will not copy all keys from options,
markText will copyObj(options) , all keys in options will be stored in the marker.
but the type is fixed to "range".
Please change the markText API (eedb58faa301551d530b34e5c765a90607232a31 codemirror.js line 7560) from

markText: function(from, to, options) {

 return markText(this, clipPos(this, from), clipPos(this, to), options, "range");

to:

markText: function(from, to, options, type) {

 return markText(this, clipPos(this, from), clipPos(this, to), options, type||"range");

In order to create bookmark with custom options properties.

Attached patch does this, but by reading the type option from the options object rather than adding an argument.


Price with VAT gots calculated wrongly at order overview.

Expected behavior:

Prices displayed to the customer should fit with prices displayed at admin site.
result of price calculation in order details should fit prices displayed at admin site.

Actual behavior:
The system displays the prices wrong at the order overview table.
see: https://ibb.co/FB5ZYPK
Reproduce steps:
Additional advise needed.
I set up the system to have a taxrate of 9% included in the prices.
Reproducing unclear, this problem occured after updating tastyigniter to the specified version.
"Include tax into prices" is set to true.
see: https://ibb.co/FB5ZYPK
Version:
3.0.4beta 28

Fixed, but please reopen if it still occurs.


rst broken on pypi for 1.5.1

https://pypi.python.org/pypi/safety (https://pypi.python.org/pypi/safety/1.5.1)

Fixed with 1.8.5 - Thanks!


join multiple vulnerability scanner

Hello,
I notice we added multiple modules to check the HTTP headers for misconfiguration/vulnerabilities  (e.g. xss protection, content type, etc.. ). It's best to join them into one module and keep the vulnerability information in a JSON and we can name it header_based_vuln?.
I glad if someone could work on this.
Regards.

how about making a directory structure for different vulnerabilities rather than having to combine all those vulnerabilities in one module as in if another developer wants to create a vulnerability for eg. header_based_vuln he will have to understand the entire data present in that module and editing those previously stable modules can cause instability and confusion.
Best Regards

Hey @pradeepjairamani,
I am not sure I quite understand you, could you please explain more?
Regards.

So, you are asking to combine multiple modules into one single module which means a single engine.py for all the http header based vulnerabilities?
Doing that might create confusion for new developers and also can make the vulnerabilities unstable with each vulnerability being added. Better we should make folder structure like we discussed in cms scanner.
http_header_vuln > options_method >engine.py
http_header_vuln > XSS_protection  >engine.py
http_header_vuln > clickjacking >engine.py , etc
Best Regards

I get it! Until http_header_vuln > engine.py be exist,  it's not a problem!

Meaning?

I meant we can combine modules just like you suggested. every module in a category could be in a subdirectory as a class.


Add support for RDBMS for PEPPOL Directory

Currently the website info says that PEPPOL directory only works with XML backend.  Add support for RDBMS backend for PEPPOL directory.  Preferred RDBMS backend is DB2.

Moved to the correct project


Upgrade of documentation

Hello team,
Hope you're well.
Customers are complaining about the documentation being outdated. For instance, in step 4 "Select your subscription", the walkthrough mentions to go to Directories+Subscriptions tab; however, it is now deprecated. It'd be much appreciated if the documentation could be keep up to date for customer reference.
Thanks a lot!

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

ID: 24e34c89-9a70-b5b2-3b38-bc81660f83c1
Version Independent ID: 77fa8d3c-56dd-294f-13ae-ac6ba44018a7
Content: Monitor Azure AD B2C with Azure Monitor - Azure AD B2C
Content Source: articles/active-directory-b2c/azure-monitor.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@marianabqu
Thanks for your feedback! We will investigate and update as appropriate.

@marianabqu
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly
@kengaderdus
Could you Please review this and update as appropriate.

@marianabqu
Thanks for bringing this to our attention. The doc have been updated #please-close.


Mime types include list

Fixes #521
lib/tessel/wifi.js was formatted after running grunt.

Perfect!