Fixes issue #1370

As described in the issue #1370 -- the method SignInManager.IsSignedIn(ClaimsPrincipal) throws when the request has a Windows Identity attached to it.
The root cause for this is described in Ph1ll's comment on this issue.
I believe this proposed solution keeps the expected behavior of the method, while fixing a relevant issue for applications on the enterprise.
This issue is blocking adoption of ASP.NET Core as it is preventing the deployment to production of an application in a Fortune 500 pharmaceutical company.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@HaoK - we appreciate your attention and feedback on this issue. We really need this fix.

If you are using windows auth, you can probably just replace the call to SignInManager with something like User.Identity.IsAuthenticated instead of using this method if its causing exceptions in this case...  This method is only really supposed to be used when you are using cookie authentication, not windows auth...

@HaoK  We use a combination of cooke and windows authentication, hence the need to call the method.

@HaoK your comments are appreciated. The PR is very simple, I'd like to understand what is the hesitation to approve it.
Thanks

We can certainly consider this for 2.1, but the benefits of a fix like this are fairly limited, you could just wrap your calls to SignInManager.IsSignedIn(ClaimsPrincipal) in your app with this try/catch today.  That's a pretty simple workaround if this is blocking you.

Or you can just check if User.Identity.IsAuthenticated first, and only call SignInManager.IsSignedIn after that is false, since that would mean its safe to call right?

@HaoK Trust me, we have tried.
Why would using a try catch wont work? Because a ClaimsPrincipal can have multiple identities. If we wrap the call to IsSignedIn with try catch then the enumeration through the identities won't complete and hence we wont know if the user is logged in.
For the second one: I will try it this evening.

@HaoK  "Or you can just check if User.Identity.IsAuthenticated first, and only call SignInManager.IsSignedIn after that is false, since that would mean its safe to call right?" This won't work because User, as a ClaimsPrincipal, has multiple Identities linked to it. When IIS sends us the Windows Authentication Identity, this is marked as the main Identity (User.Identity) and hence the call to User.Identity.IsAuthenticated will always return true (since IIS has authenticated that user) but that does not mean that he/she should access the application since we have yet to make sure that identity is registered in the EF Identity database.

SignInManager.IsSignedIn is mostly a sugar method meant to make the normal case easier.  Given that you have a more complex auth setup with Windows Auth involved, I'd recommend just writing custom method to do this functionality for you.
As you've seen the method is just a more compact way of writing:
return principal?.Identities != null &&
                 principal.Identities.Any(i => i.AuthenticationType == IdentityConstants.ApplicationScheme);

This method had more of a use when the ApplicationScheme was configurable, but now that's been removed in 2.0, its basically just a static extension method now...


Adding run crash notification

It would be useful to be notified if a run crashes due to snakemake/cluster failure so we could resubmit the job asap.

I'm not sure of all the conditions you want to be notified of, but it's possible that using the slurm notification using --mail-type (and --mail-user) would be useful to you for alerts of job failures


OTR: move smoothened binding under G-protein coupled receptor binding

Move GO:0005119 smoothened binding from under  ‘GO:0005109 frizzled binding’  to under ‘GO:0001664 G-protein coupled receptor binding’
smoothened (hedgehog pathway), although related to frizzled receptors, acts in a completely different manner to the frizzled receptors (wnt pathway). As its mechanism and biology are so divergent, it is not a frizzled receptor in any meaningful sense. GO:0005119 	smoothened binding should be moved from under GO:0005109 frizzled binding and made a child of GO:0001664 G-protein coupled receptor binding to reflect this.

The placement of GO:0005119 'smoothened binding' under GO:0005109 'frizzled binding' is coming from the logical def of 'frizzled binding':
binding
and ('has input' some 'frizzled-like G-protein coupled receptor')
Since PRO classifies 'smoothened' as a 'frizzled-like G-protein coupled receptor', 'smoothened binding' is being classified in GO as an is_a child of 'frizzled binding'.
PRO does not seem to have an entry for 'frizzled', but instead has the various frizzled and smoothened receptors as siblings with the common parent 'frizzled-like G-protein coupled receptor'.
Since we seem to be moving away from adding terms like 'frizzled 2 binding' or 'frizzled 9 binding', etc., I'm not sure it makes sense to have a grouping term for 'frizzled-like G-protein coupled receptor binding' in GO.
For now, I will remove the logical def for 'frizzled binding' and make sure the asserted parentage for 'smoothened binding' is still good.

Note: will also follow up with PRO to see what they think about adding an entry for the 'frizzled' family.


Bug Report: Low Quality Crashing

During the first transition in HOME, the game crashes if the player has low quality mode enabled.
On the first release of the unfair update, it threw this error:

And on the current commit, it either crashes with a memory error or with no error box at all.

Fixed, will be in the upcoming update.


Exit pool form updates

close #1167

Not sure I really like this behaviour, it seems inconsistent with what we have done elsewhere. Just because the user doesn't have assets we still let them view the trade form and the join pool form.


Access the relation databag if unit is leader

Description
This aims to fix https://github.com/canonical/self-signed-certificates-operator/issues/24. Within this change, non-leader unit does not try to access app relation databag and permission denied error does not appear.
Checklist

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have made corresponding changes to the documentation.
[x] I have added tests that validate the behaviour of the software.
[x] I validated that new and existing unit tests pass locally with my changes.
[x] Any dependent changes have been merged and published in downstream modules.
[x] I have bumped the version of any required library.

Are unit test cases aligned with this change?

Are unit test cases aligned with this change?

Tests are added.


Fix/chinese i18n

Short description of what this resolves:
Retranslate texts in package.nls.zh-cn.json
Changes proposed in this pull request:

