Change region plugin loop playback logic

Short description of changes:
regions plugin has doubled loop playback logic.
First can be found here, binding out event:
    /* Bind audio events. */
    bindInOut() {
        // ...

        /* Loop playback. */
        this.on('out', () => { // TODO I also believe this should be `region-out`
            if (this.loop) {
                this.wavesurfer.play(this.start);
            }
        });
    }

https://github.com/katspaugh/wavesurfer.js/blob/master/src/plugin/regions/region.js#L336
Second is in playLoop() method:
    /**
     * Play the audio region in a loop.
     * @param {number} start Optional offset to start playing at
     * */
    playLoop(start) {
        const s = start || this.start;
        this.wavesurfer.play(s);
        this.once('out', () => {
            const realTime = this.wavesurfer.getCurrentTime();
            if (realTime >= this.start && realTime <= this.end) {
                return this.playLoop();
            }
        });
    }

https://github.com/katspaugh/wavesurfer.js/blob/master/src/plugin/regions/region.js#L151
The problem is that both "loops" cannot be canceled in any way. So instead of doubling the same logics about looping and to allow disabling the loop – I propose to remove logics from playLoop() but keep method itself it for BC, but depend on this.loop: boolean flag to handle looping via bindInOut(). This way we can add setLoop(boolean) method to control the looping and remove strange logic from playLoop() if.
Breaking in the external API:
No breaking changes.
Breaking changes in the internal API:
No breaking changes.
Todos/Notes:

[ ] Update documentation.

Related Issues and other PRs:
#1868

Coverage remained the same at 80.498% when pulling aeb7824fc6fefdad667b7581c22dc4faaf9b8d49 on cadavre:patch-1 into f8544deb96cc70dbd2b5a5467645f013e30810f3 on katspaugh:master.

Overall the logic here looks correct to me.  I went to go test this code, as it turns out, region playback is totally broken in master, needs https://github.com/katspaugh/wavesurfer.js/pull/1970 to fix it.
I tested with #1970 and this pr together and the looping code appears to work just fine.

Hi, this new logic seems to break again the loop:

if a region is looping and you click outside, the cursor seems to go to the clicked position, but after that returns looping on the region.
Again, if click inside other region, it goes in loop from one region to another.
I think is that because with the previous controls, the loop was interrupted if the current time was not inside the region.

@marizuccara please provide scenarios of what should happen to your cases. I'd like to improve looping so all cases could be covered.

@thijstriemstra changelog updated. It looks like also previous merged PR had wrong entry – made for 4.0.0 instead of 4.0.1 – so I also fixed this.
@marizuccara I reused code from before committing, to update loop logic based on current playback time vs region start-end times. Could you please test it in your case?

This branch has merge conflict.

changelog updated. It looks like also previous merged PR had wrong entry – made for 4.0.0 instead of 4.0.1 – so I also fixed this.

4.0.1 was released today: https://github.com/katspaugh/wavesurfer.js/commit/aaf433ecaf2f032a28a0419401ed56958f67a17e

@thijstriemstra changelog updated. It looks like also previous merged PR had wrong entry – made for 4.0.0 instead of 4.0.1 – so I also fixed this.
@marizuccara I reused code from before committing, to update loop logic based on current playback time vs region start-end times. Could you please test it in your case?

Hi @cadavre , tested your code! I think that with this unified logic, and with your latest fix, the solution seems very nice!

@thijstriemstra conflicts resolved. Merged.


CFP: Cilium Gateway API - more Gateway features

Cilium Feature Proposal
Thanks for taking time to make a feature proposal for Cilium! If you have usage questions, please try the slack channel and see the FAQ first.
Is your proposed feature related to a problem?
No
Describe the feature you'd like

Basic rate limiting based on Services, or a more advanced approach to rate limiting, which could provide higher granularity. For example, in cooperation with HPA, sizing rate limits based on deployment replicas.
Access for a specific gateway based on roles and/or clientId in JWT.
Handling CORS requests.

Related upstream Gateway API issues for these features:

https://github.com/kubernetes-sigs/gateway-api/issues/326
https://github.com/kubernetes-sigs/gateway-api/issues/1494
https://github.com/kubernetes-sigs/gateway-api/issues/1767

Thanks for your issue.
The general  approach is to wait for features accepted in upstream first, then Cilium will plan to support them. As mentioned by above message from Mike, it seems like these GEPs are either closed or still in progress.


move content to root

I'd like to move the content folder from the storage directory to the root of the project. This shouldn't break backwards compatibility since the config file is published on install.


Use a global synchronized wavefield

Fixes #564 using Gazebo Transport.
Before this patch, all the plugins using a wave field needed to have the same <wavefield> SDF parameter block in its SDF plugin definition. This pull request changes this behavior by:

Setting one of the plugins to operate as the primary. The primary plugin requires to specify all the <wavefield> parameters as usual. Then, it should periodically publish the wavefield parameters via Transport.
The rest of the plugins should be set to operate as secondary. This means passing only one parameters inside the <wavefield> block. This parameters is the topic name where the wave field parameters are sent. When received, the wavefield can be populated with a new helper function.

Our WaveVisual plugin is set to operate as the primary. All the instances of Surface and PolyhedreaBuoyancyDrag plugins are set to operate as secondary.
How to test it?

You can run the simulation as usual:

ros2 launch vrx_gz competition.launch.py world:=stationkeeping_task

and verify that the buoys and the WAM-V move a bit up and down.

You can modify vrx_gz/models/coast_waves/model.sdf:

<wavefield>
...
  <gain>0.01</gain>
...
</wavefield>

Recompile and launch the simulation again. You should verify now that the WAM-V and the buoys stay flat at the water surface.

@M1chaelM , as we discussed offline, I've created a PublishPlugin to publish the wave field parameters every 2 seconds. All worlds in this PR use this plugin now.


MultiSelectMenu

Dotmenu now has a new api surface thanks to hazel, but it doesn't have the MultiSelectMenu which was in previous versions of dotmenu.

Will probably add next week


Add GitHub as resource in the Member Profile home page 🏡

Description
We display important resources in the home page of the Member Profile, as such:

Now that we are open-source, we should have the link to this GitHub repository.
Acceptance Criteria
A new link should be listed just below the Member Wiki with the title:
GitHub

and the description:
The codebase where our software, called Oyster, lives. Go read + contribute to the codebase!

I'll take this one!

I'll take this ticket!


Some cleanup for errorhandling refactor

See #1291
@flying-sheep: Please set your editor to use 4 spaces instead of tabs, and
don't have whitespace in blank lines (use the pep8-tool to check).

The point of the first commit is to also handle subclasses of subclasses.

whoops. i thought that python 3 doesn’t even support mixed indentation?
apparently not. sorry! i prefer tabs, but i of course adapt to my environment.

I think the whole file was consistently tab-formatted.


Refactoring of the Settings Codebase

Refactoring work that needs to be wrt settings:

Move IPC from the shell page to settingsUtils.
Have the viewmodels utilize the settings utils to communicate with runner rather than through having a delegate passed to each of them.
We should be able to make the code such that settings doesn't have to save the file directly, it can rather communicate this information to the runner and the runner can save the information.
only FZ, Shortcutguide, and powerPreview use settings repository. It should be expanded to the rest so that ideally SaveSettings is made private.
If we can somehow have runner tell the settings that it has received the information and has successfully stored it, then we can notify the user if something goes wrong there instead of having settings and runner running completely independently.

@ryanbodrug-microsoft FYI. Please feel free to add anything that I've missed.

We should also have a named mutex or synchronization mechanism that prevents access violations that can be controlled by powertoys.

@alekhyareddy28
If you work on settings codebase and mutex does it make sense to look on bug #6471?

@htcfreek, I would be modifying the settings files whereas that requires modification in the runner, which someone else would look into. Thanks for bringing it up though.

removing the in-progress label as it has been de-prioritized.

This is a very large work items, it won't be done in 0.29.

@jaimecbernardo we should have someone investigate what's left for this in v0.49.

@dedavis6797
A quick investigation by @SeraphimaZ indicates that none of the 5 items are done yet.
3 seems to be implemented but not all modules are using it.
Would be better to do a more through investigation for 0.49.

@jaimecbernardo i'm moving this to a "priority work bucket"  I think these are nice to haves but not needed


Regression: Changes to classification adapter for regression

@amueller I made some changes to the classification adapter because it was always returning 100 as the regression target.

sorry seems I hadn't pushed some part but I think there was still a bug either way.


Correct list of supported formats in Editor Documentation for OpenOffice

http://docs.cartodb.com/cartodb-editor/datasets/#supported-file-formats lists OpenOffice spreadsheets as having the extension ODB.  It's actually ODS.
Also see #6300

cc @csobier


Pause if earned x amount of xp and/or caught y amount of pokemon

I think it would be great if we had an option to pause/stop the bot at least temporary for the day (or depending on sleep schedules) once we have met our 'quota'.

+1, I read about this in other issues (and the Reddit post of course), not sure if it's WIP.

#3922

