Ability to use Extractors outside of patterns

Describe the problem you are trying to solve
Extractors offer a lot of good functionality like the ability to convert a string of JSON into a JSON object/record, or the ability to Dissect parts of some string into constituent parts. But they are currently limited to use only in patterns within match/case blocks.
Describe the solution you'd like
The ability to use Extractors outside of just patterns in match/case blocks. Perhaps something like the ability to simply assign the output of the extractor to a field like so:
let event.message = { "test": "{ foo:bar, snot:badger }" }
let foobar = %{ event.message ~= json|| };

Just an example, please consider as pseudo-code!

An alternative can be to have function equivalents for all extractors, like how we had for datetime parsing:
https://docs.tremor.rs/tremor-script/extractors/datetime/
https://github.com/wayfair-tremor/tremor-www-docs/blob/v0.7.0/docs/tremor-script/functions/datetime.md

Hi, yes we've been thinking about that it comes with a few complications. Part of extractors is their characteristic of matching, all of them have the ability to fail a pattern.
To take the example above (ignoring syntax) it creates a problem when event.message isn't a JSON.
It could return false or null but both of them also could be returned from a valid JSON so that's not really a good solution.
Alternatively, the script could simply fail that would be a safer option and perhaps make sense together with some kind of error handling as in #394?

I've been thinking about this a bit and am wondering if there is an easier improvment possible without introducing a type system.
Would lifting extractors from inside of a pattern to a pattern give you the tools you need to achive this? In other words something like:
let foobar = match "{ foo:bar, snot:badger }" of
  case res = json|| => res,
  default => null
end;

Allowing extractors to be used as described above does help, I think!

The suggested improvement is implemented.


Since carsidock is so powerful, could the author provide the training script so that we can train the model ourselves？

I would be very grateful if you could provide the training script, and its use will be non-commercial.

Thank you for your interest in CarsiDock. There is no plan to open source the training code yet.


allow tracing models on meta device

We want to facilitate running models that can only fit into memory after transforms. The current main thing in PyTorch to instantiate a model without memory is through meta devices.
So the plan is to enable:

Trace through model on meta device with real input (say on GPU).

This runs into trouble when checking for "same device". We currently have a few hacks around it here and there, but it would be good to have a more structured way.
My idea would be to add __thunder_device to the meta tensors (and/or maybe as a default option for compilation) and then use that device for proxying meta tensors.

Have transforms handle things on meta, ideally staying on meta. (I'll work this out for the bnb quant, and then we can see if/how to adapt the distributed transforms.)
Have a materialization transform that materializes models, optionally with initialization from the original code (using the transform_state_dict_for_submodule).

cc @apaz-cli

I see this as the materialization transform:
@requiresCUDA
def test_materialization():
    from thunder.transforms import MaterializationTransform

    config = litgpt_model.Config.from_name("llama2-like")
    with torch.device("cuda"):
        ref_m = litgpt_model.GPT(config).to(torch.bfloat16)
    with torch.device("meta"):
        m = litgpt_model.GPT(config).to(torch.bfloat16)
    for p in m.parameters():
        p.__thunder_device = torch.device("cuda")
    for b in m.buffers():
        p.__thunder_device = torch.device("cuda")

    init_from_sd = MaterializationTransform.from_original_state_dict(ref_m)
    jm = thunder.jit(
        m,
        transforms=MaterializationTransform("cuda", init=init_from_sd)
    )
    x = torch.randint(1, 255, (1, 10), device="cuda")
    input_pos = torch.arange(10, device="cuda")

    expected = ref_m(x, input_pos)
    actual = jm(x, input_pos)

    assert_close(actual, expected)

wdyt?

Looks super neat, to simplify we could also do
    jm = thunder.jit(
        m,
        transforms=MaterializationTransform("cuda", init=ref_m.state_dict())
    )

I thought that would be neat, but this is what kept me from offering:
we could have

no init (but garbage on first run, so only for people who do evil stuff between creating traces / prologue and running),
orig module init (through sd transforms),
orig state dict init (through sd transforms),
thunder module state dict.

And there we have two state dicts I would not know how to differentiate.

aah makes sense

With #867 and #868 we have initial support, but three of the four modes are yet to be fleshed out:

[ ] no init (but garbage on first run, so only for people who do evil stuff between creating traces / prologue and running),
[ ] orig module init (through sd transforms),
[x] orig state dict init (through sd transforms) included in #868,
[ ] thunder module state dict.

What's the benefit of allowing meta device tensors interact with other devices and not error out? What could be the alternatives? Have you considered using PyTorch's FakeTensor for initialization instead of plain meta tensors?

The advantage is to be able to trace cuda (or CPU) inputs with a network with meta weights and then transform the traced module to have cuda (or CPU) weights.
Fake tensors might work, too, but it seems that meta tensors give the best user experience for now.

This has now been achieved.


openuv: Error on device update!

Home Assistant release with the issue:

0.99.3
Last working Home Assistant release (if known):
0.99.2
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io
Component/platform:

OpenUV
Description of problem:
After the restart while upgrading to 0.99.3 this error shows up:
2019-09-27 08:25:48 ERROR (MainThread) [homeassistant.components.binary_sensor] openuv: Error on device update!
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 292, in _async_add_entity
    await entity.async_device_update(warning=False)
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 439, in async_device_update
    await self.async_update()
  File "/usr/src/homeassistant/homeassistant/components/openuv/binary_sensor.py", line 107, in async_update
    parse_datetime(data["from_time"])
  File "/usr/src/homeassistant/homeassistant/util/dt.py", line 125, in parse_datetime
    match = DATETIME_RE.match(dt_str)

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

Thanks!
Tested and working, after the restart the error is gone, while it would always show up mostly after every restart


Initial setup for packet clearing command

Closes: #158
Description

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[ ] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).
[ ] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

The following additional flags are currently required:

client the client ID on the source chain
counterparty-client the client ID on the destination chain
counterparty-channel the channel ID on the destination chain
counterparty-port the port ID on the destination chain

The optional flags are currently not implemented


Bracket Pair Colorization: Getting Started View

Once https://github.com/microsoft/vscode/pull/129231 lands, I think it would be nice to mention it in the Getting Started view.
Bracket Pair Colorization will be disabled by default and requires users to explicitly enable it, so I think many users might discover this functionality in the getting started view if it is mentioned there.

I believe that we should enable it by default, at the same time, also  ask users to make a discission whether to disable those extensions which have same feature like BPC/2, or maintain the present status

Any thoughts about just enabling by default in Insiders at least? We could always turn it off before we ship if we aren't confident in it yet. We do that a lot with new features for increased awareness.

A category dedicated to "These are some settings that we think you might want to tweak" would be nice to have, this would be a good contribution there.
Agree that enabling it by default on insiders makes good sense

This would fit nicely under the "Boost your productivity" walkthrough

Now that the feature is available in the Insider build, I think it makes sense to tackle this issue.
What do I need to do for this?

You can just add any entry here:
https://github.com/microsoft/vscode/blob/0214d59846f265d9e50f53b61e2267407d81cbfb/src/vs/workbench/contrib/welcome/gettingStarted/common/gettingStartedContent.ts#L284
You'll want to have a themed SVG to go with it, and example is here: https://github.com/microsoft/vscode/blob/920303b974e9ac5b9f7d30d5bc354db5a849c793/src/vs/workbench/contrib/welcome/gettingStarted/common/media/settingsSync.svg#L4
You can either make that yourself in {vector image editor of your choice} or ask someone on the design team nicely 😉
Also FYI the bracket-pair-colorization label should be colored c5def5, both to visually go with the other sub-feature-areas and because some tooling (for instance the triage bot) use colors as a tagging mechanism.

I'm not 100% sure where to mention bracket pair colorization options.
Shall I just add another option (e.g. "Enable Bracket Pair Colorization") here:

?
However, @isidorn had the interesting idea of adding an entire section for editor related stuff (which also includes multi-cursor stuff)

Two more things I mentioned in our chat (just to have it here)

Think about adding it to the Commonly Used section in the Settings UI fyi @roblourens
Add it to the tips-and-tricks section of our docs https://code.visualstudio.com/docs/getstarted/tips-and-tricks

Now that I think about it a bit more I am not 100% certain this should be in the Getting Started, because of course a user can get started without this feature. It is just a nice feature on top. So depends on how much we want to have "Tips and Tricks" in the "Getting Started" and how much we want it to be simple flow such that new users can just start coding.
My point is: we should market this cool feature, and maybe we should all the other things (twitter, tiktok, docs, blog) except getting started.

I'm open for not including it in this welcome screen.

