Allow block comments characters to have new line in them

Issue Type: Bug
Create an extension with a language configuration containing the following types of comments (it's coming from the VSCode Matlab extension :
"comments": {
    "lineComment": "%",
    "blockComment": [ "\n%{\n", "\n%}\n" ]
},

Run in debug mode and try to create a block comment.
The block comment should correctly be added, but when you try to remove it with the same key combination, it will add new block comments instead of removing the existing ones.
VS Code version: Code 1.22.1 (950b8b0d37a9b7061b6f0d291837ccc4015f5ecd, 2018-04-06T02:26:57.615Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4650U CPU @ 1.70GHz (4 x 2295)

Memory (System)
7.91GB (1.15GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe .

Screen Reader
no

VM
0%

Extensions (19)

Extension
Author (truncated)
Version

ng-template
Ang
0.1.9

vscode-markdownlint
Dav
0.14.1

vscode-eslint
dba
1.4.8

githistory
don
0.4.0

tslint
eg2
1.0.28

vscode-npm-script
eg2
0.3.3

fortran
Gim
0.2.0

matlab
Gim
0.8.1

Angular2
joh
2.16.1

solidity
Jua
0.0.38

ftp-sync
luk
0.3.3

HTMLHint
mka
0.4.0

csharp
ms-
1.14.0

PowerShell
ms-
1.6.0

team
ms-
1.133.0

debugger-for-chrome
msj
4.3.0

debugger-for-edge
msj
0.4.1

vscode-docker
Pet
0.0.26

yo
sam
0.9.3

How are you adding the block comments? What keybinding are you using to add and remove the comment?

On a Mac, it's   shift-alt-A.
Here's where I originally reported it in the extension.  MATLAB-style block comments must exist on their own lines, but may be surrounded by whitespace
ex.
  %{
      a block comment with leading whitespace
  %}

  myfunction(); % end of line comment

%{  mistake. because of text on this line, it is treated like an end-of-line comment
   uncommmented_function_although_intent_was_to_comment_it();
%}

I have the same issue in the language extension for the J language.
I can add the new line chars to the start and end character sequences, and they are correctly inserted.
However the same sequences don't appear to be identified/matched when attempting to toggle off the block comment and instead an additional set of start and end sequences are inserted.


let sbt handle cross building

@aliyakamercan @amotamed @jlbelmonte @DanSimon @dxuhuang

Codecov Report

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

@@           Coverage Diff            @@
##           develop     #631   +/-   ##
========================================
  Coverage    84.76%   84.76%
========================================
  Files           98       98
  Lines         4432     4432
  Branches       366      417   +51
========================================
  Hits          3757     3757
  Misses         675      675

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 e1bb232...72941b7. Read the comment docs.

Since this is @aliyakamercan PR and I approve, I shall also merge.


Cannot render marker if same location

Hi, i have an issue, if my location is same, it's not render marker, still in cluster number (ex : 2).
I have set key, but still not work

I have some issue

I thought this was the way it should work.
I am thinking of examining the children in onClusterPress to see if they are if the same location and then switching to a list view if they are.
But if anyone has a better solution, I would be glad to hear it.

entered up saying that if at max zoom and onClusterPress is called, then switch to list view

entered up saying that if at max zoom and onClusterPress is called, then switch to list view

Can you tell me how you got the current zoom level?

if (this.refs.clusteredMapView.getClusteringEngine().getClusterExpansionZoom(clusterId) == 20){

From: Nuno B<EMAIL_ADDRESS>Sent: Thursday 11 July 2019 13:18
To: novalabio/react-native-maps-super-cluster<EMAIL_ADDRESS>Cc: Eamon Kelly<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [novalabio/react-native-maps-super-cluster] Cannot render marker if same location (#80)
entered up saying that if at max zoom and onClusterPress is called, then switch to list view
Can you tell me how you got the current zoom level?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/novalabio/react-native-maps-super-cluster/issues/80?email_source=notifications&email_token=AAGGNJYX47GBDNATU7T3JNTP64QFXA5CNFSM4HBLE3X2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZWQMDA#issuecomment-510461452, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAGGNJZRDTCTZPZ7XNQOBY3P64QFXANCNFSM4HBLE3XQ.

@Filikin is right, this is actually how it should work. What @Filikin has already suggested seems reasonable to me. Or you can simply show a detail/preview upon cluster press.

this is how I get and manage clustering by zoom level:
  onRegionChange = region => {
    let zoom = Math.round(Math.log(360 / region.longitudeDelta) / Math.LN2)

    const { clusteringEnabled } = this.state
    const maxZoomForClustering = 12

    if (zoom >= maxZoomForClustering && clusteringEnabled) {
      return this.setState({ clusteringEnabled: false })
    }

    if (zoom < maxZoomForClustering && !clusteringEnabled) {
      return this.setState({ clusteringEnabled: true })
    }
  }

just in case anyone need it.
render() {
<ClusteredMapView
          data={this.deriveArrayOfEntriesWithLocation()}
          showsCompass
          loadingEnabled
          provider={'google'}
          style={styles.flex1}
          showsTraffic={false}
          initialRegion={INIT_REGION}
          ref={ref => this.map = ref}
          renderCluster={this.renderCluster}
          onRegionChange={this.onRegionChange}
          renderMarker={this.renderClusterMarker}
          clusteringEnabled={this.state.clusteringEnabled}
          customMapStyle={GoogleMapsColorScheme[googleColorScheme]}
        >
          {!this.state.clusteringEnabled && this.renderMarker()}
        </ClusteredMapView>
}


[WR-1605] add ssotFormat function to handle NULL/0, use ssotFormat function for…

… Industry Employment by Region table
I found an table that need null checking.  I created an ssotFormat function to do the checking and formatting.  I created a new function that calls ssotFormatNumber function rather than adding the null checking to ssotFormatNumber to avoid the possibility of introduction new bugs into the system.

CareerJobOpeningsByRegion.php updated to use ssotFormatNA()
CareerJobOpeningsByRegionNA.php updated to remove floatval() so function will accurately return YES|NO if data contains null or not.


POC: Cilium for everything

As part of the SIG Architecture, we'd like to consider the possibility of collapsing networking features into Cilium.
The general idea is that this would simplify the configuration and operation of these capabilities. We need to find out if that's true and evaluate the maturity of each sub-product.
The features we want to consider are:

Service Mesh
Ingress / Gateway API
API Gateway (?)
Multicluster connectivity

me and @mcharriere chatted a bit yesterday
while using cilium to replace multiple components would possible simplify operations for ourselves, we do need to generate some user facing benefits - as some random ideas:

does request time for a demo application decrease?
does overall CPU / memory requirements for the networking stack decrease?
another third thing i can't think of right now

because then we can make a compelling argument for why we're doing the work (to customers), and why customers should upgrade - e.g: you upgrade to the next version, your 99th percentile request time goes down by 50ms

We'll start with investigating Cilium Ingress in ticket #29168

We had a chat with @mcharriere recently and we both agreed we wouldn't like to bet everything on Cilium anymore.
For Gateway API and and alternative (potentially more performant and reacher in features) ingress we could try Envoy and we don't see benefits of using the Cilium built-in Evnoy.
For Multi-cluster connectivity I don't know. Maybe we should first have a use-case.


testnet radon patch upgrade test

Test for non-determinism when applying tariff security audit fixes + huckleberry fix to testnet.

bringing this test into main in #183


S3 sdk upgrade to 2.x to support netty based NIO

Hi, We are doing some high data volume upload test, it seems reached bottleneck at ~100MB/s on our setup using parquet/profobuf writer. Currently we plan to upgrade s3 sdk to leverage nio for higher work load. Wondering if you have any plan on upgrade aws sdk to 2.x to benefit from nio?

No particular plan as far as I know.  But you are welcome to try that upgrade, if there is no API change, it should be a straightforward work.

Thanks. Will update you our findings here.


Separate utilities related to pages

This pull request moves PageNo, NumPages, and PageSpan along with the operations associated with them out of Database.LSMTree.Internal.IndexCompact and into a new, dedicated module. The reason for this change is that those things are not specific to the compact index, not even to indexes at all. This pull request is not least part of the preparations for making index types selectable.

The reason for this change is that those things are not specific to the compact index, not even to indexes at all.

I’d argue that the second argument is disputable and that it depends on your point of view, but that’s beside the point.

Well, the things that are now in this new, dedicated module are only used in conjunction with indexes, from what I know; so in a way you’re completely right. However, semantically those things are not specific to index at all, as they are only about referencing and counting pages and pages not only show up in conjunction with indexes. Actually, the close relationship between those page-related things and indexes seems to be partly an implementation detail to me, considering, for example, the arrangement of storing the total number of pages along with the index.


first cut at fixing erroneous expansion toggling

In the Definitions page if you have an entry and click on the button to drop down the Select/Add a version menu, the row for that entry expands or collapses. Similarly, if you hover over the drop down and then select a menu entry.
This fix adds stopPropagation() to the handler for the menu click and the dropdown expand handler. The former seems to do the right thing. The latter feels wrong and does not actually work correctly. with this change clicking on the button does not expand/collapse the menu like it should. Without the change, the menu behaves correctly but the row also expands collapses.
The DropDown component really should be consuming this itself. Perhaps there is a setting on the Antd dropdown?
BTW, why are we using Antd for things like this. The default should be Bootstrap. For the most part the site is Bootstrap. Where there is something it does not have, we can use Antd. It feels odd to mixing these frameworks on the simple things like buttons and dropdowns.

I have used the Antd dropdown because the Bootstrap one had some rendering issues (it showed inside the definition panel and the user couldn't be able to interact with the content if the definition was not expanded)


Objects inside arrays are not cleaned

If an object happens to exist inside of an array, it is not cleaned. This seems counterintuitive for a library that has "deep" in the name.
For example, cleaning this:
foo: [
  {
    bar: null,
    baz: {},
    thing: 123
  },
  {}
],
bar: null

Currently returns this:
foo: [
  {
    bar: null,
    baz: {},
    thing: 123
  },
  {}
]

But it seems like it should return this:
foo: [
  {
    thing: 123
  }
]

@ruimarinho @ruipenso Any philosophical objections to adding this kind of deep array support to this lib? If not, I can submit a PR - but just wanted to make sure you're not against the core concept since it would require a major version bump if it defaulted to array traversal.

Interesting dilemma 🤔. I personally don't have anything against it - in fact, I would argue that for consistency, it looks more of an oversight than an enhancement. However, let's discuss how to deal with the following cases:

If we consider this a bug (even if users haven't experienced it yet), than their expectation could be that the library works exactly like the example you gave (array traversed). Hence, a patch release would be more adequate. However, it is also a breaking change, which could put the other percentage of users who have seen the library behave differently at risk. (Note: I defend a major)
If we traverse arrays, that means that [{}] could justify the concept of multiple passes, i.e., should a collection of empty values also be removed ([{ foo: {} }] => undefined)? The same principle applies to a collection of an empty collection ([[]]).
Do we even want to make traversal optional on a library named *deep?

Hence, a patch release would be more adequate.

I'd say this is too risky given my own internal use case - even though it might feel like a bug, the potential for breaking existing code that relied on existing behavior is too big IMO.

...could justify the concept of multiple passes...

Hmm, you're right 🤔 Let me try to think through this...

Do we even want to make traversal optional on a library named *deep?

I think if objects are recursed by default with no escape hatch, deep array traversal should be similar.

I think we only need to figure out if multiple passes is wanted or not. I'd say no, at least not now. Everything else looks good - major and traverse arrays by default.

I have a use case where I need to clean objects within arrays. I am currently writing JSON data from random Social sources to DynamoDB. DynamoDB does not like when any value is Null or an empty string and will not write the JSON data at all if it encounters such an error.
My initial usage of this library was to simply handle this requirement of DynamoDB without having to worry about the data coming in (I don't necessarily know all the attributes that could be passed to me) from random social sources.
was working well until I ran into an instagram post with and Array of objects that contained { attribute: "" }'s inside... 😢
would be cool to just have an option to "CleanArrayObjects" ... with a default of false?

I have a pull request which add's this feature to the library. The change is only applied if enabled via the options flag:
https://github.com/seegno/clean-deep/pull/21


mdList - Add swipe animation for items

Bug, feature request, or proposal:
feature request
What is the expected behavior?
Allow list-items to be swiped left/right to reveal additional actions on the item. Draging the item further should activate the outermost action.
What is the use-case or motivation for changing an existing behavior?
Many big applications already use this (gmail, iOS Mail, ...) and it is a rather common/already trained user behaviour.
Implemented in:

Ionic (https://ionicframework.com/docs/api/components/item/ItemSliding/)
Framework 7 (http://framework7.io/docs/swipeout.html)
Poymer? (http://motss.github.io/paper-swipe/components/paper-swipe/) (https://poly-mail.appspot.com/)

Planned in Angular Material 1 (https://github.com/angular/material/issues/1890)

This is also described in https://material.io/guidelines/components/lists-controls.html#lists-controls-types-of-list-controls as leave-behinds.

+1

+1

Using hammerJS and angular 4.2 animations this should be fairly easy. But should it really be part of core?
Supporting touch is optional and on desktop it does nothing but bloat the framework :/

That's true. Where i'm currently consulting, we did a custom implementation using Hammerjs. That was fairly easy.
It would be awesome to have something like this in a touch module, in order to reduce bloat 😊

@mackelito , @LarsVonQualen would have an example in codepen, how to do using the hammerjs

https://material.io/design/components/lists.html#behavior

As of Angular v9, we've decided to step back from the aspiration of adding gestures directly to the components, instead having the component library be agnostic to geature recognition. That said, we do still want to add documentation that explains how to go about adding gestures to the components.


Write "getting started" guide

See original issue on GitLab
In GitLab by [Gitlab user @jonathanmaw] on Oct 4, 2017, 12:17
This should go in https://buildstream.gitlab.io/buildstream/, probably as part of the installation guide
This should cover:

what it's for in a couple of sentencess.
how to install it (via a link if it's complex)
how to do 'hello-world' within 10 minutes, with words people with no bst experience can understand
where to dig deeper

In GitLab by [Gitlab user @samthursfield] on Feb 28, 2018, 14:36
mentioned in merge request buildstream-examples!1

In GitLab by [Gitlab user @samthursfield] on Feb 28, 2018, 14:45
More generally, we need a "Quick start" guide that concisely explains the basic principles of BuildStream, how to run a build, and how to author a simple project.
I had a go at drafting such a thing here: https://wiki.gnome.org/SamThursfield/BuildStreamQuickStart
I structured it in 4 sections:

"What is BuildStream?" -- this section is mostly done
"Building a project" -- showing how to use bst to build and run an existing project. I plan to use the Freedesktop SDK as an example here for want of something better, but a dedicated example from the buildstream-examples repo would probably work better if we had such a thing. This section is maybe half complete.
"Authoring a project" -- showing how to write a project from scratch. I propose using a simple Flatpak app example. This section isn't started.
"Further reading" -- link to reference manual, etc.

My example is not finished; I think the first section is pretty complete and could be used as-is in a quick start guide.

In GitLab by [Gitlab user @devcurmudgeon] on Mar 8, 2018, 17:00
How do we get Sam's doc into the official place? I'd like to submit patches to it...

In GitLab by [Gitlab user @tristanvb] on Mar 9, 2018, 07:40
[Gitlab user @devcurmudgeon]: The docs are in the doc/source/ subdirectory of the buildstream git repo.
They are in reStructuredText markdown, there is a quickstart here but following the material already in place should be fairly straightforward.
The current structure inside doc/source is:

Main page at index.rst

Docs about using the cli at main_using.rst
Docs about the format, including list of plugins at main_authoring.rst
Docs about core API, for writing plugins at main_core.rst

For a quickstart guide, I dont think it belongs in any of these, but probably a link to a new main_quickstart.rst which appears in index.rst before any of the more core reference would be appropriate, this can/should link into the more terse reference material where appropriate.
A note about main_using.rst, this is currently shallow; but there is a plan to add more here about invoking BuildStream, adding more discussion about each command, starting with this: unapplied patch (what this does is link to automatically generated docs for the CLI in sections, allowing extra custom text for each command).

In GitLab by [Gitlab user @tristanvb] on Mar 9, 2018, 08:13
Some additional notes.
To work on documentation, it is important to be able to build the docs and then browse them locally with a file:// url.
I dont know if this is at all possible when using the bst-here wrapper script.
To build documentation, it is mentioned in the HACKING.rst file, but you basically need:

BuildStream to be installed (this should be true inside the bst-here docker container)
Install some additional things:

pip install --user sphinx
pip install --user sphinx-click

Run make -C doc
Figure out where bst-here is running stuff, so that you can access the directory from the host and look at the result in your browser using a file:// url.

In GitLab by [Gitlab user @jjardon] on Mar 15, 2018, 21:41
changed title from {-Provide documentation to create a project from scratch-} to {+Write "getting started" guide+}

In GitLab by [Gitlab user @jjardon] on Mar 15, 2018, 21:41
changed the description

In GitLab by [Gitlab user @jjardon] on Mar 15, 2018, 22:14
mentioned in merge request !323

In GitLab by [Gitlab user @jjardon] on Mar 16, 2018, 11:31
actually, https://gitlab.com/BuildStream/buildstream/issues/103#note_62386189 and https://gitlab.com/BuildStrem/buildstream/issues/103#note_62392709 should be part of the docs somewhere to explain how to contrinute to the docs: opened #303

In GitLab by [Gitlab user @jjardon] on Apr 16, 2018, 17:58
mentioned in merge request !403

In GitLab by [Gitlab user @albfan] on May 11, 2018, 08:31
I'm working on examples section with real examples that will run under CI.
Will ask for MR when MR is ready!

In GitLab by [Gitlab user @albfan] on May 12, 2018, 12:38
mentioned in merge request !463

In GitLab by [Gitlab user @jennis] on May 22, 2018, 17:56
assigned to [Gitlab user @jennis]

In GitLab by [Gitlab user @LaurenceUrhegyi] on May 22, 2018, 18:14
unassigned [Gitlab user @jonathanmaw]

In GitLab by [Gitlab user @jennis] on Jun 13, 2018, 16:59
assigned to [Gitlab user @phildawson]

In GitLab by [Gitlab user @phildawson] on Jun 13, 2018, 17:08
mentioned in merge request !499

In GitLab by [Gitlab user @jennis] on Jun 13, 2018, 17:13
UPDATE: [Gitlab user @phildawson] has taken the current WIP for this issue and has created an MR which introduces another example - importing a minimal alpine image and building amhello with an Autotools build element.
This is both a simple, and minimal, example that we have agreed could serve as our Getting started example.
Note: This MR conforms to the new style of BuildStream examples.
Another note: We also intend to migrate the x86_64 example (here) into the main repo. And we are yet to decide what will be the fate of the netsurf example ([here] (https://gitlab.com/BuildStream/buildstream-examples/tree/master/netsurf-flatpak)). Currently, both of these are not in a "buildable" state.
MR here: https://gitlab.com/BuildStream/buildstream/merge_requests/499
Thank you [Gitlab user @phildawson]

In GitLab by [Gitlab user @jennis] on Jun 13, 2018, 18:14
I also think that the Getting started section should include a description/breakdown of a build element as these are what make up the core of a BuildStream project. However, I don't think it's necessary to include such work in !499, we can do this in a separate MR.
I have some material for this here.
But I would like to know what people's thoughts are before throwing up an MR. [Gitlab user @tristanvb]?
My own current concerns are:

In this example we are describing the gedit.bst element which is part of the gnome-build-meta project. Perhaps we should describe one of our own elements from our "in-house" examples...?

It's another autotools element... When !499 lands, we will have two examples both using autotools elements, perhaps we should have a bit more variety?

I also have some work which describes the motivation and concepts of buildstream (here).
If anyone could spare some time to read this, I'd appreciate your thoughts on whether this is something we should include in the Getting started section. Please don't focus on the specifics too much, just let me know if you think it's generally something that we should include and then we can focus on the specifics of the content.

In GitLab by [Gitlab user @phildawson] on Jun 14, 2018, 11:50
mentioned in commit 6946e86a352f6c99cbfc9bd3e60e9ec3a2bd475a

In GitLab by [Gitlab user @phildawson] on Jun 14, 2018, 11:50
mentioned in commit f8f58d16f61434c03823f755c1d07a36ba36c99e

In GitLab by [Gitlab user @phildawson] on Jun 14, 2018, 11:50
mentioned in commit 5d57664bb371c139a694b5da3e85240130d61ad2

In GitLab by [Gitlab user @phildawson] on Jun 14, 2018, 11:58
mentioned in commit 1195df75e58d19cb77081a0ff57ca0668160b7d2

In GitLab by [Gitlab user @phildawson] on Jun 14, 2018, 11:58
mentioned in commit 772e5a3ed85a43f6ccc8c54f3fab373fd4a89020

In GitLab by [Gitlab user @tristanvb] on Jun 16, 2018, 20:46
mentioned in commit 2d82b7471f349de510fffefb056eba9c0a0550d4

In GitLab by [Gitlab user @tristanvb] on Jun 16, 2018, 20:50
mentioned in commit 95df2d8f6495e3ba80a81094c16b6071baa3d587

In GitLab by [Gitlab user @tristanvb] on Jun 16, 2018, 21:14
mentioned in commit 1b88e2c6f0e3b060d7319e69d9ddf45bed98596f

In GitLab by [Gitlab user @tristanvb] on Jun 16, 2018, 21:51
closed via merge request !501

In GitLab by [Gitlab user @tristanvb] on Jun 18, 2018, 20:18
mentioned in commit b4105e8dea0f4aaaefbbd5a5dce344a939e736dc

In GitLab by [Gitlab user @tristanvb] on Jun 18, 2018, 20:20
mentioned in merge request !507


Use canvasCss rather than a

With this PR, grapesjs-blocks-bootstrap4 will load the css rules through the canvasCss option of grapesjs.
With this change, this bit of css code will no longer appear in the editor previews, as I believe should be the goal, given that these rules are only important for the canvas display, not the exported css code.

I would like this merged!

Hello, I'm the new maintainer of this repo, could you please update to the latest changes and resolve all conflicts so that I can merge this?


Maintenance

Unfortunately the original repository disappeared, and now that Python 2 dies a number of fixes are necessary.
Is this package still actively maintained? Is somebody accepting pull requests?

@saj Maybe the original author can provide some input, whether this project is still maintained (somewhere)?

I am surprised to see this little thing still kicking around after so long.
No, I am not aware of any actively maintained forks.  It has been many years since I last worked with Nagios, and all of my old colleagues -- some of whom may have contributed to this library -- have likewise moved on to other endeavours.  I do not personally know of anyone currently using this library in production.

@saj Well, things often live longer than expected. :wink:
I'm using it in production with Icinga, and some public plugins depend on it as well. Personally, I like it: it provides (nearly) everything needed and is self-contained; it just needs some small adaptions.
Thanks for feedback, then I'll stick to my fork.


[trade-secret] - make non-interactive method to create trade-secret

Add a way to non-interactively create trade secrets

silly me, should have replied here instead of opening https://github.com/alexbosworth/balanceofsatoshis/issues/554


Add API migration guide

So Go and JS clients can easily update with all the naming changes.
Should be human-written docs that capture all changes in a clear way.

Just some links to migration issues / docs:
This is currently for 0.6.0-rc2. Will add a follow-up for 0.6.0 final when it's ready.

poe-contracts migration doc: https://github.com/confio/poe-contracts/blob/main/MIGRATING.md
tgrade-contracts migration doc: https://github.com/confio/tgrade-contracts/blob/main/MIGRATING.md
Tgrade 0.6.0-rc2 release notes: https://github.com/confio/tgrade/pull/276#issue-1130648700

tgrade-contracts has its own issue: https://github.com/confio/tgrade-contracts/issues/461

Changes between 0.6.0-rc2 and 0.6.0 are listed in the changelog file: https://github.com/confio/poe-contracts/blob/main/CHANGELOG.md#v060-2022-02-15


About of accuracy on Washington RGB-D dataset

Hi, thanks for your good job.
I run
sh run_steps.sh step="FIX_EXTRACTION"
python main_steps.py
sh run_steps.sh step="FIX_RECURSIVE_NN"
python main_steps.py

and got Fusion result: 79.08% (121/153) which is not equal to the result in the Tabel 1 of your paper. I have two questions here:

Could you tell me how to modify and run the code if I want to get the result in Table 1?
Is the result of Top-1 average accuracy or others in  Table 1?

Thank you so much

And when I run
python main.py --run-mode 2
python main.py --run-mode 3

under ResNet101 backbone, the fusion result is: Fusion result: 79.08% (121/153) and Fusion result: 77.12% (118/153), respectively

Hi,
Thank you for your interest to this work. As stated in the paper, Washington RGB-D has 10 train/test splits (hence 10 trained models) and each split has about 7000 test images (not 153). The table results are the average results of the 10-splits. Please make sure that you're using WRGBD evaluation benchmark with the provided train/test splits.

Hi, Thank you for your interest to this work. As stated in the paper, Washington RGB-D has 10 train/test splits (hence 10 trained models) and each split has about 7000 test images (not 153). The table results are the average results of the 10-splits. Please make sure that you're using WRGBD evaluation benchmark with the provided train/test splits.

Thank you so much, I removed --debug and it worked. And I have another question: why shuffle = False is used for training dataset?
training_set = WashingtonDataset(params, phase='train', loader=custom_loader, transform=data_form)
train_loader = torch.utils.data.DataLoader(training_set, params.batch_size, shuffle=False)

Thank you so much

why shuffle = False is used for training dataset?

Glad it helped. shuffle = False is the case where the trained model is used as a feature extractor in extract_cnn_features.py. In that case, there is no need to shuffle the data.


fix(SortOptions): auctions sort default options remove

The type of this PR is: Bugfix

This PR resolves CX-760
Description
For sort by, remove the "default" option – we should show that we're already sorting by lot number ascending

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

LGTM, feel free to merge once the test is fixed.

Thanks for the quick review 🙏


Making the HTML of the page more semantic.

More explain what you change

The changes I made didn't affect the appearance of the page, they simply added meaning to the code using semantic HTML.
I wrapped the title with a header tag and wrapped the label, input and button within a form tag. For improved semantic clarity, I replaced the div tags for result and milestones with section tags. HoweverI kept the div for the container unchanged. I also wrapped the result section in a main tag, explicitly indicating it as the primary content within the body of the page.

Thank you for explaining @Mateusinbessa


Add enhancement API

Just track here:

[ ] Sync silent room with mobile  https://github.com/botorange/xiaoju-bot-pc/issues/266
[ ] Add tag. See #1627
...

Close this issue because it is outdated for a long time.


publish<EMAIL_ADDRESS>

Requested by: @philipphofmann
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave the #retract comment under this issue to retract the release (original issuer only).
Targets

[ ] github
[ ] cocoapods[sentryprivate-cocoapod]
[ ] cocoapods[sentry-cocoapod]
[ ] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

The upload failed cause of wrong CodeCov numbers https://app.codecov.io/gh/getsentry/sentry-cocoa/commit/c03c0e2cf925047d2e57bf725acecd9d02656c39. https://github.com/getsentry/sentry-cocoa/pull/3015 should fix this.

#retract


Update README.md extension configuration

Update README.md extension configuration

Hi @karuna,
I don't think that it is actually necessary because when you are installing the extension in VSCode there is a tab containing all available properties.

@KalitaAlexey yea I know, it's just my OCD kicked in because it's from old extension and some are not used anymore.

@karuna,
Yes. I thought about it.
I will remove it later.
Probably at some point we will fulfill wiki.
Close your PR if you agree.


What happens to deprecated/removed entries?

Mentioned by @eddie-knight in issue 88:

we should talk about (and document) what the project will do with an ID in the event that an entry is deprecated or removed after an official release. It will be cumbersome to users if an ID is reused for a different entry.

I can draft a PR for this in the next day or two. I don't think it needs a ton of discussion, so we can put together a draft and argue about it.

In an early stage nothing special, but at some point deprecated/removed entries should have their IDs "reserved" so they're not reused. Otherwise someone who earlier claimed to meet something might suddenly be misunderstood as doing something else.

Agree whatever we use to tombstone them it should also signal any automation that the control is gone.

Does anyone know how this has been done well elsewhere?
Perhaps we could clear all the keys except for id and add a new item signaling the removal?


is it supposed to work on a background queue?

Thanks for sharing your work. Trying to do something apparently simple but I see nothing on the console and breakpoints don't stop there. What am I getting wrong?
let backgroundQueue = DispatchQueue(label: "1sec",
    qos: .background,
    target: nil)

backgroundQueue.async {
    Timer.every(1.seconds) {
        self.log.debug("Timer tick at \(Date())")
    }
}

Cheers

It won't work, you have to call .start on an unscheduled timer with the .commonModes, IIRC. Otherwise it will only run on main thread and not when modal. If you're dealing with GCD, you can consider using GCD timers, too.

Thanks for replying, I think it worth to mention on the README. Cheers


cannot find -lpthreads when add package openssl.

I use
hunter_add_package(OpenSSL)
find_package(OpenSSL REQUIRED)

to add openssl  for my project. But I got  /usr/bin/ld.gold: error: cannot find -lpthreads in CMakeError.log. In order to decrease problems, I tried
sudo apt-get install libpthread-stubs0-dev

But don't make any affect.
os:Ubuntu 16.04

Now this line is printed:

But still with an error while link.

But still with an error while link

What is the error message? OpenSSL::SSL-NOTFOUND looks very suspicious, please post full logs. Also my advice will be to start with plain simple hunter_add_package(OpenSSL) and find_package(OpenSSL REQUIRED), without all extra modules you have added.

https://github.com/ruslo/hunter/blob/v0.23.214/README.rst


Document set_embed_options

As per https://stackoverflow.com/questions/73476232/altair-disable-the-open-in-vega-editor-option-in-a-colab-jupyter-notebook I only see a mention in the changelog https://altair-viz.github.io/search.html?q=set_embed_options&check_keywords=yes&area=default

For the record, the function itself https://github.com/altair-viz/altair/blob/3e60c99f1ee2b80461a40412e1e0a9689da82f1a/altair/utils/display.py#L39 is documented, but it's just not part of the web docs, likely because the util package is not part of the sphinx docgen.
Though I will say that additional info could be useful. For example, it looks like setting set_embed_options has no effect on the result of the various Chart.to_X methods, which was surprising to me. Any additional documentation might want to include this fact.
>>> import altair as alt
>>> import pandas as pd
>>> df = pd.DataFrame({"x": [1, 2, 3, 4], "y": [1, 2, 3, 4], "name": ["A", "B", "C", "D"]})
>>> chart_wo_embed_opts = alt.Chart(df).mark_circle().encode(x="x:O", y="y:O")
>>> alt.renderers.set_embed_options(actions=False)
RendererRegistry.enable('default')
>>> chart_with_embed_opts = alt.Chart(df).mark_circle().encode(x="x:O", y="y:O")
>>> chart_wo_embed_opts.to_json() == chart_with_embed_opts.to_json()
True

Not sure if this is intended or not. If not, I'll open a separate Issue.

embed options do not affect the chart specification, they control the HTML and Javascript code used to display the chart specification in the frontend

You can check the effect like this, for example:
>>> import altair as alt
>>> import pandas as pd
>>> df = pd.DataFrame({"x": [1, 2, 3, 4], "y": [1, 2, 3, 4], "name": ["A", "B", "C", "D"]})
>>> chart = alt.Chart(df).mark_circle().encode(x="x:O", y="y:O")
>>> alt.renderers.set_embed_options(actions=False)

>>> html = chart._repr_mimebundle_()['text/html']
>>> print('"actions": false' in html)
True
>>> print('"actions": true' in html)
False

>>> alt.renderers.set_embed_options(actions=True)
>>> html = chart._repr_mimebundle_()['text/html']
>>> print('"actions": false' in html)
False
>>> print('"actions": true' in html)
True

Thanks for the info - I figured it was intentional but I wanted to confirm.
Searching for set_embed_optoins came about because I'm trying to set some config that can be used for both colab/jupyter and flask+js. What I ended up doing on the flask+sj side was sending both the chart spec and the alt.renderers.options['embed_options'] to my HTML jinja2 template.

Related

https://github.com/marimo-team/marimo/issues/2302
https://github.com/marimo-team/marimo/pull/2303
https://github.com/streamlit/streamlit/issues/3258


resource_manager: report ru metrics to PD

What is changed and how it works?

Issue Number: Ref #14900
wait for #15100 merged
What's Changed:

Support for the upload metrics interface on the PD client, which uses the acquire_token_buckets interface in the PD server.
Change io consumption to distinguish between reads and writes for ru cost billing
Uploading metrics requires first getting the resource manager configuration from etcd in the PD
Calculate current ru cost by subtracting last io&cpu subsection from current io&cpu
When the limiter is restarted, the version will change, when the version changes, we need to recalculate the ru cost
Add mocker pd related function to check whether upload to pd

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

None

/test

/test

/merge


Custom BIO's

Since the file bio.h no longer includes the structure definition of bio_st, custom BIO's are no longer possible. Our application requires a custom BIO and has compiled under versions prior to 1.1.

Look at BIO_meth_new() and the BIO_meth_set/get functions.

Have you tried to port across to the APIs @richsalz noted? Have you identified specific items that are missing?

Okay that works. Nothing was missing, and the code even looks better.


CSS build @keyfrmes minification (CRA - v2.0)

Hey guys.
Hope you are doing fine.
I have found a bug with minified css.
create-react-app is minified css keyframes to some associative names like a, b etc. End this is bad because the new version creates two chunks main and vendor and both would have minified keyframes in each file and they would cross and conflict...

version to reproduce: 2.0.0-next.3e165448.
current stable version 1.1.4 works as expected.
p.s: sorry for my English

Do you have a repo?  I can test it out in next branch.  I've turned on safe-options in css-nano and hopefully this situation is handled as well.

I have found your improvement  - https://github.com/facebook/create-react-app/commit/52449c34eedc53e50a2a159d38604ea7df5bd997
and you are totally right, I have added this option { cssProcessorOptions: { safe: true } } to react-scripts/config/webpack.config.prod.js  locally and it works as expected

Thanks for testing it out.  A new build of 2.0.0@next was published with this change.


Need the ability to send transport headers

tcurl doesn't have a --headers flag that allows us to send transport tchannel headers

There are existing linter issues/ test issues that I haven't fixed not sure if anything should be done about those.

Is discussed on #29, please tell us more about what your needs are, transport headers are meant to be a tightly controlled internal detail, and not for arbitrary user extensible headers.

@jcorbin as @Raynos mentioned it is The use case is for tcurling ringpop services which expect shard keys in transport headers

@rssathe  We've had a change of heart; exposing --headers leaks too much implementation detail.
Can we change it to --shardKey=... ?

@Raynos done


Adds the functionality required to have CSRF be a opt-in, thus allowi…

…ng it to be disabled programmatically if it isn't going to be used.

I feel like I made sure tests were passing, I have no idea how to add a apiKeyset to the travis server, so I am going to assume this is expected?

Hey @jamesblack, thanks for the PR, but I'm afraid I can't merge this one in :(
Having CSRF tokens on the Registration / Login pages is 100% required -- disabling it won't do any good whatsoever and will only open you up to attacks.
If you want to simply disable the registration / login functionality completely, you can remove those two bits by saying:
app.use(stormpath.init(app, {
  enableLogin: false,
  enableRegistration: false,
}));

If you're trying to disable this because you're making client-side JS POST requests to create accounts or something, then you should probably create a separate endpoint all together that accepts JSON data and acts like a proper API endpoint.
I'm happy to answer any questions / etc.


fix(dragbox): Don't update if start/end coord are missing.

Fixes #150.

Looks good to me ... just can't approve :/


FAQ item request: Accessing current testcase name

I couldn't find this in the documentation or the FAQ: Is it possible to obtain the current test case's name from within that testcase's code? I mean, it should be, but how do we do it? That should be either in the FAQ or in some other relevant documentation file, IMHO.

That's a good idea. I'll probably look into this for the next version of doctest, and maybe I should also look at https://github.com/onqtam/doctest/issues/184 - basically the ability to introspect what is registered.
For now what you can do is to define a function like the following:
const char* getCurrentTestName() { return doctest::detail::g_cs->currentTest->m_name; }

in the translation unit where the framework is implemented (where DOCTEST_CONFIG_IMPLEMENT is defined) and then you can forward declare it anywhere you choose and use it.

I also think this would be a good feature to have.
My use case is when doing benchmarks with nanobench, it would be practical to use the same name for the benchmark and the test. getCurrentTestName would be great because in the case of templated test cases, it already includes the template parameters.

Besides getCurrentTestCaseName(), can we also have getCurrentSubCaseName()?
I hope we can see these API's soon, thanks!

Would love to see this feature added!


Ensure .NET Framework builds pass PEVerify

Fixes #734

I verified that without the compat flag, the builds fail :) So, hopefully we'll catch any future regressions


Debugging prevents saving, target is readonly (and outside source tree)

Type: Bug
While debugging (poor trim logic,local):
https://github.com/supaplextor/adbGoAutomation/blob/766012ff1339e1e9b4004431e52b42a3f26d0427/display/display.go#L19
I attempted to recode a few lines, but the editor says I cannot write to a readonly destination.  When I went to save as, it showed me I'm outside my editor directory.
VS Code version: Code 1.72.2 (d045a5eda657f4d7b676dedbfa7aab8207f8a075, 2022-10-12T22:16:26.920Z)
OS version: Linux x64 5.15.0-48-generic
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4570 CPU @ 3.20GHz (4 x 2474)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 2, 1

Memory (System)
31.27GB (21.68GB free)

Process Argv
. --crash-reporter-id 6eaa4cdc-e50c-44e4-9073-fd102e2a25e1

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (2)

Extension
Author (truncated)
Version

go
gol
0.35.2

makefile-tools
ms-
0.6.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
2144e591:30553903
pyindex848:30577860
nodejswelcome1cf:30587006
fc301958:30573243

Might have something to do with, sub-app in ./sample-apps/demo/demo.go references github by imports, doesn't know about ../../ (basepath of project). (GOPATH I guess? time for zzZZ 2AM lol)

/extGo

It looks like this is caused by the Go extension. Please file the issue to the Go extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!

TYVM. I'll head over there.


Questions about upgrading to newer versions

Hi there,

version 1. 12.3 is quite old, currently 1.18.2 is the up-to-date version so I was wondering whether you need to do something to get a newer version or if we are simply waiting for tailscale to release a newer arm64 version?
your upgrade instructions are to run: /mnt/data/tailscale/upgrade.sh 1.12.3
that doesn't work for me as the script is not executable.

# ls -al /mnt/data/tailscale/upgrade.sh
-rw-r--r--    1 root     root           744 Dec 30 17:29 /mnt/data/tailscale/upgrade.sh

why do you have to add 1.12.3 to the end of that command, to me it looks like the version is hard-coded in the upgrade script anyway.
how do we upgrade to newer version and how to know when there is a new release available?

###edit###
addressing my own points from  #3 #4 and #5
I manually checked and found a newer version available: https://pkgs.tailscale.com/stable/tailscale_1.18.2_arm64.tgz
after making the upgrade script executable:
chmod 744 /mnt/data/tailscale/upgrade.sh
I installed it successfully:
/mnt/data/tailscale/upgrade.sh 1.18.2

Hi @ovizii,
I'm gonna respond to your questions by number:

Yeah, when running the install script you can specify the version to install, however this wasn't particularly easy to do when running curl | sh so I've made some changes which should simplify that.
Indeed, I've updated the README to make it clear that you should specify the version you wish to upgrade to (as you've done ), let me know if that makes it clearer.
Thanks for spotting that, I've updated the install script to make sure that it is executable from the start.
You don't, there's a default-fallback version, but that tends to get out of date 😄
You should be notified of a new release on the Tailscale admin dashboard and you can upgrade to the latest version by running the same upgrade command you ran there 👍

Hi @ovizii,
I'm gonna respond to your questions by number:

Yeah, when running the install script you can specify the version to install, however this wasn't particularly easy to do when running curl | sh so I've made some changes which should simplify that.
Indeed, I've updated the README to make it clear that you should specify the version you wish to upgrade to (as you've done ), let me know if that makes it clearer.
Thanks for spotting that, I've updated the install script to make sure that it is executable from the start.
You don't, there's a default-fallback version, but that tends to get out of date 😄
You should be notified of a new release on the Tailscale admin dashboard and you can upgrade to the latest version by running the same upgrade command you ran there 👍

Thanks for clarifying, let me close this issue then.


Feature/15/maintain authors

Resolves https://github.com/PMPL-Arieken/django-locallibrary-tutorial/issues/15
added author functional, delete and update

Codecov Report

Merging #20 (103af5c) into master (c042650) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           38        38
  Lines          867       867
=========================================
  Hits           867       867

Impacted Files
Coverage Δ

catalog/tests/test_forms.py
100.00% <ø> (ø)

catalog/forms.py
100.00% <100.00%> (ø)

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 c042650...103af5c. Read the comment docs.


Bug Fixes

Codecov Report

Merging #14 (022915d) into master (9ae87f1) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 022915d differs from pull request most recent head 99f9488. Consider uploading reports for the commit 99f9488 to get more accurate results

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   91.27%   91.27%
=======================================
  Files           6        6
  Lines         711      711
=======================================
  Hits          649      649
  Misses         62       62

Impacted Files
Coverage Δ

treeo/__init__.py
100.00% <100.00%> (ø)

treeo/tree.py
94.84% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Add timeout to current status pipeline to avoid exceeds pipeline connection timeout

Description
The current status pipeline is executed into the raspberry pi server:

https://github.com/ocriado91/python-telegrambot-flashcards/blob/main/.github/workflows/current_status.yaml
In case the raspberry pi server where the FlashcardBot is deployed is down, GitHub Actions try to execute this pipeline into the server reaching the following error:

Solution

[ ] Add a timeout (e.g. 10 minutes) into the check status pipeline to avoid pipeline connection timeout.

According to: https://github.com/orgs/community/discussions/50926, Github doesn't allow configuring custom timeout to avoid stuck queued pipelines, but we can check if there are any queued workflows after a specific timeout (e.g. 10min) and cancel using GitHub CLI.

I think best option is to remove Current Status badge and CI pipeline


TM-87 Adding My Pledges page

Dependent on TM-24 and TM-43

Can we also get the SonarCloud issues back down to the minimum please?


Images are not uploading in this editor

As you said to check the error log, here is what I found in error logs.
[23-Dec-2016 15:37:56 America/Chicago] PHP Fatal error:  Call to undefined function qa_upload_file_one() in /home3/name/public_html/name/qa-plugin/q2a-trumbowyg-editor-master/qa-trumbowyg-upload.php on line 34
[23-Dec-2016 15:38:13 America/Chicago] PHP Fatal error:  Call to undefined function qa_upload_file_one() in /home3/name/public_html/name/qa-plugin/q2a-trumbowyg-editor-master/qa-trumbowyg-upload.php on line 34

which version of q2a you are using?

I am using Q2A 1.7.4 brother.
On Mon, 26 Dec 2016, 19:44 Amiya Sahu<EMAIL_ADDRESS>wrote:

which version of q2a you are using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amiyasahu/q2a-trumbowyg-editor/issues/2#issuecomment-269214878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGKdgtyAOMrlllbZ4XelowD6vGTdqy4cks5rL8vOgaJpZM4LVE17
.

f8c50609dd85fa8a84bc2f2fbc4c73a20b9ca546 this should fix the issue. Somehow I missed to test it after I made little refactor.  It should work now.

Brother now it's working fine for any live website on a server. But on localhost, when we upload an image it shows that the image is broken. But, as you save the question then, images also shows up just fine.
I tried the same with Ckeditor but that works fine for both localhost and live website. Maybe this is an issue with the editor itself but I thought to just let you know about this.

Can you check now on this issue

Yes, now it's working perfect...

Great. Closing the issue now


az functionapp config appsettings set fails to update broken keyvault reference

Describe the bug
The 'az functionapp config appsettings set' command fails to update a setting when the current value is a broken key vault reference with Error: None message.
To Reproduce

Add a setting with a key vault reference with the format @Microsoft.KeyVault(VaultName=;SecretName=)
Restart the function and check that the reference gets resolved.
Delete the secret from the key vault, restart the function and check the reference status (should be unresolved).
Run the command to update the setting with a plain text value (does not matter if the new value is another key vault reference or just plain text).

Expected behavior
The Setting value should be updated (at least if the new value is valid).
Environment summary
az CLI version 2.34.1 (Windows and Linux)
Additional context

Fails on individual or multiple assignments, as long as the broken reference is one of the values intended to update.
The current workaround is to delete the existing broken setting first and then set the new value.
I could not verify the same behavior happens when the reference is not resolved for other reason than a nonexisting secret, but I think that is possible.

route to CXP team

@CorrenSoft Thanks for reaching out to us. I used the below command to create and update the KeyVault reference in config appsettings. I tried with both existing and non-existing keyvault references. Could you please test the same and let us know if you face any issues?
----CLI Command START----
az functionapp config appsettings set --name [my-app-name] --resource-group [my-app-res-group] --settings `""PublicKey=@Microsoft.KeyVault(VaultName=[vault-name];SecretName=[secret-name])"`"
----CLI Command END----

Also if you still face the issue, could you please provide me more information about the reference status and broken reference? I am not sure if I understood it quite right. Awaiting your reply.

We had a setting with a key vault reference in place, but after a review, we decided that the value was not sensitive and decided to change it to plain text format. So, in the CI/CD process, we deleted the secret from the key vault and then tried to update the setting, but the command failed each time until we actually deleted the setting.
Today, I rebuilt the same scenario as yesterday, and I get the error again one time, and then start to work, and I could not reproduce it again. The very same command that was failing now is working. Could be another issue platform-related?
Thanks for your time, if you can not reproduce the error in the described scenario I think that you can close this, and I will be back if the issue returns in the future.

@CorrenSoft unlikely a platform issue. If you are consistently able to reproduce the issue, please feel free to reopen this thread. We would be happy to work with you with on this again.


fix(Tabs): prevent string conversion when unnecessary

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Found 5 minutes of free time and decided to fix a small bug/micro-optimization.
If we use ||, then with value = 0, it will be converted to a String. However, the model is number | string, so we can avoid unnecessarily converting the number to a string.

works now
after fix

0 -> "0"
0 -> 0

1 -> 1
1 -> 1

2 -> 2
2 -> 2

...
...

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks @hywax! 😊


Setting language string / label on model gives JavaScript error

When I try to set a language string via Builder -> Models -> Forms -> Label: Title [+] sign
I get an error when I have filled in the String key and String value and hit the ok button:
TypeError: this.options is null
http://hostname/plugins/rainlab/builder/assets/js/build-min.js?v1.0.5
Line 2
Is there away to test with an unminified build-min.js version ?
I am using:
FireFox 44.0.2 (Google Chrome also gives the same TypeError)
Builder 1.0.5
October Build 316

Can you please post the call stack from Chrome console? You can expand the exception message in the console to see the stack. Thanks.

Have set some breakpoints but the stacktrace does not reveile much:
Uncaught TypeError: Cannot read property 'plugin' of nulli.onSubmitPopupForm @ build-min.js?v1.0.5:2n.event.dispatch @ jquery.min.js?v316:3r.handle @ jquery.min.js?v316:3
TypeError: this.options is null
i.prototype.onSubmitPopupForm()
build-min.js:2
n.event.dispatch()
jquery.min.js:3
Don't bother too much.
I started from scratch with a new builder plugin and now I am able to set language strings. The initial setup takes about 20 - 30 seconds probally the first time I was impatient and clicked away while the blue bar was still running.


OutputFormatter provides no way to disable text encoding

Title
OutputFormatter provides no way to disable text encoding
Functional impact
It seems impossible to write a class derived from OutputFormatter for handling a custom binary media type that uses no text encoding.
Minimal repro steps

Create new Web API app (using "Microsoft.AspNet.Mvc": "6.0.0-beta4")

Add a MyBinaryOutputFormatter class:
 public class MyBinaryOutputFormatter : OutputFormatter
 {
     public MyBinaryOutputFormatter()
     {
         SupportedMediaTypes.Add(new MediaTypeHeaderValue("application/x-mybinary"));
         SupportedEncodings.Add(Encoding.UTF8);
     }

     public override bool CanWriteResult(OutputFormatterContext context, MediaTypeHeaderValue contentType)
     {
         return context.Object is byte[];
     }

     public override async Task WriteResponseBodyAsync(OutputFormatterContext context)
     {
         using (var stream = new System.IO.MemoryStream())
         {
             byte[] data = (byte[])context.Object;
             await context.ActionContext.HttpContext.Response.Body.WriteAsync(data, 0, data.Length);
         }
     }
 }

Add the output formatter in Startup.cs:
 services.AddMvc().Configure<MvcOptions>(options =>
 {
     options.OutputFormatters.Add(new MyBinaryOutputFormatter());
 });

Add a controller and action like the following:
 [HttpGet]
 public byte[] Test()
 {
     return new byte[] { 0xFF };
 }

Start the service and do a GET request to the action method using a HTTP client that allows you to inspect the response body contents as hex.

Expected result
Since I write to the response body stream directly, I would expect the bytes I write to be exactly those seen in the response body client-side.  That is the response body seen client side should be the single byte FF (hex).
Actual result
The response body seen client side is the single byte FD (hex).
Further technical details
Using Encoding.ASCII instead in the MyBinaryOutputFormatter constructor seems to be a hacky workaround to this problem, but then all responses have a Content-Type header of application/x-mybinary; charset=us-ascii, which doesn't look great to clients.
If the single-byte response is not convincing that the problem is the encoding, try returning a larger byte[] from the Test action method.  It quickly becomes evident that the encoding is corrupting the data.

@Tratcher Why would what got written to the body change in this case?

I actually forgot to add one other point: what I think the user expectation would be here.  My expectation was that the SupportedEncodings and the OutputFormatterContext.SelectedEncoding would never be automatically applied, and that I should be responsible for encoding any textual information according to the SelectedEncoding.
Furthermore, it should be possible to have no encoding at all - e.g. Content-Type: application/x-mybinary instead of Content-Type: application/x-mybinary; charset=....  Is it possible to do this?  Most binary formats have no concept of choosing a charset.

I wouldn't expect the above code to encode the body in any way. I can only assume that WriteResponseBodyAsync isn't being called and some other code is sending the body. @rynowak?

@Tratcher WriteResponseBodyAsync is definitely being called, I have a breakpoint and can step through it all with the debugger.  This is a minimal example that reproduces the problem, but in my original scenario I had a "real" serialization going on in my custom OutputFormatter for which there was no way something else could be sending the body.  Examining the hex of the response body client-side revealed I was definitely getting the results of my custom serialization done in WriteResponseBodyAsync - just they were corrupted because of a UTF8 encoding.

I figured out what the problem is...  The Chrome App "DHC" which I am using as my test client is failing to render the correct hex for the response body!  It appears that it is stripping the charset off of the Content-Type on the response and then modifying the response body based on the charset.  I made the request with a different test client and the hex was all correct.
In summary: The problem was with the DHC client, not MVC 6.
I still am perplexed as to how I can entirely omit any encodings and charsets from my OutputFormatter and response headers.

@timothy-shields thanks for the feedback. We're looking into making a fix for OutputFormatter to make it not set a charset if SupportedEncodings is empty (instead of throwing).
As a workaround, you might want to look into implementing IOutputFormatter directly. Take a look at StreamOutputFormatter. It's not a 100% fit for what you're doing - StreamOutputFormatter will write any content-type, but you should be able to implement what you want.

Fixed by 35ee0e3a4958db6e33d92b5f71506066b6fd842e
We won't throw anymore if SupportedEncodings is empty.


Front-end create function to read fen string

Need a function to read a fen string that will be sent from the back-end. Function needs to change the position of the pieces on the board.

02ce28f165ff4730084b6f9c4314165db6ecf12e


Hamcrest matcher builder

Hi!
I love the DSL to assert an object easily (ie: assertThat(myBean).hasName("foobar").isActivated(); ).
But I also use a lot Mockito which is based on Hamcrest to verify parameters. I would like to reduce the gab between them by having a matcher builder providing the same kind of DSL. Something like:
Matcher<MyBean> matcher = buildMatcher(myBean).hasName("foobar").isActivated();
// I could use it like (mockito):
verify(myMockedBeanDao).save(argThat(matcher));

And why not to have a possibility to use it (or any Hamcrest matcher) like that:
assertThat(myBean).with(matcher);

That's just a thought but it could be a functionality I will use each time I write unit test (almost every day)!
Thanks for considering it
Thomas

I'm not sure whether it would make more sense for an adapter to be part of Hamcrest, or part of AssertJ, or whether it would make more sense for Mockito to accept Condition. But at this point it would be a stumbling block for us in converting to AssertJ.

I would agree to have a separate jar :)

If anybody else lands here, it is now supported in assertJ Core 2.9.0 and 3.9.0. See https://github.com/joel-costigliola/assertj-core/blob/master/src/test/java/org/assertj/core/api/HamcrestConditionTest.java for an example.


Not smooth in phonegap / cordova apps.

Dear @bodymovin ,
I just rendered by using the plugin and try to embed in cordova projects.

Here my projects link
Could you help with this experience?

hi, can you share the .aep?


InfluxDB Writer: Write tags for typeNames instead of prefixing the Fi…

This fixes review comments for #601

:x: Build jmxtrans 180 failed (commit https://github.com/jmxtrans/jmxtrans/commit/5fb5385965 by @Navgeet)

@Navgeet this looks good. I'll wait for the build to go green and merge. The SNAPSHOT will be available in https://oss.sonatype.org/content/repositories/snapshots/org/jmxtrans/ a few minutes after the merge. If you need a release, let me know, I'll publish it (which I should probably do anyway).

Hello! What is the status of these changes? Can I help you somehow to speed up the process?

This has been merged and should be available in the latest SNAPSHOT. We have not done a release in quite some time, I'll get on that!


Upgrade to RC.3

Removes TemporarySources to use contentFiles packages
Remove default globs
Update test platform

Hi @natemcmaster, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Add documentation of header and footer for MultiSelect component

Subject
MultiSelect
Description
#2519
Add the functionality added by this PR to the document.
Are you willing to participate in creating this documentation page and create a pull request?
None

Please refer to #2634.

It's similar to https://github.com/yamada-ui/yamada-ui/pull/2634, so I'll take care of it!

@kaehehehe
Hi, thanks for the comment!
Yes, pls!


Seed transform tests

Follow-up to and partial revert of #6741.
Although it is ok to have "random" values in the inputs definition, we don't want actual random values inside our tests. The former communicates that the actual values don't really matter, while the latter makes sure we don't have flaky tests and can reproduce CI behavior locally.
Thus, this PR reverts the seed changes of #6741 and adds a fixture that sets the RNG seed before every test. If #6750 is resolved, we can probably also put back the "random" sizes for the gaussian_blur_* kernels.

I don't mind reverting my changes.
The reason why I didn't follow this approach is because the population of the KERNEL_INFOS dictionary happens long before the tests actually run and thus before the seed is set. This leads to different sizes for the actual data tensors which leads to the seed setting not having any effect. Am I missing something?

As a result the random calls to determine the sizes yield different results every time.

True. And I need to look into how we can fix that. This is why I put back your changes to the sample inputs definition in 7d25068f21387025fb1a7c49e6ae340ba80a6875.

Consequently the actual data tensors will be different from run-to-run making the seed setting to have no effect.

Yes, they will be different in shape. This is not desired and I will look into it as explained above. However, your PR also does not fix this as the setting of the seed only happens inside the test and thus at the same time as the fixture of this PR does.

However, your PR also does not fix this as the setting of the seed only happens inside the test and thus at the same time as the fixture of this PR does.

My original PR circumvents the problem by not having random calls for the specific problematic kernel during the KERNEL_INFOS initialisation. So right now, on main branch, the random data are deterministic (I've checked).
I'm not sure I get how this PR fixes the issue as you say. Does fix_rng_seed() execute before KERNEL_INFOS get populated?


Support for cells.zarr.zip for Xenium

Depends on https://github.com/scverse/spatialdata-io/pull/111 (should be merged first).
Closes #110

Codecov Report
Attention: 84 lines in your changes are missing coverage. Please review.

Comparison is base (944dec6) 35.76% compared to head (bc9cc35) 34.92%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
- Coverage   35.76%   34.92%   -0.85%
==========================================
  Files          16       16
  Lines        1194     1280      +86
==========================================
+ Hits          427      447      +20
- Misses        767      833      +66

Files
Coverage Δ

src/spatialdata_io/_constants/_constants.py
100.00% <100.00%> (ø)

src/spatialdata_io/readers/xenium.py
21.97% <13.40%> (-3.55%)
:arrow_down:

Tested on all the public Xenium versions using this https://github.com/giovp/spatialdata-sandbox/blob/main/tests/test_to_zarr_xenium.py

I applied the code suggestions and performed some additional cleanup. Also, I realized that the link between the labels indices for the cell segmentation and the instances of the rows required to parse additional data from cells.zarr.zip. Now this is included.
I tested the code on the Xenium Analyzer versions 1.0.2, 1.3.0, 1.4.0, 1.5.0, 1.6.0, 1.7.0, 2.0.0.
One note on the new labels: the table is still mapped to the circles/shapes but the user can change the annotation target of the table to the labels using the new set_table_annotates_spatialelement() method. I plan to show this in an example notebook.
Using the multiscale labels instead of the shapes for visualizing the data not only improves the quality of the visualization over circles and polygons, but also doesn't compromise performance. Here is an example of the new (labels) vs old (circles) napari view.

Full code to load the data, change the table and view the data.
##
import spatialdata as sd
from napari_spatialdata import Interactive

sdata = sd.read_zarr('data.zarr')

sdata['table'].obs['region'] = 'cell_labels'
sdata.set_table_annotates_spatialelement(
    table_name='table',
    region='cell_labels',
    instance_key='cell_labels'
)

Interactive(sdata)


Resources with compound keys are different than sub-resources

There are use cases where resources are identified via compound-keys, e.g. article-size-advice/{sku}/{sales-channel}. Rule 143 requires that a 'artificial sub-resource' is created, like article-size-advice/{sku}/sales_channel/{sales-channel}. This does not feel right: There is no article-size-advice/{sku} as an own resource with distinct life cycle. There are a couple of APIs that are not compliant to the rule or work around the rule (like above, or string concat). I vote for being less restrictive and allow article-size-advice/{sku}/{sales-channel} for resources with compound keys.

Rule 143 requires that a 'artificial sub-resource' is created, like article-size-advice/{sku}/sales_channel/{sales-channel}. This does not feel right: There is no article-size-advice/{sku} as an own resource with distinct life cycle.

If this doesn't feel right then neither should the following proposal:

I vote for being less restrictive and allow article-size-advice/{sku}/{sales-channel} for resources with compound keys.

Because a resource identifiers should be a single path segment, otherwise servers are exposing compound key structure to their clients. That's a detail that clients shouldn't be concerned with.

or work around the rule (like above, or string concat)

String concat (or any other mean to produce a reversable, single identifier) sounds like a good idea to me. Clients can take identifiers as opaque values without the need to treat their parts individually. That should make a) clients simpler and b) free servers to evolve their compound keys more freely.

Because a resource identifiers should be a single path segment, otherwise servers are exposing compound key structure to their clients. That's a detail that clients shouldn't be concerned with.

But in my sample scenario the server wants to expose compound key structure, and clients have to provide both compound elements as resource id.

But in my sample scenario the server wants to expose compound key structure

What's a realistic scenario when this would be desired? We already advertise to encode id properties as opaque strings:

the identity of the object. If used, IDs must be opaque strings and not numbers. IDs are unique within some documented context, are stable and don’t change for a given object once assigned, and are never recycled cross entities.
https://opensource.zalando.com/restful-api-guidelines/#174

Exposing a compound key/id structure in the URL feels inconsistent and against our own principles.

Yes, ids should be opaque. The guidance is fine, and I don't want to address this topic.
However, we have not (yet) explicitly addressed situations where resources are identified via compound keys (or compound identifier). Sometimes compound keys make perfectly sense, e.g. article-size-advice is provided for each article sku in a sales-channelspecific way. There are many similar examples in our current APIs.
Now the question is, how to handle it. I see ad-hoc the following variants:
a) Encode components into id: /article-size-advice/{id} and define in id description that it is actually build via {sku} + _ + {sales-channel}. I don't like this variant, because we cannot specify the components as OpenAPI element, and client has obligation to construct the id. It is error prone and  only partial opaque.
b) Pass the id components via query params with GET /article-size-advice&sku={sku},sales_channel={sales-channel} or via mandatory payload attributes skuand sales_channel with PUT/POST. This feels not good and is against Rule 143 to use path segments for resource ids.
c) Pass only the sku component via path, and use 'X-Sales-Channel': GET /article-size-advice/{sku} and require X-Sales-Channel={sales-channel} as mandatory. This is against our guidance about header usage Rule 183, and without explicit compound id definition.
d) Introduce sub-resource, like /article-size-advice/{sku}/sales_channel/{sales-channel}, conform to Rule 143. This is artificial and IMO misleading -- there is no sub resource with own life cycle, there is only a main resource identified via two component ids.
e) Handle id components as id path segments: /article-size-advice/{sku}/{sales-channel}. This is currently not conform to Rule 143. However, when we defined the rule we did not discussed compound id situation. I suggest, to relax the rule as to support compound ids in the path, because ids should be preferable in the path.
In our API landscape we can find diverse examples for all of the variants above.


EDITOR (Vim) prints Command terminated when opening a file

When I open a file through cfiles, in my EDITOR (Vim), it prints the following message:

before showing the file content.
I know the issue has to do with Vim in particular (since it doesn't occur with nano for example), but is there some program/command that is run before Vim is run so that this message is printed?

try doing this?

Thanks for link! After some debugging, I wen to check my plugins and it turns out, this is actually caused by tpope/vim-fugitive. So I'm closing this now. Apologies if i caused any inconvenience.


Consider supporting unix domain sockets with SocketTransport

Seems like it's possible.
https://github.com/dotnet/corefx/blob/958511d023231f120109967484fb63ae1837e6b8/src/System.IO.Pipes/src/System/Net/Sockets/UnixDomainSocketEndPoint.cs
https://github.com/dotnet/corefx/blob/b5d0c3277dc384a23fee2e57a0b5c232c99dbd88/src/System.IO.Pipes/src/System/IO/Pipes/NamedPipeServerStream.Unix.cs#L290-L301
It would be cleaner if we kept using the socket APIs and could just copy that end point logic over to kestrel. Ideally we wouldn't have to use the pipe API to support this.'
This might be the only problem https://github.com/dotnet/corefx/blob/958511d023231f120109967484fb63ae1837e6b8/src/System.IO.Pipes/src/System/Net/Sockets/UnixDomainSocketEndPoint.cs#L25
/cc @stephentoub

Acceptance checklist (check one item)

[ ] We decided not to take this fix.
[ ] The fix is tests-only.
[ ] The fix contains product changes (check all items below).

[ ] Relevant XML documentation comments for new public APIs are present.
[ ] Narrative docs (docs.microsoft.com) are updated. (check one item below)

[ ] The change requires a new article. An issue with an outline has been filed here: [ISSUE LINK]
[ ] The change requires a change to an existing article. A docs PR with these changes is linked here: [PR LINK]
[ ] The change requires no docs changes.

[ ] Verification has been completed. (check one item below)

[ ] The change is in the shared framework and was verified against the following versions

SDK installer: [VERSION]
ASP.NET Core Runtime: [VERSION]
.NET Core Runtime: [VERSION]

[ ] The change is in an OOB NuGet/NPM/JAR package and was verified against the following version of that package: [PACKAGE ID] [VERSION]

I need to test this on *nix.

Tested in a docker container:
Runtime Environment:
 OS Name:     debian
 OS Version:
 OS Platform: Linux
 RID:         debian-x64
 Base Path:   /usr/share/dotnet/sdk/3.0.100-preview6-012234/

Host (useful for support):
  Version: 3.0.0-preview6-27730-01
  Commit:  63abc77da6

.NET Core SDKs installed:
  3.0.100-preview6-012234 [/usr/share/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.App 3.0.0-preview6.19303.1 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.0.0-preview6-27730-01 [/usr/share/dotnet/shared/Microsoft.NETCore.App]


open a .cc file(2000 lines) window will be blocked and no response

Issue Type: Bug

open folder with remote-ssh
open a .cc file such as render_frame_impl.cc(in chromium project)

and then the window will be blocked and no response
VS Code version: Code - Insiders 1.35.0-insider (a5536b8f5a16a10d859f3dec1e59701671bf069e, 2019-05-20T05:15:57.974Z)
OS version: Darwin x64 18.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (7.11GB free)

Process Argv
-psn_0_340051

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

intellij-idea-keybindings
k--
0.2.31

remote-ssh
ms-
0.37.0

remote-ssh-edit
ms-
0.37.0

remote-ssh-explorer
ms-
0.37.0

vscode-gn
npc
1.0.0

This was fixed upstream at jeff-hykin/cpp-textmate-grammar in version 1.8.15, see jeff-hykin/cpp-textmate-grammar#174
The fix should make it to vscode insiders the next time the grammars are updated. If you need the fix now you can install marketplace.visualstudio.com/items?itemName=jeff-hykin.better-cpp-syntax&ssr=false

This was fixed upstream at jeff-hykin/cpp-textmate-grammar in version 1.8.15, see jeff-hykin/cpp-textmate-grammar#174
The fix should make it to vscode insiders the next time the grammars are updated. If you need the fix now you can install marketplace.visualstudio.com/items?itemName=jeff-hykin.better-cpp-syntax&ssr=false

Thank you  It's worked..


Compile docs-only CSS in eleventy build

Allows us to add scss in ./docs/assets/css/app.scss which will be included in the documentation CSS, but not in the published npm package.

Will this also work for images and JavaScript?

Will this also work for images and JavaScript?

It should do, but I haven't tried it


Prometheus adapter not working for Linear buckets and exponential buckets.

prometheus.go references types from pkg/adapter/metrics.go. Ideally that file pkg/adapter/metrics.go should be deleted and //adpater/prometheus should only know about template types or it's own config. In this case it seems like the prometheus should be referencing config. instead of old structs from pkg/adapter/metrics.go
Because of same named types, prometheus is trying to cast types from config. into //pkg/adapter/metric.. But that does not work because the types inside the adapter/prometheus/config/ dir is is different from the one inside //pkg/adapter/metric.go. NOTE:  This is not caught during compile time because in the prometheus code we convert the objects into interface{} and then try to do a type conversion. Because of the hop to interface, golang compiler will not complaint.
code that does the cast and should be fixed:
https://github.com/istio/mixer/blob/master/adapter/prometheus/prometheus.go#L264
This needs to be done for 0.2.

@mandarjog @ZackButcher @douglas-reid FYI

OK to close?

yes. fixed


[SPARK-30480][PYSPARK][TESTS] Fix 'test_memory_limit' on pyspark test

What changes were proposed in this pull request?
This patch increases the memory limit in the test 'test_memory_limit' from 1m to 8m.
Credit to @srowen and @HyukjinKwon to provide the idea of suspicion and guide how to fix.
Why are the changes needed?
We observed consistent Pyspark test failures on multiple PRs (#26955, #26201, #27064) which block the PR builds whenever the test is included.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Jenkins builds passed in WIP PR (#27159)

Test build #116449 has started for PR 27162 at commit 7c77ec7.

In case if the PR build doesn't take the test (I think it shouldn't), I'll also trigger the build in #27159 couple more times.

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-K8s/21237/
Test PASSed.

any idea on why this change is needed now?

Test build #116449 has finished for PR 27162 at commit 7c77ec7.

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/116449/
Test PASSed.

Sorry I have no idea. I guess there's some change on Jenkins env. or code change might affect, but it's not easy to figure out why it's failing in either way only to investigate on log file. The change also doesn't hurt.
cc. @shaneknapp Do you have any idea on changes in Jenkins env.? The test suddenly started to fail yesterday.

I am going to merge to unblock those PRs.
Merged to master, and branch-2.4.

Thanks for the quick reaction! Let's see builds for these PRs pass.

Looks like the builds are still failing from previous PRs, though I'm not sure why the builds in new WIP PR were succeeded.

I'll revert this and take a look tomorrow if that's still persistent.

Reverted.


Classes with names that only differ in casing.

Class POI differs only in case from Poi. Such classes will overwrite one another on case-insensitive filesystems.
scalac silently overwrites its own output producing an inconsistent set of class files in a situation when it works on a case-insensitive filesystem and there is a name conflict in our case between these two classes
Would be great to merge them or extend the parent class if needed.
Best regards,

Resolved via #19


[WIP] Adding ops for feature column functionality and feature column to workflow mapping function

Increasing NVTabular compatibility with TensorFlow feature column API by adding remaining necessary ops (cross op and bucketize) and a function which can map from a set of feature columns to an NVTabular workflow that performs all analogous preprocessing. Addresses #371
HashedCross doesn't support multi-hot yet, and I'm not sure that extending to it will be necessarily easy. For reference, the TF cross op handles multi-hots by doing a cartesian product of all indices for each feature. See the documentation here.
Still need to add bucketized support and test everything.

@benfred what's our stance on adding ops without list support? If we're ok with it, should we add a support matrix in the documentation?

With the addition of bucketize, we should have full TF feature column coverage (minus the sequence columns which I won't worry about for now). The shared embeddings and weighted shared embeddings are more of Keras layers than they would be preprocessing steps, so will still need to add layers that cover those. But overall this should put us in pretty good shape. Just need to build tests, docs, and an example and we should be good to go

rerun tests

Adding an example for using the new utilities that will hopefully act as a simple introduction to the more advanced tf notebook

rerun tests


Add FAQ

PR types
New features
PR changes
Docs
Description
add PaddleNLP_FAQ.md

 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.小五井 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@mawenjie8731 参考下OCR团队的FAQ文档撰写吧

Please resubmit a new FAQ doc and refer to PaddleOCR's FAQ doc.

Please resubmit a new FAQ doc and refer to PaddleOCR's FAQ doc.


Ironmonk88 chat message changes

resolved merge conflicts.

I think it would be good if someone could take a look if that would still work like this especially ironmonk88. The rolling in starfinder alone still works fine.

How come this PR doesn't have the if statement around the chat message creation in dice.js?

Wait nvm checking the full file I can see what's going on


Authorisation recipe verb

The current draft authorsation recipe uses the verb 'http://activitystrea.ms/schema/1.0/authorize' from activity streams . The description is:
Indicates that the actor has authorized the object. If a target is specified, it means that the authorization is specifically in regards to the target. For instance, a service can authorize a person to access a given application; in which case the actor is the service, the object is the person, and the target is the application. In contrast, a person can authorize a request; in which case the actor is the person and the object is the request and there might be no explicit target.'
I'd read this as actor(shibboleth?) authorises object (person) to target (application). But xAPI has no  target property in a statement ?
There is also a draft person requests authorisation recipe, which I think would be fine?
Ideas on the verb + usage?

David, all -
(a quick quibble with the activity streams definition: a person does not authorize a request, but the fulfilment of that request. at least I think that is what they mean. otherwise its like "you have permission to ask for more gruel")
As a general point, I'm not enthusiastic about fairly abstract recipes. Encompassing both a person responding to a request and the Shibb scenario feels too abstract to me. And I think it rather invites an over-broad scope of potential application. Surely, at this level of abstraction, we can declare that VLEs are involved in authorising users to access certain course areas. Or that the security system authorises access to the computer lab.
I would rather see verbs which are quite concrete, and give a fairly clear indication of what was done (without needing to nuance this by taking account of the object type). That just makes life easier to process the data in a less byzantine process. This isn't to say that there isn't value in looking at the abstract level and saying "there is an authorisation pattern", which several recipes might adopt so that there is some consistency.
If the requirement is a recipe which allows resource access using UKAMF services then let's be specific to that. Maybe call it the "federated access recipe". I think that could bring some attributes into scope which an abstact recipe might neglect.
And if it is about access card and building security, a different recipe, with different attributes from facilities databases.
Adam


Couldn't run case with large mesh (~8 mln cells) in parallel mode

Hello everyone,
I try to run MFEM examples (9 and 18)  with a big mesh (near 8 mln hexagonal cells). When I run a code, after several minutes I see an error: one of the processes has been killed. I see that the process has been killed during mesh reading. I haven't any problem with 1 mln cells or less. Where should I find a source of this problem? How to deal with big meshes correctly?
Mesh format is .msh
--
Best regards, Victoria

Hi @vkorchagova,
You may be running out of memory. Can you start with a coarser mesh and refine in parallel? Alternatively, can you run on a machine with more RAM?
Tzanio


Fix the cdn path of the images for Amazon s3

Hello,
At the moment the cdn replace system change images path from
../dist/img/
to
BUCKETURI/BUCKETNAME/SUBFOLDER/dist/img/
That url however return the error "The bucket you are attempting to access must be addressed using the specified endpoint. Please send all future requests to this endpoint.".
To solve the issue, you should change set as BUCKETURI the endpoint of the bucket (ex. 'http://BUCKETNAME.s3-website-BUCKETREGION.amazonaws.com') and remove the BUCKETNAME after this in the CDN replacing task.
Could this help?
Cheers,
Raffaele

Hey @oldirony thanks for making the change.
Can somebody else that uses s3 confirm? Maybe @clecompte?


⚠ Bofrost.lu has degraded performance

In 79001cf, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1646 ms

Resolved: Bofrost.lu performance has improved in 0750666 after 20 minutes.


Database tables VariableCodedValue, VariableDescription, VariableCodeDescription, and VariableCategory should be renamed

There is an inconsistency in the naming of the database tables that became apparent when writing the paper up. We have Environmental, EnvironmentalValue, EnvironmentalVariable which contrasts with VariableCategory, VariableDescription, VariableCodedValue (yikes).
This is a hang-up from the old focus on the Eth. Atlas (it's what the EA calls these), but now that we're more dataset agnostic, we should rename them.

Here are the names I've given them in paper, so that they are parallel to those assigned to Environmental tables. Just let me know if you select different names:
VariableCodedValue -> CulturalValue
VariableDescription -> CulturalVariable
VariableCodeDescription -> CulturalCodeDescription
VariableCategory -> CulturalCategory

Fixed in #218 -- great work Stef! @kirbykat the manuscript figure should match the database now.

Great!! And belated thanks for that figure, Stef. @SimonGreenhill will issue #206 change the final figure? I couldn't tell if you and @xrotwang were suggesting cutting those tables. By the way, afaik we have not yet had a case where one glottolog language maps to >1 ISO 639-3


[766] remove headings from trainee edit

Context
trainees#edit
Changes proposed in this pull request
Remove section headings from the trainee edit page
Guidance to review
🚢

 Remove section titles/headers in completed record


Fixing prompt text when asking for AWS credentials

Summary
Fixing the wording used in the quickstart prompts for AWS credentials. Current wording causes confusion as the term secret is used for both the access key and secret key

/lgtm
/approve

/lgtm

/approve

/approve


Use OIS package from separate repository in Airnode

Blocked by https://github.com/api3dao/ois/issues/2

Can you assign priority, size, and iteration?

Blocked by https://github.com/api3dao/ois/issues/6

re:testing this: I'll most probably have to make sure the examples still work and that when the repo is bootstrapped and built by yarn, it pulls the package from another repo, correct?

Hi @Siegrift and @amarthadan , as mentioned above, it seems this could be tested by only running an example, which I've done several times today without any particular issues regarding the OIS package. I assume this is now done but feel free to ping me if not.


fix issue on postgres, maintain backwards compatibility

fixed issue where postgres wasn't comparing strings versus integers. also put in a change that should make this reverse compatible with prior versions (essentially not requiring the type of resource_id to change to a string for older installs).

Please let me know if there are any suggestions. I saw no way to do this with existing methods, so wrote my own :-/.  Really looking forward to feedback though.  (BTW, tested on mySQL and Postgres.  Test suite ran on SQLite)

Thanks for working on this and testing against postrgres! :)
Could you add a few comments to the code explaining why the resource_id can either be a string or an integer and why we type cast them?

That's for issue #34 by the way!

Alright, I've got something in there.  It should also serve as backwards
compatibility too.
On Thu, Feb 23, 2012 at 2:08 AM, Philippe Creux <
<EMAIL_ADDRESS>
wrote:

That's for issue #34 by the way!

Reply to this email directly or view it on GitHub:
https://github.com/gregbell/active_admin/pull/1063#issuecomment-4132291

--
Jeff Ancel
(314) 703-8829 - Main
www.jeffancel.com


Nuxt project creation failed.

Environment
win11
Reproduction
In the local environment, whether I use the 'npx' or 'pnpm' command, creating a Nuxt project fails with the error message: 'Error: Failed to download template from registry: fetch failed'.
Describe the bug
pnpm dlx nuxi@latest init nuxt-test  or pnpm dlx nuxi@latest init nuxt-test
Additional context
No response
Logs
No response

https://github.com/nuxt/cli/issues/159

https://github.com/nuxt/cli/issues/159。maybe project should support http proxy when using nuxi init，not #21853。


Bug#1.3 Bus sometimes duplicates

A bus when tracking sometimes makes multiple entries in the database (Sometimes 4-5 at once)

Need to also figure out what causes this, as it only happens sometimes


training on different traffic histories

@YuechengLiu great PR! By the way, please take a look at all the new alerts introduced by this PR mentioned by LGTM 😸

I will fix them soon.

@YuechengLiu great PR! By the way, please take a look at all the new alerts introduced by this PR mentioned by LGTM 😸

I tried to fix these errors, and I fixed some of them. But it seems that some of the run_scripts are not compatible with rlkit.envs.get_envs, even before my last commit.

Sure, some of the run_scripts may be outdated for some time. I will check them before merging.

I will merge this PR today.

Thank you for fixing problems in run_scripts of other algorithms :rose:


In windows bit64 Could not find \expanded.

Is there something I could do to overcome this error when trying to execute some tikz latex code in Jupyter notebook cell magic %%tikz?
("C:\Program Files\MiKTeX 2.9\tex\generic\oberdiek\etexcmds.sty"
Package: etexcmds 2016/05/16 v1.6 Avoid name clashes with e-TeX commands (HO)
Package etexcmds Info: Could not find \expanded.
(etexcmds)             That can mean that you are not using pdfTeX 1.50 or
(etexcmds)             that some package has redefined \expanded.
(etexcmds)             In the latter case, load this package earlier.
)))

Script I'm executing is:
%%tikz
\documentclass{article}
\usepackage{tikz}
\usetikzlibrary{shapes,backgrounds}
\begin{document}
\pagestyle{empty}
\def\firstcircle{(0,0) circle (1.5cm)}
\def\secondcircle{(60:2cm) circle (1.5cm)}
\def\thirdcircle{(0:2cm) circle (1.5cm)}
\begin{tikzpicture}
    \begin{scope}[shift={(3cm,-5cm)}, fill opacity=0.5]
        \fill[red] \firstcircle;
        \fill[green] \secondcircle;
        \fill[blue] \thirdcircle;
        \draw \firstcircle node[below] {$A$};
        \draw \secondcircle node [above] {$B$};
        \draw \thirdcircle node [below] {$C$};
    \end{scope}
\end{tikzpicture}
\end{document}

Also other examples say "No image generated.". I have installed basic-miktex-2.9.6219-x64 and it didn't show any special errors on installation process.

Sorry for the somewhat late reply. As the error message suggests, this is usually related to and old pdftex version. It should be at least 1.50. This is rather ancient, though, so that I would expect MiKTeX 2.9 to include a much newer version. Still, can you check your version, e.g., by typing pdftex in the command line and copying the first line?

Sorry for the somewhat late reply. Your example is faulty. The TikZ magic doesn't work like that. What you want is rather something like
%%tikz -l shapes,backgrounds
    \begin{scope}[shift={(3cm,-5cm)}, fill opacity=0.5]
        \fill[red] \firstcircle;
        \fill[green] \secondcircle;
        \fill[blue] \thirdcircle;
        \draw \firstcircle node[below] {$A$};
        \draw \secondcircle node [above] {$B$};
        \draw \thirdcircle node [below] {$C$};
    \end{scope}

In addition you would like to have a LaTeX preamble containing
\def\firstcircle{(0,0) circle (1.5cm)}
\def\secondcircle{(60:2cm) circle (1.5cm)}
\def\thirdcircle{(0:2cm) circle (1.5cm)}

This is currently not supported, but not hard to add to the magic.

I added an option for specifying a preamble. The correct example should look something like the following. First, in an Python code cell, define
preamble = """
\def\firstcircle{(0,0) circle (1.5cm)}
\def\secondcircle{(60:2cm) circle (1.5cm)}
\def\thirdcircle{(0:2cm) circle (1.5cm)}
"""

This should be followed by a TikZ cell,
%%tikz -l shapes,backgrounds -x $preamble
    \begin{scope}[shift={(3cm,-5cm)}, fill opacity=0.5]
        \fill[red] \firstcircle;
        \fill[green] \secondcircle;
        \fill[blue] \thirdcircle;
        \draw \firstcircle node[below] {$A$};
        \draw \secondcircle node [above] {$B$};
        \draw \thirdcircle node [below] {$C$};
    \end{scope}


[Bug]: DNS validation fails

Description
DNS validation always fails although DNS correctly configured and propagated. Fails for united-domains.de nameserver as well as other.
Minimal Reproduction (if possible, example repository)
Go to Settings -> configuration -> General. Set the domain which has configured *.xx.domain.io wildcard subdomain as well.
Click on save -> validation error. Tried two different domains with two different nameserver providers, always shows this error:
Validating DNS (https://xx.domain.io) failed.Make sure you have added the DNS records correctly.Check this documentation for further help.
Exception or Error
Validating DNS (https://xx.domain.io) failed.Make sure you have added the DNS records correctly.Check this documentation for further help.
Version
v4.0.0-beta.274

DNS are out of scope, but is your DNS resolving to the server ip ?

@fcpauldiaz yes, DNS A record is resolving to the server ip. And the wildcard subdomain A record too.

I found the error, fixing in the next version!