Tooltips overrunning the plot

Similar to #1187, it's tricky to use plots with lots of tooltips. Unlike #1187 I think this case might require some API changes?
For example, maybe there some way to add padding around the plot such that the tooltips would display?
from lets_plot import *
LetsPlot.setup_html()

d = {
    l: [i] for i, l in enumerate(string.ascii_lowercase)
}
ggplot(d, aes(x='a', y='b')) + geom_point(tooltips=layer_tooltips(list(string.ascii_lowercase))) + ggsize(300, 200)

Hi, you can add padding around the plot panel (on top specifically) using the panel_inset parameter in theme(). But in this case you will have to increase the plot vertical size as well:
+ theme(panel_inset=[400, 0, 0]) + ggsize(600, 1000)

Similar effect can be achieve by setting wider y-scale limits or larger "extent".
However, the better solution in my opinion would be to customize the tooltip layout, for example:
value_format_list = [f'{k}=@{k}' for k in list(string.ascii_lowercase)]
format_str = ',\n'.join(', '.join(value_format_list[i:i+4]) for i in range(0, len(value_format_list), 4))
format_str

'a=@a, b=@b, c=@c, d=@d,\ne=@e, f=@f, g=@g, h=@h,\ni=@i, j=@j, k=@k, l=@l,\nm=@m, n=@n, o=@o, p=@p,\nq=@q, r=@r, s=@s, t=@t,\nu=@u, v=@v, w=@w, x=@x,\ny=@y, z=@z'

ggplot(d, aes(x='a', y='b')) + geom_point(tooltips=layer_tooltips().line(format_str))

Ah panel_inset seems great, thanks!
However, it seems to interact weirdly with facet_wrap:
from lets_plot import *

LetsPlot.setup_html()

d = {
    l: [i]*3 for i, l in enumerate(string.ascii_lowercase)

} | {'facet': [0, 1, 2]}
ggplot(d, aes(x='a', y='b')) + geom_point(tooltips=layer_tooltips(list(string.ascii_lowercase)))  + theme(panel_inset=[400, 0, 0]) + ggsize(800, 1500) + facet_wrap("facet")

Some issues I see:

It doesn't seem to add the inset anymore
the y-axis labels seem to be placed incorrectly?

Any idea what's up with that?

'facet': [0, 1, 2] : the series size here should be the sane as in the data (i.e. 26)

Maybe better try scale_y_continuous(limits=..) or scale_y_continuous(expand=..) to have adequate empty space above the point.

expanding the limit doesn't work with this toy example (since I only have one datapoint 😛 ), but would probably be sufficient in real plots.
If #1187 were fixed that would be fine since facets usually have enough space below

expanding the limit doesn't work with this toy example
To make it work you need to make plot larger (taller).
But why don't you try to re-format the tooltip's content to make it more compact?

Yep to be clear that's a good suggestion and i've starting using that, thanks!
my tooltips are still pretty beefy despite that, but it definitely helps.

Issue #1187 was fixed. Hope this resolves this issue as well.


fix(todo-comments): use toggle for Trouble todo commands

Using toggle with Trouble todo so that it can close Trouble with the same keybinding.

Thanks!


[WIP] Display cross link middleware

Display a link to vm that is linked to a middleware server in case such relationship exists.

currently depends on #9296 that should be merged first. hence wip.
@miq-bot add_label ui, enhancement, providers/hawkular, wip

cc @theute @pilhuhn @jpkrohling

A few questions about the "Underlying VM" field:

Would this only be shown if there is a VM being used to run the Middleware software?  What would we show if we are running on a physical machine?
Is the Hostname shown on the left table actually the hostname of this VM/Server?

To me, this is the server that is hosting the middleware software.  We rely on an OS and JVM being present for Hawkular or any Middleware to run, so there must be a system of some time (virtual or physical).  Therefore, I wonder if any of the following might be more accurate:

Host machine
Host server

A few questions about the "Underlying VM" field:
Would this only be shown if there is a VM being used to run the Middleware software? What would we show if we are running on a physical machine?

The relationship is shown only if it exists.
Currently we are supporting only VMs in that context.
in the future - might be a host (bare metal) or a container as well.

Is the Hostname shown on the left table actually the hostname of this VM/Server?

the hostname is something coming from EAP itself and not from a VM as it exists in manageiq.
@pilhuhn do you know what exactly is this hostname representing on the left?

To me, this is the server that is hosting the middleware software. We rely on an OS and JVM being present for Hawkular or any Middleware to run, so there must be a system of some time (virtual or physical). Therefore, I wonder if any of the following might be more accurate:
Host machine
Host server

are the suggestions above regarding the "underlying" part on the right under Relationships section or the hostname field on the left?
Keep in mind that in miq there is certain terminology that is already used (VM = virtual machine, Host = bare metal), so imo we can't deviate too much from that since users are used to it.

@abonas This should be the qualified hostname as WildFly/EAP sees it. Which is (most likely) what hostname(1) reports. We were looking at this this afternoon and for Docker it is the abbreviated container-id.

@abonas This should be the qualified hostname as WildFly/EAP sees it. Which is (most likely) what hostname(1) reports. We were looking at this this afternoon and for Docker it is the abbreviated container-id.

so would that be expected to match the VM's name?

On 22 Jun 2016, at 21:24, abonas wrote:

@abonas This should be the qualified hostname as WildFly/EAP sees it.
Which is (most likely) what hostname(1) reports. We were looking at
this this afternoon and for Docker it is the abbreviated
container-id.

so would that be expected to match the VM's name?

Well. The hostname that the VM sets internally. Not sure how this
corresponds to the view of the VMs host of its guest.

@abonas Ok, if we have standards already in MIQ about the use of "host" then I agree we should follow suit.  The suggestions were towards the "underyling" label.  Maybe we just call it "Virtual Machine" - I'm not sure "underlying" is adding any additional info other than confusion here :)

@abonas Ok, if we have standards already in MIQ about the use of "host" then I agree we should follow suit. The suggestions were towards the "underyling" label. Maybe we just call it "Virtual Machine" - I'm not sure "underlying" is adding any additional info other than confusion here :)

I am OK with removing it, but imo if it's removed, it should be agreed to be removed from containers screen too - it's a very similar functionality there, and it's called "underlying" there. @simon3z @serenamarie125

there's currently no response on aligning containers with removal of "underlying", I'm in favor of proceeding with merging the patch as is with "underlying" to be consistent with other similar ui screen look, and if needed, fixing both places to look the same later.

@miq-bot rm_label wip


style(BookAndNotepad): Replace current svg with new cropped svg

Overview

PR Checklist

[x] Related to designs: https://www.figma.com/file/TjBVb65kukF5TZRqYUn4Se/onboarding?node-id=306-11550
[x] Related to JIRA ticket: DISC-1226
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[x] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Check out the BookAndNotepad under Illustrations in Storybook. You can see its more centered than the one in production which has extra space on the right.

PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.38.2-alpha.0e00e4.0@codecademy/gamut-kit@0.6.331-alpha.0e00e4.0@codecademy/gamut-labs@35.15.4-alpha.0e00e4.0@codecademy/gamut@55.1.9-alpha.0e00e4.0@codecademy/monaco@0.7.35-alpha.0e00e4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://64adbe59ee424e419f3aa718--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.38.2-alpha.97abb4.0@codecademy/gamut-kit@0.6.331-alpha.97abb4.0@codecademy/gamut-labs@35.15.4-alpha.97abb4.0@codecademy/gamut@55.1.9-alpha.97abb4.0@codecademy/monaco@0.7.35-alpha.97abb4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://64adce370d22f04c0a3c4b4f--gamut-preview.netlify.appDeploy Logs


AzureData.getDocument() is failed with an error code "unauthorized"

Query/Question
Calling AzureData.getDocument() is failed with an error code "unauthorized". Could you please advise if  I am missing anything?
Error description:
{"code":"Unauthorized","message":"Required Header authorization is missing. Ensure a valid Authorization token is passed.\r\nActivityId: 8ff03cec-89c1-4692-9d56-fb500c936180, Microsoft.Azure.Documents.Common/2.11.0"}
Why is this not a Bug or a feature Request?
May be a bug

Hi @Raysapps,
I'm not entirely familiar with the older Track 1 library but the first thing that comes to mind is that you might be using the wrong (or malformed) credentials when you call AzureData.configure(). I tested locally by passing a malformed master key and got the following:
{"code":"Unauthorized","message":"The input authorization token can't serve the request. Please check that the expected payload is built as per the protocol, and check the key being used. Server used the following payload to sign: 'get\ndbs\ndbs/TestDatabase\nmon, 26 oct 2020 04:15:57 gmt\n\n'\r\nActivityId: 55ac3be6-e16c-44fb-b137-f4ab7d6756e7, Microsoft.Azure.Documents.Common/2.11.0"}

If you are using any of the other overloads of the cofigure() method make sure to pass the right information.
Additionally, through some trial and error I found that the library might cache some credentials and query results, so I had to uninstall my sample app and install it back again to actually see changes. I'm not entirely sure what are the limits of this, but it wouldn't hurt to try that as well.

Can you provide a sample that reproduces the issue? (Do not include a real credential though please). We know that other customers are using this library successfully so I would suspect that it's a case of a misconfigured credential but it's hard to say without a sample that reproduces the issue.

@bsiegel: I am still getting the error. Here is the sample "https://github.com/Raysapps/AzureDataTest". Please verify and advise.