Going to re-open with (new) tips and tricks label for tracking if we do add a dedicated "tips and tricks" section in the future (which I'd like to do).

Would be nice if we had a "tips & tricks" section that appeared after some time (like let's say 3 weeks) so that we don't overwhelm the walkthrough content as they start to accumulate.

This is enabled by default now so does not need the entry.


Implement users uniqueness check functionality

While creating a new user (i.e. during signup), the handler now checks if there is a user with the same username and mail in the database. If the user exists, the signup process will be canceled and the following response sent back:
# Command
curl -i -X POST -d '{"username":"samba"<EMAIL_ADDRESS>"name":"klikle", "surname":"lile", "password":"hidden"}' http://localhost:3500/users

# Response
HTTP/1.1 406 Not Acceptable
Vary: Cookie
Date: Wed, 07 Dec 2022 12:44:57 GMT
Content-Length: 45
Content-Type: text/plain; charset=utf-8

user with that username or mail already exits

It looks like backend don't recognize that user exist, pls fix fast or we will miss deadline
matteolovric@Matteos-Air digitalpaper % curl -i -X POST -d '{"username":"samba"<EMAIL_ADDRESS>"name":"klikle", "surname":"lile", "password":"hidden"}' http://localhost:3500/users
HTTP/1.1 200 OK
Date: Wed, 07 Dec 2022 16:47:09 GMT
Content-Length: 25
Content-Type: text/plain; charset=utf-8

User created successfully%
matteolovric@Matteos-Air digitalpaper % curl -i -X POST -d '{"username":"samba"<EMAIL_ADDRESS>"name":"klikle", "surname":"lile", "password":"hidden"}' http://localhost:3500/users
HTTP/1.1 200 OK
Date: Wed, 07 Dec 2022 16:47:10 GMT
Content-Length: 25
Content-Type: text/plain; charset=utf-8

User created successfully%


Repo & NPM Version mismatch with lagging peer dependency

I was installing the graphql-server-hapi@latest (0.5.2) with<EMAIL_ADDRESS>and the latest npm version doesn't match the current repo version. I'm specifically looking at the peer dependencies as the npm version doesn't allow ^0.9 graphql.

np about that @mrlannigan
the fix for that is already merged to master and we are planning on releasing a new version today. ;)

Today I installed graphql-server-express and it gave me this error:
<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^0.7.0 but none was installed.
The installed graphql version is 0.9.1...
Probably the fix refered by @DxCx  wasn't released yet...

hi @marciobueno1,
the fix was released, however, graphql-server is deperecated...
please look at the instructions for the right package name ...

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^0.7.0 but none was installed.


Line breaks not honoured

Consider this markdown:
Legenda:\n\n**H**: Hidratações\n**N**: Nutrições e umectações\n**R**: Reconstruções

The expected output would be:

Legenda:
H: Hidratações
N: Nutrições e umectações
R: Reconstruções

But markdown_viewer render it without the line breaks:

(BTW, double line breaks are NOT a solution, because they add a second line-break between the items:)
Legenda:

**H**: Hidratações

**N**: Nutrições e umectações

**R**: Reconstruções

This markdown_viewer follows the CommonMark specification. You have to type at least 2 spaces or add a \ to the end of a line in order to produce a hard break:
https://spec.commonmark.org/0.30/#hard-line-breaks

Ok.
It was confusing because GitHub markdown doesn't work that way. Maybe some documentation about hard breaks? =)

Maybe I should render the softbreak as a line break too. I will enable this feature by default.
BTW, I am busy at another project at this moment, I was working on a Notes app, this is the reason why I created this Markdown thing, but this Notes project has been paused.

BTW, I am busy at another project at this moment...

That's on you. You made a better markdown widget than Google official one, so, suffer the consequences >=)


Update README.md

Coverage remained the same at 96.651% when pulling 1198bde2bc1f4b4d785b6bf6369c273afc0352a9 on update_readme into cfbc42f05a804aa2c9d61c8acbdd53736e508460 on master.


Bug#277

Hello,
I'd fixed the problem found in the issue #277.
Would you like to review and merge it into the master?
My changes:

Bugfix in the ResourcesRepackStep.
New Integration Tests Scenarios to test the Bugfix:
2.1 WindowsFormsTestNetCoreApp - new NET Core WindowsForms application with embedded resources.
2.2 ClassLibraryCore  - new NET Core class library with WPF user control using embedded resources .
2.3. WPFSampleApplicationCore - new NET Core WPF using the user control from ClassLibraryCore.
I've also fixed the new problem in integration tests - Nuget download doesn't work without SecurityProtocolType.Tls12 anymore. In order to use SecurityProtocolType.Tls12 the integration tests had to be bumped to net472.
I've also added Nuget reference to NUnitTestAdapter to run tests in VS 2019.

Best regards,
Yury Malich

"All checks have failed"

Error during project file parsing: Microsoft.Build.Exceptions.InvalidProjectFileException: The SDK 'Microsoft.NET.Sdk.WindowsDesktop'

Hm.. It seems Gradle must be updated to build new SDK-style NET Core projects.

Hm.. It seems Gradle must be updated to build new SDK-style NET Core projects.

Hmm.. may be updating the com.ullink.msbuild to 3.8 can fix it

I have fixed the build in #281

Great! Thanks!

Hi,
I've merged it from master. https://github.com/gluck/il-repack/pull/284
It looks like two unit tests doesn't pass. I'll take a look but I recall GivenNetCore3WinFormsAppUsesImageResources_MergedCore3WinFormsApplicationRunsSuccessfully was failing prior to the merge. Maybe some ideas ?
Also, I've an issue with System.Resources.Extensions being repacked but at rutime it is still trying to load it so I've a FileNotFoundException. I'll take alook when I've time.
Regards,
Florian

Hi,
I've merged it from master. #284
It looks like two unit tests don't pass. I'll take a look but I recall GivenNetCore3WinFormsAppUsesImageResources_MergedCore3WinFormsApplicationRunsSuccessfully was failing prior to the merge. Maybe some ideas ?
Also, I've an issue with System.Resources.Extensions being repacked but at rutime it is still trying to load it so I've a FileNotFoundException. I'll take a look when I've time.
Regards,
Florian

Hi, Florian.
It was green on my side before merge.
I'm merging the latest master to my local branch right now, to fix the broken build/test.
I'll push my changes to update the pull-request then.

Hi,
I've merged it from master. #284
It looks like two unit tests don't pass. I'll take a look but I recall GivenNetCore3WinFormsAppUsesImageResources_MergedCore3WinFormsApplicationRunsSuccessfully was failing prior to the merge. Maybe some ideas ?
Also, I've an issue with System.Resources.Extensions being repacked but at rutime it is still trying to load it so I've a FileNotFoundException. I'll take a look when I've time.

Hi Florian,
I've merged the master-branch to my local branch, and fixed one blinking unit test.
All test are green right now on my side in Microsoft Visual Studio Community 2019 Version 16.8.6.

I've fixed the other unit test. https://github.com/gluck/il-repack/pull/286
I've VS 2019 as well but the pro. Maybe it depends on the msbuild installed.

Thank you all for the contributions. This has now been merged into master:
https://github.com/gluck/il-repack/commit/f808db71412282a5fe0e0cec8316d4a4d6c1aadb
https://github.com/gluck/il-repack/commit/01af4afe308ee3ff48401721931787d6f1f49eae

Thank you all for the contributions. This has now been merged into master:
https://github.com/gluck/il-repack/commit/f808db71412282a5fe0e0cec8316d4a4d6c1aadb
https://github.com/gluck/il-repack/commit/01af4afe308ee3ff48401721931787d6f1f49eae


Display execution times and periods on DAG plot

This can be done through assigning execution times as weights of edges.


Do not overallocate for tmp.ciphers_raw

Well, not as much, at least.
Commit 07afdf3c3ac97af4f2b4eec22a97f7230f8227e0 changed things so
that for SSLv2 format ClientHellos we store the cipher list in the
TLS format, i.e., with two bytes per cipher, to be consistent with
historical behavior.
However, the space allocated for the array still performed the computation
with three bytes per cipher, a needless over-allocation (though a relatively
small one, all things considered).

Checklist

[x] CLA is signed

Description of change

For 1.1.0 as well, it looks like.  @mattcaswell did the original; please take a look.
This would have been a one-line change but for that "TLS_CIPHER_LEN" is a lot longer than "n" and made the line too long.

Pushed. Thanks.


[edge] Integration for edge invitations

Add an integration for /spaces/:spaceId/join edge endpoint: when client starts guest invitation flow try contacting edge for admission.

https://github.com/dxos/dxos/pull/7941#


Expose Go memstats as metrics.

This exposes metrics for the properties available through here so that we can get some better visibility into things like GC pressure.
Here's an example of Prometheus showing HeapAlloc for our controller process:

cc @Harwayne @mdemirhan


Correct the spelling of the "curriculum" on Main page

Current Behavior

Correct the spelling of the "curriculum"
Expected Behavior

ciricullum -> "curriculum"
Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@UtkarshMishra12 can you assign me this issue

@leecalcote @Krishna2323 @Harshit-Dandriyal  Please assign it to @PanyalaSainathReddy

@UtkarshMishra12, I don't have access to this repo.


Can't access custom attributes

I just added a couple custom attributes for testing to the user pool via console, and manually added values for those attributes for a test user as well. However, when I run
u = Cognito(COGNITO_USER_POOL_ID,
                      COGNITO_CLIENT_ID,
                      COGNITO_REGION,
                      access_token=access_token)

user = u.get_user()
print(user._data)

I only see
{'phone_number': 'xxxxxx', 'email': 'yyyyyyy'}

I'm expecting to see a couple more attributes such as custom:uploads and custom:paid, but can't see them...
I think I've heard some stories that it takes 24-48 hours for cognito to update? Do I just need to wait that long or am I doing something weird here?

 Just changed those settings in the attribute read/write permissions settings page.


Enhanced Contact us page

Description
I have enanced the contact us page in following ways :

Added a gradient button
Added dark mode for contact page
Added border
fix the positioning issues with overlappign of message and textarea

Fixes #849
SIr i created contact us page as you said and also i fixed the navbar links in this pr so if you find level 3 suitable for this please
Screenshots
https://github.com/Counselllor/Counsellor-Web/assets/118350936/2e0eaf41-913c-49a0-a82c-b814f56603dd
Checklist

[ ] Tests have been added or updated to cover the changes
[ ] Documentation has been updated to reflect the changes
[ ] Code follows the established coding style guidelines
[ ] All tests are passing

Remove the bubble and remove the black border and outer blue border should be same in size

Remove the bubble and remove the black border and outer blue border should be same in size

Ok

@thestarsahil  Sir i am little bit confused like which black border i have to remove

This one or

this one

and for outer blue border are you talking about this

I am talking about both
Remove the black border and reduce the size of blue outer element in message box
Inside the text element should should some gap

@thestarsahil  Ok

@thestarsahil

Like that

Not looking good Can you try to make this same as ?

@thestarsahil  yes sir sure I can make it but can you please merge this I will surely make new pr for that because you know I am working from 2 days and I had given most of my time I hope you understand sir I will create new form exactly same as you want till now please merge it

How could i merge this ?
This changes are not even working

@thestarsahil  Sir can you please check is the mail you received or not i fixed its working

Due you fixed it ?

Yes sir i fixed this But please check once it is working in my laptop

@thestarsahil  Yes sir i fixed that


Navigation Panel (Footer)

As a User I want to see the navigation panel so that I can choose to appropriate page.
Mockup
Acceptance Criteria:

.

Assumption and Constraints:

main navigation panel`s been created.


Remove use of requestAnimationFrame from setState.

This is causing flakes/timeouts on travis for ngmodel_test and related.
As discussed offline, this is also not testable (and causes other flakes and timeouts internally), and we don't have any stabilizers or special logic for this. I recommend just a scheduleMicrotask.

I'm not working on setState.

Fixed internally by @ferhatb.


viosock: select IO control should return with 'Information' in success

When select called to poll events of vsock socket, there is a large chance of failure with WSAEINVAL code, especially with timeout param. The trace events gives "Invalid output len" output, which help us to find this bug, that IOCTL_SELECT should always return with correct 'BytesReturned' in success.

Thanks for sending the fix!


Refactored the Description/AddDescription to a clearer system

The end goal is to have this default description :

A user might want the ability to disable or override it.
Perhaps providing a route-level option to disable it would be a better solution?

I was more asking about the naming of the function. Why change to Add vs just Description. You seem to be overriding.


The behavior of model_checkpoint callback with save_last=True

I have three questions about this. The doc says..

save_last (Optional[bool]) – When True, always saves the model at the end of the epoch to a file last.ckpt. Default: None.

I assumed that if I pass, say, filename='last_model.ckpt' with save_last=True, [1] it would override the default filename for the latest checkpoint with what I pass.
Instead, it behaves differently. After epoch 1, [2] it saves two files.
last_model.ckpt
last.ckpt

[3] After epoch 2+, the callback overwrite last.ckpt and i think it makes sense. it updates last_model.ckpt with the latest model and that makes sense. BUT - then, last_model.ckpt is renamed to last_model-v0.ckpt. So at the end of training, I have two files.
last_model-v0.ckpt
last.ckpt

So I have multiple questions on this behavior.
On [1], wouldn't it make sense to use the passed filename instead of the default last.ckpt?
On [2], or if it ignores the passed filename, probably shouldn't it only saves last.ckpt?
On [3], the renaming behavior is unexpected to me and it seems a little strange. Is it intended?

On [1], additionally, I believe it actually has to use the passed filename because one could want to use two callbacks, one for the model and the other for the weights only, of course with different names.

Ok, I observed another default behavior that if I don't specify filename, it automatically saves another file (that probably has the latest model) with a name like epoch=2-step=5624.ckpt. And this is somehow explained by the doc saying

By default, filename is None and will be set to '{epoch}-{step}'.

. But I still don't get why it saves two different files for the same, latest model. Is it an intended behavior?

the renaming behavior is unexpected to me and it seems a little strange. Is it intended?

the renaming is done to ensure that files are not replaced in case same filename is already present in the directory if save_top_k > 1 or -1.. if it's 1, then it will overwrite the last_model.ckpt and won't add a version to it. This is fixed on master.

wouldn't it make sense to use the passed filename instead of the default last.ckpt

last.ckpt will be saved only when you specify save_last=True, mainly it's used to save the checkpoint for last epoch in case the training crash, to ensure the progress isn't lost. Additionally, if you specify save_last=True, it will ensure to save the last checkpoint for you along with other regular checkpoints.

Thanks for the answer. I'll take it as "save_last means whether the last model is going to be saved additionally". Maybe I can make a small PR about the documentation.


Poor performance because wrangler is not cached or in-lined

This action invokes npx wrangler via the shell: https://github.com/cloudflare/pages-action/blob/0fc25c3c282fb26e9a53abfadf6d2898adcac3de/src/index.ts#L37
This causes NPM to re-download and re-install wranger on each invocation of this action.

Notice how there is an NPM warning that it will need to install wrangler.
We're considering migrating from Netlify to Cloudflare Pages - to deploy the same site, nwtgck/actions-netlify takes 2 seconds while this action takes 19 seconds.

If wrangler was in-lined / cached by this action, the deployment would take much less time to finish.

Once we have more of the Wrangler API built out we could use it as a dependency + bundle it into the action. Right now, however, we cannot.
We could theoretically use cache here, I'm not too sure it'll make a big improvement. 19 seconds is still a pretty short time.
I'll leave this open to consider though.

Unless I'm missing something:
Given the Wrangler CLI is already an NPM package, is there any reason it doesn't export its main program to be invoked programmatically? It wouldn't increase the API surface of that package at all, but it would allow this package to import main and call it directly, at which point it would also allow this package to esbuild that package into the shipped JavaScript file. It would also resolve #74 and #75.


Can't compile APPX with VS2017 installed (and no VS2015) unless directly using VS2017 Developer cmd.exe start menu shortcut.

I receive an error about missing the Windows Mobile SDK 10.0.15063.0 (eg Windows 10 Creator Update version) even if the SDK is installed.
The issue seems to be that the HoloToolkiit-Unity is trying to use MSBuild v14 (which is present) but the Windows 10 SDK download page specifically says the 15063 SDK is not usable in VS2015 (v14?) or earlier versions.
If I manually invoke the build process using MSBuild v15 from a VS2017 developer cmd.exe (using the start menu shortcut) the APPX build works fine.
If I try to switch to MSBuild v15 using the HoloToolkit-Unity UI it can't find it. It seems like it is looking for a registry key which does not contain an entry for v15. I have to manually patch the code to hardcode the location (in C:\Windows\Microsoft.NET\v4.0.XXXX whatever\MSBuild.exe).
However there are still further issues since it still won't compile using HoloToolkit-Unity as I receive the error: MSB4019: The imported project "C:\Program Files (x86)\MSBuild\Microsoft\WindowsXaml\v11.0\Microsoft.Windows.UI.Xaml.CSharp.targets" was not found. There must be something else the Developer console shortcut for VS2017 is doing that HTU needs to do.
(It may be relevant that I am targeting XAML in Unity for Hololens keyboard support.)
I am running Unity v5.6.1 on Windows 10 x64 Build 14393.1198. (This is a work PC and that is the newest approved version of Windows to my knowledge.)

After lots of experimentation it seems you NEED VS2015 plus the VS2015-bundled Windows 10 SDK versions. Unity will display newer SDK versions in the build settings but if you select one of these the build fails.

Thanks for the report.  This has been an issue for me for a few weeks while trying to migrate to VS2017 as well.


Environments updated to RC1 and addressed feedback

Fixes #290

@rachelappel can you also merge into master?


Fix: updated category logic on backend for credit trades, approved category text - 2472 2503 2470

xls download sheet, category column is empty. Should display proper category for approved transactions.


Show livemarks button directly on feed URL pages

It would be useful if the Livemarks button was displayed directly on feed XML pages, so users can manually add a feed if the feed_finder did not detect the feed, or the user accesses the feed URL directly.

https://feeds.feedburner.com/GoogleBlog
Expected: Page detected as feed, Livemarks button in address bar.
Actual: No livemarks button
Thanks

When Firefox actually drops support for the pretty "Subscribe to this Feed" page, this will start working.

#45 got this working on some pages. This particular page https://feeds.feedburner.com/GoogleBlog doesn't work, because it uses Content-Type: text/xml and XSLT. We do show the feed icon in the URL bar.

I actually wonder if we should special case feeds.feedburner.com, it seems to be one of the most popular feed websites. Google including blogspot uses it.


Refactor pre-programmed configuration options.

What configuration options? Cache config? Others @llayman

Better captured in issue #118


Create coding guideline document

Noticed this ticket. Just wanted to say when the coding guidelines are documented, we should mention that all the code files should contain the following lines ideally at the top of the source files as a comment block like this:
// Copyright 2023 Craftsmen Ltd
// SPDX-License-Identifier: Apache-2.0

Noticed this ticket. Just wanted to say when the coding guidelines are documented, we should mention that all the code files should contain the following lines ideally at the top of the source files as a comment block like this:
// Copyright 2023 Craftsmen Ltd
// SPDX-License-Identifier: Apache-2.0

is this needed to put the file contents under the  mentioned license? or having the LICENSE.md cover all the files in the repo?

is this needed to put the file contents under the mentioned license? or having the LICENSE.md cover all the files in the repo?

I think from the repo standpoint the LICENSE does cover all the files but when someone from outside will try to re-use our code say some files s/he will automatically get the copyright on the files as well. It's more like a recommended practice other open source projects follow so it makes sense to try to follow that.


Delete ingested Proqest MMs for BagIt error clean up

We are looking at recent Proquest errors that need cleaned up. We've once again run into the BagIt error issue (Exception in thread "main" java.lang.NumberFormatException: For input string: "hin-wel-all-00001102"). I've written this issue up so as a reminder, this is the reason we have the error:

One or more parts of the multiple manifestation were ingested prior to migration and the file names were not renamed from the bnumber_hin_wel_all-00000XXX_000X.jp2 format, which does not specify which manifestation these files belong to. This file name type was acceptable in Preservica but the storage service requires files to be renamed into the bnumber_000X_000X.jp2 format, which does specify the manifestation, right after the bnumber.
When ingesting another MM item, the BagIt tool looks up for already existing manifestations in order to create a fetch file for all files that are not updated by the current manifestation. Because the file name format bnumber_hin_wel_all-00000XXX_000X.jp2 does not have the manifestation order number, the BagIt tool doesn't know which files belongs to which manifestation and therefore, cannot ingest the new item.

We can't delete these through the Delete bnumber plugin ourselves as we run into the same BagIt Error. The solution has been to:

Deleting the bnumber from the Storage Service using the Delete bnumber plugin itself will not work due to the same reasons as above: the BagIt tool doesn't know which files belongs to which manifestation and therefore cannot bag the item. Instead, ask Intranda to delete the item via the plugin and manipulate the task BagIt creation and upload by adding a parameter "purge":true to the json.

Can we please "delete" these items from the storage service so we can get everything fixed and ingested?
b12947696_0002
b12947696_0004
b12947696_0005
b1313629x_0001
b13149040_0002
b13149040_0005
b10004300_0002
b10005614_0001
b10916441_0001
b11533353_0001
b11533353_0003
b12201777_0002
b12645163_0001
b12838767_0001
b12946448_0002
b1294662x_0001
b12946795_0002
b12948457_0001
b10326273_0002

"project:SampleProject" "stepopen:automatic MMO archive status check" I've started the delete from storage service process for these now

These have all had a deletion task started for them now.

We have a few more to finish up this work that was left off the previous list:
b11533353_0001
b11533353_0002
b12181110_0002
b10005614_0001

all b-numbers are deleted


Alive?

Adhearsion is alive or dead ?

That is not a great question to ask. How does it look to you? Why do you think it's dead? Are you waiting on any feature?

Yes, i know. Question is bad, but i'm waiting for release of 3-rd version, but i see that last commit was a long time ago.
Also i have a problem in 2.6.2 version that a cannot play any sound file into a bridged call:
i mean that when two participants are bridged via dial method i'm calling next code into confirmation class
call.after 30.seconds do    play 'something'  end
but nothing happened. I hear that sound only after call is unbridged.
Main idea is to play a sound "2 minutes left" into current bridge, to all participants (or only one, don't matter).
So i'm waiting 3-rd version, may this will be possible there ... but maybe i'm doing somethig wrong.
Are help would be appreciated.
Thanks

You can use the released 3.0.0.rc1 to test your bug, and if it persists, report it fully in an appropriately titled issue.
Adhearsion is alive, but the core team's attention is currently on other projects and Adhearsion's development is not as active as it used to be. The 3.0.0 full release is currently pending performance regression testing on a large app under mine and @bklang's control. There is, however, nothing preventing you from testing the RC and providing your feedback as has previously been requested and unfortunately lacked significant responses.

On the 3.0 release, Ben said it better than I could.
As for playing audio in a call, I am sorry but I do not remember which one between FreeSWITCH and Asterisk does not support that. What platform and version are you on?

I'm on asterisk 11 (also tested on 13). I think that adhersion (or punchblock) is blocking that bridge and puts a play into a command queue

This would be better as a mailing list conversation, but in the meantime I can tell you that I am almost sure Asterisk does not play audio in a bridged call, while FreeSWITCH does.

@Syfer Please test with Adhearsion 3 and file an issue which follows the issue guidelines clearly displayed above the form if the problem persists. Locking this ticket.


support second order

Needs https://github.com/JuliaNLSolvers/Optim.jl/pull/793 to work well with Float32s, so I hope @pkofod will get around to it, but otherwise it's good to go.


"Forgot Password" link still appears even with accounts-password disabled

"Forgot Password" link still appears even with accounts-password disabled.

@SachaG You're using a custom version of accounts-ui right? Is it for Nova, if so I could possibly have a look at it.

Any resolution on this? I'm using Nova, and I get the same problem from disabling accounts-password.

Relevant method seems to be here:
https://github.com/studiointeract/accounts-ui/blob/23599d59c13f30bb0dd3f68965273d277e18a787/imports/ui/components/LoginForm.jsx#L398
So omitting passwordSignupFields in the object passed to Accounts.ui.config did the job for me, i.e. I no longer have a "Forgot Password?" link.
Perhaps there should be another boolean requirement in showForgotPasswordLink(): that accounts-password is enabled.

Closing due to inactivity.

I'm getting the same bug

@timbrandin, this seems to make sense and work after @sleexyz suggested this. What do you think?
//  file: imports/ui/components/LoginForm.jsx

export function passwordSignupFields() {
  if(!hasPasswordService) return;
  return Accounts.ui._options.passwordSignupFields || "EMAIL_ONLY_NO_PASSWORD";
};


Add new committers to the community page

Kindly check for any typo and details. Thank you.
cc @corepointer @phaniarnab @kev-inn @Shafaq-Siddiqi

I would make changes and merge. Thank you.

Thanks for making the effort to update the website.


Data type Kamelet hardcoded

When we make use of data types, we assume the presence of a Kamelet which name is hardcoded [1]. While this could be true most of the time, it may not be the case when the user decide to use a different version of Kamelets. I think we need to find a way to have this flexible and give the user the possibility to specify which version of the Data Type action Kamelet wants to use.
[1] https://github.com/apache/camel-k/blob/c8e93fc031d71f5c24764577f40ab79f6c1856d7/pkg/util/bindings/kamelet.go#L155

@christophd fyi

BTW, this problem is highlighting a compatibility problem we would have in any case as stated in https://github.com/apache/camel-k/issues/4395


Travis: define inclusion rules instead of exclusion

Crystal is being tested against Travis CI on both Linux and OSX
platforms.
When having multiple platforms defined in the Travis configuration
file (.travis.yml), any environment variable used will be
applied equally to each OS.
To reduce bogus failures or repeated builds, Travis give you options
of both inclusion and exclusion rules to consider for the test matrix.
Before this commit, an exclusion rule was applied so environment
variables used for both Linux 32 and 64 builds do not result in
duplicated builds on OSX.
This commit changes that from exclusion to an explicit matrix that
ensures no environment variable from Linux is leaked into the OSX
build, also avoiding the need to exclude the duplicate build.
While this change is simply cosmetic, it ensures clarity is taken
in consideration when quickly looking at CI results. (eg. Why it
says Linux64 for the OSX build?).
Thank you. :heart: :heart: :heart:

Hello,
Sorry about the first commit, forgot to remove os now that it is defined inside include. :sweat_smile:
If you approve, will like to squash into a single commit before merge.
Thank you. :heart: :heart: :heart:

Hello,
Images for reference.
Before:

After:

Cheers.

Awesome, thanks!!


--DO NOT MERGE YET--

Support for nested fields.
Hey,
This is far from a complete version but it exposes a new approach for handling form fields in a nested way,
see: #55
Thoughts?

I'd like some tests for that getSet.js. That's the kind of function that both a) can be easily tested, and b) really needs to work 100% correct.

Oops, sorry about that.  I didn't see the getSet test.

@erikras btw u can close it for now if the (1) bothers u, it needs more work,
The question if this is a good direction or not, your call.

No, no, @asaf, I still think this is something to look into, and there is enough demand for it, but I haven't yet found the time to actually design a "compound" form to test its viability.

@erikras Look at my concept, it doesn't break anything yet provides a powerful nested form support,
The most annoying thing is that all other corresponding states to fields (touch, errors, etc) must conform the same structure... so it takes a bit time to move everything but it shouldn't be that hard.

Implemented.


Add command alias kubectl-decode-secret

When called by that name, do not require get secret on the command line. Thus, the following commands are equivalent:
ksd get secret my-secret -oyaml
kubectl-decode-secret my-secret -oyaml

Also see discussion in https://github.com/kubernetes-sigs/krew-index/pull/173

Hey @ashleyschuett. If you'd appreciate your command land in krew.dev, feel free to reopen this PR.


[WIP]: mypy --strict-optional check

Proposed fix for #3447

Automated message from Dropbox CLA bot
@decached, it looks like you've already signed the Dropbox CLA. Thanks!

Merged the one commit so far, after fixing the testing failures (basically some dead code in the tests), thanks @decached!  We're down to just about ~200 errors; I'd encourage you to try clearing things by file/directory from here.


[Problem] Sum of primes

Sum of primes.
Sum of primes below 10 is 17 (2 + 3 + 5 + 7).
**n is a user input. 0 < n < 5000000

Find sum of the primes below n.

Hi! I'd like to work on this. Could you assign me this issue?

If you don't mind, can you just brief me on how I can submit the solution to this problem?

If you don't mind, can you just brief me on how I can submit the solution to this problem?

Check this website. opensource.com/create-pull-request-github
Hope this helps.

Hi i would like to contribute on this issue.
I am learning Open Source and I think that this will be a great issue for me to solve.

Hi i would like to contribute on this issue. I am learning Open Source and I think that this will be a great issue for me to solve.

Assigned.


Fix Issue with file disappearance upon renaming error

This Issue was brought up on thimble
Here
and the issue links back to brackets.
renaming file error with filenames greater than the size of the window pane causes the view to be shifted to the right and viewed as if the files disappeared. The solution was to move back to its original location to prevent the illusion of the file disappearance.

If there were a more efficient way of fixing this issue please let me know and I will fix it.

@davidhuang1550 I'm glad to see you diving into this, nice work.  I don't think this is the right fix, because the code you're touching is related to how Brackets interacts with Thimble, and the bug in that issue is all contained within Brackets itself.
The error dialog that's being shown in this case happens here:
https://github.com/adobe/brackets/blob/33be8022e60f1cb50c66244ce71ca20be6a3e487/src/project/ProjectManager.js#L570
You should try setting a breakpoint in your browser's JavaScript debugger (or add the debugger keyword to the code at that line and open your dev tools).  It would be interesting to get a stack track of what is happening when it occurs.
If what I'm saying doesn't make sense, let me know and I'll give further instructions.

Thanks for the feedback, i will try this.

seems that the problem has been fixed.


Doesnt work for COD: bocw

The program doesnt detect even if the streamer has drops enabled

Hi kid, this program doesn't filter by drops enabled, you are the one that must provide the filtered list.
Since you seem quite useless, there you have the filtered link for your game:
https://www.twitch.tv/directory/game/Call%20of%20Duty%3A%20Black%20Ops%20Cold%20War?tl=c2542d6d-cd10-4532-919b-3d19f30a768b
The env var to change is streamersUrl.
So yes, it works for any game, but requires you to think and read the code.

Hi kid, this program doesn't filter by drops enabled, you are the one that must provide the filtered list.
Since you seem quite useless, there you have the filtered link for your game:
https://www.twitch.tv/directory/game/Call%20of%20Duty%3A%20Black%20Ops%20Cold%20War?tl=c2542d6d-cd10-4532-919b-3d19f30a768b
The env var to change is streamersUrl.
So yes, it works for any game, but requires you to think and read the code.


[FEATURE] Parse new featureflagconfiguration CRD structure

Requirements
As a follow-up from https://github.com/open-feature/open-feature-operator/issues/72, flagd will need to parse this altered CRD structure

Redundant now that the CRD is backwards compatible


Archive file for python 2.7.12 is gone in storage.googleapis.com/travis-ci-language-archives/

My recent PR failed due to the following error. It seems that some archive file for python 2.7.12 is gone.
Travis might have dropped its support due to the end of life of 2.7.12.
Downloading archive: https://storage.googleapis.com/travis-ci-language-archives/python/binaries/ubuntu/16.04/x86_64/python-2.7.12.tar.bz2
0.10s$ curl -sSf --retry 5 -o python-2.7.12.tar.bz2 ${archive_url}
curl: (22) The requested URL returned error: 404 Not Found
Unable to download 2.7.12 archive. The archive may not exist. Please consider a different version.
https://github.com/google/personfinder/pull/760

https://travis-ci.org/github/google/personfinder/jobs/739495598
https://travis-ci.org/github/google/personfinder/jobs/739495585

This seems to be a duplicate of https://github.com/google/personfinder/issues/746. Closing this one.


Draw Component functions have inconsistencies

Description
When we draw the game object components there are multiple issues.

Sprite component

Some words overlap inputs
Some colors are incorrect.

Box Collider Component

Incorrect colors.

All other components need to be checked and adjusted to be more clean.

Prerequisites
Must have the latest.
To Reproduce
Add new components to a new game object and view the UI drawn in the object details panel.
Expected behavior
Needs to be clean and organized correctly. Correct colors, no overlap, consistency, etc.
Actual behavior
See description above.

This should be handled with the changes from f769b9c


RELATED: RAIL-3438 Fix refresh-md script in JavaScript apps

Also remove obsolete replacement (both TS and JS versions
work without the extension).
JIRA: RAIL-3438

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/229/ : SUCCESS in 1m 03s
https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-oss-check/191/ : SUCCESS in 7s

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-merge-compliance-check/84/ : SUCCESS in 17s
https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/230/ : SUCCESS in 25s
https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-fossa-scanning-zuul-el7/49/ : SUCCESS in 32s (non-voting)


_loadImage() to public

Would be nice to have a possibility to load image after the canvas is initialized. One of these options would be good to have:

canvasWhiteboardComp.imageUrl = image;
canvasWhiteboardComp.loadImage();

or simply
canvasWhiteboardComp.loadImage(image);

@Peshou Thank you for the fast response.
I just can confirm that there is ngOnChanges check for image url changes, but logs that I have put, never occurs.

Simply calling _loadImage worked fine:
this.canvasWhiteboard.imageUrl = someBase64;
this.canvasWhiteboard['_loadImage()']; // :)

Okay let me look into this since it may be some kind of bug if the method is not being invoked

Could you try to edit the canvas component directly or make a canvas options object which is linked to the component itself. Example:
 <canvas-whiteboard [imageUrl]="someBase64">
    </canvas-whiteboard>

Or
 <canvas-whiteboard [options]="options">
    </canvas-whiteboard>

options: CanvasWhiteboardOptions = {
};

then somewhere in the code
this.options.imageUrl=someBase64;

Your issue seems to occur when manually changing the input value without letting angular check for changes. Because of this the ngOnChanges method won't register input changes so the redraw code won't execute. I think the second solution with the options object should help you with your issue and it should be a bit cleaner for other inputs you may wish to change.

The example I posted, was directly editing of canvasComponent instance property, the solution that you proposed also didn`t work for me. Here is the example:
<canvas-whiteboard #canvasWhiteboard [options]="canvasOptions" (onBatchUpdate)="onCanvasDraw($event)"></canvas-whiteboard>
Options:
    this.canvasOptions = {       strokeColor: 'rgb(255, 255, 255)',       lineWidth: 5,       drawButtonEnabled: false,       clearButtonEnabled: false,       undoButtonEnabled: false,       redoButtonEnabled: false,       saveDataButtonEnabled: false,       colorPickerEnabled: false,       shouldDownloadDrawing: false,       startingColor: 'transparent',       drawingEnabled: false,       showStrokeColorPicker: false,       showFillColorPicker: false,       shapeSelectorEnabled: false,       showShapeSelector: false     } as CanvasWhiteboardOptions;
And somewhere in the code:
this.canvasOptions.imageUrl = base64;
Nothing happened. I have tried to subscribe to onImageLoaded event before changing of the URL (also with callback), nothing resolved

Okay, thanks, this is an issue and should be handled accordingly, since this is the correct way to change the imageUrl.
I will update the library with a patch version v2.0.3 and you should be able to have this functionality available.
In the meantime, I think that you can work out your problem by directly accessing the imageUrl input in the canvas without using the options object.
this.imageUrl = base64;

<canvas-whiteboard #canvasWhiteboard [options]="canvasOptions" (onBatchUpdate)="onCanvasDraw($event)"></canvas-whiteboard>

Thanks for the observation and help.
Best,
Stefan

Yeah, and until I roll this fix out, you should download the 2.0.3 version of the canvas that I released today so that you can get the fix for the flicker that appears when you draw on a canvas with an image as background.
I'll try to tend to your issue really soon.
Best Regards,
Stefan


feat: add target to Link

Adds target attribute to the <a> tag created by Link.
I wanted this for my app and noticed it didn't exist (among other attributes) so I thought I'd just add it :)

