Application show up in Mesos but not in Marathon apps list

Environment:
Mapr5, mesos 0.28, marathon 1.1.0, docker 1.9.0, ubuntu 14.04
Description:
App has been killed via Marathon
App is still running
It still showing up in Mesos
CTRL-f on a list of Marathon apps list cannot find this app
Direct link to this app in Marathon still exists ans shows that app is running
If in Marathon I press the button destroy again I get this message

Error destroying /app-prod-21: App '/app-prod-21' does not exist

Happens randomly

Hey
Further investigation shows a bug in Marathon apps list.
More details:
Direct link to app remains, but app not in list
Number of tasks is 1
App is still in Mesos
If I manually stop docker container Marathon restarts it
Container still in Zookeeper
When I try to destroy it I got this alert
Error destroying /app0: App '/app0' does not exist
When I try to suspend it I got this
Error scaling /app0: Object is not valid
When I try to scale it I got this
Error scaling /app0: Object is not valid
When I try to restart I got this
Error restarting /app0: App '/app0' does not exist
Looking in source code I see that we get this UnknownAppException when it iterates over apps list but cannot find given app id
https://github.com/mesosphere/marathon/blob/a28e6a68497c4d409cb7eef56275d1af17b67231/src/main/scala/mesosphere/marathon/api/TaskKiller.scala#L50
Nothing suspicious in logs
Please let me know if you need any additional details according this bug

Seeing the same issue. Not sure whether it is related to us deleting the apps under a higher concurrency

I have same issue. After upgrade from Marathon 0.14 to 0.15. I had problem with HealthCheck validation.
 mesosphere.marathon.ValidationFailedException: Validation failed
         at mesosphere.marathon.api.v2.Validation$.validateOrThrow(Validation.scala:18) ~[marathon:0.15.6]
         at mesosphere.marathon.state.GroupManager$$anonfun$upgrade$1$$anonfun$2$$anonfun$apply$11.apply(GroupManager.scala:160) ~[marathon:0.15.6]
         at mesosphere.marathon.state.GroupManager$$anonfun$upgrade$1$$anonfun$2$$anonfun$apply$11.apply(GroupManager.scala:158) ~[marathon:0.15.6]
        at scala.util.Success$$anonfun$map$1.apply(Try.scala:237) ~[marathon:0.15.6]
         at scala.util.Try$.apply(Try.scala:192) ~[marathon:0.15.6]
         at scala.util.Success.map(Try.scala:237) ~[marathon:0.15.6]
         at scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:235) ~[marathon:0.15.6]
         at scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:235) ~[marathon:0.15.6]
         at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:32) [marathon:0.15.6]
         at scala.concurrent.impl.ExecutionContextImpl$AdaptedForkJoinTask.exec(ExecutionContextImpl.scala:121) [marathon:0.15.6]
         at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260) [marathon:0.15.6]
         at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.pollAndExecAll(ForkJoinPool.java:1253) [marathon:0.15.6]
         at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1346) [marathon:0.15.6]
         at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979) [marathon:0.15.6]
         at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107) [marathon:0.15.6]

Do you have the issue with Marathon 1.3?

Hi Karsten.
We just upgraded to 1.3 recently. Haven't seen this issue yet.
I will keep you posted if we see this in 1.3 again.
Btw, is there any change 1.3 that you think will address this problem?
Thanks!
Yang
2017-01-23 6:25 GMT-08:00 Karsten Jeschkies<EMAIL_ADDRESS>
Do you have the issue with Marathon 1.3?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/3892#issuecomment-274500687,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJzCwvU8CZxMEcQy_RJQQb15vc-7PQWUks5rVLhGgaJpZM4IcylS
.

Hi Karsten.
Actually we are seeing it now in our test environment.
Marathon shows no apps but Mesos master still shows 200+ tasks running.
I did a "sudo rm -f /var/run/mesos/meta/slaves/latest" on all the agents
before this happens. Is it related?
Thanks!
Yang
2017-01-23 8:19 GMT-08:00 Sunshine Zeng<EMAIL_ADDRESS>
Hi Karsten.
We just upgraded to 1.3 recently. Haven't seen this issue yet.
I will keep you posted if we see this in 1.3 again.
Btw, is there any change 1.3 that you think will address this problem?
Thanks!
Yang
2017-01-23 6:25 GMT-08:00 Karsten Jeschkies<EMAIL_ADDRESS>
Do you have the issue with Marathon 1.3?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/3892#issuecomment-274500687,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJzCwvU8CZxMEcQy_RJQQb15vc-7PQWUks5rVLhGgaJpZM4IcylS
.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4499. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Show dialogs using Accessibility Service

Since showing dialogs from a service is disabled by some manufacturers, I'm creating a view similar to a dialog as an Accessibility Overlay.

Instead of notification, we now get dialogs to show Script exit messages. The script still vibrates on exit.
Error messages have a copy button which will copy the error to clipboard.
Auto Skill picker is shown on script start if you have Auto Skill enabled. The last one you used is selected by default. The selected Auto Skill settings has been removed from the UI.

If Support Image Maker is shown this way, we won't have to worry about it not showing on some devices.
The code can also be much simplified then, we can generate the images into the app's private cache directory like Android recommends and move only the images (renamed) which the user wants to the Fate-Grand-Automata/support folder.
Option for generating Friend images can also be added without making it too bothersome for the user. Right now, they have to delete what they don't want. We can change it to keep what you want, ignore the rest.


Adding AI Voice and Chat Assistant for FAQ, Page Navigation, and User Experience Improvement

Description
Issue Description:
The project currently lacks an AI-powered voice and chat assistant to enhance user experience by providing assistance with frequently asked questions (FAQs) and page navigation. This issue is created to address this gap and propose the addition of an AI voice and chat assistant feature to improve user experience, guide users through the website, and provide helpful responses to their inquiries.
Proposed Solution:
To address this issue, the following steps can be taken:

Research and select a suitable AI voice and chat assistant platform or framework that aligns with the project's requirements.
Integrate the chosen platform or framework into the project's codebase.
Implement the necessary conversational flow and responses to cover frequently asked questions and guide users through page navigation.
Test the assistant thoroughly to ensure accurate understanding and appropriate responses.
Continuously improve and expand the assistant's capabilities based on user feedback and evolving requirements.

Screenshots

Additional information
Please provide any additional information or specifications that may be helpful in implementing this feature. Thank you for your attention to this matter.

@apoorvaron ,Any specification for ai voice and chatbot  you want then specify ,I have aded FAQ and page navigation .

just add all chatbot current queries @Nishitbaria


@conda-forge-admin please rerender

Additional comment:
No response

@conda-forge-admin please rerender


Value-based MPT GC

Problem
#2095.
Solution
Previous attempts like #2137 and #2159 had about 40% overhead and even that is not complete, because MPT traversal took minutes to finish and usually time came for another GC period before the old process finished. Key-based scheme outlined in https://github.com/nspcc-dev/neo-go/issues/2095#issuecomment-1024117106 was fine on the paper, but a complete mess too in practice. With the default MemCachedStore it was like 5 times slower than simple KeepOnlyLatestState, BTree-based store worked much better, but still 2-3 times slower than KOL.
It should be noted that the higher is the chain, the more effects of good/bad MPT management are visible, so 900K+ mainnet chain is very good to check things against, previous 100-200K chains were just too small.
So what we have here is that we do the same height/refcount value trick as in https://github.com/nspcc-dev/neo-go/issues/2095#issuecomment-1024117106, but we don't touch the keys. This makes it harder to clean things up (regular persisting can't happen before the GC is finished), but it works well with ~18% overhead  (which is OK, storing a set of tries is harder than storing one trie, more read/write activity) and typical 1-2 seconds for the GC cycle.

And just to clarify: currently, MPT node's refcount (last four bytes of the node that we got from the storage) still shows the number of references to this node in the latest MPT trie and doesn't include  references from the previous MPT heights, right?

And just to clarify: currently, MPT node's refcount (last four bytes of the node that we got from the storage for active nodes) still shows the number of references to this node in the latest MPT trie state and doesn't include references from the previous MPT heights, right?

Yep. We can't really count references from old tries, technically for any unchanged node a new trie adds at least one new reference, so full-blown historic reference counting requires updating all nodes.

Does the performance slowdown differs between bolt and leveldb?

~15% for Bolt, about the same.

On the alternatives. The idea I also have is to still use the key to differentiate between active/inactive nodes. Like this (the key parts of the change):
        DataMPT        KeyPrefix = 0x03
+       DataMPTOld     KeyPrefix = 0x04
...
        case cnt == 0:
-               if !t.mode.GC() {
-                       _ = t.Store.Delete(key)
-               } else {
-                       data[len(data)-5] = 0
+               _ = t.Store.Delete(key)
+               if t.mode.GC() {
+                       key[0] = byte(storage.DataMPTOld)
                        binary.LittleEndian.PutUint32(data[len(data)-4:], index)
                        _ = t.Store.Put(key, data)
+                       key[0] = byte(storage.DataMPT)
                }

Theoretically, pros:

shorter Seek time as we're not even looking at active nodes (for the current 900K dump that's ~2.5M KV pairs evaluated at GC time vs ~850K)
storage scheme compatible with the old DB
And cons:
delete/add a key instead of value overwrite during MPT flush
MPT traversal for archived nodes requires a Get fallback, first looking at DataMPT prefix and then at DataMPTOld, so it'll certainly be slower

