evaluate approaches to mount large Grafana dashboard files

#562 introduced Grafana dashboard JSON files, which increased the output of conduit install from 19kb to 92kb.
Consider alternate approaches that would keep conduit install output small, but still allow for custom dashboards in Grafana. A few approaches:

custom Grafana Docker image
a sidecar process that creates Grafana's ConfigMap
host the dashboard files via a server, perhaps the web process
https://github.com/coreos/prometheus-operator/issues/535
https://github.com/coreos/prometheus-operator/tree/master/contrib/kube-prometheus/hack/grafana-dashboards-configmap-generator

At a glance, hosting the dashboard files in the web process seems really appealing to me because:

It doesn't clutter up install output
It's still reasonably transparent, anyone can curl web to see the dashboard json
The dashboards conceptually belong to web
Serving each dashboard json on its own URI path feels like a nice way to keep things organized.


Fix transaction details avatar size and color

Currently the avatars displayed in the transaction details does not match the designs. What needs to be fixed:

Designed avatar size is 48 but developed is 56.
The avatar with geometrical shapes and the avatar with custom generated faces has additional padding which makes the image smaller.
Additional change: the avatar borders now matches the avatar primary color so it looks the same as the one displayed in the account details home page (@onvisions).

Initial design:

Current version:

Fixed version:

The avatar now matches this header design:

Approved on my side with the suggested color change of the border.

LGTM


I need an advice on customizing a MiR platform.

Hello!
First of all, I am impressed what you have done with MiR100.
I would like to get some advice about building a robot system, If you don't mind.
I am trying to control MiR mobile robot in a custom way and all I got is a REST API documentation.
When I look into it, I cannnot find any GET or POST Encoder, IMU, Laserscan, velocity or something.
How is it possible to make it SLAM or navigating with REST API?
I would like to know how you make a bridge on the mobile robot(which has a driving unit black-boxed ) to your ROS package with REST API.
Thank you!

I'm sorry this issue got lost and I didn't reply to it. I'm closing it now due to inactivity. Feel free to reopen (or add a new one) if you want to add something.


Expose kafka with nodeport

fixes: https://github.com/stolostron/backlog/issues/23767
Signed-off-by: myan<EMAIL_ADDRESS>

/cc @clyang82 @morvencao


WIP fix(app): test cache issues

related to #4546 #4283

@pull-dog down

closed in favor of two separate PRs


scrape the number of seasons and episode details for each season.

Problem

scrape the number of season of a TV Show from sflix.is

scrape the Episode and the Episode Details  of a TV Show from sflix.is

Possible Solution
Structure of the response
{
  "result": {...},
  "season": {
    "0": [
      {
        "id": "random-id-1",
        "episodeNumber": 1,
        "episodeName": "random-name-1"
      },
      {
        "id": "random-id-2",
        "episodeNumber": 2,
        "episodeName": "random-name-2"
      }
    ],
    "1": [
      {
        "id": "random-id-1",
        "episodeNumber": 1,
        "episodeName": "random-name-1"
      },
      {
        "id": "random-id-2",
        "episodeNumber": 2,
        "episodeName": "random-name-2"
      }
    ]
  }
}

can I work on this issue ?

Ok, Sure


Typescript error

I ma getting a type error with
new L.GPX(route.url, {         async: true,         marker_options: {             startIconUrl: '',             endIconUrl: '',             shadowUrl: ''         }     })
The error is
Property 'GPX' does not exist on type 'typeof import("c:/Users/quilk/source/repos/ridehub/node_modules/@types/leaflet/index")'.ts(2339)
I see the line import * as L from "leaflet"; in the file index.d.ts for leaflet-gpx , but the editor is telling me that 'L' is declared but its value is never read.

@quilkin Did you also include @types/leaflet-gpx in your dependencies?

It's in  devDependencies of package.json but not in dependencies (as are several other type defs).
I've moved on a bit from there - I needed to get a list of coords from your package so I have forked it at https://github.com/quilkin/leaflet-gpx  to add a `get_coords()method. This will no doubt complicate a solution for typing errors.

I never implemented a get_coords() method because you can already get it from the polyline layer that gets created (with the L.Polyline.getLatLngs() method).

Thanks - but how do I get a handle on the Polyline object when using new L.GPX() ? I've now worked around by parsing the GPX data separately to get the lat/lng array, but it seems a shame because you are already parsing it in your code. Anyway I have solved the typing issue now, thanks.

You can get it from Leaflet's layers, or more easily by adding an event listener on the addline event:
https://github.com/mpetazzoni/leaflet-gpx/blob/main/gpx.js#L575

Thanks, but I  don't know how to do either of those! I tried adding addEventListener('addline',routeLineAddHandler) but the handler isn't called.

@quilkin You need to pass the async: true option to the L.GPX constructor, otherwise the parsing of the GPX happens synchronously in the constructor and therefore before you have a chance to attach the event listener. I just checked with an example locally and it works as intended, and you can get access to the latlngs with:
new L.GPX(url, async: true).on('addline', (e) => { console.log(e.line._latlngs); });

I'll make that requirement more clear in the README.


Document required linker flags

When using BOOST_TEST_DYN_LINK, the linker flag -lboost_unit_test_framework is not sufficient to successfully compile. Apparently there are more libraries that must be linked. For instance, when linking only with -lboost_unit_test_framework, I get the error:
Undefined symbols for architecture x86_64:
  "boost::unit_test::unit_test_log_t::instance()", referenced from:
      ___cxx_global_var_init in lanczos_smoothing_test-05d504.o
  "boost::unit_test::decorator::collector_t::instance()", referenced from:
      ___cxx_global_var_init.3 in lanczos_smoothing_test-05d504.o
  "boost::unit_test::ut_detail::auto_test_unit_registrar::auto_test_unit_registrar(boost::unit_test::test_case*, boost::unit_test::decorator::collector_t&, unsigned long)", referenced from:
      ___cxx_global_var_init.3 in lanczos_smoothing_test-05d504.o
ld: symbol(s) not found for architecture x86_64
clang: fatal error: linker command failed with exit code 1 (use -v to see invocation)

Hi,
Thanks for the report. This might be related to the various issues that popped out when the visibility defaults have been changed on the build system.
Would it be possible for you to check if the branch doc/various-updates-cleanups is working as expected?
Thanks!

Thanks for the quick feedback, you did a lot of work!
The test test/example/external_main_example_2.cpp can work only for the shared version, so I would say this is normal. The CmakeLists.txt is for my own devs, so I am not expecting that it would work beyond simple use cases :) In particular, this is not mimicking the released version of boost.
Now that you are on this particular branch, what about compiling boost.test and then compile the unit test module you reported as failing?
I use this from the root folder (for OSX):
export current_version="1_70_0"
export build_folder=boost_${current_version}

./bootstrap.sh
./b2 \
  cflags="-mmacosx-version-min=10.10" \
  cxxflags="-std=c++11 -stdlib=libc++ -mmacosx-version-min=10.10" \
  mflags="-mmacosx-version-min=10.10" \
  mmflags="-mmacosx-version-min=10.10" \
  linkflags="-stdlib=libc++ -mmacosx-version-min=10.10" \
  --prefix=`pwd`/../${build_folder} \
  --without-log \
  --layout=versioned \
  install

Then, once compiled, you may try to recompile your own project againts boost.1.70 and report there.
Thanks!

Hi,
I have used your suggested syntax and successfully compiled, linked, and ran on top of your doc/various-updates-cleanups branch. So it appears that this issue has already been fixed, but the fixes didn't make it into 1.69? (Or maybe I'm still screwing up?) In any case, I'm satisfied with this outcome.

Thanks for the quick feedback. The changes will be there for 1.70 (merged this morning), there were many pending issues that couldn't solve in time for 1.69, sorry about that.
Closing.

Cool, thanks for your efforts. Looks like boost.test is really improving quickly!


WIP: building sphinx docs

to-dos:

[ ] update API & FAQ
[ ] point index.html to README
[ ] set up autopush to gh-pages branch

mostly complete, two issues:

tutorial is not populating, and I don't know how to get it to read the README.md file like in fooof
some of the functions are not being recognized in API, not sure if you guys ran up to this before

other than that, should be okay to merge. I've already pushed the site to gh-pages branch

apparently function API fixed itself.
I remembered that the tutorials have to be downloaded as .py files and manually converted, and the tutorials still need to be updated. I've included the spectral tutorial as a working example.
otherwise, this is good to merge.
@TomDonoghue how do you get sphinx to read the tutorial/README.md instead of tutorial/README.txt?

I just made some changes to your API list to make it a bit clearer / organized (in my opinion).
After that, it's good to merge to me


Release 1.0.2

1.0.2 updates the image aliases used in 1.0.1 that did not bootstrap properly.

updated the aliases again, bumped versions, and tagged.


POST /study/{id}/attach

Successful response looks like:

{"study_attach":{"id":"1","main_entity_type":"task","linked_entities":"102"}}%

🪲 Users who do not own the study can attach (and detach) entities from it. This seems like an oversight (and otherwise I think it is a terrible decision).

The restriction on not attaching to legacy studies already exists in the PHP implementation. In the Python implementation it is propagated by the fact that legacy studies can not be loaded at all.

🐛  Attaching entities to a study that already exists in PHP just gave me the error

{"error":{"code":"1045","message":"Problem attaching entities. Please ensure to only attach entities that exist"}}%

which is incorrect. It seems that this is thrown on any failure while inserting the rows (in this case, it fails because of a duplicate primary key).

#135 made it possible to attach tasks to suites. runs are not supported because they require a test database update.

🪲 Users who do not own the study can attach (and detach) entities from it. This seems like an oversight (and otherwise I think it is a terrible decision).

Confirmed by Jan.


maintainers/teams: add aws team

Description of changes
With the growing list of AWS related packages and modules, I propose adding a aws team to:

have a dedicated and collectively responsible group of the AWS packages collection
keep the meta.maintainers consistent
allow interested maintainers to be informed of AWS package updates
make it easier to get help with creating AWS packages

Comment on this PR if you'd like to be added to this team.
What do you think?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2552