You can add it via the attr: syntax, for example:
<Link href="http://example.com" attr:target="_blank">
     "This is a link"
</Link>

Ah that's what I was looking for. I didn't realize it existed


Suggestion: Force plugin cache for connection class

Ansible: 2.9.9
For my performance test case with 20 empty task I've noted (with ANSIBLE_DEBUG) why loading the 'ssh' and 'mitogen_ssh' modul was always with "found_in_cache=False".
Reason is the forking the Ansbile core is doing, the connection module is only ever loaded inside the fork ....
I've added a forced loading to strategy.py inside _install_wrapper():
ansible_mitogen.loaders.connection_loader__get('mitogen_ssh', class_only=True)
My test has a total run-time of ~3,7s; this change removed 0,2-0,3s - not bad.
I don't know where to put this to avoid the harcoded ssh argument. For me this works.
Not even sure if I will start using Ansible at all considering this is still to slow for my intended use case. So take it or leave it :)
P.s. A small effects can also be seen when caching the actions 'normal' module.

ah i see that was already - tryed but ansible-core seems to counteract any performance gains :)


sort: dedup using compare_by

compare_by is the function used for sorting, we should use it for dedup
as well.
Closes #2062.
r? @electricboogie could you make sure I'm not missing something here? This feels way too simple.