And that's where we actually have different practical results for LevelDB and BoltDB. Level seems to like this change, the overhead drops from ~18% to ~12%, but Bolt jumps from ~15% to ~27%. Obviously Bolt is better at seeking at Level is better at writing new data.
Theoretically, as the chain grows we'll have bigger and bigger active set. Also my tests are configured to store just the latest 10K states, with 2M traceable blocks we'll also have much more inactive, but still kept nodes. Yet Bolt results for current chain look pale (I'll try rerunning them though).


allow different custom posts to be configured

Today we can configure the custom posts queries but it will be applied to ALL custom posts not just one.
"customPosts":
    "query":
        "per_page": 5
        "filter[orderby]": "date"
        "filter[order]": "desc"

What about post that query from taxonomies, i notice this is only work if it is custom posts, but if the listing is from taxonomies it doesn't work.

try with taxonomiesPosts instead of customPosts

Thank you~ taxonomiesPosts work!

Is there any way to pass a custom query based on the $stateParams.slug?

Now custom posts or taxo will get the conf of the type it displays.
For instance if your custom post is movie or the custom taxo post type is movie, both will read this config:
"movie":
    "query":
        "per_page": 2


The first parameter to eio_custom needs to be void (*)(eio_req *) and n

The first parameter to eio_custom needs to be void (*)(eio_req *) and not int (*)(eio_req *)
build fails on Amazon EC2 if this is incorrect.

It's up to @JustinTulloss. Personally, I would see a new release, this seems to be related to node v0.5.x, and will will probably still be at v0.6.x (next stable)

I'm another vote for merging this fix... zmq is DOA against node >= 0.5.4 w/o it.

Oh crap, I opened #58 without even thinking to see if there was already a pull request for that :/
When is this going to get merged? Anyone have news from @JustinTulloss ?

Just pushed 1.0.3 which works for me. Let me know if something's not working for you.

working for me.  thanks!
-r
On 11/8/11 1:01 AM, Justin Tulloss wrote:

Just pushed 1.0.3 which works for me. Let me know if something's not working for you.

Reply to this email directly or view it on GitHub:
https://github.com/JustinTulloss/zeromq.node/pull/47#issuecomment-2665200

Working for me as well, thanks!


Update README.md

Changed the "Toolkit" to "Synthetic Data Generation Toolkit" for GCM paper.

Sounds good!


Add constructor for stroke.

Minimal implementation of a constructor for the stroke type.
This would be especially useful in package authoring when you want to convert a length/paint/dictionary provided by the user to a stroke in order to easily access its fields.
#let my-function(my-stroke) = {
    let s = stroke(my-stroke)
    (s.paint, s.thickness, s.cap)
}
The result of #my-function(5pt) is `(auto, 5pt, auto)`

As said on Discord, I think the constructor should also offer to pass the fields as named arguments (like the dictionary, but with one level of parens less). This needs a little care, you can look at color.rs for similar functions which either take a single positional argument (for color conversion) or a bunch of other ones.

@laurmaedje Is there a reason that stroke attributes can be auto if they are set to the default value? The problem here is, that one has to check for that and replace auto with the default value (from the docs) to be able to have a value to work with.

Two reasons:

The default differs from place to place. Most of the time it's 1pt + black, but for text underline, it is text color + font-dependant thickness.
Two partial (i.e. with auto-fields) strokes can be merged. This occurs, for instance, in the example below:#set rect(stroke: 1pt)
#set rect(stroke: red)

Both become stroke values with everything but thickness / paint set to auto. And the two stroke values are merged together at a later stage (through "folding").

@laurmaedje I’ve added named arguments to the constructor. It now accepts a single value, or named arguments (or neither, but not both). The named parameters needed new docstrings, but I’m not sure that they’re in the proper style.

Ok, I'll make those documentation changes. @laurmaedje Is there a way to build or serve the docs locally, so I can check things are working?

Unfortunately, no. But I'll check before merging and fix any remaining issues.

I'm torn with the default annotations on the named parameters. It's a nice way to show the common defaults, but in reality the fields will be auto, so it's misleading.

Good point. I’ll try making the defaults auto and adding some more words to explain how the auto values resolve to defaults when needed.
Watch this space…

@laurmaedje The docs are now more technically precise about the meaning of auto and defaults.
I also snuck some other changes in along the way:

Allow gradients to be interpreted as strokes. A color like red can be used, so allowing any Paint value (including gradients) seemed correct.
You can use auto as stroke field values or dictionary values. (Before, this was only possible by omitting that field/key.)

Thanks for the great work!


v1 NPM installation lots of peer dependency issues

Using Latest node LTS on Arch Linux via NVM and NPM-6.1.0
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of webpack@1 || ^2.1.0-beta || ^2.2.0-rc.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^15.0.0 || ^16.0.0-beta || ^16.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^15.0.0 || ^16.0.0-beta || ^16.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of typescript@* but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

Manually adding dependencies with
npm i -D<EMAIL_ADDRESS>webpack@latest <EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>typescript@*

fixes issues. plz updayte package.json?

nevermind I switched to master and everything is working fine so far


Major render modifications

