fix documentation

Yay! This works now.
The hack is to execute the notebook / leave some output cells in there. Then sphinx won't try to execute the notebook while building the documentation, but just takes the outputs that are there.
And I had always deleted all output cells because I assumed that they caused the issues ...

Ah good to know!


Support EIP1102 aka. Privacy Mode or at least test to verify it doesn't break dapp browser

https://eips.ethereum.org/EIPS/eip-1102
https://medium.com/metamask/https-medium-com-metamask-breaking-change-injecting-web3-7722797916a8

https://github.com/James-Sangalli/xcontract/issues/6 would provide a good test case.

@hboon this is supported in a branch on xcontract but the web3 beta api seems to be riddled with bugs, goes hand in hand with issue #877

@hboon this is dependent on the dapp itself, currently cryptokitties has this feature and it works without asking on our wallet (we probably need to upgrade web3 to get that to ask for permission)

@hboon another note, unless privacy mode is enabled on the client, the default way will continue. We could potentially add this feature in later

@James-Sangalli re-opening


Is there a score threshold in RPN only training?

❓ Questions and Help
I followed the RPN only config but got a precision close to 0.  It seems the proposed boxes are not filtered by a score threshold like in ROI_HEAD.SCORE_THRESH and thus there are always a maximum possible number of boxes.
Is there any configuration to handle this or i need to do some coding here?

No, we always filter RPN via a maximum number of boxes, see https://github.com/facebookresearch/maskrcnn-benchmark/blob/1714b7c20d919f8c536a577e15ba3a03ce733cde/maskrcnn_benchmark/config/defaults.py#L148-L161
You could change the code where those parameters are used in order to set a limit on the threshold, but I believe you'll might end up having a lot of boxes.


[question] Can you provide example for expo detached app?

I am not sure where to put GoogleService-Info.plist in expo detached project.
I've been suffering for this whole day and need help to integrate this library to expo detached project. Hope somebody can help me..

it doesn't matter where you put it physically. you need to make sure it gets copied into binary during build.

@evollu
Thanks I've just done it and still can't resolve my problem.
FCM.getFCMToken().then((token) => {
  console.log(token);
})

token is always undefined.

check your native log in xcode. it should state what is wrong with firebase

@evollu
I've just tried it, and it just says undefined and nothing is showing related to firebase. If I am using cocoapods I don't need to do anything in xcode am I right? What I've just done is install FirebaseMessaging in podfile.

Nevermind. I just included below codes provided for without cocoapods support and it works fine now. Thanks for help.


Provide Getting Started Guide

Provide a guide that explains the steps to do when I want to get JenkinsHue up and running. Which Philips Hue products do I need and how do I set them and the application up?

Done 😄

How do I create a user when not using LDAP? This should be explained in the Getting Started Guide.

Never mind, just found it in the guide :)


Add option to perform isotropic scaling

I think it would be useful to add an option so that the same random scaling is applied to all dimensions. I can work on it.

Maybe I don't understand, but isn't this what the zoom preprocessing option does?

I forgot to mention this issue in the corresponding PR: https://github.com/NifTK/NiftyNet/pull/324
What's the zoom preprocessing option? Can you point me to some code/docs?

@Zach-ER, in the previous implementation, different scaling percentages were computed for different dimensions. This could make a prostate look like a sausage and would probably be undesired. The new flag is to make sure that a similarity transform is applied to the volume, with a single scaling factor for all axes.


Youtube-dl crashes on age restricted video

hi,
/tmp $ youtube-dl -U
youtube-dl is up-to-date (2021.06.06)
/tmp $ youtube-dl -v 'https://www.youtube.com/watch?v=3cIW1n3oP_g'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=3cIW1n3oP_g']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 2.7.16 (CPython) - Linux-4.19.0-16-amd64-x86_64-with-debian-10.9
[debug] exe versions: ffmpeg 4.1.6-1, ffprobe 4.1.6-1, phantomjs ., rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 3cIW1n3oP_g: Downloading webpage
[youtube] 3cIW1n3oP_g: Refetching age-gated info webpage
WARNING: unable to download video info webpage: HTTP Error 410: Gone
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/mrt/bin/youtube-dl/__main__.py", line 19, in <module>
  File "/home/mrt/bin/youtube-dl/youtube_dl/__init__.py", line 475, in main
  File "/home/mrt/bin/youtube-dl/youtube_dl/__init__.py", line 465, in _real_main
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2069, in download
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 808, in extract_info
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
  File "/home/mrt/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
  File "/home/mrt/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1503, in _real_extract
  File "/home/mrt/bin/youtube-dl/youtube_dl/compat.py", line 2644, in compat_parse_qs
  File "/home/mrt/bin/youtube-dl/youtube_dl/compat.py", line 2614, in _parse_qsl
AttributeError: 'bool' object has no attribute 'split'
/tmp $

lookes like it crashes...
thanks and regards
PS: where is the issue template gone where i have to answer all that questions?

Duplicate of #29333
With Git HEAD and python 2.7:
[youtube] 3cIW1n3oP_g: Downloading webpage
[youtube] 3cIW1n3oP_g: Refetching age-gated info webpage
WARNING: unable to download video info webpage: HTTP Error 410: Gone
ERROR: Sign in to confirm your age
This video may be inappropriate for some users.
Traceback (most recent call last):
... snipped ...
    raise ExtractorError(reason, expected=True)
ExtractorError: Sign in to confirm your age
This video may be inappropriate for some users.

As to the issue template, you need to be signed in to GitHub first before pushing the New issue button.

hi,
/tmp $ youtube-dl -U
youtube-dl is up-to-date (2021.06.06)
/tmp $ youtube-dl -v 'https://www.youtube.com/watch?v=3cIW1n3oP_g'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=3cIW1n3oP_g']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 2.7.16 (CPython) - Linux-4.19.0-16-amd64-x86_64-with-debian-10.9
[debug] exe versions: ffmpeg 4.1.6-1, ffprobe 4.1.6-1, phantomjs ., rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 3cIW1n3oP_g: Downloading webpage
[youtube] 3cIW1n3oP_g: Refetching age-gated info webpage
WARNING: unable to download video info webpage: HTTP Error 410: Gone
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/mrt/bin/youtube-dl/__main__.py", line 19, in <module>
  File "/home/mrt/bin/youtube-dl/youtube_dl/__init__.py", line 475, in main
  File "/home/mrt/bin/youtube-dl/youtube_dl/__init__.py", line 465, in _real_main
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2069, in download
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 808, in extract_info
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
  File "/home/mrt/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
  File "/home/mrt/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
  File "/home/mrt/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1503, in _real_extract
  File "/home/mrt/bin/youtube-dl/youtube_dl/compat.py", line 2644, in compat_parse_qs
  File "/home/mrt/bin/youtube-dl/youtube_dl/compat.py", line 2614, in _parse_qsl
AttributeError: 'bool' object has no attribute 'split'
/tmp $

lookes like it crashes...
thanks and regards
PS: where is the issue template gone where i have to answer all that questions?

are you signed in to your youtube account ??
did you use the authenticated cookies option ??

Duplicate of #29578?
Also:
Try using Python3, by making an alias in ~/.bash_aliases:
alias youtube-dl='python3 /usr/local/bin/youtube-dl'
Then you'll probably won't get these errors, but instead the age restriction warning...
All this can be fixed though. The yt-dlp fork downloads this video just fine.

Request like this works as expected:
POST /youtubei/v1/player HTTP/1.1
Host: www.youtube.com
X-Goog-Api-Key: AIzaSyAO_FJ2SlqU8Q4STEHLGCilw_Y9_11qcW8

{"context":{"client":{"clientName":"ANDROID","clientScreen":"EMBED",
"clientVersion":"16.05"},"thirdParty":{"embedUrl":"https://www.youtube.com"}},
"videoId":"3cIW1n3oP_g"}


Safari initial height ( with ng-repeat an ng-resource )

Whenever i repeat data that comes from the backend with a promise, the height of the textarea is just being blown out of proportion. I'm talking one line of text with a height of 974px. So i've fixed this temporarily by forcing angular-elastic to execute forceAdjust(); whenever the var heightValue is not NaN because that is what's being returned the first couple of times by angular-elastic.
This is probably not the best solution so i would like to hear from you guys if you have any ideas to fix this. Sadly this only happens in safari so i'm not entirely sure why it is happening. Also i'm gonna try to fix a plunker so i hope that i can reproduce the problem.

Hi,
Did you ever get anyway with this? The forceAdjust method is there for situation like this, but be interesting to hear if you got any further.
Thanks for the contribution.

Thank you for leaving this issue open, together with #67 it helped me track the issue in our app. For what it's worth, here's our situation and workaround.
We had a text area that was initially hidden. Upon user interaction (click) this happened in order:

A function on the controller is called.
That function uses a $timeout to initialize a callback after 800ms.
The callback does some DOM manipulation (yeah, I know...) and switches around with classes and whatnot, causing the boxes in the DOM to change slightly.
The callback also makes sure the text area handled by elastic becomes visible.

The textarea would then sometimes be slightly of the wrong height.
Our workaround looked like this:
this.someControllerFuncForClicks = function() {
  $timeout(function() {
    // Do DOM manipulation, set classes, etc.
    // Set a controller prop making the textarea visible
    $scope.$broadcast("elastic:adjust"); // <-- this is the workaround
  }, 800);
}

The $broadcast indirectly causes the library to do a forceAdjust, making the box resize to the appropriate height.

Archiving repository.


Add a weak reference to Webkit framework for iOS

The WKWebView class is referenced in the code here which requires a weak link to the Webkit framework. Without said weak link Xcode fails the build during the linking phase with the error:
Undefined symbols for architecture arm64:
  "_OBJC_CLASS_$_WKWebView", referenced from:
      function signature specialization <Arg[0] = Owned To Guaranteed and Exploded> of Cordova500.GeofencePlugin.evaluateJs (Swift.String) -> () in GeofencePlugin.o

Thanks


adding ability to read GeoJSON points in addition to flat points arrays