Chinese i18n texts

Fixes: #
How Has This Been Tested?

Screenshots (if appropriate):
Checklist:

[x] I have read the contribution guidelines.
[ ] My change requires a change to the documentation and GitHub Wiki.
[ ] I have updated the documentation and Wiki accordingly.

Duplicate of #962


ColumnView style update

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #5041

Details of Changes
Add details of changes here.

update column view to use 2.6 styling
left align dv status column
remove blade entrance animation
make blade bg transparent

Validation
How did you test these changes?

[x] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)

@winston-de should the dividers between the columns use the same brush as the dividers throughout the rest of the app?

good idea


prevent dialogs closing on background clicks #442

Description
Added logic on onClose prop in dialog components to prevent them closing on background clicks
Testing instructions

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage

Agile board tracking
closes #442

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

Project coverage is 93.66%. Comparing base (cd2f09f) to head (d05a1b4).
 Consider uploading reports for the commit 861798d to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #449      +/-   ##
===========================================
+ Coverage    93.64%   93.66%   +0.01%
===========================================
  Files           61       61
  Lines        16156    16194      +38
  Branches      2099     2099
===========================================
+ Hits         15130    15168      +38
  Misses        1020     1020
  Partials         6        6

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


Disable scene objects at build time

discussions about NetworkScenePostProcssing: https://github.com/MirageNet/Mirage/discussions/608
problems with disabling objects:

User does not control enable/disable of gameobjects.   This creates unexpected behaviors for users.  Complicated and brittle logic for scene objects

feature is left over from UNET, hard to know reason for initial feature

Solution:

[ ] remove this feature
[ ] Add to migration guide

I did a test solution comparing the current vs not doing the disable/enable with NetworkScenePostProcess and ServerObjectManager.
I used the ChangeScene example. In Room1 and Room2 I added 1000 SceneObjects. I ran the server in server only mode. I put a Stopwatch at the start and end of SpawnObjects().
Current: 24-26ms scene changes
Dont Disable: 17-19ms scene changes
roughly 30% faster


Set a name to the arrow function assignation

So we get a correct displayName in development

Thank you!


Global style keyframes injection

Fixes #185
This PR:

ensures that css.global are server rendered
ensures that css.keyframes are server rendered

Breaking change:
In order to be able to inject global styles, we need to make that a function.
const globalStyles = css.global({})

// call it
globalStyles()

in React

const globalStyles = css.global({})

const App = () => {
  // call here
  globalStyles()

  return <div/>
}

We could provide a nicer API, maybe a hook, in @stitches/react

This is fixed in 0.0.3-canary.1
yarn add<EMAIL_ADDRESS>
Note, this release contains the following breaking change:

Util API #209
Fix global and keyframes style injection https://github.com/modulz/stitches/pull/208


Confusing dates in readme

This is really a minor thing:
In the readme it says:
A stable v1 release is expected for March 2018, while some concepts are thought through, and prototype code refactored.

Today it is november 2019, which is faaaar after march 2018 ;)
Perhaps it would make sense to update this to reflect what the reality of today is? (what is the latest version etc...).
Or remove that sentence?

done.
Also switched to newer version of pipeline, so it's much easier to release.


Channels not captured by "attach environment"

If a user has included a number of channels when they create their environment, these will not be captured by conda env export. Not sure what to do about this...

mmm... this should be probably handled by the export mechanism in conda env... I guess...

Channels are captured in the environment on master. Closing this one now.


Feedback for “How To Use - Vehicles - Documentation”

I bought the Model-Y from Playlolly, I try to convert to Add-on mode, but after I execute the Game and try to spaw this car, will occur the "err_sys_invalidresource_5", please give me some advise, thanks

Hi B2o5T,
I play the GTA-5 for SP mode, so if I cannot convert to the add-on, how to deploy the car into the GTA-5 for spawning? thanks

@royeshao you created an issue in the wrong repo


Support for RequireJS plugins

Do you want to request a feature or report a bug?
Not sure

What is the current behavior?
Webpack supports AMD, but support for RequireJS plugins is either missing or undocumented.
If the current behavior is a bug, please provide the steps to reproduce.

Get the text plugin for RequireJS
Create a file named file.txt with some text
Define a module plugintest like this:

define(["requirejs/text!path/to/file.txt"], function(myFile){
  return {
   content : myFile
  };
});

What is the expected behavior?
plugintest should be able to get loaded just like any other module, and its output should be a POJO with a content property that contains the content of file.txt.
If this is a feature request, what is motivation or use case for changing the behavior?
The lack of support for RequireJS plugins prevents the use of Webpack for libraries that use any such plugins.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
The behavior of the plugin mentioned hereabove is identical to the behavior of the dojo/text. I would expect to be able to replace dojo/text with requirejs/text or vice versa without having to update any configuration.

Is there a reason that you need RequireJS plugin support specifically? I imagine this would work if you were to use webpack's raw loader - https://github.com/webpack-contrib/raw-loader

RequireJS plugins are not supported. You can solve this specific case with:
resolveLoader: {
  alias: { "requirejs/text": "raw-loader" }
}

I guess this is good enough for now.
Thanks!


Babel runtime error

I just following the initial steps for getting started, as mentioned I started with installing the module form npm and then, import the Typing module, using corresponding JSX code I am trying to get the typing effect added but instead I am getting an error message in my terminal.

This is the exact message I am getting after I have followed the mentioned steps, any ideas why this would happen.
TIA!!

This is resolved, in fact there was nothing wrong, I just had the library added in the wrong location.
Sorry to bother.
Thanks!