fdsan: attempted to close file descriptor 81, expected to be unowned, actually owned by ParcelFileDescriptor 0xebcfa4c

I am getting the following when trying to decode a TIFF file:
2022-08-11 17:47:41.526 9426-9426/com.therealbluepandabear.pixapencil A/libc: fdsan: attempted to close file descriptor 81, expected to be unowned, actually owned by ParcelFileDescriptor 0xebcfa4c
2022-08-11 17:47:41.526 9426-9426/com.therealbluepandabear.pixapencil A/libc: Fatal signal 6 (SIGABRT), code -1 (SI_QUEUE) in tid 9426 (bear.pixapencil), pid 9426 (bear.pixapencil)

Code:
TiffBitmapFactory.decodeFileDescriptor(parcelFileDescriptor.fd, options)

Check is this FileDescriptor not opened in one more place.

Same thing happens to me.
Occurs on android 11 and later.
It seems that the reason is that it is closed in the destructor of NativeDecoder.
NativeDecoder::~NativeDecoder()
{
LOGI("Destructor");
if (image) {
TIFFClose(image);
image = NULL;
}

Is there any way to fix it?

This is clearly a remnant of decodePath, so if you specify the filedescriptor of parcelFileDescriptor, deleting it without permission is a bug, so I would like you to fix it.
I don't know the impact on others, but if it's only decoding, it can be avoided by using the NDK that ndk-builds the one that commented out "TIFFClose(image)" from the destructor of NativeDecoder, but it's a little troublesome.

Deleted closing of IMAGE in native if it was oppened from FD.
Please try <IP_ADDRESS> build

@therealbluepandabear @adachi-hideki Deleted closing of IMAGE in native if it was oppened from FD. Please try <IP_ADDRESS> build

What is <IP_ADDRESS> build?

I quit android dev, but I trust that it's probably fixed, this can be closed.


Math Magician PR

In this Milestone, I implemented the following,

Make sure that there are no linter errors.
Documented the project in a professional way
Use create-react-app to setup a react project
ensure it ran in my browser

Hi @Ezzywealth,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
1. You have implemented the wrong workflow. You should request a merge to develop branch, not the main branch.
2. Kindly fix your linter error.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


add test case for #9522

Required for all non-trivial PRs

[x] Sign CLA (if not already signed)

@datamuc we could merge this if you wanted us to. You would have to fix the conflict on the test file. Thanks.


WIP: Test against testnet

Only last commit needed here. Should be rebased.

Codecov Report

Merging #650 (ad20736) into dev (29ddedf) will increase coverage by 12.10%.
The diff coverage is 93.98%.

:exclamation: Current head ad20736 differs from pull request most recent head 24be4c8. Consider uploading reports for the commit 24be4c8 to get more accurate results

@@             Coverage Diff             @@
##              dev     #650       +/-   ##
===========================================
+ Coverage   67.68%   79.78%   +12.10%
===========================================
  Files          36       47       +11
  Lines        6882     8866     +1984
===========================================
+ Hits         4658     7074     +2416
+ Misses       2224     1792      -432

Impacted Files
Coverage Δ

node/src/chain_spec.rs
0.00% <0.00%> (ø)

node/src/rpc.rs
0.00% <0.00%> (ø)

node/src/service.rs
0.00% <0.00%> (ø)

pallets/creditcoin/src/helpers/external_address.rs
98.05% <ø> (-0.02%)
:arrow_down:

pallets/creditcoin/src/migrations/mod.rs
0.00% <0.00%> (ø)

pallets/creditcoin/src/ocw/tasks/collect_coins.rs
98.96% <ø> (ø)

pallets/creditcoin/src/ocw/tasks/tests.rs
100.00% <ø> (ø)

...allets/creditcoin/src/ocw/tasks/verify_transfer.rs
92.59% <ø> (ø)

pallets/creditcoin/src/ocw/tests.rs
98.81% <ø> (+0.49%)
:arrow_up:

pallets/creditcoin/src/tests.rs
99.21% <ø> (-0.02%)
:arrow_down:

... and 55 more

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


Add available text editor/IDE integrations to the README file

flycheck-pbxproj is an Emacs package that checks the syntax of Xcode projects using Kin.

flycheck looks really promising! Thank you very much for your contribution and welcome to our README 😄


Doesn’t seem to support ISO? I want to install manjaro, or where is the tar.gz package needed for manjaro installation?

I downloaded an x64 iso package from manjaro.org, intending to install it on wsl2, but I tried to install the ISO file, the command is as follows:
.\LxRunOffline.exe install -n Manjaro -f'E:/manjaro-kde-21.0.7-210614-linux510.iso' -d'D:\WslDisk\Manjaro' -r root.x86_64
LxRunOffline does not support ISO installation, can there be a new version of tar.gz from Manjaro? I need a corresponding tar.gz download address, thank you very much
Looking forward to your reply

Yes ISO is not and will not be supported. It seems that Manjaro doesn't release rootfs tarballs so it's impossible to install it directly.


Chart option interaction not customizable

You cannot set the interaction option (c3js.org/reference.html#interaction-enabled). Haven't found the time to dive into the code and create a pull request.

Hey @belens, with the upcoming 0.3.0 release #55 of  angular-chart all c3 options will be included - please wait until the documentation is ready.

The 0.3.0 release is out, angular-chart now supports the full C3 API. Have a look at this graFiddle which shows how to disable all interactions in angular-chart.
Please check the upgrade guide for upgrading from 0.2.x to 0.3.0.


I keep getting the error "no such image hilschernetpi/netpi-nodered-fieldbus" when creating a container. What could be the problem?

Question:
How can I deploy Portainer on... ?

On a linux machine using Portainer 1.23.1 I was able to pull this image and deploy a container using it without seeing your error. So I guess it could be an issue with networking of the host, or the fact that Portainer isn't tested on raspberry PI so there could potentially be a bug.
Can you try pull this image through the Portainer images view and also try pulling it on the host that Portainer is on. Also if you are not already on Portainer 1.23.1 I would make sure you are before trying again (as there were recent improvements/fixes to the image pulling functionality)
I will also need more info on this issue, can you please fill out the below bug report template as a comment or update the issue description with it?

Bug description
A clear and concise description of what the bug is.
Expected behavior
A clear and concise description of what you expected to happen.
Portainer Logs
Provide the logs of your Portainer container or Service.
You can see how here
Steps to reproduce the issue:

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

Technical details:

Portainer version:
Docker version (managed by Portainer):
Platform (windows/linux):
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
Browser:

Additional context
Add any other context about the problem here.


Pull from private repo by using docker-registry info

How could I pull private repo without ssh key but username password
Eg:
<EMAIL_ADDRESS>I tried and got errors: Command failed: parse<EMAIL_ADDRESS>

Also, it would be great if we can use docker-registry info from k8s secret

Could you please send your command as a whole (without classified info) please? This is actually exactly how I am using the command to get data from private repo without issues.

I don't get this one, why would you need to do so? What is your use-case?

Here is the simple example manifest

apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: git2kube-watcher
  namespace: config
  labels:
    app: git2kube-watcher
spec:
  replicas: 1
  selector:
    matchLabels:
      app: git2kube-watcher
  template:
    metadata:
      labels:
        app: git2kube-watcher
    spec:
      serviceAccountName: git2kube-watcher
      containers:
        - name: git2kube-prometheus-rules
          image: wanderadock/git2kube
          args:
          - 'watch'
          - 'configmap'
          - '--configmap=config'
          - '--namespace=config'
          - '--branch=master'
          -<EMAIL_ADDRESS>          - '--interval=30'
          livenessProbe:
            exec:
              command:
              - /bin/sh
              - -c
              - "grep -Fxq OK /tmp/health"
            initialDelaySeconds: 10
            periodSeconds: 35
            failureThreshold: 3
          readinessProbe:
            exec:
              command:
              - /bin/sh
              - -c
              - "grep -Fxq OK /tmp/health"
            initialDelaySeconds: 10
            periodSeconds: 10
          resources:
            requests:
              cpu: 100m
              memory: 32Mi
            limits:
              cpu: 200m
              memory: 64Mi

Error log:
Error: parse<EMAIL_ADDRESS>net/url: invalid userinfo
Usage:
  git2kube watch configmap [flags]

Flags:
      --annotation strings   annotation to add to K8s ConfigMap (format NAME=VALUE)
  -m, --configmap string     name for the resulting ConfigMap
  -h, --help                 help for configmap
  -k, --kubeconfig           true if locally stored ~/.kube/config should be used, InCluster config will be used if false (options: true|false) (default: false)
      --label strings        label to add to K8s ConfigMap (format NAME=VALUE)
      --merge-type string    how to merge ConfigMap data whether to also delete missing values or just upsert new (options: delete|upsert) (default "delete")
  -n, --namespace string     target namespace for the resulting ConfigMap (default "default")

Global Flags:
  -b, --branch string             branch name to pull (default "master")
  -c, --cache-folder string       destination on filesystem where cache of repository will be stored (default "/tmp/git2kube/data/")
      --exclude strings           regex that if is a match excludes the file from the upload, example: '*.yaml' or 'folder/*' if you want to match a folder (default [^\..*])
  -g, --git string                git repository address, either http(s) or ssh protocol has to be specified
      --healthcheck-file string   path to file where each refresh writes if it was successful or not, useful for K8s liveness/readiness probe
      --include strings           regex that if is a match includes the file in the upload, example: '*.yaml' or 'folder/*' if you want to match a folder (default [.*])
  -i, --interval int              interval in seconds in which to try refreshing ConfigMap from git (default 10)
  -l, --log-level string          command log level (options: [panic fatal error warning info debug]) (default "info")
  -p, --ssh-key string            path to the SSH private key (git repository address should be 'git@<address>', example: git@github.com:WanderaOrg/git2kube.git)

time="2018-09-07T05:19:30Z" level=error msg="Command failed: parse<EMAIL_ADDRESS>net/url: invalid userinfo"

It can be because pwd contains special characters I suppose

Just and idea, as we can use same credential info both git repo and registry for gitlab. That's would be nice if we can pick up secret info which is alredy stored in k8s docker-registry

Yeah I actually thought so. Could you try using % encoding? As for the secrets you could use classic Opaque secret and pass it as env variables, while interesting idea I don't think it really fits into concept using docker-registry secret.

One more thing, if the config directory structure like this
├── config
│   ├── foo
│   │   ├── bar
│   │   │   ├── conf.ini

Could you support to create configmap with the name auto formatted form a directory recursively
Output:
config_foo_bar_conf.ini:  15 bytes

k8s doesn't support that, but git2kube have a choice to make it more convenience

Thank you


crashlytic to support uploading both apk and ipa at the same time

Feature Request
Motivation Behind Feature
To support the grow of Flutter framework, I need/wish to upload both Android and iOS app in the same lane.
Feature Description
Currently the Crashlytics allows taking both apk_path and ipa_path value at the same time, but actual setup depends on platform name (code from here: https://github.com/fastlane/fastlane/blob/6cf7c2afcd5b6532257ff23b797bd098a337c67b/fastlane/lib/fastlane/actions/crashlytics.rb)
So if I call crashlytic like this
crashlytics(     api_token: "{{MY_TOKEN}}",     build_secret: "{{MY_BUILD_SECRET"",     apk_path: "{{path/to/apk/file}}",     ipa_path: "{{path/to/ipa/file}}"   )
Then both files will be uploaded to Crashlytics after this call, as long as token and build secret match.
Alternatives or Workarounds
Currently I have to use 2 fastlane, 1 for android and 1 for ios, and run 2 lanes when I wanna upload both files to Crashlytics.
flutter/android/fastlane/Fastfile <=== to upload apk file to Crashlytics
flutter/ios/fastlane/Fastfile <=== to upload ipa file to Crashlytics

Is there any plan for this issue?

Have you tried the workaround I provided above?
I also suggested two different possible implementations, do you have an opinion on which of those would be better?


feat(structured-list): extend component from carbon-web-components

Related Ticket(s)
closes #8388
Description
Adds structured list element to carbon-for-ibm-dotcom.
Changelog
New

dds-structured-list

Remaining Tasks

[ ] Match styles to functional specs
[ ] Port WC to React

This PR replaces #8469

@andy-blum Do you mind if I adjust this PR to "Ready for Review" or do you still have other items that you'd like to push up before the larger group start to take a look?

@andy-blum Is this PR ready for another code review or is it pending design and functionality review from @oliviaflory, @jwitkowski79, and others?

Per the meeting yesterday with @jwitkowski79 @oliviaflory @kennylam and @emyarod, This PR has been updated to add author-configurable column widths. The storybook adds 4 knobs to handle this, but there are many options that would currently work right now, allowing authors to set responsive column widths

col-span-3 will set the number of carbon columns the 3rd table column will span
col-span-md-3 will override that value at the md breakpoint
col-span-lg-3 will override both at the lg breakpoint

If a column's width is not specified, it defaults to 2 carbon columns
If the total width of all table columns does not fill the available carbon columns, the last table column will stretch to fill the remaining width

@jwitkowski79 The commits above should fix all of your feed back except for the tags not aligning to the grid, that's caused from the tags code, and I'm pretty sure is an issue we've tried to push back upstream before
CC: @proeung

@andy-blum @proeung I noticed a few things while looking this over:
1. Group headers should be $text-01.
More obvious contrast difference in the dark themes

2. Tags are not following the dark theme color changes
Current example in structured list

Carbon React

3. Checkmark should be $support-02

4. Missing the ability to have a line break within the column cells.
This was one of the key differences in this structured list for @jwitkowski79
Two options:

Fix now and merge when work is complete (sorry it's so late Friday!)
Merge as-is and open an issue for next sprint to add the line break

5. Storybook naming
Sentence case for the stories to match our other stories please 🙏

Structured list

Default
With row headers
With subheaders
With complex content

@oliviaflory The tags are matching the behavior of the carbon web component version of the tag

https://web-components.carbondesignsystem.com/?path=/story/components-tag--default

@andy-blum Looking good 👍 . Still need to wrap the CSS in a mixin and export it, and resolve the private vars.

Thanks for the changes @andy-blum! Looks like the only thing left is to remove the nested mixins from the scss which are causing the CI failures. You can put them all before the main structured-list mixin and @include them where they need to be. See this example.

Not sure why the styling is not rendering in the current deploy preview but will update the branch and see if the styles will come through.


Weird behavior with KakaoTalk

I noticed very weird behavior recently with "KakaoTalk". This problem didn't occur before but at some point during updates.

After starting, KakaoTalk doesn't tile. When closing the chat list and restarting kwm, it sometimes tiles the chat window correctly, sometimes not.
When switching screen without transition through kwm, the window "sticks" and moves screens as well instead of staying on the screen it was before
The windows sometimes receive focus from follow focus, sometimes not (seen by the border).
I tried adding roles to the window like so

kwmc config add-role AXStandardWindow KakaoTalk
kwmc rule owner="KakaoTalk" properties={role="AXStandardWindow"}

without success
Accessibility inspector:

.
Using kwm 3.1.3 installed through brew

kwmc config add-role AXStandardWindow KakaoTalk

This was removed back in 2.x.x something (or 3.0.0).
The rule is the correct way to add a role.

When switching screen without transition through kwm, the window "sticks" and moves screens as well instead of staying on the screen it was before

Uh, no idea what could be up with that.
You may have to look into the whitelist option.
kwmc whitelist <process name>

Thanks for the speedy response. I tried the whitelist option as well but it didn't change anything.
Do you have some tips on how I could go on debugging what's up with that window?


Personalised benign waits

Is this:
Just place "x" to indicate selection (e.g. "[x]")

[x] feature
[ ] bug:

Instance Details
In some cases (installations) certain wait types are always present and as such need to be filtered out as noise from performance reports. It is possible however that in one installation these same wait types are not common and should be highlighted when collected by OQS.
Can we add a feature to allow each installation to configure a list of wait types that should be masked in addition to the benign waits listed at https://github.com/OpenQueryStore/OpenQueryStore/wiki/BenignWaits ? This would allow each site to have a better view of their own problematic wait types.

What about a switch/parameter in the install script? -CustomExcludeWaitTypes "c:\Temp\WaitTypes.txt"
Otherwise we could put our current list of waittypes into a txt file, that we use when installing. That way people would only have to know of the file and edit it, before installing. The file can then be stored in a central repository within the company and shared across multiple installations.

fixed with latest commit (should have used "close" in the commit to auto-close this issue)


File upload feature is not working correcting

Could you provide details? Or a screenshot?

Closing due to lack of details.


Do not use "new File(...)"

In the AbstractAnalyze class you use new File(...) for the output directory.
This is sub-optimal, because for a relative path, this method constructs a file relative to the users work directory, which means the class behaves differently depending on where the build was started from.
You should use project.file(...) (or depending on your minium Gradle version project.layout....) to get a path realtive to the project directory instead, so that it always behaves properly, no matter where the build was started from.

@jeremylong this is also an issue when configuring a suppression file; the configured value for 'suppressionFile' is passed to new File(...) which works for me on Oracle's JDK but fails on OpenJDK which happens to use a different working directory.

Sorry this took me so long to get to: https://github.com/dependency-check/dependency-check-gradle/pull/287


Static Files being trimmed

Hi,
I've been testing this today and have found an odd issue with static files being trimmed.
I create a 2mb js file and when requested it only returned 1.6mb
I ran it through express and it returned all of it. I created a separate .Net app using the UseStaticFiles middleware and that returned it all so my assumption is connect-owin is trimming somewhere.
Any ideas?

I couldn't reproduce your problem.
Can you tell me what version of the following components you are using :

nodejs
expressjs
connect-owin
Microsoft.Owin
Microsoft.Owin.StaticFiles

I'll get those over in about 90mins. I was running node on Linux so not
sure if that's a factor
On Monday, 16 March 2015, Bruno Baia<EMAIL_ADDRESS>wrote:

I couldn't reproduce your problem.
Can you tell me what version of the following components you are using :

nodejs
expressjs
connect-owin
Microsoft.Owin
Microsoft.Owin.StaticFiles

--
Reply to this email directly or view it on GitHub
https://github.com/bbaia/connect-owin/issues/4#issuecomment-81405938.

Here's a reproduction of the issue here https://github.com/jchannon/connectowinrepro
Its using latest .net components
Node = 0.10.25
Express = 4.12.2
Connect-Owin = 0.3.0

If you make a request to http://localhost:3000/static/lib.js you'll see the file is trimmed

In server.js file, if you comment the connect-owin middleware and configure the express static middleware like this :
app.use(express.static(__dirname));

You'll get the same problem.
The F12 panel indicates a 1.6mb file and the content is trimmed in the content panel, but if you download the source, the file is OK...

I didn't see that but will try again.
It first came about when an angular client was requesting the js file and it failed to initialise the angular app as it appeared the file was trimmed


The ability to silence platform warnings

https://github.com/bundler/bundler/blob/b9c12ee67661581f8a4fb5f47a375b5c7da2ed1e/lib/bundler/definition.rb#L880-L884
There really isn't any reason to ship that warning to me, and it would be nice if I could silence it, as I am well aware of the platform I am on, and I always inspect Gemfiles before I install anything, regardless of whether it's my own project or not.

I think we could accept a PR adding a setting to disable that warning, but I think we want to keep it enabled by default


ServiceBus SDK changes to support RBAC changes

Removed dependency on ADAL
Rename ManagedServiceIdentity TokenProvider to ManagedIdentityProvider
Added connection string support for Managed Identity tokenProvider
Replaced signatures for AAD TokenProvider with user defined callback interface
Added corresponding tests

Make sure there is no SpotBugs and CheckStyle errors. The CI will not pass until there is no more errors.
Current changes produces 1 SpotBugs error and 37 CheckStyle errors and 4 CheckStyle warnings after I ran maven.
"mvn -Dgpg.skip -DskipTests -Dinclude-non-shipping-modules -Ddata-plane install -f servicebus\data-plane\pom.xml"
"mvn -Dgpg.skip -DskipTests -Dinclude-non-shipping-modules -Ddata-plane site -f servicebus\data-plane\pom.xml"
I just ran the

Should be free of these error now.

In reply to: 501008301 [](ancestors = 501008301)

LGTM


VSTest task fix for dll search seems to be missing from on premise Team Foundation 2015 Update 3

I have an on premise instance of Team Foundation 2015 Update 3 but I am missing this fix from Feb 2016:
https://github.com/Microsoft/vsts-tasks/commit/fe096977af951100cfaf16e2568e4e08cc0a149b
The version of the VSTest task is 1.0.32 and looking in VSTest.ps1 in the folder \agent\tasks\VSTest\1.0.32\ I have this:
# check for solution pattern
if ($testAssembly.Contains("*") -Or $testAssembly.Contains("?"))
{
    Write-Verbose "Pattern found in solution parameter. Calling Find-Files."
    Write-Verbose "Calling Find-Files with pattern: $testAssembly"
    $testAssemblyFiles = Find-Files -SearchPattern $testAssembly -RootFolder $sourcesDirectory
    Write-Verbose "Found files: $testAssemblyFiles"
}
else
{
    Write-Verbose "No Pattern found in solution parameter."
    $testAssemblyFiles = ,$testAssembly
}

I would be expecting this:
$testAssemblyFiles = @()
# check for solution pattern
if ($testAssembly.Contains("*") -Or $testAssembly.Contains("?"))
{
    Write-Verbose "Pattern found in solution parameter. Calling Find-Files."
    Write-Verbose "Calling Find-Files with pattern: $testAssembly"
    $testAssemblyFiles = Find-Files -SearchPattern $testAssembly -RootFolder $sourcesDirectory
    Write-Verbose "Found files: $testAssemblyFiles"
}
else
{
    Write-Verbose "No Pattern found in solution parameter."
    $testAssemblyFiles += ,$testAssembly.Split(";")
}

This is my version of TFS

Looking at the release of Update 3, it looks like this fix should be included:
https://www.visualstudio.com/en-us/news/releasenotes/tfs2015-update3-vs#testing
There doesn't seem to be any way of updating the task other than doing it manually which doesn't seem like the right thing to do.

you can workaround the issue by including a trivial wildcard, e.g. ? in place of some char in your file name
the release branches correspond to sprintly releases. releases/m95 was used for TFS 2015 Update 2 and Update 3. releases/m105 was TFS 2017 RTM.
if you upload a task from a release branch, you need to follow the build process. build output gets generated under a folder _build. tfx cli can be used to upload the task to your server.
note, the goal of open sourcing the tasks is not to provide a delivery mechanism for newer tasks against an older server. the in-the-box tasks are open source examples for task authors.
another option would be (if you have a small number of agents) to edit the task cache on disk.

hello @ericsciple thanks for that info. I managed to work around the issue using the wildcard as you suggested.
Is there a record of what label goes with which TFS release?

@bthomsons In TFS 2017 onwards, build agent version corresponds to branch tag viz. Agent with version 2.105.7 shipped in TFS2017 will be using task definitons from releases/m105 branch.

@vimegh awesome, thanks for the update


No props are being forwarded.

All of my props given to a StyledComponent are getting swallowed by the styled component, and none of them are making it to the component targeted by the styled function. Seems like this is a substantial flaw.
Reproduction: https://codesandbox.io/s/vue-template-ydk8m?fontsize=14

:tada: This issue has been resolved in version 0.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Dang. 10 minutes flat. 💯


Increase test coverage for Mehtod 6: ImageUtils::checkIfImageIsDark

Increase test coverage by writing two tests

Closed by #34


ValueError: Non-consecutive added token '' found. Should have index 32100 but has index 0 in saved vocabulary.

Very exciting work, but I reported an error when loading Hugging Face's biot5-base-text2mol tokenizer. I tried transformers==4.28.1 and 4.33.3 versions.
code:
tokenizer = T5Tokenizer.from_pretrained("/workspace/LLM_ckpt/QizhiPei/biot5-base-text2mol", model_max_length=512)
ValueError: Non-consecutive added token '' found. Should have index 32100 but has index 0 in saved vocabulary.
I have another question.
When I load the biot5-base model for text2mol, I follow the text2mol code. The results are as follows:
output_selfies:

M
T
T
P
T
P
S
P
S
P
A
P
S
P
A
P
S
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P
A
P

output_smiles:
My code：
from transformers import AutoTokenizer, T5ForConditionalGeneration,T5Tokenizer, T5ForConditionalGeneration

tokenizer = AutoTokenizer.from_pretrained("/workspace/LLM_ckpt/QizhiPei/biot5-base")
model = T5ForConditionalGeneration.from_pretrained('/workspace/LLM_ckpt/QizhiPei/biot5-base')

# tokenizer = T5Tokenizer.from_pretrained("QizhiPei/biot5-base-text2mol", model_max_length=512)
# model = T5ForConditionalGeneration.from_pretrained('QizhiPei/biot5-base-text2mol')

task_definition = 'Definition: You are given a molecule description in English. Your job is to generate the molecule SELFIES that fits the description.\n\n'
text_input = 'The molecule is a monocarboxylic acid anion obtained by deprotonation of the carboxy and sulfino groups of 3-sulfinopropionic acid. Major microspecies at pH 7.3 It is an organosulfinate oxoanion and a monocarboxylic acid anion. It is a conjugate base of a 3-sulfinopropionic acid.'
task_input = f'Now complete the following example -\nInput: {text_input}\nOutput: '

model_input = task_definition + task_input
input_ids = tokenizer(model_input, return_tensors="pt").input_ids

generation_config = model.generation_config
generation_config.max_length = 512
generation_config.num_beams = 1

outputs = model.generate(input_ids, generation_config=generation_config)
output_selfies = tokenizer.decode(outputs[0], skip_special_tokens=True).replace(' ', '')
print("output_selfies:",output_selfies)

import selfies as sf
output_smiles = sf.decoder(output_selfies)
print("output_smiles:",output_smiles)

Thank you for your help！！！

Thanks for your interest in our work and bringing this issue to my attention.
The error you encountered with the Hugging Face's biot5-base-text2mol tokenizer is indeed a consequence of a discrepancy between the transformers library versions. When I initially uploaded the model and tokenizer to the Hugging Face repository, I used a higher version of the transformers library (4.34.x) than what is compatible (<=4.33.3), as specified in the requirements.txt of the GitHub repository. This incompatibility arose due to changes made in the token addition logic in the transformers 4.34.0 update, leading to inconsistencies in the dictionary when new tokens are added.
To address this, I have updated the Hugging Face repository with the transformers version 4.33.3, which should resolve the issue. Please try loading the tokenizer again with this updated version.
Regarding your second question about the biot5-base model and its application in text2mol: It is important to note that the biot5-base model, as it currently stands, has not undergone task-specific instruction tuning. Moreover, it is influenced by other pre-training tasks. Consequently, using the text2mol template with simple text input may not directly yield the corresponding molecule outputs. This limitation is inherent to the model's current training and tuning status.
Once again, thank you for pointing out this issue.


Improve feedback when executed from non-git directory

Current behaviour is confusing when running git-mob and git-solo from outside a git directory. Give user feedback that git commands only work within a git repo.

Closed by 82d2b18cc47d024bba02ca172b1dca6da8b43612


[Deployment]Add paictl cluster k8s-bootup command

Coverage decreased (-0.03%) to 50.246% when pulling ec326cfc5a7aefe78d24d5e6139967f4e26592a0 on canwan/k8sbootup into 80bb14cd8ef555ae595fdffb08905fc58c5688ce on management-dev.

Coverage decreased (-0.03%) to 50.246% when pulling ec326cfc5a7aefe78d24d5e6139967f4e26592a0 on canwan/k8sbootup into 80bb14cd8ef555ae595fdffb08905fc58c5688ce on management-dev.

Coverage decreased (-0.03%) to 50.246% when pulling ec326cfc5a7aefe78d24d5e6139967f4e26592a0 on canwan/k8sbootup into 80bb14cd8ef555ae595fdffb08905fc58c5688ce on management-dev.


AWS/Fargate: CodeDeploy Service Role cannot be longer than 64 characters

If you're project name is longer than 15 characters it will cause the template to generate a CodeDeploy Service Role name that is longer than the allowed length of 64 characters:

Technically applies to any Role Name, but the CodeDeploy role is naturally longer given the logic and phrasing use to auto-construct the name.  For example: codedeploy-test20220319-api-codedeploy-app-us-east-1-service-role.
Ideally the project would support project prefixes up to 32 characters, but whatever the limit is it should be stated and enforced when generating the templates.

A related issue is that the ELB target group names cannot be longer than 32.
Looks like the practical max for the project prefix is going to be 28.  Should accommodate most projects by default, but will truncate the name if necessary.


Remove trait definitions as part of default codegen transforms

IIUC trait definitions itself aren't needed by codegen (clients, server, types). And current code generators like typescript, ruby, go are using getModelWithoutTraitShapes currently anyways.
If it is better to leave the model with traits (note this includes traits like aws.cloudformation#cfnAdditionalIdentifier in the assembled model, even if the model file does not reference it), we can update individual code generators to use appropriate walker/transform in the specific places as needed.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Is this necessary though? Directed codegen invokes the methods to perform codegen for you anyways, so you don’t need to filter them out. I think each directive also provides methods to get the shapes connected to the service. Removing trait shapes limits the kinds of selectors you can perform too, like finding protocol bearing traits.

Thanks @mtdowling. When discussing with some other folks in the team, it seemed (maybe I misunderstood or the context wasn't right) that there isn't a need to have the trait definition shapes in the model. Yes, Directive.connectedShapes provides the Walker usage, but the current codegen logic explicitly uses getModelWithoutTraitShapes in the Walker making it seem necessary.
I wasn't sure overall and hence made this PR as a way to discuss and check if this is ok. Thanks for pointing out about the protocol traits - it indeed fails to find the protocols with this change and that fails elsewhere.


Remove side effects of scene save

Here we finally get rid of the dummy instantiation of the scene being saved,, which was a way of recreating the node path cache, needed for the editing state to be consistent and avoid bugs like #7979.
In this PR, in addition to removing the generation of the dummy scene (like #47828 tentatively did), a new way of having a good node path cache is added: namely, when a scene is packed in the editor, the data already collected for packing is leveraged to create such cache.
Benefits:

Tool scripts are not unexpectedly run anymore when saving a scene (so it fixes #17681).
The goal of #49570 may be reinforced (even better scene saving speed).

Testing-wise, I've

ensured that #7976 is not reintroduced;
noticed that #7984 still happens;
checked that #47828 alone (just removing the dummy scene thing) would indeed reintroduce #7976.

Therefore, I'd say this PR is pretty likely not to cause any observable change of behavior, either for bad or for good in addition to its purpose. As always in this kind of PR, testing every case is difficult, so user testing on alphas/betas would be a good thing to have.
As a final note, it may be better to add the possibility of user control about generating or not the node cache –pretty much like in PackedScene::instance() you can choose to generate editor data or not–. However, this already improves the state of things and something like that can be done as a future refinement.
UPDATE: I forgot to add that, since SceneState::find_node_by_path() is unreliable without a node path cache, I've added a check so we can detect easily if there's some other flow that needs it. That's not because of this PR; it would have been a good idea before it as well.

Not sure about marking as cherrypick:3.3, since this is not exactly critical.

Thanks!

Cherry-picked for 3.4.


subfolder

instead of having all modules flat, we need to subgroup them based on org!
currently there are already naming conflicts with modules.
Spryker
     ModuleName
        x.y.z
SprykerShop
     OtherModule
        x.y.z

As a side effect this also allows for any 3rd party vendor to submit recipes.

[ ] Adjust recipe generation via release app @dereuromark
[x] Adjust existing recipes to be moved in proper location - @dereuromark via small custom script
[ ] Adjust integrator usage @demkos ? or who can do it
[ ] Adjust docs

https://github.com/spryker-sdk/integrator-recipes/pull/49 PR open

Release App from now on generates nested structure.

@gechetspr Can you assign someone regarding the missing task maybe?


y2024 in copyright string

update our copyright string to 2024

Not fixed on develop (see discussion in: #2332).


Swarm bootstrapping when using TLS and domains instead of IPs

I have a basic docker swarm setup including containers for swarm-master, swarm-nodes and etcd. I also added TLS according to the docs with a self signed certificate and created a overlay network where I can ping from one container to another one by using the their names. Up to this point everything works fine.
In the next step I want to use a real certificate and also domains instead of IPs. So I start etcd as usually and then the client like this:
docker run --name node1 \
           -v /certs/cloud:/certs:ro \
           -h "swarm-node-1.cloud" \
           swarm join \
           --advertise=swarm-node-1.cloud:2376 \
           --discovery-opt kv.cacertfile=/certs/cloud.ca-bundle \
           --discovery-opt kv.certfile=/certs/cloud.crt \
           --discovery-opt kv.keyfile=/certs/cloud.key \
           etcd://discovery-1.cloud:4001

One problem is that the domain discovery-1.cloud can not be resolved (of course!). Because of that I linked the swarm node container with the etcd container (--link etcd:discovery-1.cloud). The same problem appears again for the swarm master. But the swarm master also needs to know how he should resolve swarm-node-1.cloud and therefore I added --add-host=swarm-node-1.cloud:<IP_ADDRESS>.
docker run --name manager -p 3376:3376 \
           --add-host=swarm-node-1.cloud:<IP_ADDRESS> \
           --link etcd:discovery-1.cloud \
           -v /certs/cloud:/certs:ro \
           -h "swarm-manager.cloud" \
           swarm --debug \
           manage etcd://discovery-1.cloud:4001 \
           --tlsverify \
           --tlscacert=/certs/cloud.ca-bundle \
           --tlscert=/certs/cloud.crt \
           --tlskey=/certs/cloud.key \
           --discovery-opt kv.cacertfile=/certs/cloud.ca-bundle \
           --discovery-opt kv.certfile=/certs/cloud.crt \
           --discovery-opt kv.keyfile=/certs/cloud.key \
           --advertise=swarm-manager.cloud:3376 \
           -H tcp://<IP_ADDRESS>:3376

I think that these issues are just bootstrapping issues and wont be relevant when the overlay network is setup and the hostname of containers can be resolved. If there is a better way to bootstrap these containers then please let me know!
The setup seems to be working as I get this result from the swarm manager:
➜  ~ docker --tlsverify \
       --tlscacert=/certs/cloud/cloud.ca-bundle \
       --tlscert=/certs/cloud/cloud.crt \
       --tlskey=/certs/cloud/cloud.key \
       -H=localhost.cloud:3376 info

Containers: 3
 Running: 3
 Paused: 0
 Stopped: 0
Images: 9
Server Version: swarm/1.1.2
Role: primary
Strategy: spread
Filters: health, port, dependency, affinity, constraint
Nodes: 1
 mzangerle: swarm-node-1.cloud:2376
  └ Status: Healthy
  └ Containers: 3
  └ Reserved CPUs: 0 / 4
  └ Reserved Memory: 0 B / 8.083 GiB
  └ Labels: executiondriver=native-0.2, kernelversion=4.2.0-30-generic, operatingsystem=Ubuntu 15.10, storagedriver=aufs
  └ Error: (none)
  └ UpdatedAt: 2016-03-08T16:07:39Z
Plugins:
 Volume:
 Network:
Kernel Version: 4.2.0-30-generic
Operating System: linux
Architecture: amd64
CPUs: 4
Total Memory: 8.083 GiB
Name: swarm-manager.cloud

To create an overlay network I previously had to add the --cluster-advertise and --cluster-store to the docker.conf:
[Service]
ExecStart=
ExecStart=/usr/bin/docker daemon -H tcp://<IP_ADDRESS>:2376 -H unix:///var/run/docker.sock --dns <IP_ADDRESS> --dns <IP_ADDRESS> --cluster-advertise=<IP_ADDRESS>:0 --cluster-store=etcd://<IP_ADDRESS>:2379 --tlsverify --tlscacert=/certs/cloud/cloud.ca-bundle --tlscert=/certs/cloud/cloud.crt --tlskey=/certs/cloud/cloud.key

So I tried to adjust these flags with domain names instead of IP addresses as you can see here ...
[Service]
ExecStart=
ExecStart=/usr/bin/docker daemon -H tcp://<IP_ADDRESS>:2376 -H unix:///var/run/docker.sock --dns <IP_ADDRESS> --dns <IP_ADDRESS> --cluster-advertise=localhost.cloud:0 --cluster-store=etcd://discovery-1.cloud:2379 --tlsverify --tlscacert=/certs/cloud/cloud.ca-bundle --tlscert=/certs/cloud/cloud.crt --tlskey=/certs/cloud/cloud.key

But when I try to restart the docker daemon I get following error messages and it seems that docker does not accept domains for these flags:
FATA[0000] Error starting daemon: discovery advertise parsing failed (invalid cluster advertise IP address or interface name (swarm-node-1.cloud:0) : route ip+net: no such network interface)

So my questions are:

Is there a better way to do this kind of bootstrapping?
How can I create a overlay network when using TLS and domains?

In case it's needed here is my docker version:
➜  ~ docker --version
Docker version 1.10.2, build c3959b1

same question : is there a reason why it's only possible to advertise on IPs?


Update RLViser

Using the newest RLViser uses less code in Python and sends the right data to the most recent version of RLViser (won't work without this patch)

Thanks for the PR


Enum-like points for MoveMouse task

App related points: .center, .top, .bottom, .leading or .left, .trailing or .right
MoveMouse(to: .top)

Screen related points: .screenCenter, .screenTop, .screenBottom, .screenLeading or .screenLeft, .screenTrailing or .screenRight
MoveMouse(to: .screenBottom)

App related points: .center, .top, .bottom, .leading or .left, .trailing or .right

Should the app related points default to screen related points if the application doesn't have a window like a command line app?

Should the app related points default to screen related points if the application doesn't have a window like a command line app?
Yeah, good point.

@0xLeif RobotKit is built for English language, so we will stick for now to left/right naming? and maybe later if we will add support for RTL language, we may introduce the structure that supports the naming of leading/trailing concept? what do you think?

Good point! Right now let's stick with English... but I think we can plan to add localization later on.


Editor Crashes when trying to complete _init().() on subclass

Apologies if this post is janky, was told to submit this bug from the discord server and new to github.
Godot version:
Godot 3.2.2 Stable
OS/device including version:
tested on two PCs
Device 1: W10, gtx 1080, 16gb RAM i7-7820HK
Device 2: W10, integrated graphics, 6gb RAM, Ryzen 3
Issue description:
Was setting up a basic inheritance system, when the child class would crash while filling out _init().(variable_here,var_two):
note, I had prewritten _init.().(): which seems to be the root cause
Steps to reproduce:
first, Parent script:
var charName : String
var attack : int
func _init(baseName = "placeholder", baseAtk = 1):
charName = baseName
attack = baseAtk
second (ideally), Child script:
extends "res://Assets/charTemp.gd"
func _ready():
pass
func _init().("Hunter", 5):
what happens, however, is
func _init().("Hunter",):
after adding the comma, Godot will freeze up and crash.
However, I did find that if I don't have ): written in before filling out the second set of parentheses, then it works fine.
Minimal reproduction project:
New Game Project.zip

Confirmed in 3.2.2, seems fixed in 3.2.3.rc3.

Closing then.


feat: unicode locale id type utils

Description

Linked Issues
Additional context

At this point, it is a bit difficult to implement a parser for unicode locale id with typescript type defintion.
WIP, but once merged. We may remove it later.


Create move_donation_info_to_another_voter function in donate/controller.py

Please create a function in donate/controller.py called "move_donation_info_to_another_voter". Please call this function near the bottom of "voter_merge_two_accounts_for_api" in voter/controllers.py. This function should result in donations and donation history information that are spread across two voters to be merged into a single voter:

[ ] Update all voter_we_vote_id values in the donation tables: DonateLinkToVoter, DonationFromVoter, DonationLog, and DonationSubscription.

Let's discuss problem cases that might arise.

Sure.  I would like to get basic Webhook data flowing first -- at that point all the data in postgres would stay current, including subscription expiration status.

This has been implemented

Hi Steve, I took a closer look at "move_donations_between_donors" in donate/models.py and it only seems to be moving entries from the DonationJournal. We need to move/update data from DonateLinkToVoter, DonationFromVoter, DonationLog, and DonationSubscription.

Thank you @SailingSteve for your recent check in that finishes cancelling donations and refunding subscriptions. Will you have time to work on this issue next?

Still cleaning up existing code, but I will get to this next.

@DaleMcGrew

Over the last few weeks, I looked at the pre-existing schema, and thought about what I would need to do to make subscriptions and webhooks work, and in the process I significantly simplified the schema.  Hopefully not losing out on any potential capabilities.  I left all the pre-existing tables and code in place, while building up the new capabilities.  Then when the new table was working and meeting all needs, I pulled out the parts of the older code and tables that weren’t being used.
The DonationJournal contains every bit of data we need, it has long rows, but you can find anything without a join or a multi-table lookup.  By being frugal with column widths, it takes up less disk space than the prior tables.
The stuff I pulled out:

DonateLinkToVoter is still there, but is not read by the code -- I don’t think it ever was.
DonationFromVoter is gone.
DonationLog is gone, superseded by DonationJournal that has a lot more columns.
DonationPlanDefiniton is still there but pretty useless.  The ability to donate any arbitrary amount (even before my changes) made the value for persisting the plan very low.  If you needed a 12.25 plan, you could just ask Stripe if “monthly-1225” existed and make it on the fly if needed, and then forget about it.  As I got into the webhook, it was convenient to encode the voter_we_vote_id into the plan, so persisting the plans became even less useful. See donations/models.by line 368.  The DonationPlanDefiniton code and table are still in place, but could be optimized out.
There was a half implemented table for credit cards, but that isn’t the way stripe works, and it is gone.

The implementation of the webhooks added new requirements on the data, since the transactions from the webhooks didn’t contain subscription_ids (lame from our point of view).  And the metadata that I put into stripe transactions like “voter_we_vote_id”, sticks with the one time payments from the UI and subscription setups from the UI, but does not come back with automated payments from subscriptions.  So, I prepended the voter_we_vote_id to the donation_plan_id so that I could retrieve the key from webhook transactions.
The "move_donations_between_donors" that we discussed, was just to move the donations made within a single session when an un-logged in voter makes a donation, then logs in.  That is a thin use case, but was all we decided to do.  Much more is possible (see  Dealing with donations when not logged in).
If this needs more work, I would be happy to do it.

Thank you Steve, these changes look great. Feel free to remove the tables/classes that you think we don't need. So you are saying that DonationJournal is the only donation table with voter data? If so we can close this issue.

DonationJournal is the only table we need.  DonationPlanDefiniton is still used, but is vestigial.  If it were worth it, it could be removed with a few hours of work.  Closing for now.


feat: add mark element

This PR fixes #229

This looks right. I verified that the default styles for a <mark> element don't change with with color-scheme: dark applied and so the styling is consistent with what is expected on the web.
Based on my testing in Expo Snack, the rendered results are as expected in Android, but slightly wrong on iOS.
Android:

iOS:

Is this a bug we should report to RN? Note how the yellow background continues to end of the line even when the text has already wrapped to the next line?

Is this a bug we should report to RN? Note how the yellow background continues to end of the line even when the text has already wrapped to the next line?

I'll check later on this locally in RN 0.76.1 and RN 0.75.* to verify if this is a regression. I'll share screenshots!

Pretty sure that's a known RN bug

I could reproduce the issue locally and confirm it only happens on IOS, although I only tested it on the current version of the demo app. It is indeed a RN bug.

Android
IOS


the rpc too slowly

it is spent ten seconds or more to get result when i use omnicore-cli such as omni_getinfo getinfo no mater on pc(i7 8700 32G  128g ssd 1t hdd) or cloud-server
how can i speed up ?

128G  节点应该没有同步完成吧

128G 节点应该没有同步完成吧

原来是国人开发的啊。
那是电脑配置，节点数据是存储在hdd上面的，有280多g。想弄个自有节点开发钱包，但是这个rpc调用反馈比bitcoin core慢太多，换阿里云也是这么个现象。

Same issue。Start omnicored with "rpcthreads=4" and this is also the default value of the configurations. It comes out one CPU core usage nearly 100 %, the other three CPU core seems not working. the blocks were fully block synced.
the RPCs is totally too slow.
version: v0.5.0

我的也是 上个月搭建了2次都是很慢，我还专门换了essd，26800 IOPS，还是一样，应该是那个地方不对吧，谁知道怎么解决啊

Same issue。Start omnicored with "rpcthreads=4" and this is also the default value of the configurations. It comes out one CPU core usage nearly 100 %, the other three CPU core seems not working. the blocks were fully block synced.
omni_getinfo / omni_getbalance .. are too slow. The bitcoin original RPCs seems working fine.
version: v0.5.0

I'm in the same situation as you，omni_getinfo / omni_getbalance .. are too slow. The bitcoin original RPCs seems working fine.version: v0.7.0

same issue here. omni_getinfo is too slow, while getblockchaininfo works fine.

same issue here.
version: v0.8.2

现在一般的交易所都不会自己去单独开发一套钱包系统了。
不仅因为比特币以太坊等区块节点数据庞大，同步传输慢，而且在服务器和带宽的花费成本比较高。
最重要的是养不起这样的一个技术团队！
原生钱包这种私钥放在服务器非常不安全，被偷就凉凉了。
现在市面上大家用的比较多的优盾钱包很好用
尤其是它的多币种多地址资产一键归集和冷热钱包隔离
最关键的是去中心化的
顶级私钥BOSS自主掌握，子私钥动态计算不触网，硬件加持，纯冷操作
既安全、又放心、具备丰富功能的全球领先的企业级数字资产管理系统，它不香吗

Hi! v0.10.0 seems to have the same issue for all omni specific calls (i.e being extremely slow). Will this be addressed in the next release?

same issue, omni-cli omni_getinfo takes about half minutes.


Slider responsiveness

On a wide screen, the partner slider is not centered and the prev/next arrows are not correctly aligned.

Fixed as of 29de3e1e964d73c7a800e34ef0a760655f28f87e.


[Installation] Add custom installers

This pull request adds an installer.py file to each package folder, shortens the eval command required to download the package, and makes the installation process easier. For example, it automatically inserts the packages into the config.yml file and automatically creates and runs migrations.
Most of this code hasn't been tested yet.

Also, there are some smaller miscellaneous changes you can find in the HOWTOINSTALL.md files too :)

thanks!


Improve AP tracking.

Is your feature request related to a problem? Please describe.
I would love to be able to use Foundry for my character management. But the way AP tracking is implemented makes this impossible.
When you don't use the "increase/decrease" buttons to level your attributes/skills/... you don't pay AP. Which is annoying when you are importing characters. Fine let's go slow and click all the buttons. When you click them to fast you pay AP multiple times for the same increase without increasing the "attribute score". But you can't easily fix this because you need the character sheet to not update fast enough when you are trying to revert the points.
There is also the issue that you can't really give someone a free skill increase. Let's say you let someone increase their flying from 13 -> 14 by editing the number directly so that it doesn't cost AP. Then the game doesn't realise that your score increased meaning that you only pay for the increase from 13 -> 14 while the skill improves from 14 -> 15
Describe the solution you'd like
I would love to have more control over the AP. It would be nice to be able to edit how many AP a character has expanded. I would also love the ability to update how many AP i spent on a individual skill or attribute because i like to give out free skill increase or cheaper training as a game reward.
It would also be an easy fix for wrong AP values on your character.
I would also love an update AP tracking algorithm. I implemented one for fun back in the day.
I basically had a function which would translate how many AP a single level of whatever costed. I fed this function the target score and the AP Factor (Steigerungsfaktor) and it returned the AP cost.
Example for flying:
Input 8,A -> 8 AP.
The improvement chart A costs 1 AP per Level up to Level 12 and then by 1 per level. So we just multiply 8 with 1 and return the value.
Example for Intuition:
Input 17,E -> 60 AP.
The improvement chart E costs 15 AP per Level up to Level 14 and then increases by 15 per level. 17 is 3 levels above 14 so we multiply just calculate (Inputlevel-13)*15 and return the value.
Now the value change function
Every time a number changed i would simply calculate the AP cost of the current Skill/Attribute/Whatever using the function above and substract it from the currently expended amount of AP. Then i would change the value of the Skill/Attribute/Whatever and recalculate the AP cost which i would add to the currently expended amount of AP.
That way i only needed one algorithm for Skills/Attributes/Whatever and i didn't care if the value was increased or decreased. Because i tracked the value before and after the value change it was impossible to pay multiple times and i didn't need to force the user to use any buttons for increasing or decreasing. The algorithm simply cared about the value being changed and auto applied itself every time.
Describe alternatives you've considered
I would love to have a check mark which just disables automatic AP tracking in the mean time. That way my players could at least use the AP fields to track their AP values by hand. The current system is basically only usable if you are super careful. And when you need to fix something it is often faster to just recreate the character entirely if you don't recognise the mistake immediately.

This will not be a character generator, never.
It's already too much work. See several mentions everywhere.
Also see the character calculator in the core modul.
And finally you can edit the AP sum in the attributes/eigenschaften tab

This will not be a character generator, never.

I totally understand that. But why isn't there an option to disable AP tracking entirely ? It seems to me like you could have saved yourself a bunch of explaining by not including AP tracking at all. In the current version you are basically implying that the tool is a character generator. Doesn't matter that you named it a character calculator instead.

Yeah I know it's horrible.

I think the delta tracking is brillant. No need to switch to another tool once the character is done. And the capabilities to create a character are also already damn good.
But this is also already the end of it.


Reflect new ASE master changes after tests fix

Requirements
Checklist

[ ] I have read the Contributing Guide. Don't lie! 😉
[ ] My PR is on a custom branch and is not named main.
[ ] I have used black, isort, and ruff as described in the style guide.
[ ] I have added relevant, comprehensive unit tests.

Notes

Your PR will likely not be merged without proper and thorough tests.
If you are an external contributor, you will see a comment from @buildbot-princeton. This is solely for the maintainers.
When your code is ready for review, ping one of the active maintainers.

Summary of Changes

Can one of the admins verify this patch?

Thanks! Setting this to auto-merge.


Fix typo

Codecov Report

Merging #718 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #718      +/-   ##
==========================================
- Coverage    86.6%   86.56%   -0.05%
==========================================
  Files          71       71
  Lines        2262     2262
  Branches       91       91
==========================================
- Hits         1959     1958       -1
- Misses        303      304       +1

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/JsonObject.scala
93.79% <ø> (ø)
:arrow_up:

...re/shared/src/main/scala/io/circe/KeyDecoder.scala
97.29% <0%> (-2.71%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
91.76% <0%> (ø)
: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 d0f7f83...5c8ab1c. Read the comment docs.

Good catch, thanks!


net/rpc: client's DialHTTPPath() treats http CONNECT response incorrectly

https://tools.ietf.org/html/rfc7231#section-4.3.6 - says: 'Any 2xx (Successful) response indicates that the sender (and all inbound proxies) will switch to tunnel mode immediately after the
blank line that concludes the successful response's header section;'
but DialHTTPPath() requires exactly resp.Status == "200 Connected to Go RPC"
https://github.com/golang/go/blob/master/src/net/rpc/client.go#L258
if err == nil && resp.Status == connected {
https://github.com/golang/go/blob/master/src/net/rpc/server.go#L688
var connected = "200 Connected to Go RPC"

@robpike


Ensure all issues are put in the kanban board for tracking

Use github automation - e.g. https://github.com/HL7/fhir-ig-publisher/tree/master/.github/workflows - to ensure that all issues are in the kanban

added workflow


[Bug]: PushManager never returns a value from the RequestAccess task.

Component/Nuget
Push - Native (Shiny.Push)
What operating system(s) are effected?

[X] iOS (13+ supported)
[ ] Mac Catalyst
[ ] Android (8+ supported)

Version(s) of Operation Systems
iOS 17.2 (physical device) and others
Hosting Model

[X] MAUI
[ ] Native/Classic Xamarin
[ ] Manual

Steps To Reproduce

Include nugets: Shiny.Core, Shiny.Push, Shiny.Hosting.Maui
Configure the project correctly to use push notifications
Requests platform permission to send push notifications using method RequestAccess from PushManager.

Android: works fine
iOS: never returns the task result and hangs
Expected Behavior
The RequestAccess method should return a value.
Actual Behavior
When the RequestAccess method is called, nothing happens and the tank result is never returned.
In AppDelegate for RegisteredForRemoteNotifications I get the appropriate deviceToken which is then passed to OnRegisteredForRemoteNotifications():
 [Export("application:didRegisterForRemoteNotificationsWithDeviceToken:")]
 public void RegisteredForRemoteNotifications(UIApplication application, NSData deviceToken) => Shiny.Hosting.Host.Lifecycle.OnRegisteredForRemoteNotifications(deviceToken);

I don't know why, but this PushManager method is not called:
   public void OnRegistered(NSData deviceToken) => this.tokenSource?.TrySetResult(deviceToken);
As a result, in the RequestRawToken method we wait forever for the result from the TaskCompletionSource tokenSource.
  protected async Task<NSData> RequestRawToken(CancellationToken cancelToken)
  {
      this.tokenSource = new();
      using var cancelSrc = cancelToken.Register(() => this.tokenSource.TrySetCanceled());

      await this.platform
          .InvokeOnMainThreadAsync(
              () => UIApplication
                  .SharedApplication
                  .RegisterForRemoteNotifications()
          )
          .ConfigureAwait(false);

      var rawToken = await this.tokenSource.Task.ConfigureAwait(false);
      return rawToken;
  }

It looks as if PushManager is not a singleton at all or the OnRegisteredForRemoteNotifications method uses a different PushManager instance than the one registered for the application.
Exception or Log output
No response
Code Sample
No response
Code of Conduct

[X] I have supplied a reproducible sample that is NOT FROM THE SHINY SAMPLES!
[X] I am a Sponsor OR I am using the LATEST stable/beta version from nuget (v3.0 stable - ALPHAS are not taking issues - Sponsors can still send v2 issues)
[X] I am Sponsor OR My GitHub account is 30+ days old
[X] I understand that if I am checking these boxes and I am not actually following what they are saying, I will be removed from this repository!

You need to provide a full sample showing the issue

3.3.3 has been tested here: https://github.com/shinyorg/pushtester and verified to be working

This sample repository additionally uses Shiny.Push.FirebaseMessaging and Shiny.Push.AzureNotificationHubs
I am not analyzing the use of these additional libs

These are built on top of the "basic" implementation

For Android it does not matter, but in the case of iOS we wait for the result for the TaskCompletionSource which is never completed because SetResult is made for a completely different instance than we expect.

If you want me to look at this, you have to supply a reproducible sample or this will be locked.

Hello, I'm facing the same situation, android works great, ios doesn't want to
@HollowAccount, did you manage to solve this?

@xarlammm please don't hijack closed issues.  If you have an issue, file it with all of the requested information


Reaction Emoticons update

The reactions (happy/sad) emoticons have been updated by Jared:

This PR updates the emoticons inside HSDS React.
While doing so, I noticed that we have <Emoticon /> and <RateAction /> components and their intended functionality was a bit mixed, specifically some stuff was on Emoticon that should have been on RateAction.
So I refactored both components so that:

Emoticon is like Icon, but for svgs that have more than one colour (so shades do not apply for example)
RateAction uses Emoticon under the hood, as before, but it takes care of the animation and interactions
RateAction: added the "okay" animation
RateAction: added all 3 sizes
Removed TypeScript on both
Added prop-types
Uses JSDocs where appropriate
Restructured folders to recently adopted pattern
Added new stories for both
Updated docs
Maintains backwards compatibility
Removes 'propConnect' from both

Screens
Emoticons

RateAction

Pull Request Test Coverage Report for Build 2888

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

Totals

Change from base Build 2885:
0.0%

Covered Lines:
82

Relevant Lines:
82

💛  - Coveralls

@plbabin not sure i follow your comment on the focus state issue? You dont think the green border is clear?

@plbabin not sure i follow your comment on the focus state issue? You dont think the green border is clear?

green border is for the selected state. I thought the focus was only increasing the size, but it is the size, color and animation. I am fine with that https://c.hlp.sc/4guxJje2.
Disregard my comment 😄

@plbabin You're right ... my bad. But yea, think we're okay there. 👍


Displayname

Hi, I'm trying to put a space between my prefix and ign. I can't seem to figure out how to do it. I'd like it to say; [Prefix] ign instead of [Prefix]ign.

Either set the prefix itself with a space "[Prefix] " or configure your chat plugin format to add a space between the prefix and username.


Fix building process with test

For start testing only on test command. (#3)
Actually, when you launch build command. the test is run in all command (see https://docs.gradle.org/current/userguide/java_plugin.html#sec:java_tasks )
prefer to use assemble gradle task to build.

Linked to #2

Thanks! The problem with using assemble is that it's not available on the Android plugin for Gradle. I think the most flexible way would be to write a Gradle plugin that checks which one is available and act accordingly. What do you think?

@esteve assemble is a gradle Task for java and android.
http://tools.android.com/tech-docs/new-build-system/user-guide#TOC-Build-Tasks
http://stackoverflow.com/questions/4597850/gradle-build-without-tests
But I'am ok for Gradle plugin

@Theosakamg I thought I had tried using assemble, let me try it again with Android and maybe we won't need a Gradle plugin for now.

I check on external android project :
./gradlew assemble
:Application:preBuild UP-TO-DATE
:Application:preDebugBuild UP-TO-DATE
:Application:checkDebugManifest
:Application:preReleaseBuild UP-TO-DATE
...
:Application:packageRelease
:Application:assembleRelease
:Application:assemble

BUILD SUCCESSFUL

And on ROS2 Android WS :
Process package 'ros2_listener_android' with context:
--------------------------------------------------------------------------------
 source_space => /home/micky/ros-workspace/ros2java/ros2_android_ws/src/ros2_java/ros2_android_examples/ros2_listener_android
  build_space => /home/micky/ros-workspace/ros2java/ros2_android_ws/build_isolated/ros2_listener_android
install_space => /home/micky/ros-workspace/ros2java/ros2_android_ws/install_isolated/ros2_listener_android
   make_flags => -j4, -l4
  build_tests => False
--------------------------------------------------------------------------------
+++ Building 'ros2_listener_android'
==> '/usr/bin/gradle -Pament.build_space=...  -Pament.build_tests=False -Pament.android_stl=gnustl_shared -Pament.android_abi=armeabi-v7a -Pament.android_ndk=/opt/android/android-ndk-r12b assemble' in '/home/micky/ros-workspace/ros2java/ros2_android_ws/src/ros2_java/ros2_android_examples/ros2_listener_android'
Incremental java compilation is an incubating feature.
:cleanNativeLibs UP-TO-DATE
:copyNativeLibs
:preBuild UP-TO-DATE
:preDebugBuild UP-TO-DATE
:checkDebugManifest
:prepareDebugDependencies
:compileDebugAidl UP-TO-DATE
:compileDebugRenderscript
:generateDebugBuildConfig
:mergeDebugShaders UP-TO-DATE
:compileDebugShaders UP-TO-DATE
:generateDebugAssets UP-TO-DATE
:mergeDebugAssets UP-TO-DATE
:generateDebugResValues UP-TO-DATE
:generateDebugResources
:mergeDebugResources UP-TO-DATE
:processDebugManifest UP-TO-DATE
:processDebugResources UP-TO-DATE
:generateDebugSources
:incrementalDebugJavaCompilationSafeguard UP-TO-DATE
:compileDebugJavaWithJavac
None of the classes needs to be compiled! Analysis took 0.052 secs.
:compileDebugNdk UP-TO-DATE
:compileDebugSources
:prePackageMarkerForDebug
:transformClassesWithDexForDebug
Merged dex #1 (2 defs/2,2KiB)
Merged dex #2 (12 defs/8,2KiB)
Merged dex #3 (9 defs/9,1KiB)
Merged dex #4 (7 defs/7,6KiB)
Merged dex #5 (1 defs/1,2KiB)
Merged dex #6 (13 defs/7,3KiB)
Result is 44 defs/39,3KiB. Took 0,1s
:mergeDebugJniLibFolders UP-TO-DATE
:transformNative_libsWithMergeJniLibsForDebug
:processDebugJavaRes UP-TO-DATE
:transformResourcesWithMergeJavaResForDebug
:validateDebugSigning
:packageDebug
:zipalignDebug
:assembleDebug
:preReleaseBuild UP-TO-DATE
:checkReleaseManifest
:prepareReleaseDependencies
:compileReleaseAidl UP-TO-DATE
:compileReleaseRenderscript
:generateReleaseBuildConfig
:mergeReleaseShaders UP-TO-DATE
:compileReleaseShaders UP-TO-DATE
:generateReleaseAssets UP-TO-DATE
:mergeReleaseAssets UP-TO-DATE
:generateReleaseResValues UP-TO-DATE
:generateReleaseResources
:mergeReleaseResources UP-TO-DATE
:processReleaseManifest UP-TO-DATE
:processReleaseResources UP-TO-DATE
:generateReleaseSources
:incrementalReleaseJavaCompilationSafeguard UP-TO-DATE
:compileReleaseJavaWithJavac
None of the classes needs to be compiled! Analysis took 0.004 secs.
:compileReleaseNdk UP-TO-DATE
:compileReleaseSources
:lintVitalRelease
:prePackageMarkerForRelease
:transformClassesWithDexForRelease
Merged dex #1 (2 defs/2,2KiB)
Merged dex #2 (9 defs/9,1KiB)
Merged dex #3 (7 defs/7,6KiB)
Merged dex #4 (12 defs/8,1KiB)
Merged dex #5 (1 defs/1,2KiB)
Merged dex #6 (13 defs/7,3KiB)
Result is 44 defs/39,1KiB. Took 0,0s
:mergeReleaseJniLibFolders UP-TO-DATE
:transformNative_libsWithMergeJniLibsForRelease
:processReleaseJavaRes UP-TO-DATE
:transformResourcesWithMergeJavaResForRelease
:packageRelease
:assembleRelease
:assemble
:deployArtifacts

BUILD SUCCESSFUL

From clean workspace, and worked on device.

I've made some changes to your branch and merged it, see https://github.com/esteve/ament_java/pull/4
Thanks!


2.0 compat

Fix various 64bit issues on 2.0-compat branch

2.0-compat doesn't need to be merged into master, that is an old compatibility branch of 2.x versions.


Dynamic Island Broken in Dev

After updating from 4.1.4 to 4.3, the Dynamic Island and Lock Screen notification appears to no longer work.
Both worked in the previous version of Dev I was using.
Screen cap video here as too large to upload to git - https://share.icloud.com/photos/08eCAEfodxQa1-kO1CdcxegJg

Phone restart fixed issue, sorry!


Allow asset deprecate to asset from different namespace

Desired Behavior
I want to deprecate
https://bonsai.sensu.io/assets/jspaleta/sensu-plugins-disk-checks
and point people to
https://bonsai.sensu.io/assets/sensu-plugins/sensu-plugins-disk-checks
Problem
The deprecate dialog doesn't expose or allow me to search by namespace.
I can search for and select 'sensu-plugins-disk-checks'  but that ends up just selecting the asset I'm trying to deprecate.... effectively pointing people who got to
https://bonsai.sensu.io/assets/jspaleta/sensu-plugins-disk-checks
back to itself.

Pull Request #158


Add version info in include/c.h

What steps will reproduce the problem?
1. There isn't any version info in b binding file include/c.h

What is the expected output? What do you see instead?
I can get version info in c interface

What version of the product are you using? On what operating system?
ANY

I'm working on a php binding using c binding, but I can't get version info,
If it was supplied as a macro will be much appreciated.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 13 Jun 2012 at 3:42

Fixed in 1.7.0.

Original comment by<EMAIL_ADDRESS>on 16 Oct 2012 at 11:30

Original comment by<EMAIL_ADDRESS>on 16 Oct 2012 at 11:30

Changed state: Fixed


NaNs cause hanging issue while training

Hey there, thank you for this awesome project!
At first I thought this was an issue with joblib but found out it had to do with NaNs in my dataframe.
Problem: when I try and train using NaNs the job basically just hangs and is pegging only one core. It seems like a better case would be to return an error or something because I had no idea it was just hanging since #7 isn't implemented yet.

I am running Fedora 30 with python 3.7.3.
Perhaps a simple check for NaNs in the array would be useful as an error to prevent this from happening? Lemme know how I can help and thank you! Great speech at Strata btw
Thank you
-Matt

Hi @hexgnu - glad you enjoyed the talk at Strata, we had a blast!
Thanks for reporting this. This is a bug on our part, we had some NA handling before release but decided to pull it.
You're welcome to submit a pull request throwing an exception on NaNs. Otherwise, we'll produce a fix on this around end of week.

Hi @hexgnu - This issue has been fixed in our latest 0.1.19 release.  Thanks again for reporting this to us.

InterpretML Team


Error initializing Wakatime: System.Net.WebException: The request was aborted: Could not create SSL/TLS secure channel.

Hi!
I'm sure I've done the installing procedure in the correct way but in logs I have this:
2018-09-14 19:35:52.3064|INFO|WakaTime.Logger|Initializing WakaTime v1.0.3 2018-09-14 19:35:53.0935|ERROR|WakaTime.Logger|Error initializing Wakatime: System.Net.WebException: The request was aborted: Could not create SSL/TLS secure channel.    at System.Net.WebClient.DownloadFile(Uri address, String fileName)    at System.Net.WebClient.DownloadFile(String address, String fileName)    at WakaTime.Downloader.DownloadCli(String url, String dir)    at WakaTime.WakaTime.OnConnection(Object application, ext_ConnectMode connectMode, Object addInInst, Array& custom)
using ssms 2014
can suggest the reason

I've installed python 3.7 (64bit) on Windows 10.  Restarted SQL Server and still get the same "Let's download and install python now?" pop up ....

Anyone ? Do I need a different version of Python installed?

I'm Using Windows Server 2012 R2, therefore probably problem is with whole Windows OSs

Just released a new version to support v2012+
Download and follow the instructions
https://github.com/gandarez/ssms-wakatime/releases/tag/v2.0.0


Issue 388

Closes #388
Fix for adding type="button" to <button> elements

Coverage remained the same at 97.448% when pulling 41dae28324789bc2af65713bebfc4ff74067af3d on tobiaswest83:issue-388 into 3ebbfde4d16ae97a52cb4f9e98e83b266ad0d986 on ngOfficeUIFabric:dev.


Action State and Buttons

Currently actions triggered by buttons are stateless. When you click to send a transaction the user is not aware if anything happened until the actions responds from the call being made. There could be a few approaches to solving this issue.
Solution Ideas:
In the case of a button,

Manage action states in such a way that when clicked the button state is disabled until a response is received and then re enabled.
Additionally produce an optional busy or spinner/loading modal that shows the user the transaction is processing and optionally automatically closes when done.

Kindly open this live chat to talk to the customer service directly on your issue https://direct.lc.chat/13898481/

thank you for the submission. please re request with the latest if desired to submit. thank you!


CE-3, CE-26

Description
Please provide a summary of the change and the issue fixed. Please include relevant context. List dependency changes.
Fixes # (issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

@afwilcox something weird is going on here, I thought it was ready, but I need to take another look


ESPDuino Connect/Reconnect

Hi,
I have been using an arduino + ESP-01 module and wired them as depicted on your github page.
Apparently an arduino sketch has been functional from a couple of days, and when I started it today, I get the following error
TCP: Reconnect to: m11.cloudmqtt.com:17478
DNS: found ip <IP_ADDRESS>
TCP: connecting...
TCP: Reconnect to m11.cloudmqtt.com:17478
Free memory
TCP: Connect to domain m11.cloudmqtt.com:17478
TCP: Reconnect to: m11.cloudmqtt.com:17478
DNS: found ip <IP_ADDRESS>
TCP: connecting...
TCP: Reconnect to m11.cloudmqtt.com:17478
Free memory
TCP: Connect to domain m11.cloudmqtt.com:17478
TCP: Reconnect to: m11.cloudmqtt.com:17478
DNS: found ip <IP_ADDRESS>
TCP: connecting...
TCP: Reconnect to m11.cloudmqtt.com:17478
Free memory
TCP: Connect to domain m11.cloudmqtt.com:17478
TCP: Reconnect to: m11.cloudmqtt.com:17478
DNS: found ip <IP_ADDRESS>
TCP: connecting...
TCP: Reconnect to m11.cloudmqtt.com:17478
I could not figure out why it would do this. Is this because some setting has changed on the broker?

@rupin Can you share the code?

I am using the debug version of the MQTT bridge. I reuploaded code on the ESP-01 very recently.
My arduino code is the same as the mqtt demo given in this repo.

I assume you tried pointing to another MQTT broker, just to rule out an issue with connecting to the one you are pointed  at (m11.cloudmqtt.com)? Did you try pointing to test.mosquitto.org:1833?

I have been trying to use espduino and borrow off the mqtt example.
I tried making the system persistent by making cleansession = 0 in the call to mqtt.begin
But the broker does not accept the client on reconnecting, and instead shows a new client connection.
We suspect that the persistent connection does not work. Is there something I am missing?
Also, which version of Mqtt does this library use?