Getting started is a nightmare for new users.

FYI: Breaking changes from RC1 to RC2
I really feel like the wider vnext project needs someone to own a global 'Getting started with RC1' manual - perhaps on [docs.aspnet.org], that is directed to from all current documentation areas/packages.
I know a lot is going on with the move to dotnet, but I have literally spent hours and hours and hours trying to make sense of everything. From what I've seen of the milestones, RC2 is months away and therefore this would probably be worthwhile.
It took far too long to understand (as a brand new vnext user as of a week ago) that:

RC1-final is best to work with right now
because of huge (good) changes across the board undergoing in RC2
because RC2 seems months away (which is fine)
It's easiest to stick with dnx451 or dnx46 native frameworks right now
AspNetCore.* dependencies use a new namespace for RC2+, and I should be using AspNet.* packages instead
Packages are managed most easily in Visual Studio via the context menu 'Manage packages'
Adding the non-standard URL to the package manager in VS was required to get RC2 packages
Look at code samples in master branches are probably for RC2, better to refer to release* or RC1-* branches
RC2 packages need a custom package provider setting up
dotnet replaces dnx, dnvm etc
dotnet is essential for RC2+
dotnet is not essential for RC1, can use dnx etc at the time (not clear from package repos)
dnx46,dnx451 are .NET Native frameworks
dnxcore50 is RC1* name for .NET Core
project.json schemas documented had slight differences between RC1, RC2
The official documentation website shows 'vnext' as the default, which seems to be RC2 related.
Using dnxcore50 as a framework doesn't seem to work easily because it needs ".NET Native" libraries - how do I make it use these?

I could go on. Instead of making stuff, I've been running around in circles. I know vNext is new, but RC1 should be easier to get going with.
What would clear this up:

Explain how and why RC1 is what most people should use right now
Getting started with RC1
Explain who should use RC2 nightlies (contributors, dark souls players, etc)
Getting started with RC2

If RC2 was ready tomorrow, I wouldn't bother with this post, but it seems months away. Which is fine because the changes are great and clarify so much with this project.

It's a waste of time until RC2 is out. I'm not going near it until then

@simonmurdock  The point of this issue is that it took me hours and hours of frustrating research to even understand that different errors/terminologies/documentation/syntaxes were all down to big changes in RC2.
I was very close to giving up completely until I had a small breakthrough.
The current state of affairs is confusing for new users and therefore damaging to the project's goals. I think this issue needs to be escalated.

I agree. Not everything is well documented yet but for me it's ok.
This is just my opinion but... it's still a release candidate. Either they spend a ton of time on documentation for things that are going to change and by the time RC2 comes throw 20-50% of the documentation out anyway.
RC1, as I see it, is for those who want to get the new toys before everybody else. You aren't getting a packaged device. You are essentially getting an IKEA package of your device. Sometimes there is instructions, sometimes there isn't. If you are left a few screw at the end, it might be normal.
If you are learning RC1 right now, learn it for the concepts that are going to drive the RC2/RTM in the end. Some things in there aren't going to change.

Core is hosted Out-Of-Process. It's not going back inside IIS.
Middleware is the way your pipeline is built and managed.
Dependency Injection is the way everything should be built.
There will be a command line scenario (whether it's call dnx or dotnet)
There's not going to be another web.config. Everything is in .json files now.
Things that were Windows-only aren't going to find themselves in .NET Core unless there is a cross-platform alternative and even then... if the team has the time.

So yeah... I know it's frustrating but it's even more hard for those of us who've been looking at this project since they had k, kre, etc.
This project has basically been renamed twice.

Runtime:  K to DNX to dotnet.
Framework: vNext, ASP.NET 5, ASP.NET Core 1

So here's what I might suggest.
Stay on RC1 for now. Don't get stuck in the RC2 build. Things will get broken often and what work one day might not work the next. To avoid any moniker (dnxcore50, dnx451, etc.) confusion, just recreate a project with the updated Web Tooling/yeoman once the new bits are released and copy/ paste over your business code over. There is no upgrade scenarios since we are literally in beta-ish software right now.
If you are looking for what broke, add a watch to aspnet/Announcements and you will be aware of all breaking changes.
I don't know what else to add. We are not 1.0.0 yet. We are still in pre-release.

@MaximRouiller I don't think they should spend more time documenting at this point - just a little time updating signposts with more contextual information regarding the huge/breaking changes with RC2.
If I found such information, it would have saved me hours.

@simonmurdock And that's ok. Especially if you are not of the adventurous kind.
I would recommend however to look at the core concepts of the framework.
Install RC1, get to know what makes this release tick and get to make a simple WebAPI. Add a few packages and understand what's going on.
I've been playing with this project since K. Things are light years ahead of what they were 2 years ago. I've been warning people of the changes. Things will have moved. It's time today to get to know what makes this framework what it is.
It's a fundamental change. Things will break. If you are not ready yet, it's OK. But please try it out.
It got some wonderful ideas and the team need all the feedback we can give them.

Also, I want to add this post is not really about my wasted time, rather it's about the typical issues new users face.

For those wondering, here are the Breaking changes between RC1 -> RC2
You just have to change the milestones to get more info. :smiley:

@MaximRouiller Yes I did see that, I updated the OP with its inclusion.

I think that the ASP.NET Community stand ups have explained this pretty well. Of course you need to know they exist, but that is the same with any documentation.
With this Pre-release stuff you just have to know that you are gonna use some/alot of time reading and tinkering.

In you situation it would make sense to wait for rc2
rc1 unfortunately was a bit premature (especially given it had a go live license) and as you pointed out there are too many breaking changes. rc1 -> rc2 to make it a release candidate.
These are however documented on the announcements repo and at least MS is allowing change in order to get it right in the long run.

Maybe calling them release candidates in the first place is a bad, RC1 should be beta9 and RC2 should be beta10 and the betas need to continue on for a very long time (minimum one year) so that all the nonsense stuff is shaken out... don't even think about release candidate until it's ACTUALLY a release candidate... it's been way too much misleading and potentially harmful political grandstanding with the way these releases have been named and presented as "almost final" when that's the farthest thing from the truth.

@stgeorge I think it as to do with having a "Go Live" and not being in Beta.
I agree that things are changing way too much for a Release Candidate. But we're there.
Personally, I know a client that is already in production with the RC1 on a SaaS product he is building.

BTW my experience is things are working well.  I also have a site I will probably go live on rc2.
Its just things are still changing but most of that is tooling (dotnet-cli) .
The breaking changes do not require much coding effort to fix more than a find and replace.  The main concepts are good to go IMHO.

Nevermind, I'll write a blog post I think. May as well get involved!

:+1:


NeutralResourcesLanguage Warning on UWP

I'm getting the following warning on a Universal Windows App.

warning MSB3817: The assembly "Microsoft.Framework.OptionsModel.dll" does not have a NeutralResourcesLanguageAttribute on it. To be used in an app package, portable libraries must define a NeutralResourcesLanguageAttribute on their main assembly (ie, the one containing code, not a satellite assembly).

Hmmm... we might want to set that on all our DLLs. We normally have en-US embedded in all our "main" DLLs and use satellite resources only for non-en-US resources, and so the neutral language is always en-US.

:+1: Making it part of KoreBuild would be better than doing it in each of the projects that target dotnet/netcore50.

@bricelam @muratg given aspnet/Universe#246, is there anything actionable to track here on Options or can this bug be now closed?

Fixed in https://github.com/aspnet/Options/commit/9902f77b89390733fb38667bfe62f20eff71216b


fix time bucket issue and fix issue  that check style failed

@pengys5

Changes Unknown when pulling a4c0bb5db4368e29b4afb59ab045e468afb12a19 on ascrutae:feature/debugging into ** on wu-sheng:feature/debugging**.

Changes Unknown when pulling 6d46e85a6fe5cdc96f89ee7ef4d3eee6c42200c2 on ascrutae:feature/debugging into ** on wu-sheng:feature/debugging**.


Version of rocThrust compatible with older versions of ROCm?

I am attempting to run some benchmarks that use thrust in them through the gem5 simulator, which currently only supports ROCm 1.6.x.  While hipify'ing the applications, I need to port the thrust calls to use rocThrust (which I did).  However, after looking through the release tags and the commits for rocThrust, it appears that rocThrust only supports ROCm 2.5+.  Is this correct?  Or is there an (older) commit that has support for older versions of ROCm?
Thanks,
Matt

Hi Matt,
The rocThrust library is only supported on newer rocm releases. I recommend upgrading but it looks like you have no choice. Please try https://github.com/ROCmSoftwarePlatform/thrust. This is an unsupported library that will be archived soon. It may support an older rocm version.

Thanks @saadrahim   Yeah, unfortunately upgrading is not an option here.  Looking at that repo, I do see a branch for ROCm 1.5, so hopefully something there works for ROCm 1.6.  I'll check with them, thanks!
Matt


Error creating a region in the postgres database

While creating a region, I get an error:
SQLSTATE[22P02]: Invalid text representation: 7 ERROR:  invalid input syntax for type bigint: "{{resourceId}}" (SQL: select count(*) as aggregate from "nova_page_manager_regions" where "template" = footer-region and "id" <> {{resourceId}} and "locale" = en)

In Mysql(and mariadb), this sql is executed normally, but in the postgres it does not allow it to be executed due to the fact that "{{resourceId}}" string does not correspond to the bigInt type.

That's weird. Nova is supposed to replace {{resourceId}} with the actual resourceId. What's your Nova version?

3.13.0

Ah yes, I think I got it. Please try out version 3.2.3 and let me know if it works better now. :) Thanks!

Great, everything works! thanks


gotype fails to include other files in the same package

Really love this plugin, but it has one fatal flaw which is that when passing gotype a single filename, it will ignore other files in the same package, so you get undeclared name errors constantly.
It seems to me that you could locate the directory the file is in, run gotype -e <dir>, then filter the errors that it dumps out to only that file.

I don't see any current problem with long import names, so I'm closing this.


Little docs correction

Comma out

Pardon for distraction :smiley_cat:

Thanks! Can you sign the CLA please?


[CalendarDialog] when using disabledTimeline property, disabled days are not shown instead of being disabled

Using disabledTimeline = PAST or FUTURE results in disabled days not being shown in the dialog, which in turn results in the following representation:

Spoiler

Which looks unsightly. This behavior is not customizable.
Proposed solution: add a parameter that would allow disabled days to show up as "greyed-out".
Update: because of the issue above, for months that are lacking any enabled days, the dialog is completely empty.

I agree. I will make them look exactly like individually disabled dates.
Regarding your update: when the past or future is disabled, you are not supposed to be able to navigate to the past or future months and years. How did you manage to do that?