This is a work in progress; i have it working but need to debug the lookup function for click handling. Looking to the lines.js file for a reference... any tips appreciated, thanks!
Basically, I have a huge amount of data in the following GeoJSON format, rather than a flat nested array of [lat, lon] tuples:
{
  "type": "FeatureCollection",
  "crs":{"type":"name","properties":{"name":"urn:ogc:def:crs:OGC:1.3:CRS84"}},
  "features":[
    {"geometry":{"coordinates":[-71.4178125,41.81443759],"type":"Point"},"properties":{"name":"WEISS & GEISLER, INC.","sic_name":"Jewelry & Other Miscellaneous Manufacturing","city":"Providence","year":1953,"open":1953,"sic_color":"#008000","street":"409 Pine St","close":1953,"employees":"0-49"},"type":"Feature"},
    {"geometry":{"coordinates":[-71.41780474,41.81480583],"type":"Point"},"properties":{"name":"WEISS & GEISLER, INC.","sic_name":"Jewelry & Other Miscellaneous Manufacturing","city":"Providence","year":1959,"open":1959,"sic_color":"#008000","street":"147 Summer Street","close":1959,"employees":"0-49"},"type":"Feature"}
  ],
}

I found Leaflet.glify wasn't expecting that, so I made some modifications to allow for this format of GeoJSON, which also allow the properties of each point to be used in the popup, which wasn't previously possible for point data.
Update - ready for review!

OK, i found that it was more efficient to run the following on the dataset to "clean it up" first, and to give it non-nested 0 and 1 object properties, so it could be accessed as feature[0] and feature[1] for example. This simplified the changes to the code dramatically. But it requires an initial filtering step like this, which I wasn't sure where to put:

   // standardize lat/lon instead of lon/lat
   // and add non-nested coords for feature[0], feature[1]
   data.features.forEach(function(f) {
     f[1] = f.geometry.coordinates[0];
     f[0] = f.geometry.coordinates[1];
     f.geometry.coordinates[0] = f[0];
     f.geometry.coordinates[1] = f[1];
   });

This also served to correct the longitudeFirst, in part because I found relying on longitudeFirst() to be really hard to code around. I kept getting bugs related to that, so I wanted to suggest there be an initial prefilter step that just corrects the datasource, rather than constantly keying the correct order, because managing state is pretty rough that way. I dunno what you think about that, though?

Hi @robertleeplummerjr if you had a moment to look over this, I'd be grateful! Thanks for a great library!

I will look at this soon, sorry, been busy.

Looking at this now.

Added support from recommendations here in https://www.npmjs.com/package/leaflet.glify/v/3.0.0
Ty!


Support messages that are greater than 1024 characters

The v1 Event API supports large messages, but if new fields are specified, the Kubernetes API converts it to a events.k8s.io/v1 Event which has a limit of 1024 characters.
The solution was to stop setting the eventTime field so that it stays a v1 Event.
Relates:
https://issues.redhat.com/browse/ACM-13720

/cherry-pick release-2.11

/cherry-pick release-2.10

/cherry-pick release-2.9

/cherry-pick release-2.8


Data is not reloaded if initial request returns no results

If the first query for a model returns an empty set then one of the query params is changed the resource will not be reloaded and will continue to return an empty set.

hitting this same issue, any progress on incorporating the fix?