Thanks for providing the repro, it was very helpful. I'm not able to exactly reproduce the issue you're seeing - I think it's possible that the device is caching a response that may have been returned in the past when no credentials were supplied. Try disabling caching as follows:
        // provide Valid  cosmosDB accountName and accountKey
        val accountKey = "<your-master-account-key>"
        val accountName= "<your-account-name>"

        AzureData.isOfflineDataEnabled = false
        AzureData.configure(this, accountName, accountKey, PermissionMode.All)

When I did this, I was able to submit requests to the service but I ran into an additional problem - on my emulator, the Date header was being mis-formatted by the library. Can you please try the above change and let me know if you receive a different error similar to the following?
{"code":"Unauthorized","message":"The input date header is invalid format. Please pass in RFC 1123 style date format.\r\nActivityId: <some id>, Microsoft.Azure.Documents.Common/2.11.0"}

Thank you. The API call was successful on Windows 10 Emulator and  Samsung Galaxy s8, but failed with unauthorized error on Pixel3 Android Version 11. I haven't got any new errors. Please find the images attached of my test results. Hope these are helpful in providing further advice/suggestions.

Okay, thank you - that confirms what I was seeing too, my emulator was also on Android 11. It seems like there is a bug in the AzureCore library that AzureData depends on, specifically here - this code is mistakenly using the ROOT locale rather than Locale.US to format the Date header, and it seems like on newer versions of Android the ROOT locale no longer contains month names.
Unfortunately, the library is deprecated and we don't have an upgrade path at present for you. My best recommendation would be to either
1.) Check out the Azure.Android branch of this repo and build your own copies of the AzureCore and AzureData libraries locally, applying the fix mentioned above to DateFormat.kt in AzureCore; or
2.) Use the CosmosDB client library for Java in your Android application, which is a much more complete client library but unfortunately it is not designed specifically to work on Android. You can use the example code from this project to help, which would allow you to subscribe to the reactive results from the Android UI thread.

Thank you for the suggestions. I will explore both options and implement in my project.

Closing this issue as it appears to be inactive. If there are further issues with the Track 1 SDK, please file an issue and we'll make a best effort to assist you.


Fix MacOS/BSD specific commands

There's some differences between the Linux and MacOS versions of common commands (echo -e for instance doesn't seem to work), so there needs to be a way to tell nix-watch before build time what variation of command it should use for which system. This should be fairly straight forward since things are working already on Linux, it's just a matter of sorting out how they will work on MacOS.
What isn't working:

[x] Terminal does not clear when -c is passed, but manually running the command clear does. This may be an issue in relation to the argument parser.
[x] Debug logs do not print when --debug is passed (likely due to the -e arg passed to MacOS's version of echo)
[x] MacOS uses a different version than the Linux stat, so unless you run nix develop and acquire stat from the nix store, it fails to update the modification timestamp.
[ ] Making changes to files seems to only trigger a new process once, and without the above debug logging it's difficult to tell why this is. If I had to guess it could have something to do with stat as well since it's a darwin dependency and not linux.

There may be other things not working but this is what I've found so far. Currently testing this on my M1 Macbook Pro (aarch64-darwin) with nix version 2.22

@sezna I've figured out 3 of the 4 issues above, any idea what would be causing it to only recognize that changes were made once? The program still loops but does nothing after the first pass. Works perfectly on Linux, this is just on MacOS.

In the man page for stat on MacOS (the BSD one), there's a -x flag that prints its output in in a more compatible way with Linux's stat. Did you give that a whirl? I haven't dug into your code, but that might be a good starting point.

In the man page for stat on MacOS (the BSD one), there's a -x flag that prints its output in in a more compatible way with Linux's stat. Did you give that a whirl? I haven't dug into your code, but that might be a good starting point.

stat seems to be working as expected, I added the GNU coreutils package version of stat as a dependency. Since this will only ever be used as a dev-tool, and by people using nix it shouldn't be a problem. The same goes for echo, pwd, getopt, and clear.

After many debug statements, I've found that indeed stat is working correctly so it isn't that.
Some things to note:

the last_mod_time variable which is outside of the loop and tracks whether to start another process for some reason doesn't get updated until the second process runs. So order of ops is:

start nix-watch
command runs, sees last_mod_time is 0, updates it (debugs show that it does indeed get updated)
command finishes

save the file to proc the fswatch and start the next iteration, it then shows that the last_mod_time is 0 and updates it again
this time it actually does update it, but then this happens:

the last_mod_time is no longer 0, it's always<PHONE_NUMBER>, always... the stat that's included on the system by default shows the same thing:

shstat -f %m .
1722837808

then from there, it always returns from the function until fswatch detects anther update but the stat time never changes.

I've figured out that MacOS doesn't update the directory modification timestamp if one of its files' timestamp is updated. If I add a touch $WATCH_DIR then it will continue to pick up the new timestamp, but this causes an infinite loop. It also doesn't solve the problem of updating the last_mod_time variable. For some reason that's now always 0 for every process that runs.

One solution that comes to mind is to use some other file to track the last known modification time, since we don't get that from the directory that has files changing in it, and the modification time variable won't update. If we touch some other file and check that for the timestamp that might work.


Result::unwrap() on an Err value: No touchbar device found

Just install Pop!_OS without Nvidia on my 13 inch Macbook Pro 2019 with integrated GPU. All went well. Everything is working after following the instructions here:  https://t2linux.org/ except for the touchbar. So I can´t use ESC, FN-keys etc.
I've installed tiny-dfr with the command: $ sudo apt install tiny-dfr. Then I've restarted the laptop, but... no touchbar. Then I've run 'sudo apt install tiny-dfr' but... no touchbar. Instead I got the message:
thread 'main' panicked at src/main.rs:658:43:
called Result::unwrap() on an Err value: No touchbar device found
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Any help on this?

Unfortunately, i can not provide that much support for intel machines, since i do not have any t2 macbooks. To the first glance, the issue is that the t2 display output can't be found, but i can't offer any further help. You should try asking in the t2 discord for help.

@Ikuyu  Have you found a solution yet?

Unfortunately not. When I rebooted into macOS and then again into popOS I saw the Touch Bar a brief moment, but after logging out/in it disappeared again. Also, when I saw it I got the message: Result::unwrap() no Touch Bar found. Same with Fedora. I removed popOS and installed the T2 Fedora Gnome edition. Now the Touch Bar remains visible, but when I run your Touch Bar app, it keeps giving me the same message.

Same issue on Arch, rebooting w/o booting into macos seems to work most of the time. The difference between working and failing appears to be the existence of /dev/dri/card1. When that device exists everything works, and when it doesnt the touchbar disappears

The bug is in the t2 kernel driver. There is nothing tiny-dfr can do so this issue should be closed.

Any updates?

This is a driver-related bug on t2


Error building mod (dll error)

Description
I am using a Macbook Pro Sierra.
I am trying to build my mod, but I get an error message related to DLL files apparently:
Missing dll files for RonitrocketsMod
Must have either All.dll or both of Windows.dll and Mono.dll
All.dll must not have any references to Microsoft.Xna.Framework or FNA
Windows.dll must reference the windows Terraria.exe and Microsoft.Xna.Framework.dll
Mono.dll must reference a non-windows Terraria.exe and FNA.dll
Steps to Reproduce

I first used the mod skeleton to make a simple mod with one sword.
Building it the first time a different error message came up, which I fixed with noCompile = true
After that I began getting the message above.

Expected behavior
Following Jofairden's tutorial, (the first one) it explained that the mod should build, however I got the first error message, which I fixed, and now I got this one.
Reproduction frequency
This is my first mod so I can't say
Versions

tModLoader v0.10.1
Mods
No mods were active
Thanks for your help.

I don't know what tutorial you are talking about, but since you didn't mention which IDE you are using, I'm guessing you aren't actually building the dll files you need.
That said, remember, this is NOT where you get support for making mods, this issue tracker is only for actual bugs in tModLoader. Come to http://discord.me/tmodloader for modding help.


Eng-574 header; Eng-572 member count; Eng-568 join button; lint fixes

Left padding for active proposal header needs to be adjusted

Right padding on closed proposals needs to be adjusted

@germanurrus , @dbslone , have added changes for comments


How does this project keep up to date with Google?

I noticed Google's roboto repository updates constantly. How do this repo then? Are the fonts files identical with Google's?
https://github.com/google/roboto/commits/master
Thanks.

No this project is not automatically updated from this repository. The readme of the google/roboto repo says:

A subset of an earlier version of Roboto is available from Google Fonts, and can be used as a web font."

So, even if the main repository updates regulary, the webfont part, which is what relevant for this package, has been last updated January 2015. This version has been integrated into this repo with #9.

I didn't look into fonts in google/roboto, not sure what was updated. Is there any plan this repo will update to Google's current fonts?
Relate event http://google-opensource.blogspot.sg/2015/05/roboto-googles-signature-font-is-now.html

No, there is no plan at the moment to keep on track with the general Roboto font. If you can supply a recipe on how to deduce the needed files from google/roboto feel free to create a PR with a script that updates the files. I would be highly interested in that.

@roozbehp ping... what's the updates in Roboto since January please?

Too many to count, is the short answer. The git log is your best guide.

Pages of commits >_<
Do you use this tool in Google? I saw it on you video and it's quite amazing: https://youtu.be/6WxACOHm0_g?t=56

@jiyinyiyong Are there any more questions 'bout this, or can this be closed?


Upgrade libphonenumber to 8.12.1

Simple version bump.

@nashfive thx for merging, could you release a new version to pub?

@ened thanks for the PR! The new version has been released on pub ;)


[mobile] Can't scroll properly

Swiping up/down middle and middle-left doesn't always work.

As far as I can tell scrolling doesn't work at all on iOS.

Oh ok, what's going on is that even though the sidebar is hidden, it's still intercepting pointer events. So setting pointer-events to none while the sidebar is hidden should re-enable scrolling.