Oof, I will create a new issue for that


ClientOptions base class

All client options should derive from a base client options.  Allows for common client options to be specified.
Common client options:

LoggingOptions
ProxySettings
RetryOptions
PerCallPolicies
PerRetryPolicies
TransportAdapter

okay, I can do this

I see the label Azure.Core. So in my understanding, core sdk is going to provide a base client options and storage sdk is going to inherit from it

Working on it. Expect a PR later


ModuleNotFoundError: No module named 'infer'

File "C:\Retrieval-based-Voice-Conversion-WebUI-main\infer\modules\train\train.py", line 7, in
from infer.lib.train import utils
ModuleNotFoundError: No module named 'infer'

the same file had an error about logger being instantiated before it was imported. I fixed that in my code to get to this next error, that I entered in the above section.

When did this error occur?

I downloaded your code Friday and ran into the bug the first time I tried to one click train which was yesterday.

Try the latest 1006 version.


How to access assets from an .a statically linked library

I am having issues using a Rust statically linked library that has a dependency on txt files.  I can access it fine using raw Dart, but once i try to use iOS in flutter it will not work.  The assets are in the pubspec.yaml, and in the xcode project, but I keep getting an error back from the statically linked file that the directory doesn't exist.  Any help would be appreciated.

The Github issue tracker is for bugs and features requests for the Flutter SDK. Help and support questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community .


Hiding label with fieldViewconfiguration

Hi !
Thanks for your amazing job ! I started using your project since 2 weeks.
I need to custom some fields, and create new one.. So after reading the cookbook, I have a little problem.
I have a new field : Fieldset. I've created a new FieldsetField, + FieldsetView + fieldsetDirective to manage the template. So far, so good ! But I want to hiding the fieldset label (not just the fieldset but all the parents div, to win some width..).
I know It's possible in the field configuration, if the second parameters of template is === true.
But I can't find a way in my situation...
Any ideas ?
Cheers.

You can try calling:
myField.templateIncludesLabel(true)


Upgrade to protostream-4.0.0.Alpha3 and fix test failures

https://issues.jboss.org/browse/ISPN-7183

CI looks good.

Merged, thanks


 If you send other transaction after this even with enough gas, the transaction will get stuck as well.
System information
Semux version: 1.5.0-RC1-021d510
OS & Version: Windows 10
Expected behaviour
Transaction will fail if insufficient gas is supplied, gas will be consumed and will allow you to send other transaction
Actual behaviour
Transaction will be accepted by the TransactionManager
account nonce will increase
no validator will pickup the transaction
if you send another transaction, it will get stuck
account nonce will increase