also checking out the fix's PR, looks like the wrong variable name (forceReload vs force) might be used in reloadRelationship, which might be causing the CI error:
https://github.com/mdehoog/ember-data-has-many-query/pull/26/files#diff-db478f9ef87f5a766fd804b373628213R89
    reloadRelationship: function (propertyName, forceReload) {
    ...
            if (isLoaded || force) {

I'm also hitting this issue. For now I'm going forking and run against @arschmitz's changes, but it would be great to get this merged in.

+1 on this. Only when initial query returns an empty set.

What's the hold up? How can we move this across the finish line.


Version 2.0 coming soon!

New version of this project will be coming soon and im pretty exited.
*New changes include:

Sending an email consisting of a data file
More lag
UI improvement

Bugs:
No bugs as of Version 1.1
*Some of these features might not come in version 2.2

UI Development Complete

Version 2.0 Live!


Check for Qm or ba

This is not a nice fix but it does solve the specific Optimism bug
Should also be fixed in the next rollout: https://github.com/graphprotocol/graph-node/pull/4089

Interesting. How is it optimism specific though?


Stacked widget

Looked into using stacked widget to provide some separation to the programme and make ui feel more modern


Rename logo-white.png to Src/logo-white.png

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

@gstraccini codacy bypass


Dialogue

Describe your bug here.
startDialogue(dialogueJson); crashes the song and lua dialogue started the dialogue but also starts the song at the same time, version 0.5.2 please fix this. and yes I DID NOT modify anything in the lua script shadow give in the wiki and also exact script from wiki
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Mac
Did you edit anything in this build? If so, mention or summarize your changes.
no

Im not sure but i think lua don't work properly in other os than Windows


How to handle Multiple types of items?

I have a music app, and I want to use this recommendation engine on it.
I want to be able to recommend “Music” and “Artists”. So ideally, music and artists should be items.
How do I differentiate this 2 different recommendations? Also how do I distinguish music and artist in the items table?
I mean how do i get the API to return a music recommendation alone?

what you could do is create 2 categories music and artists then GET /api/recommend/{user-id}/{category}


When no bridging is required we are returning this wrong gas_limit

When no bridging is required we are returning this wrong gas_limit. We should avoid mutable variables here and only construct Transaction when all its fields are valid.
Original comment

Migrated to Linear: https://linear.app/reown/issue/CR-595/when-no-bridging-is-required-we-are-returning-this-wrong-gas-limit


Update README.md

adding typewriter effect

adding typewriter effect

Please resolve the conflicts.

I'm not getting, why this conflict occurred?


Update conversion between IR and QCircuit

These updates will support global phases and make the codes more elegant.

tests are failing.

The PR itself looks good to me.

@Roger-luo @GiggleLiu How to fix the test without register a new version of ZXCalculus.jl? The codes it needs is in the #master branch.


pom: should add FindBugs static analysis to jetcd

It seems to me a lot of java projects includes FindBugs maven plugin to help developer finding potential bugs. We probably should add that to Jetcd too.
http://findbugs.sourceforge.net/

Maybe we can use http://errorprone.info, not sure which one is the best but we try them

sounds good. close in favor of https://github.com/coreos/jetcd/issues/83


v.30 on Openshift cluster references nonexistent image of ose-kube-rbac-proxy

Description
devworkspace operator v.30 installation on Openshift 4 cluster tries to pull registry.redhat.io/openshift4/ose-kube-rbac-proxy@sha256:fde6314359436241171f6361f9a1e23c60bdf2d421c0c5740734d1dcf5f01ac2
How To Reproduce

For new install, find operator v.30 in OperatorHub
Installed with default configs
pods for devworkspace-controller-manager and devworkspace-webhook-server fail with ImagePullBackOff error

Expected behavior
devworkspace-controller-manager and devworkspace-webhook-server pods should deploy properly
operator should install with existing ose-kube-rbac-proxy image in redhat registry
Additional context
Originally was on v.29. Subscription is set to automatic approval

Hi,
I can confirm this is also happening to us when updating from 0.29.0 to 0.30.0:

Failed to pull image "registry.redhat.io/openshift4/ose-kube-rbac-proxy@sha256:fde6314359436241171f6361f9a1e23c60bdf2d421c0c5740734d1dcf5f01ac2": reading manifest sha256:fde6314359436241171f6361f9a1e23c60bdf2d421c0c5740734d1dcf5f01ac2 in registry.redhat.io/openshift4/ose-kube-rbac-proxy: manifest unknown.

This helps https://access.redhat.com/solutions/7084768.
Edit devworkspace-operator.v0.30.0 CSV to replace the image from registry.redhat.io/openshift4/ose-kube-rbac-proxy@sha256:fde6314359436241171f6361f9a1e23c60bdf2d421c0c5740734d1dcf5f01ac2 to registry.redhat.io/openshift4/ose-kube-rbac-proxy@sha256:514e9e03f1d96046ff819798e54aa5672621c15805d61fb6137283f83f57a1e3
Edit devworkspace-webhook-server deployment to sub out the same image.

Sorry for the delayed reply, we've been working on addressing this issue downstream, see https://issues.redhat.com/browse/CRW-7172.
This issue only affects the downstream productized version of DevWorkspace Operator (i.e. when installed through OLM or OperatorHub on OpenShift) and not the upstream version.
The workaround mentioned by @Ruckus0143 is our currently provided workaround until this issue is resolved, see https://access.redhat.com/solutions/7084768.

The fix for this issue on x86 and IBM Z (s390x) platforms has been shipped: https://access.redhat.com/errata/RHBA-2024:6449
Please note:

If DevWorkspace Operator 0.30.0 was previously installed, you will need to apply the workaround from https://access.redhat.com/solutions/7084768 OR delete the clusterserviceversion for DevWorkspace Operator 0.30.0 with oc delete clusterserviceversion devworkspace-operator.v0.30.0 -n <operator-install-namespace>. The <operator-install-namespace> refers to the namespace where DevWorkspace Operator is installed, which is openshift-operators by default, e.g. oc delete clusterserviceversion devworkspace-operator.v0.30.0 -n openshift-operators.
This fix only works for x86 and IBM Z (s390x) platforms. The fix for PPC is currently being worked on and will be coming soon.

I will mark this issue as resolved when the fix for PPC becomes available.

The fix for this issue that works for all platforms (x86, s390x and PPC) has been shipped: https://access.redhat.com/errata/RHBA-2024:6628
An update for DevWorkspace Operator 0.30.2 should appear in the Installed Operators page on OpenShift.
Please note:
If upgrading from DevWorkspaceOperator v0.30.0 or v0.30.1, you may have to delete the existing devworkspace-operator.v0.30.0 or devworkspace-operator.v0.30.1 ClusterServiceVersion after initiating the upgrade (depending on which version of DevWorkspace Operator is installed). This is only required if your install of DevWorkspace Operator v0.30.0 (on all platforms) or v0.30.1 (on PPC) is broken.
The above can be done from the OpenShift Console UI or with oc delete clusterserviceversion devworkspace-operator.v0.30.0 -n <operator-install-namespace> or oc delete clusterserviceversion devworkspace-operator.v0.30.1 -n <operator-install-namespace>. The  refers to the namespace where DevWorkspace Operator is installed, which is openshift-operators by default.


Remove debian folder

Now that the build produces Docker images, we can remove the debian folder entirely.

This change is

Time to merge this one too.


abolished/normal discrimination of growing end PMID:10574765

Jacky requested:
Boundary of non growing cell end maintained
During monopolar growth the boundary of the non growing cell end is maintained and is distinct from the cell body
Boundary of the non growing cell end not maintained
During monopolar growth the boundary of the non growing cell end is not maintained and is indistinguisable from the cell body
This is a  phenotype where a marker, which is usually cortical and excluded for the growing ends, is no longer excluded.
Monopolar mutants are used to demonstrate this, because they still have a non growing end, but I think the phenotype should be "generic".
perhaps:
abolished/normal discrimination of growing end
figure 2

Hmmm. This is one of those awkward cases where what you want to say is actually an interpretation of an observed phenotype. The observation in this figure is simply "protein mislocalized to cell cortex of non-growing cell tip" (which doesn't exist in FYPO but would be easy to add), or not.
To go beyond the observation it would help to konw how a normal process could be named and defined - it's something along the lines of "setting the boundary of a cell end", but all of the words I've thought of so far would probably clash with more specific developmental biology usage (e.g. boundary determination or boundary specification).
In any case, "discrimination" doesn't really work here because it means recognizing an existing distinction between two or more different things, not bringing the distinction into existence.
(I would also love to have a way to track directly observed vs. inferred-by-curator phenotypes some day ...)

The problem is that isa all a bit artificial.
The marker which is used  (Cor1 fragment)  actually turned out to be a fragment of Nse6.
So, itself is not normally locilised to the cortical region, but its  fragment is.
So its actually
"protein mislocalized to cell cortex of non-growing cell tip, of a protein which  has a  mutant which locates to the cortex , but is normally nuclear". Which clearly we can't capture.
This fortuitous marker is then used  to show that orb1 mutants can no longer discriminate their growing end.
This  phenotype is observed in mutants which only grow monopolarly so they know there is only a single growing end. Then they show that orb1 does not recognize that this single end is growing, but I'm interpreting that this is only  to make a simpler model  and that orb2 is generally deficient in  discriminating between growing and non-growing ends (which also fits with the other data).
Jacky, did I  understand that correctly ?  @jvhayles

Oh I see what you mean about discrimination. I'm thinking...

me too. maybe "delineation"? we should run it by David Hill ...
there are probably other cases where a region within a cell is somehow defined, so GO should come up with something generally applicable (if only they weren't dysfunctional ...)

I was just thinking
"loss of cell end specification" ? which sounds along the same lines...

unfortunately, "specification" is one of those words that has a really specialized meaning in dev biol, so it would be better if we can think of something that won't clash :/


[UI] Need separate configurations for setting URL for general help and context sensitive help

Currently CloudStack has "docBase" in config.json file where one can change the base URL for all sections. Changing the "docBase" to a new URL will effect all sections which may not be a convenient setting.
For example, changing the "docBase" URL from "http://docs.cloudstack.apache.org/en/latest/" to "http://dochelp.custom.url/" this change effect the general help section here (below image) which is good

But it also changes the base URL for other sections like context sensitive help (below image) for example on a specific virtual machine page the help URL is updated to "http://dochelp.custom.url//adminguide/virtualmachine.html"

It is good to have two different settings for URLs, one for general help and another for all context sensitive help sections.
Also if feasible, a single place like a json file where one can update the bookmarks/URLS to the general help and individual context sensitive help sections (eg: VM, template, snapshot, volume, etc.)
ISSUE TYPE

Improvement Request

COMPONENT NAME
UI

CLOUDSTACK VERSION
4.15

Replaced by - https://github.com/apache/cloudstack/issues/4731 (a mapping proposed)


Compatible with Ecto?

Hi! Is this library compatible with Ecto as the library this was forked from? If not, would love to see that feature!

Sorry, it is not compatible. I think, it does not make sense, because there are two different concepts. Ecto is built for rational databases while MongoDB is document based. So, adding an additional layer on the driver which is not compatible due to the different concepts will have more disadvantages than advantages.
I think, it is the best to use the driver directly without using a wrapper like Ecto. If you want to use defstruct instead of simple maps you can convert directly manually. Using this approach you don't need to think about the syntax of the DSL to build queries, so the DSL query is translated back to the correct MongoDB query. Since the MongoDB-API uses JSON/BJSON to express queries and updates, the danger of sql-injection is not very high, because you need to build maps and keyword lists. Strings are not used to build query and update expressions. So, the DSL query language makes no sense at all in case of MongoDB.

Thanks for the response! I think I actually framed my question wrong -- it was more accurate to ask if they're an ecto adapter for this driver a la https://github.com/ankhers/mongodb_ecto The driver this was forked from doesn't interface with ecto -- the adapter does the integration.
As for using Mongo with Ecto, we've been using it for 3 years and really love the combination. We have reasonably complex documents in Mongo and haven't encountered anything that Ecto can't model.
We lean on Ecto for fetching data. We drop down to the mongo driver for doing atomic updates ($set and friends) -- all places where we want to very precisely use what Mongo offers.
So, for us, we don't see an either or with Mongo or Ecto -- we see them very much as complimentary and really appreciate the the other Mongo driver lets us use it stand alone or integrate with Ecto. It lets us have the best of both worlds.
I haven't looked into how the mongodb_ecto adapter was written, but if this library added instead of changed the API from the other driver, I imagine it either works now or would be in a good place to add compatibility. On a related note, how stable do you feel your work is now? Is the api something that could be relied on as is? Or, do you have plans to change it in small or large ways?

I haven't looked into how the mongodb_ecto adapter was written, but if this library added instead of changed the API from the other driver, I imagine it either works now or would be in a good place to add compatibility.

It is possible to the this driver for the mongodb_ecto adapter, but I think it needs some work. An interesting part is the transaction API, for example.

On a related note, how stable do you feel your work is now? Is the api something that could be relied on as is? Or, do you have plans to change it in small or large ways?

The API is stable so far. I try to integrate the test files from the specification and adding stuff like 'retryable' reads and writes. But before that I need to extend the event broadcasting module, so the test cases can fetch events in order to receive curtain events.
My plans are

add more test from the driver specs
add missing features like retryable writes, errorLabels, retryable commits


Feature: Create a new logo for the CrawlerX platform

Create a new logo for the CrawlerX platform and insert it inside the folder called logo in the root directory. Also, update the favicon of the web-app

I would like to create a logo.  Please assign it to me.


[Networking] Building A Single Server TCP Route Using Python

Topic Suggestion
Networking
Writing sample(s):
https://www.section.io/engineering-education/authors/srikumar-r-pilladin/
Proposal Submission
Proposed title of article
Building A Single Server TCP Route Using Python
Proposed article introduction
In most of the applications, we are following centralized server - multiple client architecture. The underlying architecture requires networking where TCP is the most commonly preferred protocols. In this tutorial, we will understand how a single server can interact with multiple clients using TCP protocol. We will also be learning from scratch to build a single server connecting single/multiple clients using sockets in Python.
Key takeaways

The reader will understand how TCP connection works
The reader will understand the basics of server-client architecture
The reader will understand threading in TCP connections
The reader will build a server-client architecture using Sockets in Python

Article quality

Most of the articles on the internet does not deal with threading that binds any server or client. In this article, we will learn how to bind any server or client and establish a connection that accepts any number of clients dynamically.
Moreover, we will building the server-client architecture from scratch. Enable dynamic acceptance of clients to a server with threading enabled connections.

References

en.wikipedia.org/wiki/Transmission_Control_Protocol
https://stackoverflow.com/questions/17453212/multi-threaded-tcp-server-in-python

Sounds like a helpful topic - lets please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it).
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved

I'm going to close this issue due to inactivity. It can be re-opened.

Closing due to inactivity.
This topic was approved a month ago - to make sure all our topics are up to date and unique we will be closing this issue.
You are welcome to re-submit topic for review and re-approval.
Since this topic may have been covered here or elsewhere since then.


Adjust notebook to changes in bnn-pynq brevitas example

Fixes #299

This is indeed due to newer versions of Brevitas restructuring the BNN-PYNQ examples. Thank you for the PR!


Editor Download

Cannot download the editor it links me to a 404 page. https://www.dropbox.com/s/11pif5xn4a0t6uk/Screenshot 2015-06-26 17.55.35.png?dl=0
Have you discontinued the native editor? I wanted to the use native editor because I was having trouble adding plugins in the online editor.

You can download it from here: https://github.com/GitbookIO/editor-legacy/releases


Uncaught TypeError: Cannot read property 'push' of undefined

Note: it throws this error but still appears to work.
To Reproduce:

Write a comment, e.g. "#doing:20 Task in a text file like markdown" (in a .md file)
Save the file (throws this error)

Atom Version: 1.0.5
System: Microsoft Windows 10 Pro
Thrown From: imdone-atom package, v1.1.10
Stack Trace
Uncaught TypeError: Cannot read property 'push' of undefined
At C:\Users\bfunkhouser\.atom\packages\imdone-atom\node_modules\imdone-core\lib\repository.js:405

TypeError: Cannot read property 'push' of undefined
    at C:\Users\bfunkhouser\.atom\packages\imdone-atom\node_modules\imdone-core\lib\repository.js:405:19
    at C:\Users\bfunkhouser\.atom\packages\imdone-atom\node_modules\imdone-core\lib\mixins\repo-fs-store.js:86:11
    at C:\Users\bfunkhouser\.atom\packages\imdone-atom\node_modules\imdone-core\lib\tools.js:220:24
    at FSReqWrap.oncomplete (fs.js:77:15)

Commands
     -0:09.8.0 imdone-atom:tasks (atom-text-editor.editor.is-focused)
     -0:03.1.0 core:paste (atom-text-editor.editor.is-focused)
     -0:02.8.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "one-light-ui",
      "pen-paper-coffee-syntax"
    ]
  }
}

Installed Packages
# User
imdone-atom, v1.1.10
pen-paper-coffee-syntax, v0.9.1

# Dev
No dev packages

Fixed by modifying uri with encodeURIComponent

Hooray!  Thanks!

Hello, I still got the same error message,
does this issue be fixed? or what should I do to fix it?
thank you.


Lack of description in Settings - "Crop and Lock"

