Adding in Base Processor - exposing Umbraco context services

As discussed on issue #200.  I've done the work to add UmbracoContext and ApplicationContext in the base DittoProcessorAttribute. I used the singleton UmbracoContext.Current and ApplicationContext.Current as mentioned here accessing ApplicationContext through UmbracoContext is something you'd want to avoid doing.
I've also reworked the DittoProcessor methods which called UmbracoContext.Current to use the new Umbraco property.
There are a couple of internal statics I didn't touch upon though:

https://github.com/leekelleher/umbraco-ditto/blob/8387c47bc5f0a2c01ed5b3299506f91c28bd4234/src/Our.Umbraco.Ditto/ComponentModel/Processors/UmbracoDictionaryAttribute.cs#L61-L76
https://github.com/leekelleher/umbraco-ditto/blob/8387c47bc5f0a2c01ed5b3299506f91c28bd4234/src/Our.Umbraco.Ditto/ComponentModel/Processors/UmbracoPickerAttribute.cs#L142-L154

Let me know what you think!

I realise there's going to need be a lot more work done in order to get this to work with the tests as UmbracoContext.Current & ApplicationContext.Current are both null. :/

Hey folks, struggling with this one. I've been back and forth with how this setup but in the end I took an idea from the core on UmbracoContext access (see notes here: https://github.com/leekelleher/umbraco-ditto/pull/204/commits/bd4c79aaf5ae61a2a83d1f6190e536a545063058).
Happy to rename things, the structure may not be how you guys would see the files being placed and the naming is a tad verbose. This was a bit of a brain racker!

I've managed to removed the static membershiphelper but removing the static dictionary helper might be a bit more challenging. I've left it in for now and it can be reviewed later on.

Hey @jamiepollock, thanks for taking it on and making a start with this! 🎉
I'm not sure what to make of the IUmbracoApplicationContextAccessor yet, (it's the first time I've heard about IUmbracoContextAccessor in Umbraco core too). I'll give it some thought.

@leekelleher Yea, I know what you mean. The whole reason is to have a testable in for UmbracoContext & ApplicationContext.
I've been back and forth with this over the weekend after I saw the first push failed I've been trying to get this in. In the end this was the best way I could find. I noticed the Umbraco core was doing something like this in their newer code for Umbraco Identity (https://github.com/umbraco/Umbraco-CMS/search?utf8=✓&q=SingletonUmbracoContextAccessor) and phasing out stuff like plugincontroller.
I'm not suggesting this is how we should go but I've certainly spent a good amount of time looking over a few options.
I realise adding it to the constructor is out as it'd need to be public to be testable and also then appear in the attribute setup.
It's been a fun one :)

On 22 Jan 2017, at 18:05, Lee Kelleher<EMAIL_ADDRESS>wrote:
Hey @jamiepollock, thanks for taking it on and making a start with this! 🎉
I'm not sure what to make of the IUmbracoApplicationContextAccessor yet, (it's the first time I've heard about IUmbracoContextAccessor in Umbraco core too). I'll give it some thought.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This is one of the biggest drawbacks to the "Context" pattern imo. Testing umbraco v5, which was uber context heavy, was a real pain as you end up spending ages mocking the rabbit hole.
RE the passing in attribute, you could make an internal accessor as I think we make internals visible to the testing library so it should be allowed to be set from the testing lib.

@mattbrailsford yea, I've not found it so bad. The goal with the accessor was to eliminate the need for any references to UmbracoContext in production code, as if I did put it in then it would cause tests to fail. Hence in the tests we can just setup a mocked up version and be done with it. In theory we could pass in an entirely different UmbracoContext & ApplicationContext constructed setup, somehow. not sure how that'd work mind or if it even should.
On top of this, the parameter nesting of passing the accessor result comes through comes about because we're using extension methods rather than a class we can DI something into.
I might be missing something though. I've tried going down the route of adding in internal properties this weekend and couldn't find the best place to populate those values in the production code which then wouldn't cause tests to fail. Hence coming back around to the accessor.
More than happy to take on edits, I might be close to the trees to see that wood. :)

@mattbrailsford I've looked at implementing the code review changes. Only one I've not implemented is helpers into the Context. I wasn't quite sure how to achieve that at this point or if we're going that way.

@jamiepollock no worries. I'm not 100% on that suggestion so happy for it to stay as is and we can always move it later if needs be. It wouldn't change the API on the processors so won't be a big deal to move if we need to.

Sorry, one last thing. I think I'd drop the use of the word "Default" for the UmbracoApplicationAccessor stiff as there can be only one anyway, so it's not like it's a default of a collection (like the Processor one is). Do you think?

Sounds like a good idea. It was based, as you can imagine, on the default processor type but we can't specify an accessor on a per model basis. I'll add it in later on when I get back home.

On 24 Jan 2017, at 11:19, Matt Brailsford<EMAIL_ADDRESS>wrote:
Sorry, one last thing. I think I'd drop the use of the word "Default" for the UmbracoApplicationAccessor stiff as there can be only one anyway, so it's not like it's a default of a collection (like the Processor one is). Do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@mattbrailsford sorted!

I've added an instance of UmbracoHelper to UmbracoDictionaryAttribute.cs.
I feel this might be something we could add at Processor level as it has useful methods outside of wrapping up Services methods or ContentCache.

@mattbrailsford @leekelleher Okay, chaps. I bit the bullet and added in a reference to UmbracoHelper along side all the other new additions. Much like Members (MembershipHelper) & UmbracoHelper referenced elsewhere this is simply called "Umbraco".
This means UmbracoContext has been renamed from Umbraco to UmbracoContext.
All references have been updated! I think we're getting closer! :D

I think the main stickler here is the naming of the UmbracoApplicationContextAccessor, As mentioned in this commit (https://github.com/leekelleher/umbraco-ditto/pull/204/commits/bd4c79aaf5ae61a2a83d1f6190e536a545063058) I took a cue from the core but that's probably not commonly used code as it was mostly put in for Umbraco Identity.
I don't think UmbracoContextAccessor is quite right as it also accesses ApplicationContext. In the current version of Umbraco we're using IUmbracoContextAccessor is internal but in newer versions it's public.
Happy to rename this to whatever you guys think is best :)

I've created a new feature branch called "feature/umbraco-context", I'll merge this in to there... then we can review/amend accordingly (then get it merged into "develop" branch).
@jamiepollock Thanks again for this, it's very much appreciated! #h5yr 💯

Hey guys, is there anything I can do to help this along? Just wondering where we're at. I'm sure you're all super busy at the moment. :)

@jamiepollock It's the first time I've had chance to look at Ditto in ages.  Reviewing stuff now and we'll see where it's at. 🤘

@leekelleher thats okay :) just want to make sure it's in the pipeline. I was happy with the code where I left off but time makes way for uncertainty.


MIAD-202 /bulk-alerts/review-prisoners screen

MIAD-202

Codecov Report
Attention: Patch coverage is 75.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 89.97%. Comparing base (30d3090) to head (bffef8e).

Files with missing lines
Patch %
Lines

...ourneys/bulk-alerts/review-prisoners/controller.ts
58.33%
5 Missing :warning:

...journeys/bulk-alerts/select-prisoner/controller.ts
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
- Coverage   91.06%   89.97%   -1.10%
==========================================
  Files          91       93       +2
  Lines        1980     2004      +24
  Branches      308      310       +2
==========================================
  Hits         1803     1803
- Misses        173      196      +23
- Partials        4        5       +1

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


bugfix/DeliveryFrequency

Added DeliveryFrequency to the cloudformation template for DeliveryChannel so that it records the state periodically, fixing issue where periodic config rules only ran once

Looks good! Merging.

@sjpalf Just realised, would you mind adding that new variable to the readme.md as well please? :)


Update Composer dependencies (2021-10-03-00-08)

Visual regression test passed!
View the visual regression test report


issue compiling example/led/all