Rewriting the renderer to be more efficient (and hopefully flash less, see issue #8 ).

Going to recreate the branch so its up to date

Done, next build shouldn't fail & I can get back to work

Says that some checks were not successful but I just rebuilt on veyor with no problem. Next commit it'll probably straighten itself out.

Persistence is key...

No conflicts from github, hopefully the veyor build will pass painlessly

This is getting out of hand, all commits are going directly to master to avoid conflicts


Validate additional field when login

In the session controller, I also permit passing a code in the params:
before_filter :configure_permitted_parameters

def configure_permitted_parameters
    devise_parameter_sanitizer.for(:sign_in).push(:code)
end

When somebody tries to login I would like to check for the correctness of the code.
And redirect him back with errors if its not correct. I tried to yield create with:
def create
  super do |resource|
     resource.errors[:base] << "You must have a Sunday in the time range!" if params["user"]["code"] != "test_code"
  end
end

It seems not to work, because the yield takes place after the user is logged in https://github.com/plataformatec/devise/blob/aa3f64976fd886b25e40dfc6043aa7b151145f94/app/controllers/devise/sessions_controller.rb#L20
How can I check if the code is correct, and only if the code is correct, sign in the user? Thanks!

@emmanuelmillionaer
I think you should do something like:

def create
  if params["user"]["code"] != "test_code"
    resource.errors[:base] << "You must have a Sunday in the time range!"
    # maybe a redirect or render a failure page
  else
    super
  end
end

This is jus an example, you can customize to your application needs. Please use the mailing list or StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.


Textbox redraw optimization

Optimize the rendering of the scrolling Textbox on slow displays/processors (such as Arduino).

Goal: minimize flicker when text lines are scrolled up.
Ideal method: use a graphics RAM copy (within ILI9341) to shift old lines up on the display without having to redraw all characters.
Alternate optimization: batching of consecutive character options into string operations. One complexity would be in handling color change requests part-way along a row.

Note that I have implemented the "alternate" option (which will be released as part of the updates to #42 ). This mode will be available with the config option #define GSLC_FEATURE_XTEXTBOX_EMBED 0.


Update Hasura infra / documentation

 [XS] Upgrade Hasura image to v1.2.0 - https://github.com/hasura/graphql-engine/releases/tag/v1.2.0


Micrometer support

Hello,
(not sure if this is the right place or if that's rather a topic for the micrometer maintainers)
We are using the datadog-lambda-java to send custom metrics from a lambda function implemented with spring-cloud-function. Therefore we have already micrometer as a common metrics facade in place.
We tried to use the direct datadog api approach, however that registry does not work properly within a lambda, as it does not flush the metrics at the end of the invocation and it gets confused when a (warm) lambda function is re-activated for the next call.
Would it be possible to provide a micrometer integration that works within AWS Lambda?

Hi @tom-mi ,
This is mildly complicated. Datadog-lambda-java handles metrics by writing them to the Lambda function's Cloudwatch log stream. The metrics are then picked up by the Datadog Forwarder lambda function, converted into metrics, and sent to Datadog's metrics backend. There are a couple things we could try here:

Create a new Micrometer registry that flushes metrics to logs in the format the Datadog forwarder expects
Modify the existing Micrometer registry to flush metrics to the API immediately (this is one of the supported behaviors of Datadog-lambda-js and -py. It doesn't add a lot of overhead to the Lambda invocation, especially if flushing is accomplished in a separate thread)
Wait for the Datadog Lambda Extension to support Java Lambda functions. The Lambda extension has a modified version of dogstatsd built in, and that will support the dogstatsd approach documented in the Micrometer docs.

I'll create a feature request. I think option 3 is already on the roadmap, but I'll have to check with the PM on the Lambda Extension project. In the mean time, I'll investigate how much effort would be required to add an immediate flushing option to the existing Micrometer registry.

Closing this since our Lambda Extension already support sending custom metrics for Java.


Password generator to UI

Before this is done, we could just link to https://passwordsgenerator.net


[HybridApp] Get latest XCode 15 and Android Studio working for OldApp

Problem
Our OldApp does not support the latest XCode and Android Studio
Solution
Get OldApp to run on both XCode 15 and the latest Android Studio
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013d543aaa30637bae
Upwork Job ID:<PHONE_NUMBER>573650944
Last Price Increase: 2023-10-20

One pre-requisite will be this: https://github.com/Expensify/App/issues/30104, but @staszekscp or @mczernek if you want to work on this one, just leave a comment and I can assign you!

XCode 14.2 isn't supported on latest MacOS, so we should do this asap!


Spelling mistake on github/using-git/ignoring-files

https://docs.github.com/en/github/using-git/ignoring-files
Under the section "Further Reading":
the second list item ".gitignore in the man pages for Git"
should be  ".gitignore in the main pages for Git".

See #3898, this is not a typo.

A man page is short form for manual page, which would be the second time someone has attempted to correct this. The possible mistake of adding the "i" leaves it ambiguous to either "main" or the true "manual".
Should this not be expanded to "manual" at the very least?

Thanks for pointing out this duplication @ajs256! It's called a man page because man is the command from unix, so unfortunately we can't change it to "manual" either @edwardASavage 💛


Master broken?

https://travis-ci.org/rust-lang-nursery/rustfmt/jobs/419970670
This fails on a simple linux build, not only on the integration tests.
Specifically:
   Compiling rustc-ap-rustc_errors v235.0.0
error: found removed `do catch` syntax
    --> /home/travis/.cargo/registry/src/github.com-1ecc6299db9ec823/rustc-ap-syntax-235.0.0/parse/parser.rs:1762:43
     |
1762 |             let pat_arg: PResult<'a, _> = do catch {
     |                                           ^^
     |
     = help: Following RFC #2388, the new non-placeholder syntax is `try`
error: aborting due to previous error
error: Could not compile `rustc-ap-syntax`.

Weirdly, I can't recreate locally - any idea for how this could happen?
It works fine on MacOS. I tried building on Linux to ensure that wasn't creating any diff, and that also worked (docker run -it -v (pwd):(pwd) -w (pwd) rust cargo test)

Should be fixed by https://github.com/rust-lang-nursery/rustfmt/pull/2964.

#2965 is a better version of that change, but still failing some tests.


Service Facades / Service Custom Inspectors

Overview
Adds editor-only 'service facade' objects to MRTK singleton. Facades are lightweight scene objects representing active services. They resemble classic MonoBehaviour singletons.
They give developers a better sense of what services are active and offer a quick way to check up on the state of the service.

Testing Goals
On the whole we're looking for feedback on ease of use and inspector clarity. In addition:

Deleting / unparenting / duplicating facade objects should have no effect. Look for situations where these actions cause errors or where facade objects persist in an odd state.
Look for cases where inspector errors are caused by the state of a service.
Look for app or service states where toggling 'Use Service Inspectors' in the mixed reality toolkit config profile causes errors.

Documentation Links
To help new devs get their bearings, facades link to online docs by way of the DocLink attribute. This functions similarly to the HelpUrl attribute in Unity.

Service Custom Inspectors
Facades may implement custom inspector / scene GUI / gizmo functions. This is handled similarly to Unity inspectors. A class implementing the IMixedRealityServiceInspector interface is given the MixedRealityServiceInspector attribute, and the facade uses that class to draw the contents of its inspector.
If no inspector is defined, the facade will draw the contents of the service's configuration profile. If the service uses no profile, the facade simply says no inspector is defined for the service.
I've created the following custom inspectors:

FocusProvider

HandJointService

SpatialAwarenessSystem

Performance
Facades are only used in editor so they won't affect runtime performance. Impact should be minimal but needs to be formally tested. Devs can opt out of using them by un-checking a 'Use Service Inspectors' field in the MRTK configuration profile.
Issues

After starting / stopping a scene, facades vanish until the scene is started again or until recompilation occurs. (This may be an issue with services rather than facades.)

#3545

@davidkline-ms Just to recap - the plan is to merge this into mrtk_development with facades set to OFF by default. This will make it easier for folks to test and provide feedback with minimum disruption.
Once we feel confident that the feature is in good shape we'll set facades to ON in the default config profile.

Just to recap - the plan is to merge this into mrtk_development with facades set to OFF by default.

Sounds perfect! Will review and test ASAP. Will also help with the known issues :)

I also have not tested the facades in a branch, and I now David Kline is working on something similar. Could you please make sure to get his sign off?

@julenka this is the implementation of the POR @davidkline-ms laid out in #3545 - he has seen it and is on board.

Finally, for the hand visualization custom editor, is there a way to check / uncheck all of the joints?

There are 'All / None' buttons you can click. You can also just show fingers / fingertips.

/azp run mrtk_ci

/azp run mrtk_CI

There appears to also be a bug here where choosing another profile under one of the facades doesn't have any effect:

Basically if you:

Go to the InputSystem facade
Click the "targety" button to choose a different input system profile.
Click on any other profile to choose them (or use your keyboard to choose them)
Notice that it doesn't update the profile.

There are some new warnings that I can see in this branch (actually, just one I'm sure about, another one I'm not certain)
I'm not sure about:
A meta data file (.meta) exists but its asset 'Assets/MixedRealityToolkit/Inspectors/InputTemplates' can't be found. When moving or deleting files outside of Unity, please ensure that the corresponding .meta file is moved or deleted along with it.
The one I'm sure about i'm adding right to the line that is causing it

It looks like there may also be a bug in the way we handle array types under inspectors for example:

In this picture the user should be able to add Observers (but there is no + button to do so) whereas they can in the main MRTK thing.

It's a bit odd that you can Add Component to these facade objects, and I'm not really sure how those play out - I don't think it's worth doing magic here to remove user-added components, so maybe in the docs that we write for this we should call it out?
I don't see any way to disable the Add Component button, but also my Unity-fu isn't as strong as yours.

I'm also hitting a bug now where:

Enable service facades.
Notice that facades are correctly added to the scene graph.
Hit play
After coming back from play, the facades aren't re-added to the scene graph. I have to toggle the "service facades" checkbox.

This seems like maybe one of the more critical bugs to fix (if you can repro this)

The focus provider inspector looks to be empty by default, so I'm not sure what scenarios there are where this should be populated:

The back navigation-ability of modifying profiles is a bit weird:
For example:

Go the boundary scene facade

Double click the visualization profile.

Click "back to configuration profile"

It doesn't take you back.

This feels somewhat busted, though as far as the other issues called out here is probably less troublesome.

The teleport system inspector doesn't appear to have any way of adding events:

Other comments:

The teleport system inspector doesn't appear to have any way of adding events:

As a general rule I'm avoiding inspectors that let you alter services. I think it will be much cleaner if they just let you view its state.

It's a bit odd that you can Add Component to these facade objects

This is harmless enough but we should try to prevent it. Let's think of ways to catch it in the validation check.

This seems like maybe one of the more critical bugs to fix (if you can repro this)

This is one of the known issues called out in the Issues section.

The focus provider inspector looks to be empty by default, so I'm not sure what scenarios there are where this should be populated:

If you hit Play, and pointers are instantiated, the list will be populated. We can mention this in a help box, maybe.

/azp run

I still don't fully understand how the active pointers thing is supposed to show up - i.e. even in play mode my scene looks like this:

And this is with "Use Service Inspectors" on.

Overall this looks this looks great - I think there are still some bugs/issues (i.e. back navigation, edit vs read only) that would prevent us from enabling this on by default.
If I were to summarize the known issues here, would it be accurate to say:

Profile navigation (going back up) isn't working because it goes to the 'root' profile.
Disjoint set of edit vs read functionality (i.e. you still have to use the full profile experience to configure things until we can do the work to make it super

For any other other bugs that weren't addressed above, can you open an issue to track them just so that we don't lose them, or go through the rediscovery process again in the future.

/azp run mrtk_pr

/azp run mrtk_docs

I don't know what's going on with the builds right now, but I think that there's a docs break:
[19-04-29 08:54:15.042]Warning:BuildCommand.BuildCore.Build Document.LinkPhaseHandlerWithIncremental.ConceptualDocumentProcessor.SaveInvalid file link:(~/Documentation/Images/MixedRealityToolkitConfigurationProfileScreens/MRTK_EnableServiceInspectors).
2 Warning(s)

/azp run mrtk_docs

/azp run mrtk_docs

/azp run mrtk_docs

/azp run mrtk_docs

/azp run mrtk_docs


Can't download private vimeo vid

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 2018.09.26. 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 2018.09.26

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: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.09.26
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

sams-MacBook-Pro:~ samsotudeh$ youtube-dl -v youtube-dl https://player.vimeo.com/video/90307349
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'youtube-dl', u'https://player.vimeo.com/video/90307349']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.09.26
[debug] Python version 2.7.10 (CPython) - Darwin-16.7.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
ERROR: u'youtube-dl' is not a valid URL. Set --default-search "ytsearch" (or run  youtube-dl "ytsearch:youtube-dl" ) to search YouTube
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 792, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 507, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2214, in _real_extract
% (url, url), expected=True)
ExtractorError: u'youtube-dl' is not a valid URL. Set --default-search "ytsearch" (or run  youtube-dl "ytsearch:youtube-dl" ) to search YouTube--
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
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
Hi, I want to download this private video but i keep getting an error. I'm new to this so could you please help me

Remove second occurrence of youtube-dl in your command line.

This is what happens
sams-MacBook-Pro:~ samsotudeh$ youtube-dl https://player.vimeo.com/video/90449804
[vimeo] 90449804: Downloading webpage
[vimeo] 90449804: Extracting information
ERROR: This video is protected by a password, use the --video-password option; 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.

Maybe you'll bother to read the error message?


add definitions

What's a CPU? Provide a definition for beginners?
Add full acronym names, and what they stand for

[ x] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Some addresses showing as "West Bromwich" when they're not

See http://alpha.openaddressesuk.org/addresses?postcode=b74+3ud
The post town should be Sutton Coldfield.
Also occurs on some Solihull and Birmingham addresses

The two West Bromwich addresses appear to be honest mistakes made by people registering their companies there and not realising that their location is actually in Sutton Coldfield.
@murraydata has checked, and, interestingly, the B74 3 postcode sector appears to be actually a good split between Sutton Coldfield and West Bromwich. The kind of confidence calculations we are doing at the moment - whose granularity stops at postcode sectors - will not easily spot the error as we will likely observe the same distribution of addresses between the two towns. Spatial confidence will fix that in the future.
In the meantime, we will rely on validation votes to make the address score less.


increase retention days to a year

increase retention days of Yakut test to a year

@pavel-kirienko Merge this?

@maksimdrachov please pull it into your https://github.com/OpenCyphal/yakut/pull/109, then close it.


feat: merge small chunks for sink executor

Is your feature request related to a problem? Please describe.
It has been found that the chunks of a Kinesis sink executor are too small, with an average cardinality of 1.5. Since the Kinesis sink sends one chunk at a time to the external Kinesis system, if the distance between RisingWave and Kinesis is too great, it will result in low sink throughput.
I think we can support a chunk merge executor for sinks to avoid small chunks.

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

IIUC, this problem eixsts for all internal operators: any chunk that passes an Exchange will be splitted into smaller chunks. Saying, the parallelism = 128, then every chunk will be divided by 128 = 8 rows
I am thinking that, since the problem was caused by HashDispatcher, shall we solve it in the Merge? For example, waiting for all Ready chunks and merge them into a single bigger chunk.

https://github.com/risingwavelabs/risingwave/issues/15713

IIUC, this problem eixsts for all internal operators: any chunk that passes an Exchange will be splitted into smaller chunks. Saying, the parallelism = 128, then every chunk will be divided by 128 = 2 rows
I am thinking that, since the problem was caused by HashDispatcher, shall we solve it in the Merge? For example, waiting for all Ready chunks and merge them into a single bigger chunk.

It will hurt the latency so we need to make it configurable.

I tested the PR and it works great. There is no back pressure in the sink anymore. Let's discuss whether to apply the merging broadly (like for all types of sink or even the merge executor).

Let me add a histogram metric to monitor the input chunk rows for actor first to see in which workload, small chunks will appear. Because I think nexmark is not a good benchmark for this issue.


Change type of final script witness to Witness from Vec<Vec>

Doing this would certainly help APIs downstream that operate on &Witness because they would not conversion from &Vec<Vec> to &Witness.

Addressed the nits.


ui.router: ng-strict-di breaks templateProvider in $stateProvider.state

I've been trying to load my templates on '.run' and loading them via templateProvider. TemplateProvider never executes if 'ng-strict-di' is set on the application. Took me a long time to track this down to ng-strict-di. Currently unsure what's the problem and why ng-strict-di could affect templateProvider from running with no errors at all the console.
Here's a plunker to show it's not running when ng-strict-di is set:
http://plnkr.co/edit/YJDk8EAx5pM04lZpx4cj?p=preview
Another exact same plunker without ng-strict-di working:
http://plnkr.co/edit/HWC54bDeAa8dZdubMZdL?p=preview
Thanks and please correct me if I'm just horribly wrong.

ok i had the same issues, but it's seem not work in my project.
Removing the ng-strict-di, still break the templateProvider.
But the diferences is i am use angular 1.4.0 beta 1, and according to this i assume it's gonna work if returning a promise. If the issues goes when you and me returning a promise in templateProvider, i hope somebody will update the documentation to should return a promise.
see you @samy100 , i will try to return promise first

hai @samy100 , this is to your issue.
i assume the bug is gone when remove the dependency in the templateProvider
here take a look
but sadly it's not work in my project :disappointed:
and maybe you should take a look on templateFactory if you still need the injection.

Thanks @brutalcrozt
The second plunker you added is using an angular version where ng-strict-di is not working as it does in newer versions.
Looking this again, the problem was that template provider also need ['depn', func(depen)..] annotation and it was missing. Usually strict di tells you about this, not silently fail as it's doing with templateProvider for some reason. See working plunker now: http://plnkr.co/edit/wzDsRvXNVuje1mThWFCS?p=preview


osmc implementation

How difficult would it be to adjust this script to install in OSMC.

@luckymedog we currently don't support OSMC since it's a media center. It's based on Debian, so it shouldn't be too hard to port this over (you will probably have to bypass the OS checks here ). Let me know if you succeed :smiley:


Added v3-redirect-warning to the frameworks pages

@samejr you should use this when you copy the v2 docs to that other repo

This was used for the new v2docs repo


Add node 5.x

Would like to see node 5.x included...
Perhaps relabel 4.x/0.12.x as LTS and 5.x as Stable to better align with nodejs.org, removing 0.11.x

It actually is available, it just doesn't get listed. See #111.


Catch up to Ecto 2.1.3, released 18 Jan 2017.

Coverage remained the same at 84.101% when pulling e63459520517521c7d4ea0a9d642b8b56100a692 on ecto-2.1.3 into 2b9d58fe787d5845b7ca7438b4efb0f00ceb54ec on master.


"ValueError: negative dimensions are not allowed" and suggested fix

Hello,
i just got the error message  "ValueError: negative dimensions are not allowed",
when running the comprehensive example.
To fix it i had to change "Pymonics/pymonics/Memo_esprit.py"
line 48: inputDataSamples = inputSignal.iloc[0, :]
to: inputDataSamples = inputSignal.iloc[:, 0]
I am new to github and programming, so sorry if i disregarded some conventions.

Dear @Bronnnn Thanks for posting here. I tried to run the comprehensive example on my machine. I didn't receive any such error. Could you provide more details of the error? Also, if you are interested to collaborate and contribute to this package, feel free to approach us. We can add you to contributor list.


losing JS when switching to blocks in file list

If you edit JS and then switch not with "Blocks" button, but instead by clicking on main.blocks in the file list, there is no warning and JS is lost.

Fixed by issue #196


.env is .gitignored so isn't in the repo

had me stuck for a while 😅

Em... You're right. Guess that's why you won't be able to capture the flag.
I've updated the commit to fix the issue.

Cheers! 😄


Could not find artifact org.jacoco:jacoco-maven-plugin:pom:0.7.6-SNAPSHOT

When trying to build the package:
mvn clean package

[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] uMirrorMaker
[INFO] uMirrorMaker-Controller
[INFO] uMirrorMaker-Worker
[INFO] uMirrorMaker-Distribution
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building uMirrorMaker 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ uMirrorMaker ---
[INFO]
[INFO] --- maven-resources-plugin:2.5:copy-resources (copy-bin) @ uMirrorMaker ---
[debug] execute contextualize
[WARNING] Using platform encoding (UTF-8 actually) to copy filtered resources, i.e. build is platform dependent!
[INFO] skip non existing resourceDirectory /root/uReplicator/target/uMirrorMaker-pkg/bin
[INFO]
[INFO] --- maven-resources-plugin:2.5:copy-resources (copy-repo) @ uMirrorMaker ---
[debug] execute contextualize
[WARNING] Using platform encoding (UTF-8 actually) to copy filtered resources, i.e. build is platform dependent!
[INFO] skip non existing resourceDirectory /root/uReplicator/target/uMirrorMaker-pkg/repo
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building uMirrorMaker-Controller 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[WARNING] The POM for org.jacoco:jacoco-maven-plugin:jar:0.7.6-SNAPSHOT is missing, no dependency information available
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] uMirrorMaker ...................................... SUCCESS [0.336s]
[INFO] uMirrorMaker-Controller ........................... FAILURE [0.020s]
[INFO] uMirrorMaker-Worker ............................... SKIPPED
[INFO] uMirrorMaker-Distribution ......................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 0.507s
[INFO] Finished at: Wed Nov 02 15:20:45 GMT 2016
[INFO] Final Memory: 9M/241M
[INFO] ------------------------------------------------------------------------
[ERROR] Plugin org.jacoco:jacoco-maven-plugin:0.7.6-SNAPSHOT or one of its dependencies could not be resolved: Failed to read artifact descriptor for org.jacoco:jacoco-maven-plugin:jar:0.7.6-SNAPSHOT: Could not find artifact org.jacoco:jacoco-maven-plugin:pom:0.7.6-SNAPSHOT -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginResolutionException

Removed the jacoco plugin as like suggested here:
https://github.com/uber/uReplicator/issues/5
Now it builds the package.


Hibernate-cache: Fix deprecation warnings from XML

I am seeing deprecation warnings that I believe are coming from these XML files.

Sorry, I submitted this prematurely -- guess it wasn't coming from this XML after all.


Annotate peaks with chipseeker

annotatePeak with tssRegion=c(-2000,2000).
txdb limits this step to genomes known by chipseeker -- don't run on custom genomes.

usage in ASPEN https://github.com/CCBR/ASPEN/blob/master/workflow/scripts/ccbr_annotate_peaks.R


Expose person object to watcher hollabacks

Since there's no way of getting this information today, this would be a nice addition and doesn't interfere with existing properties.

I think I might make a Person class like I did for Channel with a proper toString.

That sounds cool. I had to change dependency from official to my fork on socialhapy because I was missing this. :)