Give me a sec to kick this around?

This looks right and works on all our tests.  Don't know why I didn't see it myself.

excellent, thanks :)


[BUG]: Uncaught (in Promise) {} (stripe.confirmPayment)

What happened?
I'm in the process of migrating our cardElement to a generic paymentElement. The setupPay works fine, but then on the following method:

async function handleSubmit(e) {
    console.log("HANDLING SUBMIT ===========");
    e.preventDefault();
    const stripe = Stripe(window.stripePub);

    const { error } = await stripe.confirmPayment({
      elements,
      confirmParams: {
        return_url: "http://localhost:3000",
      },
    });

    if (error.type === "card_error" || error.type === "validation_error") {
      showMessage(error.message);
    } else {
      showMessage("An unexpected error occured.");
    }
}

it leads to the following error in the console:

Unclear what the issue is because the error message is empty.
Environment
Chrome 101.0.4951.64 on MacOS Big Sur 11.6
Reproduction
No response

Error occurs if Stripe is called twice, once for initializing/mounting, and once for calling confirmPayment. Error goes away once the same stripe = Stripe(..) object is re-used. The error message should be much better though, as this was truly awful to debug.


Improved sanitization and formatting.

fixes #2
no longer depends on universal_html:
custom output formatting that keeps no-content tags as self-closing: <img />