Microsoft PowerToys version
0.82.1
Installation method
Microsoft Store
Running as admin
None
Area(s) with issue?
Settings
Steps to reproduce
Unclear what Crop and Lock does :( Would be good to ensure each of these settings has adequate details. Most of the rest were fine, but this one left me confused
✔ Expected Behavior
Expected a brief helpful explanation like for "Find my Mouse"

❌ Actual Behavior
Instead just the title, a small icon and two commands without explanation of what the feature is all-up

Other Software
No response

Hi @msheets1
The dashboard provides a quick way to find out how to activate the utilities.
You can find a description of the utility at the top of the setting page and also on the "Lean more" link.

/needinfo


Must define exactly one of HIP_PLATFORM_AMD or HIP_PLATFORM_NVIDIA Error

Hello, I am currently building on Crusher where I am linking Kokkos and Kokkos Kernels to a Ascent+Genten+VTKm build and I am encountering the following error in when I am building Ascent (all other libraries build and install):
/opt/rocm-5.2.0/include/hip/hip_runtime.h:66:2: error: ("Must define exactly one of HIP_PLATFORM_AMD or HIP_PLATFORM_NVIDIA");
#error("Must define exactly one of HIP_PLATFORM_AMD or HIP_PLATFORM_NVIDIA");
I have encountered this error in two different builds: one that uses the cray compilers and one using amdclang.
I've attached txt files of my scripts so you can see the compiler variables I set.
If you need anymore information please let me know. Any help would be very appreciated!
Here is the verbose build line:
[ 40%] Building CXX object ascent/CMakeFiles/ascent.dir/runtimes/flow_filters/ascent_runtime_genten_filters.cpp.o
cd /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/build/ascent && /opt/rocm-5.2.0/bin/..//bin/amdclang++ -DASCENT_EXPORTS_FLAG -DHAVE_ROCBLAS -DHAVE_ROCSOLVER -DVTKMDIY_MPI_AS_LIB -DVTKMDIY_NO_THREADS -Dascent_EXPORTS -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/thirdparty_builtin/lodepng -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/thirdparty_builtin/tiny_obj -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/ascent -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/build/ascent -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/ascent/utils -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/ascent/runtimes -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/ascent/hola -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/flow -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/build/flow -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/flow/filters -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/ascent/runtimes/flow_filters -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/rover -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/genten-gitlab/install/include/genten -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/libs -I/gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/build/libs -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/vtk-m-v1.8.0/install/include/vtkm-1.8/vtkm/thirdparty/lcl/vtkmlcl -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/vtk-m-v1.8.0/install/include/vtkm-1.8/vtkm/thirdparty/diy/vtkmdiy/include -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/conduit-v0.8.3/install/include -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/conduit-v0.8.3/install/include/conduit -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/vtk-m-v1.8.0/install/include/vtkm-1.8 -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/kokkos-3.6.01/install/include -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/vtk-m-v1.8.0/install/include/vtkm-1.8/vtkm/thirdparty/optionparser -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/vtk-m-v1.8.0/install/include/vtkm-1.8/vtkm/thirdparty/diy -isystem /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/vtk-m-v1.8.0/install/include/vtkm-1.8/vtkm/thirdparty/diy/vtkmdiy/include/vtkmdiy/mpi -I/opt/rocm-5.2.0/include -Wno-pass-failed -I/opt/cray/pe/mpich/8.1.16/ofi/amd/5.0/include    -O3 -DNDEBUG -fPIC -fvisibility=hidden -ffunction-sections -std=c++14 -MD -MT ascent/CMakeFiles/ascent.dir/runtimes/flow_filters/ascent_runtime_genten_filters.cpp.o -MF CMakeFiles/ascent.dir/runtimes/flow_filters/ascent_runtime_genten_filters.cpp.o.d -o CMakeFiles/ascent.dir/runtimes/flow_filters/ascent_runtime_genten_filters.cpp.o -c /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/ascent/runtimes/flow_filters/ascent_runtime_genten_filters.cpp
In file included from /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/ascent-v0.9.0-pre/src/ascent/runtimes/flow_filters/ascent_runtime_genten_filters.cpp:27:
In file included from /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/genten-gitlab/install/include/genten/Genten_ComputePrincipalKurtosisVectors.hpp:41:
In file included from /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/kokkos-3.6.01/install/include/Kokkos_Core.hpp:51:
In file included from /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/kokkos-3.6.01/install/include/Kokkos_Core_fwd.hpp:52:
In file included from /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/kokkos-3.6.01/install/include/Kokkos_Macros.hpp:110:
In file included from /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/kokkos-3.6.01/install/include/KokkosCore_Config_SetupBackend.hpp:47:
In file included from /gpfs/alpine/scratch/nic8504/csc340/2022_08_kokkos_genten_ascent/ascent-crusher-build-scripts/kokkos-3.6.01/install/include/setup/Kokkos_Setup_HIP.hpp:53:
/opt/rocm-5.2.0/include/hip/hip_runtime.h:66:2: error: ("Must define exactly one of HIP_PLATFORM_AMD or HIP_PLATFORM_NVIDIA");
#error("Must define exactly one of HIP_PLATFORM_AMD or HIP_PLATFORM_NVIDIA");
amdclang_build.txt
cray_build.txt

@nicolemarsaglia , try adding -D__HIP_PLATFORM_AMD__ to both your CFLAGS and your CXXFLAGS. This is needed so the HIP headers know you're building for the AMD runtime (rather than trying to target CUDA).
As a general FYI, if you run into issues on Crusher or Frontier you're more likely to get machine specific help using the OLCF support system at https://www.olcf.ornl.gov/for-users/getting-started/submit-ticket/

Thanks, that worked!


Add all AWS cards to OpenShift on AWS

Adds database and network AWS cards to the OpenShift on AWS overview page.
Note there is an issue with the /reports/openshift/infrastructures/aws/cost API returning the same cost, regardless of the product family. See https://github.com/project-koku/koku/issues/789
Fixes https://github.com/project-koku/koku-ui/issues/667
OpenShift on AWS:

Codecov Report

Merging #702 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #702      +/-   ##
==========================================
+ Coverage   79.92%   79.94%   +0.01%
==========================================
  Files         221      221
  Lines        3378     3380       +2
  Branches      542      542
==========================================
+ Hits         2700     2702       +2
  Misses        599      599
  Partials       79       79

Impacted Files
Coverage Δ

...tore/ocpOnAwsDashboard/ocpOnAwsDashboardReducer.ts
100% <ø> (ø)
:arrow_up:

...tore/ocpOnAwsDashboard/ocpOnAwsDashboardWidgets.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update aws-sdk to v3 in 'Use AWS:S3 As Storage' Documentation

In the documentation for using AWS:S3 As Storage, aws-sdk is at version 2.240.1, which now causes the following warning:
NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.

Please migrate your code to use AWS SDK for JavaScript (v3).
For more information, check the migration guide at https://a.co/7PzMCcy

@ThaumRystra would you love to send a PR?

I'm too dumb to get it to work right, so I am stuck on AWS 2.240 until I figure it out.
I'll submit a PR if I manage to get it working properly.

Also note that trying to use AWS SDK v3 will throw a warning that Node 14 is no longer supported from 1 May 2024, so until meteor switches to a later Node version, there will be warnings.

@jacoatvatfree true, would like to submit a PR, at least as WIP, I will help to finalize, and I'm sure community would join too. Perhaps you have a project from where you can copy-paste general changes required for v3?
If you're going to submit a PR let's postfix files with v2 and v3 so we can reference S3 integrations in the docs for all of its versions


Updating Umami should not lead to re-iimporting all my accounts

I have been using Umami v2.3.3 on Linux (Ubuntu 24.04.1 LTS, if you can call that Linux). I click the in-app link to update to v2.3.4.
Upon restart, I need to re-import all my accounts for scratch.

The issue lies with electron-builder config changes. Name was changed from umami to. umami-desktop thats why the old files could not be found after the update.


push bot fails to generate spikes

the pynn example of using the push bot fails to execute correctly.
Untill such time as this works correctly with the master branches of the tool chain, the code cannot be pulled into the system.
This issue is to keep track of the problem, whilst allwoing us to close the pull request, for metric purposes.

FIXED!


Effects for damage

It would be great to have one time effects that increase the damage of the subsequent roll for:

Sneak attack
Colossus Slayer
Hunter's Mark
Tempest Domain Divine Strike

Here two examples how I currently do it using separate features within the player's item sheets.
Special expiry is always set to DamageDealt:

Sneak Attack:

Hunter's Mark (it just adds damage, the player has to check for himself if he targets the correct target)

Tempest Divine Strike

Just read through DFreds answer on the other issue you wanted before opening this one.
If you are willing to, or already use Midi-qol and DAE there is a way to automate all these extra damage options that you mention already.
There isn't an easy straightforward add an active effect solution, but needs to add some extra damage flags to link attacker and target.
I can walk you through it if you want.

@thatlonelybugbear Feel free to put the answer in here for future reference

So for Sneak Attack and Hunter's Mark, I use the premade items that tposney has included in the Midi-QOL sample items compendium. They work as intended and also can, with a small macro change to update the actor to make sure that the timing (once per turn for Sneak Attack for instance) is 100% respected even if the Token moves or casts a spell in the same round (edge cases, but the Midi flag that these items set is not passed to the actor to be stored, so they get deleted with each such update to the token).
For Tempest Divine Strike (works similarly for all Cleric (8th level if not mistaken) features like that or Blessed Strikes optional ones) there are 2 ways.
The long and convoluted way... that will create a damage card in one line with both damage and extra damage
One more prerequisite for this to actually work is to use the Next Up's module Auto Control Next Token: ALL
This is an example for Blessed Strikes but you just need to change the damage types and attack types for the rest.

Step 1: Create the proper feature on the actor, giving it a unique name, in my example: Blessed Strikes Thodadin

Set Target and Range to Self and a duration of 1 Turn.
Add a DAE and delete the Effect icon if you don't want an icon effect to be added on the token in the start of every round.
Add a Special duration of 1Hit: Expires on the next hit by the character
Add the effects that suit the feature, in my example:

Step 2: Create a Blessed Strikes Thoradin script macro.
Either drag and drop the feature you created in Step 1 on the macros bar or create a new macro with Type: Script and input game.dnd5e.rollItemMacro("Blessed Strikes Thoradin");
Step 3: Create manually a passive effect on the token (you could also automate this with a feature on the actor that uses Transfer Effect to Actor on Item Equip, but no reason for me as you end up with 2 features with the same more or less name).

In Duration Macro Repeat: Start at each turn: Run effect macros at the start of the characters turn
In Effects macro.execute and the name of the macro you created in Step 2: "Blessed Strikes Thoradin" (with the quotes " ")

The maybe programmatically sexier way
is to make use of the flags.dnd5e.DamageBonusMacro like the ones used by the Sneak Attack and Hunter's Mark ... to be continued (have to go to work :P)!
About the Colossus Slayer (haven't been using that but still doable) its a pretty straightforward bonus damage macro
where you will need to just include a check on the target as to whether it has Full HP or not and then just trigger the extra damage.

Create a feature for Colossus Slayer, with Special activation cost and Target and Range of Self.
Give it a duration of lets say 24 Hours to make it an Active Ability
Create a DAE on it with Transfer Effect to Actor on Item Equip checked.
In the Effects add a flags.dnd5e.DamageBonusMacro with Effect Value of ItemMacro.Colossus Slayer (best to use Item Macro module as well)
In the Item Macro space on the Colossus Slayer feature you created, you can input (crude macro example)

if (!["mwak","rwak"].includes(args[0].item.data.actionType)) return {};           //only weapon attacks
if (args[0].hitTargets.length < 1) return {};                                      //need a hit to continue
token = canvas.tokens.get(args[0].tokenId);
actor = token.actor;
if (!actor || !token || args[0].hitTargets.length < 1) return {};

let target = canvas.tokens.get(args[0].hitTargets[0].id);

if (game.combat) {
    let combatTime = game.combat.round + game.combat.turn /100;
    let lastTime = getProperty(token.data.flags, "midi-qol.ColossusSlayerTime");

}

const diceMult = args[0].isCritical ? 2: 1;
if (game.combat) {
    let combatTime = game.combat.round + game.combat.turn /100;
    let lastTime = getProperty(token.data.flags, "midi-qol.ColossusSlayerTime");
    if (combatTime === lastTime) {
        MidiQOL.warn("Colossus Slayer Damage: Already used it this turn");
        return {};
    }
    if (combatTime !== lastTime) {
        setProperty(token.data.flags, "midi-qol.ColossusSlayerTime", combatTime)
    }
}
//checking hp max versus hp current value on the target and if they are not different just stop, otherwise go on to return the extra damage
let hpmax = target.actor.data.data.attributes.hp.max;
let hpval = target.actor.data.data.attributes.hp.value;
if (hpmax === hpval) return {};

return {damageRoll: `${diceMult}d8`, flavor: "Colossus Slayer damage"};

This is taken from the original Midi-QOL sample item Sneak Attack (without changes to respect the time even after token updates). So if you attack and then move or cast a spell, the midi-qol.ColossusSlayerTime flag is nuked as all these flags are rewritten on each update of the token.
You can quickly and dirty change the let lastTime = getProperty(token.data.flags, "midi-qol.ColossusSlayerTime"); to actor instead token but with other problems.
The 100% accurate way is to do an actor.setFlag, so change the
let lastTime = getProperty(token.data.flags, "midi-qol.ColossusSlayerTime"); to  await actor.setFlag('world','ColossusSlayerTime', combatTime);
and also all the rest of the entries to let lastTime = getProperty(actor.data.flags, "ColossusSlayerTime");
This flag is persistent on the actor though and so in order to make sure that at some point this flag is deleted, you can add in the macro the following:
if (!game.combat) {
await actor.unsetFlag('world','sneakAttackTime');
}

and just have the character roll an attack outside of combat :D
Phew that was a lot. Will revisit in the future...

Hi and thanks for the detailled reply.
Actually, I'm aware of those options, I implemented it in a quite similar way as you described it (I even fully automated spells like Booming Blade)... and my players didn't like it.
Such a high grade of automation often results in problems and issues within the game flow (a lot of popups, things that need to be clicked in the right order, problems when actions are interrupted through reactions, problems when all the required modules get updates,...)
That's in fact the reason why I want to use this module and not have every spell contain and transfer it's active effect directly (effects like bless, bane, faerie fire, guiding bolt,... - which are included in this module - could also be implemented directly within the spells, but that is what I don't want to do): Give the players and me as a gm a unified UI that allows the manual selection of the effects.

Yeah I think that this will work for you when DFreds finds a solution for the CE to include editable custom effects.
Just for argument's sake (feel free to ignore what is under this line, just my thoughts on the subject as I am drinking a coffee during break and bored to death by reading the same news over and over again :P)
Midi-QOL and DAE pretty much are updated to always play nice with each other.
Probably you have seen that nowadays Midi is able to intercept and handle (baby steps) interruption of the normal attack and damage workflow for reactions (spells like shield, or uncanny dodge or even bardic inspiration)
Automation is always an issue (and a developing problem of what feels like too much automation) but as the modules are evolving it is much less fuss than clicking for example, almost every round on a Sneak Attack CE and then attacking for me and my players. Even more for me as a DM that have to do that for many more creatures per round (sneak attacks, blessed strikes, marks they add up :D)
Needs some time to set up properly but now no matter what, as a DM I can't do without. Even clicking on Add Convenient Effects, find what you need and click again on it, it adds up in the end.
The latest addition of Midi being able to recognize automatically spell names that have a corresponding CE is a godsent and I have to say a HUGE thanks to both @DFreds and tposney one more time, that they took the time to work out a good co-operation between the 2 modules!
So how I have set my game up right now, has no pop-ups except the occasional

message, no need to click on anything between turns to keep using character features, a good amount of actions automatically interrupted through reactions, like shield, uncanny dodge, indomitable (midi still needs to find a way to do the same with spells) and not a noticeable higher degree of problems when the required modules get updates (thanks to the efforts again of the aforementioned developers).
Of course if you don't use midi to auto roll and auto target swith spells etc, all the things I wrote go down the drain.
Have a good day!

Jeah, you of course are right with your arguments and I also tried to use midi, dae,... in that way (in fact I provided tposney with the initial coding for the concentration automation ;-) ). Unfortunately my players don't like this kind of automation, it feels too much of a video game for them.
Convenient Effects is the ideal solution for such players, especially as soon as custom effects are possible. If you don't want to implement the suggested effects I posted in this issue, then it's fine for me - I'll just wait until custom effects can be implemented :-)

If you don't want to implement the suggested effects I posted in this issue, then it's fine for me - I'll just wait until custom effects can be implemented :-)

Just to say that I am not the owner of the module, so I have no say on this!


Output containing reserved characters isn't escaped allowing invalid JSON and XML

Some code we were running that used libxo added single quote characters to a string that ended up in a value (via a few levels of indirection) and we were surprised to find that invalid XML was produced.  It would be helpful if libxo did this for us or at a minimum least provided a simple vis(3) like function that did the escaping required for the current format mode.
In our case I ended up using a strsnvis() to blindly encode XML and JSON special characters.

Do you have some example code?  My goal is definitely to prevent generation of invalid xml/json wherever possible.
Thanks,
Phil

On Apr 20, 2015, at 4:11 PM, Brooks Davis<EMAIL_ADDRESS>wrote:
Some code we were running that used libxo added single quote characters to a string that ended up in a value (via a few levels of indirection) and we were surprised to find that invalid XML was produced. It would be helpful if libxo did this for us or at a minimum least provided a simple vis(3) like function that did the escaping required for the current format mode.
In our case I ended up using a strsnvis() to blindly encode XML and JSON special characters.
—
Reply to this email directly or view it on GitHub https://github.com/Juniper/libxo/issues/35.

I wrote some code and then realized the person who reported the issues to me was wrong.  You are escaping < > and &.  ' and " have encodings, but don't need to be encoded and I now think libxo is doing the right thing.  Sorry for the noise.

No problem.   Always happy for non-problems. ;^)
Can I ask what the app is?
Thanks,
Phil

On Apr 20, 2015, at 4:20 PM, Brooks Davis<EMAIL_ADDRESS>wrote:
I wrote some code and then realized the person who reported the issues to me was wrong. You are escaping < > and &. ' and " have encodings, but don't need to be encoded and I now think libxo is doing the right thing. Sorry for the noise.
—
Reply to this email directly or view it on GitHub https://github.com/Juniper/libxo/issues/35#issuecomment-94558583.

It's a unit test program used in our fork of FreeBSD support the CHERI processor which has hardware enforced memory safety and fast hardware supported application compartmentalization.
The test program is in github at:
https://github.com/CTSRD-CHERI/cheribsd/tree/master/bin/cheritest
Project info at http://www.cl.cam.ac.uk/research/security/ctsrd/cheri/
The SOAAP component of the project also uses libxo to produce structured output from LLVM/Clang, but I'm enough out of the loop there that I don't know exactly what it's used for.

Awesome!  Thanks…

On Apr 20, 2015, at 4:27 PM, Brooks Davis<EMAIL_ADDRESS>wrote:
It's a unit test program used in our fork of FreeBSD support the CHERI processor which has hardware enforced memory safety and fast hardware supported application compartmentalization.
The test program is in github at:
https://github.com/CTSRD-CHERI/cheribsd/tree/master/bin/cheritest https://github.com/CTSRD-CHERI/cheribsd/tree/master/bin/cheritest
Project info at http://www.cl.cam.ac.uk/research/security/ctsrd/cheri/ http://www.cl.cam.ac.uk/research/security/ctsrd/cheri/
The SOAAP component of the project also uses libxo to produce structured output from LLVM/Clang, but I'm enough out of the loop there that I don't know exactly what it's used for.
—
Reply to this email directly or view it on GitHub https://github.com/Juniper/libxo/issues/35#issuecomment-94560355.


add support for Databricks

This adds support for the Databricks CLI.
This CLI needs to know a "host" URL to the instance of Databricks you're working with, as well as your credentials that are specific to that instance. Personal access tokens are they primary way to authenticate, while username+password is possible but discouraged.
I've tested this on my own config, and both import and usage seems to work just fine 💯  Please let me know if you'd like to see more test cases or anything else before accepting.

A couple notes:

I've opted for the "Person Access Token" credentials type, despite username+password being an option. The code here handles both cases, but the label of "Personal Access Token" isn't quite right in both cases. I still think it should be as I've proposed here to encourage the recommended flow, but an alternative would be to use a more generic "Configuration" label. A second opinion here would be nice 🙂
When doing op plugin list, this plugin shows up with REQUIRED FIELDS: host. This is because while you do need to provide either a token or a username and password, neither of them can be marked as required on their own. Probably not a big deal, but resolving it can't really be done without making some changes to schema.CredentialField and related code.
The config files at the moment typically also includes a jobs-api-version = 2.x line. This is ignored by the plugin with the following reasoning:

Unlike the other config options, the CLI does not accept setting the API version with an environment variable. In order to support this the plugin would have to make use of the file-based provisioning variant, as well as add a highly specific field definition to the SDK. While possible, I don't believe it is justified given the next two points.
The version can be set with a command line flag, and is only relevant for the databricks jobs subcommand.
Perhaps most importantly, this version flag is temporary, and future versions of the CLI will default to the newer API version.

@DCKcode @SimonBarendse for sec review.

I've opted for the "Person Access Token" credentials type, despite username+password being an option. The code here handles both cases, but the label of "Personal Access Token" isn't quite right in both cases. I still think it should be as I've proposed here to encourage the recommended flow, but an alternative would be to use a more generic "Configuration" label. A second opinion here would be nice :slightly_smiling_face:

I agree that the product can be opinionated towards the desired option. This also provides the best experience for the most common case of following Databricks recommended method of authentication (more closely matching wording).
I'd like to consider if we can even go one step further and choose to not support username+password at all. This will allow us to set the personal access token as required, which has the following advantages:

op plugin init currently only prompts the user to set required fields. For this plugin, we'll definitely want to prompt for the personal access token during initialization.
Simpler documentation, as we can omit username+password specifics. So the user does not need to consider which to use. They just have the single option.
We help the user towards Databricks recommended form of authentication.

I'd indeed remove the username and password fields and make the token field required. If we want to support both credential types, then we should create separate CredentialType structs for them.

I've removed username+password and made the token field required. Let me know if there's anything else you would want changed 👍🏻


Get 500 when editing or creating new users

Hey,
since the last update from packager.io I get error code 500 in nginx when I want to create new users or editing existing ones. My gogs-version is 0.8.10-1451618975.4993ab1.jessie.
access.log
x.x.x.x - - [15/Jan/2016:18:50:41 +0000] "GET /gogs/admin/users/new HTTP/1.0" 500 0 "-" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.111 Safari/537.36"

The gogs-logs don't show anything interesting.

Works since last update from packager.io


[Rendering] MS Teams does not scroll down, when adaptive card is sent

Target Platforms
Android
SDK Version
1.5
Application Name
Microsoft Teams
Problem Description
If a new card appears in the chat, MS Teams does not scroll down. That was earlier the case and it is also the standard for any kind of messenger app.
Screenshots
https://github.com/microsoft/AdaptiveCards-Mobile/assets/6587992/3bfd77df-22d7-443c-b83c-d5f8ba6f4b37
Card JSON
It does not depend on any json, its a general issue

Sample Code Language
No response
Sample Code
No response

@ardlank I hope it's fine to mark you on that issue, as I think you are responsible in this topic? Let me know if I'm wrong and if there is someone else in charge of this kind of problems on teams for android.
We got already multiple complains from our customers using our but that they are confused by this behaviour. We could not yet find any workaround. We would be happy to provide you with any further information which would help you to fix this problem.

I just tested on the latest version. Scrolling is still broken. Are we here in the correct repository or should we report this somewhere else?

We are getting lots of feedback to this issue, when we are talking to our users. What is the state on that scrolling issue?

It is not only scroll down issue. There is also issue with horizontal scrolling. I am using table component in adaptive card, and set widths as pixels. Horizontal scroll just disappeared recently.

I still see this problem on my mobile. Is there any feedback on this? Usability is currently super bad with this bug as you are constantly scrolling within the conversation.

Adaptive Cards is now a semi private repo

Pull requests and issue creations will no longer be accepted and will be closed. Please send all issues with Adaptive Cards to Microsoft Teams docs repo (msteams-docs)
Issues will be reviewed and prioritized from there
Source code will still be available to the public.
Packages will still be posted to the public.


[BUG]Duplicate Footer Display

🐛 Bug Report
Issue: Duplicate Footer Display
Description:
The footer section appears twice on the page, creating a redundant display issue. This might be caused by the Footer.ejs component being rendered multiple times.
Steps to Reproduce:

Open the [URL where the issue occurs] (e.g., Contact Us page).
Scroll to the bottom of the page.
Observe that the footer is displayed twice.

Expected Behavior:
The footer should appear only once at the bottom of the page.
Possible Solution:

Check if `Footer' is being called twice in the main layout file.
Review conditional statements to ensure the footer isn’t rendered twice under specific conditions.

Let me know if you need further customization!
Severity
How critical is this issue?

[X] 🔴 Critical
[ ] 🟠 Major
[ ] 🟡 Minor

Screenshots

Can u assign me this task


Decide whether to compress files in the FLX based on filename extensions

Previously the FLX builder compressed assets only if they were not dynamically
generated.  This meant that the license file was not compressed.

LGTM


Strange behavior on field sorting

Situation

I am using the method IFSStore.index(String indexName, String hash, String id, String contentType, List indexFields) with several index fields, also "title"
I upload and index several new files with different titles: "my title 1", "my title 2", "my title 3", "my title 4" (I also tried inserting them in a different order: no difference)
When I query for the data with "query/search?index=documents&page=0&size=1000&sort=title&dir=ASC" then the order is as expected ("my title 1", "my title 2", "my title 3", "my title 4")
but
if I upload new files with the titles "2018-10-30 16:58:47", "2018-10-30 16:59:11", "2018-10-30", "16:19:48" then the order (after querying) is completely wrong ("2018-10-30 16:58:47", "2018-10-30 16:19:48", "2018-10-30 16:59:11")
Other behavior:
When I only use hex-strings as titles (0-9a-f), the order is also correct
When I mix the previous entries ("my title 1", "my title 2", "my title 3", "my title 4") with the other entries ("2018-10-30 16:58:47", "2018-10-30 16:19:48", "2018-10-30 16:59:11"), the order is also wrong:
("my title 1", "2018-10-30 16:58:47", "2018-10-30 16:19:48", "2018-10-30 16:59:11", "my title 2", "my title 3", "my title 4")

Hi,
Thanks for reporting this issue. After investigation, this is due to the fact that ElasticSearch uses the Standard Tokenizer (Standard Token Filter - Lower Case Token Filter - Stop Token Filter) for text field by default. So basically when you index a document { "title": "my title 1"} , the index creates three references to this document (my, title, 1) to allow full text search natively.
But in terms of sorting, it sorts all references by alphabetical order and remove duplicate after:

1
2
3
4
2018-10-30
my
title

That's how ElasticSearch works by default and this can be tweaked a little bit but not really through IPFS-Store.
As a short term solution, you can configure manually the index field mapping in ElasticSearch like this:
POST http://<IP_ADDRESS>:9200/documents/documents/_mapping
{
    "documents": {
      "properties": {
        "title": {
          "type": "text",
          "fields": {
            "raw": {
              "type":  "keyword"
            }
          }
        }
      }
    }
}

Run the query (with sort=title.raw):
GET query/search?index=documents&page=0&size=1000&sort=title.raw&dir=ASC"

In the future, I will try make IPFS-Store more configurable for this.
Thanks again for raising this issue!
Greg

As per the recent refactoring, it is now possible to pre-configure an ElasticSearch index mapping in the API in order to pre-create the index on startup with the necessary index fields mapping.
In your case, you could create a mapping file index_mapping.json like this:
{
  "mappings": {
    "_doc": {
      "properties": {
        "__hash": {
          "type": "keyword"
        },
        "__content_type": {
          "type": "keyword"
        },
        "title": {
          "type": "text",
          "fields": {
            "keyword": {
              "type": "keyword"
            }
          }
        }
      }
    }
  }
}

The index field title is indexed in two manners:

text: use for full text and approximate search
keyword: use for exact search and sorting

(__hash and __content_type are two required field by Mahuta)
Once the file setting on the server, we need to pass the following arguments to the API
-Dspring-boot.run.arguments=--mahuta.elasticsearch.host=localhost, --mahuta.elasticsearch.port=9300, --mahuta.elasticsearch.clusterName=docker-cluster, --mahuta.elasticsearch.indexConfigs={"name":"document", "map":"index_mapping"}

indexConfigs takes an array of index name / config, so the API creates accordingly these index with the config on startup.


cli: some improvements for password

This will probably fix #103, but I can't check it.

Nice work as usual my friend, thank you so much for all of these improvements!


parser, ast: accept syntax about chain, release when commit or rollback

What problem does this PR solve?
Fix #649.
This pr just make parser accept the syntax without actually handling.
What is changed and how it works?

add field CompletionType uint8for CommitStmt and RollbackStmt, which overwrites system variable completion_type within current transaction.

Check List
Tests

Unit test

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

PTAL @tangenta


Move deprecation utilities to private namespace

Splitting this off from https://github.com/modal-labs/modal-client/pull/2633. We're not very clear about whether functions in nested public modules: we should clean that up.
In particular the exception classes in modal.exception are public, but it makes sense not to pull them up into the the top-level namespace, since there's a bunch and referencing them is niche. So it seems particularly important to keep this namespace clean. That said, can't imagine any users are calling these utilities directly.
Changelog

The internal deprecation_error and deprecation_warning utilities have been moved to a private namespace

@prbot approve; big diff and want to avoid merge conflicts


asm and llvm_asm

asm is not behind a feature gate on nightly and will be stable on 1.59.0: https://doc.rust-lang.org/nightly/core/arch/macro.asm.html
llvm_asm will be removed so you should remove it as well

Hi, I have made the changes as you suggested. I don't remember why I enabled llvm_asm feature though it is not used anywhere in the codebase. (Maybe I was experimenting something and left it as it is).
Yeah about asm I am using rustc 1.59.0-nightly (532d2b14c 2021-12-03) and it throws an error if #![feature(naked_functions)] is removed. So leaving it untouched.


Module documentation

This would be really helpful for the external reader, or to anyone not particularly familiar with the codebase to have a short top-level description for each module. As for instance what we did for revaultd https://github.com/re-vault/revaultd/blob/3141908c8c1e847b2e76f9240c109734d1324599/src/daemon/control.rs#L1-L7 .
Something like a "tour" of the software.

yes
❯ tree src
src
├── main.rs
├── revaultd               <---- revaultd client
│   ├── client
│   │   ├── error.rs
│   │   └── mod.rs
│   ├── config.rs
│   ├── model.rs           <---- revaultd resource (https://github.com/re-vault/revaultd/blob/master/doc/API.md)
│   └── mod.rs
└── ui                     <---- everything concerning the User Interface
    ├── app.rs             <---- the main Iced app
    ├── color.rs
    ├── component          <---- components we try to reuse in the views, they are message agnostic, theorically
    │   ├── button.rs            the only dep is Iced. It is where we will add our custom widgets later.
    │   └── mod.rs
    ├── error.rs
    ├── font.rs
    ├── icon.rs
    ├── image.rs
    ├── message.rs         <--- message and enum structs which are send through the app states when an event occurs.
    ├── mod.rs
    ├── state              <--- state of the application like Controllers in a MVC app they manipulate the revaultd models.
    │   ├── charging.rs         They handle message and load commands.
    │   ├── cmd.rs
    │   ├── history.rs
    │   ├── installing.rs
    │   ├── manager.rs
    │   ├── mod.rs
    │   ├── stakeholder.rs
    │   └── util.rs
    └── view              <---- Structs which role is to display stuff, every core logic should be in the state mod
        ├── charging.rs
        ├── history.rs
        ├── home.rs
        ├── installing.rs
        ├── layout.rs
        ├── manager.rs
        ├── mod.rs
        ├── network.rs
        ├── sidebar.rs
        └── vault.rs


Package refresh

Create/update the package with the bluemixServiceName parameter set to messagehub so that package refresh works.

Depends on #34. Once that PR is merged, most of the changes (currently) attached to this PR will just get rebased away.

LGTM 👍


[Minor cleanup] Remove TrieNode

Codecov Report

Merging #9649 (0587e8b) into master (b2da310) will increase coverage by 0.95%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #9649      +/-   ##
============================================
+ Coverage     62.83%   63.79%   +0.95%
- Complexity     5162     5284     +122
============================================
  Files          1935     1893      -42
  Lines        103815   101769    -2046
  Branches      15758    15519     -239
============================================
- Hits          65236    64921     -315
+ Misses        33708    32086    -1622
+ Partials       4871     4762     -109

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
67.34% <ø> (+<0.01%)
:arrow_up:

unittests2
15.66% <ø> (?)

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

Impacted Files
Coverage Δ

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/MinionQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ache/pinot/server/access/AccessControlFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/SegmentReloadMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/TableDeletionMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...pinot/core/data/manager/realtime/TimerService.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 617 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove Application of Known Charges

Description
This PR removes the ability to apply 'known' charges to molecules from the BuildSmirnoffSystem.
This feature was originally required as a work around for the OpenFF toolkit not supporting specifying the charges on molecules (such as water or ions) directly. This is now well supported by library charges, and hence these feature is no longer required.
Status

[X] Ready to go

Btw, with the removal of the _generate_known_charged_molecules, the ions are no longer parametrized by the toolkit. Is there a workaround for this?
openforcefield.typing.engines.smirnoff.parameters.UnassignedMoleculeChargeException: The following molecules did not have charges assigned by any ParameterHandler in the ForceField:
[Na+]
[Cl-]

@jeff231li you now need to specify these charges in the SMIRNOFF force field directly using a LibraryCharges handler. @j-wags should be able to help with this.

Thank you, I can get it to work by adding "test_forcefields/ion_charges.offxml" to ForceField.


Support multi channels by supporting channel splitting

Support multi channels by supporting channel splitting.
Channel splitting enables creating a set of sub-channels in SPMD way. It is powerful and flexible way to create a channel, which should cover major needs of creating a sub-channel. Each rank in a group will get a channel for the group. We may extend the group semantic more to make it more flexible, but for now channel splitting is sufficient to support all use cases for StableHLO collective ops.
Each value in the groups represents a rank in the input channel and
all ranks must be specified in the groups. Each group is specified
with parentheses and comma separated. Each rank specified in a group
is also comma separated. Groups can have a different sizes but cannot
be empty. Here are some examples with 4 ranks:
"(0,1),(2,3)" - Two groups with size 2 each
"(0),(1,2,3)" - Two groups with size 1 and 3, respectively
"(0),(1),(2),(3)" - Four groups with size 1 each

Now, the channel provider does not create the unique ID. Instead, the root rank sets the ID and each participant including the root rank calls the provider function to broadcast the ID. In the MPI implementation, only color and key are necessary to split a communicator, but for other communication methods, it may need more information, so the original groups string is passed in the provider function too. It would be useful for debugging too.
The missing implementation here is the use of the input channel to split. Currently, we always split the default channel. It is marked as TODO. This needs a work to attach the MPI communicator to iree_hal_channel_t. The stablehlo always creates a group from the default channel, so that's okay for now, but this needs to be fixed to make the channel split op complete.
The code is tested with all_reduce, all_gather, and reduce_scatter with 2 and 4 processes setups.

Thanks @benvanik for the early comments! I am rebasing the changes onto your channel provider changes.
The channel split (MPI_Comm_split) to get the unique ID will go into another channel provider function, e.g., exchange_id_for_group. Then, the runtime side can just create a channel with the unique ID.
Can you point me to the variadic list example? Or we can simply follow how mhlo does using -1 for empty data. Another way is to pack group ID and rank in the group in a 64-bit (32-bit for group and 32-bit for rank). It will pack the data in binary form and do not waste space.
I can keep the generic channel creation. My preference is that removing unused features if it is not used and adding it when a necessity show up. I'd like to hear more from Calin's use case.

Rebasing is done and it passes my local tests, but I still need to discuss and resolve a few things. Switching to a draft PR.

We do not want to do that (byte buffers/etc) and shouldn't need to. Variadic arguments should be used instead.

We do not want to do that (byte buffers/etc) and shouldn't need to. Variadic arguments should be used instead.

Can you point me to an example?

command buffer push_constants
https://github.com/google/iree/blob/main/compiler/src/iree/compiler/Dialect/HAL/IR/HALOps.td/#L996
https://github.com/google/iree/blob/main/compiler/src/iree/compiler/Dialect/HAL/hal.imports.mlir/#L263-L268
https://github.com/google/iree/blob/main/runtime/src/iree/modules/hal/module.c/#L750
(CiD indicating a single i32 repeating list)
I'm happy to help handle this if you want to split the PR up or can take it on - it's something I'm familiar with and can spend cycles on it if you'd like to work on the other parts of the stack.

@benvanik I am a bit lost here because the push constants seem to be related to a dispatch and command buffer, but the collective ops have nothing to do with dispatch.
Anyway, it would be nice to have your help here. You can work on the PR and change the the HAL interface directly. If you want me to split the PR in a preferred form, let me know. I can clean up some stuffs for you. Thanks!

Thanks Ben!

This is fused into Ben's recent PRs. Thanks!


[MM-39119] Auto complete categories in categorize playbook run

Summary
When configuring the optional sidebar category into which to categorize a new playbook run channel, it uses react-select Creatable component to expose user existing categories and also allow creating non existing category as before.
Before

After

Ticket Link
https://github.com/mattermost/mattermost-server/issues/18562
https://mattermost.atlassian.net/browse/MM-39119
Checklist

[ ] Telemetry updated
[ ] Gated by experimental feature flag
[ ] Unit tests updated

Thank you for review. I'll get home and look onto requested changes

Thanks for participating in Hacktoberfest! You can claim your sticker set here: https://get.printfection.com/hacktober21/4144583267 @dunak-debug

Hi, @dunak-debug, is there anything we can do to help you with this?

Hi, sorry I had corona and I couldnt really get into work on computer 😏 I feel bad for this big delay on this PR and as soon as I arrive home I will push changes and we can resolve any other reviews. Is that ok? 🙂

Oh please, don't worry at all! Take all the time you need and get well soon, your health is way more important than any PR. Ping us when you're ready, there's no rush at all. Stay safe :)

I did some changes. Please @agarciamontoro, take a look and let me know

Thank you, useful comments ;) Didn't know about intl and menuPlacement.. Updating it

Could you merge with master @dunak-debug ? Also please run make i18n-extract and commit en.json file changes <3

Also I wanted to ask where could I push tests for this PR.  It's likely a part of my school assignment :D to do these tests... So if I can push it inside this PR or make a new one with the same ticket number?

@dunak-debug you can push Tests here.
Squash will happen automatically on merge

@dunak-debug you can push Tests here. Squash will happen automatically on merge

So if I am right these specific tests should go inside https://github.com/mattermost/mattermost-plugin-playbooks/blob/master/tests-e2e/cypress/integration/playbooks/edit_spec.js

So if I am right these specific tests should go inside https://github.com/mattermost/mattermost-plugin-playbooks/blob/master/tests-e2e/cypress/integration/playbooks/edit_spec.js

@dunak-debug, that would a be a great place, yes! Thank you :)

I am experiencing an issue when I run tests on is disabled in a new playbook
cy.request() failed on:

http://localhost:8065/plugins/playbooks/api/v0/playbooks

The response we received from your web server was:

  > 403: Forbidden

@dunak-debug sometimes tests are a bit flaky. Just fix the lint error, tests should be ok.

@dunak-debug sometimes tests are a bit flaky. Just fix the lint error, tests should be ok.

I did but I get still this 403 error.
 POST /plugins/playbooks/api/v0/playbooks
                        Status: 403 - Forbidden
                        Response body: {
                          "error": "Not authorized"
                        }
                        ```

I saw warning.. Not authorized; internal details: restricting playbook editing to specific users is not available with your current subscription: not covered by current server license This may be the issue

Not sure why CI wants from me to extract i18n. Maybe somebody forgot to do?

I saw warning.. Not authorized; internal details: restricting playbook editing to specific users is not available with your current subscription: not covered by current server license This may be the issue

Do you have the testing and developer modes enabled? That should fix it

Not sure why CI wants from me to extract i18n. Maybe somebody forgot to do?

Probaly the merge added some conflicting change to en.json. Run make i18n-extract and commit whatever comes out of that, it will fix the errors.

Do you have the testing and developer modes enabled? That should fix it

I have, still 403. ✘  POST /plugins/playbooks/api/v0/playbooks
I am really confused 😕

Oh, I think why this might be. Can you try opening cypress like this?
CYPRESS_developerMode=true npm run cypress:open

Also, first double-check that the testing and developer modes are indeed enabled, running the Cypress tests without the env var might have overwritten that at some point.

Cypress developerMode fixed 🎊

Cypress developerMode fixed :tada:

Yay! Sorry, should have warned you about that, but it's a relatively new thing and I forgot!

Hope I covered all possible cases 👍 If you can think of any I forgot let me know :)