Hi,
This is my first post on here so I hope I do it correctly!
When trying to make the example/led all command I get the error
CC /Volumes/case-sensitive/esp-open-sdk/esp-open-rtos/open_esplibs/libmain/ets_timer.c
make: xtensa-lx106-elf-gcc: No such file or directory
make: *** [build/open_esplibs_libmain//ets_timer.o] Error 1
I have tried this on Ubuntu (all the way updated, fresh install on virtual machine) and in macOS but keep getting the error.
Also on both machinesWhen ring to run the make STANDALONE command it always ends with:
checking how to run the C preprocessor... gcc -E
checking for ranlib... ranlib
checking for gobjcopy... no
checking for objcopy... no
configure: error: missing required tool: gobjcopy objcopy
make[1]: *** [_ct-ng] Error 1
make: *** [crosstool-NG/ct-ng] Error 2
Don't know if these are related but any help would be appreciated.
I did clone the repositories and set the PATH and SDK_PATH correctly I believe, its just missing the resource(?)
Also, I have looked into others having the same problem with the gobjcopy/objcopy and they noted it should not necessarily be an issue for native work, however I am not sure if that is the case or not.

I went through and redid everything the same again, but now I am getting a different error.
[INFO ]  Installing cross-gdb
[ERROR]    collect2: error: ld returned 1 exit status
[ERROR]    make[5]: *** [gdb] Error 1
[ERROR]    make[4]: *** [all-gdb] Error 2
[ERROR]    make[3]: *** [all] Error 2
[ERROR]
[ERROR]  >>
[ERROR]  >>  Build failed in step 'Installing cross-gdb'
[ERROR]  >>        called in step '(top-level)'
[ERROR]  >>
[ERROR]  >>  Error happened in: CT_DoExecLog[scripts/functions@257]
[ERROR]  >>        called from: do_debug_gdb_build[scripts/build/debug/300-gdb.sh@120]
[ERROR]  >>        called from: do_debug[scripts/build/debug.sh@35]
[ERROR]  >>        called from: main[scripts/crosstool-NG.sh@646]
[ERROR]  >>
[ERROR]  >>  For more info on this error, look at the file: 'build.log'
[ERROR]  >>  There is a list of known issues, some with workarounds, in:
[ERROR]  >>      'share/doc/crosstool-ng/crosstool-ng-1.22.0-60-g37b07f6f/B - Known issues.txt'
[ERROR]
[ERROR]  (elapsed: 42:47.87)
[42:48] / ct-ng:152: recipe for target 'build' failed
make[2]: *** [build] Error 2
make[2]: Leaving directory '/home/parallels/esp-homekit-demo/esp-open-sdk/crosstool-NG'
../Makefile:135: recipe for target '_toolchain' failed
make[1]: *** [_toolchain] Error 2
make[1]: Leaving directory '/home/parallels/esp-homekit-demo/esp-open-sdk/crosstool-NG'
Makefile:130: recipe for target 'crosstool-NG/.built' failed
make: *** [crosstool-NG/.built] Error 2
not sure about this one either, I have plenty of free disk space in both the .dmg file (11gb) on my main system (macOS) but also enough space available for the Ubuntu virtual machine being ran through parallels.
Any help would be appreciated.

@maximkulkin

This is the problem with esp-open-rtos, not with esp-homekit. You need to report to that project.

Its always what I obvious, isn't it.
Thanks!


refactor: group clusters, fix event iam, create efs

Fixes & refactor:

Launch asset discovery using FARGATE SPOT with FARGATE as the fallback
Remove volume from dockerfile since it's not needed as well as interfering with directory ownership
IAM permissions so that eventbridge can launch the step function
Discontinue one cluster per container. Clusters will now be grouped by service
Create EFS persistent mount so that Neo4J data isn't lost when containers cycle
Elasticsearch password special character list reduced due to conflicts with pythons urlparse library

The multiple containers to one EFS volume that access points provide isn't necessary for this use case. So we can continue with the mount target that will dedicate the entire efs volume to the Neo4j data directory.


Remove unneeded Deserialize code once JSONRPC crate is fixed

JSONRPC crate currently is broken in that it requires Serialize and Deserialize regardless of whether a type features only as an argument or a return.

@tomusdrw is there any progress on this in JSONRPC crate?

@gavofyork yup, it's backward compatible. Example how to set up Deserialize only bound here: https://github.com/paritytech/jsonrpc/pull/336/files#diff-d41fba166fb0f3c701b3199592549237R13
Requires implementation in substrate now.


Option to force all assets to be loaded at startup

Useful to validate all assets in the system during development.

@qcapen this looks resolved to me. should we close it?

Looks resolved to me as well.  Closing as suggested.


Change build pipeline

Enhance the existing automated build pipeline to remove dependency with external servers

Build workflow has been completely modified to make it independent of any external servers. Github Actions handled the whole process for both UI and server.


Blazor support?

Is this a Bug or Feature request?: Feature
Will Blazor be supported, or is it already in work to be supported?

Will Blazor be supported, or  is it already in work to be supported?

It's not currently supported but we do plan to add support for Blazor.
Right now (no promises) I'd keep an eye on the alpha 2 Razor VSCode release. What's currently planned will enable us to get a very basic Blazor experience. All that being said, alpha2 is a few months out, a lot can happen between now and then, so stay tuned 😉!
Leaving this open for any others who are curious.

Thanks for clearing up!

We expect to add C# and directive completions for Blazor apps in alpha 1, but there will be no support for tag helpers or blazor components. As a result, Blazor apps will likely display some false errors for our first release. We plan to address this for alpha2 as @NTaylorMullen indicated.

Closing this issue as the question was answered and we are tracking the work with other issues.


Bing Scraper not working

Hey aaron + the internet,
Seems like the bing scraper has broken. I'm on a mac and receive the following error when trying to scrape.
query bing images
script.py:126: DeprecationWarning: use driver.switch_to.frame instead
driver.switch_to_frame("OverlayIFrame")
Traceback (most recent call last):
File "script.py", line 222, in
scrape(scrapeSubject)
File "script.py", line 150, in scrape
popup.click()

hey thanks for pointing it out
i don't have much time right now to address it
but i did a quick so at least it doesn't break
hope that helps in the meantime =)


[Bug]: Nested contracts and relative paths to contracts do not compile correctly

What happened?
When compiling a multifile smart contract

where files are nested inside additional directories in the contracts,

OR

where the path to the files that make up the smart contract are relative paths,

do not compile.
What should have happened?
Taqueria should traverse through the directories inside the contracts folder to locate the files relevant to the smart contract.
AND
Taqueria should compile multifile smart contracts that are referenced using a relative path
Reproduction Steps:

Have a multi-file smart contract
Have the main contract in the /contracts folder
Have the additional files in a folder inside the /contracts folder
Compile the multi-file contract

Should be fixed with the contract registry @sinapsist needs to find dev ticket and close the bug.

Closing, fixed via contract registry work


Disable eBPF host routing in cni chaning mode

In cni chaining mode, the other chained plugin is responsible for underlying connectivity, so cilium eBPF host routing shoud not work, and let it fall back to the legacy routing mode
Fixes: #20135
Signed-off-by: smwyzi<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

Could you also add a note to the upgrade guide? For example, mention that setting cniChaining now auto-enables legacy host rotuing. https://github.com/cilium/cilium/blob/master/Documentation/operations/upgrade.rst#helm-options-1
Thanks!

Could you also add a note to the upgrade guide? For example, mention that setting cniChaining now auto-enables legacy host rotuing. https://github.com/cilium/cilium/blob/master/Documentation/operations/upgrade.rst#helm-options-1
Thanks!

done

According to checkpatch.pl, there also is a typo in your commit message, could you fix that too? Thanks!
  Running on b33759f6550eccc3539c86844f323f6b32885303
  Warning: WARNING:TYPO_SPELLING: 'shoud' may be misspelled - perhaps 'should'?
  #7:
  so cilium eBPF host routing shoud not work, and let it fall back to the legacy routing mode
                              ^^^^^

/test

@gandro hi，one workflow failed，but it seems not caused by this pr，could you give some advise，thx！

@gandro hi，one workflow failed，but it seems not caused by this pr，could you give some advise，thx！

Hey, unfortunately the CI links get lost when you push to the branch. Let me re-run CI again and take a look.

/test

CI triage:

The bpf checks job complains about a discrepancy between the author vs. the sign-off line: https://github.com/cilium/cilium/actions/runs/3433623465/jobs/5776142761
Travis reports known flake #21730 , no action necessary.
k8s-1.23-kernel-5.4 reports known flake #21735, no action necessary.

Could you fix up the commit signoff/author discrepancy per the report above? With that I think it's OK to merge this.

fixed

I'm not quite sure what's going on with CI but I suspect it's because the image builds failed. That doesn't seem to be related to your changes, so I think that if you address the latest feedback then hopefully next time that won't be a problem.


Add Blazor samples

It would be great to have one or more samples that show how to integrate the ChatGptNet library in Blazor projects.

I like the idea. Should we think about implementing it with Blazor Server or Blazor Wasm? The main difference would be that in Blazor Server we can actually inject the service directly into the page, meanwhile in Wasm we have to go though an API layer, unless the openai api supports also the CORS.

@nicolaparo, you're welcome if you want to work on the current samples! Can I assign this issue to you?

Yes, with pleasure! Also, since the two implementations are almost identical, should we start to focus mainly on Blazor Wasm?

Great, I have assigned the issue to you :-) Yes, let's start with Blazor Wasm, it makes sense.


onChange doesn't work

I tried to use dist/luminous.min.js (commit https://github.com/imgix/luminous/commit/1c53f53e2471022e95ebcd69567e948b6ef225b9) and found that onChange doesn't work:
new LuminousGallery(
  document.querySelectorAll('.gallery-item a'),
  {onChange: ({ imgEl }) => alert('This alert will never be triggered')},
);

But this code works:
new LuminousGallery(
  document.querySelectorAll('.gallery-item a'),
  {M: ({ j }) => alert('It works WHAT THE HECK!!!11')},
);

The minification minified things that shouldn't have been minified, lol

Hey @andreymal thanks for bringing this to our attention. I've made a note of this and will ask one of our engineers to investigate.


Add while_some; impl FromParallelIterator for Option/Result

The new while_some filters for Some values, and halts if it encounters any
None.  This is inspired by Itertools::while_some, although being parallel
means it doesn't consider the order.
Using this, we can implement a short-circuiting collect for Option and
Result like the standard library, returning just None or Err respectively
if any such item is found in the iterator.  Since this operates in parallel,
it's not deterministic which items will have already been visited, nor which
Err will be returned if there are multiple.

This is inspired by Itertools::while_some, although being parallel
means it doesn't consider the order.

For reference: Itertools::while_sum()
The difference is similar to Itertools::find vs. ParallelIterator::find_any, which we deliberately chose to name differently to avoid any ordering assumptions by the user.  I'm not sure of a good rename in this case -- while_all_some? until_any_none?  meh...

Interesting. I'm not too worried about the naming here.

I also thought of a similar idea for a plain while, something like an unordered take_while, but it's not as useful for the collects here since the predicate is separate from the value.  I'd have to do while(|x| x.is_some()) and then unwrap it in a separate step.  But maybe while could be useful in general.

Isn't while a keyword?

Ah, indeed.  Phooey.

The name while_some definitely only makes limited sense in the parallel context. I'm struggling to come up with a better one, though. It's basically "keep processing until a some is seen anywhere", so "while" is not inappropriate I guess.

I say we merge it =) I was debating if we wanted to call it "unstable" till we find a name we like better, but I think the analogy to itertools alone makes this name a good choice, and anyway we have the option of breaking changes to rayon for a while yet.

It's basically "keep processing until a some is seen anywhere"

The opposite - while some, until none.

anyway we have the option of breaking changes to rayon for a while yet.

Indeed!  We've already made a few since 0.7 too. :)

The opposite - while some, until none.

Right yes


Clock hang up

Hi
more than once a day the clock quits working.... after a few hours pm2 restarts automatically and the clock works again for a while....
Platform [ Raspberry Pi 3]:
Node Version [latest ]:
MagicMirror Version [ V2]:

Please check http://forum.magicmirror.builders for support. Thanks!


Add plugin FAIL

ionic cordova plugin add https://github.com/arnesson/cordova-plugin-firebase
(node:1648) UnhandledPromiseRejectionWarning: CordovaError: Failed to fetch plugin https://github.com/arnesson/cordova-plugin-firebase via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
OR
cordova plugin add cordova-plugin-firebase --save
stay stuck
OR
npm install cordova-plugin-firebase
Stuck Here "extract:fsevents: sill pacote<EMAIL_ADDRESS>extracted to..."
And my connection is OK.

AND
npm install cordova-plugin-firebase
Stuck Here "extract:fsevents: sill pacote<EMAIL_ADDRESS>extracted to..."

Finally I had no problem:
cordova plugin add<EMAIL_ADDRESS>--save

closing as resolved

still showing issue

(node:2501) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): CordovaError: Failed to fetch plugin<EMAIL_ADDRESS>via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: npm: Command failed with exit code 1 Error output:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