I've wanted to create a separate library for the formatting (e.g. allow indents and a canonical formatting of the html markup), but the edges cases are hard to deal with, and I've settled with this simple solution.
The runZoned part could be probably removed? /cc @jonasfj

PTAL.

@jonasfj I don't have merge right, please merge this one.


add tooling to convert endpoints to a tree and back

add tooling to convert endpoints to a tree and back

@maksymvavilov I think we should check for nils more aggressively and add some nil tests also as there are a lot of pointers and derefrencing pointers to get the value in here so could easilly end up with a panic

@maksymvavilov I think we should check for nils more aggressively and add some nil tests also as there are a lot of pointers and derefrencing pointers to get the value in here so could easilly end up with a panic

safeguarded all public calls from nil pointers


Ubuntu repos install 3.x series, not 4.0 as said in the wiki

In the wiki, here:
https://github.com/tesseract-ocr/tesseract/wiki#linux
It says:

...you can install Tesseract 4.x and it's developer tools on Ubuntu 18.x bionic by simply running:

sudo apt install tesseract-ocr
sudo apt install libtesseract-dev

However, I'm running Bionic and I got version 3.x installed:
↪ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.2 LTS
Release:	18.04
Codename:	bionic

↪ tesseract -v
tesseract 3.04.01
 leptonica-1.73
  libjpeg 6b : libpng 1.6.34 : zlib 1.2.11