I'll get to this tomorrow morning, @dunak-debug, sorry for the late reply! Jesse also added Cass to help review the E2E tests :tada:

Thank you 👍 I am happy


[kube-enforcer] Unable to set/inherit airgapped settings (proxy and image repository) on trivy

With version 2022.4.29 you migrated from Starboard to Trivy-Operator.
The OSS trivy-operator offers the ability to configure every setting of the operator and trivy itself:
https://github.com/aquasecurity/trivy-operator/blob/main/deploy/helm/templates/configmaps/trivy.yaml
Unfortunately the embedded trivy-operator here doesn't support customization.
We are in a airgapped environment where connectivity to the internet is only possible via a forward proxy.
The params

trivy.repository
trivy.dbRepository

are hardcoded and the params

trivy.httpProxy
trivy.httpsProxy
trivy.noProxy
trivy.serverInsecure
trivy.sslCertDir

are missing completely.
This way I assume we cannot start trivy scans as the images and the db cannot be pulled from the operator.

Hi @mkilchhofer
Sorry for the delay in the response.
Today, integration of Trivy-operator inside commercial offering only utilizes the feature of ConfigAuditing from Trivy-operator. And hence other options are not available in this embeded trivy-operator.
Any reason to use workload image scan from trivy-operator, and not from commercial flow of workload image scan via aqua scanner.
Today, if we enable trivy-operator to perform workload image scan then result will only be stored in CRD and it will not be surfaced on Aqua UI. If you are interested to use this feature from trivy-operator then we can include these options in helm values.
If you need more clarity then feel free to open support ticket on aqua support portal.
Regards
Devendra
cc @semyonmor @adishaull