npm ERR! code EINTEGRITY
npm ERR! sha1-SFwMaF6cwyJtdco+L7l9xiDNB+Q= integrity checksum failed when using sha1: wanted sha1-SFwMaF6cwyJtdco+L7l9xiDNB+Q= but got sha1-QWCJLAk+6IfRUYulxXVroPIqNa0=. (13187910 bytes)
npm ERR!  In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Now the version is 1.0.4. Each time you change the version, a connection problem occurs.
@Anujmoglix you can try cordova plugin add<EMAIL_ADDRESS>--save

Same problem with 1.0.4 version
<EMAIL_ADDRESS>--save
it working

cordova plugin add cordova-plugin-firebase hangs stuck
And all the specific versions mentioned here are stuck too.
I'm using latest Cordova and latest Android SDK

still stuck on it
many time used Ctrl+c
C:\Drive D\test\ionic\fcmDemo>ionic cordova plugin add cordova-plugin-firebase

cordova plugin add cordova-plugin-firebase --save

C:\Drive D\test\ionic\fcmDemo>
C:\Drive D\test\ionic\fcmDemo>ionic cordova plugin add cordova-plugin-firebase

cordova plugin add cordova-plugin-firebase --save

C:\Drive D\test\ionic\fcmDemo>
C:\Drive D\test\ionic\fcmDemo>ionic cordova plugin add cordova-plugin-firebase

cordova plugin add cordova-plugin-firebase --save

C:\Drive D\test\ionic\fcmDemo>
C:\Drive D\test\ionic\fcmDemo>ionic cordova plugin add<EMAIL_ADDRESS>
cordova plugin add<EMAIL_ADDRESS>--save

C:\Drive D\test\ionic\fcmDemo>ionic cordova plugin add<EMAIL_ADDRESS>
cordova plugin add<EMAIL_ADDRESS>--save

yes, with latest nodejs and waiting over 10 min,it's done.


Need uniformity in buttons within Learn Mores

Across the app wherever we have Learn More, the buttons should be "Ok" and "Need Help?" Right now there isn't consistency in buttons within Learn More.
https://github.com/bithyve/bitcoin-keeper/assets/68264080/9f5b1d8a-b256-488d-950c-f63e8d1b4efd

Completed can see this changes


Intellisense based on template.

What problem does this feature solve?
Improve DX. User will have automatical intellisense. For example:
before
<script setup>
import type { CompRenderFunc } from 'xxx-ui'

const renderFunc: CompRenderFunc  = ({ /** intellisense! */ }) => {
  return /** intellisense! */
}
</script>

<template>
  <Comp :render-func />
</template>

after
<script setup>
const renderFunc = ({ /** intellisense! */ }) => {
  return /** intellisense! */
}
</script>

<template>
  <Comp :render-func />
</template>

What does the proposed solution look like?
Maybe unplugin-auto-import will be helpful.

It's not possible. In ts you can't do:
const foo = (props) => {}

bar(foo) // Can't infer props' type from this usage

Sorry for my ambiguous words. I mean, we have already got a Comp component, and we have the type declaration of bar. Like:
<script setup lang="ts">
// Comp.vue

defineProps<{
  bar: (data: number) => string
}>()
</script>

<template>
  <div />
</template>

When we use it, maybe there is some way to achieve like that:
<script setup>
import Comp from './Comp.vue'

const bar = (data /** intellisense - number */) => {
  return /** intellisense - string */
}
</script>

<temlpate>
  <Comp :bar />
</template>

It is just like below in ts:
// first we have a type declaration of bar
function bar (foo: (data: number) => string) {}

// so we can have intellisense when we use it
bar((/** intellisense */) => { return /* intellisense */  })

I know what you mean, but your two code examples have different semantics. The first defines the function separately, while the second passes it directly as an anonymous function as a parameter.

Thanks for your answers, that is a pity 😢.


Annotate manifests for single-node-developer cluster profile

This partially implements phase 1 of openshift/enhancements#482
and does not change behavior. Initially, all cluster-image-registry-operator
manifests are included in the single-node-developer cluster profile.
Follow-on PRs may exclude any of these that are not needed in the
profile.

/lgtm

/retest


Fix certificate template generation

Added createTemplate() to fill a certificate template with all required fields.
Added a line to the Makefile in test directory to compile libcryptosec before running tests.

The new method is actually not necessary since it is basically a macro to two existing functions.
We'll take in some of the makefile fixes but not the template function.


Use biber instead of bibtex for bibliography processing

Bibtex with its lack of unicode support and also lack of support of @online and @digital reference classes is deprecated and the class should rather us Biber in conjunction with biblatex (if must with the natbib compatibility option).
Any thoughts?

I just checked with the biblatex documentation, because I do not know the differences too well.
In summary, traditionally the use of bibliography is this:
\documentclass{...}
\begin{document}
\cite{...}
...
\bibliographystyle{...} % This will be ignored, because
\bibliography{...}      % ...this currently includes \bibliographystyle{plainnat}
\end{document}

Biblatex would require this:
\documentclass{...}              % We load natbib and doi here
\usepackage[style=...]{biblatex} % A user can add this package to their document
\addbibresource{...}             % ... and load resources
\begin{document}                 % This is the same
\cite{...}                       % as is this...
...
\printbibliography               % If no \bibliography is used, this should work
\end{document}

According to the biblatex documentation, the default processing of the document also needs to be changed in order for biblatex to work properly (run biber instead of bibtex). I am not sure, many people know how to do this... I myself haven't changed this for my documents yet...
But, actually, nothing should stop an author from just using biblatex, if they wish to. Unless our natbib and doi packages interfer with biblatex...
How about we use the default processing (i.e. bibtex)? But we make sure, we do not load packages imcompatible to biblatex? Authors may then decide to use the traditional toolchain or to switch to biblatex on their own account.
Or, we force authors to switch to biblatex. But we should then be prepared to receive a few questions...

As far as I know, biber is now shipped with all LaTeX distributions. But you are right not many know about this. Just like many are clear over that the command latex is nowadays an alias for pdflatex with a post-script option (i.e., a fall back mode really).
I'm fine with having the example file use bibtex and not biber. The problem is that the current modelica.cls file conflicts with biblatex since it loads the conflicting natbib package.
I can think of 2 solutions:

(more elegant): Add an option backend=bibtex|biber to the modelica class (just like the biblatex package has) which defaults to bibtex for now. This will then load the correct compatible packages and settings.
Remove all bibliography related things from the class file and put this in the example-paper.tex instead.

As an example have a look at these biber & biblatex settings to get the same output as natbib.

There are people who do not use latexmk? There are so many commands you need to remember to run if you do things manually that I do not think this is an issue.

We've just established that the presence of latexmk is not guaranteed on LaTeX systems. Though using it is an improvement we should add a check to the Makefile and use fallback option in parallel.

See above pull-request. Better to have somebody look at it before I force it in.


User gets 404 after exception handling

Hi,
I was trying to respond to an invalid request by using this code:
exception(Exception.class, (exception, request, response) -> {
      System.err.println("=========================");
      System.err.println("Exception: " + e.toString());
      final StackTraceElement element = e.getStackTrace()[2];
      System.err.println("Thrown at: " + element.getClassName() + "." + element.getMethodName() + ":" +
              element.getLineNumber());
      System.err.println("Request IP: " + request.ip());
      System.err.println("Request body: " + request.body());
      System.err.println("=========================");
      response.status(400);
});

This request will respond with a 404 code. If I'm submitting a body like "bad request" the response code is still 404.
Thank you.

Hi!
I don't know(now) why, but if you will set body in response
response.body("some body"); then it will be working (request will respond with a 400 code)

Yes, if body.notSet() the status will be 404.
You can see this in spark.http.matching.MatcherFilter#doFilter

i have the same issue
I would like to be able to customize the response code simply with response.status(500) and avoid to have to set the response.body to prevent spark to overwrite my code


Adding "Open Source School"

@John-Loup http://oss.fr seems to be a website of some industry-oriented commercial venture, not a university, nor a college. Could you please clarify why you have requested to add this domain to this repository?

@philipto Hello, OSS means "Open Source School", here is the right website : http://www.opensourceschool.fr/

@John-Loup Thank you. Please note that you requested to add a file lib/domains/fr/oss.txt which corresponds a domain of oss.fr. Please ensure you are familiar with this repository rules located at the bottom of this page and rename the file to correspond to your actual email domain. Thank you!

Oh, my mistake.
Done !

@John-Loup Pull request merged. Thank you!


usb: dc_dw: Improve TX perf and reliability

During large USB transfer it's pretty common to call ep_write whereas
the previous TX transfer is not achieved and so the TX FIFO space is
not available. Sleeping 20ms in this case introduce a relatively high
latency and reduce the throughput.
This can be observed when pinging the board with CDC-ECM net class.
ping reply is split into 2 USB TX transfer, the second one is only
triggered after 20ms, making ping latency > 20ms.
To fix this, just continuously read the FIFO availabilty and fire TX
as soon as possible.
Signed-off-by: Loic Poulain<EMAIL_ADDRESS>

Have you tested it working?

Yes tested with USB-ECM on genuino 101.


Fix crash with guessing return type

Minimal sample to reproduce crash:
function noReturnDocTag() {
  const o = {};
  return { ...o };
}

Esdoc try to guess function return type when not @return tag found. When return type is Object, most times it can determine it's shape from AST. But when this object contains rest spread operator, it became compicated (for ex return { ...obj }, theorethically we can try track deeper and determine shape of obj, but for now we simple break guessing with *). In any case, this commit fixed crash with this case, because when AST node contain object with type SpreadProperty, it of course has no key.

Anyone? (If it helps, I can rebase branch to latest master)

I just rebase and fixed this branch.
@h13i32maru, can you look at this, please! With this fix esdoc now can successfully parse our react-native project.

I just rebase and fixed this branch.
@h13i32maru, can you look at this, please! With this fix esdoc now can successfully parse our react-native project.

@vovkasm I got the same problem, you can try my fork if you can not wait

@vovkasm I got the same problem, you can try my fork if you can not wait

@vovkasm Sorry for late reply 😓
Thanks for this PR!

And I improved to guess object spread.
https://github.com/esdoc/esdoc/commit/b613502c8eaa010979f3344012a2543801c18d38
Thanks.

Cool! Thank you!


Update README.md to display screenshot

@ldanzinger ready for review. Thanks.

please feel free to merge once reviewed.


[BUG] - Databricks absolute path must be written differently