https://packages.ubuntu.com/bionic/tesseract-ocr

My mistake, I had an old version lingering around. Sorry about that.


Why does it install itself in the Windows/Installer folder?

Like the title says.
Just wondering if this was an error or a requirement of some kind.

I'm not sure what you mean with "Windows/Installer folder"?
The default path to install should be %programfiles%\PowerPlanSwitcher.
I can't think of many situation where this would be different.
Can you elaborate?

I have uninstalled it now, but the installer had placed it in a "C:\Windows\Installer<someID>" folder.
It's possible it happened because of a problem on my end.
Anyway the program runs just fine in portable mode so that's what I'm doing now.

I don't even have a single folder in C:\Windows that starts with "Installer". I'm really not sure why it would install it there.
But as long as it works now, it's fine.
If you have any further problems, just create a new issue.

Just to clarify, I meant "C:\Windows\Installer<some-id>
You certainly have the "Installer" folder in that location (may be hidden in your explorer), and a series of subfolders in it with IDs as names.

You're right. I had the option to hide protected system files active.
It seems like the folders inside usually have icons for installed apps.
Maybe just some meta information for show the app in the list of installed apps.
The weird part is, for PowerPlanSwitcher, it has the whole executable in there. Though it's called "ProductIcon.exe". I'm not sure that is 100% correct.
I would assume it has to do with the way I put my icons in the app.
I will take a look.