Please Ashish please elaborate. I'm having the same issue and don't seem to be able to figure this out.


PLAT-51382: Syncing DOM structure and logic between JS version and Native version lists

Issue Resolved / Feature Added
There was a big difference of DOM structure between JS versions and native versions of lists, and it leads missing patches for bug fixing or enhancing features even both versions share a lot of internal logic.
Resolution
Changed DOM structure of JS versions as same as native versions and removed redundant code.
Links
PLAT-51382
Comments
Enact-DCO-1.0-Signed-off-by: Seungcheon Baek<EMAIL_ADDRESS>

LGTM

:shipit:


Refactor file viewer

Proposed Changes
(Describe your changes here. Also describe the motivation for your changes: what problem do they solve, or how do they improve the application or codebase? If this pull request fixes an open issue, use a keyword to link this pull request to the issue.)
...

Screenshots of your changes (if applicable)

Associated documentation repository pull request (if applicable)

Type of Change
(Write an X or a brief description next to the type or types that best describe your changes.)

Type
Applies?

🚨 Breaking change (fix or feature that would cause existing functionality to change)

✨ New feature (non-breaking change that adds functionality)

🐛 Bug fix (non-breaking change that fixes an issue)

🎨 User interface change (change to user interface; provide screenshots)

♻ Refactoring (internal change to codebase, without changing functionality)

🚦 Test update (change that only adds or modifies tests)

📦 Dependency update (change that updates a dependency)

🔧 Internal (change that only affects developers or continuous integration)

Checklist
(Complete each of the following items for your pull request. Indicate that you have completed an item by changing the [ ] into a [x] in the raw text, or by clicking on the checkbox in the rendered description on GitHub.)
Before opening your pull request:

[ ] I have performed a self-review of my changes.

Check that all changed files included in this pull request are intentional changes.
Check that all changes are relevant to the purpose of this pull request, as described above.

[ ] I have added tests for my changes, if applicable.

This is required for all bug fixes and new features.

[ ] I have updated the project documentation, if applicable.

This is required for new features.

[ ] If this is my first contribution, I have added myself to the list of contributors.

After opening your pull request:

[ ] I have updated the project Changelog (this is required for all changes).
[ ] I have verified that the pre-commit.ci checks have passed.
[ ] I have verified that the CI tests have passed.
[ ] I have reviewed the test coverage changes reported by Coveralls.
[ ] I have requested a review from a project maintainer.

Questions and Comments
(Include any questions or comments you have regarding your changes.)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

115 of 145 (79.31%)  changed or added relevant lines in 10 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.04%) to 91.795%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/javascript/Components/Result/binary_viewer.jsx
27
30
90.0%

app/javascript/Components/Result/file_viewer.jsx
21
25
84.0%

app/javascript/Components/Result/text_viewer.jsx
35
44
79.55%

app/javascript/Components/Result/image_viewer.jsx
0
14
0.0%

Files with Coverage Reduction
New Missed Lines
%

app/javascript/Components/Result/file_viewer.jsx
1
71.93%

Totals

Change from base Build<PHONE_NUMBER>9:
0.04%

Covered Lines:
41320

Relevant Lines:
44339

💛  - Coveralls


Unhelpful error message if you accidentally ask for too many clusters in cluster studio

What happens?
If you accidentally ask for more clusters than exist when calling linker.cluster_studio_dashboard() (either explicitly using sample_size, or if using the default value of 10), which could happen e.g. if your clustering threshold_match_probability is too low, if your model is too matchy, if your blocking_rules_for_prediction are too small, etc.
The process will hit an error with e.g.
duckdb.ParserException: Parser Error: Sample sample_size 133.333333 out of range, must be between 0 and 100
as the attempted sample proportion is > 1.
Should probably adjust proportion to be == 1 if would-be proportion is > 1, and perhaps give a warning that the required sample size could not be satisfied (particularly as this may well be indicative of issues with the linkage model).
To Reproduce
import pandas as pd

from splink.duckdb.duckdb_linker import DuckDBLinker
import splink.duckdb.duckdb_comparison_library as cl
from splink import __version__

print(__version__)

df = pd.DataFrame(
    [
        {"id": 1, "forename": "Stuart", "surname": "Smith"},
        {"id": 2, "forename": "Maria", "surname": "Williamson"},
        {"id": 3, "forename": "Charlotte", "surname": "Davies"},
    ]
)

print(df.head())

settings = {
    "unique_id_column_name": "id",
    "link_type": "dedupe_only",
    "comparisons": [
        cl.exact_match("forename"),
        cl.exact_match("surname"),
    ],
    "retain_intermediate_calculation_columns": True,
    "retain_matching_columns": True,
    "probability_two_random_records_match": 0.2,
}

linker = DuckDBLinker(df, settings)

linker.estimate_u_using_random_sampling(1e7)
linker.estimate_parameters_using_expectation_maximisation("1=1")
df_predict = linker.predict()

df_clusters = linker.cluster_pairwise_predictions_at_threshold(df_predict, threshold_match_probability=0.8)
print(df_clusters.as_pandas_dataframe().head())
# only have three clusters - the individual records, as they are unlinked @ 80%
linker.cluster_studio_dashboard(df_predict, df_clusters, "cluster_studio.html", overwrite=True, sample_size=4)

OS:
macOS
Splink version:
3.6.0
Have you tried this on the latest master branch?

[X] I agree

Have you tried the steps to reproduce? Do they include all relevant data and configuration? Does the issue you report still appear there?

[X] I agree

NB Splink 4 reprex:
import duckdb

import splink.comparison_library as cl
from splink import DuckDBAPI, Linker, SettingsCreator, splink_datasets