@SachaG this is working for me now – what page and device/browser?

Any page on iPhone 5s.


Incorrect code generation when using generic types

I'm trying to create some simple helper functions to aid in finding and creating objects.
    @:generic
    public static function AddSubObject<T:UObject>(parent:AActor, name:String, transient:Bool = false):T
    {
        var object:T = FObjectInitializer.Get().CreateDefaultSubobject(parent, name, transient);
        return object;
    }

...

        var sphere:USphereComponent = AddSubObject("Sphere", false);

Which generates the following code:
  @:glueCppIncludes("Engine.h", "<OPointers.h>", "Object.h", "UObject/NameTypes.h", "<TypeParamGlue.h>")
  @:glueHeaderIncludes("<UEPointer.h>")
  @:glueHeaderCode("static void * CreateDefaultSubobject_AddSubObject_T(unreal::helpers::UEPointer * self, void * Outer, unreal::helpers::UEPointer * SubojectName, bool bTransient);")
  @:glueCppCode("void * __pvt::_hx_unreal::FObjectInitializer_GlueGeneric_obj::CreateDefaultSubobject_AddSubObject_T(unreal::helpers::UEPointer * self, void * Outer, unreal::helpers::UEPointer * SubojectName, bool bTransient) {\n\treturn TypeParamGlue<T *>::ueToHaxe( ( (FObjectInitializer *) (::unreal::helpers::UEPointer::getPointer(self)) )->CreateDefaultSubobject<T>(( (UObject *) Outer ), (*(( (FName *) (::unreal::helpers::UEPointer::getPointer(SubojectName)) ))), bTransient) );\n}")
  @:noCompletion
  @:noUsing
  @:genericInstance
  @:glueCppIncludes("Engine.h", "<OPointers.h>", "Object.h", "UObject/NameTypes.h", "<TypeParamGlue.h>")
  @:glueHeaderIncludes("<UEPointer.h>")
  @:thisConst
  @:typeName
  @:ifFeature("unreal.FObjectInitializer.CreateDefaultSubobject_AddSubObject_T")
  public static function CreateDefaultSubobject_AddSubObject_T(self : unreal.PExternal<unreal.FObjectInitializer>, Outer : unreal.UObject, SubojectName : unreal.FName, bTransient : Bool) : ue4hx.internal.PExternalDef<T> {
    return (unreal.helpers.HaxeHelpers.pointerToDynamic( __pvt._hx_unreal.FObjectInitializer_GlueGeneric.CreateDefaultSubobject_AddSubObject_T(@:privateAccess self.getWrapped().get_raw(), @:privateAccess Outer.getWrapped().rawCast(), @:privateAccess SubojectName.getWrapped().get_raw(), bTransient) ) : ue4hx.internal.PExternalDef<T>);

  }

Which throws the error: Type not found: T.

Unfortuantely, this is a Haxe limitation. If you need this, you're going to use macros, like we did at https://github.com/proletariatgames/unreal.hx/blob/master/Haxe/Static/unreal/TArray.hx#L173 . Just beware that you need to guard any Unreal code with #if !macro on the class you need to do this


getChat Error ETELEGRAM: 400 Bad Request: chat_id is empty

When I use getChat method get this error:
Unhandled rejection Error: ETELEGRAM: 400 Bad Request: chat_id is empty
    at request.then.resp (/root/Documents/saveSampad/SaveSampadBot/node_modules/node-telegram-bot-api/src/telegram.js:280:15)
    at tryCatcher (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/promise.js:512:31)
    at Promise._settlePromise (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/promise.js:569:18)
    at Promise._settlePromise0 (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/promise.js:614:10)
    at Promise._settlePromises (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/promise.js:693:18)
    at Async._drainQueue (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/async.js:133:16)
    at Async._drainQueues (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/async.js:143:10)
    at Immediate.Async.drainQueues (/root/Documents/saveSampad/SaveSampadBot/node_modules/bluebird/js/release/async.js:17:14)
    at runCallback (timers.js:781:20)
    at tryOnImmediate (timers.js:743:5)
    at processImmediate [as _immediateCallback] (timers.js:714:5)