Okay, I found the issue. The windows\installer folder is used by MSI installers (possibly others too) to store relevant files. Like icons to show in the list of installed apps or files to perform a repair on the installation.
I changed it to use the actual icon file instead of the executable.
But the installer should have put the actual executable PowerPlanSwitcher.exe in the program files folder. And add start menu entries and a auto-start entry for it.
I haven't released a new setup yet. I don't think it's necessary. Your stuff works. The ProductIcon.exe is not a problem in itself. MSI explicitly allows either ico files or exe files to be used as a reference for the icon. Though it might not be a smart idea to use the executable, it's not a real problem.
Installing a new version replaces the ProductIcon.exe with the new ProductIcon.ico file. By the magic of MSI.
Here more details if you're interested:
The reason for the ProductIcon.exe there was that I used the executable as a reference to the icon when creating the installer. The name ProductIcon.exe was given by me. (It's basically more than 10 years old and copied from an example I found on the internet.)
I expected it to extract the icon from the executable. But instead it just takes the executable as THE file for the icon. Hence it stores it in windows\installer.
Btw: I tried running the ProductIcon.exe. But it seems to be blocked. Probably due to it's location. Not sure.

I haven't released a new setup yet. I don't think it's necessary.

If the shortcut in the start menu points to Producticon,exe in the Windows\Installer folder (as it did in my case), the whole app is ran from there. I don't think it should be doing that.

I just double and triple checked. I don't think it's the case that the start menu shortcut or auto start shortcut starts the executable that is in Windows\Installer.
The wxs file clearly shows that the shortcuts belongs to the ProductComponent and is marked as advertised.
The ProductComponent is the PowerPlanSwitcher.exe in the platform specific ProgramFiles folder with the PowerPlanSwitcher as the subfolder.
Afaik I couldn't even directly reference the ProductIcon.exe that is in Windows\Installer.
I double checked the resulting MSI file with Orca. The shortcuts reference the ProductComponent and from there on out the actual executable in the proper path.
I also checked on 4 machines. Two existing ones and two clean VMs I set up. One VM is Win11, the rest is Win10.
It's not possible to directly check the target of the shortcuts since they are marked as advertised. When checking the properties of the shortcut, it shows "PowerPlanSwitcher" as the target. Nor does it short the path it's executed in.
But I verified by checking the image path of the executable that is started when I use the shortcut.
On all machines it shows the image path to be C:\Program Files\PowerPlanSwitcher\PowerPlanSwitcher.exe.
And I tried to inspect the ProductIcon.exe.
Keep in mind that the files name is just defined by the ID of the Icon element in the MSI file.
It turns out, the ProductIcon.exe isn't the actual executable of the project. But instead the DLL that is created along side the executable. Before the setup is created, the executable and all necessary binaries are combined into a single executable. I thought $(var.PowerPlanSwitcher.TargetPath) would refer to the actual executable. But when checking out the size, content and headers of the executables, I noticed that the content differ severely. But the PowerPlanSwitcher.dll has the exact content of the ProductIcon.exe in Windows\Installer.
So I doubt there is anything directly executable there anyways. It's a DLL file that has .exe at the end of the filename.


Publications List Ignored

Describe the bug
I have a KMM library in which I define two publications aar and framework. I declare that these two publications should be published by artifactory plugin. The artifactoryPublish task publishes these two publications and additionally publishes .module metadata about publications which are not listed in defaults block.
We used the plugin version 4.33.1 and it worked. It started to happen after migration to 5.2.0.
Current behavior
The :library module is kotlin multiplatform library. According to this article kotlin multiplatform plugin along with maven-publish plugin creates publications for each target.
Configuration of artifactory plugin in root build.gradle.kts
configure<ArtifactoryPluginConvention> {
    val artifactoryContextUrl: String by rootProject
    val artifactoryKeyRelease: String by rootProject
    publish {
        contextUrl = artifactoryContextUrl
        repository {
            repoKey = artifactoryKeyRelease
            username = System.getenv("ARTIFACTORY_LOGIN")
            password = System.getenv("ARTIFACTORY_PASSWORD")
        }
        defaults {
            publications("aar", "framework")
            setPublishIvy(false)
            setProperties(mapOf("q.os" to "kmm", "dev.team" to "xxx"))
        }
    }
}

Declaration of publications that should be published by the artifactory task.
publishing {
    val libraryVersion: String by rootProject
    val artifactoryGroupId: String by rootProject
    val artifactoryArtifactId: String by rootProject
    val artifactoryArtifactIdIos: String by rootProject

    publications {
        register<MavenPublication>("aar") {
            groupId = artifactoryGroupId
            version = libraryVersion
            artifactId = artifactoryArtifactId

            artifact("$buildDir/outputs/aar/library-release.aar")
            pom.withXml {
                val dependencies = asNode().appendNode("dependencies")
                val common = configurations.commonMainImplementation.get().allDependencies
                val android = configurations.implementation.get().allDependencies
                (common + android).forEach {
                    dependencies.appendNode("dependency").apply {
                        appendNode("groupId", it.group)
                        appendNode("artifactId", it.name)
                        appendNode("version", it.version)
                    }
                }
            }
        }
        register<MavenPublication>("framework") {
            groupId = artifactoryGroupId
            version = libraryVersion
            artifactId = artifactoryArtifactIdIos

            artifact("$buildDir/faktory/zip/frameworkarchive.zip")
        }
    }
}

The artifactoryPublish task publishes publications framework, aar and module info about publications defined in KMM. Module info shouldn't be deployed. See enclosed build log.
> Task :library:generateCommonMainxxxDatabaseInterface UP-TO-DATE
> Task :library:androidReleaseSourcesJar UP-TO-DATE
> Task :library:preBuild UP-TO-DATE
> Task :library:preReleaseBuild UP-TO-DATE
> Task :library:mergeReleaseJniLibFolders UP-TO-DATE
> Task :library:mergeReleaseNativeLibs NO-SOURCE
> Task :library:stripReleaseDebugSymbols NO-SOURCE
> Task :library:copyReleaseJniLibsProjectAndLocalJars UP-TO-DATE
> Task :library:generateReleaseResValues UP-TO-DATE
> Task :library:extractDeepLinksForAarRelease UP-TO-DATE
> Task :library:checkKotlinGradlePluginConfigurationErrors
> Task :library:generateReleaseResources UP-TO-DATE
> Task :library:packageReleaseResources UP-TO-DATE
> Task :library:parseReleaseLocalResources UP-TO-DATE
> Task :library:generateReleaseRFile UP-TO-DATE
> Task :library:compileReleaseKotlinAndroid UP-TO-DATE
> Task :library:extractReleaseAnnotations UP-TO-DATE
> Task :library:javaPreCompileRelease UP-TO-DATE
> Task :library:compileReleaseJavaWithJavac NO-SOURCE
> Task :library:mergeReleaseGeneratedProguardFiles UP-TO-DATE
> Task :library:mergeReleaseConsumerProguardFiles UP-TO-DATE
> Task :library:mergeReleaseShaders UP-TO-DATE
> Task :library:compileReleaseShaders NO-SOURCE
> Task :library:generateReleaseAssets UP-TO-DATE
> Task :library:packageReleaseAssets UP-TO-DATE
> Task :library:prepareLintJarForPublish UP-TO-DATE
> Task :library:prepareReleaseArtProfile UP-TO-DATE
> Task :library:processReleaseManifest UP-TO-DATE
> Task :library:processReleaseJavaRes UP-TO-DATE
> Task :library:mergeReleaseJavaResource UP-TO-DATE
> Task :library:syncReleaseLibJars UP-TO-DATE
> Task :library:writeReleaseAarMetadata UP-TO-DATE
> Task :library:bundleReleaseAar UP-TO-DATE
> Task :library:generateMetadataFileForAndroidReleasePublication
> Task :library:kspKotlinIosArm64 UP-TO-DATE
> Task :library:compileKotlinIosArm64 UP-TO-DATE
> Task :commonizeNativeDistribution UP-TO-DATE
> Task :library:transformCommonMainDependenciesMetadata UP-TO-DATE
> Task :library:compileCommonMainKotlinMetadata UP-TO-DATE
> Task :library:metadataCommonMainProcessResources NO-SOURCE
> Task :library:metadataCommonMainClasses UP-TO-DATE
> Task :library:transformNativeMainDependenciesMetadata UP-TO-DATE
> Task :library:compileNativeMainKotlinMetadata NO-SOURCE
> Task :library:metadataNativeMainProcessResources NO-SOURCE
> Task :library:metadataNativeMainClasses UP-TO-DATE
> Task :library:transformAppleMainDependenciesMetadata UP-TO-DATE
> Task :library:compileAppleMainKotlinMetadata NO-SOURCE
> Task :library:metadataAppleMainProcessResources NO-SOURCE
> Task :library:metadataAppleMainClasses UP-TO-DATE
> Task :library:transformIosMainDependenciesMetadata UP-TO-DATE
> Task :library:compileIosMainKotlinMetadata UP-TO-DATE
> Task :library:metadataIosMainProcessResources NO-SOURCE
> Task :library:metadataIosMainClasses UP-TO-DATE
> Task :library:iosArm64MetadataJar UP-TO-DATE
> Task :library:iosArm64SourcesJar UP-TO-DATE
> Task :library:generateMetadataFileForIosArm64Publication
> Task :library:kspKotlinIosSimulatorArm64 UP-TO-DATE
> Task :library:compileKotlinIosSimulatorArm64 UP-TO-DATE
> Task :library:iosSimulatorArm64MetadataJar UP-TO-DATE
> Task :library:iosSimulatorArm64SourcesJar UP-TO-DATE
> Task :library:generateMetadataFileForIosSimulatorArm64Publication
> Task :library:kspKotlinIosX64 UP-TO-DATE
> Task :library:compileKotlinIosX64 UP-TO-DATE
> Task :library:iosX64MetadataJar UP-TO-DATE
> Task :library:iosX64SourcesJar UP-TO-DATE
> Task :library:generateMetadataFileForIosX64Publication
> Task :library:generateProjectStructureMetadata UP-TO-DATE
> Task :library:allMetadataJar UP-TO-DATE
> Task :library:sourcesJar UP-TO-DATE
> Task :library:generateMetadataFileForKotlinMultiplatformPublication
> Task :library:generatePomFileForAarPublication
> Task :library:generatePomFileForAndroidReleasePublication
> Task :library:generatePomFileForFrameworkPublication
> Task :library:generatePomFileForIosArm64Publication
> Task :library:generatePomFileForIosSimulatorArm64Publication
> Task :library:generatePomFileForIosX64Publication
> Task :library:generatePomFileForKMMBridgeFrameworkPublication
> Task :library:generatePomFileForKotlinMultiplatformPublication
> Task :library:artifactoryPublish
> Task :artifactoryPublish
> Task :extractModuleInfo
> Task :library:extractModuleInfo
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/XxxXxxXxx/library-android/unspecified/library-android-unspecified.module
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/XxxXxxXxx/library-iosarm64/unspecified/library-iosarm64-unspecified.module
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/XxxXxxXxx/library-iossimulatorarm64/unspecified/library-iossimulatorarm64-unspecified.module
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/XxxXxxXxx/library-iosx64/unspecified/library-iosx64-unspecified.module
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/XxxXxxXxx/library/unspecified/library-unspecified.module
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/cz/xxx/modules/xxx-android/1.0.5/xxx-android-1.0.5.aar
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/cz/xxx/modules/xxx-ios/1.0.5/xxx-ios-1.0.5.zip
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/cz/xxx/modules/xxx-android/1.0.5/xxx-android-1.0.5.pom
[pool-52-thread-1] Deploying artifact: https://xxx.jfrog.io/xxx/xxx-xxx-xxx/cz/xxx/modules/xxx-ios/1.0.5/xxx-ios-1.0.5.pom

I believe that the issue is in PublicationExtractor in extractModuleInfo. There is a filter on GenerateModuleMetadata tasks with respect to publications. I think there should be included a filter for declared publications. I debugged extractModuleInfo and found out that KMM publication tasks are not filtered out. See screenshot below.
    public void extractModuleInfo() {
        for (GenerateModuleMetadata generateModuleMetadata : artifactoryTask.getProject().getTasks().withType(GenerateModuleMetadata.class)) {
            Publication publication = generateModuleMetadata.getPublication().get();
            if (!isApplicablePublication(publication)) {
                continue;
            }

            File moduleMetadata = generateModuleMetadata.getOutputFile().getAsFile().get();
            if (!moduleMetadata.exists()) {
                continue;
            }

            DeployDetails.Builder builder = createArtifactBuilder(moduleMetadata, generateModuleMetadata.getPublication().get().getName());

            //noinspection unchecked
            ActualPublication actualPublication = (ActualPublication) publication;
            PublishArtifactInfo artifactInfo = new PublishArtifactInfo(getPublicationArtifactId(actualPublication), "module", "module", null, moduleMetadata);
            addArtifactToDeployDetails(actualPublication, builder, artifactInfo);
        }
    }

Publications named androidRelease, iosArm64, iosSimulatorArm64, iosX64, kotlinMultiplatform are filtered.

Reproduction steps

Install KMM plugin to Android Studio.
Create a basic KMM library using wizard.
Add additional publication in the KMM library that is going to be published to Artifactory.
Add artifactory plugin in the version 5.2.0 and configure it so the given publication is the only one to be published.
Run artifactory plugin and watch what is published to artifactory.

Expected behavior
I expect that only declared artifacts along with their POM (or other metadata) are published to artifactory.
Artifactory Gradle plugin version
5.2.0
Operating system type and version
macOs 14.3.1 (23D60)
JFrog Artifactory version
No response
Gradle version
8.2.2 / 8.3.0

We have the same problem in our project (KMM library too).
It works until 5.1.10, then the error occurs - all targets are build and deployed.
When trying 5.1.11. some other build-error occurs, in 5.1.12. this build-error is fixed, but the publication list isn't working anymore.

@hollmannlu Related #104


Controls should hide automatically when video is playing and no input has been made recently

Many video players will automatically hide controls if the video is playing and there has not been any input (mouse or keyboard) made recently.

Please see #6.


Feature request..... Enable live wallpapers option

Hi.... I really love this rom but i still miss using KLWP. Is there any way to make live wallpapers work in this rom?

Live wallpapers aren't included

I know that live wallpapers aren't included in this rom. But i will like very much if this will be possible. Why the reason that isn't included?

Not many users are using it. That's why, and it has been out since MM days


Use Unicode functions consistently and remove helper extension methods that were made public

Fixes #72
This one is not urgent. I have improvements I'd like to do in other PRs.
Please review carefully in case I made a mistake. The test suite does execute the affected code paths, and I checked that HidDevice.Description remained the same.
I also removed the public Extensions class with ToUTF8String and ToUTF16String because I don't think it was meant to be public. It has the unfortunate side effect of appearing as an extension method on byte arrays in intellisense for all projects that use HidLibrary. It was added in https://github.com/mikeobrien/HidLibrary/commit/aad5c64a5dcf0c02c689a5905eebf04f2eb75a56. If any consuming projects did directly call these extension methods, they will fail to recompile against the new version of HidLibrary. If a consuming assembly is not recompiled but runs using the new version of HidLibrary anyway, the .NET runtime will throw MissingMethodException if the consuming project did directly call these extension methods.
An alternative is to obsolete them and put [EditorBrowsable(EditorBrowsableState.Never)] on them so that they stop polluting intellisense for all byte arrays in consuming projects.

I modified the original commits to no longer delete things in Extensions.cs, and then I added three new commits.
Unfortunately, both Visual Studio and ReSharper's intellisense still show the extension methods even though they have EditorBrowsable(Never). I filed a VS bug for this (https://github.com/dotnet/roslyn/issues/47551). I tested this using a new project in a new solution that directly referenced the HidLibrary.dll file.
Would you like to make them non-extension methods? This would get them to stop showing up in byte array intellisense. It would not be a binary-breaking change: no MissingMethodException for assemblies consuming HidLibrary that run against the new version without being recompiled. It would be a source-breaking change: projects consuming HidLibrary would stop compiling once they start pulling in the new version of HidLibrary. Instead of the obsoletion message explaining that the extension method is deprecated, they'd get "CS1061 'byte[]' does not contain a definition for 'ToUTF16String' and no accessible extension method 'ToUTF16String' accepting a first argument of type 'byte[]' could be found (are you missing a using directive or an assembly reference?)"
Visual Studio:

ReSharper:

Good news! The Roslyn team fixed this very fast and Visual Studio 16.8 will respect EditorBrowsable(Never) on the extension methods the way I hoped! I struck out the text above that no longer applies.
That answers the last outstanding question, I think?

@amullins83 @jnm2 hey, sorry for the late reply. IMO depending on extension methods in a 3rd party library always comes with the risk they they can be changed, esp if they aren't documented as part of the official API. So I don't see any problem with pulling them. Plus this project doesn't even have official API doc's so we have even more latitude.... ;) I guess we should bump the major version since we are following semver and this may be a breaking change for some.