For playlists, get newly added videos first.

When creating a feed for a playlist that is sorted as oldest-to-newest,
if the playlist has more than 50 videos you can never get updates for the newer videos,
since the most recent videos will always be outside of the max results of 50.
This could be fixed by generating the feed from the most recently added videos always,
regardless of the sort order that the playlist has.

That is a great point! Unfortunately, YouTube's API doesn't provide an efficient solution to retrieve the latest items from a playlist if it isn't sorted by most recently added at the top. I was trying to avoid any excessive API usage by just looking at the first 50 items in a playlist, but it seems like to be more broadly compatible I will have to pull all the items in the playlist to find the most recently added items.
I went ahead with this and added logic to find the most recently added ones whenever it looks like the playlist is not sorted by most recent already. Hopefully, the API usage is low enough that it won't cause any issues.
Let me know if the latest changes aren't solving your issue!

That did the trick, thanks.

Unfortunately, this has proven too costly to continue. Since it takes too many resources to collect all the elements of a playlist to find the newest ones, I have disabled this feature in the hosted version of this project. See the README for more information on how to achieve this functionality with a self-hosted solution.


Scrolling should have an offset that keeps the 1-9 labels in view

Gif to demonstrate what I mean:

It should start scrolling when the 9 in the leftmost column reaches the top line instead of the numbers scrolling off screen.

Looks like its not currently possible to fix this but will be at some point in the future.
https://github.com/fdehau/tui-rs/issues/328

Should probably be fixed but it's definitely a niche issue, so I wouldn't put priority on this

Resolved by #524!


Synch with the apache/flink master repo, upstream

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[ ] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[ ] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Merged automatically.

confirmed merge this time.


[Automation] Altenative for Power Cap Good case test execution