tried resending the transaction reusing the old nonce to overwrite the low gas but get error invalid Nonce
close and reopen wallet and now getting
09:01:05.917 INFO     SemuxBft         Entered validate: proposal = true, votes = [0, 1] [0, 0] [0, 0] 09:01:05.918 WARN     Block            Transaction result does not match for TransactionResult [code=INVALID_NONCE, returnData=, # logs=0] 09:01:05.918 WARN     SemuxBft         Invalid transactions

Steps to reproduce the behaviour

getPendingTransactions

{
"success" : true,
"message" : "successful operation",
"result" : [ {
"hash" : "0x676fee4fe62a7adc09818a3b02994443fd1f7d28d9d1f6a7717939cf8f2264a6",
"type" : "CALL",
"from" : "0x25291f9f681868ab7f70f0bb58f024e3ee086c01",
"to" : "0x6b8e0e12f51a98c0c4c5990c3876686329785e65",
"value" : "0",
"fee" : "5000000",
"nonce" : "2",
"timestamp" : "1542847211654",
"data" : "0xa9059cbb0000000000000000000000006b8e0e12f51a98c0c4c5990c3876686329785e6500000000000000000000000000000000000000000000000000000000000003e8"
}, {
"hash" : "0x27570c94a895c5105ae50c02debed6a9bead6c095a2ceeebc34c10f3e2240fea",
"type" : "CALL",
"from" : "0x25291f9f681868ab7f70f0bb58f024e3ee086c01",
"to" : "0x25291f9f681868ab7f70f0bb58f024e3ee086c01",
"value" : "0",
"fee" : "5000000",
"nonce" : "3",
"timestamp" : "1542847422855",
"data" : "0x70a082310000000000000000000000006b8e0e12f51a98c0c4c5990c3876686329785e65"
}, {
"hash" : "0x5c714a521da8ab9424a2dbd3887f6943615a8491b3b91cf94aec826690d5f25f",
"type" : "CALL",
"from" : "0x25291f9f681868ab7f70f0bb58f024e3ee086c01",
"to" : "0x25291f9f681868ab7f70f0bb58f024e3ee086c01",
"value" : "0",
"fee" : "5000000",
"nonce" : "4",
"timestamp" : "1542847602367",
"data" : "0x70a082310000000000000000000000006b8e0e12f51a98c0c4c5990c3876686329785e65"
}, {
"hash" : "0x30a72a57d6760f5b0af126b71a10eab7feed470faaa91be264d7ee30d89c97b7",
"type" : "TRANSFER",
"from" : "0x25291f9f681868ab7f70f0bb58f024e3ee086c01",
"to" : "0xe5f0adab7559cc3920ce454bd32410cbf82f96bd",
"value" : "10005000000",
"fee" : "5000000",
"nonce" : "5",
"timestamp" : "1542847729529",
"data" : "0x"
} ]
}

Restarting the wallet cleared the pending transaction but still getting
09:25:49.733 WARN     Block            Transaction result does not match for TransactionResult [code=INVALID_NONCE, returnData=, # logs=0] 09:25:49.733 WARN     SemuxBft         Invalid transactions

deleted the wallet and importing the validator address back to a newly generated shows the same error
transactions are no longer stuck but still shows the Invalid transactions in the terminal

It will clear after a few hours.  Thanks for the report!

Fixed in #113


Offer programmatic API

The main function in src/index.ts or something similar/new could be exposed for programmatic usage. Yet needs attention in terms of API: what should it look like for external usage?

Adding my 2 cents here:
knip seems to be doing a very good job at building the dependency graph of the project.
One other of my tooling problem is to generate human readable (dot, convertible to images) dependency graphs for my workspaces. So far I haven't found a tool which is doing it exactly the way I'd like…
Quoting knip's README:

When building a dependency graph of the project, an abstract syntax tree for each file, and traversing all of this, why not collect the various issues in one go?

Would it also be possible to have a programmatic hook into that graph to use it for other purposes? I'd rather have a single tool build a single dependency graph and use it as source of truth for all my dependency graph needs (whether pruning unused exports or documenting code structure)…

Not in scope, closing this for now.

Thanks @mathieucaroff!
A great example is this GitHub Action: https://github.com/marketplace/actions/knip-reporter

They really should have been called post-processors.

A bit off-topic, but I guess it depends on how you look at it:
knip produces results/data → processor(s) → reporter(s)

The initial idea was that it's processing the data before reporting it. But yeah, one can definitely argue it's processing data after it's assembled.


Fix endpoint_status filter and bug in ad-hoc finding creation

Fix bug where endpoint_status objects are not being created for ad-hoc findings being created from the test level. Also removed endpoint_statuses from all filters as they were not helpful and cluttered things.

[x] Give a meaningful name to your PR, as it may end up being used in the release notes.
[x] Your code is flake8 compliant.
[x] Your code is python 3.6 compliant (specific python >3.6 syntax is currently not accepted).
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Model changes must include the necessary migrations in the dojo/db_migrations folder.
[ ] Add applicable tests to the unit tests.
[x] Add the proper label to categorize your PR.

fixes #3090 #3123

I am still experiencing that issue, despite the merged PR.


Dynamic past and future for neural machine translation

Hi there! I open this issue to suggest this EMNLP'19 paper Dynamic past and future for neural machine translation, which proposes a guided dynamic routing mechanism upon capsule networks to distinguish translated and untranslated contents during translation.
Thx!
zaixiang

Hi! Thanks for recommendation, and I've added the paper to the list~

@minicheshire thank you very much! I feel like it'd better be also put in "coverage constraint" due to its motivation. Sorry for not mentioning that.

Got it! Sorry for late reply. Will address it!


SAK-43730 - T&Q: If instructor grades by question, instructor attachments are missing

https://jira.sakaiproject.org/browse/SAK-43730
Note to reviewers:
The attachments that are rendered when students take a quiz are in /jsf/delivery/item/attachment.jsp.
The attachments that are rendered when viewing the Questions tab when grading is jsf/evaluation/questionScoreItemAttachment.jsp, (included from the questionScore.jsp page).
The former loads attachments from an ItemContentsBean, the latter loads attachments from a PublishedItemData instance. The attachments lists consist of the exact same types, and are rendered in the exact same manor, but the EL expressions pointing to these attachments lists differ.
I wanted to eliminate code duplication, so I used tomahawk's 'aliasBean' tags so that both contexts can use the same attachment rendering code. Then I moved the attachment rendering code into samigo/samigo-app/src/webapp/jsf/shared/itemAttachmentList.jsp. Seems like a logical place for code that's 'shared' between views, but if this isn't ideal, let me know. Also, the aliasBean tags aren't used anywhere else in samigo, but they are used by other tools.

Codacy's issues are about comments. I was matching the format that existed before; but if I should make changes wrt these issues, let me know.


Add and host rustdoc for crates

The UI does not have much code to be documented, but the VM part has enough code and the latter might be used by others as well.
rustdoc is Rust's standard documentation system, which is also compatible with Cargo (i.e. cargo doc).
The documentation should be hosted online, e.g. via GitHub Pages. This URL should then be used in the crate metadata as well as being mentioned in the README.
Ideally the hosted documentation would automatically be updated upon pushes to the Git repository, possibly using Travis CI.

While adding/writing rustdoc itself will most likely take a few commits, I got a plan for the automation part which I'd just take from my robo9k/rust-magic project.
The only tricky part would be obtaining a GitHub token to publish the GitHub Pages, which would need to be generated by the repo owner @jakerr

Generating and publishing rustdoc works after the additional commits ab31d08ba1504bb86fe52ca22b9fd14c4c2b4ab4 and 10a30c258b631f5fee6b5355b4a248408d96e723. The GitHub Pages are live at https://chip8-rust.github.io/chip8-rust/ and redirect to chip8_vm/, as the other crates (empty chip8_ui or dependant crates) are not so much of interest.
The build script is currently using a personal access token of mine to publish to the gh-pages branch. Looking at the repository settings it seems like the Travis CI build is running in the context of my user, otherwise decrypting the accesss token would not have worked anyways.
Overall I assume this means that the Travis build might break if I am removed from the @chip8-rust organization.
Apologies for pushing to master directly. Publishing rustdoc should only happen on Travis builds for this repository (no forks) and in the master branch, so this was the only way to test this. As far as I'm aware there is no way to test Travis configurations without an actual Git commit, so I had to give it a few tries.
The Travis config could be split into shell scripts to

download and compile SDL2
build and publish rustdoc

Since the config is working as-is, I'd consider this to be an optional, cosmetical refactoring for now.
I'll push a few more commits to mention the rustdoc in the README and in chip8_vm's crate metadata.


Fix history button for nodes present in tree with hidden VMs

Last selected hidden VM/Template stored in @sb[@sb[:active_accord]] replaced @hist[:id] in replace_right_cell. So it has to be changed in x_history.
Introduced by https://github.com/ManageIQ/manageiq/pull/12504 .
Effected trees:
Compute -> Infrastructure -> Virtual Machines -> VMs & Templates
Compute -> Clouds -> Instances -> Instances by Provider
Compute -> Clouds -> Instances -> Images by Provider
Before:
If History button was used when on summary page of a VM/Template it never changed right side to correct info.
After:
Shows info corresponding to object selected in History button.
@miq-bot add_label ui, bug
@h-kataria please review. BZ has no blocker flag or euwe/yes so I didn't add blocker and euwe/yes labels. Please add them if they are needed. Thanks.
https://bugzilla.redhat.com/show_bug.cgi?id=1395795

verified fix in UI.

Euwe Backport details:
$ git log -1
commit aa6a3a278363a7536ce573c701d56b2fa4c63378
Author: Harpreet Kataria<EMAIL_ADDRESS>Date:   Fri Nov 18 13:17:00 2016 -0500

    Merge pull request #12738 from ZitaNemeckova/another_fix_history_button

    Fix history button for nodes present in tree with hidden VMs
    (cherry picked from commit 6a49592189d0c213046ca68970899f29930d931c)

    https://bugzilla.redhat.com/show_bug.cgi?id=1395795


Add kwargs to from_loaders

Most of the from_documents functions have more than a few args. It's really nice using the single line loader, so lets pass along kwargs

Sounds good to me.  Should probably remove the embedding arg as well and just move that into the vectorstore_kwargs

Just took a shot at it @hwchase17  - thoughts?

Good call - fixed

ill update this and get it in next release!


Auto run tour at startup

Is it possible to make something like this work?
{   "auto-run-command.rules": [       {           "command": "vsls-contrib.codetour.startTour"       }   ] }

More information, settings.json has this:
{
"auto-run-command.rules": [
{
"command": "vsls-contrib.codetour.startTourByTitle?["1. Astra Getting Started"]"
}
]
}
Error is this:
[Auto Run Command] Condition met but command [vsls-contrib.codetour.startTourByTitle?["1. Astra Getting Started"]] raised an error - [Error: command 'vsls-contrib.codetour.startTourByTitle?["1.' not found]
I've tried the file name, etc.  The tour in question has this:
{
"$schema": "https://aka.ms/codetour-schema",
"title": "1. Astra Getting Started",
"nextTour": "2. Start Building a React app",
"steps": [
{
"title": "Authentication",
"description": "## Authentication\n\n>> npx create-react-app astra-tik-tok; cd astra-tik-tok\n\n>> astra-setup workshops tiktok\n\nVisit Astra and follow the instructions in the console to get your token for the astra-setup step.\n\n"
},
{
"title": "Setup Git",
"description": ">> git init\n\n>> git config --global user.email<EMAIL_ADDRESS>git config --global user.name 'Tik Tok'\n\n>> git add -A\n\n>> git commit -a -m 'Initial commit'\n\n>> git rm -rf yarn.lock src/App.test.js src/reportWebVitals.js src/setupTests.js\n"
}
]
}
So why is it not working?

I'm not familiar with the auto-run command extension, but the name of the command that you want to execute is simply codetour.startTour. So maybe try updating the command name and see if that works?

I tried this:
"vsls-contrib.codetour.startTour?["1. Astra Getting Started"]"
And this:
"codetour.startTour?["1. Astra Getting Started"]"
Neither worked.

I changed it to startTour with no arguments, but now it's just giving me a dropdown at startup and not picking a particular tour (the primary one).

This is mostly working, would just love to know how to describe the tour I want in the call

The codeTour.startTourByTitle command allows you to start a specific tour by title. However, I'm not entirely familiar with how the Auto Run Command extension allows you to specify args. It looks like you provide them as space-seperated values. So you could try codeTour.startTourByTitle \"<title>\"

[Auto Run Command] Condition met but command [codeTour.startTourByTitle "1. Astra Getting Started"] raised an error - [Error: command 'codeTour.startTourByTitle' not found]

Apologies, it's codetour. startTourByTitle. Note that "codetour" is all lowercase.

Closing this as resolved


Dead Link to Image

In https://github.com/tensorflow/lucid/blob/master/notebooks/building-blocks/SemanticDictionary.ipynb there is a dead link in the section "Spritemaps" (https://storage.googleapis.com/lucid-static/building-blocks/sprite_mixed4d_channel.jpeg).
This is what I get instead of the actual image:
<Error>
AccessDenied
<Message>Access denied.</Message>

Anonymous caller does not have storage.objects.get access to lucid-static/building-blocks/sprite_mixed4d_channel.jpeg.

</Error>

I would appreciate a working link. No biggy though.

@SeanPedersen thanks for the report!
Looks like we rearranged the bucket—I have made the bucket access public again and updated the notebook to point at the correct sprite in 2137cdb.
Test the link:

Works like a charm!


How to disable the Virtualmin logo shown during logging in?

When logging in the following logo is displayed:

Is it possible to disable or replace it?

@iliajie this sounds like a theme question ...

Is it possible to disable it?

Yes, you can disable it by using browser's console and entering two following commands:
settings_embed_product_branding_privileged = false;
theme.config.save();

Is it possible to replace it?

Yes, you could place custom brand content (based on the standard content, which can be found in the theme root under images/brand directory) to /etc/webmin/authentic-theme/brand.

Thanks, that made the trick.


Change message when not subscribe to notification (#361)

Summary
I was learning about the test app, fixes #361
Ticket Link

And a test needs to be fixed

Codecov Report
Base: 21.12% // Head: 21.13% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (39677eb) compared to base (74fd3db).
 Consider uploading reports for the commit c532fa5 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #402   +/-   ##
=======================================
  Coverage   21.12%   21.13%
=======================================
  Files          81       81
  Lines        5216     5215    -1
=======================================
  Hits         1102     1102
+ Misses       4000     3999    -1
  Partials      114      114

Impacted Files
Coverage Δ

utils/logger.go
0.00% <0.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Rework clientlib permission checks

The client library sling console plugin now prints for the impersonation user:

informational message what client libraries are unreadable
errors when a client library is readable but one of it's elements is not
errors when a client library references unreadable client libraries or files
errors when something is referenced from a client library but not present in JCR

BTW: there are the following broken references discovered by the checker. I didn't fix that so far, since I'm not sure whether these are unnecessary:
ERROR: can't find element jslibs/bootstrap/3.3.5/js/bootstrap.min.js of resource folder /apps/composum/prototype/themes/solid/clientlib/body/js
ERROR: can't find element jslibs/bootstrap/3.3.5/js/bootstrap.min.js of resource folder /apps/composum/prototype/themes/solid/clientlib/body/js
ERROR: can't find element composum/pages/components/clientlibs/page of resource folder /apps/composum/prototype/pages/documentation/clientlibs/page/css
ERROR: can't find element composum/pages/components/clientlibs/page/body of resource folder /apps/composum/prototype/pages/documentation/clientlibs/page/body/js
ERROR: can't find element fonts/awesome/4.6.1/css/font-awesome.min.css of resource folder /apps/composum/prototype/pages/documentation/clientlibs/page/css/fonts
ERROR: can't find element composum/nodes/browser/js/nodes.js of resource folder /libs/composum/assets/manager/clientlibs/manager/js
ERROR: can't find element /etc/clientlibs/granite/jquery/granite/csrf/source/granite.http.externalize.js of resource folder /libs/composum/nodes/commons/components/clientlibs/base/js/aem-6.1
ERROR: can't find element /etc/clientlibs/granite/jquery/granite/csrf/source/csrf.js of resource folder /libs/composum/nodes/commons/components/clientlibs/base/js/aem-6.1
ERROR: can't find element /libs/clientlibs/granite/jquery/granite/csrf/source/granite.http.externalize.js of resource folder /libs/composum/nodes/console/clientlibs/base/js/aem-6.4
ERROR: can't find element /libs/clientlibs/granite/jquery/granite/csrf/source/csrf.js of resource folder /libs/composum/nodes/console/clientlibs/base/js/aem-6.4
ERROR: can't find element /etc/clientlibs/granite/jquery/granite/csrf/source/granite.http.externalize.js of resource folder /libs/composum/nodes/console/clientlibs/base/js/aem-6.1
ERROR: can't find element /etc/clientlibs/granite/jquery/granite/csrf/source/csrf.js of resource folder /libs/composum/nodes/console/clientlibs/base/js/aem-6.1
ERROR: can't find element fonts/awesome/4.6.3/css/font-awesome.css of resource folder /libs/composum/pages/wiki/clientlibs/page/css/fonts
ERROR: can't find element composum/nodes/console/page/js/console.js of resource folder /libs/composum/nodes/console/clientlibs/codeeditor/js


[Preview 2 Release]Show warning for registry admin password

Proposed changes
Now if users re-use the PVC, the admin password will not take effect.

Types of changes

Bugfix (non-breaking change which fixes an issue)

Linked Issues
https://github.com/rancher/rancher/issues/20797

Further comments

Updated

LGTM


Freeze the name column on the Roster table

Classroom teacher mentioned that scrolling right is awkward on roster. Freeze the name column on the left-hand side.

Resolved by @nate01776 in #818!
Note to self, "resolved by nate in eight-one-eight" rhymes.


Back to top button fix

Revert back to top scroll behaviour (We previously used a flexbox layout that prevented the traditional back to top behaviour, required a some JS, now removed)
Fix back to top pattern alignment and height

Before:

After:

:+1:


Issues with data returned by downloadFile()

I'm not sure if this is a error in the library or if I just need to handle the returned data differently.
When I save what I get returned on an invoice download, I get a PDF that doesn't show me any errors when opened, but the content is just a blank page.
I think with this request you would have to use responseType: 'arraybuffer' because the expected response contains binary data. With this change the request works for me and I successfully get back a working PDF.
Am I doing something wrong here or is it actually due to the missing responseType?

@joeried The suggested responseType: 'arraybuffer' is not a Header or body property that gets forwarded to the API – so it won't change, what the API returns. This is a parameter for Axios, which will treat the returned value differently depending on this setting. I think it depends on what you want to do with the result of the call. In some cases responseType: 'arraybuffer' might be necessary, in other cases it might not. Can you show your code example, what/how you're trying to download/view  the file?

in the latest release your change is implemented, thank you for the remark!


[Hub Generated] Review request for Microsoft.KeyVault to add version stable/2021-11-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
By the end of the month (June 2022). Likely in a week or so.

When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
ETA: 2022-07-01

By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

The version should be todays date @adarce
https://github.com/microsoft/api-guidelines/blob/vNext/azure/Guidelines.md#api-versioning
" DO NOT use the same date when transitioning from a preview API to a GA API. If the preview api-version is '2021-06-04-preview', the GA version of the API must be a date later than 2021-06-04"

@adarce The version should be todays date https://github.com/microsoft/api-guidelines/blob/vNext/azure/Guidelines.md#api-versioning " DO NOT use the same date when transitioning from a preview API to a GA API. If the preview api-version is '2021-06-04-preview', the GA version of the API must be a date later than 2021-06-04"

Thanks for the heads up. I'll abandon this PR and create a new one with a newer API version.


feat: add zammad integration for toggl button

Please remember the Contributing Guidelines :heart:
:star2: What does this PR do?
I request the merge of the zammad integration. Zammad is a helpdesk and support software see  https://zammad.com.
The toggl button is placed in the ticket details in the tool-area at the top right (see screenshot below).
The changes have been linted and the integration has been checked in Chrome and Firefox.

Hey @athanasius-kircher, thank you for the PR!
The button on the looks good.
However, for me, the project selector always returns "Zammad Helpdesk". Is this correct? Would it be possible to change it to the customer name instead?
I also think it would be good to have the mini button appear on the overview list:

Ok I have created a new commit. Changes to the version before and the questions above:

The project string is not taken from the site name anymore, but is a combination of the client name and the organization. The later is only taken if the client is associated to an organization. The project string can be described by the following pseudo code client.fullName + ((organization.exists)?' (' + organization.name + ')'). This pattern resemble the Zammad client name pattern in different lists.
@lise-toggl I have checked the options to integrate the button into the overview list, but can not see a good solution. The problem is, that the different lists are very unspecific and relevant information are missing. The lists use very generic markup (overlapping with lists for users etc.) and do not have very reliable selectors, as it seems. Some ticket list use a special wrapper content_permanent_TicketOverview. But on the other hand ticket-results in a search are quite difficult to distinguish from users for example. And even in the overview list the public ticket id is missing and does not occure in the markup. Therefore I would propose to do not implement the button in the list (currently), because it may lead to more tracking inconsistency due to different identifiers or different list behaviour.

Great, thank you for the update! :)


Add Junit ClassRule "SpannerEmulator"

This ClassRule enables automatic startup and shutdown of the cloud spanner emulator during unit testing.
Updated Readme to indicate the two separate samples.

ping @snehashah16

Hey @nielm - sorry for the slow response. could u please accommodate Knut's comments in a subsequent PR. merging this for now since I missed it for too long.


Add github repository link

Repository links are useful if you want to figure out where a particular crate is hosted. I have wondered the same thing and was able to find this repository, but it would have been easier if I could just have clicked the link on docs.rs.

Thanks!


How to get landmark screan position?

Hi, is it possible to get a landmark screen position in C# script? Thanks

You can use this method.
https://docs.unity3d.com/ScriptReference/ComputeBuffer.GetData.html

Just leaving a code snippet here because it took me a minute to get it to work:
float[] data; data = new float[64]; if(isRight)     holisticPipeline.rightHandVertexBuffer.GetData(data); else     holisticPipeline.leftHandVertexBuffer.GetData(data); var targetImage = isRight ? rightHandTargetImage : leftHandTargetImage; var b = new Vector3(data[0]-0.5f, data[1]-0.5f,targetImage.GetComponent<RectTransform>().position.z); targetImage.GetComponent<RectTransform>().position = b;

I believe this issue is resolved.
I close this issue.


Topic/operational frames

Reworked version of operational_frames :

@jcarpent new version. Is the computation of the jacobian ok ?

:+1:


Failed to launch the browser process! Unable to open X display.

Hello
I'm on Debian 64 bits
~/destreamer$ ./destreamer.sh -i "https://web.microsoftstream.com/video/XXXXXX" -o /home/XXX/XXX/

[WARNING] .token_cache not found.

Launching headless Chrome to perform the OpenID Connect dance...

[ERROR] Error: Failed to launch the browser process!
[25638:25638:0911/192128.402447:ERROR:browser_main_loop.cc(1512)] Unable to open X display.

TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/master/docs/troubleshooting.md

    at onClose (/home/XXXX/destreamer/node_modules/puppeteer/lib/Launcher.js:750:14)
    at ChildProcess.<anonymous> (/home/XXXXX/destreamer/node_modules/puppeteer/lib/Launcher.js:740:61)
    at ChildProcess.emit (events.js:326:22)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:276:12)

I don't know how to solve this error
Thanks in advance

Are you running a graphical desktop? We spin up a browser window for login so XFCE/Gnome/KDE/whatever has an X Server is a must.

Yeah i'm not running a graphical desktop, only by command line.
Do i have to install a graphical desktop or is there a way to do it with command line like in this project ?

Sorry, no. You need X to run destreamer.


export Mrv{File,Block}IsReaction to python

I didn't notice that this was missing during the initial review

@bjonnh-work @tadhurst-cdd, could one of you please take a look at this small PR?

@ptosco you ok with merging this one now that your suggestions are in?

@ptosco you ok with merging this one now that your suggestions are in?

@greglandrum yes, thanks!


Update UsingAlice.js

Adding language to clarify that projects will not be listed in ALICE until the project data begins to retire (nb the proposed change is based on common feedback from ALICE users).

Thanks Delilah!


Switch to proper Starlark args + docstring

The Ethereum package uses the old style of run args, args = {}, which is very opaque as to what the args are.
Kurtosis now supports proper Starlark arg style in the run function, and with a docstring in a special syntax.
The two benefits of using this normal Starlark style:

the Cloud will autogenerate a user-friendly form for running the package
Users calling the Ethereum package will get a nicer interface (e.g. ethereum.run(plan, some_arg1, some_arg2))

Oops closing as duplicate of #364


support for zsh auto-completion - #983

Apologies for the delay in responding, I've been on vacation.
Can you separate the commit into two please, one for the vendor stuff (the update of cobra) and one with your actual changes.
I note that this changes the syntax of rclone genautocomplete in a backwards incompatible way which is OK since I think it will give a sensible error.

Don't worry about splitting the commit - I'll do it shortly!

I split the commit into 7e93567b18a822ed8b32428b51ff52e8601da570 and f1f7e0e6f9ea6da5be51a272e94869209aefb487 and merged - thank you very much :-)