Major/minor version is set here: https://github.com/mikeobrien/HidLibrary/blob/master/azure-pipelines.yml#L5

@mikeobrien I ended up leaving them in; would pulling them be your preference? I'm thinking the major version update could be saved for something more exciting then.

Ah ok cool, however you have it is fine then. Honestly I'm pretty disconnected from this project so don't have much of a preference on anything. ;) Thx for the PR!

Cool! @amullins83 @mikeobrien Think it's ready to merge?


Implement caesar cipher encryption

Given a non-empty string of lowercase letters and a non-negative integer representing a key, write a function that returns a new string obtained by
shifting every letter in the input string by k positions in the alphabet, where k is the key.
Note that letters should "wrap" around the alphabet; in other words,
the letter *z shifted by one returns the letter a
Sample Input
string = "abz"
key = 2

Sample Output
"cdb"

I will take this up. Can you assign it to me ?

@Karthik-B-06 assign to you. Send your pull request.

can you assign this to me , i would like to work on this!

@Aditya489 This problem already solved in c language.you send improved version of this code or you solve same problem in different programming language & send the pull request.

its already closed i did it in another programming language , can i submit now?

@Aditya489 You can send pull request.


Bump to Gradle 5.2.1

This includes monkey patches to fix errors:

https://circleci.com/gh/int128/gradle-swagger-generator-plugin/203
Project directory '/home/circleci/gradle-swagger-generator-plugin/acceptance-test/codegen-v2/multiple-sources' is not part of the build defined by settings file '/home/circleci/gradle-swagger-generator-plugin/settings.gradle'. If this is an unrelated build, it must have its own settings file.

This will fix #142.

It seems OutOfMemoryError occurs:
https://circleci.com/gh/int128/gradle-swagger-generator-plugin/211
org.hidetake.gradle.swagger.generator.test.codegen2.CustomClassSpec > generateSwaggerCode task should generate customized server code FAILED
    java.lang.IllegalStateException at CustomClassSpec.groovy:29
        Caused by: org.gradle.tooling.GradleConnectionException at CustomClassSpec.groovy:29
            Caused by: org.gradle.launcher.daemon.client.DaemonDisappearedException

Tested on my macOS:
> Task :resolveSwaggerTemplate NO-SOURCE
Exception in thread "Thread-513" java.lang.OutOfMemoryError: Metaspace
	at com.fasterxml.jackson.databind.deser.BeanDeserializerBuilder.build(BeanDeserializerBuilder.java:351)
...
	at io.swagger.codegen.v3.config.CodegenConfigurator.fromFile(CodegenConfigurator.java:611)
	at io.swagger.codegen.v3.cli.cmd.Generate.run(Generate.java:216)
	at java.base/java.lang.Thread.run(Thread.java:834)
> Task :generateSwaggerCodePetstoreV1
> Task :generateSwaggerCodePetstoreV2 FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':generateSwaggerCodePetstoreV2'.
> Metaspace

* Try:
Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Exception is:
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':generateSwaggerCodePetstoreV2'.
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter$2.accept(ExecuteActionsTaskExecuter.java:121)
...
Caused by: java.lang.OutOfMemoryError: Metaspace


[ENH] Add a time-dependent gaussian distribution.

Describe the solution you'd like
Addition of time-dependent parameters to the base gaussian distribution.
Parameters- mean and variance will be estimated from the temporal data using standard time series statistical techniques.

Describe alternatives you've considered
Functionality can be extended to accept parameters as functions of time

Interesting idea!
I think there are two ideas here - estimation and representation.
I would also suggest to start with:

representation, i.e., how to represent temporal correlation
and as a special case, multivariate normal with the index dimension being independent, see https://github.com/sktime/skpro/issues/22

One design quesiton is, how would the user represent a multivariate?
E.g., if we pass (co-)variance to be in a certain shape? Or, do we need to pass additional variables?

Interesting idea!
I think there are two ideas here - estimation and representation.
I would also suggest to start with:

representation, i.e., how to represent temporal correlation
and as a special case, multivariate normal with the index dimension being independent, see [ENH] roadmap of probability distributions to implement #22

One design quesiton is, how would the user represent a multivariate?
E.g., if we pass (co-)variance to be in a certain shape? Or, do we need to pass additional variables?

hmm, I suppose we could use the shape of the variance to interpret the number of dimensions for a multivariate distribution. I don't think much work has been taken up for the multivariate normal in issue #22, the implementation of the base class may make it easier since the time dependency can simply be extended to it as in the case of univariate.

Yes, I suppose that would work.
Would you like to sketch a design for the multivairate normal first?

sure!


The test case about String2Base function under llvm backend.

Currently with llvm backend String2Base function would fail if the string is in the format of "+N", e.g. "+123", with error message:
terminate called after throwing an instance of 'std::invalid_argument'
  what():  Not a valid integer

But under other backend, such as java, "+123" would be parsed as Int: 123

Fix is under review: https://github.com/kframework/llvm-backend/pull/191


Give dom-repeat a context

The dom-repeat snippet doesn't have a context and so it isn't working. This PR fixes that.

Thanks!