There is existing design discussion going on https://github.com/openbmc/openbmc/issues/557
However there are Good test cases path for Min, Max, in tests/test_occ_powercap.robot:
Untill the issue #557 is addressed or come to conclusion we would move the writing to sensor/host/powercap to /org/openbmc/settings/host0   power_cap  which make a logical sense for GOOD case test case.

Will  address a couple of them using https://github.com/openbmc/openbmc-test-automation/issues/118
Will close this as there are pending design going on for power cap settings..


Implements operator and CSI subcommand

Description
Reimplements the operator and CSI subcommands as Cobra commands.
They do not yet replace the actual code, since I wanted to break up the changes into multiple smaller PRs.
Please review the refactoring and restructuring and tell me what you think.
N.B.: I moved the original files from src/cmd/operator to src/cmd/ but added new ones with the same name. That is why some old files are listed as new ones
How can this be tested?

Deploy the DTO on a cluster
All DTO components must start as usual

Checklist

[x] Unit tests have been updated/added
[x] PR is labeled accordingly
[x] I have read and understood the contribution guidelines

@meik99 What's the idea behind hiding methods for different structs behind type Provider interface?

@meik99 What's the idea behind hiding methods for different structs behind type Provider interface?

Making the command builders and built commands more testable.
By using interfaces instead of specific classes they can be more easily exchanged for mocks


Fixed developer guide

Pull Request Test Coverage Report for Build 122

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

Totals

Change from base Build 118:
0.3%

Covered Lines:
1988

Relevant Lines:
2412

💛  - Coveralls


[gitlab-ci-pipeline-exporter] Update serviceAccountName template in the Deployment

The name of the template was changed from exporter.serviceAccountName to app.serviceAccountName in acea05b, but not updated in the Deployment. This caused the templating to fail when rbac.enabled was set.
Error: template: gitlab-ci-pipelines-exporter/templates/deployment.yaml:30:38: executing "gitlab-ci-pipelines-exporter/templates/deployment.yaml" at <{{template "exporter.serviceAccountName" .}}>: template "exporter.serviceAccountName" not defined

Thanks @marcfrederick !


Error on interactive map example

It looks like the Interactive Map example isn't working as expected: https://stackblitz.com/edit/solid-map-gl-interactive?embed=1&file=src%2FApp.tsx. Actually none of the examples are working due to this error.

Temporarily manual method is to add maplibre-gl to package.json

Hi David,
I fixed this issue in the latest release. I'm using a different way now to include MapLibre.
Let me know if the latest version fixes it for you.


Customised Bridge initialisation fails for iOS

Description
Our project is a hybrid of React Native modules and native UI. We ideally want only the React Native developers to run the development server, while the rest pick up the bundle file.
We have implemented the RCTBridgeDelegate protocol which is passed to the bridge during initialisation. Below is a snippet of the methods which return the source URL. In debug, the source URL should be the development server, with the bundle as fallback, if it's not running.
- (NSURL *)sourceURLForBridge:(RCTBridge *)bridge
{
#ifdef DEBUG
    NSURL* sourceURL = [NSURL URLWithString:@"http://localhost:8081/index.ios.bundle?platform=ios&dev=true"];
#else
    NSURL* sourceURL = [[NSBundle mainBundle] URLForResource:@"main" withExtension:@"jsbundle"];
#endif

    return sourceURL;
}

- (NSURL *)fallbackSourceURLForBridge:(RCTBridge *)bridge
{
    return [[NSBundle mainBundle] URLForResource:@"main" withExtension:@"jsbundle"];
}

I must be doing something wrong because I still get the Red Screen if I try to load a React Native module. Any suggestions will be appreciated.
Additional Information

React Native version: 0.35
Platform: iOS
Operating System: MacOs

Do you see the red screen while running in debug mode, running outside of debug mode, or both? What does the error say? Does your server properly respond with a bundle when hitting that URL?

I get the standard error that the development server is not reachable. I have only tried in Debug. This will not fail in Release since we are picking up the bundle.
If the server is running, it responds with the bundle.

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


statistics: Does not record previously available jobcode

For jobcode that was previously present but was deleted, the statistics does not take note of this. There may be some jobcode that has 0 contacts, and this is not captured.

no need. give scenario that there is a job code 10 years ago


chore: update test coverage for ibc client

Description
Improve test cases for ics-02 client

updated existing test cases
added new test cases to cover edge cases

Commit Message
type: commit message

see the guidelines for commit messages.
Changelog Entry
version: <log entry>

Checklist

[ ] I have performed a self-review of my own code
[ ] I have documented my code in accordance with the documentation guidelines
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have run the unit tests
[ ] I only have one commit (if not, squash them into one commit).
[ ] I have a descriptive commit message that adheres to the commit message guidelines
[ ] I have added version bump label on PR.

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.54%. Comparing base (3767742) to head (25407cb).
Report is 1 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #843      +/-   ##
============================================
+ Coverage     70.25%   70.54%   +0.28%
  Complexity      414      414
============================================
  Files           155      155
  Lines         14028    14028
  Branches        292      292
============================================
+ Hits           9856     9896      +40
+ Misses         4022     3982      -40
  Partials        150      150

Flag
Coverage Δ

rust
68.71% <ø> (+0.33%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes


aurora package not correctly working on windows

i would say this is not something urgent to address, since it is strictly cosmetics and doesn't impact any functionality of the cli. just putting this here first so we don't forget about it.

Follwoing issue https://github.com/mesg-foundation/core/issues/355 we decided, colorized output (aurora) will be dropped.

We're using https://github.com/fatih/color/ now and it's working on windows (tested on win10)


Acheron x Showdown

Song Name
Acheron x Showdown
Artist Name
savylui
Source
Youtube
Youtube Link
fcae91CaPe7zF4Jq
Direct File Link
No response
Song ID
770520
Start Offset [ms]
10000
Extra
Created from Auto Nong.
Created for level: 73667628

reject
invalid YouTube link


(PUP-6636) Only match /environment/:env route as the whole path

Previously, the /environment/:env route wasn't properly anchored to the
start of the path, so routes with other prefixes would also match.
Because auth is performed separately based on the path, this allowed
requests like /status/environment/production to bypass auth checks and
retrieve environment catalogs. This also meant that other valid requests
ending in /environment/:foo, such as for files, would be handled
incorrectly.
We now properly anchor the /environment/:env regex to the beginning of
the path, so only explicit environment requests will match.
This is a cherry-pick

I'm going to let the tests finish just in case, but this looks like a clean cherry-pick to me

Looks good to me 👍

CLA signed by all contributors.


[DOCS] Improving the README.md

Description
I would like to be assigned for GSSOC'23 for this issue
Suggestions:

Restructuring the header for a more visually appealing look
formatting text in a better manner
SEO friendly changes to link and text
adding badges for a cool effect
other minor changes

Screenshots
No response
Additional information
No response

Their is spelling mistake in documentation of backend which is written as "backnd " .
@J0SAL  can you assign me for better enhancement  and changes in  README documentation.

Could you please assign me this as a GSSOC Contributor.

Assign this task to be as I can make good changes in it.As I am a GSSOC Contributor.

@the-amazing-atharva @Aditi323 @Aditya-Aware, we'll keep the documentation part for the end, (already discussed in discord)


flutter_localization conflict with kalender 0.3.7

Hello,
i want to use in my project the library component  kalender  but the latest version of this library use the intl 0.19 that
goes into conflict with the flutter_localization that use intl 0.18.1.
Is there a possibility that flutter_localization will be updated, so that it can also use the new version of the intl library?
Thank you,Regards

@GSFdev24
I'm sad to say that I cannot update the intl. Since this package (flutter_localization) is not dependent on intl directly but depends on Flutter SDK flutter_localizations (the one that uses the intl 0.18.1). We have to wait for the Flutter update for this.
There is a workaround, you need to switch the Flutter channel to master (which is not recommended) and the issue is solved.

@GSFdev24 I'm sad to say that I cannot update the intl. Since this package (flutter_localization) is not dependent on intl directly but depends on Flutter SDK flutter_localizations (the one that uses the intl 0.18.1). We have to wait for the Flutter update for this.
There is a workaround, you need to switch the Flutter channel to master (which is not recommended) and the issue is solved. But you might get other issues caused by the master channel.

Ok, i understand thank you

Look like Flutter SDK flutter_localizations has been updated their intl version in the latest Flutter version. I will close this case for now. Thank you.


config: enable activation service by default on public endpoinnt

while looking at https://github.com/spacemeshos/go-spacemesh/pull/4701 i found that "activation" wasn't
enabled by default on public endpoint

bors merge


Need to download image from a url not from  a file or buffer

I need to render dynamic image from a url in pdf file

This is not built into PDFKit, but you can do it yourself fairly easily. Make the AJAX request with xhr.responseType = 'arraybuffer', then pass xhr.response to doc.image.


Simplify React JS mixin cursors

Hi there,
I have been thinking about the mixins and how you define cursors. I think it would be a lot more powerful and easier to understand if you narrowed down how mixin cursors work to just this:
var MyComponent = React.createClass({
  mixins: [tree.mixin],
  cursors: {
    todos: ['todos', 'items'],
    isLoading: ['todos', 'isLoading']
  },
  render: function () {
    return (

{this.state.todos.length} and {this.state.isLoading}

    );
  }
});

Just remove cursor and cursors property using arrays. What we gain by this is:

There is only one way to do it, and it is the best way to do it (most flexible)
You can name the property on the state, which can differ from the name on the cursor
You can put the cursor data directly on the state, not under a this.state.cursors.todos
By only having a store mixin you always have access to all data (best practice)

It is really annoying that you can not use this state directly as it is just too much to write inline with your JSX. this.state.cursors.todos and this.state.todos.
I am not sure how a transition to this one single format could work, but a suggestion is:

Breaking API change
Deprecate the cursor and cursors array format, but keep it for a couple of more versions
Deprecate cursor mixins

I think it would really clean up the library and make it easier to use and understand :-)