Description
I try to distribute my ingestion areas on several containers in an azure data lake mounted on a dbfs path on the databricks platform (ex: /mnt/starlake).
For that I specified in environment variable SL_AREA_PENDING=dbfs://mnt/pending_area for example for the pending area.
Problem, when you specify an absolute path on databricks with the filesystem included in it, you have to write it with a single '/'.
%fs ls dbfs://mnt/starlake gives us this error:
IllegalArgumentException: Hostname not allowed in dbfs uri. Please use 'dbfs:/' instead of 'dbfs://' in uri: dbfs://mnt/starlake
While %fs ls dbfs:/mnt/starlake works correctly.
However I can't change the absolute path of SL_AREA_PENDING=dbfs:/mnt/pending_area because according to your code, in DatasetArea.scala line 49, a path is recognized as absolute only if it contains "://".
So if I include the variable this way, starlake will recognize this path as relative and concatenate me SL_DATASETS with SL_AREA_PENDING : "dbfs:/mnt/datasets_area/dbfs:/mnt/pending_area"
This will cause an error at runtime.
So it might be useful to change this condition to contains(":/") or just create a specific condition for databricks.

Thank you very much @hayssams, using dbfs:///mnt/pending_area as SL_AREA_PENDING is working fine !
However, I have an other trouble here.
When doing the import with this setting, starlake pushes all the source files without filtering by domain. This was done by default in the DatasetArea.scala path function that would add the /$domain variable at the end of the path. But it doesn't do it anymore.
So in my situation if I wanted to create the domain directories in my pending area, I thought using dbfs:///mnt/pending_area/$domain would work in my cluster environment. It didn't change anything ($domain was ignored), so I tried dbfs:///mnt/pending_area/{domain} instead and it created a {domain} directory.
Do you know how can I use variables in my cluster environment ?

Could you try {{domain}} instead and let me know

Can We setup a call 5:30 PM?

Sure ! Here is my email<EMAIL_ADDRESS>


Fixed proxy types matching

#8693e7xgf

Task linked: CU-8693e7xgf Android. Not enough permissions for start staking, vote on Staking, Governance proxy


implement mimeo-style regex handlers

Addresses #57 and sort of indirectly addresses #56.
I added the ability to associate arbitrary commands with files and URLs that match regular expressions, similar to what another resource opener, mimeo, supports. See my changes to README.md for detailed info on how it would be used.
They would be defined in ~/.config/handlr/config.toml, which I figured would be fine since it was previously only used for two options.
However, I have only implemented this functionality as it pertains to  handlr open. So they will currently not be included in handlr list or handlr get or assignable with handlr set.
Also, I should mention that I used sort of a hack that essentially creates a faux instance of DesktopEntry so that I could leverage the existing command execution code without making major changes to it.
At one point, I had to update handlr's dependencies to fix an issue with the regex crate, so I had to fix a few lines of unrelated code due to a breaking change in clap.
Edit: I think updating Clap makes this pull request address #53 as well.

After no response for about a month and no activity on this repo since August 2021, I have decided to spin off my fork as its own project.
Github repo: https://github.com/Anomalocaridid/handlr-regex
AUR packages: handlr-regex, handlr-regex-bin
However, just in case this repository becomes actively maintained again, I will keep this pull request open and maintain the associated branch. I would still be happy to have my changes added to handlr if the maintainer would be open to that.
Needless to say, any questions or concerns should be directed towards my fork's Github repo or AUR listing rather than made here unless it pertains to this pull request and whether it will be merged or not.

More than two years after I made this PR, I have long since accepted this repo is dead.
To the owner of this repo: thanks for everything and I wish you nothing but the best.


Blocks / loops no longer works in Appfile

In previous version of Fastlane (0.15.0) I was able to use blocks / loops to make something like:
app_identifier "*"
apple_id<EMAIL_ADDRESS>
for_platform :ios do
  [
    :lane_name1,
    :lane_name2,
  ].each do |lane_name|
    for_lane lane_name do
      app_identifier "*"
      apple_id<EMAIL_ADDRESS>      team_id "TEAMID"
    end
  end
end

But since version 0.16.0 I'm receiving this error:
/Library/Ruby/Gems/2.0.0/gems/credentials_manager-0.7.1/lib/credentials_manager/appfile_config.rb:10:in `rescue in try_fetch_value': uninitialized constant CredentialsManager::AppfileConfig::Helper (NameError)
	from /Library/Ruby/Gems/2.0.0/gems/credentials_manager-0.7.1/lib/credentials_manager/appfile_config.rb:7:in `try_fetch_value'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-0.10.3/lib/sigh/options.rb:33:in `available_options'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/actions/sigh.rb:42:in `available_options'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/configuration_helper.rb:17:in `parse'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/runner.rb:134:in `block (2 levels) in execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/actions/actions_helper.rb:62:in `execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/runner.rb:128:in `block in execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/runner.rb:127:in `chdir'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/runner.rb:127:in `execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/fast_file.rb:120:in `method_missing'
	from (eval):67:in `block (3 levels) in parse'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/lane.rb:32:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/lane.rb:32:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/runner.rb:50:in `block in execute'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/runner.rb:44:in `chdir'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/runner.rb:44:in `execute'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/lane_manager.rb:36:in `cruise_lane'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/lib/fastlane/command_line_handler.rb:31:in `handle'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/bin/fastlane:35:in `block (2 levels) in run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:153:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/runner.rb:428:in `run_active_command'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/runner.rb:68:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/delegates.rb:15:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/bin/fastlane:124:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.16.0/bin/fastlane:130:in `<top (required)>'
	from /usr/bin/fastlane:23:in `load'
	from /usr/bin/fastlane:23:in `<main>'

That's not good, I'll look into it as soon as possible.

I can reproduce this issue and am working on a fix.

There is a small progress since version 0.17.0. Now when I'm running Fastlane I'm receiving:
[13:22:48]: -------------------------------------------------
[13:22:48]: --- Step: Verifying required fastlane version ---
[13:22:48]: -------------------------------------------------
[13:22:48]: fastlane version valid
[13:22:48]: ------------------------------
[13:22:48]: --- Step: default_platform ---
[13:22:48]: ------------------------------
[13:22:48]: Driving the lane 'ios lane_name1' 🚀
[13:22:48]: ------------------
[13:22:48]: --- Step: sigh ---
[13:22:48]: ------------------
undefined method `call' for #<Array:0x007fad925a2ea8>
undefined method `call' for #<Array:0x007fad925ba788>
[13:22:48]: Starting login
[13:22:48]: To not be asked about this value, you can specify it using 'username'
Your Apple ID Username:

It is asking me for Apple ID, but I've just provided it in the Appfile.
When I remove for_platform and use only each block with for_lane everything starts working.

I just pushed a new version, please update using sudo gem update credentials_manager :+1:

Let me know if that works for you

Yes, it works. Thanks!

Perfect :rocket:


Made following enhancements:

Instead of using urllib, moved to requests
Added support of python 3.x

Definitely better to use requests than urllib.

Definitely better to use requests than urllib.


x/crypto/ssh: make ClientConfig HostKeyCallback non-permissive by default

ClientConfig.HostKeyCallback interprets nil as "accept any host keys". This is not a great default from a security perspective. Many clients probably should set HostKeyCallback to something real but are not.
It was written this way in golang.org/cl/9922043 to preserve backwards compatibility with the original implementation, but that was probably not the right balance to strike.
This issue is to make HostKeyCallback=nil mean "reject all host keys" and at the same time provide at least
func InsecureSkipVerifyHostKey() HostKeyChecker
func FixedHostKey(key []byte) HostKeyChecker

and maybe also
func HostKeysFile(file string) HostKeyChecker

Thanks to Phil Pennock for pointing out this problem.

see https://go-review.googlesource.com/c/38701/

For reference, this is now: CVE-2017-3204

Hello,
This issue not occurred on "go1.8.1 darwin/amd64". I think that "go1.8.1 darwin/amd64" and " go1.8.1 linux/amd64" is not same in ssh package.