Ok, thx. I was going to do it, but you were faster.

@bpicode no worries!  Sorry about changing my mind - I thought I'd kept you waiting long enough while I was on vacation.


codecs/mp4sys: fix valgrind issue caught for invalid file

Can you provide a bit more of a description on how it's fixed? I see a ton of tag stuff in there.

These changes are due to two corrupted video files.
1- By default we hit line desc = lsmash_malloc_zero( sizeof(mp4sys_descriptor_t) ); even though desc is not deallocated for invalid/forbidden mp4sys_descriptor_tags, causing a memory leak (caught by valgrind).
2- The additional condition in the if clauses are actually more of an assertion as the remaining operations are only valid and safe if they are true. One of these statements is enforced because of a corrupt file - the input pointer didn't have proper header tag, which was triggering some errors in valgrind. The rest are extrapolated from that check to add extra security.

It seems the memory leaks cause in that parent descriptor does not hold allocated unknown/invalid child descriptor in its children list. Any descriptor shall be held by its parent descriptor if any. So, your fix is not appropriate I think.


Pushing events to Elasticsearch

hi @johnSchnake @timothysc
How to push event to es cluster directly？

It creates structured logs that are easily indexed by elastic-search OOTB.


Shoot command animation very imprecise

I was using the shoot command to create weapons long ago, The "bullets" that had origin: went in a straight line
Now i tried using the shoot command again and the animation of the "bullets" was very imprecise. I still could hit the target, but the snowball flew in a different location. the short script i was using is:
"WeaponTest":
type: world
events:
On player right clicks with i@MP5:
- if < player.flag[MP5Speed].is_expired> {
- shoot snowball origin:< player> speed:6 script:MP5Hit spread:0 gravity:0 height:0
- flag player MP5Speed duration:0.2s
- determine fulfilled
}
"MP5Hit":
type: task
context: location
script:
- hurt 5 entities:< def[hit_entities]>
I tried to use the push command instead, but i gave the same unprecise animations

Looks like the problems appear when you set the speed over 4.  You can circumvent the speed limit if you make a wormhole and bend the fabric of space time though.

Anthony suggestion sounds solid. Is it possible for you mcmonkey? Otherwise i'll ask Morph ;)
Closing now.


Sattellite view and battery status

Hello!
I want to display among the lat,long,speed and course the battery percentage. Also change the map to sattellite view.
Can you help me?
Thanks!

Traccar Manager is a relatively new project, so it might lack some features. I definitely plan to add more features including the ones you mention.

And is not possible to tell me how I can add those features?

How exactly do you expect me to do it?

I was thinking maybe it's just a code line to insert in some files :)

Here is a line to switch to satellite view:
map.setMapType(GoogleMap.MAP_TYPE_SATELLITE);

To add more information to the marker popup, change MainFragment.formatDetails method.

This is what I wanted to know.
Thank you!


Update readme.md

Not bad, but still left out the countdown list, none of the desktop images show the countdown list.


Using TCK Tested JDK builds of OpenJDK

The AdoptOpenJDK has been discontinued since July 2021. When using Zulu you get all the latest updated (TCK Tested) builds for all versions of OpenJDK.
Signed-off-by: Carl Dea<EMAIL_ADDRESS>

I don’t mind switching, however Temurin doesn’t support some of the JDK versions that are specified in the build matrix.
Let me know how you want to proceed. One option I could use Zulu for the versions Temurin doesn’t support.

In #42 we overhaul the workflow, including the build matrix. Then, the used Java version will be 8, 11, and 16, for which Temurin binaries exist.
So, you can either commit it here, then I will cherry-pick it. Or you create a new PR targeting the overhaul-gha branch. In both cases you will be credited as the original commit author.