df = splink_datasets.fake_1000

settings = SettingsCreator(
    link_type="dedupe_only",
    comparisons=[
        cl.ExactMatch("first_name"),
        cl.ExactMatch("surname"),
        cl.ExactMatch("dob"),
        cl.ExactMatch("city").configure(term_frequency_adjustments=True),
    ],
    retain_intermediate_calculation_columns=True,
)

con = duckdb.connect()
db_api = DuckDBAPI(con)

linker = Linker(df, settings, db_api)

df_e = linker.inference.predict()
df_c = linker.clustering.cluster_pairwise_predictions_at_threshold(df_e, 0.01)

linker.visualisations.cluster_studio_dashboard(
    df_e, df_c, "tmp_csd.html", overwrite=True, sample_size=500
)


[Game Crash] Was doing a jump on a d-force pad

Describe how it crashed by giving us those informations:

What you were doing before it crashed and what action made the game to crash.
Doing a jump on a d-force pad

Upload the crashinfo.txt (GameFolder/crashinfo.txt)
log.txt
crashinfo.txt
info.txt

Upload the info.txt and log.txt (GameFolder/Logs)

Explain how we can reproduce the error

Start the game
Go to screen '...'
Select '...'
Game crashed

Sreenshots/Videos
If possible, take screenshots or record the crash happening.
Additional information
If there's any information that you think would help us.

06:49.006: OutFox Engine (SDL Input Debug) - Controller Removed 00
06:49.201: InputFilter::ButtonPressed: Invalid device 39
it seems when you jump the pad disconnects, does this happen in windows itself when you jump cuz it it does it might be that your pad or the cable of the pad is malfunctioning

Fixed a while ago, sorry for not replying here ;_;


[swagger] Add javadoc-swagger-generator

This is a very rough PR. Currently it just parses this:
/**
 * My very nice description.
 * It can be multiple lines,
 * because why not!
 *
 * @path /users/:user-id
 * @pathParam user-id a very nice description
 * @queryParam my-query-param some other description
 * @formParam my-form-param a terrible description
 * @result SerializeableObject object
 */

Into this:
description:    My very nice description. It can be multiple lines, because why not!
path:           /users/:user-id
path-params:    [(user-id, a very nice description)]
query-params:   [(my-query-param, some other description)]
form-params:    [(my-form-param, a terrible description)]
return-type:    SerializeableObject

The idea is to use this information, together with some global settings, to produce a full swagger spec.
If we make this feature complete, we should probably not merge it into Javalin, but release it as a separate javadoc-swagger-generator project.
Alternatively we can make an annotation:
@SwaggerApi(
        description = "My other nice description. It's just a single line.",
        path = "/users/:user-id",
        pathParams = [
            PathParam("user-id", "a very nice description"),
            PathParam("other-id", "another description")
        ],
        queryParams = [QueryParam("my-query-param", "some other description")],
        formParams = [FormParam("my-form-param", "a terrible description")],
        result = SerializeableObject.class
)

Definitely use the annotation in my opinion

@HexRealm why is that? I've created two screenshots to show the two different solutions in action:
Annotation:

Javadoc comment:

Maybe this is just me, but the annotation just really rubs me the wrong way, it looks so noisy compared to the javadoc comment. Type safety is nice though...

Hi @tipsy and @HexRealm,
I'm developing something like this.
This is the link of the project (https://github.com/gibran/tupi) and I will appreciate a lot if you can to looking that.
I have some catalog issues to improve it like to create a possibility to set any kind of authentication for the routes and other.
Let me know if it is something like this that you are thinking.
Thanks

@gibran Looks interesting. Why did you choose to create four different annotations, instead of just one?

@tipsy mostly because of type safety & IDE hinting, but that's my opinion. As for @gibran's project looks promising. Swagger already has a library that contains a collection of annotations, maybe we could look into using it? https://github.com/swagger-api/swagger-core/wiki/Swagger-2.X---Annotations

Why do we have to use annotations as Kotlin is a good fit for a good DSL? Take a look at this repo: we can make a dsl based on this structure making our code pleasant to read!

@gibran Looks interesting. Why did you choose to create four different annotations, instead of just one?

@tipsy I did it because I thought to create separated processors per annotations, and I thought to create a feature to generated a file to configure the binds between route and controllers.

Hi @tipsy / @hamnis do you have any suggestions? do you see any possibility to introduce it as javalin extension?

Hi @gibran. What do you mean as a Javalin extension?
@rbygrave showed an interesting approach to generating Javalin applications in https://github.com/tipsy/javalin/issues/450, I'm thinking we should collaborate closer with him and generate Swagger docs the same way. The default (purely programmatic) way of writing Javalin apps is a poor fit for Swagger.
I'm on vacation soon, so I'll think a bit more about it in the coming weeks.

@tipsy when I told extension in though create a extension method to set configuration for swagger.
When I started to build the tupi project I just though about dinject javalin. As I developed for long time in .net and I based on https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/master/README.md and in create this anotations to intercept  the lifecycle and invoke the right controller or method inside the controller.
In my first step in tupi was to create a oas 3.0 to swagger documentation, my second step will be to generate a file to bind routes with controllers like get("/api/hello") -> helloController.sayHello.


NDT_CUDA crash

Hi author,
I build fast_gicp with option(BUILD_VGICP_CUDA "Build GPU-powered VGICP" ON). Set reg_method is NDT_CUDA_D2D or NDT_CUDA_P2D, hdl  will crash when received laser points. But when using with FastVGICP_CUDA and FastVGICP(disable BUILD_VGICP_CUDA) , there is no problem. It seems that it is  fast_gicp::NDTCuda lib problem. My OS is ubuntu20.04, noetic, PCL is 1.10. When debug with gdb, there is no backtrace, so i only have the terminal log. Thanks for your time very much :)