The only thing that scares me with this approach is a potential incompatibility with some components already using specific keys on their states that would conflict by the names of the keys in your store.

Hi :-)
I see your point, but the key you use to define a cursor is the key that
will be used on the state object. You define your "getInitialState" and
cursor keys in the same file, so you have to work pretty hard to get a
conflict. And if you do, it is as easy as changing either the
"getInitialState" key or cursor key. Could even state an error from Baobab
in case of conflicting keys.
On fre. 6. mars 2015 at 11:55 Guillaume Plique<EMAIL_ADDRESS>wrote:

The only thing that scares me with this approach is a potential
incompatibility with some components already using specific keys on their
states that would conflict by the names of the keys in your store.
—
Reply to this email directly or view it on GitHub
https://github.com/Yomguithereal/baobab/issues/110#issuecomment-77541418
.

This would also allow for letting the user use a function instead of a map:
var MyComponent = React.createClass({
  mixins: [tree.mixin],
  cursors: function () {
    return {
      todos: this.props.todosCursor,
      isLoading: ['todos', 'isLoading']
    };
  },
  render: function () {
    return (

{this.state.todos.length} and {this.state.isLoading}

    );
  }
});

Allowing you to use instance properties etc.

@christianalfoni  I love this.

I like where this is going. This feature, as well as the removal of state.key will probably make it to the v1.

:+1: for removing the ".cursors" bit and putting the data directly in state
also, I would find it very useful to be able to use instance properties in cursor paths like this:
    var Channel = React.createClass({
      mixins: [tree.mixin],
      cursor: ['channels', this.props.id] // this.props not valid here
      },
      render: function () {
          ...
{this.state.cursor.name}

      }
    });

This seems like a nice option to have, and would solve the problems i'm having with dynamic cursor paths:
    var Channel = React.createClass({
      mixins: [tree.mixin],
      cursor: function(){ return ['channels', this.props.id]; },
      },
      render: function () {
          ...
{this.state.cursor.name}

      }
    });

This will make it into the 1.0.0 version, normally. Even if I have some difficulties to sketch how the mixin would work under the hood to perform this.

@Yomguithereal We built a mixin like this at our company that has this behavior. Would be happy to contribute with a 1.0 mixin :-)

@christianalfoni I'd like to see that, is it already published somewhere?
Discovered Baobab from your blog post btw, so thanks for that :)

@RJ You are very welcome :-)
Let me just paste the code here with comments. I did some changes to this after pasting it so might be a bug here, but you will figure that out if you want to test, I am sure :-)
This mixin allows you to do:
var Comp = React.createClass({
  cursors: function () {
    return {
      list: ['admin', 'todos'] // cursor key decides state key
    }
  },
  render: function () {
    // this.state.list
  }
};

Have to merge this with existing cursor to also allow using a map as cursors.
// The mixin needs a reference to your Baobab instance
var stateTree = require('./stateTree.js');

/*
 * Grabbed code from React lib as it will not change and we reduce
 * a dependency with one. Also testing becomes easier
 */
var shallowEqual = function shallowEqual(objA, objB) {
  if (objA === objB) {
    return true;
  }
  var key;
  // Test for A's keys different from B.
  for (key in objA) {
    if (objA.hasOwnProperty(key) &&
        (!objB.hasOwnProperty(key) || objA[key] !== objB[key])) {
      return false;
    }
  }
  // Test for B's keys missing from A.
  for (key in objB) {
    if (objB.hasOwnProperty(key) && !objA.hasOwnProperty(key)) {
      return false;
    }
  }
  return true;
};

var ReactComponentWithPureRenderMixin = {
  shouldComponentUpdate: function(nextProps, nextState) {
    return !shallowEqual(this.props, nextProps) ||
           !shallowEqual(this.state, nextState);
  }
};

module.exports = {
  mixins: [ReactComponentWithPureRenderMixin],
  cursorSubscriptions: null,
  ignoreCursorUpdate: false,
  onCursorUpdate: function () {

    // There can be multiple updates on the same tick,
    // ignore those, as we only need one update
    if (this.ignoreCursorUpdate)  {
      return;
    }

    this.ignoreCursorUpdate = true;
    setTimeout(function () {
      this.ignoreCursorUpdate = false;
      this.setCurrentCursorsValues();
    }.bind(this), 0);

  },

  // Remove data from component when cursor is removed
  onIrellevant: function (event) {

    // Wait for next event tick
    setTimeout(function () {
      var key = event.target.path.slice(0).pop();
      delete this.state[key];
      this.setState(this.state);
    }.bind(this), 0);
  },
  getCurrentCursorsValues: function () {

    var cursors = this.cursorSubscriptions;
    return Object.keys(cursors).reduce(function (cursorsValues, cursorKey) {
      cursorsValues[cursorKey] = cursors[cursorKey].get();
      return cursorsValues;
    }, {});

  },
  setCurrentCursorsValues: function () {

    // If there is a cursor update while unmounting,
    // let us just return
    if (!this.isMounted()) {
      return;
    }

    var cursorsValues = this.getCurrentCursorsValues();

    // attach to state
    this.setState(cursorsValues);

  },
  componentWillMount: function () {

    // Might not have any cursors
    if (!this.cursors) {
      return;
    }

    // Make sure cursors is a function
    if (typeof this.cursors !== 'function') {
      throw new Error('You have to use a function and return cursors in an object on ' + this.constructor.displayName);
    }

    var cursors = this.cursors();

    // Add actual cursors to map and listen to changes
    this.cursorSubscriptions = Object.keys(cursors).reduce(function (subscriptions, key) {
      var cursor = stateTree.select(cursors[key]);
      cursor.on('update', this.onCursorUpdate);

      // Have to wait for next event loop, as the data might still be comitting
      setTimeout(function () {
        cursor.on('irrelevant', this.onIrellevant);
      }.bind(this), 0);

      subscriptions[key] = cursor;
      return subscriptions;
    }.bind(this), {});

    this.setCurrentCursorsValues();

  },
  componentWillUnmount: function () {
    if (this.cursorSubscriptions) {
      Object.keys(this.cursorSubscriptions).forEach(function (key) {
        this.cursorSubscriptions[key].off('update', this.onCursorUpdate);
        this.cursorSubscriptions[key].off('irrelevant', this.onIrellevant);
      }, this);
    }
  }
}

Okay, let me know if you try it and get into trouble, would maybe we can work on it here and make it ready as a suggestion for 1.0 :-)

@christianalfoni, I think you can use the library combinations to simplify your mixin's update listeners. Example here.
I don't think those are accessible outside of the library but you can still use the cursors' or method to build it.

Would be possible for the mixin to not depend on the Baobab instance and instead pass the instance (and maybe also the cursors) somehow from inside the component?
Are you looking at coming up with an alternative to the mixin for react 0.13 so that it can be used with ES6 classes? Higher-order Components could be an alternative maybe https://gist.github.com/sebmarkbage/ef0bf1f338a7182b6775
I just started playing out with Baobab but I'll gladly help if come up with something.

Another great example would be FluxComponent from flummox
https://github.com/acdlite/flummox/blob/master/docs/why-flux-component-is-better-than-flux-mixin.md

Those are interesting concepts @nickdima, I'll check that more thoroughly soon.
What should be done to enhance ES6 classes compatibility?

As ES6 classes don't have any standard way of doing mixins, React decided they won't implement their own proprietary system. More details here, look for the Mixins section: http://facebook.github.io/react/blog/2015/01/27/react-v0.13.0-beta-1.html

Hi guys!
Got the flue last weekend so been on the couch all week, but back now :-)
Here are my thoughts on this.
I think it would be nice to create three different interactions with Baobab and React:

Mixin from the tree instance

var tree = require('./tree.js');
var Comp = React.createClass({
  mixins: [tree.mixin],
  cursors: function () {
    return {
      items: ['items']
    };
  },
  // or
  cursors: {
    items: ['items']
  },
  render: function () { ... }
});

ContextMixin where the tree is passed in the context
Unsure how to handle the key of the context. It probably needs to have a specific key, like baobab.

var Baobab = require('baobab');
var Comp = React.createClass({
  mixins: [Baobab.ContextMixin],
  contextTypes: {
    baobab: React.PropTypes.Object
  },
  cursors: function () {
    return {
      items: ['items']
    };
  },
  // or
  cursors: {
    items: ['items']
  },
  render: function () { ... }
});

Extend class

var tree = require('./tree.js');
class MyComponent extends React.Component {
  constructor () {
    tree.applyCursors(this, {
      items: ['items']
    });
  }
}

Just some suggestion to try to narrow down a spec for the different scenarios Baobab could handle :-)

Moving this discussion to this repository.

Been so busy lately @Yomguithereal , just checked out the repo. It looks freakin awesome! I think you did a great choice here moving it to its own repo!


[FEATURE] Be able to reference Environment Variables on WADM App Manifest for Secrets and Config

Being able to reference environment variables on App Manifest for config and secrets would be really wonderful. Areas on the manifest this may be relevant include Link Defs and Config.
An example of using Environment Variables with AWS Secrets Manager can be found here:
https://docs.aws.amazon.com/AmazonECS/latest/developerguide/secrets-envvar.html
I wonder if HashiCorp Vault and others allow similar use.
@brooksmtownsend mentioned the potential of storing values in JetStream also.

It also aligns with the 12 factor app principle for config:
https://12factor.net/config

Secrets may not be a good use case for this as per discussion in this thread:
https://wasmcloud.slack.com/archives/CS38R7N9Y/p1682949268796849

Are there use cases where there would be value in referencing an environment variable on a WADM app manifest instead of just typing the value on the manifest?

Need to investigate if Secrets are a good use case for env variables. Also, list other use cases for referencing env variables on the app manifest before moving forward with this.