I cherry-picked your commit here:
https://github.com/junit-pioneer/convert-junit4-to-junit5/pull/42/commits/5c1d37e51857216d652dd8f82fd57de9154f633f
Thx!

I cherry-picked your commit here:
5c1d37e
Thx!

Nice! Glad to have helped contribute.


Data dictionary V1.1 Updates

[x] @steph-sing Releases V1.0 data dictionary version to Hadley for publishing (1st week of Dec)
[x] @HadleyKing Run error reports for V1.0
[x] @steph-sing Review results from Error Testing and update V1.1 as needed

https://github.com/FDA-ARGOS/data.argosdb/tree/main/tests/v1.0

Completed V1.1 updates - All HL data sets were updated per V1.1 as part of the V1.41 data release.


Feature identifiers

Has there been discussion around including an id_column or something similar in the file metadata?  I think it would assist in round-tripping features from other formats if it were known which column represented the feature identifier.
It looks like GDAL has a FID layer creation option.  But I'm assuming that the information about which column was used when writing would be lost when reading from the parquet file (@rouault would need to confirm).
I grant that this doesn't feel "geo" specific, and there may be existing conventions in Parquet that would be appropriate.

GDAL currently uses an extension gdal:schema metadata domain where it puts information such as the fid column name or GDAL specific typing.
e.g:
{"fid":"fid","columns":{"AREA":{"type":"Real"},"EAS_ID":{"type":"Integer64"},"PRFEDEA":{"type":"String","width":16}}}

Interesting. Seems like it might be good for GeoParquet to at least make a recommendation for reliable FID. Perhaps just use what Pandas does, and then GeoPandas and GDAL could align?

I'll note that pandas uses a list of fields for index_columns, to support its MultiIndex (I think similar to a composite key in some flavors of SQL). It looks like GDAL uses a string, at least in Even's example.

It looks like GDAL uses a string, at least in Even's example.

yes, GDAL only supports a single numeric column as feature identifier

Perhaps just use what Pandas does, and then GeoPandas and GDAL could align?

I'd be hesitant to mimic the Pandas metadata exactly because it's very Python-specific, at least the pandas_type and numpy_type.
One other thing to note is that pandas stores its metadata in two places: once in the file metadata under the pandas key, but again in the file metadata within the Arrow schema metadata. Arrow also encodes its schema in a file-level metadata key named 'ARROW:schema' and is a base64 encoding of a custom binary format (I forget exactly where this layout is defined, @jorisvandenbossche maybe would know?) so that it's possible to exactly round trip Arrow data through Parquet without losing metadata like for extension types.
# continuing from above
meta.metadata[b'ARROW:schema']
# b'/////6gCAAAQAAAAAAAKAA4ABgAFAAgACgAAAAABBAAQAAAAAAAKAAwAAAAEAAgACgAAAAQCAAAEAAAAAQAAAAwAAAAIAAwABAAIAAgAAADcAQAABAAAAM0BAAB7ImluZGV4X2NvbHVtbnMiOiBbImEiXSwgImNvbHVtbl9pbmRleGVzIjogW3sibmFtZSI6IG51bGwsICJmaWVsZF9uYW1lIjogbnVsbCwgInBhbmRhc190eXBlIjogInVuaWNvZGUiLCAibnVtcHlfdHlwZSI6ICJvYmplY3QiLCAibWV0YWRhdGEiOiB7ImVuY29kaW5nIjogIlVURi04In19XSwgImNvbHVtbnMiOiBbeyJuYW1lIjogImIiLCAiZmllbGRfbmFtZSI6ICJiIiwgInBhbmRhc190eXBlIjogInVuaWNvZGUiLCAibnVtcHlfdHlwZSI6ICJvYmplY3QiLCAibWV0YWRhdGEiOiBudWxsfSwgeyJuYW1lIjogImEiLCAiZmllbGRfbmFtZSI6ICJhIiwgInBhbmRhc190eXBlIjogImludDY0IiwgIm51bXB5X3R5cGUiOiAiaW50NjQiLCAibWV0YWRhdGEiOiBudWxsfV0sICJjcmVhdG9yIjogeyJsaWJyYXJ5IjogInB5YXJyb3ciLCAidmVyc2lvbiI6ICI3LjAuMCJ9LCAicGFuZGFzX3ZlcnNpb24iOiAiMS4zLjEifQAAAAYAAABwYW5kYXMAAAIAAABMAAAABAAAAMz///8AAAECEAAAABwAAAAEAAAAAAAAAAEAAABhAAAACAAMAAgABwAIAAAAAAAAAUAAAAAQABQACAAGAAcADAAAABAAEAAAAAAAAQUQAAAAGAAAAAQAAAAAAAAAAQAAAGIAAAAEAAQABAAAAA=='

// Function to read and parse the above buffer
arrow_schema = pq.read_schema('test.parquet')
print(arrow_schema)
# b: string
# a: int64
# -- schema metadata --
# pandas: '{"index_columns": ["a"], "column_indexes": [{"name": null, "fiel' ...

print(arrow_schema.metadata)
# {b'pandas': b'{"index_columns": ["a"], "column_indexes": [{"name": null, "field_name": null, "pandas_type": "unicode", "numpy_type": "object", "metadata": {"encoding": "UTF-8"}}], "columns": [{"name": "b", "field_name": "b", "pandas_type": "unicode", "numpy_type": "object", "metadata": null}, {"name": "a", "field_name": "a", "pandas_type": "int64", "numpy_type": "int64", "metadata": null}], "creator": {"library": "pyarrow", "version": "7.0.0"}, "pandas_version": "1.3.1"}'}