nodelet: /build/pcl-gWGA5r/pcl-1.10.0+dfsg/kdtree/include/pcl/kdtree/impl/kdtree_flann.hpp:138: int pcl::KdTreeFLANN<PointT, Dist>::nearestKSearch(const PointT&, int, std::vector<int>&, std::vector<float>&) const [with PointT = pcl::PointXYZI; Dist = flann::L2_Simple<float>]: Assertion `point_representation_->isValid (point) && "Invalid (NaN, Inf) point coordinates given to nearestKSearch!"' failed.
[velodyne_nodelet_manager-1] process has died [pid 279759, exit code -6, cmd /opt/ros/noetic/lib/nodelet/nodelet manager __name:=velodyne_nodelet_manager __log:=/home/jxl/.ros/log/6f74d344-5d6d-11ec-9054-af2fc89854e3/velodyne_nodelet_manager-1.log].
log file: /home/jxl/.ros/log/6f74d344-5d6d-11ec-9054-af2fc89854e3/velodyne_nodelet_manager-1*.log

I ran into this issue as well but it was crashing on FastVGICP_CUDA as well. I was able to resolve this by going to commit fa23f9b493347452ea3a029b088b7694f766053f. It seems the Eigen upgrade broke the CUDA stuff.

Hi @rzhao88, sorry for late reply. I tried to switch to  fa23f9b493347452ea3a029b088b7694f766053f commit of fast_gicp. When using with NDT_CUDA_D2D and NDT_CUDA_P2D, hdl complains tf error. But when using with FastVGICP_CUDA, hdl is normal.

Error:   TF_NAN_INPUT: Ignoring transform for child_frame_id "base_footprint" from authority "unknown_publisher" because of a nan value in the transform (nan nan nan) (0.000000 0.000000 0.000000 1.000000)
	 at line 240 in /tmp/binarydeb/ros-noetic-tf2-0.7.5/src/buffer_core.cpp
[ERROR] [1640833458.491322637, 53.560000000]: Ignoring transform for child_frame_id "base_footprint" from authority "unknown_publisher" because of a nan value in the transform (nan nan nan) (0.000000 0.000000 0.000000 1.000000)

Hi,
I tried to use NDT_CUDA_D2D and NDT_CUDA_P2D, and I got the same error than @narutojxl, is there any upcomming update to fix this @koide3 ? Thank you for the answer.

I get the same error. Is the update to Eigen that has to be done ?

Anyone have a complete solution?

sudo apt purge libpcl-dev
git clone https://github.com/ros-perception/perception_pcl.git


(SERVER-2345) Support autosigning when generating certs

Previously if the user tried to generate a cert that was autosigned by
the CA, the command would error, because it would request that the cert
be signed after it was already signed by autosign. This commit updates
the generate logic to take into account autosigning.

Also tested this out manually with a local puppetserver configured with autosign = true.

Updated.

So the problem with this now is that you can go in and sign the CSR after receiving a 400, but then the cert will never get placed in the cert dir. It still appears in the signeddir, so not sure how much that matters. But it is different from how generate usually works. Still might be better than forcing a clean and regenerate.


Problem: sticky seems to calculate sidebar height wrong, overlaps footer

Hi!
I'm using your plugin on one of my sites, and I'm very happy with it, except for one thing that I'm not able to resolve:
It seems it calculates the height wrong.
You can see it in action here, with the ad on the right:
http://delbart.se/saab-blocket/
When the white are ends, the sticky item is supposed to stop, but it keeps on for a couple of hundred pixels.
Any idea what might be wrong?

Sorry, I solved it by myself. I had misinterpreted a setting, but now it all works as expected! Thanks for a wonderful plugin!


Not able to update token image , name and SYMBOL (basically any information)

I tried to merge the json but I`m getting the following message.
This repository has been archived by the owner. It is now read-only.
Neither strata or token creator, none of them worked as well, can somebody provide me a solution or a simple guide on how to do that?
thanks,

I think you're in the wrong repo! This isn't a Strata repo, and the token list isn't managed here either.


fix JSON error