Make Begin[Route]Form include antiforgery

@dougbu - any other feedback on this PR?

:shipit: w/ one doc comment phrase that I commented on in a comment.

Also, why are the CI builds failing?

Also, why are the CI builds failing?

Everyone knows that I don't pay attention to travis. I ignore him, and he fails my builds 100% of the time. We have a pretty good (albeit unconventional) working relationship.


Feature: Fine-tuned Models in Bedrock

Can we add support of fine-tuned models?
The following code snippet could be added in list_bedrock_models():
        response_custom_models = bedrock.list_custom_models()
        bedrock_custom_models = response_custom_models.get("modelSummaries", [])
        custom_models = [
            {
                "provider": "bedrock",
                "modelId": model["modelName"],
                "streaming": False,
                "type": "text-generation",
            }
            for model in bedrock_custom_models
            if (
                model["modelName"].startswith("<<fine-tuned-model-prefix>>")
            )
        ]
        models.extend(custom_models)

In addition, it should include an error handler for API requests.

With the release of v3.0.0 we also list custom models from Bedrock


Update SLE reader to newest data version and other measurement configurations

[x] Move SPECS readers into subfolder
[ ] Update parsing of binary data to read in Prodigy version > 4.63
[ ] Allow for different measurement configurations (need to make parsing of XML experimental schedule more robust)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
48