If there is potential to use providers for secrets and grab the secrets through the defined LinkDefs like Jochen suggested, could consider starting secret manager providers in a preliminary stage of deployment.

I will look at this after config json and provider scaler issues

I no longer have time for this one, moving onto some other work.


Minification: Create local variable for prototype during class declaration

Creates a local variable storing a class's prototype to support better minification of class declarations when emitting for ES5 and below.
Example
Source
class A {
    a(): void { }
    b(): void { }
    get c(): { return 1; }
}

Old ES5 emit
var A = (function () {
    function A() {
    }
    A.prototype.a = function () { };
    A.prototype.b = function () { };
    Object.defineProperty(A.prototype, "c", {
        get: function () { return 1; },
        enumerable: true,
        configurable: true
   });
}());

New ES5 Emit
var A = (function () {
    function A() {
    }
    var proto_1 = A.prototype;
    proto_1.a = function () { };
    proto_1.b = function () { };
    Object.defineProperty(proto_1, "c", {
        get: function () { return 1; },
        enumerable: true,
        configurable: true
   });
}());

Fixes #9638

@dmichon-msft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Isn't this worse after compression? Do you have some numbers?

Hey @dmichon-msft, I think @alfaproject is correct. When using gzip compression, my understanding is that you end up with a table referencing these repeated segments anyway. Now, you have the same table lookup with additional code for the prototype alias that's inserted.
aving tried the given samples out on this page it looks like this change is less efficient. Do you have concrete numbers indicating otherwise?

Working on it.
@DanielRosenwasser Is there a guide somewhere for how to successfully npm link a local build of typescript?

@alfaproject @DanielRosenwasser
So, for a single, small class, this change results in slightly larger files after gzip. All following analytics are conducted by assuming that the minifier will perform optimum renaming of variables for maximum compressibility, then passing them through this page that Daniel linked.
Single, small class:
var A = (function () {
    function A() {
    }
    A.prototype.a = function () { };
    A.prototype.b = function () { };
    Object.defineProperty(A.prototype, "c", {
        get: function () { return 1; },
        enumerable: true,
        configurable: true
   });
}());

Result: 273 bytes before, 163 bytes after gzip
vs.
var A = (function () {
    function A() {
    }
    var a = A.prototype;
    a.a = function () { };
    a.b = function () { };
    Object.defineProperty(a, "c", {
        get: function () { return 1; },
        enumerable: true,
        configurable: true
   });
}());

Result: 268 bytes before, 168 bytes after gzip
Winner: before change, by 5 bytes/168 ~ 3%
If we add a couple more classes:
var A = (function () {
    function A() {
    }
    A.prototype.a = function () { };
    A.prototype.b = function () { };
    Object.defineProperty(A.prototype, "c", {
        get: function () { return 1; },
        enumerable: true,
        configurable: true
   });
}());

var B = (function () {
    function A() {
    }
    A.prototype.c = function () { };
    A.prototype.d = function () { };
    A.prototype.e = function () { };
    A.prototype.f = function () { };
}());

var C = (function () {
    function A() {
    }
    A.prototype.g = function () { };
    A.prototype.h = function () { };
    A.prototype.i = function () { };
    A.prototype.j = function () { };
    A.prototype.k = function () { };
}());

Result: 716 bytes before, 206 bytes after gzip
vs
var A = (function () {
    function A() {
    }
    var a = A.prototype;
    a.a = function () { };
    a.b = function () { };
    Object.defineProperty(a, "c", {
        get: function () { return 1; },
        enumerable: true,
        configurable: true
   });
}());

var B = (function () {
    function A() {
    }
    var a = A.prototype;
    a.c = function () { };
    a.d = function () { };
    a.e = function () { };
    a.f = function () { };
}());

var C = (function () {
    function A() {
    }
    var a = A.prototype;
    a.g = function () { };
    a.h = function () { };
    a.i = function () { };
    a.j = function () { };
    a.k = function () { };
}());

Results: 671 bytes before, 206 bytes after gzip
Winner: neither.
Adding more classes or members from this point gradually leans in favor of the change, though the delta is relatively small.
If I change the starting assumption about the minifier performing optimal variable renaming, then things get a bit stranger. I attempted to invoke the modified compiler against a much larger internal project, but due to some difficulties my numbers for the modified compiler include upstream files still built using the old compiler, and more local files build with the modified version.
Nonetheless, some partial numbers for some real world projects (all post gzip):

479.26 KB -> 479.08 KB
554.55 KB -> 554.41 KB
496.75 KB -> 496.63 KB
556.62 KB -> 556.44 KB
458.36 KB -> 458.16 KB (largest improvement, at 0.20 KB)
289.69 KB -> 289.74 KB (largest regression, at 0.05 KB)

So, to sum up, gzip is pretty fantastic at compressing away .prototype., but there is still some room for improvement, and this is a pretty minor change. It also results in a reduction in uncompressed code footprint, which is unfortunately rather difficult to usefully quantify the effects of.

This should also result in a small runtime performance improvement by avoiding the need to lookup the prototype property repeatedly. The engine's optimizer might be able to help once it warms up, but this code likely all runs during application startup while you're inside the interpreter.

Can we revive this PR anyone?


docs(readme): Document debugging with node --inspect-brk

To me, this is the most important type of debugging. It's what I think of when I think of debugging my test. I was happy to find the instructions buried here.

@melissachang Thank you for the PR!


Create Chip abstraction under Cluster

Now that tt_SiliconDevice has moved to tt::umd::Cluster, we can work forward towards the diagram https://docs.google.com/drawings/d/1-m1azdsBqMA0A6ATYRMfkhyeuOJuGCEI62N5a96LXj0/edit
Create proper Chip abstraction which takes away most of functionalities under the Cluster.

I'll create several PRs to come to resolution of this:

Create Chip abstraction underneath.
Move Simulation and Mockup to extend Chip rather than Cluster.

tt_metal changes

Delete tt_device

tt_metal changes

Delete unnecessary Cluster api's, move them to Cluster->get_chip(id)->something().

tt_metal_changes


Generating candidates that aren't in the text verbatim

For the IoT project with tables, we would like to be able to have candidates that are inferred from a span without actually being present. For example, given a list of part numbers “BC348...350”, we would like to return ["BC348", "BC349", "BC350"], despite "BC349" never actually showing up in the text. Basic options I see:

Expand this list in preprocessing. I don't love that idea.
Incorporate this in a Matcher. I don't know that this will work with our current setup, since Matchers are basically just filters that return a True/False on their inputs.
Incorporate this in a CandidateSpace. This seems like a better fit than the matcher, because here we can actually "generate" options for becoming candidates. Though it does go against the usual flow that we've seen so far where the Matcher is where most user preferences are expressed, rather than the CandidateSpace.

The main issue I see here then is just that the text of the candidate won't actually correspond to true character offsets, so get_span('words') and friends may not work as expected. I'm sure there's a way to make this all work regardless, but I wanted to see if any of you (esp. @ajratner, @stephenbach, @lukehsiao) have any immediate thoughts on this for a particularly elegant solution.

Similar to #138
I agree that #3 is the nicest way to go!

(Meant option 3, not issue number 3)

Ah yes, that is a relevant thread. There could be two issues here though:

composite mentions (maybe allowing char_start and char_end to be lists?)
hallucinated mentions (adding a mention that isn't actually in the text, but is in inferred, as in our case)

Yeah so I agree with you completely that modifying the candidate space is
the way to go. And then yes we just need to think though the downstream
effects
On Thu, Aug 11, 2016 at 2:47 PM Braden Hancock<EMAIL_ADDRESS>wrote:

Ah yes, that is a relevant thread. There could be two issues here though:

composite mentions (maybe allowing char_start and char_end to be lists?)
hallucinated mentions (adding a mention that isn't actually in the
text, but is in inferred, as in our case)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/snorkel/issues/392#issuecomment-239303422,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABgw_XQ68Po1X8SpV3Stwypeh4Ip-eQTks5qe5h1gaJpZM4JilUE
.

Additional examples:
Lists: BC348A/348B/348C
Ranges: BC348...350
Groups: Group A (where BC348A, BC349A, and BC350A are all referenced in the table somewhere)

A thought on the implicit spans that we may use to address this issue:
Since they're created after the initial parse that creates the corpus, they won't necessarily have all the CoreNLP tags that all verbatim spans have. We could either resolve this by:
a) allowing the user to specify if/how they would like to define the POSes, lemmas, etc. for the implicit spans they create and add additional error handling for those methods (such as get_span()) that expect these attributes to always be non-None
b) calling the CoreNLP server again on the words in these implicit spans and have all those attributes populated automatically. pro: less handling of edge cases. con: more calls to CoreNLP, which may cause a performance hit?
I vote (b), since most of the time, people won't be using a CandidateSpace that does range splitting, so it's a non-issue, and those that are providing their own range-splitting rules and likely willing to take the (probably very small) hit to have their ranges enumerated.

This is incorporated via ImplicitSpan objects in the tables_master branch. Coming soon to a master branch near you.


settingsScreen: Add icons to the menu options

Added Feather icons to all the options in the Settings screen.
Closes #2068

The code looks good. You should not center the text though.

@jainkuniya Appended a period at the end of my commit messages, as requested. 😃

Looks good! Both the code and the UI :)
Thanks for the effort 👍


[client] Shouldn't be able to place building if not enough resources

The building should be greyed out and a tooltip appear if not enough resources

taking issue


Research a markdown-based qt resource method for popup tooltips and overlays.

Tooltips are explicitly defined in the code. It may be better dynamically source a static markdown resource on-display of the popup / tooltip. This would allow for richer tutorials and help because we can embed images / videos and subsections.
The users may also want to interact with the text so a tooltip may not be sufficient.

Duplicate


CDK DRAG&DROP : Transfer item from on group list to another on click or keyup event (keyup.code ===13)