Having metadata in the arrow schema might make it more interoperable than in the pandas-specific metadata, but not usable for non-Arrow-based readers (the Java world I think). If we want to be able to round-trip from e.g. GeoJSON which has feature identifiers, maybe it would make sense to add an option to the geoparquet-specific metadata describing an id column (though we'd have to guard against our metadata being out of sync with other metadata)

GDAL currently uses an extension gdal:schema metadata domain where it puts information such as the fid column name or GDAL specific typing. e.g:

Other than the fid, is there a reason why GDAL can't reuse the Arrow schema metadata, given that GDAL is using the Arrow C++ libraries to read/write Parquet? Do GDAL types not map 1:1 to Arrow types?

Do GDAL types not map 1:1 to Arrow types?

at 99%, but they are subtelties. Like GDAL can have a hint for the maximum width of a string, or JSON or UUID "subtypes" for strings. Those are generally not essential metadata, but GDAL can write them for perfect round-tripping of its abstraction model.

I would personally not follow (or be inspired by) the pandas' metadata here. The information in there is very pandas specific.
If we want to support this, I think a simple new (and optional) field in our geo metadata where you can say which column is the ID column would be sufficient? (similarly to what GDAL does now, i.e. "fid":"fid")

One other thing to note is that pandas stores its metadata in two places: once in the file metadata under the pandas key, but again in the file metadata within the Arrow schema metadata. Arrow also encodes its schema in a file-level metadata key named 'ARROW:schema' and is a base64 encoding of a custom binary format (I forget exactly where this layout is defined, @jorisvandenbossche maybe would know?) so that it's possible to exactly round trip Arrow data through Parquet without losing metadata like for extension types.

@kylebarron yeah, this duplication of the pandas metadata in both the normal Parquet metadata and inside the serialized schema is not super ideal. There is a JIRA about this: https://issues.apache.org/jira/browse/ARROW-14303
The format itself is the IPC message for a schema, and then base64 encoded: https://arrow.apache.org/docs/dev/cpp/parquet.html#serialization-details
But, as you mention yourself, I don't think it is interesting to look at this place to put FID information, as that is not readily available for Parquet readers that are not based on an Arrow library.

If we want to support this, I think a simple new (and optional) field in our geo metadata where you can say which column is the ID column would be sufficient? (similarly to what GDAL does now, i.e. "fid":"fid")

I was envisioning the same.  Though unless space is a concern, I think “id_column” is a bit friendlier and fits well with “primary_column” (nit).

Call 10/24 says we should add some 'best practice' that says parquet doesn't have a primary key, so it's not part of this spec. GDAL should do what it does, if other systems are also interest in roundtripping a feature id then we'd consider it as part of the spec.


Fix legends

Related to:

[x] https://github.com/CartoDB/support/issues/1349
[x] https://github.com/CartoDB/support/issues/1339

Changes

Hide legend title ad header when there is no legend
Fix title injection in legend
Fix legend resize from tab

Acceptance

Resize an embed map with no legends

Create a map with a legend, select it in the tab and resize

(After)

In production! :rocket:


Term request: email address + phone number, postal address

From<EMAIL_ADDRESS>on January 03, 2012 19:23:03
From: https://groups.google.com/group/information-ontology/browse_thread/thread/cb03fa62269dd995 Subject: [IAO] email address
------------------------
From: Chris Stoeckert<EMAIL_ADDRESS>Date: Tue, Jan 3, 2012 at 6:15 PM
To: IAO Discuss<EMAIL_ADDRESS>Hi (and Happy New Year)
I have multiple use cases that require specification on an email address. 'postal address' (IAO_0000422) is actually quite close (defined as: A textual entity that is used as directive to deliver something to a person, or organization) however it is logically defined as 'is about' some 'geographical location' which does not fit.
I propose:

adjusting the textual definition of postal address to reflect its dependency on geographical location.
addition of the term 'email address' with a definition such as " A textual entity that is used as directive to deliver something electronically to a person, or organization." For differentia from 'postal address', we need to indicate what 'email address' is about but I don't think we have something appropriate for an electronic account (not really a device).

apologies if this has been discussed before but hard to search for in the discussion list.
Thanks
Chris
----------
From: Alan Ruttenberg<EMAIL_ADDRESS>Date: Tue, Jan 3, 2012 at 7:20 PM
To: Chris Stoeckert<EMAIL_ADDRESS>Cc: IAO Discuss<EMAIL_ADDRESS>Clearly a mixed message here. The reason that the textual definition
doesn't mention geographical location is that there are postal
addresses that are not geographically fixed, for example addresses on
warships. I would say it is loosely about geographical location in
that eventually it is a material entity that is delivered somewhere
specifically located (to a site, perhaps). So the text should be
adjusted, but with a mind to not confuse postal address with names for
geographic location such as longitude and latitude coordinates.
We probably should include something about email addresses used as
intended, and for other uses. Legitimate uses are as identifiers for
persons and organizations, such as when used as a login, and to record
provenance, as when used in the "From:" field. Illegitimate uses are,
for example, spoofing From: addresses by spammers. I think that the
definition of email address should make reference to the protocols
that define delivery, something along the lines of delivering
information to a mailbox by certain protocols. The protocols are well
documented on the wikipedia page: http://en.wikipedia.org/wiki/Email_address In addition, since there is a standard for email address URIs, these
should be used for instances. I.e. the URI for my gmail box would be<EMAIL_ADDRESS>and we would then have assertions
<<EMAIL_ADDRESS>> instanceOf(i.e. rdf:type) <iao:email
address> (whatever the id is)
etc. Protege lets you explicitly set the URI of a term.
I'm going to add an issue once I send this email, and copy the
conversation (or leave pointers) there.
If you agree with the strategy and want to have a go at a definition,
please add it to the issue thread?
Original issue: http://code.google.com/p/information-artifact-ontology/issues/detail?id=130

From<EMAIL_ADDRESS>on January 03, 2012 16:46:55
Added provision ID http://purl.obolibrary.org/obo/IAO_0000429 and editor note pointing to this thread.

From<EMAIL_ADDRESS>on January 03, 2012 18:59:03
I had not realized that there is such a mechanism as creating addresses with '+' on the fly. Given that, I agree with you completely.

Bjoern

On Tue, Jan 3, 2012 at 7:56 PM, Bjoern Peters<EMAIL_ADDRESS>wrote:
> I thought email addresses are 'staggered CRIDs', with DNS servers
>  registering each domain (e.g. gmail.com), and in each domain there is a
> second CRID mechanism that deals with the part before the @.
There is a mechanism, but it is not necessarily CRID based - the gmail
addresses that you can make up on the fly by adding + being
an example. Or do you not agree that those are counterexample?
> I would consider 'made up addresses' to be outside of the scope. Made up
> postal addresses should also be, otherwise what 'geographic location' would
> they be about?
For mail addresses I agree. For email addresses I'm not so sure. If
they are defined in terms of how they are manipulated by protocols,
then email addresses with valid domains but invalid parts before the
"@" are an in-between case. The protocol for delivering them (SMTP)
defines the behavior of mail servers even when the mail arrives and
there is no mailbox. Various actions can be configured such as
responding with an error message, forwarding, or specialized handling
such as in the case of the "+" addresses.
I've taken the CRID case to mean there is a single database of some
kind that has an entry for each valid identifier.
There's no question that email addresses have a somewhat organized way
to be created, but my gut would be define another kind of identifier
creation process that matches how email addresses (and similar
identifiers) work.
-Alan
I'm having some doubts about the "about geographic location" axiom on
postal addresses. Since about is in the eye of the creator and because
creators would not know geographic location in some cases (the ships)
it seems like it can't be universal.
>
> - Bjoern
>
> ________________________________
>
> On Tue, Jan 3, 2012 at 7:23 PM, Bjoern Peters<EMAIL_ADDRESS>wrote:
>> (and I would argue more specifically CRIDs, but that is not completely
>> true
>> for postal addresses)
>
> Interesting question about CRIDs. Some points to consider:
>
> - There isn't any single registry for email addresses. They are
> allocated by providers, people who run smtp servers, and users that
> are enabled to add email aliases.
> - Consider google email addresses, e.g.
><EMAIL_ADDRESS>Anyone can make an email address
> with arbitrary "+" additions and they will all be delivered to the
> email address without the "+". There is no registration necessary for
> these addresses.
> - We need to decide whether email addresses that are made up (e.g. by
> spammers), but syntactically correct, are IAO email addresses. This
> adds a further wrinkle.
>
> -Alan
>
> ps. I added provisional id http://purl.obolibrary.org/obo/IAO_0000429 > so you can use it in OBI pending getting the definition right.
>
>
--
<EMAIL_ADDRESS>To change settings, visit http://groups.google.com/group/information-ontology

From<EMAIL_ADDRESS>on January 04, 2012 13:29:16
From: Bill Hogan<EMAIL_ADDRESS>Date: Wed, Jan 4, 2012 at 12:49 PM
Subject: Re: [IAO] email address
To: Alan Ruttenberg<EMAIL_ADDRESS>Just to emphasize Alan's point, there is a huge difference between
street address and mailing address for a significant number of
individuals living in rural areas.  See here: http://painintheenglish.com/case/3604 Bill

From<EMAIL_ADDRESS>on January 04, 2012 13:52:57
I agree with Alan about inclusion of protocol in the definition of email address as it obviates the issue of spoofing since SMTP servers don't use header addresses according to the wikipedia link Alan provided.
Proposed definition: A textual entity that is used by Simple Mail Transfer Protocol (SMTP) servers as directive to deliver something to an electronic mailbox. It is about some computer which is used to provide access to the email.

From<EMAIL_ADDRESS>on August 13, 2012 10:02:18
At ICBO, I discussed phone number with Alan Ruttenberg along with email address and postal address as these seem to all be of the same type in some way. They share a communication objective but have different modalities (phone~voice, email~digital, postal address~physical). They are also about some addressable device or devices associated with a person or organization (phone/ voice mail, email server, mailbox).
Are these all "A textual entity that is used as directive to deliver something to a person, or organization" (postal address definition) but delivering different things (voice, email, letters and packages)?
Summary: Term request: email address + phone number, postal address
Cc<EMAIL_ADDRESS>


Sodarace prompt mutation

(merging with main was toooo messy. It's faster to delete and redo the PR)
Add in the prompt mutation component. The goal is to make Sodarace ELM run with what we currently have.

[x]  Add an abstract Model class and separate DiffModel and PromptMutation... as different subclasses.
[x]  Uniformize the json return from sandbox for image task and sodarace.
[x]  Separate out PromptMutationImageOptim  from my image toy task, and uniformize the api for the env of both image task and sodarace.
[x]  Sodarace runs and generates a map via ELM.

Some bigger changes:

Added the Model abstract class over DiffModel. Also added PromptMutationModel, PromptMutationForSodarace since currently we are swapping out diff model by prompt mutation. In the future, we can have other subclasses that talk to Triton inference server. Basically this component is fully in charge of genome creation from a seed.
Added a error_code field in the Sodaracer seed (alongside program_str and result_dict) to indicate the execution result. Added .valid attribute for Sodaracer.
When n_steps >= initsteps, still force MapElites to continue calling random() when the map is empty (this will prevent an error).
Put default seeds (IMAGE_SEED, SQUARE_SEED) together in elm.environment and also combine them into convenient templates image_init_args and sodarace_init_args to instantiate their own environment (see explanations here).

Notes:

In diff_model.py, there are some codes that ultimately belong to the sandbox. Probably won't do anything to those in this current PR, but I think after this, we should have another update that refactors those into sandbox.

To test:
Since we don't have a lot of unit tests yet, the way I test is to run (should be safe without sandbox if we only generate a handful)
cd elm/sandbox/server
export FLASK_APP=index.py
flask run

and the following in another terminal:
python3 run_elm.py run_name=test
python3 run_elm.py --config-name=elm_image_cfg  run_name=test

They seem to run. Feel free to try those in your own computers to make sure they work for different environment.

Looks good, thanks for the PR! I tested it with codegen-350m and it does work, albeit slowly. I will merge now! 🥳


NK3AM fails to register a Credential in the clean state

NK3AM seem to fail to register a Credential in the clean state during a Heads setup operation.
The subsequent attempt works.
From the discussion:

ok seems like if you fail to touch it during the hotp check (a window around 1s) then it is not recognized after that and the OEM reset failed - I suspect this is true for all the hotp checks

ah interesting - it does just happen for the first oem reset - if you do a second one after that it does not occur

this problem only happens, if you do a hotp_verification info before a gpg --card-edit on a fresh key
after the first time gpg --card-edit (without doing anything) it works without the touch

also hotp_verification info does not show the firmware version and the serial;
only after this gpg --card-edit

Touch implementation for the NK3 AM:

https://github.com/Nitrokey/nitrokey-3-firmware/blob/6fe0e242b92a4d58806360a10fe1dda792e7a174/runners/embedded/src/soc_nrf52840/trussed_ui.rs#L96

CCID interface implementation in Nitrokey HOTP Verification Tool:

https://github.com/Nitrokey/nitrokey-hotp-verification/blob/master/src/operations_ccid.c

Secrets App versions on NK3 firmware:

v1.4 -> 0.10
v1.5 -> 0.11

Reproduce with:
a fresh nk3am from produktion, execute
hotp_verification info
it will ask for a touch, after that it will be not responsive anymore even with succesfull touch (1.5) on 1.4 it was only unresponsive if the touch was not there.  to be responsive again it needs a power cricle

I don't think https://github.com/trussed-dev/usbd-ccid/pull/13 would fix the issue. All it does is reset the state when there is an unexpected input rather than panicking. It still would not properly deal with the badly formatted input.

Yes - I was referring to the crash inducing behavior (which is a direct cause of the problem).

The following is not crashing the firmware, but is one of the causes of the further malfunction:
_msg = (&str) "Reset\n"
_msg = (&str) "Trussed backend error: FilesystemReadFailure (line 1233)\n"
_msg = (&str) "OTP command execution error: UnspecifiedNonpersistentExecutionError\n"

which connects to:
https://github.com/Nitrokey/trussed-secrets-app/blob/a95b8ed534c3c7bbaa9a5a4c07ccbb83fcc66492/src/authenticator.rs#L1226-L1236

I took a look at it. The error is simply that the timeout is too short. For some reason the device does not respond within 1s, which leads to the libusb_bulk_transfer call in ccid_receive function to fail. This leaves the device in a Processing state without being cleaned up, so the next commands error.
Increasing the timeout fixes the issue.
This also explains why some circumstances are faster than others, I guess that the first calls after factory reset require re-building a default state, which explains why the timeout is only reached in that case and not when the device is operating properly.

Thanks for checking. I've noticed that as well, and increased the timeout to 3 seconds, making it work reliably again. I do not understand though, why the first call in the session always timeouts.
I am perplexed by the cause of the panic. Should not the outgoing buffer be cleared, if the message was not received, and another request comes?
What about https://github.com/Nitrokey/trussed-secrets-app/issues/88#issuecomment-1594567227 @sosthene-nitrokey ?

I do not understand though, why the first call in the session always timeouts.

To copy what was said in private: my initial theory was that it's rebuilding/storing the state.

I am perplexed by the cause of the panic. Should not the outgoing buffer be cleared, if the message was not received, and another request comes?

Yes see https://github.com/trussed-dev/usbd-ccid/pull/13

The following is not crashing the firmware, but is one of the causes of the further malfunction:
_msg = (&str) "Reset\n"
_msg = (&str) "Trussed backend error: FilesystemReadFailure (line 1233)\n"
_msg = (&str) "OTP command execution error: UnspecifiedNonpersistentExecutionError\n"

which connects to:
https://github.com/Nitrokey/trussed-secrets-app/blob/a95b8ed534c3c7bbaa9a5a4c07ccbb83fcc66492/src/authenticator.rs#L1226-L1236
cc @sosthene-nitrokey Is this worth investigating? Can we recover from this?

Yes, it looks concerning. Not sure what can cause read failures. It feels like something that should never happen

Re: Filesystem corruption
Maybe the error is not actual filesystem corruption but rather an incomplete state? A non-existent file being read would lead to this error.

The file was claimed to be existing actually:
trussed-auth-53c577462e9e97b3/d704b04/src/backend/data.rs
pub(crate) fn get_app_salt<S: Filestore, R: CryptoRng + RngCore>(
    fs: &mut S,
    rng: &mut R,
    location: Location,
) -> Result<Salt, Error> {
    if !fs.exists(&app_salt_path(), location) {
        create_app_salt(fs, rng, location)
    } else {
        load_app_salt(fs, location)
    }
}

Just FYI, timeout value increase is not the sole fix for this one I am afraid @sosthene-nitrokey :
~/w/n/cmake-build-debug (master|✚1) $ ninja; ./hotp_verification set GEZDGNBVGY3TQOJQGEZDGNBVGY3TQOJQ "" 2
>&1 | grep Time
ninja: no work to do.
*** Time for ccid_send: 0
*** Time for ccid_receive: 4000
*** Time for ccid_send: 0
*** Time for ccid_receive: 25
*** Time for ccid_send: 0
*** Time for ccid_receive: 2
*** Time for ccid_send: 0
*** Time for ccid_receive: 336
~/w/n/cmake-build-debug (master|✚1) $ ninja; ./hotp_verification set GEZDGNBVGY3TQOJQGEZDGNBVGY3TQOJQ "" 2
>&1 | grep Time
ninja: no work to do.
*** Time for ccid_send: 0
*** Time for ccid_receive: 4000
*** Time for ccid_send: 0
*** Time for ccid_receive: 4000
*** Time for ccid_send: 1
*** Time for ccid_receive: 360
~/w/n/cmake-build-debug (master|✚1) $

I do not understand what I am seeing.
Some operations take more than 4 seconds?

Nope. Some operations are timing out regardless of the set timeout value. 4 seconds is a lot. 10 seconds even more.

In the further tests it turned out, that setting timeout to 200ms results in the exact same behavior, that is the very first request is not responded to (timed out), but further requests are working as expected.
Further work: check usb-ccid crate in the NK3 firmware.

Does it reproduce with the usbip runner?

I do not support CCID in my USB/IP setup currently. Do you have any working where I could draw inspiration from?

https://github.com/Nitrokey/opcard-rs/blob/main/examples/usbip.rs#L36
Enable the ccid feature of the pc-usbip-runner and it will work.

Ah, I thought this one does not work. Will check it on the next occasion.

You might get the kernel bug, but we ever only reached it when using pcscd so you should be fine


Kali fails to build chroot - libxreder1 404 not found

this is the command i used to build the chroot
sudo sh crouton -r kali -t xiwi
Get:44 http://http.kali.org/ kali/main x11-utils amd64 7.7~1 [233 kB]
Get:45 http://http.kali.org/ kali/main xclip amd64 0.12+svn84-2 [22.6 kB]
Fetched 29.0 MB in 22s (1281 kB/s)
Failed to fetch http://security.kali.org/kali-security/pool/main/libx/libxrender/libxrender1_0.9.7-1+deb7u1+b1_amd64.deb  404  Not Found
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/kali...

Possible a transient error.  Try again?

rebuild worked.  thanks


Add CLIUtils C++11 command-line parser

I'm interested in package CLI11
Here is official site of package https://cliutils.gitlab.io/CLI11Tutorial/
Official GitHub repository: https://github.com/CLIUtils/CLI11
Used build system: CMake
This package has no dependencies

Already available:

https://docs.hunter.sh/en/latest/packages/pkg/CLI11.html

Thanks, @ruslo - you're absolutely correct.
I thought that it wasn't available because I didn't find it under the "Commandline Tools" section in the package list:
https://docs.hunter.sh/en/latest/packages/commandline.html
It might help others in future if it was added to this list.

It might help others in future if it was added to this list

@nzbart Feel free to send a pull request


Added warning for deprecation methods for xcode 14

Proposed changes
Adding Pragma warning for deprecated methods
Type of change

[ ] Feature work
[x] Bug fix
[ ] Documentation
[ ] Engineering change
[ ] Test
[ ] Logging/Telemetry

Risk

[ ] High – Errors could cause MAJOR regression of many scenarios. (Example: new large features or high level infrastructure changes)
[x] Medium – Errors could cause regression of 1 or more scenarios. (Example: somewhat complex bug fixes, small new features)
[ ] Small – No issues are expected. (Example: Very small bug fixes, string changes, or configuration settings changes)

Additional information

CPP pipeline line are broken due to the same :(


new login method

Previously WW had it's own login page. Now we must click on the login button after visiting the homepage.
There we need to click on a button

Thankfully, it's the first button of the webpage.
Hence, below code will return the button we need.
document.getElementsByTagName("button")[0];

We can click it
document.getElementsByTagName("button")[0].click();

And then click on the login button.

Using this script
let btnList = document.getElementsByTagName("button");

for (let btn of btnlist) {
  if (btn.innerText.toLowerCase() == "log in") {
    btn.click();
  }
}

And so, the final script will become.
document.getElementsByTagName("button")[0].click();
let btnList = document.getElementsByTagName("button");

for (let btn of btnlist) {
  if (btn.innerText.toLowerCase() == "log in") {
    btn.click();
  }
}

We just need to get to the homepage, and run this script in the console. After that the method to enter email and password should be the same.

40c82139ffbf7ce29e3191cea317072974670c73
Done


Bluepy error after apt update && upgrade

I updated and upgraded the RPI and now I get this error after launching the daemon:
Xiaomi Mi Flora Plant Sensor MQTT Client/Daemon
Source: https://github.com/ThomDietrich/miflora-mqtt-daemon

[2021-08-13 11:10:00] Connecting to MQTT broker ...
[2021-08-13 11:10:00] MQTT connection established

Adding sensor to device list and testing connection ...
Name:          "olea"
Internal name: "olea"
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/btlewrap/bluepy.py", line 27, in _func_wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/btlewrap/bluepy.py", line 75, in read_handle
    return self._peripheral.readCharacteristic(handle)
  File "/usr/local/lib/python3.7/dist-packages/bluepy/btle.py", line 529, in readCharacteristic
    self._writeCmd("rd %X\n" % handle)
  File "/usr/local/lib/python3.7/dist-packages/bluepy/btle.py", line 302, in _writeCmd
    raise BTLEInternalError("Helper not started (did you call connect()?)")
bluepy.btle.BTLEInternalError: Helper not started (did you call connect()?)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/opt/miflora-mqtt-daemon/miflora-mqtt-daemon.py", line 213, in <module>
    print('Device name:   "{}"'.format(flora_poller.name()))
  File "/usr/local/lib/python3.7/dist-packages/miflora/miflora_poller.py", line 77, in name
    _HANDLE_READ_NAME
  File "/usr/local/lib/python3.7/dist-packages/btlewrap/bluepy.py", line 33, in _func_wrapper
    raise BluetoothBackendException() from last_error
btlewrap.base.BluetoothBackendException

I'm having the same issue.


IAPP click / draw box FIX

Overview
This PR includes the following proposed change(s):

IAPP point on click fix
IAPP draw box on map fix
Blanked out InvasivesBC Table

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots if applicable.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

/rebase

/rebase


Krayon: background colour flooding line

when a new line is written, write an error space using the default background colour

Posted question to stackoverflow
Write-Host with background colour fills the entire line with background colour when window is resized in powershell

This is a confirmed bug is anythin that depends on conhost: Console doesn't handle colored regions when reflowed


Preprocessing steps for training ELMO

Hello,
I was wondering if the preprocessing steps for training ELMO is specific to ELMO or applies in general to AllenNLP in general?
The part I am referring to are below part in the README.md.

All tokenization/normalization is done before training a model, so both the vocabulary file and training files should include normalized tokens. As the default settings use a fully character based token representation, in general we do not recommend any normalization other then tokenization.```

Just to be clear ..

For using the existing pre-trained models with this library or allennlp, don't apply any pre-processing other then tokenization.
For training your own models, you can use whatever pre-processing you want, as long as you are consistent when both training and predicting.


Build protoc in Travis

This mostly fixes the Travis-CI build

@zhangkun83, would you mind reviewing this? I know it's in a strange repository :)

LGTM

Merged as 3cc6568


Some documentation enhancement and preparation for automated Garden Linux Website

How to categorize this PR?
/kind enhancement
/area documentation/website-automation
/os garden-linux
What this PR does / why we need it:
Preparation for automated documentation extraction for garden-linux website:
Delimiter are always surrounded by <> in the beginning, and </> in the end.
Delimiter always starts with website- followed by the subpage, e.g.
website-main
compare with README.md

/squash


IntegerParameter value should have type int

I would expect that IntegerParameter value would have type integer, but it seems to have type float.
For example
foo=IntegerParameter("some description",optional=True,default=20)

Then in the run function, self.foo appears to have type float.

How did you check that? For me it seems it is an int:
#!/usr/bin/env python3

from mlprocessors import Processor, Input, IntegerParameter, StringParameter, IntegerListParameter
from mlprocessors.registry import registry

class TestProc(Processor):
  p1 = IntegerParameter()
  def run(self):
    print(isinstance(self.p1, int))

registry.register(TestProc)

if __name__=="__main__":
  import sys
  registry.process(sys.argv)

$ ./test_int.py TestProc --p1 1
True

Closing. If you can reproduce the issue, feel free to reopen.


Move more MQTT platforms to config entries

Description:
#16904, but for these new platforms:

binary_sensor
camera
cover
light
sensor
switch
alarm_control_panel

fan and lock need to be dealt with later because their base classes haven't been converted to config entries yet.
Checklist:

[ ] The code change is tested and works locally. It's a rather simple change, and I have tested the sensor platform quite well.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

@OttoWinter After this is merged, MQTT discovery only seems to accept a single device per platform.
Is it a bug, or does the configuration need to be udpated somehow?
Example of error of the 2nd switch (and all subsequent):
2018-09-29 09:58:19 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry configuration.yaml for switch
Traceback (most recent call last):
  File "/mnt/d/development/github/home-assistant_fork/homeassistant/config_entries.py", line 225, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/mnt/d/development/github/home-assistant_fork/homeassistant/components/switch/__init__.py", line 84, in async_setup_entry
    return await hass.data[DOMAIN].async_setup_entry(entry)
  File "/mnt/d/development/github/home-assistant_fork/homeassistant/helpers/entity_component.py", line 108, in async_setup_entry
    raise ValueError('Config entry has already been setup!')
ValueError: Config entry has already been setup!

@emontnemery Hmm, it only happens to platforms that you also have entries in your configuration.yaml for, right?
It seems like the check in entity_component.py when forwarding  a  condig entry is because the component is already loaded as a normal platform through the config file, but not as a config entry. That check probably needs to be fixed in the entity component file.

@ottowinter It's reproduced also with an empty configuration.yaml
Should I open an issue for this with some dumps of mqtt discovery data to reproduce?

@emontnemery Yes that would be good. I can't replicate the issue here.

@OttoWinter Perhaps it is timing related: There's no issue if I manually publish discovery topics, but I can reproduce if the discovery messages are retained when hass starts. Issue here: #16968

Merged PRs should not be used for enhancement discussion or bug reports. If you've found a bug it's ok to make a review with inline comments and link to an issue that reports the bug.
Thanks!


Only add the CoreGameplay module if the module selection is empty

This will allow headless servers to run without CoreGameplay,  but still avoid breaking if there are no modules selected.  For the most part,  the default.cfg will take care of ensuring CoreGameplay gets selected at minimum.  This should still catch the times where there is invalid module names in the config file.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/269/
Hooray Jenkins reported success with all tests good!

Makes sense to me. Merged!


Reapply IOStreamer DDR sampling fix, and also fix QSPI

Please see individual commits.

Temporarily setting this to draft, to see what happens to the smaller PRs first.


mseed crate handles timing conversion incorrectly

The api raw_pack does a wrong conversion of time for starttime. Inside pack_raw it calls util::time_to_nstime(start_time); and expected to return unix timestamp in nano secs, but it  returns seconds instead. Due to this wrong conversion of time miniseed record always provides invalid start time.
Fix: util::time_to_nstime() should call  unix_timestamp_nanos()  instead unix_timestamp.

Thank you for reporting.
Though, the proposed fix doesn't work unix_timestamp_nanos() returns an i128, while libmseed defines the nstime_t as i64.
One of solution which comes to my mind (and should work) is using libmseed's ms_time2nstime() function. I'm going to fix this once I fully understand the details.

Thank you for reporting.
Though, the proposed fix doesn't work unix_timestamp_nanos() returns an i128, while libmseed defines the nstime_t as i64.
One of solution which comes to my mind (and should work) is using libmseed's ms_time2nstime() function. I'm going to fix this once I fully understand the details.

I have seen that internally it casted into i128, I would say casting to i64 is would be good enough.

I'd rather leave it up to libmseed whether casting is good enough or not.

Does e114b82a8ca1745519109fcfe24a71cf52fe7bd4 solve this issue on your side?

Does e114b82 solve this issue on your side?

I have tried it for host system and it works as expected, whereas the cross compliation fails
ERROR
error[E0308]: `?` operator has incompatible types
   --> src/record.rs:286:39
    |
286 |         unsafe { util::nstime_to_time(check_nst(raw::msr3_endtime(self.0) as _)?) }
    |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `i32`, found `i64`
    |
    = note: `?` operator cannot convert from `i64` to `i32`
help: you can convert an `i64` to an `i32` and panic if the converted value doesn't fit
    |
286 |         unsafe { util::nstime_to_time(check_nst(raw::msr3_endtime(self.0) as _)?.try_into().unwrap()) }
    |                                                                                ++++++++++++++++++++

Thank you.
933c27e8d948dddcd85c6398ee691a87c9659dcb should fix the time related compilation issue on ARM.
Do you mind checking once more?

Thank you.
933c27e should fix the time related compilation issue on ARM.
Do you mind checking once more?Thank you.

933c27e should fix the time related compilation issue on ARM.
Do you mind checking once more?
Yeah, I have verified for both host and ARM,  and also the examples.

Great. Thanks again.

Great. Thanks again.

Thanks @damb for the quick fix..much appreciated:)


Splitting worker shutdown into request+join+deinit.

This ensures that all workers (and the poller) have exited and that no
threads are live by the time we go to deinitialize their data structures.
Fixes #8863.

(this supersedes #8872, which explains the issue nicely)

I've now read this code and think it's correct :-) (and i had tried to implement the same idea). I can't find an 'approve' button anymore, presumably because it's already merged.

whoops - not sure how that happened - thought I just set automerge-on-approval >_<


Added reCaptcha verification option on sign up.

Added support for [reCaptcha] (https://www.google.com/recaptcha/intro/index.html), which inhibits bots from creating accounts. The user can configure the following options:
siteKey: The key needed by the reCaptcha widget.
secretKey: The key needed to verify the reCaptcha response on the server.
theme: (OPTIONAL) User can select a light or dark theme for the widget. Default is light.
data_type: (OPTIONAL) User can specify image or audio verification. Default is image.
Both of the keys can optionally be provided in a [Meteor settings file] (http://docs.meteor.com/#/full/meteor_settings), obviating the need to pass the private key in a separate server file.
The guide has been updated with configuration instructions.
See this pull request for an example on how the reCaptcha templates are used.
This solves issue #238.

Hello @theplatapi!
Brilliant work!
I have only a  few concerns about the way you're picking up public/private key.
I'll try to add comments to the code.

What if we pick up keys' value at init enforcing they're presence with a warning in case they were not correctly set?
...including that if on the client you see there's a privateKey you blind it out and print a warning on the client console?
I'd say it should be put here for the server and here for the client.
in case AccountsTemplates.options.showReCaptcha is set:

on the server you check the value AccountsTemplates.options.reCaptcha.secretKey and if not present you try to pick up Meteor.settings.reCaptcha.secretKey otherwise you throw an error.
on the client  you check the value AccountsTemplates.options.reCaptcha.siteKey and if not present you try to pick up Meteor.settings.reCaptcha.siteKey otherwise you throw an error. And also you check that there's no AccountsTemplates.options.reCaptcha.secretKey (and in case you delete it, printing a warning or throwing an error).

This done, for the templates you can simply refer to AccountsTemplates.options.reCaptcha.* being sure you have them.
Does this make any sense?

Makes total sense. Good catch with the first comment; you got my bug! I'll update this with your recommended changes.

I added checks for the keys, seeing if they're present and if the secret key leaked to the client. There are three other things I need help with though.

If the account creation fails, I need to manually reset the reCaptcha form. I can do this by calling grecaptcha.reset(), but I get the console error below. How should I get around this? Destroy and recreate the template? Not everyone has https I presume.

Uncaught SecurityError: Blocked a frame with origin "https://www.google.com" from accessing a frame with origin "http://localhost:3000".  The frame requesting access has a protocol of "https", the frame being accessed has a protocol of "http". Protocols must match.

The reCaptcha form takes like half a second to actually appear. Is this alright? I tried loading the script earlier in client.js [after the key checks pass] (https://github.com/meteor-useraccounts/core/pull/268/files#diff-50cfa59973c04321b5da0c6da0fdf4feR426), but there was no speed improvement.

If the reCaptcha verification fails, it usually gives me a list of reasons. I'm passing this [here] (https://github.com/meteor-useraccounts/core/pull/268/files?diff=unified#diff-8b91d379b444e1568e2a37a2becb73e1R79), but it doesn't appear on the client. Should I pass the error reason into the first parameter?

Let me know what you think of these questions and my recent commits.

Good tip on the SO question. The form now properly resets when there's an error. Only catch is that I had to add the code in three separate places.
I tried reset the reCaptcha widget once, regardless of success or failure, but there's a small delay between the widget resetting and the user being signed in. It looked sloppy. It's either clean code or a clean login interface.
Regarding question 2 in my last comment, the widget loads a bit slow on other sites as well. I saw on ticketmaster that it took like ~1 second to appear.
What do you think about this latest change, or my previous question 3? Anything else I should try adding?

If you say it's now stable and bug free I'd say to merge ;-)
...there will be time for small improvements after a bit of testing I guess.

It's bug free from all of the config permutations I tested. Should I try writing some actual tests, or is that not needed?

...mmm, no need for testing right now
We have no tests at the moment, I'd say we should go for them with v2.0 :'(

Awesome work,
Tnx!

My pleasure! For all of the different style packages, should I fork them and add reCaptcha templates? Or will those package maintainers do it?

I'm gonna change them!
...but if you're keen to do it, I'll happily accept PRs ;-)


64x64 resolution?

I'm re-uploading some Telegram stickers to a service that has a 512 KiB size limit for each gif, and it would be nice to have a 64x64 resolution.

To convert Telegram sticker to GIF with custom resolution, use this tool as follows:
$ node cli.js /home/ed/Downloads/super_pack/ --height 64 --width 64


Drag and drop into the software.

What feature or improvement do you think would benefit Files?
Hi,
I am graphic designer and I continuously need to drag images, videos and pdfs into the software. But 'files' doesnt support it at all which is great kill for me. kindly update the software with drag and drop feature for softwares.
thank you
Requirements
Hi,
I am graphic designer and I continuously need to drag images, videos and pdfs into the software. But 'files' doesnt support it at all which is great kill for me. kindly update the software with drag and drop feature for softwares.
thank you
Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.1928
Comments
Hi,
I am graphic designer and I continuously need to drag images, videos and pdfs into the software. But 'files' doesnt support it at all which is great kill for me. kindly update the software with drag and drop feature for softwares.
thank you

Thanks for the feedback what software are you trying files into drag to.

I use coreldraw.
On Sat, 1 Jul, 2023, 4:28 pm Josh65-2201, @.***> wrote:

Thanks for the feedback what software are you trying files into drag to.
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/12815#issuecomment-1615849285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BA7ANVVFBCOAYWEPCHZCJMDXN77GRANCNFSM6AAAAAAZ2T6GFM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks it may be fix when #11735 is also fixed.

@SobanMub thank you for your feedback, as Josh mentioned, this issue is being tracked in #11735. You can subscribe to that issue to be notified when it's fixed.


Update README.md

Changed README.md for easier convenience

python3.11 -m venv major_env not python -m venv <env_name>. we need a specific version. the export export KMP_DUPLICATE_LIB_OK=TRUE doesn't have proper reasons.

Mine worked in python 3.10
Though i can place note: python version >=3.10