On the todo list as well (#4004). I'm going to begin work on this tonight, less I find another PR already up.

Contained in #3922.

#3922 Duplicate


Styling window in golang

Hi there,
I am developing an app in golang and using nice library nuklear. But when i styling window of my app etc background color, i can not set background color. I want to use ctx->style.window.background example. But in golang has not same code. I try ctx.Style() , but this code not continue about background. Could you add styling code in golang, ASAP or Can you help me? I am waiting for this.

Hi, related:
https://github.com/golang-ui/nuklear/issues/6
TL;DR: You'll have to write a setter

Setter is placed in etc.go? Cause Style() in there

You can place in any file that is not auto-generated, etc.go there is my file for such stuff, yes.


debian/jessie64

Hello,
Did you test this on debian/jessie64? I get the error message
TASK: [php | Add PPA Repository] **********************************************
failed: [default] => {"failed": true}
msg: Invalid repository string: ppa:ondrej/php

FATAL: all hosts have already failed -- aborting

@PieterCappelle role has been updated, but I had to comment out php7.0-mbstring package (for debian is unstable)


Export Template Manager downloads automatically stops when window not in focus.

Godot version:
3.0.2
OS/device including version:
Windows 10
Issue description:
Whenever your 'focus / active window' changes, the download automatically stops. If I go away from my PC, the download actually works and completes successfully, which contrasts other issues.
Steps to reproduce:

Start downloading the latest export template.
Once it starts, change the active window. The console should display something like this:

I'm dumb. Found an duplicate issue where that's already fixed on master. Closing.


Juniper mist Configuration options

In the initial scope, it was understood that these values where fixed: RF_TEMPLATE_ID, SITE_GROUP_IDS, ORG_ID
and Enable GovWifi,Enable MoJWifi should be stored as custom_vars on the mist console.
These understanding where incorrect.
Values RF_TEMPLATE_ID, SITE_GROUP_IDS, ORG_ID are dynamic depending on what environment you are running on.
Enable GovWifi,Enable MoJWifi should be stored within "site_groups" within the mist console AND SITE_GROUP_IDS are required to be defined so that when a field gets read in as TRUE or FALSE from the spreadsheet, it can assign the site to the correct site_group_id.
DOD:

Add support for to allow user to inject vars: RF_TEMPLATE_ID, SITE_GROUP_IDS, ORG_ID
Add input validation
Update docs
Update unit tests

related to https://github.com/ministryofjustice/juniper-mist-integration-spike/issues/4


TypeConsumer is actually a visitor

Per discussion with Player about the game rule api, he noticed that the TypeConsumer is actually a visitor. So here is a PR.
And map that one bootstrap method that uses the visitor

Since player points out, guess we can make acceptBoolean visitBoolean etc


test that connection close whilst idle works

Motivation:
I wasn't sure if we actually properly observed an idle connection
suddenly closing. Ideally, if an idle connection just goes away, we just
want to remove it from the connection pool (which is what happens).
Modification:
Add a test that verifies idle connections that get closed from the
server don't get reused in the pool.
Result:
Better test coverage for more complicated scenarios.

CC @adtrevor


Use Borrow trait bound for bip158 Iterator::Item

Currently the generic iterators in bip158 return items of type slice reference. We can make the code cleaner (no explicit lifetimes) and more general if the generic iterator argument iterates over Borrow<[u8]>

I also don't understand the distinction between Borrow and AsRef but ack either one.

Rebase only, no other changes.


feat(srr520): initial driver

PR Type

New Feature

Related Links

Description
Initial hardware interface, decoder, and ros wrappers for the continental SRR520

Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[ ] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR: Required to pass before the merge.

@mojomex
Although it is marked for review, we still need to merge the ARS548 PR before addressing this one (also, the priority for this one is quite low)

Codecov Report
Attention: Patch coverage is 0% with 3222 lines in your changes are missing coverage. Please review.

Project coverage is 6.11%. Comparing base (4651feb) to head (199193a).

Files
Patch %
Lines

...inental/continental_ars548_decoder_ros_wrapper.cpp
0.00%
534 Missing :warning:

..._hw_interfaces/continental_srr520_hw_interface.cpp
0.00%
401 Missing :warning:

...inental/continental_srr520_decoder_ros_wrapper.cpp
0.00%
401 Missing :warning:

...al/continental_ars548_hw_interface_ros_wrapper.cpp
0.00%
347 Missing :warning:

...ontinental/decoders/continental_ars548_decoder.cpp
0.00%
284 Missing :warning:

..._hw_interfaces/continental_ars548_hw_interface.cpp
0.00%
243 Missing :warning:

...al/continental_srr520_hw_interface_ros_wrapper.cpp
0.00%
243 Missing :warning:

...ontinental/decoders/continental_srr520_decoder.cpp
0.00%
232 Missing :warning:

...ti_continental_ars548_hw_interface_ros_wrapper.cpp
0.00%
196 Missing :warning:

...terfaces/multi_continental_ars548_hw_interface.cpp
0.00%
171 Missing :warning:

... and 14 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##            main    #124      +/-   ##
========================================
- Coverage   6.58%   6.11%   -0.48%
========================================
  Files        136      81      -55
  Lines      10987   11259     +272
  Branches     869     859      -10
========================================
- Hits         724     688      -36
- Misses      9677    9987     +310
+ Partials     586     584       -2

Flag
Coverage Δ

differential
6.11% <0.00%> (?)

total
?

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

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

This PR should be reviewed after https://github.com/tier4/nebula/pull/151 is merged

@knzo25 As

#151

has been merged, I will review this PR after it has been rebased. Do you prefer rebasing yourself or should I do it?

I will do it, but it will not look like rebase (remember that this PR started 3 month ago and so the commit history is already quite destroyed)

@mojomex
Thanks for your review !
I think I addressed all the comments so please have a look when you have time

@knzo25 I just noticed on the commits on this PR that your commits are not verified.
Although we don't enforce this in Nebula right now, in future I would like to. When you get a chance, please update your SSH and GPG keys. Thanks!

@knzo25 I just noticed on the commits on this PR that your commits are not verified. Although we don't enforce this in Nebula right now, in future I would like to. When you get a chance, please update your SSH and GPG keys. Thanks!

Ooh, I was using -s but since I formatted the computer the keys changed :scream:, Will do !


Musicmanager.download_song - GetDownloadLink: 404 Client Error: Not Found for url

When attempting to call music_manager.download_song(song_id) I'm getting the following exception:
GetDownloadLink: 404 Client Error: Not Found for url: https://music.google.com/music/export?songid=20f89b83-8021-3f22-af05-02ce6d5f601a&version=2
(requests kwargs: {u'url': u'https://music.google.com/music/export', u'headers': {u'X-Device-ID': u'B8:F6:B1:13:7D:3E', u'Authorization': u'<omitted>'}, u'params': {u'songid': u'20f89b83-8021-3f22-af05-02ce6d5f601a', u'version': 2}, u'method': u'GET'})
(response was: u'\n\n<TITLE>Not Found</TITLE>\n\n\n
Not Found
\n
Error 404
\n\n\n')

I first did the Musicmanager.perform_oauth step which finished with "succeeded".
I then wrote a script to test the download_song method as follows:
#!usr/bin/env python3

import time
import gmusicapi

def main():

    # Create the APIs.
    music_manager = gmusicapi.Musicmanager()
    mobile_client = gmusicapi.Mobileclient()

    # Log in.
    music_manager_logged_in = music_manager.login()
    if music_manager_logged_in:
        print("Musicmanager login succeeded")
    else:
        print("Musicmanager login failed")
        return
    mobile_client_logged_in = mobile_client.login(user, password, gmusicapi.Mobileclient.FROM_MAC_ADDRESS)
    if mobile_client_logged_in:
        print("Mobileclient login succeeded")
    else:
        print("Mobileclient login failed")
        return

    all_songs_generator = mobile_client.get_all_songs(incremental=True)
    for song_list in all_songs_generator:
        for song in song_list:
            song_id = song['id']

            # This is where the exception occurs.
            filename, audio = music_manager.download_song(song_id)

            time.sleep(1)

if __name__ == "__main__":
    main()

If I ignore the exception and continue to loop, the same exception is raised each iteration, so it doesn't seem to be song-specific.
pip lists gmusicapi as version 10.0.1
Perhaps the URL might have changed from https://music.google.com/music/export to something else?
Any ideas on what might be going on here would be appreciated 😸

Can you try with http://unofficial-google-music-api.readthedocs.io/en/latest/reference/musicmanager.html#gmusicapi.clients.Musicmanager.get_uploaded_songs instead of mobileclient.get_all_songs? I'm wondering if the musicmanager will 404 if given an All Access track that's been added to the library (rather than an uploaded track).

It seems you're right - when replacing mobile_client.get_all_songs with music_manager.get_uploaded_songs the music_manager.download_song method works without issues. Is this expected behaviour? Or perhaps the result of a recent change to the google music protocol?

I think this is expected, though I've never actually tried Google's music manager with my subscription test account. I think Google's music manager would download your uploaded and purchased tracks, but not anything from your subscription.
Seems like we should update the docs!

I seem to be getting the same behaviour when attempting to use Webclient.get_stream_audio or Webclient.get_stream_urls - both respond with 404.

Hm. What about the mobileclient? The webclient isn't as well tested, so it might just be broken in some unrelated way.
Also, can you give an example of the kind of id you're using?

I just tried mobile_client.get_stream_url and it seems to work fine with the ids retrieved from the get_all_songs song dictionaries 👍

I'll leave this open for you to close in case you'd like to wait until the docs are adjusted.

Sounds good. Glad you got it working!


workaround to a minor bug in mux which filters out empty query

Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

Thanks for the clean fix.
We can later try to register a custom func in mux to prepare the vars map.
LGTM


Editor improvements

Codecov Report

Merging #83 (90f2c78) into master (872f756) will decrease coverage by 0.60%.
The diff coverage is 65.78%.

:exclamation: Current head 90f2c78 differs from pull request most recent head b1e0090.


AcessViolation in FileDrop event

 private static void OnWindowFileDrop(object sender, FileDropEventArgs e)
    {
        foreach (var item in e.Filenames)
        {
            Console.WriteLine(item);
        }
    }

Exception:
Fatal error. System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
at System.Runtime.InteropServices.Marshal.ReadByte(IntPtr, Int32)
at GLFW.Util.PtrToStringUTF8(IntPtr)
at GLFW.NativeWindow.OnFileDrop(Int32, IntPtr)
at GLFW.NativeWindow.b__115_4(IntPtr, Int32, IntPtr)
at GLFW.Glfw.WaitEvents()
at GLFW.Glfw.WaitEvents()
at Program.Main(System.String[])

If you haven't fixed it already:

how are you setting the callback? are you doing something like

GLFW.Glfw.SetDropCallback(Window, DropHandler) {     //etc }
where DropHandler is the name of the method that does the handling?
If so, try defining a field of type GLFW.FileDropCallback and writing the "value" DropHandler to it after creating the window


default Docker parameters

Hello,
Is there any way to modify the run string of a container through the GUI without going through the CLI? Context is deploying a container to students who aren't technically savvy and having them use Kitematic for this. I tried searching through Docker docs to see if maybe this could be done on the build end, but it seems everything related to RUN, CMD, ENTRYPOINT, etc. wasn't capable of doing something like mapping ports? The string we would want to run is this:
docker run -e "OFFLINE_IP=$(IP)" -e "OFFLINE_PORT=8080"
--name $(CON_OFF) -d
-p 5050:5050 -p 8080:8080 $(IMG_OFF) 2>/dev/null
Thanks for the help!
Best,
Colton

@coltonoscopy Does the image have the above ports declared in the Dockerfile via 'EXPOSE' ?
All of the -e parameters can be set within the container's General tab, under Environment variables.
Custom port editing/mapping is still a work in progress. If you have them created during the initial docker run they should be retained after changes to the general settings (environment)

@FrenchBen Thank you for the reply; the ports are indeed exposed via EXPOSE, but I was hoping there was a way to configure it in such a way that we could get a 1:1 mapping (i.e., 5050:5050 and 8080:8080 as in the above string). As best I can tell, EXPOSEing the ports only makes them outwardly available from within the image but doesn't let the host access them except via default port mapping?

The 'random' default port mapping in Kitematic is in place to prevent port conflicts. If you map them during the initial creation, they should remain mapped afterwards.
Perhaps you could create a compose file for your student to use as a base, then use Kitematic to do the rest of the config?

@FrenchBen Thank you for the reply! We would ideally like to avoid having the students build their images and instead have everything happen from within Kitematic. Can an image built with compose, for example one with ports mapped within it, be uploaded to the Docker Hub and then run from within Kitematic without it remapping the ports?

@coltonoscopy I don't think you can force the port-mapping of the images. Compose is simply to organize all images that would be built/linked on the system along with port maps, see:
https://docs.docker.com/compose/

@FrenchBen Ah I see... thank you for your help!

You're welcome :+1:
Port mapping/editing should be added soon to Kitematic, so keep an eye on the release details


Update Programming Playground with Gremlin section and reference

What does this PR do?
Add resource(s)
For resources
Programming Playground
Description
Adding Gremlin section to Programming playground with first reference.
Why is this valuable (or not)?
Having a place to play with gremlin
How do we know it's really free?
Its under donation policy only and its sponsored by AWS neptune to promote gremlin open source language.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

the changes to the index all break links. don't do that!

I got a lil problem with one vs code extension and it passed by. Fixed already @eshellman

Thanks, and welcome to free-programming-books!


[FR] Use query param examples in demo/code example

Is your feature request related to a problem?
Not sure if this is a bug or a feature request.  I would like to show examples of query parameters in the code sample snippets, but adding them as examples to my parameters does not cause them to be included either there or in the demo sidebar.
Describe the solution you'd like
Adding an example to a query param should make it be included in the demo box and the code snippets, similar to the way that examples are added to request bodies.
Describe alternatives you've considered
N/A

Hi @IanVS, this is a known limitation. Currently, the behavior is that requestBodies are "pre-loaded" into code snippets but parameters require user input to appear. You're welcome to open a FR for this as I can see how it makes sense to pre-load parameter values. I suppose the initial params should be based on defaults/examples and fall back to data type.

You're welcome to open a FR for this

Sorry, but isn't that what this is?  Do you mean open a PR for it?

Yup, disregard. There's so many issues that I confused this with another one. 😅

Haha I know how that goes.  Sorry I've been opening so many issues / discussions.  I've been really liking the tool overall though.  I even just tweeted about it!  https://twitter.com/IanVanSchooten/status/1626668097984765953

Are there any updates regarding this issue?

related to #544


Many Popovers on a Page Causes Very Slow Popover load times

Bug type
Component
Component name
MudSelect, MudTooltip, MudMenu
What happened?
It's possible the answer is "this is an issue with Blazor, not MudBlazor", but I ran into a very difficult to debug problem with MudSelects being slow to load, instant locally but anywhere from 10-60 seconds depending on connection speed in production.
After a month of debugging I realized the tooltips I had on repeated elements (I.E in a table) were all creating popovers that would get re-rendered when page content changed. This meant there were about 500 popovers that a user would have to wait to load before a MudSelect was interactable.
My short term answer was to remove the tooltips from repeated content, but this is less than ideal.
https://github.com/DrMurloc/PumpItUpScoreTracker/blob/main/ScoreTracker/ScoreTracker/Pages/Charts.razor Here's one of the affected pages, note that the tooltips were around the icons in the rows inside the table. https://piuscores.arroweclip.se/Charts
Expected behavior
Popover content to be available on demand as opposed to waiting for all popover content to try to pre-load.
Reproduction link
https://piuscores.arroweclip.se/TierLists
Reproduction steps
(Note that I removed tooltips on the above link so it's not reproducable there anymore)

Add about 500 tooltips to a page
Set network speed to slow
Change page content causing a rerender
Observe that a MudSelect takes multiple seconds to render
...

Relevant log output
No response
Version (bug)
6.11.0
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I guess its blazor server side?
Try for now:
services.AddMudServices(config =>
{
	config.PopoverOptions.Mode = PopoverMode.Legacy;
});

I have the same issue as DrMurloc. If a page has many popovers (or elements that utilize them), they can take some time to appear (4-5 secs for me) after the page has loaded.
In my case, I have a custom grid/table, where each row has several IconButtons, each with their own tooltip. When loading this table with 20+ rows, the slowdown becomes apparent and impacts all controls that use Popups on the page.
If one waits (4-5 secs for me) after the page has loaded, they function as normal.  This does not appear to occur in local development, but does occur in production (Azure App Service, Blazor Server).
Could this be a JS timer issue?
I am happy to report, however, that setting the PopoverMode.Legacy seems to have resolved my issues.  :) The pages with lots of Tooltips also seem to load a lot faster (feel more responsive).

In my use case (lots of buttons on the page with ToolTips), I will probably refactor to just use the HTML title attribute to help with performance and reduce overhead.

I am happy to report, however, that setting the PopoverMode.Legacy seems to have resolved my issues. :) The pages with lots of Tooltips also seem to load a lot faster (feel more responsive).

Thanks for confirming this. I will leave this issue open to report later when i will fix the default mode, since legacy mode has it's own performance problems during dispose on other platforms.

The fix is in the MudBlazor 6.17.0 version if anyone wants to test with
config.PopoverOptions.Mode = PopoverMode.Default;

The fix is in the MudBlazor 6.17.0 version if anyone wants to test with
config.PopoverOptions.Mode = PopoverMode.Default;

On my quick dev machine testing, this appears to be working well and performance does not seem to be impeded with a lot of popovers on the page. I tested with adding about 100 tooltips to a large table.
I'll post back here if we notice any production issues.
Thanks for the fix and the other performance tweaks in 6.17 ScarletKuro!


Hide outdated comments in file view

Fix #4991

Codecov Report

Merging #5017 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5017      +/-   ##
==========================================
- Coverage   37.41%   37.41%   -0.01%
==========================================
  Files         306      306
  Lines       45374    45374
==========================================
- Hits        16978    16975       -3
- Misses      25944    25946       +2
- Partials     2452     2453       +1

Impacted Files
Coverage Δ

models/issue_comment.go
48.37% <100%> (ø)
:arrow_up:

models/repo_indexer.go
50.84% <0%> (-1.28%)
:arrow_down:

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 d7ca839...095b284. Read the comment docs.


open & run every example app in the apps repo. validate that run results display in the CTRL panel

To Do:
open & run every example app in the app’s repo. validate that run results display in the CTRL panel

need more automation
may need to add mocking, if so - convert to new tasks

I think that this e2e test is supposed to do this:
https://github.com/flojoy-io/studio/blob/develop/cypress/e2e/example_apps.spec.cy.ts
It could use some enhancements, though:

[ ] verify that it's working and running in CI 😆
[ ] instead of using https://github.com/flojoy-io/studio/blob/develop/cypress/e2e/config_example_app_test.json to decide which apps to run, we should simply run all apps in the apps repo. there shouldn't be anymore sample apps within studio - they should all be in apps now
[ ] in addition to running the app and verifying a successful run, should validate the control panel state for apps that have a control panel. changing a widget and rerunning the app should result in a different output.
[ ] would be nice to add Percy to CI for image screenshots of the 3 tabs after runs (App, Debug, and CTRL)


Suggestion: entries in "Display" can be accurately searched in the search box of Zotero.

Dear Developer,
Thank you very much for developing this plugin.  I am very grateful.
During my use, I have encountered some issues that I would like to share with you. When using Preview, I often use "temporary placeholders" while writing articles. However, when inserting references, I use these "temporary placeholders" to search for entries in Zotero to trace the original entries. The problem arises because the search results are not always unique, as shown in the attached image.
I hope you can optimize this feature. Below is a comparison image between Better BibTeX for Zotero and Preview, as shown.
I hope that the entries in "Display" can be accurately searched in the search box of Zotero.
Thank you very much!
Best regards,

I'm not this is possible


Code cleanup

Improve docstrings and documentation. Additionally, make some minor efficiency and naming changes.

Also format everthing with isort and black using default arguments.


Create Solidity tests benchmark to understand JS overhead

Definition of Done
Create Solidity tests benchmark to understand JS overhead using forge-std (local tests only). Compare with forge test as baseline at forked version.

Done in https://github.com/NomicFoundation/edr/pull/506


Add field indexer to include kind and API Version for AWSNodeTemplates

Tell us about your request
Add field indexer as described here: https://github.com/aws/karpenter/pull/2812/files/47690c9e57f918b94da87c1ec1b2c587ffd172a3#r1046402188
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Field Indexer used for watching provider ref only does it for the providerRef and name, but we may need to also index on Kind and API Version to be more robust in the future. It is technically sound now since the only Provider Ref is the AWSNodeTemplate.
Are you currently working around this issue?
Not a problem right now
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Closing this since it's no longer relevant.


[Feature] A method to answer wether a subtree has non-null values?

Similar to standing on a root directory and asking whether any of the sub-directories has files in it

what is the use case? this does not sound like something that is needed often enough to be implemented in OmegaConf.

not convinced this should be in OmegaConf.


Remove undefined defaultNavigationOptions from createFluidNavigator

defaultNavigationOptions is not exported by react-navigation, what makes its usage ineffective in this context, since it will always be undefined.
@chrfalch, is there any underlying reason why we should keep this, or it's just a leftover? Tks.

Hi, @guiccbr :) Are you sure about this?


Add M1 Mac support using ARM Docker image

From: @conblem

We are using lowkey-vault for local testing of our .netcore api's. Some of our devs use M1 MacBooks and docker has to resort back to amd64 emulation on these devices, because there are no native images. The startup time of our whole stack depends on first having a running lowkey-vault instance. At the moment this is really slow for our engineers using ARM machines. We would really appreciate a ARM build of your project.

Thanks for creating this issue. First off really appreciate the project. Docker natively supports building multi arch images using buildx. The whole topic is documented here: https://docs.docker.com/build/building/multi-platform/ . The eclipse temurin images does seem to have builds for arm you can see this referenced here: https://hub.docker.com/_/eclipse-temurin

Hi @conblem
thank you! I have started to look into this. It is not that easy to make this part of my build pipeline as:

the Gradle plugin I am using does not support buildx yet
The base image is not the same (the Alpine based JRE image does not appear to have an arm64 variant) and I would love to keep the version pinned with the SHA as well
I will need to figure out how to build it on Github as well.

Anyway, I have managed to build an experimental version locally (although I couldn't figure out how to test it) using this Dockerfile:
FROM eclipse-temurin:11.0.17_8-jre-jammy@sha256:aa4584a9a4a59653192d854382049b4c8e028c7c62710344c0beeb30ffdb4314
LABEL<EMAIL_ADDRESS>EXPOSE 8443:8443
ADD lowkey-vault.jar /lowkey-vault.jar
RUN mkdir "/import"
CMD [ "sh", "-c", "ls /" ]
ENTRYPOINT ["sh", "-c", "java ${JAVA_OPTS} -jar /lowkey-vault.jar ${LOWKEY_ARGS}"]

If you wish to give it a try, please check it out here: https://hub.docker.com/layers/nagyesta/lowkey-vault-experiment/1.13.23_arm64/images/sha256-11d5b470679e7beca3f4702aa707401deea5616bd9bfb85a004fb9a374aae182?context=repo
I will continue the work to figure out how I can solve this on the long run, maybe I could set up a different project that would just grab the latest jar and build the arm image and tag it using a different tag name (e.g. 1.13.23_arm64 or similar).
Do you think this could work or would you need the same tag name with different architectures?
Thank you in advance!

Hello thanks for the effort the manually built image works 🎊 . As far as I can see the Docker Plugin supports buildx and different platforms:

Hi @conblem
Thank you for testing the image and thank you for pointing out the buildx support in the plugin! I have reached the wrong conclusion based on the changelog here: https://github.com/palantir/gradle-docker/tree/develop/changelog/@unreleased
In the meantime I have found a Github action too, that should be able to set up buildx for the build: https://github.com/marketplace/actions/docker-setup-buildx
I will spend some time on this and figure out how to integrate everything and have automatic builds for every platform.

Hi @conblem ,
I have ended up creating a new repository for these non-AMD64 images and managed to push the first two to the main Docker repo:
https://hub.docker.com/layers/nagyesta/lowkey-vault/1.13.23-arm/images/sha256-8868d257f0e16c937565d23332cffb4aa4f77a4cb92e755eeef650371335c438?context=repo
https://hub.docker.com/layers/nagyesta/lowkey-vault/1.13.23-arm64/images/sha256-36fb72a4d2410ef7dda10ca286703fac41ed35aa302f59848d7b208766ca3d3b?context=repo
Please feel free to use them, I will make sure to trigger a build for these after every Lowkey Vault release.
Looking forward to your feedback on how it works, thank you!

Hello this is a very welcome change thanks for the effort!
I have not been able to test the new images but will do that soon :)
Is there a particular reason you have choosen to use different tags? For us this causes some akward hacks in our codebase. We use a shared docker-compose file for all our developers. Docker then just pulls the correct image arch based on the system. Now we would have to override this for all our developers using M1 devices to use the new arm64 tags you have setup. This is not a huge hassle however and just having native images is already a huge improvement, thanks alot!

Hi @conblem and @irby ,
I have some good news! Finally had some time to look into this a bit better. Just released the new image that should just do the trick. Please find it here: https://hub.docker.com/layers/nagyesta/lowkey-vault/1.13.33-jammy/images/sha256-c2a85999fb172a2fde4570e80f891d90372500fb0daf79c35f749d41851e3bb7?context=repo
I have decided to keep the suffix of the base image I am using in this case as users will be able to select between the original, Alpine, variant and the multi-arch variant based on their needs (smaller image vs. more convenience in case of a mixed setup).
The amd64 version will be always tested using the Docker example project, so the chance of having a broken image for the arm64 and arm/v7 variants should be minimal, but should you ever face an issue with it, feel free to open a bug.
Thank you again for your feedback and patience!


Actor Information Concept

Make a concept for the feature to click on the actors in the Movie Page and get either a infoItem or gets you to a information page about that actor with information from a api (maybe even showing other movies he made by looking for that, checking if there are in the db to display the poster + title and maybe crawl the rest via api....consider if this is worth the api requests)
https://en.wikipedia.org/api/rest_v1/#/Page content/getContent-media-list
https://api-ninjas.com/api/celebrity

https://github.com/Maxiboy441/PixelPrime/wiki/[TC]-Actor-Feature


ProcStat VSize is unsigned

see man 5 proc
This should fix the integer overflow of this field on 32bit platforms.
See also https://github.com/ncabatoff/process-exporter/issues/80

FYI.
From manpage (i.e. http://man7.org/linux/man-pages/man5/proc.5.html ):

       /proc/[pid]/stat
              Status information about the process.  This is used by ps(1).  It is defined in the kernel source file fs/proc/array.c.

              (23) vsize  %lu
                        Virtual memory size in bytes.

%lu - long unsigned.
Unfortunately it is not mentioned at all in the Documentation/filesystems/proc.txt in kernel docs itself.


[Improvement]: Introduce a way to run tests in a docker container

Description
We have bal build --cloud but not  bal test --cloud but to verify the package functionality in a cloud platform, we must run the tests also in a container. This applies to GraalVM verification as well. bal test --graalvm --cloud should run the tests in a container.
For this, there should be a standalone testable jar that can be run inside a container. We can check the possibility of generating a single testable jar or separate jars for each module.

Following are the sub tasks for $subject.

[ ] Create an uber jar for test modules
[ ] Create test executable task and emit executable
[ ] Verify code-coverage, mocking with the new test executable
[ ] Lifecycle plugin implementation at C2C for running tests in docker container

@Ramith-D-Rodrigo
Let's create a spreadsheet with all the cases we need to address

--graalvm --cloud=docker is supported now. Implemented using the testable fat jar while also considering the mocking features.
Added the relevant tests for the changes. --cloud flag changes are added in c2c repository. The tests added in this repo check the changes in BTestMain and the execution of testable fat jar.


[Bug Report] el-table can not checked all

Element UI version
1.2.9
OS/Browsers version
all
Vue version
2.3.0
Reproduction Link
https://jsfiddle.net/leopoldthecuber/pj71jkyw/78/
Steps to reproduce
see jsfiddle.
What is Expected?
click "check all checkbox", the all checkbox will be checked.
What is actually happening?
click "check all checkbox", the all checkbox will be checked,but not.

Refer to the top section of #4505.

1.3.0-beta.1 already fixed this.


AutoComplete dropdown in ios is not scrollable

I can't scroll my drop down list in ios but in android i can.
I checked a lot of suggestions and nothing doesn't help.
this is my code
`return (

{showAddChar && (

+

)}
<Autocomplete
data={data}
defaultValue={query}
onChangeText={(text: string) =>
this.setState({ query: text, hideResult: false })
}
keyExtractor={index => index['id'].toString()}
inputContainerStyle={styles.inputContainerStyle}
listContainerStyle={{}}
listStyle={styles.listStyle}
style={{ fontSize: 18 }}
hideResults={hideResult}
placeholder={placeholder}
renderItem={({ item }) => (
<TouchableOpacity
style={styles.item}
onPress={() => this.onChoosenItem(item)}
>
{item['name']}

)}
/>

);
}
}
const styles = StyleSheet.create({
container: {
left: 0,
position: 'absolute',
right: 0,
top: 0
},
addChar: {
zIndex: 1,
position: 'absolute',
fontSize: 30,
right: 0,
top: 0
},
inputContainerStyle: {
borderColor: 'rgb(70, 48, 235)',
borderBottomWidth: 3,
borderTopWidth: 0,
borderRightWidth: 0,
borderLeftWidth: 0
},
//   listContainerStyle: {height:filteredMembers.length * 70},
listStyle: {
borderColor: 'grey',
margin: 0,
overflow: 'scroll',
maxHeight: 200
},
item: {
fontSize: 16,
borderBottomColor: 'grey',
borderBottomWidth: 2,
padding: 12
}
});`
***** that what i get in IOS

Same issue in IOS. Any Solution?

Hi, @barak109  I am also facing the same issue in iOS and Android. Did you find any solution?

I can scroll my dropdown in iOS but not Android. I have another similar issue open as well. Wondering if most people who use this package only have small data sets that do not need scroll? This is a major issue IMO and will be a deal-breaker for me :(

Hey, @tayloraleach I could fix the scrolling issue in Android and iOS by following the https://github.com/mrlaessig/react-native-autocomplete-input/issues/64#issuecomment-507388606 and by sending listStyle={{ maxHeight: 350 }} prop to AutoComplete Input.

The solutions suggested here didn't work for me, the solution that worked:
adding flex: 1 to listContainerStyle which is the parent of FlatList.

The solutions suggested here didn't work for me, the solution that worked:

adding flex: 1 to listContainerStyle which is the parent of FlatList.
adding style: {height: 400} to flatListProps

After trying all other options, this is the only one that worked for me on both iOS and android, thank you so much for sharing!


revising articles connections will throw error when 1 revisions remaining

bug just gone and can not be reproduced any more, close this issue


What is the ongoing vector search API going to look like?

I was using private preview, and during search I have to create azure.search.documents.models.Vector to pass that into search_client.search.
Now i am on azure-search-documents==11.4.0b6 installing by pip install azure-search-documents --pre, now according to the example I can skip the step of creating the Vector object, and instead pass in a list for the vector, and there is one topk argument instead of one in search_client call and one in Vector definition.
# Perform vector search
results = search_client.search(
    search_text=None,
    vector=generate_embeddings(query, cogSvcsEndpoint, cogSvcsApiKey),
    topk=3,
    vector_fields="imageVector",
    select=["title", "imageUrl"]
)

https://github.com/Azure/cognitive-search-vector-pr/blob/main/demo-python/code/azure-search-vector-image-python-sample.ipynb
It is all good but it is not matching what's shown in the public documentation https://learn.microsoft.com/en-us/rest/api/searchservice/preview-api/search-documents#request-body. Here in the request body there is clearly a vectors field and it is using the old object definition.
     "vectors": [
      {
        "value": "a vector representation of the query",
        "k": an integer (number of nearest neighbors to return as top results),
        "fields": "a comma-delimited list of vector fields to use in the query"
      }

So, which way is it gonna be and what is the source of truth?

@mikelam92 thanks for your feedback. We are still in preview and while the 2023-07-01-Preview REST API version won't have any further breaking changes, the SDK's may modify their syntax over time. You can find the most up-to-date python SDK sample here: https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/search/azure-search-documents/samples/sample_vector_search.py
Additionally, all the python notebooks in this repo have been updated.
For now, we are supporting both a vector object and a vectors field but in the new API version moving forward, we plan to only support vectors array as part of the request body. Please use vectors moving forward.


Add setting for tiles preview to appear expanded when tiles selected

Current behavior is to do so, but could be annoying if repeatedly editing other data
If disabled, have to click inside TBT inspector or collapsed panel to expand
Could add as checkbox to preview panel, and keep preference updated via https://github.com/dandeliondino/tile_bit_tools/issues/18

Will implement #56 instead


Users SBAT activate, revoke votes via CLI

[ ] Implementation
[ ] Documentation

i think it makes sense to base it on this PR: https://github.com/celo-org/celo-monorepo/pull/1750

Documentation is here:
https://github.com/celo-org/celo-monorepo/blob/master/packages/docs/command-line-interface/election.md#activate
https://github.com/celo-org/celo-monorepo/blob/master/packages/docs/command-line-interface/election.md#revoke


How to handle the adaptive card response in Composer

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
What package version of the SDK are you using.
Describe the bug
Give a clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

@KoradaRajesh As far as I know (@vishwacsena, correct me if I'm wrong), there's no way to capture this through a prompt, since Adaptive Card input is sent through Activity.Value and not Activity.Text.
So, you need some way of capturing those. Here's what I did:
Adaptive card json:
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "TextBlock",
            "text": "Text Input - id: textInput"
        },
        {
            "type": "Input.Text",
            "placeholder": "Enter some text",
            "id": "textInput"
        },
        {
            "type": "TextBlock",
            "text": "Number Input - id: numberInput"
        },
        {
            "type": "Input.Number",
            "placeholder": "Enter a number",
            "id": "numberInput"
        }
    ],
    "actions": [
        {
            "type": "Action.Submit",
            "title": "Submit"
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.0"
}

Then, construct a condition with turn.activity.value && !turn.activity.text and output the result with<EMAIL_ADDRESS>

yeah i have tried it but @mdrichardson how do you sent the adaptive card , if we sent in prompts it not accepting the action and again it re prompting if we sent in the (send a response) it will not wait for the submission of card ,it will immediately check the if condition in the flow.

@KoradaRajesh As far as I know (@vishwacsena, correct me if I'm wrong), there's no way to capture this through a prompt, since Adaptive Card input is sent through Activity.Value and not Activity.Text.
So, you need some way of capturing those. Here's what I did:
Adaptive card json:
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "TextBlock",
            "text": "Text Input - id: textInput"
        },
        {
            "type": "Input.Text",
            "placeholder": "Enter some text",
            "id": "textInput"
        },
        {
            "type": "TextBlock",
            "text": "Number Input - id: numberInput"
        },
        {
            "type": "Input.Number",
            "placeholder": "Enter a number",
            "id": "numberInput"
        }
    ],
    "actions": [
        {
            "type": "Action.Submit",
            "title": "Submit"
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.0"
}

Then, construct a condition with turn.activity.value && !turn.activity.text and output the result with<EMAIL_ADDRESS>

@KoradaRajesh I had to add it as a separate trigger. It's not ideal and it's difficult to get working in your flow, but that's the only way I was able to do it. If you want to point me to a copy of your bot, I might be able to help you implement it.

Adaptive dialogs automatically promote up adaptive card response as an intent if an intent = xxx is included in the data. See here for a code based sample

thank you @mdrichardson  and @vishwacsena

Adaptive dialogs automatically promote up adaptive card response as an intent if an intent = xxx is included in the data. See here for a code based sample

but in composer we can trigger this as an intent, only .lu format intents it will accept

@KoradaRajesh I had to add it as a separate trigger. It's not ideal and it's difficult to get working in your flow, but that's the only way I was able to do it. If you want to point me to a copy of your bot, I might be able to help you implement it.
That being said, the Composer team marked this as an [Enhancement], so they'll work this into an actual feature. Targeted release is in March.

@mdrichardson can you tell me the type of trigger i need to use for triggering the on submit of adaptive card

@mdrichardson  i have tried by triggering an activity on submitting the adaptive card but it's evaluating

@KoradaRajesh  I used Activity trigger with message type. I'm not sure why your error is being given, but maybe try switching turn.activity.text == null to !turn.activity.text

@mdrichardson  i have tried both but it throwing an error

did you kept any condition here

New Trigger
Trigger type: Activities, Activity type: Activities (Activity received), Submit
Click "Activities", leave Condition blank, type "message" for "Activity type"

If condition, Condition: turn.activity.value && !turn.activity.text

On the True branch, Send a response: -<EMAIL_ADDRESS>
That takes care of the adaptive card part. The trick is getting it to trigger something else if turn.activity.value && !turn.activity.text is false. For that, I have it launch a separate dialog on the False branch (which is a simpler copy of the RespondingWithCards sample):

Here's a link to my repo of this

I wasn't able to get this to work in a non-code, Compose-only example. @vishwacsena Is this possible?

@mdrichardson  it's working i'm able to get the adaptive card response in activity received trigger but for text message also it's triggering that activity i want to trigger an activity only submitting the adaptive card.
In this scenario it's failing to trigger the OnIntent because it is not going to match the user text with any recognizer and it triggering the Activity received trigger.

Hi @KoradaRajesh - Could you please let me know how you have configured the adaptive card json in the bot messages and reading the card in the Flow. I am trying to send an adaptive card but unable to load the JSON. Thanks in Advance.

@KoradaRajesh As far as I know (@vishwacsena, correct me if I'm wrong), there's no way to capture this through a prompt, since Adaptive Card input is sent through Activity.Value and not Activity.Text.
So, you need some way of capturing those. Here's what I did:
Adaptive card json:
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "TextBlock",
            "text": "Text Input - id: textInput"
        },
        {
            "type": "Input.Text",
            "placeholder": "Enter some text",
            "id": "textInput"
        },
        {
            "type": "TextBlock",
            "text": "Number Input - id: numberInput"
        },
        {
            "type": "Input.Number",
            "placeholder": "Enter a number",
            "id": "numberInput"
        }
    ],
    "actions": [
        {
            "type": "Action.Submit",
            "title": "Submit"
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.0"
}

Then, construct a condition with turn.activity.value && !turn.activity.text and output the result with<EMAIL_ADDRESS>

@mdrichardson - Could you please let me know how you have added the card JSON in bot messages as i am getting the Syntax error if i add the same json in my bot. PFB the screenshot for your reference.

Hi, I have configured the adaptive card json(input form) in the bot messages  but I couldn't read inputs from card when submit button is clicked. Thanks in Advance.

@prasadreddybv
I believe you need to format it more like:
@adaptivecardjson()
- ```
{
    "type"...

Note how you need the line: - ```
If that doesn't work, please open a new issue. I haven't worked with Composer is awhile and since it's in Preview, things may have changed.

@KoradaRajesh I had to add it as a separate trigger. It's not ideal and it's difficult to get working in your flow, but that's the only way I was able to do it. If you want to point me to a copy of your bot, I might be able to help you implement it.
That being said, the Composer team marked this as an [Enhancement], so they'll work this into an actual feature. Targeted release is in March.

Is this issue resolved? Can you tell me, how to get the value from action.submit in adaptive card?

@GowthamDisys see this sample. It shows both using adaptive card to collect input in an input action as well as to route to appropriate dialog. In short, adaptive dialog include a recognizer by default that transforms any payload posted by action.submit into intent + entities.
adaptiveCardHandling.zip

@vishwacsena  thats a great example.. is there an example for multiple textbox fields from adaptivecard?

@mganeshphani you can just add additional inputs. Just give them all unique IDs. Then you can access them via an entity @propertyId. Here is an example ..
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "TextBlock",
            "size": "Medium",
            "weight": "Bolder",
            "text": "Fill in the form"
        },
        {
            "type": "Input.Text",
            "placeholder": "Enter your name",
            "id": "userName"
        },
        {
            "type": "Input.Number",
            "placeholder": "Placeholder text",
            "id": "userAge"
        },
        {
            "type": "ActionSet",
            "actions": [
                {
                    "type": "Action.Submit",
                    "title": "Submit"
                }
            ]
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.2"
}

Then you can access the data returned by adaptive card submit via @userName, @userAge etc.

@vishwacsena where does the reading part go into?
I am going by template and would like to know where i can map @userName, @userAge to the properties in bot composer?

If you are asking for one piece of information at a time, you can simply set the value property in the input action to =@entityName.
If you are collecting multiple pieces of information, then it's better to just use set properties action right after the input to set them to what ever memory scope you want to set it to. E.g. user.name etc.

Got it.. thanks for the info. i will try.
On Wed, Sep 9, 2020 at 6:15 PM Vishwac Sena Kannan<EMAIL_ADDRESS>wrote:

If you are asking for one piece of information at a time, you can simply
set the value property in the input action to =@entityName.
If you are collecting multiple pieces of information, then it's better to
just use set properties action right after the input to set them to what
ever memory scope you want to set it to. E.g. user.name etc.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/BotFramework-Composer/issues/1820#issuecomment-689872517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFPMMIEIJQ3KEISUWTHRTJTSFAEC5ANCNFSM4KCFUCNA
.

--
Ganesh Phani Majeti.

I'm still trying to figure out how to collect multiple inputs.  The single userName works.  Adding an additional field and I keep getting an object error.

@vishwacsena thats a great example.. is there an example for multiple textbox fields from adaptivecard?

Could you provide an example gathering multiple inputs.  This works but for multiple you get an error with object to string conversions.

@vishwacsena thats a great example.. is there an example for multiple textbox fields from adaptivecard?

Could you provide an example gathering multiple inputs. This works but for multiple you get an error with object to string conversions.

@CopenhagenJames  as simple as this(this is just a modification of adaptivecardhandling example)

@vishwacsena thanks for the quick response greatly appreciated

I am using adaptive cards with Bot framework composer, I am facing an issue while submitting empty fields in the adaptive card.
How we can apply validation in adaptive cards with bot composer.

We ran into a problem executing your composer dialog. Can not convert Object to String.

Sorry, the bot can't talk for a while. It's something the bot's owner needs to address. Error code: 2023. Conversation ID: IT7ZlBeSrMGLRuFHaliH1X-c. Time (UTC): 4/29/2021 9:12:58 AM. Can not convert Object to String.
We ran into a problem executing your composer dialog. Can not convert Object to String.


Have a mock execute a transaction

I'm in a situation where I need a mock to execute a transaction.  Is there a way to do this?

You will probably need a custom built test contract for this use case.

So I'll need to extend the Doppelganger contract and add a helper JS function.  Right.


Fix assets paths for Whitehall and remove an unnecessary domain suffix.

The assets upload job was defaulting to nonexistent paths for the three Whitehall deployments. Since whitehall-admin, whitehall-frontend and draft-whitehall-frontend each run the same image (though not necessarily the same version) with a different .Release.Name and none of those names match the assets directory inside the image, we need to override it.
We still run the assets upload job independently for each of the three deployments because it's possible for them to be running different versions and therefore need to upload different assets files - even though the vast majority of the time it'll just be fruitlessly uploading the same thing three times.
Also clean up an unnecessary domain suffix while we're there.

Forgot to update values-test.yaml (force-pushed just now to fix).


adding or updating the resize plugin for mobile devices

Hi,
Firstly, thank you for your amazing work with jsPsych! I'm making more of a feature request – maybe with some time, I could try and contribute but it is currently not within my powers.
I'm working on building an experiment that works via browsers on mobile devices, and coming up against issues with how the stimulus is displayed in portrait and landscape mode. Unfortunately, it appears that portrait and landscape are treated differently on different phone systems (iOS and Android). I was hoping a fix would be to use the resize or browser-check plugins but the click-and-drag function (i.e. when matching the rectangle to the credit card) doesn't seem to work on mobile devices.
My idea is to add some sort of plugin that allows the user to simply click rather than drag to resize the rectangle (like they would in resize) to cover most of the screen. This could then maybe set CSS style for the jspsych-display-element or determine a scale-factor, allowing tasks to essentially scale to any mobile device screen regardless of their handheld orientation.
Sorry if this has been brought up elsewhere – I couldn't see to find any related issues on this. In the meantime, I'm happy on restyling the CSS/HTML of my stimulus to make it look okay on devices (hopefully!)
Kindly,
William

Thank you @DominiqueMakowski, that is helpful! I'm using canvas-button-response for most of what I am doing – my stimuli has a few moving parts – but I had not thought to use the width and height parameters from browser-check to maybe adjust my canvas. One feature of iPhone devices is that I think the browser full-screen doesn't close or hide the top toolbar – so I'm not sure if the window parameters adjust for that. A thing to try and test in my next study!


[BUG] Management.EventGrid Client Throws Null Reference When Deleting EventSubscription

Describe the bug
The management event grid client sdk version 6.0.0 is throwing a null reference exception when trying to delete an event subscription. The event subscription is successfully deleted, and when I hit "continue" on visual studio debugger, the code goes on as expected.
Expected behavior
The client.EventSubscriptions.DeleteAsync(scope, subscriptionName) should delete the event subscription without throwing null reference exception on the azure rest client.
Actual behavior (include Exception or Stack Trace)
Microsoft.Rest.ClientRuntime.Azure is throwing null reference exception even though I am able to successfully delete the event subscription.

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

On visual studio 2019, install management event grid package version 6.0.0.
using valid credentials, create an instance of the management event grid client
call the delete async call on the event subscriptions
run the code through debugger

Environment:

Name and version of the Library package used: [e.g. Microsoft.Managmenet.EventGrid 6.0.0]
Hosting platform or OS and .NET runtime version
PS C:\Users\marher> dotnet --info
.NET SDK (reflecting any global.json):
Version:   5.0.102
Commit:    71365b4d42

Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19042
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\5.0.102\

IDE and version : Microsoft Visual Studio Enterprise 2019
Version 16.8.4

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Looking at your repro step, it may because you have enabled debug exception setting to always break on NullRefException (catch first chance exception). This exception may be thrown due to probing for a few different supported LRO mechanisms. The exception is handled and will move onto other checks. If this is the case, it is normal.

Looking at your repro step, it may because you have enabled debug exception setting to always break on NullRefException (catch first chance exception). This exception may be thrown due to probing for a few different supported LRO mechanisms. The exception is handled and will move onto other checks. If this is the case, it is normal.

@allenjzhang , how do I go about disabling "catch first chance exceptions" to test your hypothesis?
Mario

@allenjzhang , how do I go about disabling "catch first chance exceptions" to test your hypothesis?
Mario

@allenjzhang, I enabled the "Just my code" debugging option in Visual Studio and proceeded to not hit the error.
Thanks for the help here.

@allenjzhang, I enabled the "Just my code" debugging option in Visual Studio and proceeded to not hit the error.
Thanks for the help here.


Fix position from vertex shader partially uninitialized

A vertex shader in Godot can write to a subset of the fileds of POSITION, which would lead to the vec4 position being partially unitialized. The DirectX Shader Compiler will reject that. This PR initializes the variable to the most sensible value so the user can re-override whatever members they want. Shader optimization is expected to remove the redundant assignments.
A different fix may be put in place: that the Godot shader compiler rejects the shader unless it writes to the whole vector. This PR shouldn't be an obstacle to doing that later anyway.

Thanks!


Fastlane fails to create an app on iTunes Connect

New Issue Checklist

[x] Updated fastlane to the latest version - Updated
[ ] I have read the Contribution Guidelines

Issue Description
Fastlane fails to create a new app using fastlane produce
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

Dev-Deployment-Mac-mini.local (14:54):
[03:58:31]: [DevCenter] App 'com.abc.xyz' already exists, nothing to do on the Dev Center
[03:58:41]: Creating new app 'XYZ' on iTunes Connect
Looking for related GitHub issues on fastlane/fastlane...
➡  uninitialized constant CredentialsManager::PasswordManager
https://github.com/fastlane/fastlane/issues/7653 [closed] 11 💬
3 weeks ago
➡  Fastlane crashes after updating to 2.5.0 on Linux Fedora 22
https://github.com/fastlane/fastlane/issues/7765 [closed] 12 💬
3 weeks ago
➡  Fastlane crashes when disabling colors
https://github.com/fastlane/fastlane/issues/8100 [closed] 3 💬
a day ago
and 5 more at: https://github.com/fastlane/fastlane/search?q=uninitialized constant Class::UI&type=Issues&utf8=✓
🔗  You can ⌘ + double-click on links to open them directly in your browser.
/Users/automation/.rvm/gems/ruby-2.2.3/gems/fastlane-2.28.8/spaceship/lib/spaceship/tunes/application.rb:81:in create!': [!] uninitialized constant Class::UI (NameError) 	from /Users/automation/.rvm/gems/ruby-2.2.3/gems/produce-2.0.0/lib/produce/itunes_connect.rb:25:in create_new_app'
from /Users/automation/.rvm/gems/ruby-2.2.3/gems/produce-2.0.0/lib/produce/itunes_connect.rb:12:in run' 	from /Users/automation/.rvm/gems/ruby-2.2.3/gems/produce-2.0.0/lib/produce/manager.rb:8:in start_producing'
from /Users/automation/.rvm/gems/ruby-2.2.3/gems/produce-2.0.0/lib/produce/commands_generator.rb:37:in block (2 levels) in run' 	from /Users/automation/.rvm/gems/ruby-2.2.3/gems/commander-fastlane-4.4.4/lib/commander/command.rb:178:in call'
from /Users/automation/.rvm/gems/ruby-2.2.3/gems/commander-fastlane-4.4.4/lib/commander/command.rb:178:in call' 	from /Users/automation/.rvm/gems/ruby-2.2.3/gems/commander-fastlane-4.4.4/lib/commander/command.rb:153:in run'
from /Users/automation/.rvm/gems/ruby-2.2.3/gems/commander-fastlane-4.4.4/lib/commander/runner.rb:476:in run_active_command' 	from /Users/automation/.rvm/gems/ruby-2.2.3/gems/fastlane-2.28.8/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:40:in run!'
from /Users/automation/.rvm/gems/ruby-2.2.3/gems/commander-fastlane-4.4.4/lib/commander/delegates.rb:15:in run!' 	from /Users/automation/.rvm/gems/ruby-2.2.3/gems/produce-2.0.0/lib/produce/commands_generator.rb:135:in run'
from /Users/automation/.rvm/gems/ruby-2.2.3/gems/produce-2.0.0/lib/produce/commands_generator.rb:12:in start' 	from /Users/automation/.rvm/gems/ruby-2.2.3/gems/fastlane-2.28.8/fastlane/lib/fastlane/cli_tools_distributor.rb:58:in take_off'
from /Users/automation/.rvm/gems/ruby-2.2.3/gems/fastlane-2.28.8/bin/fastlane:20:in <top (required)>' 	from /Users/automation/.rvm/gems/ruby-2.2.3/bin/fastlane:22:in load'
from /Users/automation/.rvm/gems/ruby-2.2.3/bin/fastlane:22:in <main>' 	from /Users/automation/.rvm/gems/ruby-2.2.3/bin/ruby_executable_hooks:15:in eval'
from /Users/automation/.rvm/gems/ruby-2.2.3/bin/ruby_executable_hooks:15:in `'
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
✅ fastlane environment ✅
Stack

Key
Value

OS
10.11.6

Ruby
2.2.3

Bundler?
true

Git
git version 2.10.1 (Apple Git-78)

Installation Source
~/.rvm/gems/ruby-2.2.3/bin/fastlane

Host
Mac OS X 10.11.6 (15G31)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.2.3/lib

OpenSSL Version
OpenSSL 1.0.2k  26 Jan 2017

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.2.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane gems

Gem
Version
Update-Status

fastlane
2.28.8
✅ Up-To-Date

deliver
2.0.1
✅ Up-To-Date

produce
2.0.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

executable-hooks
1.3.2

bundler-unload
1.0.2

rubygems-bundler
1.4.4

bundler
1.14.6

CFPropertyList
2.3.5

i18n
0.8.1

minitest
5.10.1

thread_safe
0.3.6

tzinfo
1.2.3

activesupport
4.2.8

public_suffix
2.0.5

addressable
2.5.1

babosa
1.0.2

claide
1.0.1

fuzzy_match
2.0.4

nap
1.1.0

cocoapods-core
1.0.0

cocoapods-deintegrate
1.0.0

cocoapods-downloader
1.0.0

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

netrc
0.7.8

cocoapods-trunk
1.0.0

cocoapods-try
1.0.0

colored
1.2

escape
0.0.4

fourflusher
0.3.1

molinillo
0.4.5

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.4.4

cocoapods
1.0.0

highline
1.7.8

commander-fastlane
4.4.4

fastimage
2.1.0

dotenv
2.2.1

excon
0.55.0

multipart-post
2.0.0

faraday
0.12.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
<IP_ADDRESS>

gh_inspector
1.0.3

jwt
1.5.6

little-plugger
1.1.4

multi_json
1.12.1

logging
2.2.2

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

hurley
0.2

mime-types-data
3.2016.0521

mime-types
3.1

uber
0.0.15

representable
2.3.0

retriable
2.1.0

google-api-client
0.9.28

json
2.1.0

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.3.0

rubyzip
1.2.1

security
0.1.3

slack-notifier
1.5.1

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

tty-screen
0.5.0

word_wrap
1.0.0

rouge
2.0.7

xcpretty
0.2.7

xcpretty-travis-formatter
0.0.4

generated on: 2017-05-05

Thanks for reporting this, I can see the problem, and will work on a fix 👍

This will be fixed with https://github.com/fastlane/fastlane/pull/9089/files

It seems like you're running an old version of fastlane. Can you please run gem cleanup and install fastlane again? Make sure that produce isn't installed any more.


Getting Started with Basscss

I'm trying to get to grips with Basscss to use it on a project I'm working on during the holidays. It's worth noting that it's my first time using Rework, but for some reason runs for cover I can't seem to understand how it all fits together even after reading through the docs. I wondered if someone on here could help answer a few questions?

Is gulp dev the command I want to run when developing a page? I ask this, because if I edit the ./index.html file and run gulp dev later, it reverts the changes I've made to the page. Is there a different command I should be running when developing?
While running the watch-css task changes made to ./src/basscss.css are saved out to ./css/basscss.css and ./css/basscss.min.css, I take it ./src/basscss.css is the primary CSS file I should be editing?
What's with the ./scss directory then?
What would be a typical workflow for using something like Basscss?

Sorry if these questions border on stupidity. I'd really appreciate if someone could help point me in the right direction. Any help is appreciated.
Thanks!

Thanks for shedding a bit more light on this for me, @jxnblk. I'm going to go through this right now.

Thanks for shedding a bit more light on this for me, @jxnblk.  But if I'm importing the Bower files like so:
@import 'basscss/scss/defaults';
@import 'basscss/scss/base-reset';

then what is the point of the files found in ./scss? And which scss files should I be making edits to? Surely not the ones found in bower_components if those are the ones being imported?

Not sure if I understand the first question, but those import statements are using the files in /scss so that's the point of them.
As a general rule, you shouldn't edit vendor libraries, NPM packgages, or Bower components. You might set up your own Sass base file like this:
@import 'my-variables';
@import 'basscss/scss/defaults';
@import 'basscss/scss/base-reset';
@import 'my-custom-styles';

If you want to edit files from Basscss, you'd have to manually copy and paste the files you want to edit, but then you lose the benefit of getting updates.
Let me know if that answered your questions.

Sorry I mean I've installed Basscss via Bower, and added bower_components/basscss/scss as an includePaths option. And I was importing them into my base Sass file like this:
@import 'basscss/scss/defaults';
@import 'basscss/scss/base-reset';

I then noticed there's an scss file in the root directory, so what's to stop me from importing these files over the files found in bower_components/basscss/scss?

Sorry for the slow response.
The bower_components folder should contain all the packages you've installed with Bower, including Basscss if you ran bower install basscss. Basscss only has one scss folder at its root, and those are the files you're importing. When you use the includePaths option, that just provides shortcuts to folders.
Just to clarify, did you use bower install basscss in a new folder or another repo (i.e. not a cloned version of Basscss)?


Explanation of project

I know it's early in the development, but I'm trying to figure out how this will work. Will it be so the pico can act as a bridge and/or a brain? I tested the code from last week on my custom brain, it compiles...but I didn't get any console logs out. Should it show up as a usb device yet? or console over uart?
This looks really exciting anyway, and I hope to contribute to is when I understand more about the goals.

Sorry for the delay. The pico will be running DeviceScript, see https://microsoft.github.io/devicescript .


speed up float/double parsing

use fast_float (conan) for parsing float and double. A private dependency should be fine

Now using std::from_chars which is based on fast_float in gcc and clang


How to update the weight of one class in another class

self.nn = update will raise a error.
How to change the weight of self.nn

I think the pattern you are looking for is some kind of partial update via filters.
You can hack things using object.__setattr__(self, 'nn', ...) but I suspect that is not what you should do longer term.

great thanks,
cottrell

Definitely please don't use object.__setattr__. You will probably get silently wrong results if you do.
eqx.tree_at is probably best. Note that this will return the modified result out of place, i.e. you must use its return value.

Do you guys know how to print the value of the error in my clip?

https://github.com/google/jax/issues/196


Console error

When I called 'php yii podium/queue/run' from command line (I was in the correct directory), I got

Error: Unknown command "podium/queue".

I edited console.php  as you said in installation wiki

Right. Forgot one thing in wiki.
In console configuration add 'podium' value in the 'bootstrap' key like:
'bootstrap' => ['log', 'podium'],

Should be ok with it.

Yes, that worked, thanks


Authereum Integration is using Kovan

Now that Authereum is released, the wallet integration should be updated to use the mainnet url. Currently, the url used for auth is kovan.authereum.com. It looks like this should be possible by updating the @web3-react/authereum-connector package to 6.0.8 and configuring it to use mainnet.

Closed by #21


Android Studio Chipmunk | 2021.2.1 Support

Short description
Last version of the plugin is not compatible with latest androii version Android Studio Chipmunk | 2021.2.1
So I just follow the workaround posted on https://github.com/mplushnikov/lombok-intellij-plugin/issues/1107
and I want to share the zip file for it for someone who need it.
File:
lombok-plugin (2).zip
Steps:

Open Android Studio, press Ctrl + Alt + S (or open settings via file menu), go to plugins
Click on the settings gear icon right to installed
Select "Install Plugin from Disk..."
Select the zip file and press ok.
Lombok is installed

Best Regards.

After log search i found this. Thanks @eriknyk

Thank you very much @eriknyk

thank you very much @eriknyk

thank you very much @eriknyk

thank you @eriknyk :)
@mplushnikov  Do you need some help on adjusting this plugin?

this works and help me solved a problem : when use the plugin( modified it`s limited version for compatibility) in AS-2021.3.1,the code completion does not work and there is no any code editing tips for some variable!


Dev wanting to help, but git clone fails because "This repository is over its data quota."

Cloning into 'MnB2-Bannerlord-CommunityPatch'...
remote: Enumerating objects: 319, done.
remote: Counting objects: 100% (319/319), done.
remote: Compressing objects: 100% (200/200), done.
remote: Total 7728 (delta 155), reused 231 (delta 117), pack-reused 7409
Receiving objects: 100% (7728/7728), 2.83 MiB | 8.53 MiB/s, done.
Resolving deltas: 100% (5246/5246), done.
Downloading tools/LocateSteamApp/Facepunch.Steamworks.Posix.dll (546 KB)
Error downloading object: tools/LocateSteamApp/Facepunch.Steamworks.Posix.dll (8ad78ee): Smudge error: Error downloading tools/LocateSteamApp/Facepunch.Steamworks.Posix.dll (8ad78ee417779e059d0dda560f0403efb61122c39169248fa011d116a4d6ae9c): batch response: This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.

Looks like this is how you can purchase more: https://docs.github.com/en/github/setting-up-and-managing-billing-and-payments-on-github/upgrading-git-large-file-storage
I'd be happy to pay for one data pack for the next two months ($5 for 1/mo)
This is probably the best way for me to help: https://github.myshopify.com/products/gift-card

Hello @TommasoPieroncini !
Please, tell if I understand this situation right:

You want to continue to work on the Community Patch by yourself -  to make the mod playable for the last update (and hopefully, for the upcoming updates - TaleWorlds seems to be more active in this direction in the lately)

But you need to pay for additional data massive to copy files to your github directory

So the right way will be to gather some people who is interested in this mod and split the payment for all. By the way, I'm in 👍

Sorry, if I was wrong in something, I'm not a developer myself and I don't know how GitHub works in this case.

If you're interested in working on this mod, contact me on Discord.
My contact information (as well as that of other authors) is in the Readme.
I can add you as contributors to the project.
In the meantime I'll mirror this to GitLab.

There's now an .lfsconfig in master that should solve the issue.

Mirrored master here:
https://gitlab.com/tyler-in/MnB2-Bannerlord-CommunityPatch

The entire project and LFS is mirrored there now.
Still try to use GitHub for everything, with the .lfsconfig you should be able to git lfs pull to verify you're pulling LFS from GitLab while pulling code from GitHub.

Pinging @TommasoPieroncini

Hi @Tyler-IN , actually none of the Discord links in the Readme work for me. And the cloning now fails with the following error:
Downloading tools/LocateSteamApp/Facepunch.Steamworks.Posix.dll (546 KB)
Error downloading object: tools/LocateSteamApp/Facepunch.Steamworks.Posix.dll (8ad78ee): Smudge error: Error downloading tools/LocateSteamApp/Facepunch.Steamworks.Posix.dll (8ad78ee417779e059d0dda560f0403efb61122c39169248fa011d116a4d6ae9c): batch request: Host key verification failed.: exit status 255

Also, your discord user links for some reason are non-functional


Feature Request: Auto detect previously used display and apply last setting

I use Mons on my laptop on a daily basis. I love the simplicity but I wish there was a way that it could be totally automated. I would appreciate if the -a option which daemonizes the process would also detect when a monitor was plugged in and set the display to some pre configured state, from either a configuration file or just by remembering how the user had it set previously.
I'm going to start perusing your code and see if there is a simple way for me to add this feature and submit a pull request. Let me know if this is something you feel fits your project and if so please suggest how you think this should be implemented.
Thank you!

I'm just commenting to share a script that I wrote to solve this issue for my specific use case. I have a udev rule that detects changes to my specific monitor and then calls this script. When two displays are detected, and it is not already in extend mode, it extends the display and moves all of my i3 windows over to the new display.
I doubt much of this will be useful for use in mons but it might give others who come here searching for a solution an idea of how they can solve it themselves. Thanks again for your awesome minimalist software! I am using mons, xpub, batify, and pug!
My script, called by my udev rule on display change:
#!/bin/bash
logfile="/tmp/setup-display"
if [[ -f "$logfile" ]]; then
	cat $logfile | tail -n10 > $logfile.tmp
	mv $logfile.tmp $logfile
fi

exec >> $logfile
exec 2>&1

sleep 3

mons_out=$(mons)
num_monitors=$(echo "$mons_out" | grep "Monitors" | awk '{print $2}')
mons_mode=$(echo "$mons_out" | grep "Mode" | awk '{print $2}')
edp=$(echo "$mons_out" | grep "eDP" | awk '{print $2}')
hdmi=$(echo "$mons_out" | grep "HDMI" | awk '{print $2}')

echo
date
echo "$mons_out"
if [[ $num_monitors -eq 2 ]]; then
	echo "Two displays."
	if [[ "$mons_mode" == "extend" ]]; then
		echo "Already extended."
		exit 0
	fi
	echo "Extending..."
	mons -e left
#	sleep .5
	while [[ "$ret" != '[{"success":true}]' ]]; do
		echo "Moving workspaces to output $hdmi..."
		ret=$(i3-msg "[class=\".*\"] move workspace to output $hdmi" | grep '"success"')
		#sleep .25
	done
else
	echo "One display."
	mons -o
fi

Thanks @AdamSLevy for contribution.
I've just read your answer. I'll handle it next week.


Fixed typo for Back to the Future reference

The reference of Back to the Future fell a little flat when the typo "capitor" stood in for "capacitor". Don't worry Doc didn't see it, yet!
Before submitting a pull request, make sure it's targeting the right branch:

For fixes to Ink 1.0, use master.
For fixes to Foundation for Emails 2, use v2.0.

Happy coding! :)

Hi @david-gagne,
thank you for your contribution to Foundation Emails.
We appreaciate it.


Create "Implementing Keptn Applications" section

Goal
This is a section that will have guide chapters about specific use cases that can be implemented, special topics and facilities that are useful, and so forth.
Problem
New users need material that puts together the whole process of implementing a Keptn Application or using specific facilities.  These guide pages can point to various CRDs that must be defined and other actions that are required and include guidelines for "best practices".
Technical Details
Some starter information is included in the existing documentation, we can glean more from video demonstrations that we have, and some information needs to be written fresh.  We will start by pulling together the information we can and then grow the content over time as we learn from user experiences.
DoD
The initial implementation will include the following pages; order may change:

Implement a Keptn application (end-to-end list of required and optional tasks to do to implement an application).  This is the text on the landing page for the "Implementing..." section with links to pages that have more details
Deployments, workloads, and Kept applications -- give a somewhat high-level albeit technical overview of these three entities and how they fit together.  The purpose of this section is to give the "big picture".  It should link to other Keptn and Kubernetes docs for details
Implement Observability

Evaluations -- What Keptn provides "out of the box", how to view results, how to expand that functionality
Site metrics -- How to implement some basic metrics, examples of more sophisticate metrics
Visibility -- Discuss use of k8s results and dashboards such as Grafana to view results.  Link to details about installing Grafana and other dashboards in the "Installation" section.
Tracing -- How to implement, give some basic and some more interesting examples

Working with secrets
Working with contexts
Implement Slack notifications

### Tasks
- [ ] https://github.com/keptn/lifecycle-toolkit/issues/1200
- [ ] https://github.com/keptn/lifecycle-toolkit/issues/1201
- [ ] https://github.com/keptn/lifecycle-toolkit/issues/1202
- [ ] https://github.com/keptn/lifecycle-toolkit/issues/1203
- [ ] https://github.com/keptn/lifecycle-toolkit/issues/1205
- [ ] https://github.com/keptn/lifecycle-toolkit/issues/1468

@StackScribe is something more needed to be documented? AFAIK this should be done

Yup, this is done.  The section now has additional pages and has been renamed to "User Guide"


Feature Suggestion: Enabling the execution of node commands from the host with 'docker-compose exec'

Info:

Docker version 18.03.0-ce, build 0520e24
Laradock commit: 8a090f3e46f4d04f9111952b5d12f2e2c80ff7b9
System info: Linux
System info disto/version: Ubuntu 16.04

Issue:
When trying to execute npm commands in the workspace container by using docker-compose exec workspace npm ..., you get the following error:
OCI runtime exec failed: exec failed: container_linux.go:296: starting container process caused "exec: \"npm\": executable file not found in $PATH": unknown

WORKSPACE_INSTALL_NODE is set to true, and I see the relevant output when building the workspace image.
For it to work, you have to docker-compose exec workspace bash into the container beforehand.
You get the same behavior when trying to run any node module command (bower, gulp, etc).
After some probing around, I realized this issue came from the fact that the docker-compose exec command somehow uses the host's PATH environment variable instead of the container's.
After I manually added the path to the '/home/laradock/.nvm/versions/node/v9.11.1/bin' folder in the host's PATH, then things started working. However, using this solution feels a bit dirty.

Expected behavior:
You should see the output of the command running in the container. For example, you should see something like 5.6.0 when you run docker-compose exec workspace npm -v

Reproduce:
After a fresh install, run the containers with docker-compose up -d nginx mysql.
Then, try to get the 'npm' version with 'docker-compose exec workspace npm -v'.

Feature Suggestion:
To fix this, I added symbolic links in 'usr/local/bin' directly pointing to the node modules executables. I added the following code into the workspace Dockerfiles. I tested it on mac os X and Ubuntu 16.04
Relevant Code:
# Make it so the node modules can be executed with 'docker-compose exec'
# We'll create symbolic links into '/usr/local/bin'.

Thanks.
I'm in the process of creating a pull request. I'll get to it when I have some free time.

+1 please merge this =)