Remember this syntax:
bot.sendMessage(CHAT_ID_HERE, "Message);
So, can you write here your part of code?

I try send img in chanel
get chat id => https://api.telegram.org/bot/getUpdates => "chat":{"id":-1001382271711,
use code
var chatId = -1001382271711;
a.bot.sendDocument(chatId, 'img.png');

And see erorr =>  400 Bad Request: wrong file identifier/HTTP URL specified
Where I do mistake?

@tyt34 Maybe you must specify full file path, otherwise use file_id (best choice).
File_id can be retrieved catching message.document.file_id (or message.photo.file_id) on bot.on('message', ...).
Then sendDocument(chatId, file_id).
Anyway try to use as possibile sendPhoto or sendSticker, etc. sendDocument only if you don't know type of file are you sending.

@sidelux Hi. I use this api to get chat information
bot.getChat(chatId, function(msg) {
  console.log(msg.chat.username)
})

Thanks @sidelux! This worked! But what was problem?

@aliir74 Honestly i don't know, in my bot i use that syntax with getChatMembersCount and it works, so... btw very good ;)

I dont know, but in one place code dont work in other place - work.
Shortly speaking - all goods.
Thx.

can I wrap in a block ty catch?

can I wrap in a block ty catch?

In the end the library use bluebird currently, so internally this method have a try catch. But you can use .then and .catch in your code for catch the error and not display it in the console


fix a bug in bytecode optimization code

Fixes #289
As @ozanh pointed out, this is a bug in Bytecode optimization code. While removing duplicate constants, some indexes were re-mapped 2+ times if duplicate CompiledFunctions exist.
@ozanh @geseq let me know what you guys think.

Looks good @d5

Looks good to me :+1:


csc_matrix fails on unsorted coordinates when using 64 bit indicies

For me the following three function calls work:
csc_matrix(((1.0, 1.0, 1.0), ((0, 1, 2), (0, 0, 0))), shape=(2**31, 1))
coo_matrix(((1.0, 1.0, 1.0), ((0, 2, 1), (0, 0, 0))), shape=(2**31, 1))
csc_matrix(((1.0, 1.0, 1.0), ((0, 2, 1), (0, 0, 0))), shape=(2**31-1, 1))`

But the following fails:
csc_matrix(((1.0, 1.0, 1.0), ((0, 2, 1), (0, 0, 0))), shape=(2**31, 1))

The following line (998) in sparse/compressed.pyc produces a "ValueError: could not convert integer scalar"
fn(M, N, self.indptr, self.indices, self.data)

which is called when running the following in sparse/coo.pyc (line 297):
A.sum_duplicates()

It seems to something with the indices not being sorted.
I tested this on scipy 0.17.0 and python 2.7.9

I can't reproduce this on master. If you can, try updating and check if the same issue occurs on your end.

Check it against master (66e7ff0) and it works now.  Sorting will work in the meantime.


Peer Review Summary - Milestone #3

Hello,
I started reviewing your application. Following are my observations:

Idea is very impressive.

Well structured and clearly defined Wiki page.

Feature Improvement points:

Origin and destination address drop down can be provided so that user doesn't need to type in complete location.

Error loading page when refreshed page results (https://github.com/airavata-courses/Envy/issues/99)

No way to close the travel guide once clicked on it.

Submit button not working (https://github.com/airavata-courses/Envy/issues/100).

Hi Pulkit,
Thank you for the suggestions,
We will work on it by the next milestone.
Thanks,
Alakh


Tag updater is not compatible with GNU sed

From @flatcap on PR #6:

I get an error.
sed: can't read /url/ s/tag => "(.)", :revision => "(.)"/tag => "neomutt-20160416", :revision => "fc12fdf30b1789ee46dbc134dd488ca22f7cb3cd"/: No such file or directory

Gnu sed is happy with -i'' (no space) or just -i, but not -i '' (space).

I think we'll just use -i.bak and then delete the resulting neomutt.rb.bak file.

That does the trick.  It's a shame it was necessary.

Indeed 😞
Thanks for checking.


Is there a missing '}' ?

Is there a missing '}' ?

@Co5mos thank you, yes there was missing }, fixed it now.


Getting a Navbar (top menu) on header?

Is there a way (no just title) to add some top menu links to the header?
thanks
Steve,

+1

+1

Just made this. Cheers, :+1:
http://joedf.github.io/strapdown-topbar


TypeScript type definition ReactNode not compatible with React Native strings.

Description
Note
This report is similar to this existing bug report. The existing bug report is not actionable due to a missing reproducer and the use of an outdated React Native version.
In this bug report, a reproducer is included and the latest version is used.
Additionally, more specific details regarding the issue are provided.
Context
React and React Native come with TypeScript support.
Both the React documentation and the React Native documentation reference npm package @react/types as the package used to provide types.
In addition to the types defined by the package, Typescript itself includes some special, extra JSX type checks. Prior to TypeScript version 5.1, it would ensure that components that you used with JSX syntax such as <MyComponent/> could actually be used as JSX by making sure their return value matched global type JSX.Element. From TypeScript 5.1 onwards, it improved this type check to make sure that the return type matches JSX.ElementType.
@types/react makes sure that JSX.ElementType resolves to a function that returnsReact.ReactNode when type checking functional components.
Clearly, the type React.ReactNode has found its place as an integral part of typed React and React Native.
It is defined as the following type in TypeScript 5.0.4, the most recently supported version :
type ReactNode =
        | ReactElement
        | string
        | number
        | Iterable<ReactNode>
        | ReactPortal
        | boolean
        | null
        | undefined
        | DO_NOT_USE_OR_YOU_WILL_BE_FIRED_EXPERIMENTAL_REACT_NODES[
            keyof DO_NOT_USE_OR_YOU_WILL_BE_FIRED_EXPERIMENTAL_REACT_NODES
        ];

This type matches the list of types belonging to a React Node as listed here in the React Native documentation.
Problem
Conceptually, the ReactNode is anything that can be rendered by React, and this is the same in React in the DOM as in React Native.
But in practice, React Native cannot actually render strings directly. They need to be wrapped with a <Text/> node to be used.
Since the same TypeScript package is used in both React in the DOM as in React Native, this leads to incorrect type checking at compile-time in React Native. By default, the types will assert that components can accept raw strings as children, when really only the Text component can. The result is possible run-time crashes due to broken type safety.
When React Native gets upgraded to support TypeScript 5.1, this will turn into a new issue. Functional components written
as JSX will be type checked by making sure that the function returns a ReactNode, as explained above. That means thay any functional component that returns a string will pass this check, but break at runtime. For instance, this will pass the type check but break:
<View><ComponentThatReturnsString /></View>
Possible resolution
A separate type could be created for React Native that more accurately reflects the types that the components actually support. Specifically, string could be removed from this new type, as string appears to explicitly be for React on the web. This solves the issue pre TS 5.1 where strings can be passed as children, and the issue post TS 5.1 where components can return strings. Next, the Text component should get its own separate type so that it can allow children to include a string as an exception to the rule.
Steps to reproduce

Clone the reproducer app here:
https://github.com/MCOrdina/stringsinrn
install the node modules: npm install
run a typescript check in the project to see that there are no type issues: ./node_modules/.bin/tsc --noEmit
run the project. For instance using npm run start and npm run android
See that the project crashes with the following error:

React Native Version
0.73.4
Affected Platforms
Runtime - Android, Runtime - iOS, Build - MacOS, Build - Windows, Build - Linux
Output of npx react-native info
System:
  OS: macOS 14.3
  CPU: (12) arm64 Apple M2 Pro
  Memory: 121.06 MB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.10.0
    path: ~/.nvm/versions/node/v20.10.0/bin/node
  Yarn: Not Found
  npm:
    version: 10.2.3
    path: ~/.nvm/versions/node/v20.10.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.14.3
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 22.2
      - iOS 16.2
      - macOS 13.1
      - tvOS 16.1
      - watchOS 9.1
  Android SDK: Not Found
IDEs:
  Android Studio: 2022.2 AI-222.4459.24.2221.10121639
  Xcode:
    version: 14.2/14C18
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.9
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.4
    wanted: 0.73.4
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: Not found
  newArchEnabled: false

Stacktrace or Logs
ERROR  Error: Text strings must be rendered within a <Text> component.

This error is located at:
    in RCTView (created by View)
    in View (created by App)
    in App
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in ReproducerApp(RootComponent), js engine: hermes

Reproducer
https://github.com/MCOrdina/stringsinrn
Screenshots and Videos

⚠	Newer Version of React Native is Available!
ℹ	You are on a supported minor version, but it looks like there's a newer patch available - 0.73.4. Please upgrade to the highest patch for your minor or latest and verify if the issue persists (alternatively, create a new project and repro the issue in it). If it does not repro, please let us know so we can close out this issue. This helps us ensure we are looking at issues that still exist in the most recent releases.

Reproduced with 0.73.4

string being included in the ReactNode type feels strange, especially in combination with React Native. Feels like a custom type is in order to prevent misuse of ReactNode

I don't think the tag "Never Patch Available" still applies. This has been reproduced with the latest version.


Revert "Revert "Cleanup run docker machinery in run_tests.py, task_runner.py and elsewhere (#28704)" (#28741)"

Reintroduce #28704.
(fedora34 seems to have no "find" command? /var/local/jenkins/grpc/tools/run_tests/dockerize/docker_run.sh: line 54: find: command not found)

Adhoc run of full python distribtests:
https://fusion2.corp.google.com/invocations/6cc87bb8-eaf7-48fc-86a0-f526b9540fbc/log

This seems to have broken a few jobs on the test fusion dashboard...

This seems to have broken a few jobs on the test fusion dashboard...

Which ones?


cant detached many apps

i just can detach 11 apps
more that get error
log.txt

yes thats the limit. detach.bin cant be larger than 512 bytes. i thought that would be enough for everybody. why are you trying to detach that many apps

it's good if no limit


Service Bus: Custom Endpoint Address Support

Summary
When using an Express Route circuit, there are some configurations which are not able to open the initial connection to the Service Bus service using the fully qualified namespace as the connection endpoint.  In these cases, a custom endpoint address is needed to ensure the correct routing for the connection, but should only be used when first establishing the connection; after that point, the fully qualified namespace is expected by the service for normal operations.
In order to support this scenario, an approach is needed to allow for the endpoint address for connections to be overridden with an explicit value, taking precedence over the default behavior of assuming the fully qualified namespace.
Scope of Work

Implement the design for supporting a custom endpoint address.

Extend the endpoint validation used by the AMQP library to allow for using the specifically configured custom endpoint address without disabling validation or allowing a wider set.

Success Criteria

The design has been implemented and the targeted clients are able to provide a custom endpoint address for connecting to the Service Bus service which is respected by the client library types.

The tests necessary for its validation have been created or adjusted and pass reliably.

The existing test suite continues to produce deterministic results and pass reliably.

Related Issues and References

Design Proposal for Custom Endpoint Address Support (https://gist.github.com/jsquire/f0948c9a5ffe2741b03d312dcf8d899e)
Custom endpoint for connection establishment (#27143)
[Pull Request] Event Hubs Custom Endpoint Support (#18014)

I noticed this is set for the June milestone, are you planning on working on this @jsquire?

I don't have the bandwidth to take this on at the moment.   We're close enough to code complete now that we'll want to push into July.


Update README.md

Added input handling documentation.

Coverage remained the same at 91.083% when pulling e41475d6e5730d2a5865eb567ab9f456029b663f on robotex82:patch-1 into 3841c2ff8db1cf0d660c0407117d9d094d210db3 on piotrmurach:master.

Thanks!


Deprecate bco:0000064

This was the grouping class 'darwin core class', which is no longer needed.

The superclass (entity) was specified as an axiom in obi-bco-imports, so that file changed with this change.


[CWOD V20] Bug Fixes and New Features

This update covers a number of requested features from users as well as bug fixes.
Updates:

Added an expanded roll template that factors in normal rolls, initiative rolls and also rituals being used.
Most multi-button options have been gutted to 1 button at that controls all rolls, this was to simplify the sheet and help keep the sheet clean
RItual template rolls have a scroll bar to show the full description of a ritual as well as a handy page listing feature to source the content
The smaller notes section has been removed in favor of a section that calculates Movement. This was one of the larger requests I received on messages, most players have a hard time tracking movement, now its not a problem!
Combination Disciplines have expanded to include more information
Added some features for more folks who house rule such as enabling options to Backgrounds to roll them and set a difficulty

Fixes:

Fixed the issue for Boosts not being factored into rolls.
Fixed the issue with Initiative rolls not working correctly
Intiative roller has an option to include to the Tracker or not

Github says you have conflicts. Those need to be fixed before any merge can be made.

Thanks for pointing that out, never had that happen before, still a bit of a rookie at using GitHub


Compute FFE based off of Data in Structure inventory

During the creation of the fdai file, the structure inventory is loaded from dbf. Check the inventory if FFE is present. If it is, use it.
There should be attributes from the user that declare if the GE was stored in the structure inventory via an atribute. There should also be an attribute about if the user imported an FFE or a FH, use that boolean also.
https://github.com/HydrologicEngineeringCenter/GeoFDA/blob/bbb1a9de9ff12369afcded5ea4842cf09290e626/GeoFDA/Output/OutputChildTreeNode.vb#L473
The line above needs to be only done if necessary - if the FFE is not previously provided, and if the user did not provide ground elevation on structure import.
https://github.com/HydrologicEngineeringCenter/GeoFDA/blob/bbb1a9de9ff12369afcded5ea4842cf09290e626/GeoFDA/Output/OutputChildTreeNode.vb#L474
this line needs to be fixed to only compute FFE if it was not provided and all of the necessary info exists (FH and some source for GE)
if it is not, check the following basic case logic:

if the user stored ground elevations on the structure inventory use them (attribute is stored with each structure independently, because user's could be modifying the structure attributes post import)
if the user did not store ground elevations or FFE, get ground elevations from the terrain file and add them to foundation heights
if the user does not have GE data or FH data at compute time record an error message.

Closing. This bug was fixed, but I left the issue open because I was going to further optimize the code around it, but other projects have taken priority recently.


Use of eval in "xxx.vue" is strongly discouraged as it poses security risks

Environment

Operating System: Windows_NT
Node Version:     v20.16.0
Nuxt Version:     3.14.159
CLI Version:      3.15.0
Nitro Version:    2.10.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      default
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
if(!/^[0-9a-zA-Z\u4e00-\u9fa5]{3,30}$/.test(dat.name)){
    dat.verify_name_info = 'invalidate name'

    eval(`${op}_name_input.value.focus()`)

    show_toast('error', 'invalidate name')
    return false
}

Describe the bug
I use eval() in my code, and when I build, I get a warning message like this:
Use of eval in "pages/manage/vedio.vue" is strongly discouraged as it poses security risks and may cause issues with minification.

Additional context
No response
Logs

This isn't a bug - it's a valid warning and is coming from rollup - see https://rollupjs.org/troubleshooting/#avoiding-eval.


Fix typos in README.md

I can fix the typos and update the README. md. So, kindly assign me this issue.

Hey @kichloo Thanks for contributing to LoginRadius Open Source.  As part of hacktoberfest, LoginRadius is also giving free swags and goodies to its contributors. Please check out our Hacktoberfest Page and fill the form mentioned there to become eligible for free swags. :)

@anilswm Thanks for the information. And I wanted to know whether I should start working now or after the labelling?

@kichloo You can start work. thank you


feat: Alpha 3

:tada: This PR is included in version 1.1.1 :tada:
The release is available on:

GitHub release
v1.1.1

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.0-alpha-4.1 :tada:
The release is available on:

GitHub release
v1.2.0-alpha-4.1

Your semantic-release bot :package::rocket:


a3_stereo_image_proc benchmark

As follows from #8, the a3_stereo_image_proc benchmark now reuses the image_input_component code from the a1 benchmark. Slight modifications were made to the image_input_component.hpp and image_input_component.cpp files to better accommodate the a3 benchmark. I also created a yaml file instead of a readme file.

@vmayoral Analyzed the data and added the latency to the yaml file!

@jasonjabbour I'll have time tomorrow to review this. Is it ready to be reproduced in other machines? including the analysis script?

The analysis script doesn't terminate properly. Trying to fix it now ...

The analysis script doesn't terminate properly. Trying to fix it now ...

Mmmm, from the output of this, are you sure you're finding any matching sets at all? Meaning, do you complete the chain at least once? Try printing the len of the resulting list (of lists) after the processing routine.
If you're not finding any matching sets at all, then there might be some issues in the processing of the sets you need to debug further.
I'll plan to have a look at this myself tomorrow, so feel free to describe in here your debugging process. I'll build on top of it.

Using babeltrace, a preview of my data looks like this:

and I can identify these tracepoints in my data:
robotperf_benchmarks:robotperf_image_input_cb_init
robotperf_benchmarks:robotperf_image_input_cb_fini

robotperf_benchmarks:robotperf_image_input_cb_init
robotperf_benchmarks:robotperf_image_input_cb_fini

robotperf_benchmarks:robotperf_image_output_cb_init
robotperf_benchmarks:robotperf_image_output_cb_fini

So, I tried to modify the target chain (and the other lists below it)

By printing the image_pipeline_msg_sets list of lists I get the following output and a length of 326:

I think I am finding matching sets:

I believe the error might be an issue within the traces() function

@jasonjabbour been searching for the dataset and couldn't find it anywhere. Any chance you can facilitate it to reproduce results?

@vmayoral Thanks for the help today! I cleaned up some very minor things and I added the max latency of running the benchmark on my Intel i9 CPU. I made a note of the other statistics in the yaml file. I also uploaded the data for this benchmark here: https://drive.google.com/drive/u/0/folders/1Z9JtllJd0X7aVcBZoYZ56WEHRPEbUB_N
I think it should be ready! Let me know if I should change something!

Thanks for this @jasonjabbour, great contribution.
Tested locally (Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz):

Benchmark Mean
Benchmark RMS
Benchmark Max
Benchmark Min
Mean
RMS
Max
Min

RobotPerf benchmark
29.10 ms
29.22 ms
34.94 ms
6.36 ms
43.88 ms
44.33 ms
65.14 ms
10.80 ms

Awesome!! Thanks, Victor!


Lab2

Fixes #286
Review of colleague's PR #
Changes proposed in this PR:
-- Complete Lab 2
--Bubble sort and Insertion sort

Codecov Report

Merging #186 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #186   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      3    +1
  Lines          18     52   +34
=====================================
+ Hits           18     52   +34

Impacted Files
Coverage Δ

01_fib/shahriys/main.go
100% <100%> (ø)

02_bubble/shahriys/main.go
100% <100%> (ø)

Continue to review full report at Codecov.

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

Changes done requested by @eastrd

The go-course is now closed. Thank you very much for participating.


Improvements proposal

Proposals for improvement:

when connected read mac and lock status automatically
add battery percentage with indicator e.g.: going up (charging) or going down (discharging)
add button status, e.g. when you calibrate game pad you need to check what is the result and you need to use e.g Gamepad tester web page
add indicators of 100% for analog sticks, e.g. for up, down, left, right and L2/R2 buttons. sometimes after calibration you cannot reach 100% in all directions, and you cannot reach L2/R2 max value
add pre and post calibration results e.g. as on gamepad tester web page: one color for pre and different one for post calibration. percental deviation is an excellent score result.

Additional test proposal, the ability to set the trigger motors on a DualSense for motor testing. I've had a few come through with cracked/slipping gears that I only caught because I used the motor test in DualSenseX.

Thank you for these suggestions. Just added the first one (automatically read mac and lock status). Will add the others in future.

If possible, there should be a hardware deadzone adjustment tool, it will be useful for BDM-010 and BDM-020 dualsense since these are the models that i had to fix the most due to how easy it is to get stick drift on these models. I'm getting tired of having to recalibrate a lot to get it just right. Permanently writing the changes does not solve the issue, but makes it a lot easier to get the stick range an calibration just right, only to realize that a deadzone adjustment tool is actually necessary and would help combat stick drift.


Comments / descriptions

Add a comment and/or description to ride along with the data model for steps, and pipelines.

Done in c2cbee3d758cfd159e5ca9a18ad3ea1e79247dbb


tabular regression and classification

RNN (LSTM) , CNN

I guess you want a new template for this? Can you describe in more detail what your use case is and which framework you would want?

I guess you want a new template for this? Can you describe in more detail what your use case is and which framework you would want?

tabular data regression

tabular data regression

@atwahsz I'm closing this for now. If you have a more complete explanation of what you want, please feel free to write it here, then I'll re-open :)


Add #dearMoon and Mars BFR missions to future launches.

It would be cool add #dearMoon mission in 2023 to future launches
Since Elon Musk personally confirmed it, it will most likely happened.
There are 2 planned missions to Mars:

cargo mission to Mars in 2022
cargo and crew, is targeted for 2024

Why not add them to future launches too?

We could add them, but outside of the SpaceX website, no source even has it on the manifest, because there's too many unknowns on when it will fly.
If the subreddit wiki manifest doesn't even list it, there's probably too many unknowns at this point to add it.
If it gets added to the wiki, I'd be happy to add it 👍

OK, NP.
I'll add them manually to my graphic then.

FYI, SpacexNow has added these launches to the list:


CASSANDRA-11950: wait for schema agreement

Once PR https://github.com/datastax/python-driver/pull/617 is merged, this change will eliminate the UnknownColumnFamilyException warnings from the logs. You can merge this PR as soon as you would like, but it won't have any effect until the driver change is also available.
I've also fixed some typos and pep warnings.

+1. @mambocab, when should we merge this?

Sorry for the delay. Merging.


Data preprocessing

Thank you for sharing your awesome work.
What preprocessing is required before feeding volumes to the network?

Thanks for your interest in our work. I used Zscore normalization before feeding volumes.
Cheers,
Himashi


Small organizational changes

@jstickelmaier does this pass the tests on your computer? We should merge this soon.

@cyrillustan I updated it with more changes and yes it passes all tests.


add two sample within subject permutation test function to stats module

This should be easy as it is simply the difference between the two inputs and then running through one sample t-test


add 126 (2020-12-02)

I think this is a bit confusing.  The improper preimage revelation problem was already addressed by not revealing local preimages for passthrough payments, which is what the devs thought LND was doing in the first place.  This change would've also prevented the problem by a forbidding the node to release the preimage to anyone who didn't have the payment secret, which is contained in a field that's not available for passthrough payments.
Additionally, I'd prefer (but it's your choice) that we use the name from BOLT4 of "payment secret" rather than LND's pet name of "payment addr(ess)".  I personally find the name "payment address" confusing and I think it has the same problem "Bitcoin address" has, which is that "address" implies it should be reused (like a mailing address), which is even less true of LN payment secrets than it is of Bitcoin output scripts/witness programs.


Cannot get theme to work

So I tried to get the agnoster theme (as in your README example), but the only thing i get is:
.znap.source:shift:4: shift count must be <= $#
.znap.source:.:16: not enough arguments

when I try to source .zshrc or just restart the shell.
Is there something I missed?
Also, how would one go about adding a custom theme?
Thank you

@alexghergh Sorry about the cryptic error message. I've now added more informative ones. Please try the following:
znap clone https://github.com/agnoster/agnoster-zsh-theme.git
znap pull

Then restart your shell.
Does that work?

Right. So I cloned and pulled, but now I get this:
.znap.compile:4: command not found: add-zsh-hook
znap source: not enough arguments

For the reference, my .zshrc only contains:
source ~/.zsh/zsh-snap/znap.zsh

znap prompt agnoster

If I remove the prompt line, I only get the first error message.

@alexghergh Please again znap pull and restart. It should be fixed now. Sorry about the hassle!

Thanky you, worked great!
Also thank you for the cool project, been looking for something lightweight like this for a while!
Closing.


Firefox : new tab opened on far right when duplicating tab

New Issue Checklist (tick off like this: [x])

[x] I checked existing issues (open and closed) for possible duplicates
[x] I can reproduce the problem on the latest stable version (not nightly!)

What is your browser?

[x] Firefox
[ ] Chrome

What is your operating system?

[ ] Linux
[ ] Mac
[x] Windows

Description (please include examples/screenshots where applicable)
Using Firefox 57.0 on Windows 7 with open tab next to current v2.0.10
When I right click on a tab and select duplicate, the new tab is opened on the far right, as all the tabs are before installing the extension.
Can the extension also handle this use case ?

I've got a fix, but it's weird. I think the problem got introduced when fixing #30. I expected duplicate tabs to be opened right next to the original, but never tested it. As you figured, it doesn't. Now the fix does to duplicated tabs, what it does to other tabs, moves them next to the current one. So when I duplicate the current tab, it does what I'd expect. But if I duplicate another tab, the new tab is still moved next to the current, not the original one. This feels weird, but there is nothing that I can do about it. Unless I introduce inconsistent behaviour. What do you think?

Thanks for your answer !
By current tab, I suppose you mean selected tab ?
I would understand if : you have 3 tabs A, B*, C (B* = selected tab), you duplicate A with a right click
=> result : A, B*, A', C.
But if I duplicate B I would expect to have A, B*, B', C
I this the behavior you're describing ?

Yes, this is what I meant and I think it's weird. I'd always expect a duplicated tab to end up next to the original. But if you think it's OK, I can just make the change. I am not using duplicate tab myself. Not even sure whether anybody even duplicates other tabs than the current one.

Totally agree !
And even if the behavior seems weird if it's documented as is it, well, users will say it is how it is ;)
Seriously indeed i agree with you, duplicating a tab certainly happens most of the time on the current/selected tab... And this fix would definitely improve the extension user experience !


change force_aba_ name

In GitLab by @proyan on Aug 27, 2019, 13:42
force_aba sounds misleading. It is difficult to understand what this flag does, without going through the code.
I would suggest changing it to with_armature instead.

In GitLab by @cmastalli on Aug 27, 2019, 14:01
Sounds OK for me! I will handle this soon

In GitLab by @cmastalli on Aug 27, 2019, 14:01
assigned to @cmastalli

In GitLab by @cmastalli on Aug 29, 2019, 15:14
mentioned in merge request !210

In GitLab by @cmastalli on Aug 29, 2019, 16:12
closed


Application Information Reporting

I want to have a REST API that allows me to pull application specific information from RPS (Application Version and Protocol Version) so that I can check version compatibility between RPS and RPC.

implemented in 1.2 release


Print detected stage sizes

As a y2f user,
I would like to make sure y2f detected the right structure.
Notes:
It would be very useful to print the detected stage dimensions before generating code with FORCES so one can quickly check whether something went wrong in the detection. I recently worked with this code to the file optimizerFORCES.m (line 353), which could be extended or made nicer but it does the job:
% print size of variables
    fprintf('Number of variables          : ');
    for n = 1:length(stages{i})
        fprintf('%2d  ', stages{i}(n).dims.n);
    end
    fprintf('\n');

    % print size of equality constraints
    fprintf('Size of equality constraints : ');
    for n = 1:length(stages{i})
        fprintf('%2d  ', stages{i}(n).dims.r);
    end
    fprintf('\n');

    % size of polytopic constraints
    fprintf('Size of polytopic constraints: ');
    for n = 1:length(stages{i})
        fprintf('%2d  ', stages{i}(n).dims.p);
    end
    fprintf('\n');

Solved in v0.1.15


SelfSigned Doc Changes

Quite a large rework of the SelfSigned issuer docs, intended to highlight potential gotchas and clarify language.

I think I've now addressed all the comments that were raised :+1:

/lgtm

/hold

Up to you whether you want to hold this till v1.3 when that event trigger will be true

The final forcepush I just made was to add this change:
diff --git a/content/en/docs/configuration/selfsigned.md b/content/en/docs/configuration/selfsigned.md
index fe6363a..fef8a1a 100644
--- a/content/en/docs/configuration/selfsigned.md
+++ b/content/en/docs/configuration/selfsigned.md
@@ -148,6 +148,6 @@ To avoid this, be sure to set a Subject for `SelfSigned` certs. This can be
 done by setting the `spec.subject` on a cert-manager `Certificate` object
 which will be issued by a `SelfSigned` issuer.

-In any case, cert-manager will emit a Kubernetes [warning event](https://github.com/jetstack/cert-manager/blob/45befd86966c563663d18848943a1066d9681bf8/pkg/controller/certificaterequests/selfsigned/selfsigned.go#L140)
+Starting in version 1.3, cert-manager will emit a Kubernetes [warning event](https://github.com/jetstack/cert-manager/blob/45befd86966c563663d18848943a1066d9681bf8/pkg/controller/certificaterequests/selfsigned/selfsigned.go#L140)
 of type `BadConfig` if it detects that an certificate is being created
 by a `SelfSigned` issuer which has an empty Issuer DN.

/unhold


Added: markup extension

Markup extention to convert from FontAwesomeIcon to string without bindings. Useful for styles and document element such Run, etc.
Example:
<TextBlock>
  Info <Run Text="{fa:FontAwesome InfoCircle}" FontFamily="pack://application:,,,/FontAwesome.WPF;component/#FontAwesome"/>
</TextBlock>

Why not also add:
FontFamily="{fa:FontFamily}"

It's a shame to see such little response from the owner of this repo.

I'm always a bit weary on long strings floating around the place.
Would it be worth creating a second markup extension just for setting the font?

Well it helps to read stuff before posting! @drewnoakes had already posted what I was refering to :) - woops!

Reason for this markup extension was that I'm tired of searching for codes for icons.  Especially when you need to work with 's
With font family there is no such a problem. You can define own style or static resource for it
<FontFamily x:Key="FontAwesomeFamily">pack://application:,,,/FontAwesome.WPF;component/#FontAwesome</FontFamily>
...
<TextBlock FontFamily="{StaticResource FontAwesomeFamily}"

I don't think that worth to create another markup extension. Maybe just define a shared static resource


Strapi and Cloudinary working with gatsby-transformer-cloudinary

Hey Guys! nice job with the plugin!
Unfortunately I can't make this works, I have a CMS with Strapi and Cloudinary as a plugin, when I try to use the
gatsby-transformer-cloudinary, all the Cloudinary existing images are upload again in a cache folder, i wanna try the existing images steps but is not clear to me, could you explain to me a little more how can i do for integrate this!
Thanks in advance @jastuccio @jlengstorf

Hi @jdortegar, I know you posted this about a year ago, but I wonder if you found any solutions. I am in the same situation.
The docs for this plugin say that it should work with images that are already uploaded to Cloudinary:

The coverPhoto property in the node above will be deleted and replaced by gatsby-transformer-cloudinary with a CloudinaryAsset node that can be used with gatsby-image. This transformation will be done for any top-level properties of nodes that have cloudinaryAssetData: true, and values cloudName, publicId, originalHeight, and originalWidth properties

Therefore, I believe the problem is that Strapi does not provide those fields to Gatsby via gatsby-source-strapi.
I guess the solution would be to somehow massage the strapi data to include these fields. But I haven't the foggiest idea how!
Any ideas from the community would be very much appreciated

@tylerpaige @jdortegar did you ever figure this out?

@MrChrisRodriguez thanks for your kind comment. Absolutely I can share my solution. Basically, I added custom transformer plugins to create new nodes that would have the properties that gatsby-transformer-cloudinary needs:
cloudinaryAssetData
cloudName
publicId
originalHeight
originalWidth

I wish I could paste some code, but I've unfortunately lost access to that branch. The Gatsby docs do have a really good tutorial on how to create a transformer plugin though, and that's what I used to write one myself.
Basically, the procedure was like this:

onCreateNode look for Strapi nodes
If the node has any image properties, create a new node for the image. Copy all the original data and then add new fields (listed above)
Create another new node to can replace the image's parent node. Copy all the original data, then add a new property for association to your newly created image node.

E.g. if you have a Product with an image, you would create a ParsedProduct node with a parsedImage field that points to a ParsedImage node
You do this because the original record will still be pointing to the un-modified Image node without the fields you want.
At some point, I had hoped to just use createNodeField to skip all this node creation and just modify the original Strapi Image nodes... I didn't have much luck, but maybe there's a way to get it working!

At this stage, I had no use for the gatsby-source-cloudinary plugin, as I had all the nodes I needed.
For me, though, I found that this wasn't nearly as polished as I wanted, so I decided to handle all of my image resizing in a custom component on the front-end. I read the image URL from Strapi, then construct a list of sources myself, and pass them off to gatsby-image in a format it understands. I don't think this is really the ideal outcome, but my builds are indeed a lot faster now. Here's a gist that has it all together.
At any rate, gatsby-source-strapi was still downloading all of my images for Gatsby's transformation layer, which was causing me a fair amount of bandwidth (hence my PR that you linked). That remains unsolved, but hopefully my PR can be merged soon :-)

Thanks @tylerpaige -- I'll pass this to my team and will circle back if we make any improvements that could be helpful to you or others!

The newest beta supports this by letting you configure the mapping of data needed.
yarn add<EMAIL_ADDRESS>Take a look at the documentation and let me know how it goes so I may improve if needed.

@jdortegar, @MrChrisRodriguez and @tylerpaige have you been able to test the 4.6.0-beta.1? Looking to make this a current version, but would love an external test before I do.


add permissions to SELECT events/exposures data

Currently the permissions outlined only allowed us to create/delete/select in the statsig folder. They need to be able to also select from their folder path to grab the metric source data or what not

#1369  👈
main

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

Join @danielwest-statsig and the rest of your teammates on  Graphite

🚀 Deployed on https://66020630398f2b6571773846--cozy-fox-0defec.netlify.app


Need help with using gpu tuner option of tflite delegate

I am trying to use the gpu tuner option of tflite delegate.
armnn_delegate = load_delegate( library="/home/lzm/Desktop/project/aarch32_build/delegate/libarmnnDelegate.so", options={"backends": "GpuAcc,CpuAcc,CpuRef", "logging-severity":"info","enable-fast-math": "true", "reduce-fp32-to-fp16": "true", "number-of-threads": "8","gpu-tuning-level": "3","gpu-tuning-file": "/home/lzm/Desktop/project/tuning"})
However, it doesn't seem to work. The inference runs without any errors or warnings but no tuning file is created and there is no improvement in the performance. I feel like I am doing something wrong here. Do I need to manually create a tuning file before running?  If yes, does the extension matter and what is it?
I can confirm that the inference is running on the GPU.

Hi,
Sorry for the delay. I have verified that the gpu-tuning-file does get passed down to the GPU backend when using the delegate through execute network. Can you give me some details on the execution environment please. You're using "tf.lite.experimental.load_delegate", what version of TF are you using?
Colm.

@Colm-in-Arm I am using tflite-runtime 2.5 on Odroid XU4 board running Ubuntu Mate 22.04. I am unable to create a GPU tuning file (or atleast I cannot find it if it is being created, not in the path that I am providing) to begin with.

@Colm-in-Arm I am using tflite-runtime 2.5 on Odroid XU4 board running Ubuntu Mate 22.04. I am unable to create a GPU tuning file (or atleast I cannot find it if it is being created, not in the path that I am providing) to begin with. Could you elaborate what argument should I provide to gpu-tuning-file to create the tuning file? Is it the path to the folder or path to the file? And if it is the latter, should I first manually create the file or does ArmNN create it?

Hi,
There are two parameters that control the tuning file"gpu-tuning-file" and "gpu-tuning-level".
gpu-tuning-file refers to the path name of the file we pass to ACL. If this file exists then will use its contents to tune the inference.
gpu-tuning-level tells ACL to do some model tuning. The default value, 0, implies that the file referred to in gpu-tuning-file will be read and its parameters used. Values from 1-3 indicate that a tuning file should be created.
I tested this functionality using our tflite delegate and execute network rather than through the TfLite runtime but it did work.
For example to create an initial tuning file I did:
./ExecuteNetwork -m mobilenet_v2_int8_terminus.tflite -c GpuAcc --tuning-path tuning.txt --tuning-level 1
(Note the CLI parameters do match gpu-tuning-file and gpu-tuning-level. The model was something I choose from the Arm model zoo)
This did result in the creation of a tuning file. This was on an Android hikey but essentially the parameters do get passed to ACL which is what I'm testing.
I could then apply that tuning file with:
./ExecuteNetwork -m mobilenet_v2_int8_terminus.tflite -c GpuAcc --tuning-path tuning.txt
In your case the parameters you're specifying are correct. I would expect a tuning file to be created.
I went on to try the same with the tflite benchmarking_tool
./android_aarch64_benchmark_model --graph=mobilenet_v2_int8_terminus.tflite --use_xnnpack=0 --disable_nnapi_cpu=0  --external_delegate_path=libarmnnDelegate.so --external_delegate_options="backends:GpuAcc;gpu-tuning-file:/data/local/tmp/GitHub_706/tuning.txt;gpu-tuning-level:1"
However it did not create the tuning file. Which suggests there's a parameter handling problem in there some place.
Some small suggestions for you. You said the inference is running on Gpu but could you remove CpuAcc and CpuRef as compute options just in case there's an underlying problem being hidden by fall back to those compute devices.
I'm also going to investigate the parameter handling here.
Colm.

Hi,
I've identified the reason why the parameters from TfLite runtime are not getting down to the backend. This patch on Arm NN main branch will solve that:
GitHub_706_patch.txt
However, even though the values are now being passed to the backend the tuning file still isn't being created.
I'd appreciate it if you could try this patch and let me know if it works for you. In the meantime I'll continue to debug.
Colm.

I'm making progress now. I've just submitted this review
https://review.mlplatform.org/c/ml/armnn/+/8836
It ensures that the parameters are correctly handled from the delegate and execute network. However, there is is another known issue at the moment around enabling the GPU profiling. I'll update this ticket when that has progressed.
Colm.

Closing due to lack of activity, please reopen if the issue persists.
Regards,
Francis.


Get Started

[x] Add Vscode devcontainer with tasks and settings for project management
[x] Cleanup Readme but keep references and acknowledge source
[x] Get acquainted with automated release management (release-please)

New github actions for conventional commits and generating release notes #38

added release-please but haven't really tested it. Let's just look what it creates and tweak from there

hope that release-please is no working as expected. Time will tell ;)


Undo/redo buttons only show their status in Blocks

In the Blocks section, the undo/redo buttons helpfully change colour to show whether they can do anything or not. In the JavaScript section, they always look like they can do something.

I do think this is limitation of Monaco editor. @microsoftsam?

Limitation in the monaco editor.

Closing as Dependency issue on external Monaco editor


Gitlabsource additional Conditions and Printer Columns

This PR contains two UX improvements for Gitlabsource:

living condition set is extended with additional conditions to reflect Webhook and Receive Adapter states,
CRD has additional printer columns to get more details in kubectl outputs.

/test pull-knative-eventing-contrib-go-coverage

/lgtm

Argl, concurrent reviewers.


⚠ Bofrost.at has degraded performance

In 5192752, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1007 ms

Resolved: Bofrost.at performance has improved in 8b7a0e7 after 41 minutes.


feat!: Add support for FLOAT32

BREAKING CHANGE: The default mapping for values of CLR type float was FLOAT64 and it is now FLOAT32.

@jskeet I'm still running integration tests locally, it'll be a while.
It's also not clear to me if the emulator supports FLOAT32, so that check may fail here. In that case I'll skip FLOAT32 tests when running on the emulator.
Otherwise, this is ready for review.

OK, so FLOAT32 is not supported on the emulator. I'll skip those tests tomorrow.

Filed b/333670648 for the decimal discussion.

These emulator failures are just because some FLOAT32 tests that I didn't skip. Doing that now.

All green on the emulator now.


compiled releases?

Would it be possible to include releases that actually have the 'compiled' js files?
This looks like it could be a useful library, but I can't really use it because quite frankly, I have no desire, nor a good reason to involve myself with the nightmare that is npm/nodejs. I don't use it, in fact, I go out of my way not to. Generally not a problem, aside from when I find libraries that just HAVE to have it for some reason...
Beside the nodejs annoyances, releases really should have the finished product, anyways. At least far as I can tell, this is the purpose github intended for them.

Did any of this approaches worked for you?
https://github.com/stampit-org/stampit#install
Those are browser ready compiled released.
Tell me if it doesn't work or you need something else.

IDK how i missed those...
You might want to make them a bit more noticeable since some people see console commands and quit reading...
however, i do get THIS problem, now
stampit()
TypeError: stampit is not a function [Learn More]
typeof stampit
"object"
i also did a toSource() , but other than confirm it's an object, nothing of note.
So, how am I supposed to create a new stamp, then? your documentation all says to call stampit()
do i just use stampit.() , now?
another question, can i define the methods like
{
methodName:function(){}
}
?
I'm certainly not willing to trust certain ES6+ syntax to be supported just enough to be safe to use everywhere, also... I might use either typescript or haxe, and i have to use not so exciting things as-is to do what I'm planning to do.

You might want to make them a bit more noticeable

I'll be glad to accept your contribution! Feel free to submit a PR.

TypeError: stampit is not a function [Learn More]

This is an unfortunate error of "too much compilation" so to say. There is an opened issue about that. Have no time to fix it properly. A quick workaround is to replace stampit().bla with stampit.bla.
To dive deeper into the stamps I'd recommend you this blog post series: https://medium.com/@koresar/fun-with-stamps-episode-1-stamp-basics-e0627d81efe0
Have I answered all your questions?

All you need really to do is put "download compiled version" or something
Or note that Bower and whatever else aren't mandatory
I don't feel comfortable editing an MD file... I generally don't use them, thus never bothered to learn their syntax
Alright... As long as it still works
Neat!
Probably... we'll just see further down the road

kinda? i started a few years ago and was only writing things by hand. eventually was made aware that there are tools to build JavaScript from much nicer-looking code, Like the Haxe compiler and VS:Code. I still have quite mixed feelings about using it to build entire programs, though. I am aware it's not "true" compilation... lol. Definitely heard permutations of "trans" + "piled" , as well. Just don't use them because I rarely see any reason to.
There is one totally pointless one I forgot multiple times now...
what's the "umd" stand for in the filename?
and yes, thanks for the help.

UMD = Universal Module Definition
Used to expose code to different module systems

Alright
Well, I'll let you know if I have any other problems/questions
Should be safe to close this now.

Thanks!


Registrar: Decouple AtSign reservation from secondary activation

Is your feature request related to a problem? Please describe.
Currently as soon as the reservation process is complete, the secondary is activated. When we have IOT customers who want to reserve very large numbers of atSigns many of which will not need to be activated for a very long time, then, if we still have activation happening as soon as reservation is complete, we will consume lots of infrastructure for no reason.
Describe the solution you'd like
Would like to decouple reservation from activation, so that an IOT partner can reserve atSigns at will, but only activate them at a later date when the activation is actually needed.
See the sequence diagram here for an outline of what we need in terms of process
Additional context

I am defining "AtSign Reservation complete" as "The registrar has supplied the AtSign and CRAM Key to the requester"
The 'activate' API method should require only the AtSign and its CRAMKey as parameters
The 'reserve' parts of the flow will need a small bit of reworking so that it is possible at some stage for the requester to provide a 'deferActivation=true' parameter in order to prevent immediate activation

@gkc to generate cramkey we need uuid, it's hash of uuid.
Best we can do is to give an API to assign free atsign and when user will login, we can give cramkey.
Like we are doing in registrar site, use can get atsign and can activate later on

Closing this issue as we completed design and @athandle has a working spike implementation. Created new issue at #796 for completion of the implementation


UE 4.19.2 packaging not possible. Can't produce UE4-UnrealEnginePython.lib

I've tried every possible constellation with either the embedded Python or a system-wide Python but I just can't package a game with the latest plugin version. I've also included an empty ue_site.py file like described in the readme.
My testscenario is a blank C++ project in UE 4.19.2 with the Python plugin.
This is my output log:

UATHelper: Packaging (Windows (64-bit)): ********** BUILD COMMAND STARTED **********
UATHelper: Packaging (Windows (64-bit)): Running: C:\Program Files\Epic Games\UE_4.19\Engine\Binaries\DotNET\UnrealBuildTool.exe tfbuildtest Win64 Development -Project=D:\ss\tfbuildtest.uproject  D:\ss\tfbuildtest.uproject -NoUBTMakefiles  -remoteini="D:\ss" -skipdeploy -noxge -generatemanifest -NoHotReload
UATHelper: Packaging (Windows (64-bit)):   Using Embedded Python at: D:\ss\Plugins\UnrealEnginePython\ThirdParty\Python36
UATHelper: Packaging (Windows (64-bit)):   full lib path: D:\ss\Plugins\UnrealEnginePython\ThirdParty\Python36\Lib\python36.lib
UATHelper: Packaging (Windows (64-bit)): Took 1.2170991s to run UnrealBuildTool.exe, ExitCode=0
UATHelper: Packaging (Windows (64-bit)): Running: C:\Program Files\Epic Games\UE_4.19\Engine\Binaries\DotNET\UnrealBuildTool.exe tfbuildtest Win64 Development -Project=D:\ss\tfbuildtest.uproject  D:\ss\tfbuildtest.uproject -NoUBTMakefiles  -remoteini="D:\ss" -skipdeploy -noxge -NoHotReload -ignorejunk
UATHelper: Packaging (Windows (64-bit)):   Using Embedded Python at: D:\ss\Plugins\UnrealEnginePython\ThirdParty\Python36
UATHelper: Packaging (Windows (64-bit)):   full lib path: D:\ss\Plugins\UnrealEnginePython\ThirdParty\Python36\Lib\python36.lib
UATHelper: Packaging (Windows (64-bit)):   Parsing headers for tfbuildtest
UATHelper: Packaging (Windows (64-bit)):     Running UnrealHeaderTool "D:\ss\tfbuildtest.uproject" "D:\ss\Intermediate\Build\Win64\tfbuildtest\Development\tfbuildtest.uhtmanifest" -LogCmds="loginit warning, logexit warning, logdatabase error" -Unattended -WarningsAsErrors -installed
UATHelper: Packaging (Windows (64-bit)):   Reflection code generated for tfbuildtest in 7.2668511 seconds
UATHelper: Packaging (Windows (64-bit)):   Performing 8 actions (4 in parallel)
UATHelper: Packaging (Windows (64-bit)):   PCH.UnrealEnginePython.cpp
UATHelper: Packaging (Windows (64-bit)):   d:\ss\plugins\unrealenginepython\source\unrealenginepython\private\Wrappers/UEPyFSoftSkinVertex.h(15): error C3646: 'ss_vertex': unknown override specifier
UATHelper: Packaging (Windows (64-bit)):   d:\ss\plugins\unrealenginepython\source\unrealenginepython\private\Wrappers/UEPyFSoftSkinVertex.h(15): error C4430: missing type specifier - int assumed. Note: C++ does not support default-int
UATHelper: Packaging (Windows (64-bit)):   d:\ss\plugins\unrealenginepython\source\unrealenginepython\private\Wrappers/UEPyFSoftSkinVertex.h(22): error C2065: 'FSoftSkinVertex': undeclared identifier
UATHelper: Packaging (Windows (64-bit)):   ERROR: UBT ERROR: Failed to produce item: D:\ss\Plugins\UnrealEnginePython\Binaries\Win64\UE4-UnrealEnginePython.lib
PackagingResults: Error: UBT ERROR: Failed to produce item: D:\ss\Plugins\UnrealEnginePython\Binaries\Win64\UE4-UnrealEnginePython.lib

(I checked the ominous FSoftSkinVertex errors but couldn't detect any issues with that)
What am I doing wrong? Do I miss something?
Thank you very much in advance for the help!

I haven't tested or confirmed packaging yet. It's possible it is broken atm. Check the parent unrealenginepython branch and see if it packages for you, I might be missing a few upstream commits to ensure it compiles in monolithic mode.

Hi, first of all, thanks for the great work.
i also have the same problem, it wont let me package your fork, 20tabs works fine.
i use your fork because of the multithreading functions you added, unfortunately i cannot locate the changes you talk about here:
https://github.com/20tab/UnrealEnginePython/issues/181
Anyways - realy cool work, i would be stoked if you could make it package somehow...
best,
sebastian pirch

Hey @sepppirch, do you still use UE4.19? I've integrated a changelist to fix the issues for UE 4.22 a while back, so that version should work if you get the latest code version. (Pull Request for that fix)


Dev

Update, Add and Move Chapters
Update to Chapter 14
Moved workflows chapter to 18-Workflows
Added new chapter on Diffs

Changes are accepted


Segregate the resources in README into each folder

Let's not have all the resources stored in the README. Instead create a new folder for a resource (if not already) and store the resources in that. Make sure to add hyperlinks and small description. You can take the this repo as a reference.
Thank you!

LGTM!


Normalisation through assert_norm crashes F*

The following code crashes when using F* master, commit 95984a8067e8be3e2.

let sprop = bool -> prop

let pred (args: list bool) : sprop =
  let rec aux (args:list bool) (out:sprop) : sprop =
  match args with
  | [] -> out
  | a::q -> let out:sprop = fun s0 -> out s0 in aux q out
  in aux args (fun _ -> True)

let lemma_pred (args:list bool) : Lemma (pred args true) =
  match args with
  | [] -> assert_norm (pred args true)
  | _ -> admit()

In interactive mode, typechecking lemma_pred yields
[F* error] FStar_SMTEncoding_Env.Inner_let_rec
F*: subprocess exited.
Replacing assert_norm (pred args true) by assert_norm (pred [] true) solves the issue. Nevertheless, this should probably be a warning instead of a crash

F* now gives a warning (at the definition of aux) and an error (in lemma_pred) as follows:
(Warning) Definitions of inner let-rec aux and its enclosing top-level letbinding are not encoded to the solver, you will only be able to reason with their types
(Error) Could not encode the query since F* does not support precise smtencoding of inner let-recs yet (in this case aux)

Closing the issue, thanks for the report!


Storage layer scaffolding for aggregated apiserver

in order to assist developers to build aggregated apiserver upon different storage persistent layers quick and easily, we will provide a set of pre-built storage layer implementation for several commonly-used storage backends:

[ ] Mysql/PGsql
[ ] ElasticSearch
[ ] Git
[ ] Filepath

How about supporting mongoDB, which is a popular NoSQL and may be used in many projects

@weixiao-huang sgtm, i will start w/ one low-hanging fruit like Mysql and we will see how that goes..

I would be happy to take on the MongoDB implementation if needed, I have some background with that product.


command line execution of Docker based Digits using text-classification plugins

I'm new to workng with both Docker and Digits. Couldn't find an example of using Docker Digits to create a dataset , test etc. from command line.
Referenced Crepe article and this one.
Tried this but no luck:
sudo nvidia-docker run qlua -nographics ../ref/text-classification-model.lua -input rtda_csv/train.csv -output train.t7b
Using default tag: latest
Pulling repository docker.io/library/qlua
nvidia-docker | 2016/12/01 13:01:56 Error: image library/qlua:latest not found
text-classification-model.lua came from the text plugin page: https://github.com/NVIDIA/DIGITS/tree/ade0287f15b2ba57ebfee41fd1fb5f1645803921/examples/text-classification
Suggestions? Or can you point me to an example?
Thanks,
Elliot

Greetings @elliotekoch ! You might find this project useful:
https://github.com/kertansul/dl-docker/blob/master/Dockerfile.digits
@kertansul maintains a Dockerfile for DIGITS which points to the 5.0 pre-release.
Also make sure you review https://github.com/kertansul/dl-docker/blob/master/README.md - there are instructions there for setting up a shared volume between the host and the container (you will probably want to copy your dataset in a shared volume).

Thanks Greg! Digesting...


Suggestion to change .targets to .props

I suggest to rename these files from .targets to .props:
https://github.com/devlooped/json/blob/main/src/JsonPeek/JsonPeek.targets
https://github.com/devlooped/json/blob/main/src/JsonPoke/JsonPoke.targets
Because that way they will be imported at the beginning of and SDK style Project, and not at the end.
See:
https://docs.microsoft.com/en-us/nuget/concepts/msbuild-props-and-targets#packagesconfig-projects
That's safer, the UsingTask declared by the nuget would be available in the whole Project.

Hi there!
The UsingTask is always evaluated before any targets, so the declared tasks are available always available to any targets always, regardless of whether they (the targets that use JsonPoke) exist in .props or .targets files. See the evaluation phases for reference.
That said, the reason I put it in targets, is that for UsingTask, the first to declare it "wins", unlike targets, where the last one to declare a target (by name) is the one that determines the implementation. Having the JsonPoke task declared in a .targets, therefore, allows you to override its declaration by just putting it in the .csproj or in a .props (i.e. Directory.Build.props). It makes it more flexible.
I hope that makes sense!


Return fast from transpile with empty input

Summary
Previously transpile([]) raised a warning.  Now we just return quickly without further tests, similarly to how we return early if we're asked to transpile a list of Schedules.
Details and comments
Fix #7287.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0006%) to 82.541%

Totals

Change from base Build<PHONE_NUMBER>:
0.0006%

Covered Lines:
49830

Relevant Lines:
60370

💛  - Coveralls