This pull request was made in order to solve an issue.
When slack authentication fails (in my case fails when I don't use 'identity.basic').
So when slack API returns a error passport-slack throws the following exception:
ReferenceError: json is not defined
   at /Users/vfernandes/project/node_modules/passport-slack/lib/passport-slack/strategy.js:92:16
   at passBackControl (/Users/vfernandes/project/node_modules/oauth/lib/oauth2.js:125:9)
   at IncomingMessage.<anonymous> (/Users/vfernandesproject/node_modules/oauth/lib/oauth2.js:143:7)
   at emitNone (events.js:91:20)
   at IncomingMessage.emit (events.js:185:7)
   at endReadableNT (_stream_readable.js:974:12)
   at _combinedTickCallback (internal/process/next_tick.js:74:11)
   at process._tickCallback (internal/process/next_tick.js:98:9)

The pull request #23, solves this issue but isn't the best way to do it. Because done callback doesn't return the error message.

+1

+:1+ Please fix!


Scene centric mode

Hi,
I noticed that UnifiedDataset is not working with the centric="scene" option ,because some methods are missing such as load_all_agent_data. Is this intentional and functionality will be added at a later point?

Hi @Leon0402 , thanks for reaching out!
Yes, we have actually just implemented this feature internally, and will soon roll it out to this public version of the codebase, sorry that I did not include it in the initial release!


Merge pull request #51 from OfficeDev/dev

November 2015 release

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


Incompatibility with Either Galleries or a Photo Page

So now we have two options for placing an image: either a Markdown image or a Hugo shortcode. Both seem to have certain limitations and right now I'm not seeing a possible way to rectify this.
If you use a Hugo shortcode, then these photos aren't picked up on the Photos page, because Hugo doesn't recognize them.
If you use a Markdown insertion, then the photo will be shown on the Photos page. But because of the way Markdown syntax processing works, you can't have a div with an img-gallery class right before. Markdown expects an empty line between the previous text and the images and creates an empty  there that ruins the layout.
I'd be happy to contribute a solution but at the moment I'm not even sure if there's a clean way.

Yup-- I don't know the solution right now. If I had one, I'd be putting it into place. Happy for collaborators or ideas.

So now we have two options for placing an image: either a Markdown image or a Hugo shortcode. Both seem to have certain limitations and right now I'm not seeing a possible way to rectify this.
If you use a Hugo shortcode, then these photos aren't picked up on the Photos page, because Hugo doesn't recognize them.
If you use a Markdown insertion, then the photo will be shown on the Photos page. But because of the way Markdown syntax processing works, you can't have a div with an img-gallery class right before. Markdown expects an empty line between the previous text and the images and creates an empty
there that ruins the layout.
I'd be happy to contribute a solution but at the moment I'm not even sure if there's a clean way.

If you use a div and enter new lines between everything, the galleries work. For example:
<div class="img-gallery">

![](urltoimg1.jpg)

![](urltoimg2.jpg)

</div>

Will render properly like the shortcode would have.


Add UMD wrapper

Closes #208.
Used return exports wrapper from umdjs/umd.

Do you think just the UMD wrapper is fine, or should I use something like fid-umd or browserify instead?

I have no idea how to test this. Any advice?

yeah maybe browserify would be better so we don't touch the main code. if you wanna keep it this way, i'd rather have the UMD stuff be at the bottom of the file

Personally, I think those who want to use this module in an AMD environment should use browserify as part of their own process... just my $.02 here.

@tracker1 :+1:

@bucaran I think it's mainly to bridge the awkward transition phase from amd to commonjs. Fwiw I agree that people should just use browserify instead of wrapping co in umd.

@bucaran The only reason I could think of is if someone wanted to pull this module in via bower, then a UMD wrapper around it might make sense.  Though I have similar opinions on bower, and choose to "just use npm" for my modules.

@tracker1 I see. What a baggage :)


Add nested.index_select_along_batch/seq

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

Project coverage is 100.00%. Comparing base (e902825) to head (1ed6ffb).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #66   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           37        37
  Lines          801       805    +4
  Branches        56        56
=========================================
+ Hits           801       805    +4

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


build go dev container from source

i've already pushed this to iron/go:dev cuz we sadly need it to work for our internal tests and this is probably easier than rewriting those, this will be useful cuz alpine community go wasn't even up to date, now we can just install latest from source every time. sidebar, it's a total pain keeping this up to date, we should play with something more official

@rdallman do we need to autobuild this image after commit?

We could switch to official images now that they use Alpine.


Increment version for spring releases

Increment version for spring releases

Can one of the admins verify this patch?


use buildjet runners

converted this to draft -- it depends on fixes to upload-module action when running on buildjet
tracking here https://viam.atlassian.net/browse/RSDK-5379

testing close/reopen flow to clear stale CI labels


[Feature Request] Run on startup

Great app! Wonderful work. It would be nice if the program had an option to start when Windows boot up.
The method I am using now is adding the app's shortcut to the Windows Startup folder but the problem is that the settings window shows up every startup and  I have to hit the "Run" button every time. It would be nice if the program had an option to startup without having to show the settings window. It like logging in to the current weather. Thanks and keep up the good work!

I added a command line flag in the new 1.8 release that you can use to auto start it, like this:
"Chameleon.exe -a"

Thanks. Nice update. I got it working now.
It would be nice though, I know I’m picky, but if the settings windows doesn’t show at all. At each startup, Chameleon does run automatically and changes my wallpaper but the settings window is minimized.


Fleet Rolling Update doesn't seem to be rolling

What happened:
When updating a Fleet, it doesn't seem to do a rolling update, even when using a Rolling Update strategy. It seems like it deletes the entire old GameServerSet, then creates a new one at the original size.
What you expected to happen:
We should get a gradual rollout of shutting down a set of gameservers, while spinning up another set. Then it should wait for that to complete before moving to the next set.
How to reproduce it (as minimally and precisely as possible):
root@7551c789cd4f:/go/src/agones.dev/agones# kubectl apply -f ./examples/simple-udp/fleet.yaml
fleet.stable.agones.dev/simple-udp created
root@7551c789cd4f:/go/src/agones.dev/agones# kubectl scale fleet simple-udp --replicas=50
fleet.stable.agones.dev/simple-udp scaled
root@7551c789cd4f:/go/src/agones.dev/agones# kubectl get flt
NAME         SCHEDULING   DESIRED   CURRENT   ALLOCATED   READY     AGE
simple-udp   Packed       50        50        0           50        27s
root@7551c789cd4f:/go/src/agones.dev/agones#
root@7551c789cd4f:/go/src/agones.dev/agones# kubectl edit flt simple-udp

# change the container port to a different value, that will force an update

root@7551c789cd4f:/go/src/agones.dev/agones# kubectl get gs | grep -v Shut | wc -l
51

You can see that 50 gets created straight away. If you kubectl get gs you can also see all the other gameservers are moved to shutdown.
Anything else we need to know?:
Not sure if it shows anything, but this is what the graph looks like