Relevant Lines:
48

💛  - Coveralls


EADDRINUSE - Sails 0.11

Hi. I installed sails js 0.11 yesterday. Since then I have tried re-installing it multiple times. But I get error Error: listen EADDRINUSE
at exports._errnoException (util.js:746:11)
at Server._listen2 (net.js:1156:14)
at listen (net.js:1182:10)
at Server.listen (net.js:1267:5)
at Array.async.auto.start (/usr/local/lib/node_modules/sails/lib/hooks/http/start.js:29:35)
at /usr/local/lib/node_modules/sails/node_modules/async/lib/async.js:484:38
at _each (/usr/local/lib/node_modules/sails/node_modules/async/lib/async.js:46:13)
at Object.async.auto (/usr/local/lib/node_modules/sails/node_modules/async/lib/async.js:455:9)
at Sails.startServer (/usr/local/lib/node_modules/sails/lib/hooks/http/start.js:16:11)
at Sails.emit (events.js:104:17)
at Sails.emitter.emit (/usr/local/lib/node_modules/sails/lib/app/private/after.js:50:11)
at afterBootstrap (/usr/local/lib/node_modules/sails/lib/app/private/initialize.js:56:11)
at bootstrapDone (/usr/local/lib/node_modules/sails/lib/app/private/bootstrap.js:51:14)
at Object.module.exports.bootstrap (/Users/rishabh/Desktop/sails/config/bootstrap.js:16:3)
at Sails.runBootstrap (/usr/local/lib/node_modules/sails/lib/app/private/bootstrap.js:44:25)
at Sails.bound [as runBootstrap] (/usr/local/lib/node_modules/sails/node_modules/lodash/dist/lodash.js:729:21)
I have to run killall -9 node everytime I need to do sails lift. How can I solve that?