Hello again.
I forget do "go get -u github.com/pkg/sftp" with "-u" flag. Sorry for attention :(

I had a question about the approach used to solve this.  I understand the reason for the breaking change, but I'm wondering if there is a way to implement this so that the "break" occurs during build as opposed to at run time.  That would lessen the impact and the surprises that come up after a build was successful and then the first run in production fails for some error that you've never seen before.
Thoughts?

@CameronGo, had there been a nice solution with that property (failing at compile time instead of run time), we would've used it. But e.g. renaming renaming ssh.ClientConfig to ssh.ClientConfig2 would've been super gross.

Understood - (and agreed on the latter point for sure).

How can agent forwarding be supported this this case, the mux ssh server
client -> mux -> remote


Infinite reverse scroll breaks on Safari

Describe the bug
Infinite reverse scroll breaks on Safari. It only updates layout when scroll is released for more couple hundreds of milliseconds
To Reproduce
Scroll to top and add items on top of the list when range is near first element
https://github.com/yokenfoundation/bilft/blob/992177dbabb21df29ee312d2aef0fd18659019e6/source/features/CommentsPage/CommentsPage.tsx#L770-L781
https://github.com/user-attachments/assets/66e979b9-e3a4-4a99-ac70-9ad219a017a1
Expected behavior
It must update list while scrolling
Platform:

OS: iOS
Browser: Safari
Version of this package: 0.33.3
Version of framework: solid-js [1.8.17]

Additional context
Telegram mini app safari

https://github.com/inokawa/virtua/issues/403#issuecomment-1978126177


[OpenMVG] Fix open mvg

Describe the pull request

What does your PR fix? Fixes linking to flann_s due to #16724

Which triplets are supported/not supported? Have you updated the CI baseline?
All triplets.
Updated versions/baseline.json

Does your PR follow the maintainer guide? yes.

Failed to create static pcl.

Yes, this is fixed in the other PR #16792.

Closing as it is included in #16724.


ci(actions): detach the lint workflow

Split CI workflows into lint and tests, working in parallel
Affected Packages
None

Can the incremental tests discussed earlier be submitted?

Yes, it will be in the next PR


docker with invalid command returns status code of 0 in 1.13 rc

Description
I noticed when trying to work with the official docker:1.13-rc2-git tag, that I couldn't execute any non-docker commands, and it seems to be because in 1.13-rc's that running docker nonsense --help returns a 0 status code (successful), whereas in previous versions it would return 1.
Steps to reproduce the issue:

from your host, run docker run --rm -ti docker:1.13-rc2 sh (or have 1.13-rc2 installed on the host instead)
run docker somenonsense --help

Describe the results you received:
I was presented with the docker command list / usage output and a status code of 0 (success).
 Likely the output should have also included a statement that the command was not valid.
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
Client:
 Version:      1.13.0-rc2
 API version:  1.25
 Go version:   go1.7.3
 Git commit:   1f9b3ef
 Built:        Wed Nov 23 17:40:58 2016
 OS/Arch:      linux/amd64

Server:
 Version:             1.13.0-rc2
 API version:         1.25
 Minimum API version: 1.12
 Go version:          go1.7.3
 Git commit:          1f9b3ef
 Built:               Wed Nov 23 17:40:58 2016
 OS/Arch:             linux/amd64
 Experimental:        false

Output of docker info:
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 0
Server Version: 1.13.0-rc2
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 0
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 03e5862ec0d8d3b3f750e19fca3ee367e13c090e
runc version: 51371867a01c467f08af739783b8beafc154c4d7
init version: 949e6fa
Security Options:
 seccomp
  Profile: default
Kernel Version: 4.4.27-moby
Operating System: Alpine Linux v3.4 (containerized)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 1.951 GiB
Name: 4d42f7a94662
ID: 7IMJ:46PA:IDUG:D4NM:EX36:EX5B:B3OP:BZK7:4EGV:L64S:DJE2:E6XB
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
No Proxy: *.local, 169.254/16
Registry: https://index.docker.io/v1/
WARNING: No kernel memory limit support
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Actually, closing. It's a dupe of #28329


How do I load configuation in 2.2.1

I just updated to 2.2.1 and have the issue that I can not find the way to load my global/server configuation with the new storage management. Is all my work I put in the config gone and I can not load it anymore ?
Thanks for your help

Hi. It should have upgraded the local browser config. Unless you had never connected to KIP with that device. Is this the case?
Good news is it's not gone!
Look for ~/.signalk/applicationData/global/kip/1.0.0.json. Make a backup.
Rename to 9.0.0.json and see if you can see it from KIP Configuration tab.
If you use Sign in. You have to copy global to "user / default".
If not using Sign in, copy to local storage.

Thank you very much. Will try to do it with the jason. Have a nice day. I really like kip

Me pleasure. Let me know if it works.

Just released 2.2.2 that fixes configuration upgrade issues. Try it out and provide feedback.
Thanks for your patience!

FYI. V2.2.2 was released and fixes the problem. Please try this version and report back. Thank you!


Display collected container labels and selectors in the UI

Labels table was added in the views of the following entities:

Container Groups (pods)
Replication Controllers
Services

Selector table was added in the views of the following entities:

Replication Controllers
Services

Please notice it was added in container_summary_helper for code reusability.

@miq-bot add_label providers/containers

@miq-bot add_label ui

@h-kataria @simon3z @abonas please review

@miq-bot add_label UI

How does it look when there are a lot of labels and a lot of selector parts? is each section internally scrollable or the whole page becomes scrollable?
For selectors I recommend a name change - each entity has only one selector, and what's plural there are the selector's parts. (as it's also called in the model itself). So either it has to be called selector or selector parts (we can see what other reviewers think is most clear), but selectors is not the right name.

Selector table was added in the views of the following entities:
Container Groups (pods)

seems like a mistake, cg don't have selector, replicators do. I think the code itself is correct, only the description above is not.

abonas commented 6 minutes ago
Selector table was added in the views of the following entities:
Container Groups (pods)
seems like a mistake, cg don't have selector, replicators do. I think the code itself is correct, only the description above is not.

fixed, thanks!

How does it look when there are a lot of labels and a lot of selector parts? is each section internally scrollable or the whole page becomes scrollable?

the whole page becomes scrollable:

For selectors I recommend a name change - each entity has only one selector, and what's plural there are the selector's parts. (as it's also called in the model itself). So either it has to be called selector or selector parts (we can see what other reviewers think is most clear), but selectors is not the right name.

Agreed. I'll change it to Selector for now (as this is how it appears in the api) but i'm open for other opinions :)

@miq-bot add_label enhancement

@abonas @simon3z Please review.

@dkorn can you try to fix the rubocops warnings? (I know that you're trying to be consistent with the style of the file but we should make sure not to introduce new code that is not rubocop-compliant). Thanks!

@simon3z alternatively, we can merge this PR matching the style in the code and a separate PR can do a Rubocop cleanup.

@abonas Can these tables be empty? If so, what is the correct behaviour for the UI in this case?

@dkorn also nodes can have labels

@abonas Can these tables be empty? If so, what is the correct behaviour for the UI in this case?

well, an entity doesn't have to have a label.
I would present an empty section in such case to indicate that there are no labels (as opposed to not presenting anything which can be interpreted as "not displaying labels in general")

@dkorn also nodes can have labels

It seems that node labels aren't parsed and saved at the moment. After speaking with @abonas I prefer to add the beckend + UI for node labels in a separate PR. Is that OK?

well, an entity doesn't have to have a label.
I would present an empty section in such case to indicate that there are no labels (as opposed to not presenting anything which can be interpreted as "not displaying labels in general")

@zeari this is how it would look like:

@abonas, @simon3z, this looks probably ready to merge .. thoughts?
@dclarizio, you wanna give this a quick check?  LGTM

cc @abonas LGTM :+1:


Missing parentheses in z-score calculation?

Noticed this line when enabling z-score calculation for expected rewards.  Should it be
if z_score:
    expected_rewards = (
        ( expected_rewards - expected_rewards.mean() ) / expected_rewards.std()
    )

?

@zhnhou
Yes I agree.


IPFS performance improvments and experiments

We need to get to the bottom of why "providing" time for the experiments (https://github.com/lazyledger/ipld-plugin-experiments/pull/9) is different to the MVP, and in general, work towards more confidence in the performance and scalability of IPFS.
Experiments that can be done:

Remove sleep from ipld-plugin-experiments https://github.com/lazyledger/ipld-plugin-experiments/blob/a4c8edf11ac9a465b0053f124804a64c637acb0e/experiments/clients/main.go#L62
https://github.com/lazyledger/ipld-plugin-experiments/issues/4
https://github.com/lazyledger/ipld-plugin-experiments/issues/10 (use NMT in experiments)
In lazyledger-core MVP, what's the total time to sample a chunk, when the sample request is made immediately after the block is generated (but before provided is finished) and when the block interval is 30 seconds?

Experimental improvements:

What's the providing time when a private IPFS network is used?
What's the providing time when Hydra nodes are added as bootstrap nodes?
What's the minimum providing time when multiple validators are providing the same block?
What's the providing time when new DHT (crawler-based) client used?

In lazyledger-core MVP, what's the total time to sample a chunk, when the sample request is made immediately after the block is generated (but before provided is finished) and when the block interval is 30 seconds?

@musalbas, I guess you mean unconnected node. I've done enough experiments to say that it would take >1.5min

But that's on real IPFS network, in the case of a private network the time would be much lesser, but we still need to do deploy it and exec experiments

If we imagine that our network grows to the size of the IPFS's one, I am confident that there is no way around it, besides moving on new DHT client with full routing table

In lazyledger-core MVP, what's the total time to sample a chunk, when the sample request is made immediately after the block is generated (but before provided is finished) and when the block interval is 30 seconds?

@musalbas, I guess you mean unconnected node. I've done enough experiments to say that it would take >1.5min

Is this also true for subsequent blocks?


How can I get positions of markers

In the following case, how can I get the positions of markers?
I would like to measure the distance between two markers ("hiro" and "kanji").
<a-scene>
  <a-marker preset="hiro">
    <a-box></a-box>
  </a-marker>
  <a-marker preset="kanji">
    <a-box></a-box>
  </a-marker>
  <a-entity camera></a-entity>
</a-scene>

Take the box, computer the world position
It select the market entity and look at the position of the 3d object
On Mon, Sep 11, 2017, 12:27 AM Mitsuteru Kokubun<EMAIL_ADDRESS>wrote:

In the following case, how can I get the positions of markers?
I would like to measure the distance between two markers ("hiro" and
"kanji").

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jeromeetienne/AR.js/issues/183, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPcIrjxQIbW2yE4OJBjHQOrTrrnTunIks5shHBVgaJpZM4PSfhw
.

I'm looking for a solution to this as well, not sure I understand the solution posted @jeromeetienne - I've been trying to get the position of a marker and the position of the camera using getAttribute('position') and both are returning 0,0,0 - Is there a way to get the camera's position in relation to the markers position?

My understanding, at least for the three.js version of these applications, is that the camera is always at (0,0,0) and the rest of the scene is rotated according to the position of the marker. For the position of the marker, you could try the function getWorldPosition() -- perhaps (0,0,0) is its local position?

This PR has been closed due to inactivity. Please open a new, updated issue if the problem or the question is still relevant.


Error while model_training

This is the error I am encountering while running model_training.py
  File "C:\Users\Vishishta\Desktop\pr\computer\model_training.py", line 8, in <module>
    X_train, X_valid, y_train, y_valid = load_data(input_size, data_path)
  File "C:\Users\Vishishta\Desktop\pr\computer\model.py", line 30, in load_data
    X = np.vstack((X, train))
  File "C:\Users\Vishishta\AppData\Local\Programs\Python\Python36-32\lib\site-packages\numpy\core\shape_base.py", line 234, in vstack
    return _nx.concatenate([atleast_2d(_m) for _m in tup], 0)
ValueError: all the input array dimensions except for the concatenation axis must match exactly

It seems it's an array shape mismatch issue, double check your data shape as well as the shape of X_train, X_valid, y_train, y_valid.

Hey hamuchiwa,
Thanks for the reply, I have found the error and got it fixed. I just changed input_size to 76800 from 320*120 and it worked.
Thanks again. Keep up the good work.


KB size's toHumanReadable() method is incorrectly returning the size as MB

toHumanReadable() method is not returning the correct human readable string for KB

@irtazanasir Please review the latest version. This is fixed.


I can't download this video

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.02.08. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2019.02.08

Before submitting an issue make sure you have:

[ x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[x ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: [u'--no-mtime', u'-o', u'/home/josh/queue/%(title)s.%(ext)s']
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-o', u'/home/josh/best hits backup/%(title)s.%(ext)s', u'https://www.youtube.com/watch?v=qHI16SXjiZY']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.02.08
[debug] Python version 2.7.12 (CPython) - Linux-4.15.0-45-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: ffmpeg 2.8.15, ffprobe 2.8.15
[debug] Proxy map: {}
[youtube] qHI16SXjiZY: Downloading webpage
[youtube] qHI16SXjiZY: Downloading video info webpage
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://r1---sn-vgqsrnll.googlevideo.com/videoplayback?txp=5431432&keepalive=yes&requiressl=yes&ipbits=0&sparams=aitags%2Cclen%2Cdur%2Cei%2Cgir%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Ckeepalive%2Clmt%2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Crequiressl%2Csource%2Cexpire&fvip=3&ms=au%2Conr&source=youtube&mv=m&ip=<IP_ADDRESS>&key=yt6&lmt=1550227398631048&id=o-AP8SldgY78dRIOJ6OFKr4LfUjG827nDqxYzeEhn9BO6N&dur=99.040&expire=1550279677&mn=sn-vgqsrnll%2Csn-ab5l6nsy&gir=yes&itag=248&clen=10180830&ei=nQ9nXIXGM5aaiwS0gZPYDg&mt=1550257967&pl=16&mm=31%2C26&c=WEB&initcwndbps=1835000&mime=video%2Fwebm&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&signature=132460EA51FF6C6DE8D7EA9EED892221EB7D0176.163608AEC47B62726A5CF1CA1C3DEF413BEA06E8&ratebypass=yes'
ERROR: unable to open for writing: [Errno 36] File name too long: '/home/josh/best hits backup/\xe3\x81\x8b\xe3\x81\x90\xe3\x82\x84\xe7\x89\xb9\xe9\x9b\x86 1_6\xe3\x80\x90\xe4\xb8\xad\xe5\x9b\xbd\xe4\xba\xba\xe3\x81\x8c\xe6\x97\xa5\xe6\x9c\xac\xe3\x81\xae\xe6\x9b\xb2\xe3\x81\xa7\xe8\xb8\x8a\xe3\x81\xa3\xe3\x81\xa6\xe3\x81\xbf\xe3\x81\x9f\xe3\x80\x91\xe3\x83\x81\xe3\x82\xab\xe3\x81\xa3\xe3\x81\xa8\xe3\x83\x81\xe3\x82\xab\xe5\x8d\x83\xe8\x8a\xb1\xe3\x81\xa3\xe2\x99\xa1\xe3\x80\x90\xe5\x92\xac\xe4\xba\xba\xe7\x8c\xab\xe3\x80\x91\xe3\x80\x90\xe3\x81\x8b\xe3\x81\x90\xe3\x82\x84\xe6\xa7\x98\xe3\x81\xaf\xe5\x91\x8a\xe3\x82\x89\xe3\x81\x9b\xe3\x81\x9f\xe3\x81\x84\xe3\x80\x91.f248.webm.part'
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 250, in download
    ctx.tmpfilename, ctx.open_mode)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 458, in sanitize_open
    stream = open(encodeFilename(filename), open_mode)
IOError: [Errno 36] File name too long: '/home/josh/best hits backup/\xe3\x81\x8b\xe3\x81\x90\xe3\x82\x84\xe7\x89\xb9\xe9\x9b\x86 1_6\xe3\x80\x90\xe4\xb8\xad\xe5\x9b\xbd\xe4\xba\xba\xe3\x81\x8c\xe6\x97\xa5\xe6\x9c\xac\xe3\x81\xae\xe6\x9b\xb2\xe3\x81\xa7\xe8\xb8\x8a\xe3\x81\xa3\xe3\x81\xa6\xe3\x81\xbf\xe3\x81\x9f\xe3\x80\x91\xe3\x83\x81\xe3\x82\xab\xe3\x81\xa3\xe3\x81\xa8\xe3\x83\x81\xe3\x82\xab\xe5\x8d\x83\xe8\x8a\xb1\xe3\x81\xa3\xe2\x99\xa1\xe3\x80\x90\xe5\x92\xac\xe4\xba\xba\xe7\x8c\xab\xe3\x80\x91\xe3\x80\x90\xe3\x81\x8b\xe3\x81\x90\xe3\x82\x84\xe6\xa7\x98\xe3\x81\xaf\xe5\x91\x8a\xe3\x82\x89\xe3\x81\x9b\xe3\x81\x9f\xe3\x81\x84\xe3\x80\x91.f248.webm.part'

...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
I can't download this video
https://www.youtube.com/watch?v=qHI16SXjiZY  is says the file name is too long.

Limit title length in output template.


Add translation tests and improvements

(This is a continuation of https://github.com/hidden-spectrum/swift-translate/pull/31)

Add some translation tests. Not very many, basically just for what I saw failures with when I trying it out on my string catalogs.
Add saving the string catalog periodically during translation. To allow cancelling any time without loosing progress.
Add a --set-needs-review flag to allow marking strings for review after being translated. I wanted this to allow to run the review command on the newly revised translations. (I realize now that the naming is not consistent with the mark-needs-review command I added previously, maybe one of them should change.)
Fix choosing Y when pressing enter. Which is what is expected when prompting "Y/n".
Fix a bug that caused the translated value to show up in the localizableString.sourceKey variable (#585feb8) and a related naming confusion by renaming sourceKey -> sourceValue. And tests for these in StringCatalogTests.

Feel free to have opinions on any changes/additions, I'm just hacking at things I like to see for myself!
Cheers,
Jonas

See disussion in #31


Add comprehensive Modrinth Maven guide

Rendered view
Please squash, I dunno what happened to the merging hell :P

Looks good to me :)


Not recognizing building

Hi,
How can I troubleshoot that?
When I start it for the first time, I see the progress % going up, in addition to that I can see it clicks on a building (without clicking the info button) and it doesn't write down that a building was found.
When the progress ends, it just click on the build option and gems buying.
In the CMD I see at the end issue with barracks, going to the building file I can see it is empty.
Thanks!

Hi,
How can I troubleshoot that?
When I start it for the first time, I see the progress % going up, in addition to that I can see it clicks on a building (without clicking the info button) and it doesn't write down that a building was found.
When the progress ends, it just click on the build option and gems buying.
In the CMD I see at the end issue with barracks, going to the building file I can see it is empty.
Thanks!

I notice that, especially when city hall level is lower then 16.
below is temporary solution:

create a json file under bot/save/ip_port_building_pos.json (ex: <IP_ADDRESS>_5555_building_pos.json)
copy & paste following

{
    "alliance_center": [
        628,
        251
    ],
    "tavern": [
        691,
        251
    ],
    "goldmine": [
        376,
        407
    ],
    "quarry": [
        439,
        459
    ],
    "barracks": [
        754,
        355
    ],
    "stable": [
        817,
        303
    ],
    "blacksmith": [
        943,
        303
    ],
    "archery_range": [
        880,
        355
    ],
    "lumber_mill": [
        439,
        511
    ],
    "farm": [
        502,
        563
    ],
    "siege_workshop": [
        754,
        459
    ],
    "hospital": [
        628,
        563
    ]
}

replace x and y position to your building position
open ip_port_config.json (ex: <IP_ADDRESS>_5555_config.json), find and set hasBuildingPos to true

Another solution(may also fix it):
https://github.com/Dylan-Zheng/Rise-of-Kingdoms-Bot/issues/9#issuecomment-791843808

Thanks!  I mean I also saw the other comment but can't understand associate the red square to a numbers location.

You can use paint.net / photoshop to get your building position:

open paint.net and set view as follow:

move you mouse to the building, then zoom in:

x in top ruler, y in left ruler

ex:
{
    "goldmine": [
        513,
        295
    ],
}

New version allow to manually configure building position
Link: https://github.com/Dylan-Zheng/Rise-of-Kingdoms-Bot/releases/tag/v<IP_ADDRESS>121_beta

You are amazing!!


Typo: "first" should be "last" in syntax.md page

On the https://github.com/nishihatapalmer/byteseek/blob/master/syntax.md page where it is written:
 &7f                 # 0x7f = 01111111. Match bytes with all first seven bits set.

I think "first" should be "last", i.e.:
 &7f                 # 0x7f = 01111111. Match bytes with all last seven bits set.

Oops. Duh. The original wording is correct. I'm reading my binary backwards. Sorry for the noise. Closing.

Hey, no problem.  Thanks for trying to improve byteseek!

@nishihatapalmer Thanks. Without opening another new issue, is the line ^~87                # 0x87 = 10000111. Match bytes without any of the bitmask bits set. on the same syntax.md page correct? It seems to me that ^~87 should match all bytes with the first three and the last bits unset, i.e.,
00000000
00001000
00010000
00011000
00100000
00101000
00110000
00111000
01000000
01001000
01010000
01011000
01100000
01101000
01110000
01111000

Am I missing something here again?

That's correct.  An any match ~87 on its own would match all the bytes with any of those bits set, but the expression is negated with ^, so it will match all bytes without any of those bits set, which is the list you give.

A simple way to understand the negation operator ^ is that it gives you a set of bytes containing all the bytes which weren't in the original set.

Oh, yes, I understand the semantics of ~, ^, and &; I was confused about the wording, but now I see that "without any of the bitmask bits set" means "without any of the bits in 10000111 set", i.e., all integers between 0 and 255 whose binary representation (as a string) starts with 0 and ends with 000.

I can see how it could be confusing... I'll either have to come up with some better examples, or a better explanation...


Change cluster filtering by permits instead of roles

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/811

@sjd78 Done. Please, review.

@sjd78 @gregsheremeta Added external permissions fetch as temporary solution, because of this bug https://bugzilla.redhat.com/show_bug.cgi?id=1639784

ci build please

ci build please

ovirt-web-ui-1.5.0-0.20181026git9b9bd99.el7.noarch.rpm
ovirt-web-ui-1.5.0-0.20181026git9b9bd99.fc28.noarch.rpm

ci build please

ci build please

@leistnerova rebased.

LGTM


Treat 'some' types as having non-simple representations by default

That is, if they appear inside a larger type, they may need to be parenthesized. ASTPrinter is careful about this already, so this doesn't actually change any existing behavior, but it makes the default more conservative when no PrintOptions are in play.

@swift-ci Please smoke test


Missing microbe 🦠 emoji

I am using Noto on my raspberry pi. It won’t render the microbe emoji. Just get the missing glyph box.

Maybe you're using an old version?

You are absolutely right. Just rebuilt from source and it's there. Must have picked up a stale TTF file somewhere. Sorry.


urql support

Current dependencies on/for this PR:

main

PR #6   👈

This comment was auto-generated by Graphite.


Asset modules enhancement

What kind of change does this PR introduce?
Feature
It works but there is some stuff to do:

[x] fix tests
[x] add more tests
[x] add generator option validation by a scheme

Did you add tests for your changes?
Yes
Does this PR introduce a breaking change?
No
What needs to be documented once your changes are merged?
About generator option

Thanks


core-js polyfill vs babel-polyfill

Version
4.3.1
Environment info
System:
    OS: macOS Mojave 10.14.6
    CPU: (8) x64 Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz
  Binaries:
    Node: 12.14.1 - ~/.nvm/versions/node/v12.14.1/bin/node
    Yarn: Not Found
    npm: 6.13.4 - ~/.nvm/versions/node/v12.14.1/bin/npm
  Browsers:
    Chrome: 80.0.3987.163
    Firefox: Not Found
    Safari: 13.1
  npmPackages:
    @twcapps/vue-cli-plugin-build: 0.1.9 => 0.1.9
    @vue/babel-helper-vue-jsx-merge-props:  1.0.0
    @vue/babel-plugin-transform-vue-jsx:  1.1.2
    @vue/babel-preset-app:  4.2.2
    @vue/babel-preset-jsx:  1.1.2
    @vue/babel-sugar-functional-vue:  1.1.2
    @vue/babel-sugar-inject-h:  1.1.2
    @vue/babel-sugar-v-model:  1.1.2
    @vue/babel-sugar-v-on:  1.1.2
    @vue/cli-overlay:  4.2.2
    @vue/cli-plugin-babel: ~4.2.2 => 4.2.2
    @vue/cli-plugin-e2e-nightwatch: ~4.2.2 => 4.2.2
    @vue/cli-plugin-eslint: ~4.2.2 => 4.2.2
    @vue/cli-plugin-router:  4.2.2
    @vue/cli-plugin-typescript: ~4.2.2 => 4.2.2
    @vue/cli-plugin-unit-jest: ^4.2.3 => 4.2.3
    @vue/cli-plugin-vuex:  4.2.2
    @vue/cli-service: ~4.2.2 => 4.2.2
    @vue/cli-shared-utils:  4.2.2 (4.2.3)
    @vue/component-compiler-utils:  3.1.1
    @vue/eslint-config-airbnb: ~5.0.2 => 5.0.2
    @vue/eslint-config-typescript: ~5.0.1 => 5.0.1
    @vue/preload-webpack-plugin:  1.1.1
    @vue/test-utils: ^1.0.0-beta.32 => 1.0.0-beta.32
    @vue/web-component-wrapper:  1.2.0
    eslint-plugin-vue: ~6.2.1 => 6.2.1
    jest-serializer-vue:  2.0.2
    typescript: 3.7.5 => 3.7.5
    vue: ~2.6.11 => 2.6.11
    vue-class-component: ~7.2.3 => 7.2.3
    vue-cli-plugin-markdown: ~0.1.2 => 0.1.2
    vue-eslint-parser:  7.0.0
    vue-hot-reload-api:  2.3.4
    vue-i18n: ~8.15.3 => 8.15.3
    vue-jest:  3.0.5
    vue-loader:  15.9.0
    vue-property-decorator: ~8.4.0 => 8.4.0
    vue-router: ~3.1.5 => 3.1.5
    vue-spatialnavigation: ^1.2.1 => 1.2.1
    vue-style-loader:  4.1.2
    vue-template-compiler: ~2.6.11 => 2.6.11
    vue-template-es2015-compiler:  1.9.1
    vuex: ^3.1.3 => 3.1.3
  npmGlobalPackages:
    @vue/cli: 4.2.3

Steps to reproduce
core-js polyfill is not working the same way as babel-polyfill.
With core-js (or it's indirect use like @vue/cli-plugin-babel/preset) the app behaves weird, not updating the UI from events on Safari 6 (Samsung Tizen 2016).
What is expected?
App works after the preset applies all polyfills
What is actually happening?
Weird behavior, UI not updating even though the event method is called. (Could be related to macrotaks? Which polyfills would wrongly cause this?)

On Tizen 2016, the regular Vue CLI 4 project is not working properly after applying the polyfills. If I use babel-polyfill with useBuiltIns: "entry" it works just fine. Is there any fundamental difference between those?

https://babeljs.io/docs/en/babel-polyfill

🚨 As of Babel 7.4.0, this package has been deprecated in favor of directly including core-js/stable (to polyfill ECMAScript features) and regenerator-runtime/runtime (needed to use transpiled generator functions):

As for your specific issue, please provide a reproduction.

Hi @sodatea, I understand that the cli is using core-js directly because babel-polyfill has been deprecated. I just wanted to understand what could cause the issue.
As for reproduction, it would be very difficult for me to provide it, as it is very specific to a Samsung tizen TV. The use case follows:
The TV SDK will send me an event with the current video progress (a number). I'm using xstate to manage a state machine that will handle the transitions and this event is passed up to the app via a callback. So the app is doing:
this.player.subscribe(this.handleStateUpdate);

handleStateUpdate(state) {
      this.progressTime = state.progressTime;
}

It's a very simple event that will call the function and update the ui reactively, nothing fancy there. If I put a console.log to see the state variable and check on a remote devtools the UI updates, as if the event loop was delayed sufficiently for Vue to detect the changes. If I use a setTimeout wrapping the assignment it also works with core-js, maybe it's the queue-microtasks polyfill that's included from babel but not from core-js? I'm quite lost at this point.
I'll try to download an old version of Safari and try to repro locally (as the TV is using Safari 6 equivalent of webkit).
Thanks

Honestly, I'm not sure.

I use babel-polyfill with useBuiltIns: "entry" it works just fine

Does it work if you replace babel-polyfill with core-js/stable. Because in the default setup, the core-js polyfills are required on-demand. So if core-js/stable also works, we can narrow down the issue to the polyfills that are not loaded by default.

It doesn't work, unfortunately. I'll try experimenting a bit with the different polyfills that are being applied by babel. It's super strange.

So I tested corejs@3 by including only
import "core-js/fn/set";
import "core-js/fn/map";
import "core-js/fn/promise";
import "core-js/fn/object";
import "core-js/fn/array";

which is the bare minimal to make my app run on that device. The UI update works just fine. I'll try to find the offender now by including each package from core-js.

Tested<EMAIL_ADDRESS>by including only
import "core-js/stable/symbol";
import "core-js/stable/object";
import "core-js/stable/array";
import "core-js/stable/promise";
import "core-js/stable/map";
import "core-js/stable/set";

which is the bare minimal to make my app run on that device. The UI update stopped working.

Found the breaking package from core-js@3: core-js/stable/promise
Using core-js@2 core-js/fn/promise works fine, even using everything else from core-js@3 as in:
import "core-js3/stable/symbol";
import "core-js3/stable/object";
import "core-js3/stable/array";
import "core-js2/fn/promise";
import "core-js3/stable/map";
import "core-js3/stable/set";

Super weird edge case. Now I think it's out of scope of vue-cli, right @sodatea ?

Yeah. It's either in the core-js polyfill or the Vue nextTick implementation.
You can check if the new polyfill breaks any logic of nextTick https://github.com/vuejs/vue/blob/dev/src/core/util/next-tick.js


Expose dom as an attribute on Test::Mojo

Summary
Expose dom as an attribute on Test::Mojo to allow testing Mojo::DOM objects without involving a user agent. Also makes some tests in t/mojo/mojo-dom.t a bit simpler to read.
Motivation
This makes it easy to use Test::Mojo to test the contents of HTML/XML strings.

I've wanted this on the off occasion where there are mixed api return types, like a JSON api for which one of the values is HTML.

I'm less sure about changing how the t/mojo/dom.t tests test Mojo::DOM (now via Test::Mojo) but the rest I'm mildly 👍  on

👎 Really don't like the changes in dom.t. If you want better test coverage for Test::Mojo, then i believe the right way to do it would be a new test file with tests for pass and fail. Not sure how to do it best, but i'm sure other test modules have tests too.

Ok, I'm definitively not attached to the t/mojo/dom.t changes, will add a
few tests to t/test-mojo.t or so instead. Closing this pull request and
will reopen when I'm done.
M
On Mon, Jan 30, 2017 at 10:04 PM Sebastian Riedel<EMAIL_ADDRESS>wrote:

👎 Really don't like the changes in dom.t. If you want better test
coverage for Test::Mojo, then i believe the right way to do it would be a
new test file with tests for pass and fail. Not sure how to do it best, but
i'm sure other test modules have tests too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kraih/mojo/pull/1045#issuecomment-276189788, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAVluzXdN-N6j1ziVGo0vTe8IOFGow0ks5rXlBOgaJpZM4Lx5Mw
.

FYI, once Perl ships a major (minor, sigh) version with Test2, I intend to rewrite the Test::Mojo internals and test them better (with approval of course).


Create smoke test for edit collection

Approach

Add test for clicking edit details on private collection and/or revision.
Attempt cancel.
Attempt save.

This is already at least partially covered by the revision e2e tests. The reason they didn't catch this error is that they had to be disabled (skipped) for a few weeks while we reconfigured the testing infrastructure.


remove patches

should be no longer necessary with conda-build update

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Best to hold off on this until conda-build 1.21.10 or above is available.  1.21.11 may be necessary to appropriately create entry points for conda-build itself.  Issue at https://github.com/conda-forge/conda-build-feedstock/pull/2

Roger. I saw a PR somewhere else that I thought meant this was done, but I must have misunderstood.

Closing this, as upcoming 15.4 release will remove the need for the patches anyway


Impressive

This is very impressive widget.
The format that the data is saved in is similar to quill perhaps. Is it json by any chance.
Looking to help extend this and am thinking that the quill format is a nice way to do it.
You know quill ?

Not yet, but I can learn. Can you give me some refs?
Pe 23 dec. 2017 15:52, "gedw99"<EMAIL_ADDRESS>a scris:

This is very impressive widget.
The format that the data is saved in is similar to quill perhaps. Is it
json by any chance.
Looking to help extend this and am thinking that the quill format is a
nice way to do it.
You know quill ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/AI2nVnTVCzz-pjY-EohxuYao_zhHorQVks5tDQWjgaJpZM4RLrE5
.

https://pub.dartlang.org/packages/angular_quill
So already written in dart. That's why I suggested it.
It's a very popular rich text editor too.
https://quilljs.com
There is an add-on that uses y.js to give it the ability to sync changes
with other users too so you get Google docs like functionality.
https://github.com/y-js/y-richtext
So I am planning to integrate the y.js functionality in as dart. Should not
be too hard.
Let me know what you think.
I would love to extend from what you have so dart and flutter users have a
solid and syncing editor. Very powerful
On Sat, Dec 23, 2017, 14:55 Razvan Lung<EMAIL_ADDRESS>wrote:

Not yet, but I can learn. Can you give me some refs?
Pe 23 dec. 2017 15:52, "gedw99"<EMAIL_ADDRESS>a scris:

This is very impressive widget.
The format that the data is saved in is similar to quill perhaps. Is it
json by any chance.
Looking to help extend this and am thinking that the quill format is a
nice way to do it.
You know quill ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AI2nVnTVCzz-pjY-EohxuYao_zhHorQVks5tDQWjgaJpZM4RLrE5
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4#issuecomment-353727375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATuCwq8TJxL4PMHgDIUz0BOK_Dba7m5Yks5tDQZOgaJpZM4RLrE5
.

Thanks! I'll take a look and come back with an answer.
Pe 23 dec. 2017 16:02, "gedw99"<EMAIL_ADDRESS>a scris:
https://pub.dartlang.org/packages/angular_quill
So already written in dart. That's why I suggested it.
It's a very popular rich text editor too.
https://quilljs.com
There is an add-on that uses y.js to give it the ability to sync changes
with other users too so you get Google docs like functionality.
https://github.com/y-js/y-richtext
So I am planning to integrate the y.js functionality in as dart. Should not
be too hard.
Let me know what you think.
I would love to extend from what you have so dart and flutter users have a
solid and syncing editor. Very powerful
On Sat, Dec 23, 2017, 14:55 Razvan Lung<EMAIL_ADDRESS>wrote:

Not yet, but I can learn. Can you give me some refs?
Pe 23 dec. 2017 15:52, "gedw99"<EMAIL_ADDRESS>a scris:

This is very impressive widget.
The format that the data is saved in is similar to quill perhaps. Is it
json by any chance.
Looking to help extend this and am thinking that the quill format is a
nice way to do it.
You know quill ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4, or mute the thread
<
https://github.com/notifications/unsubscribe-
auth/AI2nVnTVCzz-pjY-EohxuYao_zhHorQVks5tDQWjgaJpZM4RLrE5
.

—
You are receiving this because you authored the thread.

Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4#issuecomment-353727375,
or mute the thread
<https://github.com/notifications/unsubscribe-
auth/ATuCwq8TJxL4PMHgDIUz0BOK_Dba7m5Yks5tDQZOgaJpZM4RLrE5>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4#issuecomment-353727660,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI2nVn4aEwilx46d09vTReMgRLebZ-DUks5tDQfrgaJpZM4RLrE5
.

ok, so who are going to port this library to Flutter? what's your plan?

Right now i have to chicken out. Just no bandwidth.
I used y.js witha  web / javascript client on an old project and it worked really well with quill.
This is why i brouht it up.
here is another one btw:
https://github.com/ipfs-shipyard/peerpad/blob/master/docs/ARCHITECTURE.md

Hey again,
I have been doing some research on this and have a MUCH better approach now.
There is a library called automerge.
It is able to take json and keep it in sync between multiple instances on different devices. It's based on CRDT and works very well.
The maintainers are talking about making a port to Dart, but it's not started yet. It should be a reasonable easy port.
So the only thing then that matters is how you serialise the HTML widget to JSON. I suspect you won't need to do anything special but I am not sure yet until I do more testing.

In your code I don't see any code that serialises the contents of the widget....
You never got to the functionality of saving and loading content to and from json ?

appreciate your comments.
Yours is the only one i have found, and if you can illustrate how you would rewrite it then maybe me or someone will do it.
Is that ok ?

Sorry but I don't have time.
vin., 20 apr. 2018, 18:53 gedw99<EMAIL_ADDRESS>a scris:

appreciate your comments.
Yours is the only one i have found, and if you can illustrate how you
would rewrite it then maybe me or someone will do it.
Is that ok ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4#issuecomment-383140323,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI2nVrILNGniiYg64dhMeEbTTuWRpakeks5tqgRrgaJpZM4RLrE5
.

Do you mind if I work and change it substantially ?
On Sat, Apr 21, 2018, 7:09 PM Razvan Lung<EMAIL_ADDRESS>wrote:

Sorry but I don't have time.
vin., 20 apr. 2018, 18:53 gedw99<EMAIL_ADDRESS>a scris:

appreciate your comments.
Yours is the only one i have found, and if you can illustrate how you
would rewrite it then maybe me or someone will do it.
Is that ok ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<https://github.com/long1eu/rich_editor/issues/4#issuecomment-383140323
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AI2nVrILNGniiYg64dhMeEbTTuWRpakeks5tqgRrgaJpZM4RLrE5
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/long1eu/rich_editor/issues/4#issuecomment-383313359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATuCwrdpelcpoO5PAGKP_8Md4l0SjWhmks5tq2e7gaJpZM4RLrE5
.

not at all

compiler message: file:///Users/shenma/Documents/Flutter/projects/rich_editor/rich_editor/lib/src/material/dropdown.dart:267:7: Error: The non-abstract class '_DropdownRoute' is missing implementations for these members:
compiler message:   'barrierLabel'.
compiler message: Try to either
compiler message:  - provide an implementation,
compiler message:  - inherit an implementation from a superclass or mixin,
compiler message:  - mark the class as abstract, or
compiler message:  - provide a 'noSuchMethod' implementation.
compiler message:
compiler message: class _DropdownRoute extends PopupRoute<_DropdownRouteResult> {
compiler message:       ^^^^^^^^^^^^^^
compiler message: file:///Users/shenma/Documents/Flutter/flutter/packages/flutter/lib/src/widgets/routes.dart:831:14: Context: 'barrierLabel' is defined here.
compiler message:   String get barrierLabel;
compiler message:              ^^^^^^^^^^^^
compiler message: file:///Users/shenma/.pub-cache/hosted/pub.dartlang.org/material_color_picker-0.0.1/lib/src/dropdown.dart:33:7: Error: The non-abstract class '_WidgetDropdownRoute' is missing implementations for these members:
compiler message:   'barrierLabel'.
compiler message: Try to either
compiler message:  - provide an implementation,
compiler message:  - inherit an implementation from a superclass or mixin,
compiler message:  - mark the class as abstract, or
compiler message:  - provide a 'noSuchMethod' implementation.
compiler message:
compiler message: class _WidgetDropdownRoute extends PopupRoute {
compiler message:       ^^^^^^^^^^^^^^^^^^^^
compiler message: file:///Users/shenma/Documents/Flutter/flutter/packages/flutter/lib/src/widgets/routes.dart:831:14: Context: 'barrierLabel' is defined here.
compiler message:   String get barrierLabel;
compiler message:              ^^^^^^^^^^^^
compiler message: file:///Users/shenma/Documents/Flutter/projects/rich_editor/rich_editor/lib/src/widgets/rich_editable_text.dart:873:18: Error: Too many positional arguments: 0 allowed, 1 given.
compiler message:     renderObject.setCaretPrototype(currentStyle.fontSize);
compiler message:                  ^
Compiler failed on /Users/shenma/Documents/Flutter/projects/rich_editor/rich_editor_demo/lib/main.dart
FAILURE: Build failed with an exception.


fix: update dockerfile to install chromium packages

 Target ghcr.io/dedsxc/unixbot:4-dev-1.4.1 (alpine 3.17.5)
Vulnerabilities (2)

Package
ID
Severity
Installed Version
Fixed Version

libcrypto3
CVE-2023-5363
MEDIUM
3.0.11-r0
3.0.12-r0

libssl3
CVE-2023-5363
MEDIUM
3.0.11-r0
3.0.12-r0

No Misconfigurations found
Target Python
Vulnerabilities (1)

Package
ID
Severity
Installed Version
Fixed Version

pip
CVE-2023-5752
MEDIUM
23.2.1
23.3

No Misconfigurations found


Scrolling Delete Card Positioning

Hello how are you?
When you scroll all cards to left and delete de active card the scroll position dont update example in photo

can you help me or tell me what i can do for that?

Hello, @edgarscr. Look closer to library example implementation. You may try to change it to acquire behavior you need. When it will behave the way you need apply it to your project.

Hello, my problem its this look normal recycler view cards

scrolling work perfect
but if i scroll all to right

Looks like that, but if i remove one card (last) they stay like this

but i search in ur documentation and dont found nothing to help .

This feature is specific to your project so you'll need to figure out how to implement it. If you'll need then you can fork this lib and customize it the way you need.


First letter of title cut of in "td l"

The first letter of the task/document title is cut of if using "td l" under Windows.
Should'nt we have a unit test for this?
Please fix and add unit test(s)

Issue is resolved but we still need a test for this

Parser has been improved, is now line based and uses String.split(System.lineSeparator())


feat: test action

Hello from Github Action.
Risks: []
Summary: SummaryResponse(coverage=0.05734406438631791, risks=0)
Coverage: CoverageResponse(coverage=0.05734406438631791)

Drill4J CI/CD report:

Coverage: "0.05734406438631791"%
Risks: "0"

Drill4J CI/CD report:

Coverage: "0.05734406438631791"%
Risks: "0"

Drill4J CI/CD report:

Coverage: "0.0"%
Risks: "0"

Drill4J CI/CD report:

Coverage: "0.0"%
Risks: "0"

Drill4J CI/CD report:

Coverage: "0.0"%
Risks: "0"

Drill4J CI/CD report:

Coverage: "0.05734406438631791"%
Risks: "0"

Drill4J CI/CD Report

Coverage: "0.0"%
Risks: "0"

Drill4J CI/CD Report

Coverage: "0.05734406438631791"%
Risks: "0"


Import speaker's introductory note into metadata database.

Researcher wants the note introducing the speaker to be displayed prior to reconstructed text in notebook.

Related welfare_state_analytics#21


Issue 5 woozie mode

Makes woozie a major mode.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Alex Santoro seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Error with Facebook video

I get the following error when trying to download a video from Facebook:
ERROR: Cannot parse data
The command `youtube-dl -U' gets the following:
youtube-dl is up-to-date (2021.12.17)
Here's the whole output with the '--verbose' option:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.facebook.com/[...]']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.8 (CPython) - Linux-6.0.0-5-amd64-x86_64-with-glibc2.36
[debug] exe versions: ffmpeg 5.1.2-1, ffprobe N-98534-g5eb4405fc5, rtmpdump 2.4
[debug] Proxy map: {}
[facebook]<PHONE_NUMBER>441603: Downloading webpage
[facebook]<PHONE_NUMBER>441603: Downloading webpage
ERROR: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/facebook.py", line 680, in _real_extract
return self._extract_from_url(real_url, video_id)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/facebook.py", line 594, in _extract_from_url
raise ExtractorError('Cannot parse data')
youtube_dl.utils.ExtractorError: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Thanks.

Follow #30839, please.
This issue is a duplicate of #30872. There is a new FB extractor in PR #30700. If you can, you might try that and report there. Otherwise, the testing for the PR will take account of this report.


BizTalkSample cacls command fails with warning in server deployment

BizTalkSample cacls command fails with warning in server deployment
This work item was migrated from CodePlex
CodePlex work item ID: '3743'
Assigned to: 'tfabraham'
Vote count: '0'

[UnknownUser@2/3/2009]
Resolved with changeset 18268.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]