There is one interesting thing. if we kubectl describe the fleet, the events look correct. Maybe it's moving too fast?
Doesn't look like we are capturing the rolling nature in an e2e test - just the final result (that everything has changed).
root@7551c789cd4f:/go/src/agones.dev/agones# kubectl describe fleet simple-udp
Name:         simple-udp
Namespace:    default
Labels:       <none>
Annotations:  kubectl.kubernetes.io/last-applied-configuration={"apiVersion":"stable.agones.dev/v1alpha1","kind":"Fleet","metadata":{"annotations":{},"name":"simple-udp","namespace":"default"},"spec":{"replicas":2,...
              stable.agones.dev/sdk-version=0.10.0
API Version:  stable.agones.dev/v1alpha1
Kind:         Fleet
Metadata:
  Creation Timestamp:  2019-05-29T22:38:32Z
  Generation:          3
  Resource Version:    4150979
  Self Link:           /apis/stable.agones.dev/v1alpha1/namespaces/default/fleets/simple-udp
  UID:                 7c9ec43a-8262-11e9-83d0-42010aa80085
Spec:
  Replicas:    50
  Scheduling:  Packed
  Strategy:
    Rolling Update:
      Max Surge:        25%
      Max Unavailable:  25%
    Type:               RollingUpdate
  Template:
    Metadata:
      Creation Timestamp:  <nil>
    Spec:
      Health:
      Ports:
        Container Port:  8000
        Name:            default
      Template:
        Metadata:
          Creation Timestamp:  <nil>
        Spec:
          Containers:
            Image:  gcr.io/agones-images/udp-server:0.9
            Name:   simple-udp
            Resources:
              Limits:
                Cpu:     20m
                Memory:  64Mi
              Requests:
                Cpu:     20m
                Memory:  64Mi
Status:
  Allocated Replicas:  0
  Ready Replicas:      50
  Replicas:            50
Events:
  Type    Reason                 Age   From              Message
  ----    ------                 ----  ----              -------
  Normal  CreatingGameServerSet  5m    fleet-controller  Created GameServerSet simple-udp-vx4qx
  Normal  ScalingGameServerSet   5m    fleet-controller  Scaling active GameServerSet simple-udp-vx4qx from 2 to 50
  Normal  ScalingGameServerSet   4m    fleet-controller  Scaling inactive GameServerSet simple-udp-vx4qx from 50 to 37
  Normal  CreatingGameServerSet  4m    fleet-controller  Created GameServerSet simple-udp-m96x4
  Normal  ScalingGameServerSet   4m    fleet-controller  Scaling inactive GameServerSet simple-udp-vx4qx from 37 to 24
  Normal  ScalingGameServerSet   4m    fleet-controller  Scaling active GameServerSet simple-udp-m96x4 from 13 to 26
  Normal  ScalingGameServerSet   4m    fleet-controller  Scaling inactive GameServerSet simple-udp-vx4qx from 24 to 11
  Normal  ScalingGameServerSet   4m    fleet-controller  Scaling active GameServerSet simple-udp-m96x4 from 26 to 39
  Normal  ScalingGameServerSet   4m    fleet-controller  Scaling inactive GameServerSet simple-udp-vx4qx from 11 to 0
  Normal  ScalingGameServerSet   4m    fleet-controller  Scaling active GameServerSet simple-udp-m96x4 from 39 to 50
  Normal  DeletingGameServerSet  4m    fleet-controller  Deleting inactive GameServerSet simple-udp-vx4qx

Environment:

Agones version: 0.10.0
Kubernetes version (use kubectl version): 1.11.9
Cloud provider or hardware configuration: GKE
Install method (yaml/helm): helm
Troubleshooting guide log(s): N/A
Others:

I think we should create empty new GS Set first and then it would be scaled in parallel with rest shutting down using this function applyDeploymentStrategy. So Spec.Replicas should be 0 to start with.
https://github.com/GoogleCloudPlatform/agones/blob/e6d91905ccb400b534aca70d2927836ca4043cee/pkg/fleets/controller.go#L306

Which.. I think is what happens? Or at least, is what is supposed to.

Any update on this?

Hello @lftakakura I am wokring on a fix. Will send draft PR in a moment.


wasapi: Implement S16NE-format streams

Brings the WASAPI backend up to parity with the other backends, making it easier to write cross-platform code for S16 audio. I tried to base my approach off of the AudioUnit backend (in fact, auto_array_wrapper is now in cubeb_utils and used by both) and a little bit from the Jack backend (the bytes_per_sample member).
The official documentation is (purposefully?) obtuse as to when S16 is a supported format, but it seems to hint that it's always available in non-exclusive  mode, and the rest of the internet seems to believe that that's true.

Thanks!


[chip-tool] Tracking paired device's data model state

Reproduction steps / Feature
Hello,
I would like to know if it is possible to leverage the chip::app::DataModel::Provider interface to create a provider that tracks the data model of the device the chip-tool is paired to. I need to track the data model metadata tree of the remote device (logical nodes, endpoints, clusters and attributes) to analyze it in my application.
I see the CodegenDataModelProvider is used in the InteractionModelEngine and, as of my knowledge, it uses the generated ZAP data to get information about the available clusters, so I assume it provides the data model of the chip-tool client, which I don't need.
Can this provider be adapted to behave as I want? Should an implementation of a remote provider follow the Provider interface or use a different approach, for example just sending Descriptor commands and storing the results from the callbacks?
May someone introduce me to how these providers actually work?
Thank you in advance.
Platform
core (please add to version below)
Platform Version(s)
No response
Type
Manually tested with SDK
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response

@andy31415

The general intent of the provider interface is to be used server-side as a backend for the interaction model....  You could try using it for something on the client side, but it's not really designed around that.


Setting default filename in web client files_upload causes AttributeError for file being IOBase instance

https://github.com/slackapi/python-slackclient/blob/033fb44f64d5c5f5664a8b6752adf3e4beba7854/slack/web/client.py#L1496
When the input parameter file is type IOBase, the following attribute error is raised.
Traceback (most recent call last):
  File "/afs/ThisCell/data/vlsi/eclipz/ct6/verif/p10d2/pscripts/user_modules/change_stats.py", line 456, in <module>
    rc = main()
  File "/afs/ThisCell/data/vlsi/eclipz/ct6/verif/p10d2/pscripts/user_modules/change_stats.py", line 356, in main
    plotter(df)
  File "/afs/ThisCell/data/vlsi/eclipz/ct6/verif/p10d2/pscripts/user_modules/change_stats.py", line 286, in plotter
    initial_comment='')
  File "/afs/ThisCell/func/vlsi/eclipz/ct6/verif/p10d1/.pscripts/pong/user_modules/site-packages/slack/web/client.py", line 1305, in files_upload
    kwargs["filename"] = file.split(os.path.sep)[-1]
AttributeError: '_io.BufferedReader' object has no attribute 'split'

Since the Slack API files_upload does not require a filename to upload, I do not believe this method should either.
If this is not acceptable, then at the very least files_upload should check for filename in kwarkgs to be True when isinstance(file, IOBase) == True. If the former is false, when the latter is true, it should raise a SlackRequestError as it does in the following examples.
https://github.com/slackapi/python-slackclient/blob/033fb44f64d5c5f5664a8b6752adf3e4beba7854/slack/web/client.py#L1487
https://github.com/slackapi/python-slackclient/blob/033fb44f64d5c5f5664a8b6752adf3e4beba7854/slack/web/client.py#L1489

@csaska Thank you very much for taking the time to flag this (and fix it). This is an obvious bug.


Error of type in duration variable on parseRoad

Hi, I found an error in your package, when I pass the coordinates for the getRoad() an error is throw on line 41 and 43 of utils.dart, the duration variable get a value double but the type is int and throws the error type 'double' is not a subtype of type 'int' in type cast .

[√] Flutter (Channel stable, 2.8.0, on Microsoft Windows [versÃ£o 10.0.19042.1348], locale pt-BR)
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)