Feature Description
CDK DRAG&DROP with transfer feature on events(mouse, key etc)
Use Case
it help to transfer a input filed from one group to another

Using the keyboard isn't really drag and drop anymore. You can implement this yourself already by binding a keydown event.


How to setup a Virtual Environment and install Django in it

Description
Give a step by step creation of a virtual environment and install Django in the environment
Domain
Backend Dev (Python)
Type of Contribution
Video, Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign I'm interested

/assign


[1Password] ...

Extension
https://www.raycast.com/khasbilegt/1password
Description
When i close the window, I was asked to log in every time
Steps To Reproduce
1.Close the Raycast window
2.Open "my password" in Raycast
Current Behaviour
No response
Expected Behaviour
No response

@raycastbot close this issue


consider tagging v1.0.0

please considers tagging
git -a v1.0.0 tag 6daaf91
so we connect to that specific version (related to https://github.com/jspm/jspm-cli/issues/1851)

6daaf91 is not the version 1.


Deprecate some Vector specializations

Deprecate VectorExpandImageFilter and VectorCentralDifferenceImageFunction in favor of their generic versions.
According to #725.

It is VectorCentralDifferenceImageFilter which is from ITKReview. Did you mean to swap the names ?

@maekclena I think Matt did not confuse the classes. VectorExpandImageFilter is in ITKImageIntensity and should stay there while ITK major version is 5. But it can/should be deprecated.
VectorCentralDifferenceImageFilter is currently in ITKReview. ITKReview is meant as a temporary place for filters, and filters from it can be moved to other modules at any time. So moving VectorCentralDifferenceImageFilter into ITKDeprecated is fine.


SAK-47228 Enable building webcomponents based on vue

This adds an vuecomonents folder, where vue components can be created. Those will be globally available across Sakai as webcomponents. This also includes some tutorials, on how to work with vue components in Sakai.

@ern It should be all ready now!


xpath/py-dom-xpath/abbreviations_t01 test fails on chrome on macos

The test
co19/LayoutTests/fast/xpath/py-dom-xpath/abbreviations_t01
has started failing on chrome on macos, with the error:
          value: test @name failed:raised NoSuchMethodError: method not found: 'get$nodeName' (J.getInterceptor$x(...).get$nodeName is not a function).
          timestamp: 0.053
    },
    {
          type: print
          value: test @* failed:raised NoSuchMethodError: method not found: 'get$nodeName' (J.getInterceptor$x(...).get$nodeName is not a function).
          timestamp: 0.054
    },
    {
          type: print
          value: test ../@lang failed:raised NoSuchMethodError: method not found: 'get$nodeName' (J.getInterceptor$x(...).get$nodeName is not a function).
          timestamp: 0.056

Also failing on windows.  LInux is not yet updated to Chrome 45.


front 3.5mm headphone jack does not work properly

First thank you for your contribution,this EFI work fine on my machine.
But the sound card does not seem to work properly. When the iphone earpod is inserted into the front 3.5mm headphone jack, there is no sound when it is inserted completely, and there is sound when it is inserted halfway.

Yes i know , i have fixed on my machine but i don't update the repo the correct layout is 14 or 15.
As soon as possible i update the repository

14 can work on my machine, thank you again for your work
One more question, is there a way to turn on HiDPI on Big Sur 11.2? I tried almost all the solutions I found on the Internet, but none of them work properly. My monitor is DELL U2515H, which can only display 2K on Big Sur 11.2, I successfully turned on HiDPI on Catalina  10.15.7 before, and it can be displayed at 1080P resolution,but the same method does not work on Big Sur.
The main method I tried is to put the DisplayVendorID folder under /System/Library/Displays/Contents/Resources/Overrides

Try switchresx app

switchresx does not work, I will continue to research, thank you.

14 can work on my machine, thank you again for your work
One more question, is there a way to turn on HiDPI on Big Sur 11.2? I tried almost all the solutions I found on the Internet, but none of them work properly. My monitor is DELL U2515H, which can only display 2K on Big Sur 11.2, I successfully turned on HiDPI on Catalina 10.15.7 before, and it can be displayed at 1080P resolution,but the same method does not work on Big Sur.
The main method I tried is to put the DisplayVendorID folder under /System/Library/Displays/Contents/Resources/Overrides

layout 14 work on my machine on input set audio input ¢

switchresx does not work, I will continue to research, thank you.

have you try with this https://github.com/xzhih/one-key-hidpi ?

switchresx does not work, I will continue to research, thank you.

have you try with this https://github.com/xzhih/one-key-hidpi ?

Yes, I have tried all of these, and finally I found that the problem is not in the display settings.
When I was looking for the solution, someone mentioned that if the performance of the graphics card is not enough, the 4k monitor cannot be used, so I wonder if there is some problem with current graphics card configuration? after some attempts, HiDPI can be turned on now.
The graphics card settings are as follows , Although I don't know what it means, but it works fine.

The effect of turning on HiDPI:


Cannot load library libgit2

I'm not 100% sure if I post on the right repo.
I have a ipynb file calling Julia 1.0.x code. I wanted to use my unregistered package so downloading from my GitHub repo with the following code (I'm a newbie with Julia, so yet not sure of the correct code):
using Pkg; Pkg.add(PackageSpec(url="https://github.com/xxx/yyy.jl"));

On my GitHub page, on https://nbviewer.jupyter.org and on my own jupyter-lab I have the same error (while working perfectly on my Julia 1.0.x):
error compiling #add_or_develop#13: error compiling shred!: error compiling #26: error compiling with: error compiling #27: could not load library "libgit2"
/usr/lib/x86_64-linux-gnu/libmbedx509.so.0: undefined symbol: mbedtls_mutex_unlock

Stacktrace:
 [1] #add_or_develop at ./none:0 [inlined]
 [2] #add_or_develop#12 at /build/julia-wJr69F/julia-1.0.3+dfsg/usr/share/julia/stdlib/v1.0/Pkg/src/API.jl:29 [inlined]
 [3] #add_or_develop at ./none:0 [inlined]
 [4] #add_or_develop#10 at /build/julia-wJr69F/julia-1.0.3+dfsg/usr/share/julia/stdlib/v1.0/Pkg/src/API.jl:27 [inlined]
 [5] #add_or_develop at ./none:0 [inlined]
 [6] #add#18 at /build/julia-wJr69F/julia-1.0.3+dfsg/usr/share/julia/stdlib/v1.0/Pkg/src/API.jl:69 [inlined]
 [7] add(::Pkg.Types.PackageSpec) at /build/julia-wJr69F/julia-1.0.3+dfsg/usr/share/julia/stdlib/v1.0/Pkg/src/API.jl:69
 [8] top-level scope at In[1]:1

It looks like you should include a Julia compiled against the static lib libgit2 but against a shared lib

Hm, no, this isn't the right repo. Jupyter code is written in Python and Typescript. Your problem with Julia in a notebook is about installations of a Julia kernel.
I suggest that you join the Jupyter Forum at https://discourse.jupyter.org/ and post your question/suggestion in the Q&A section there to get in contact with the people running nbviewer.jupyter.org. For GitHub, you'd have to contact GitHub, but they are probably running some default configuration. There's a chance that changes rolled out to nbviewer.jupyter.org might be picked up by GitHub later.
As for the behavior of your own jupyter-lab installation, that's clearly your own responsibility. If you followed somebody else's instructions for installing the Julia kernel, you might ask whoever provided the instructions.
But if you're doing something that fails everywhere except on your own, non-Jupyter Julia installation, then you might be doing something that just isn't compatible with how the Julia kernel works. In that case, the Jupyter Forum would also be a good place to figure out what's going on and get in contact with people that can help you. Another place to reach out to would be the IJulia kernel community. Their GitHub repo is https://github.com/JuliaLang/IJulia.jl
I'm closing this issue as off-topic.


lack of kafka "Default node configuration and virtual machine sizes f…

…or clusters" information
I add Kafka Default node configuration and virtual machine sizes for clusters into Tables. I check Azure Portal and filled default VM size, but I didn't find recommended VM size. So, could you please add recommended VM size into that table?

 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.:x: takeshi-masuda sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@takeshi-masuda : Thanks for your contribution! The author, @kkampf, has been notified to review your proposed change.

@mamccrea Please triage if you have time. Takeshi is in support at Microsoft. Thanks, Jason

@kkampf can you check the accuracy of the Kafka column that was added? Thanks, Jason

Thanks for the edit, we are updating this information in a separate request #please-close


Release v2.3.0

Rationale
I couldn't push directly to main as written in the release instructions so I'm opening a branch instead. We should also do a GH release afterwards.
The PR should be merged using rebase and merge strategy so the commit hash is not changed.

Ugh. The commit SHA changed. Per GitHub's docs:

Rebase and merge on GitHub will always update the committer information and create new commit SHAs, whereas git rebase outside of GitHub does not change the committer information when the rebase happens on top of an ancestor commit.

Rebase and merge on GitHub will always update the committer information and create new commit SHAs, whereas git rebase outside of GitHub does not change the committer information when the rebase happens on top of an ancestor commit.

Oh, that's a pity. I assumed if the author of the PR merges it then the metadata stays the same (I knew that when someone else merges a PR he is added as co-author so the metadata changes). I can see only two workarounds:

Use merge commits (I dislike this)
Change the tag manually after merge (I dislike this as well)

I agree neither is ideal (went with change tag manually for this since it was already merged). Though there is an option 3- have push access to main 😄

I added @Siegrift to the Airnode releasers team


Add Dragan Theodora to Contributors list

@theodoradragan, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Broken formatting for jsx in case :: function binding

VSCode Version:1.10.1
OS Version: Ubuntu 16.04

Steps to Reproduce:
1.Open BadFormatting.js as jsx or babel.
2. Format file (Ctrl+Shift+I).
As result formatting is brken:

If I remove :: function binding all become ok.

This issue was moved to Microsoft/TypeScript#14491