@narangrishab I'm not sure why you have to run killall... So short check list:

What OS?
How do you start your sails application?
What's the output of (assuming you're on some unix system, and you're using the default port 1337) lsof -i :1337?

@RWOverdijk

Mac Yosemite.
sails lift to start the application.

@narangrishab You're already running things with node... Try sudo killall node -9 and shut down chrome. Then do sails lift. From there on out, keep track of what's running on which port. As to shutting down sails, use ctrl+c. I've never seen that not work before so you're probably doing something wrong.
Are you using ctrl+z? What happens when you type jobs in your terminal? If so, you're perhaps just suspending processes which would make sense. Or are you perhaps using screen? There are too manu possibilities.

When I type jobs I get this
jobs
[2]+  Stopped                 sails lift
What can I do to override this behaviour? Will changing ports help? Why would node interfere with this anyway? @RWOverdijk

@narangrishab Ok. Small intro to jobs:

Unix has jobs.
You can suspend a job (anything running) by typing ctrl+z
You can recover a job by typing either fg (bring back to front) or bg (to run in background).
You can call the jobs command to see your suspended jobs.

Take a look at: http://superuser.com/questions/262942/whats-different-between-ctrlz-and-ctrlc-in-unix-command-line

@RWOverdijk  Would changing the localhost:1337 to some other port fix the problem because I am not very comfortable with suspending jobs?

@narangrishab Seriously, read it and learn what it's about. Make sure you get comfortable with it, or you'll have the same problem with a different port in the future. It's common knowledge, and important to have in your utility belt :)

@RWOverdijk  alright. Thanks man :)

Thanks for your help @RWOverdijk!


Improvement in Color of The Address and it's detail

Is there an existing issue for this?
I have searched the existing issues
What happened?
The address and its details in the same color do not look good. I will change its color and make it appear in a better way.
Record:

I agree to follow this project's Code of Conduct
I'm a GSSOC'24 contributor
I want to work on this issue.

@anuragverma108, Please assign me this task.

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Updated validation, moved it to shared since used by vpn, other code …

Using db rating validation from shared istio project.
Updated README to reflect this.
Renamed so variables.

/lgtm


--list-snapshots returns html page with error

Command: swiftenv install --list-snapshots
Actual result: html page with error
Expected result: List of snapshots
Error HTTP status code: 520

This is an upstream issue with our hosting provider (https://status.heroku.com), hopefully should be sorted shortly.

Heroku Platform is stable now and this is working.


Support calling setup() from Vimscript?

Neovim supports calling Lua functions, using syntax like call v:lua.require'neocord'.setup({}) (see v:lua-call). This should provide a way for VimL init-script users to call neocord's setup function.
However, calling setup in this way currently throws an error:
Error detected while processing /home/XXX/.config/nvim/init.vim:
line  X:
E5101: Cannot convert given lua type

This seems to be the result of setup returning a Lua object reference, which Neovim cannot convert back to a native VimL type.
While I realize that this plugin is written entirely in Lua (and therefore probably expects users' init scripts to also be in Lua), it would be nice if we had a version of the setup function which were compatible with this calling format (i.e. one that returned nil).
If this is in scope, I'd certainly be willing to PR it, though that should be pretty trivial.

If this is in scope, I'd certainly be willing to PR it, though that should be pretty trivial.

if you would be will I would love to have the pr!

Since #3 has been merged, this is now resolved. See the Readme for usage details!

Thanks for your work! :heart:


Revert "replace oauth endpoint url (#1133)"

This reverts commit ecda503e419858021670f39c17b5fda99edbfc0e.
The initial commit is trying to fix a problem that only one customer had, however the change might introduced a regression to most TFS on-prem customer.
I would like to revert this change for TFS 2018 RTM, and add the right fix for that one customer in next agent release from Master branch.

@ericsciple @bryanmacfarlane

makes sense


An argument to specify the exact output directory

There is --storage_path but dice-embeddings creates a (unpredictably named) subdirectory to store the experiment results. It would help in automation scenarios to specify exactly where output files (model.pt and others) would be created.

Great. The potential fixes are being merged.


playInBackground not working on RN 0.42.0

playInBackground not working on RN 0.42.0
I've resolved to fixing this by adding the following snippet to the end of didFinishLaunchingWithOptions method under AppDelegate.m.
  AVAudioSession *audioSession = [AVAudioSession sharedInstance];
  NSError *setCategoryError = nil;
  BOOL success = [audioSession setCategory:AVAudioSessionCategoryPlayback error:&setCategoryError];
  if (!success) { /* handle the error condition */ }
  NSError *activationError = nil;
  success = [audioSession setActive:YES error:&activationError];
  if (!success) { /* handle the error condition */ }

@a-koka Thanks for opening this issue! I was driving myself crazy wondering if I was implementing things wrong.