check the new version 0.1.1

Your update solved the problem, thank you!

Hi @liodali , I am implemanting trip service but I have a problem with type of variable.
E/flutter ( 5881): [ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: Exception: type 'int' is not a subtype of type 'double' in type cast
E/flutter ( 5881): #0      new Road.fromOSRMJson
package:routing_client_dart/…/models/road.dart:43
E/flutter ( 5881): #1      parseTrip
package:routing_client_dart/…/utilities/utils.dart:88
E/flutter ( 5881): #2      _IsolateConfiguration.apply
package:flutter/…/foundation/_isolates_io.dart:83
E/flutter ( 5881): #3      _spawn.<anonymous closure>
package:flutter/…/foundation/_isolates_io.dart:90
E/flutter ( 5881): #4      _spawn.<anonymous closure>
package:flutter/…/foundation/_isolates_io.dart:89
E/flutter ( 5881): #5      Timeline.timeSync (dart:developer/timeline.dart:163:22)
E/flutter ( 5881): #6      _spawn
package:flutter/…/foundation/_isolates_io.dart:87
E/flutter ( 5881): #7      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:300:17)
E/flutter ( 5881): #8      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192:12)
E/flutter ( 5881):

Att.

ok i will fix that

check 0.3.0+1

Your update solved the problem, thank you!

But, how use the road response with drawRoadManually? Because drawRoadManually need a List<GeoPoint> and the getTrip don't return this.

Thanks man, I will test!

Maybe the type of the list that getTrip should receive is not correct, because as the type of the list is LngLat, and GeoPoint's is LatLng, it ends up shuffling the data, changing the order drawnRoadManualy should receive.
But that's my opinion of what might be returning the error.


How to use nested ValidationObserver together with ValidationProvider to get common errors for several inputs?

A simple example - a bank card. There are two inputs - month and year:
<validation-observer v-slot="cardDateObserver" tag="div">
  <validation-provider name="card_month" rules="required|between:1,12" v-slot="{ errors }" tag="div">
    <text-input
      v-model="formCardMonth"
      :error-messages="errors"
      name="card_month"
      type="text">
    </text-input>
  </validation-provider>

  <validation-provider name="card_year" rules="required|between:19,99" v-slot="{ errors }" tag="div">
    <text-input
      v-model="formCardYear"
      :error-messages="errors"
      name="card_year"
      type="text">
    </text-input>
  </validation-provider>

  <pre>{{cardDateObserver.errors}}</pre>
</validation-observer>

In pre I see the following:
{
  "_vee_2": [
    "The Card expiration field must be between 1 and 12"
  ],
  "_vee_3": [
    "The Card expiration field must be between 19 and 99"
  ]
}

And I don’t understand what to do with it. I made nesting to combine errors from two inputs. For this I used the documentation: http://vee-validate.logaretm.com/v2/guide/components/validation-observer.html#nested-observers But there are no examples for combining errors.
Tell me, please, what have I done wrong?

@rafaelrenanpacheco How to use it?
For example, I have a line:
<validation-observer v-slot="cardDateObserver" tag="div">

I add the ref attribute:
<validation-observer ref="testObserver" v-slot="cardDateObserver" tag="div">

Further, when I want to start using $refs inside the template or inside computed, I get undefined.