plan for layout-padding, layout-margin

Is there a plan to include the "Layout Extra" options
as in angular1-material?
layout-padding, layout-margin, layout-fill
https://material.angularjs.org/latest/layout/options#layout-margin-padding-wrap-and-fill

Already exists:
https://github.com/angular/flex-layout

@aldocaetano
There is fxFlex, fxColumn, but I can't find -padding, -margin, -fill

For this i am using a pure css. I had no enough time for wait they finish :(

I did the same, I am just thinking if I should use fxLayoutPadding, fxLayoutMargin, etc
or there will be different naming scheme ..

Discussions for layout features so go over in https://github.com/angular/flex-layout


Next and Back buttons in onboarding too close to bottom

The pager and next and back buttons are too close to the bottom of the screen. It should be lifted up to a consistent 40px margin. This margin should be consistent across the entire app.

@jleach is this fixed as a result of this ticket? Shall this be moved to done?

Yes/No. I've bumped it up quite a bit to let shel do some testing. I need to put ti back to a sane location.

Ok, I've moved it to a better location.


Not working in android

confirmed. And no information provided about android support!

It's working on my Nexus 6P running Android 6.0.1. I'm still on RN 0.29.2 though so perhaps a recent update to RN broke it?

Any pull requests to fix this issue gladly accepted.

working on rn 30

@chromonav in my case, indicators are not showing in Android, and in iOS, they are showing out of bounds.

The problem seems to be indicatorOffset. It's original value is 250, which sends the indicators nearly to the top of the screen.

it is not working on my android project either, any help?

@javaxiu have you able to solve your issue. ?
Works seamlessly on "react-native": "^0.29.1"

don't work on react-native 0.38

this is maybe a react-native bug, I have read the document but can't find it

For anyone saying it "doesn't work", you'll probably have to give an explanation.
If your problem is "the view shows up empty", and your <Carousel> is in a <ScrollView>, you may be encountering https://github.com/facebook/react-native/issues/8088. Set the containing <ScrollView> to removeClippedSubviews={false} and things should work (though it's not a great solution for performance reasons).

@symmetriccurve  it turns out canvas was not suitable to display(too big or too small). try to print container's size on your console for debug. i haven't use this module for a while. wish it is helpful

Running rn0.41 I was able to merge both the Android and iOS component definitions. This resolved the issue for me. see PR #76

hi everyone, this project seems abandoned for a while. So I've make lots of improvements of this library. If you are looking for alternative solutions see here https://github.com/Canner/react-native-carousel-view. thanks!

@tioback Setting indicatorOffset to 10 dd'nt help me

@mikelambert I am using Carousel within ScrollView
But adding removeClippedSubviews={false} to ScrollView dd'nt help me either


Dependency is not resolved correctly

A subdependency is not correctly included by saucectl when uploading node_modules, which results in a runtime error due to missing dependencies:
Preparing npm environment
Detected node_modules, running npm rebuilding.

Rebuilding packages:
Error: Cannot find module 'pump'
Require stack:

    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\node_modules\got\node_modules\get-stream\index.js
    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\node_modules\got\node_modules\cacheable-request\src\index.js
    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\node_modules\got\dist\source\core\index.js
    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\node_modules\got\dist\source\as-promise\types.js
    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\node_modules\got\dist\source\as-promise\index.js
    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\node_modules\got\dist\source\create.js
    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\node_modules\got\dist\source\index.js
    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules\saucelabs\build\index.js
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>    D:\sauce-playwright-runner\2.14.0\bundle_project_\node_modules@cucumber\cucumber\bin\cucumber-js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:956:15)
    at Function.Module._resolveFilename.sharedData.moduleResolveFilenameHook.installedValue [as resolveFilename] (D:\sauce-playwright-runner\2.14.0\bundle_project\node_modules@cspotcode\source-map-support\source-map-support.js:811:30)
    at Function.Module.load (node:internal/modules/cjs/loader:804:27)
    at Module.require (node:internal/modules/cjs/loader:1028:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object. (D:\sauce-playwright-runner\2.14.0\bundle_project\node_modules\saucelabs\node_modules\got\node_modules\get-stream\index.js:3:14)
    at Module._compile (node:internal/modules/cjs/loader:1126:14)
    at Module.extensions..js (node:internal/modules/cjs/loader:1180:10)
    at Object.require.extensions. [as .js] (D:\sauce-playwright-runner\2.14.0\bundle_project\node_modules\ts-node\src\index.ts:1608:43)
    at Module.load (node:internal/modules/cjs/loader:1004:32)
    at Function.Module.load (node:internal/modules/cjs/loader:839:12)
    at Module.require (node:internal/modules/cjs/loader:1028:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object. (D:\sauce-playwright-runner\2.14.0\bundle_project\node_modules\saucelabs\node_modules\got\node_modules\cacheable-request\src\index.js:6:19)
    at Module._compile (node:internal/modules/cjs/loader:1126:14)

See https://app.eu-central-1.saucelabs.com/tests/bd09a8a633fb4ec3aa194c52547471ba
Originally posted by @samplehub in https://github.com/saucelabs/saucectl/issues/615#issuecomment-1297187622

@samplehub Please update @saucelabs/cucumber-reporter to version 0.4.0 or above. There's a fix in there that addresses some incorrect dependencies that saucectl falls victim to.

Will check and let you know shortly. Thanks Alex.

That worked like a charm. Thanks @alexplischke

@alexplischke ,  I ran a smoke category(@Smoke tag) which is a set of scenarios but they all ran serially in one cucumber test  .
So if 4 scenarios are tagged as Smoke , I was expecting to see 4 different test runs in sauce labs . If we have 4 VM's they would run concurrently , if not they shall queue up and run consecutively .. Isn't that ideal behaviour ??
Check out https://app.eu-central-1.saucelabs.com/tests/9852499920a2420ca101b76818c725cf - 2 passed & 2 failed
Please correct me if am wrong. Thanks.

Generally speaking, a suite in saucectl results in one job launched, unless otherwise stated, like sharding. For general questions, please our https://github.com/saucelabs/saucectl/discussions section 🙏


Move to Netlify

move the site to Netlify so you can get PR previews

also take advantage of Netlify DNS

@tsurdilo fyi

awesome!

@ricardozanini @tsurdilo @msmagnanijr
👋 Are you admins for the project? If not, who is?
I'm happy to walk you through creating a Netlify team and deployment. You need:

Someone with admin access to the serverlessworkflow.github.io repo
Someone with access to the project account's email to fill out Netlify's open source plan application

Let me know when you've got these squared away!

Hi @zacharysarah Chris Aniszczyk and myself have repo admin access. Ok, will look into this asap. Thanks

@zacharysarah I have a question regarding:
"Someone with access to the project account's email to fill out Netlify's open source plan application"
This form wants us to have:

So we need to put this link on our existing index page before we even get granted a license?

@tsurdilo you should be invited to https://app.netlify.com/sites/serverless-workflow/overview

@tsurdilo

So we need to put this link on our existing index page before we even get granted a license? Seems a little bit backwards

Looks like it's not necessary--the CNCF handles billing at the org level. Happy to be wrong about this one. 😄
Another question: what custom domain would you like? serverless-workflow.io? .com?

@zacharysarah
we currently have
serverlessworkflow.io
serverlessworkflow.org
would like to keep those two if possible.
Thanks!

@zacharysarah thats what I had questions on too :) Would you be able to set this up for us? I do not own the domain and @caniszczyk has helped us before to set up the redirect for serverlessworkflow.io/org to https://serverlessworkflow.github.io/
So how does this change our process, we still make changes here to the source and it will "magically" be picked up by Netlify and deployed there? Sorry never done this so I am confused.

@caniszczyk thanks! got the email and logged in. Many thanks.

@tsurdilo

So how does this change our process, we still make changes here to the source and it will "magically" be picked up by Netlify and deployed there?

Yes, basically.
I've added serverlessworkflow.io and serverlessworkflow.org to the domain list. We need to repoint the DNS servers on the CNCF side--I've opened a ticket to do that now, and will keep you updated on progress.

@tsurdilo

serverlessworkflow.io is successfully aliased and using Netlify DNS: http://serverlessworkflow.io/

serverlessworkflow.org is successfully aliased and using Netlify DNS: http://serverlessworkflow.org/

@zacharysarah just tested it. thank you!
will try making a change (to add some kubecon links) and see if it gets picked up :)

@zacharysarah oh wow you were right this just works now:

Thanks again, you guys rock!!

@tsurdilo Great! If you're happy, feel free to close this issue. 🙇

@zacharysarah @tsurdilo can I have access to the Netlify account as well? We need to create a custom subdomain for the VSCode Plugin services. This custom endpoint will redirect to a custom shared service deployed in the cloud, so we can change the route anytime just changing the DNS without impacting our users. Many thanks!

@tsurdilo @zacharysarah guys have you had the chance to take a look at the comment above? We really need to create the DNS entry to redirect to our services. Thanks!

@ricardozanini i think with netlify we could try to deploy that service on netlify itself and make it part of the site. Maybe that would be something we could look at ?

@tsurdilo not sure of netlify has the option to deploy an application in there. I haven't found any information about it... or I'm just too dumb to see. Can you give me access to the SW dashboard so I can take a look? As far as I understood, their service is related to DNS only, not a cloud provider itself.

Closing issue (long overdue :)) Thank you for allowing us to host our website!


Not working on fedora 25

Error: Command failed: cat /var/lib/dbus/machine-id
cat: /var/lib/dbus/machine-id: No such file or directory
After sudo ln -s /etc/machine-id /var/lib/dbus/machine-id works fine

same error on Archlinux Linux version 4.10.13-1-ARCH (builduser@tobias) (gcc version 6.3.1 20170306 (GCC) ) #1 SMP PREEMPT Thu Apr 27 12:15:09 CEST 2017

fixed in my fork - issuing a pull request to get the fix integrated
tested on ubuntu and fedora - please test on arch

Should be fixed in 1.1.6 2626bdde516047da16958a5dde5514940766dff2


HEIMAN SMHM-I1 - Can you please update the plugin to support it

Device description
It is a Heiman Motion sensor - recognized as SMHM-I1. Normally listed in devices js. Messages from device appear normal. I saw there are other HEIMAN devices marked as supported for this version of the plugin but not this one. I would be glad if you could prepare that.Thanks in advance.
MQTT message with device information
PUT here the device data you receive when publish message with topic
zigbee2mqtt/bridge/config/devices, as example
zigbee2mqtt:info  2020-03-28 10:52:56: HEIMAN Smart Motion Sensor (0x005043c917387fa0): SMHM-I1 - HEIMAN Smart motion sensor (EndDevice)

MQTT messages from device
Put here examples of MQTT messages with different states you've received from device, as example
zigbee2mqtt:info  2020-03-28 11:13:22: MQTT publish: topic 'zigbee2mqtt/HEIMAN Smart Motion Sensor', payload '{"occupancy":false,"tamper":false,"battery_low":false,"linkquality":110}'
zigbee2mqtt:info  2020-03-28 11:16:00: MQTT publish: topic 'zigbee2mqtt/HEIMAN Smart Motion Sensor', payload '{"occupancy":true,"tamper":false,"battery_low":false,"linkquality":115}'
zigbee2mqtt:info  2020-03-28 11:17:01: MQTT publish: topic 'zigbee2mqtt/HEIMAN Smart Motion Sensor', payload '{"occupancy":false,"tamper":false,"battery_low":false,"linkquality":115}'
zigbee2mqtt:info  2020-03-28 11:17:03: MQTT publish: topic 'zigbee2mqtt/HEIMAN Smart Motion Sensor', payload '{"occupancy":true,"tamper":false,"battery_low":false,"linkquality":110}'
zigbee2mqtt:info  2020-03-28 11:18:04: MQTT publish: topic 'zigbee2mqtt/HEIMAN Smart Motion Sensor', payload '{"occupancy":false,"tamper":false,"battery_low":false,"linkquality":115}'
zigbee2mqtt:info  2020-03-28 11:19:31: MQTT publish: topic 'zigbee2mqtt/HEIMAN Smart Motion Sensor', payload '{"occupancy":true,"tamper":false,"battery_low":false,"linkquality":113}'

MQTT state change commands
If device supports state change put MQTT topics + payload examples to change device state here

Done, could you please update plugin and check

Hi Stas. Thanks for the speedy action - of course it works. Closing the issue.


Add media_image_url for mpd to get cover art from web server

Description:
This adds support for cover art in mpd. A web server like nginx will need to have access to the mpd music directory and a image of the cover art needs exist in each album.
Pull request in home-assistant.io with documentation: https://github.com/home-assistant/home-assistant.io/pull/726
Example entry for configuration.yaml (if applicable):
media_player:
  platform: mpd
  server: <IP_ADDRESS>
  port: 6600
  baseurl: http://localhost/mpd/cover/
  covername: cover.jpg
  location: bedroom

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If code communicates with devices, web services, or a:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

We just got support to route media player images via Home Assistant in #2693 . You should be able to fetch the image directly from MPD.

With regards to #2693 that is very cool but MPD does not have support for cover art and know nothing about it. This is a bit of a hack to enable cover art. It is based off the same approach that MPDroid uses. With that said there is most likely a better way of doing this, I will look into #2693 further to see how I could use that to solve my problem.

Ah, I thought this was part of how MPD does it.
If it is a custom hack, I don't want to add it to Home Assistant as it will confuse the users.

@deonspengler As this method is a defacto standard I support the reopening of this pull request.


Command to list all known versions of a package

From a hackathon:
No easy way to list known versions of packages
Pub must have commands that show available versions of packages. Something along the lines of pub list unittest that shows the versions.

This issue has been moved to dart-lang/pub#504.


docs: add reference to Fastly's adapter/runtime/template

Now that #4841 has been merged, it will be possible to use Fastly's runtime/adapter/templates on the next release. Thanks all.
As suggested in https://github.com/remix-run/remix/pull/4860#issuecomment-1414383681, I've made a new PR that adds Fastly's runtime/adapter/template to docs as community contributions.
Most of the PR should be self-explanatory, but I have a few points to note:

Community Runtimes: There was no existing list of community runtimes.

For now I have created a new list and added to the bottom of the @remix-run/server-runtime README file, because this README had a list of the official runtimes and also described the specs of what a runtime has to implement.
The problem with putting it here though is that since this is part of a package's README, it will get out of sync with the remix website.
This content itself might actually be fit for putting inside the /docs in the same way as adapters.md, since it does talk about how Remix works and is useful for others who need to write runtime packages.

Community Templates: These was no existing list of community templates.

For now I have created a "Community Templates" section in the stacks.md page, since that was the page that talks about templates as tips on working with them.

Please let me know if there are thoughts on better ways of incorporating the above info into the docs.
These are docs-only (except for the server-runtime thing), so it's been branched from the main branch.

@MichaelDeBoey @machour have made updates:

alphabetized list of adapters
removed list of stacks and runtimes.

@MichaelDeBoey oof, I re-requested review from you and I didn't realize that would remove the requests for @brophdawg11 and @mcansh. can you add them back in?

@MichaelDeBoey bump
Can I get a re-review from you? I would be nice to be listed soon. =)


microk8s.status takes forever (almost 2 minutes) as user but not as sudo

Running microk8s.status takes forever while it comes instantly when executing with sudo. Other commands like microk8s.kubectl come instantly as normal user.
Before, I added the user to the microk8s group and relogged in.
on ubuntu 18.04

the same with microk8s.kubectl get all

it seemed to be my WSL Debian terminal. After restarting it all is smooth again. Super weird. So it's 99% not related to microk8s, so I'll close this ticket.

I also have encountered similar delays running microk8s.status, microk8s.inspect, or anything similar unless I run it with sudo.  In my case, I'm using Ubuntu 18.04 as the host on a laptop that should have ample resources.  I can run microk8s in a new VM, add my user to the microk8s group, and performance is normal, but for some reason the problem is present only on my laptop on what should be a fairly generic 18.04 desktop installation.  I wanted to add a comment to this issue just in case others encounter it.  Unlike the original poster, I'm not using WSL.
inspection-report-20191231_105717.tar.gz

Ah good to know. I haven't experienced this since I restarted my WSL Debian terminals but if I do I'll come back again.
I reopen the ticket, just to have more visibility and the issue doesn't show up anymore, I will close again!

Is running microk8s.kubectl also very slow?  Underneath microk8s.status is primarily running kubectl.  Thanks

Yes, running something like microk8s.kubectl get pods has similar slow performance (on the order of ~3 minutes) whereas running sudo microk8s.kubectl get pods takes less than one second.

@mrhoads thanks.  If you try to run an external kubectl (not microkk8s.kubectl) pointing to your microk8s cluster is it also slow?
You can get the kubeconfig by doing this microk8s.kubectl config view --raw > $HOME/.kube/config. After that you can use kubectl get
Can you also attach the inspect tarball?

for the record and what I remember,

microk8s.status was slow/never ending
microk8s.kubetctl get all was slow/never ending
just microk8s.kubectl was fine though
(all as normal user)

@balchua thanks for the info.  I've attached the inspect tarball.  I see similar differences in using kubectl as a snap or the kubectl binary from here.  It's not as long as running something like microk8s.status, but is still a noticeable difference (like 1.5 minutes for kubectl get pods as a regular user and ~1 second for sudo kubectl get pods).
Also just for what it's worth, I'm OK with just using  sudo for now, especially if @2jj isn't seeing issues now and if others aren't reporting it.  Thanks for your help.
inspection-report-20191231_200059.tar.gz

Thanks @2jj and @mrhoads these are all good info.

I couldn't reproduce this issue.  The performance of using microk8s.status with or without sudo is the same for me.

I'm having the same issue under 18.04.
inspection-report-20200103_150211.zip

Actually, wiping out ~/.kube (which was owned by root) fixed the problem.

Removing ~/.kube did the trick for me.  Something in ~/.kube/cache/discovery was owned by root.  kubectl responsiveness without sudo is now very quick.

Thanks @bobbyrward and @mrhoads for finding a solution.

thanks @mrhoads for this solution, I have run sudo chown -R $USER. ~/.kube and the magic happened!

Thank you everyone.  Was running microk8s in a ubuntu VM ... and was facing same issues.  Followed the above suggestion of chown - R $USER .kube .... and it fixed the long-timing issues immediately.

Hi, I am trying to configure microk8s on my Ubuntu machine to learn about Kubernetes.
I installed microk8s on Ubuntu 20.04 from snap (with the command sudo snap install microk8s --classic --channel=1.18/stable), following the instructions from microk8s website.
I added my user to microk8s group, chownd the .kube folder and logged out and logged in.
Then I run:
sudo microk8s status --wait-ready

and it just hangs, without showing any information, nor progress (I don't know if it's pulling the required containers or settings things up etc.).
Is there any way I can debug what's going on, or if I installed it the wrong way?
Thanks a lot for the help!

@lucacerone wait-ready will check two things a) the apiserver responds b) there is a node available.
Try microk8s.inspect and attach the tarball in an issue so we take a look.

@ktsakalozos microk8s status --wait-ready just hangs forever
`[appz@appz3 ~]$ microk8s inspect
Inspecting services
Service snap.microk8s.daemon-cluster-agent is running
Service snap.microk8s.daemon-flanneld is running
Service snap.microk8s.daemon-containerd is running
Service snap.microk8s.daemon-apiserver is running
Service snap.microk8s.daemon-apiserver-kicker is running
Service snap.microk8s.daemon-proxy is running
Service snap.microk8s.daemon-kubelet is running
Service snap.microk8s.daemon-scheduler is running
Service snap.microk8s.daemon-controller-manager is running
Service snap.microk8s.daemon-etcd is running
Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system information
Copy processes list to the final report tarball
Copy snap list to the final report tarball
Copy VM name (or none) to the final report tarball
Copy disk usage information to the final report tarball
Copy memory usage information to the final report tarball
Copy server uptime to the final report tarball
Copy current linux distribution to the final report tarball
Copy openSSL information to the final report tarball
Copy network configuration to the final report tarball
Inspecting kubernetes cluster
Inspect kubernetes cluster
WARNING:  IPtables FORWARD policy is DROP. Consider enabling traffic forwarding with: sudo iptables -P FORWARD ACCEPT
The change can be made persistent with: sudo apt-get install iptables-persistent
Building the report tarball
Report tarball is at /var/snap/microk8s/1608/inspection-report-20200915_235000.tar.gz
`

@rasoolasik could you attach the tarball created by microk8s inspect? Thank you.

this is happening as a root user in my case

Same here.
I executed the above commands, but kubectl returns very slowly, I used the root user to operate.


Photo from camera: crash Android 11

Hello
I have Android 11 installed on my Pixel 2.
Every time I try to select picture by camera the library crashes with the following stacktrace:
2020-09-30 11:08:05.444 7462-7462/com.landwirt E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.landwirt, PID: 7462
java.lang.RuntimeException: Unable to start activity ComponentInfo{com.landwirt/lv.chi.photopicker.utils.CameraActivity}: java.lang.IllegalStateException: intent.resolveActivity(packageManager) must not be null
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3449)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3601)
at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:85)
at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2066)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:223)
at android.app.ActivityThread.main(ActivityThread.java:7656)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
Caused by: java.lang.IllegalStateException: intent.resolveActivity(packageManager) must not be null
at lv.chi.photopicker.utils.CameraActivity.requestImageCapture(CameraActivity.kt:85)
at lv.chi.photopicker.utils.CameraActivity.onCreate(CameraActivity.kt:30)
at android.app.Activity.performCreate(Activity.java:8000)
at android.app.Activity.performCreate(Activity.java:7984)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1309)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3422)

Solution is to add the following lines to AndroidManifest

is there any way to provide this through the library or add this at least to readme

For others having the same issue, the lines to add to AndroidManifest are the following:
    <queries>
        <intent>
            <action android:name="android.media.action.IMAGE_CAPTURE" />
        </intent>
    </queries>

For others having the same issue, the lines to add to AndroidManifest are the following:
    <queries>
        <intent>
            <action android:name="android.media.action.IMAGE_CAPTURE" />
        </intent>
    </queries>

This is fixed by #24

This is fixed by #24

For others having the same issue, the lines to add to AndroidManifest are the following:
    <queries>
        <intent>
            <action android:name="android.media.action.IMAGE_CAPTURE" />
        </intent>
    </queries>

Nice ! its work for me

For others having the same issue, the lines to add to AndroidManifest are the following:
    <queries>
        <intent>
            <action android:name="android.media.action.IMAGE_CAPTURE" />
        </intent>
    </queries>

Nice ! its work for me

For others having the same issue, the lines to add to AndroidManifest are the following:
    <queries>
        <intent>
            <action android:name="android.media.action.IMAGE_CAPTURE" />
        </intent>
    </queries>

Works for me!


Issue 499 domain property

Description
Change domain property to be more consistent between submodels
Fixes #499
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Key checklist:

[ ] No style issues: $ flake8
[ ] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works
[ ] Any dependent changes have been merged and published in downstream modules

I definitely can't be bothered to do that one ...


Todo sample refactoring

New UI and new functionality

This change is

Changes Unknown when pulling b0574fab3175b69e512e34df01a7d02f4058429b on todo-refactoring into ** on master**.

Changes Unknown when pulling 5d7199371957c0d85ccd12b71b30b60a618c9ab0 on todo-refactoring into ** on master**.

Changes Unknown when pulling 3fa35636c85bf66189750b4ed11f9dc91fc0256f on todo-refactoring into ** on master**.

Changes Unknown when pulling 70e72e87da71864136f0308c2825ecb43b73a638 on todo-refactoring into ** on master**.

Changes Unknown when pulling 2aec9c153f85bd76d100998a52a5a54459b9c64a on todo-refactoring into ** on master**.

Changes Unknown when pulling 76b9502ef4745f01c466cb79642478c467b88a95 on todo-refactoring into ** on master**.


Add sql operations to python code

When adding all records we should automatically run create_tables if it hasn't been run
when adding new test data we should automatically run create_tables either way

Just make a method to run a SQL file on a database. Then I can just call it at the appropriate time


Read/Write nested dictionaries under FixedSizeList in IPC

Which issue does this PR close?

Closes #1609.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Codecov Report

Merging #1610 (1706cf8) into master (fd9cb23) will increase coverage by 0.00%.
 Consider uploading reports for the commit f645b64 to get more accurate results

@@           Coverage Diff           @@
##           master    #1610   +/-   ##
=======================================
  Coverage   82.95%   82.96%
=======================================
  Files         193      193
  Lines       55435    55460   +25
=======================================
+ Hits        45988    46010   +22
- Misses       9447     9450    +3

Impacted Files
Coverage Δ

arrow/src/ipc/writer.rs
80.27% <62.50%> (-0.25%)
:arrow_down:

arrow/src/ipc/reader.rs
89.43% <100.00%> (+0.26%)
: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 fd9cb23...f645b64. Read the comment docs.

Thank you @nevi-me


ESPHome 2023.9.1 - error "Platform missing. You must include one of the available platform key esp32, esp8266, rp2040, host"

I updated to ESPHome 2023.9.1.

It does mention that libretiny is natively supported now.
However get the error below:

LibreTiny ESPHome - no errors there

Try removing
framework:
  version: dev

Same error.
The error is one the bk72xx line....
Cleaning build files did not solve the issue:

That's a different error.  You need to replace ledc with libretiny_pwm moving forward.

And that worked :) - thanks.
Are any more changes like this? I don't think I saw anything documented for this.

Nope, that is pretty much it.


font is listview too small with responsive sizing

this might apply to compoents other than ListVew:   The list of elements is too small with responsive sizing -- check for both phones and tablets.
Here's an issue of someone reporting it, but also finding a clever way to patch around it using Javascript.   The real fix (in Java) presimably would not do this, but you can see how he's setting the scaling.
https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/mitappinventortest/_b6jmQGNugI/zBOZzuA_BAAJ

@halatmit Hello , I'am Mahima and I want to contribute in Component Projects( Improvements in Listview) . Can you please suggest how I start working on it ? from where I get the source code ?

Closed by cac9f88ce


[DebugInfo] Set the autogenerated bit to request recycled locations

IRGenDebugInfo expects to see an autogenerated bit before allowing reuse
of the last seen debug location for a new instruction.
rdar://28544248
(cherry picked from commit 5f6d8e8)

@swift-ci test and merge

@swift-ci test and merge

@swift-ci test and merge


Sync with GitLabs?

More of a question, can I use Gitlabs?

No, only github supported
https://github.com/shanalikhan/code-settings-sync/issues/256 will provide other options.


Add an option for keeping trailing semicolons in rule

See https://github.com/jakubpawlowicz/clean-css/issues/980.
I'm not certain I have implemented this in the way you wanted me too, so feel free to offer improvements
Thanks in advance

Sorry, don't want to pressure you, I am just not sure, whether you were notified of my last commit

No worries, I think I didn't see the change. Looks good now! Merging 👍

Thanks a lot

Thanks for committing your time to do this. It'll end up in 4.2 release although you can safely use master if needed.


[prim_flop_en/aes] Update interface and include scanmode

This adds a scanmode input to the prim_flop_en primitive.
The reason for this is that not all std cell libraries provide enable flops, hence we have to construct the functionality with a clock gate and a regular D flop.
In that case, the clock gate needs a scanmode enable.

Please ignore the first two commits in this PR.

@vogelpi if this looks good to you, please go ahead and merge it.

Addresses #6097

@vogelpi If you are ok with that, let's try and switch over to the mux version. I think it will probably be much less intrusive (although it may be a tick bigger and more power consuming).

@vogelpi If you are ok with that, let's try and switch over to the mux version. I think it will probably be much less intrusive (although it may be a tick bigger and more power consuming).
Edit: in that case we do not need this PR anymore.

I think it's significantly better for SCA-resistance to use a mux version rather than the clock-gate, if I followed all of these comments/diffs correctly.

Ah yes, now that you mention it @cdgori I remember that we this had also come earlier when discussing the LFSR implementation.
@msfschaffner do you need anything from me or inside AES for the mux version or can you do that completely inside the prim? Noticing that the gate count went up by roughly 10kGE over night I assume you already made that change, right?

Hey @vogelpi, the change is transparent. I basically instantiated a mux2 primitive by hand.
The gate count increase is likely due to the fact that both the flop and the mux receive a size_only constraint, thereby inhibiting some of the logic optimization.

@vogelpi If you are ok with that, let's try and switch over to the mux version. I think it will probably be much less intrusive (although it may be a tick bigger and more power consuming).
Edit: in that case we do not need this PR anymore.

I think it's significantly better for SCA-resistance to use a mux version rather than the clock-gate, if I followed all of these comments/diffs correctly.

yeah I didn't like the huge diff this CG version was introducing. the mux version is a bit bigger and more power hungry, though.

Okay, all good then. Thanks Michael.


How do I cancel my NetNewsWire subscription?

Don't bill me, Brent Simmons!
https://justuseapp.com/cancel/1480640210/netnewswire-rss-reader
We're entering a new age of AI-generated spam/fraud.
Anyway, I was just googling how to set NNW as the default reader. Any help with that?

This is the only way I know to change how. I don't know if it works in macOS 12.
Closing since this is a question, not an issue.

Ideally we’d take our old code for setting the default RSS reader and make that into a separate mini-app that just does that one thing. It’s not easy to find time, though, given everything else on our plate.


Implement HTTPS support

Ticket
#163
Changes

Add DNS records using Route 53.
Add a certificate using ACM.
Enabled HTTPS for the load balancer.
Connect everything together.
Add terraform variables for the dns zone.

Context for reviewers
Creating the DNS zone in Route 53 is out of scope of this as it needs to be permanently in the AWS account for testing purposes. That is because it is a subzone that is delegated from the nava-internal account.
Testing
Tested in platform-test:

AWS objects created:

@jamesbursa i think we should put the https certificate in the network layer similar to grants equity PR https://github.com/HHS/grants-equity/pull/536
we can use config parameters to configure how the certificate is created (i.e. whether it's imported or created by aws). take a look at the tech spec in here to see how i'm imagining it: https://github.com/navapbc/template-infra/milestone/4 and the

Done in #163


Co-annotation analysis - enrichment extension

Are any of the CL terms in the initial seed also in the object column of the enrichment table?
If so, repeat co-annotation analysis including everything that maps to this term, directly or via the enrichment table.
e.g.
Co-annotation (in this case, everything is 1:1 => predicate cluster_matches).
print(ade._anndata.obs[['author_cell_type', 'cell_type', 'cell_type_ontology_term_id']].drop_duplicates().to_csv(sep='\t'))

author_cell_type
cell_type
cell_type_ontology_term_id

naive B cell
naive B cell
CL:0000788

memory B cell
memory B cell
CL:0000787

gamma-delta T cell
gamma-delta T cell
CL:0000798

plasmablast
plasmablast
CL:0000980

regulatory T cell
regulatory T cell
CL:0000815

CD4-positive, alpha-beta memory T cell
CD4-positive, alpha-beta memory T cell
CL:0000897

CD8-positive, alpha-beta memory T cell
CD8-positive, alpha-beta memory T cell
CL:0000909

naive CD8+ T cell
naive thymus-derived CD8-positive, alpha-beta T cell
CL:0000900

naive CD4+ T cell
naive thymus-derived CD4-positive, alpha-beta T cell
CL:0000895

mucosal invariant T cell (MAIT)
mucosal invariant T cell
CL:0000940

TissueResMemT
memory T cell
CL:0000813

double-positive T cell (DPT)
double-positive, alpha-beta thymocyte
CL:0000809

double negative T cell (DNT)
double negative thymocyte
CL:0002489

TCRVbeta13.1pos
T cell
CL:0000084

enrichment table:
print(ade.minimal_slim_enrichment(['simple_term_enrichment']).to_csv(sep='\t'))

| s | s_label | p | o | o_label
-- | -- | -- | -- | -- | --
0 | CL:0000895 | naive thymus-derived CD4-positive, alpha-beta T cell | rdfs:subClassOf | CL:0000084 | T cell
1 | CL:0000897 | CD4-positive, alpha-beta memory T cell | rdfs:subClassOf | CL:0000084 | T cell
2 | CL:0000900 | naive thymus-derived CD8-positive, alpha-beta T cell | rdfs:subClassOf | CL:0000084 | T cell
3 | CL:0000909 | CD8-positive, alpha-beta memory T cell | rdfs:subClassOf | CL:0000084 | T cell
4 | CL:0000940 | mucosal invariant T cell | rdfs:subClassOf | CL:0000084 | T cell
5 | CL:0002489 | double negative thymocyte | rdfs:subClassOf | CL:0000084 | T cell
6 | CL:0000897 | CD4-positive, alpha-beta memory T cell | rdfs:subClassOf | CL:0000813 | memory T cell
7 | CL:0000909 | CD8-positive, alpha-beta memory T cell | rdfs:subClassOf | CL:0000813 | memory T cell
8 | CL:0000798 | gamma-delta T cell | rdfs:subClassOf | CL:0000084 | T cell
9 | CL:0000809 | double-positive, alpha-beta thymocyte | rdfs:subClassOf | CL:0000084 | T cell
10 | CL:0000813 | memory T cell | rdfs:subClassOf | CL:0000084 | T cell
11 | CL:0000815 | regulatory T cell | rdfs:subClassOf | CL:0000084 | T cell
T-Cell; CL:0000084  is in original seed (from obs.cell_type...) and in the object column of the enrichment table.
Co-occurence analysis without enrichment => TCRVbeta13.1pos cluster_matches T cell (CL:0000084)
Co-occurence analysis without enrichment =>
TCRVbeta13.1pos subClusterOf T cell (CL:0000084)
regulatory T cell (CL:0000815) subClusterOf T cell( CL:0000084)
(Note: working through like this makes it clear that we need to start making the distinction between ontology terms (where subClassOf is appropriate) and cell sets defined by tagging with ontology terms (cell sets are related by subClusterOf).  This will be important for the model, but needs to be handled carefully for user facing assertions.  For now, I think we need to gloss this distinction.)

CC @hkir-dev

Co-occurence analysis without enrichment =>
TCRVbeta13.1pos subClusterOf T cell (CL:0000084)
regulatory T cell (CL:0000815) subClusterOf T cell( CL:0000084)

@dosumis did you mean with or cluster_matches?

Are any of the CL terms in the initial seed also in the object column of the enrichment table?

Isn't this the default for the simple enrichment? Am I missing something here?
First, I thought you wanted apply some sort of transitive relationship between author cell types with the other cell types, i.e. regulatory T cell rdfs:subClassOf TCRVbeta13.1pos. It is not clear what kind of output do you wish to see with this report.


Update the content of content.ts to the corresponding template content.

Feature Request
When initializing a project, if a certain framework is selected, the corresponding content entry content may be changed to the corresponding framework code to be more friendly.
https://wxt.dev/guide/content-script-ui.html#integrated
Is your feature request related to a bug?
N/A
What are the alternatives?
N/A
Additional context
N/A

I don't plan on adding additional features to the templates. I will add support for community templates if there's interest.
The built-in templates need to stay minimal.

Makes sense.


Subtitle per track separation preview misleading when performing Export

I've assigned different Anki fields to different subtitle tracks. These settings are applied when updating the last card. However, when I'm trying to Export a new card, both tracks appear under the Sentence field, and the 2nd track field is empty.
This seems to be mainly a visual glitch, as when the card is exported, the tracks are assigned correctly
https://github.com/killergerbah/asbplayer/assets/6875790/9e8ec060-8fed-4f7a-a969-b00f9b18bbcf
Tested on commit f14bd91fa85f1fb65cae6bda604613b0e7ae82fd

Ah I see, I've missed the 3 dots in the UI. But that's only part of the problem - in my video you can see Sentence Translation is mapped in Anki, but that field appears blank on the Export window popup

Sorry, I'm not seeing how that is not the same problem as the fields being hidden. For what it's worth, any of the default fields with special behavior (like sentence, word, track 1, track 2, etc) will always be populated as long as they are showing in the dialog. But they might not actually be used in the actual anki card.

Yes you’re right, I’ve hidden the default fields so it’s no longer confusing. Thank you so much!


Export Annotations to COCO format

Hey Britefury,
Great app, is there an option to convert the output annotation labels to COCO format?
Kind Regards,
Edwin

hi! how did you get this done?


Validation discrepancy between classpath model and imported model

When using the smithy-gradle-plugin to build/validate a model, if that model is pulled in via the classpath by existing in a model/ directory, then the validation errors for that model are different than if that model is explicitly imported via the smithy-build.json file.
Here's an example minimal model that should produce a DANGER error (it doesn't like teapots, unless something changes due to #1386):
$version: "1.0"
namespace aws.protocoltests.misc

use aws.protocols#restJson1
use smithy.test#httpRequestTests
use smithy.test#httpResponseTests

@restJson1
service MiscService {
    operations: [ResponseCodeRequiredOperation],
}

@http(method: "GET", uri: "/responseCodeRequiredOperation", code: 418)
operation ResponseCodeRequiredOperation {
    input: EmptyStructure,
    output: ResponseCodeRequiredOutput,
}

structure EmptyStructure {}

@output
structure ResponseCodeRequiredOutput {
    @required
    @httpResponseCode
    responseCode: Integer,
}

This model, if pulled in by existing in the models/ directory, emits a WARN due to the IDL version being 1, but does not emit the DANGER message. If instead, it is placed outside the models/ directory and referenced directly from the imports section of smithy-build.json, then the DANGER message is emitted and the build fails.

then the DANGER message is emitted and the build fails.

Worth mentioning too that the WARN is not emitted, only the DANGER message.

I wasn't able to reproduce this by moving a model between an explicit import vs placing it in the model/ directory, but that might be because I don't understand the full setup of how to get this to fail.
You mentioned the models/ directory, but the Gradle plugin actually looks for the model/ directory. Is it possible that is the root cause of the issue here (that is just didn't load your model)? One quick way to check this kind of thing is note the number of shapes that are validated (e.g., you might see something likeFAILURE: Validated 299 shapes (DANGER: 1, WARNING: 2, NOTE: 2) output from the plugin).
One other consideration that may or may not be relevant here: when generating a JAR from a Gradle project, the Smithy Gradle plugin will validate that the inputs it's given are correct as a first pass at validation. It also then does another pass at validating using only the generated JAR and your implementation/api dependencies to ensure that the generated JAR is valid and doesn't, say, refer to unknown shapes.

Worth mentioning too that the WARN is not emitted, only the DANGER message.

We removed the IDL 1.0 warning since it was so noisy, so that might explain this.

Sorry, models/ (plural) is a typo. We're using a model/ directory.

Ah ok, good to know. When trying to reproduce this with Smithy (1.26.4) and the Gradle plugin (0.6.0 or HEAD), I see the DANGER event when your example model is an imports model or found in the model/ directory. The output is different though, which I don't think is a good DX. For example, when loaded via model/, I get the formatted text like:
DANGER: aws.protocoltests.misc#ResponseCodeRequiredOperation (HttpResponseCodeSemantics)
     @ /Users/dowling/projects/smithy-test-1387/model/foo.smithy
     |
  13 | @http(method: "GET", uri: "/responseCodeRequiredOperation", code: 418)
     | ^
     = Expected an `http` code in the 2xx range, but found 418

When loaded via smithy-build.json/imports, I get:
[DANGER] aws.protocoltests.misc#ResponseCodeRequiredOperation: Expected an `http` code in the 2xx range, but found 418 | HttpResponseCodeSemantics /Users/dowling/projects/smithy-test-1387/foo.smithy:13:1
[WARNING] aws.protocoltests.misc#ResponseCodeRequiredOperation: The output of this operation should target a shape that starts with the operation's name, 'ResponseCodeRequiredOperation', but the targeted shape is `aws.protocoltests.misc#ResponseCodeRequiredOutput` | OperationInputOutputName /Users/dowling/projects/smithy-test-1387/foo.smithy:14:1
[WARNING] aws.protocoltests.misc#ResponseCodeRequiredOperation: This operation uses the `GET` method in the `http` trait, but is not marked with the readonly trait | HttpMethodSemantics /Users/dowling/projects/smithy-test-1387/foo.smithy:13:1

Which still fails the build for me.
Here's my setup:
~/projects/smithy-test-1387〉tree
.
├── build.gradle.kts
├── model
│   └── foo.smithy
├── settings.gradle.kts
└── smithy-build.json

1 directory, 4 files

~/projects/smithy-test-1387〉cat smithy-build.json
{
  "version": "1.0",
  "imports": [
    //"foo.smithy"
  ]
}

~/projects/smithy-test-1387〉cat settings.gradle.kts
rootProject.name = "test-1387"

pluginManagement {
    repositories {
        gradlePluginPortal()
        mavenCentral()
    }
}

~/projects/smithy-test-1387〉cat build.gradle.kts
plugins {
    id("software.amazon.smithy").version("0.6.0")
}

repositories {
    mavenCentral()
}

dependencies {
    implementation("software.amazon.smithy:smithy-model:[1.0, 2.0[")
    implementation("software.amazon.smithy:smithy-aws-traits:[1.0, 2.0[")
    implementation("software.amazon.smithy:smithy-protocol-test-traits:[1.0, 2.0[")
}

~/projects/smithy-test-1387〉cat model/foo.smithy
$version: "1.0"
namespace aws.protocoltests.misc

use aws.protocols#restJson1
use smithy.test#httpRequestTests
use smithy.test#httpResponseTests

@restJson1
service MiscService {
    operations: [ResponseCodeRequiredOperation],
}

@http(method: "GET", uri: "/responseCodeRequiredOperation", code: 418)
operation ResponseCodeRequiredOperation {
    input: EmptyStructure,
    output: ResponseCodeRequiredOutput,
}

structure EmptyStructure {}

@output
structure ResponseCodeRequiredOutput {
    @required
    @httpResponseCode
    responseCode: Integer,
}

Are there other settings you're passing to the plugin?

I looked through our build.gradle.kts files and didn't see any additional settings for the plugin. Perhaps the situation has improved since this was originally filed.
I don't have time right this moment, but I can try to repro it again in smithy-rs with its upgraded Smithy version. We could also see if things have changed by trying out the above with an August release of Smithy.

I'm not able to repro this anymore in smithy-rs with Smithy 1.26.2.


⚠ Bofrost.at has degraded performance

In 8dbe5ed, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1528 ms

Resolved: Bofrost.at performance has improved in e641e20 after 10 minutes.


Pipeline updates to improve MPI CI jobs.

Move some MPI jobs from AMDCI to nla-gpu.
SLURM_GRES=4 for horeka MPI jobs.

Closes #1174

If the pipeline works on nla-gpu, then it closes the issue, because that means the problem was indeed GPU oversubscription (the 3 MPI processes were sharing the one Radeon VII of amdci and that made things very slow).
Another comment, currently I believe we are not using this parameter: https://github.com/ginkgo-project/ginkgo/blob/develop/.gitlab/scripts.yml#L32 to use it, it should be BUILD_MPI instead of GINKGO_BUILD_MPI I think. So is this needed at all?

Yes, oversubscription was indeed the issue. I removed all remaining MPI jobs from the AMDCI machine and moved them to the nla-gpu machine.


Restarting Sensu-Client results in stale checks.

Expected Behavior

I make a configuration change (usually via ansible) and restart the sensu-client on a node (typically via ansible, but I've also repro'd the issue using a bash script, ssh one liners with -t flag, and manually ssh'ing in myself and restarting the service.)
The service restarts and check requests/results keep being pushed/pulled, with new check results being added to the client.

Current Behavior

I make a configuration change and restart sensu-client.
Most of the time (I'd say at least 80%) the client gets stuck on "Stopping reactor" and no sign in the log that it started back up or did anything.
If I fully stop the sensu-client and wait ~90 seconds, and then start it, about 50% of the time it recovers.
The other 50% of the time I have to do stop/90 second wait/start/10 second wait/restart before it finally starts working again.
This makes managing my 80+ sensu clients via ansible completely untenable.

Possible Solution
I'm not sure what is causing the issue. Are child processes not being killed? Are sockets/ports not being released on a server stop/restart?
I tried adding timeouts to all my checks, so far things seem okay, but this is a huge issue and our monitoring basically goes down any time I make config changes, and I have to spend sometimes an entire day manually fixing it by restarting the sensu-client manually on each affected node (I've had 40+ go out on me at a time) until it works again.
Steps to Reproduce (for bugs)

Set up a 2-node HA sensu server setup.
Set up one or more subscriptions with checks. Assign TTL values to all the checks.
Set up the sensu client on several nodes and assign them to these subscriptions. My particular setup has each client running ~10 sub checks and ~2 standalone checks.
Let the data populate.
Run sensu-client restart however you want. Ansible, ssh, etc.
The checks will go stale and you'll get TTL events in sensu until you go back in and restart the service or stop/start/restart until it works again. I had to manually tail logs to make sure it was actually coming back up.

Context
I'm trying to manage 80+ nodes as sensu clients via ansible but if I can't restart the client on any of them without bringing my whole system essentially down (and not paging people) it's a no-go.
Your Environment

Sensu version used: 1.2.0, tried downgrading to 1.1.3 to see if that would resolve the issue. It did not.
Active Sensu extensions (e.g. WizardVan 0.1.0): None
Operating System and version (e.g. Ubuntu 14.04): Ubuntu 14.04 servers and CentOS 6.5/6.8/7.3 and RHEL 6.5/6.8 clients
Transport and runtime versions (e.g. RabbitMQ 3.6.2 on Erlang 18.2): RabbitMQ 3.6.2 on Erlang 20

{"timestamp":"2018-02-15T09:43:25.039601-0800","level":"warn","message":"received signal","signal":"TERM"}
{"timestamp":"2018-02-15T09:43:25.039787-0800","level":"warn","message":"stopping"}
{"timestamp":"2018-02-15T09:43:25.040054-0800","level":"info","message":"completing checks in progress","checks_in_progress":[]}
{"timestamp":"2018-02-15T09:43:25.540817-0800","level":"info","message":"closing client sockets"}
{"timestamp":"2018-02-15T09:43:25.541198-0800","level":"warn","message":"stopping reactor"}
{"timestamp":"2018-02-15T09:47:08.703004-0800","level":"warn","message":"loading config file","file":"/etc/sensu/config.json"}
{"timestamp":"2018-02-15T09:47:08.703113-0800","level":"warn","message":"loading config files from directory","directory":"/etc/sensu/conf.d"}
{"timestamp":"2018-02-15T09:47:08.703144-0800","level":"warn","message":"loading config file","file":"/etc/sensu/conf.d/checks.json"}
{"timestamp":"2018-02-15T09:47:08.703185-0800","level":"warn","message":"config file applied changes","file":"/etc/sensu/conf.d/checks.json","changes":{"checks":{"check_autotest_nodes":[null,{"interval":60,"command":"/opt/sensu/embedded/bin/fdsjfkldsj-endpoint-check.rb -u 'http://localhost/endpoints/fiodhsiofhds' -i 30 -m 45 -t heartbeat -c 0 -w 0","handlers":["staytus","slack"],"standalone":true}],"check_fdsfdsfds_nodes_mm":[null,{"interval":60,"command":"/opt/sensu/embedded/bin/fdsjfkldsj-endpoint-check.rb -u 'http://localhost/endpoints/fiodhsiofhds' -t maintenance_mode -w 25 -c 25","handlers":["staytus","slack"],"standalone":true}]}}}
{"timestamp":"2018-02-15T09:47:08.703234-0800","level":"warn","message":"applied sensu client overrides","client":{"name":"HOSTNAME_REDACTED","address":"IP_REDACTED","subscriptions":["automation","client:HOSTNAME_REDACTED"]}}
{"timestamp":"2018-02-15T09:47:08.705184-0800","level":"warn","message":"loading extension files from directory","directory":"/etc/sensu/extensions"}
{"timestamp":"2018-02-15T09:47:08.705246-0800","level":"info","message":"configuring sensu spawn","settings":{"limit":12}}
{"timestamp":"2018-02-15T09:47:08.705274-0800","level":"debug","message":"setting eventmachine threadpool size","size":22}
{"timestamp":"2018-02-15T09:47:08.719306-0800","level":"debug","message":"connecting to transport","name":"rabbitmq","settings":{"host":"SENSU_HOST_REDACTED","port":"PORT_REDACTED","vhost":"/VHOST_REDACTED","user":"USER_REDACTED","password":"PASSWORD_REDACTED","ssl":{"cert_chain_file":"/etc/sensu/ssl/cert.pem","private_key_file":"/etc/sensu/ssl/key.pem"}}}
{"timestamp":"2018-02-15T09:47:08.822810-0800","level":"debug","message":"transport connection open"}
{"timestamp":"2018-02-15T09:47:08.823034-0800","level":"debug","message":"binding client tcp and udp sockets","options":{"bind":"<IP_ADDRESS>","port":3030}}
{"timestamp":"2018-02-15T09:47:08.823348-0800","level":"debug","message":"binding client http socket","options":{"bind":"<IP_ADDRESS>","port":3031}}
{"timestamp":"2018-02-15T09:47:08.823472-0800","level":"debug","message":"scheduling keepalives"}
{"timestamp":"2018-02-15T09:47:08.823566-0800","level":"debug","message":"publishing keepalive","payload":{"name":"HOSTNAME_REDACTED","address":"IP_REDACTED","subscriptions":["automation","client:HOSTNAME_REDACTED"],"version":"1.1.3","timestamp":1518716828}}
{"timestamp":"2018-02-15T09:47:08.823732-0800","level":"debug","message":"subscribing to client subscriptions"}
{"timestamp":"2018-02-15T09:47:08.823807-0800","level":"debug","message":"subscribing to a subscription","subscription":"automation"}
{"timestamp":"2018-02-15T09:47:08.823966-0800","level":"debug","message":"subscribing to a subscription","subscription":"client:HOSTNAME_REDACTED"}
{"timestamp":"2018-02-15T09:47:08.824071-0800","level":"debug","message":"scheduling standalone checks"}
{"timestamp":"2018-02-15T09:47:28.824020-0800","level":"debug","message":"publishing keepalive","payload":{"name":"HOSTNAME_REDACTED","address":"IP_REDACTED","subscriptions":["automation","client:HOSTNAME_REDACTED"],"version":"1.1.3","timestamp":1518716848}}
So it looks like the list is empty. On restart, the log seems to indicate all the right subscriptions, etc but then the client ONLY runs standalone checks... I'll join the slack and try to work this out.

This related sensu-client code in question
"completing checks in progress": https://github.com/sensu/sensu/blob/e895d5958f1a3857601d3ef321f59dfc4d9e8f1f/lib/sensu/client/process.rb#L639-L642
"stopping reactor": https://github.com/sensu/sensu/blob/e895d5958f1a3857601d3ef321f59dfc4d9e8f1f/lib/sensu/daemon.rb#L215-L216
What's really bizarre is that EM::stop_event_loop should be immediately ending to process.

For what it's worth, we had a very similar problem -- all the checks were defined and the subscriptions were set up correctly, but for some reason Sensu stopped scheduling the checks.
The cause was that we had a rogue sensu-server process running in our environment, connected to the same transport as the real server. The rogue server had no checks defined and apparently when that happens no checks are scheduled even though there's a server that contains all the check definitions.
Removing the rogue server process brought everything back into sync. I hope this helps some other Googlers!

As pointed out above there are already multiple forms of protections against a rouge sensu server from joining a cluster:

rabbitmq username/password
rabbitmq TLS certs
rabbitmq vhost ACL's

Depending on how your setup is I would also suggest maybe restricting your firewalls (iptables, aws security groups, etc) to prevent nodes outside of the expected subnets from talking to RMQ.
My guess would be that you need to update your automation to not by default give credentials and certs that work in a real environment but should randomly/statically generate it for development/ci.

@anherrera have you been able to resolve this issue? Please let us know. If we haven't seen any update on this scenario by next week I think we'll close the issue due to inactivity.

I think I narrowed the issue down to rabbitMQ configuration being off. I'll let you know once new configs are in place whether this continues. Thanks for the followup.

I'm going to go ahead and close this for lack of update.


How can i use Jackson CamelCase Property Naming  Strategy?

Hi, how can I use this jackson feature?
ObjectMapper mapper = new ObjectMapper();
 mapper.setPropertyNamingStrategy(PropertyNamingStrategy.CAMEL_CASE_TO_LOWER_CASE_WITH_UNDERSCORES);

Thanks, was helpful.


🐛 Bug ➾ in the Taco-shop Scaffold docs, there is a linter error for the suggested aps.tsx code

🚥 Status (Internal Taqueria Team Use Only)

[x] 🔬 Investigated and Verified
[ ] ⚗ Solution Identified and Designed
[ ] 🧫 Dev Implementation of Fix
[ ] 🧪 Fix Tested and Validated
[ ] 🏆 PR Merged

🆘 What happened?
When trying to set up the Taco-Shop scaffold in the React setup, I get a linter warning

🆘 Steps to Reproduce?

go to https://taqueria.io/docs/scaffolds/taco-shop/
taq scaffold https://github.com/ecadlabs/taqueria-scaffold-taco-shop taco-shop
cd taco-shop
Insert the Contract Address
When you add the code suggested in the doc, there is a linter waning

🪵 Relevant log output
No response
🐘 How impactful is this bug?
🤯 Major disruption
⏱ Prevalance
everytime
💻 Operating System
Windows -> Linux VM
🕸 System Architecture
x64
🌮 Taqueria Version
v0.23.6-rc.1
🌿 Node.js Version
v16.13.1
☎ Contact Details
<EMAIL_ADDRESS>Code of Conduct

[X] I agree to follow this project's Code of Conduct

It turns out you substitute the contract for the get code, not add it. So the doc needs some clarification


Error: Cannot ready property 'queryRunner' of undefined

Here is my controller code to create a new Node:
`export default class JobsController {
 public createJob = async (request: express.Request, response: express.Response): Promise<void> => {
     await Jobs.createOne({
        company: request.body.company,
        title: request.body.title,
        location: request.body.location,
        salary: request.body.salary,
        seniority: request.body.seniority,
        topApplicant: request.body.topApplicant
    })
        .then(res => response.json(res))
        .catch(error => console.error(error.message));
}`

Thanks for submitting this issue. Could you include the code you used for creating the Neogma instance, the Jobs model and the folder structure?

It may be possible that you're importing this file before creating the Neogma instance.
This results in the error you reported:
import { JobsController } from "./src/controllers/Jobs";

export const neogma = new Neogma(
  {
    url: "bolt://localhost:7687",
    username: "neo4j",
    password: "password",
  },
  {
    logger: console.log,
    encrypted: false,
  }
);

while this doesn't:
export const neogma = new Neogma(
  {
    url: "bolt://localhost:7687",
    username: "neo4j",
    password: "password",
  },
  {
    logger: console.log,
    encrypted: false,
  }
);

import { JobsController } from "./src/controllers/Jobs";

that's why I recommend creating the Neogma instance as soon as you've run dotenv.config(), before importing anything else!

Thanks, @themetalfleece  that worked for me.

Added documentation here: https://themetalfleece.github.io/neogma-docs/docs/Models/Defining-a-Model#importing-neogma

@themetalfleece first of all, thank you and congrats for the library - it helps a lot.
I'm having an issue while integrating with Nest.js. I'm trying to decouple everything to keep my app "testable" but it is causing Cannot read property 'queryRunner' of undefined error by the time the code bootstrap.
I tried to create the connection early, during the app bootstrap but it didn't work either - the dependency injection doesn't even have a chance to forcefully inject the connection via a module's custom provider.
What would be your suggestion for scenarios where we rely on DI to create/manage the class instances and DB connections as well?
Our scenario also requires to connect to different Neo4j instances which means different connection strings are managed dynamically (via service logic) to determine which connection string the DI needs to inject in the constructor and then to create a Neo4j connection.
How can we accomplish something like that?
I will leave some snippets of the code for your reference:
app.module.ts
@Module({
    imports: [
        ConfigModule.forRoot({ isGlobal: true }),
        NeogmaModule.forRootAsync({
            imports: [ConfigModule],
            inject: [ConfigService],
            useFactory: (configService: ConfigService): Neo4jConfig => ({
                scheme: configService.get('NEO4J_SCHEME') || 'bolt+s',
                host: configService.get('NEO4J_HOST') || 'localhost',
                port: configService.get('NEO4J_PORT') || '7687',
                username: configService.get('NEO4J_USERNAME') || 'neo4j',
                password: configService.get('NEO4J_PASSWORD') || 'neo4j',
                database: configService.get('NEO4J_DATABASE') || 'default',
            })
        }),
        AutomapperModule.withMapper(),
        CommonModule,
    ]
})
export class ApplicationModule { }

neogma.service.ts
import { Driver, int } from 'neo4j-driver'
import { Injectable, Inject, OnApplicationShutdown, Logger } from '@nestjs/common';
import { NEOGMA_CONFIG, NEOGMA_DRIVER } from './neo4j.constants';
import { Neo4jConfig } from './neo4j-config.interface';
import { Neogma, QueryRunner } from 'neogma';
import { LoggerService } from '../common/provider/logger.service';

@Injectable()
export class NeogmaService implements OnApplicationShutdown {

    private readonly logger = new Logger(NeogmaService.name);

    public static neogmaConnection: Neogma;
    neogma: Neogma;
    runner: QueryRunner;

    constructor(
        @Inject(NEOGMA_CONFIG) private readonly config: Neo4jConfig,
        @Inject(NEOGMA_DRIVER) private readonly driver: Driver
    ) {
        this.logger.debug('Initialized');

        this.runner = new QueryRunner({
            driver: this.driver,
            logger: LoggerService || console.log
        });

    }

    public static async createNeogmaConnection(config: Neo4jConfig) {

        return new Neogma({
            url: `${config.scheme}://${config.host}:${config.port}` || 'bolt://<IP_ADDRESS>:7687',
            username: config.username || 'neo4j',
            password: config.password || 'Test1*'
        },
            {
                logger: console.log,
                encrypted: false,
                disableLosslessIntegers: true
            })
    }

    getDriver(): Driver {
        return this.driver;
    }

    getNeogma(): Neogma {
        NeogmaService.neogmaConnection = this.neogma;
        return this.neogma;
    }

    getConfig(): Neo4jConfig {
        return this.config;
    }

    int(value: number) {
        return int(value)
    }

    onApplicationShutdown() {
        return this.driver.close()
    }

}

neogma.module.ts
import { DynamicModule, Module, Provider } from '@nestjs/common';
import { NEOGMA_CONFIG, NEOGMA_DRIVER } from './neo4j.constants';

import { ConfigModule } from '@nestjs/config';
import { Neo4jTransactionInterceptor } from './neo4j-transaction.interceptor';
import { NeogmaService } from './neogma.service';
import { Neo4jConfig } from './neo4j-config.interface';
import { NeogmaConnection } from '../../neogma.config';

@Module({
    exports: [
        NeogmaConnection,
        NeogmaService
    ]
})
export class NeogmaModule {

    static forRoot(config: Neo4jConfig): DynamicModule {

        return {
            module: NeogmaModule,
            global: true,
            providers: [
                {
                    provide: NEOGMA_CONFIG,
                    useValue: config,
                },
                {
                    provide: NEOGMA_DRIVER,
                    inject: [NEOGMA_CONFIG],
                    useFactory: async (factoryConfig: Neo4jConfig) => {
                        return NeogmaService.createNeogmaConnection(factoryConfig)
                    },
                },
                NeogmaService,
            ],
            exports: [
                NeogmaService,
                Neo4jTransactionInterceptor,
            ]
        }
    }

    static forRootAsync(configProvider): DynamicModule {
        return {
            module: NeogmaModule,
            global: true,
            imports: [ConfigModule],

            providers: [
                NeogmaService,
                {
                    provide: NEOGMA_CONFIG,
                    ...configProvider
                } as Provider<any>,
                {
                    provide: NEOGMA_DRIVER,
                    inject: [NEOGMA_CONFIG],
                    useFactory: async (config: Neo4jConfig) => NeogmaService.createNeogmaConnection(config),
                },
            ],
            exports: [
                NeogmaService,
            ]
        }
    }

}

@jotamorais Hey Jonathas, thanks for your message and the code snippets.
This is very interesting and I want to try Neogma with Nest.js and provide a guide and a module for it.
This is gonna take a bit though, but I'll let you know as soon as I have news, or if I need any more info.
Thanks!


Support alignment of protein sequences containing "*"

I'm using the C API of block-aligner to align protein sequences from UniProt database. There are *s in some protein sequences. Currently using block-aligner to align sequences containing * will cause a Segmentation Fault. Although the users can resolve it by mapping * to other supported chars, it would be nice if we can support * internally! :)

I'm not sure if * will every be directly supported internally. It will always have to be mapped to some character that fits within the scoring matrix, so SIMD lookups can be done. Right now, the amino acid matrix supports alphabetical characters A-Z.
There are a couple of ways this could be solved:

An unused letter like J could be used to represent *, like what you said. On the Rust side, the scores in the amino acid matrix can be cloned and changed, but this is not yet exposed in the C API. Without changing the scores, matches and mismatches with J incur a score of -128.
A letter not part of the original 20 amino acids but still has predefined scores can be used. For example, * can be translated to X.
Letters are required to be mapped to numerical values 0-20. Currently, all letters are mapped to 0-25 by simply subtracting A.


SDK update for TemplateSpecs GA API 2021-05-01

Regenerated Resources SDK for Templatespecs GA API version. No changes apart from the API version.
PR for updated swagger spec - https://github.com/Azure/azure-rest-api-specs/pull/14065
All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

@JonathanCrd Can you please review this PR? Thanks.


Having problem when import createLocalVue

I write
import { shallowMount, createLocalVue } from '@vue/test-utils';
in my *.spec.ts file, and when I run the test file with jest, it report the error that
Module '"@vue/test-utils"' has no exported member 'createLocalVue'
My @vue/test-utils version is ^2.0.0-rc.12, and my jest version is ^26.6.3, and I am using vue3.

Hi! 👋
This is because Vue Test Utils 2 no longer needs a local Vue instance, so localVue is removed.
Here are the docs about the change: https://next.vue-test-utils.vuejs.org/migration/#no-more-createlocalvue
Hope it helps!


Fix unencrypted incremental streams

Add the raw flag to incremental ZFS send commands too.
Missed with #46.

Thanks for catching this!


Implement commit history traversal

Currently, the tool only analyses and summarises the latest commit for a repo.
Modify the inspect_repo() function to traverse the entire commit history of the specified branch, rather than just analysing the last commit.
This will likely involve iterating through the commits in the branch using GitPython and calling extract_commit_info() and analyse_commit() for each commit.
We should probably LIMIT this to a given number of commits to go back - supplied via a command-line option, and with a sensible fairly low default value (e.g. let's just start with 10).
Commits should be processed in chronological order. I.e. start with Commit -10, then Commit -9, etc., until the last / newest Commit 0. It may be a good idea to provide the Summary (not the full Analysis) of each commit processed so far to the LLM when asking for the next analysis - for historical context.

I've made a PR for this issue

Implemented in #9


Loss convergence is very slow！

This problem is a multi-label classification problem and does require BCEloss. But I don't know why the training convergence is very slow. I use 4*1080Ti at a time, 64 frames input,  per 80 videos update the parameters. Do you have the same problem? Trained for one night, the mAP is only 6, or is there a problem with my environment, or does it require training for a long time? Using pytorch0.3.1

I have also the same problem. The training convergence is really slow and with batch_size=5*8 the validation mAP after 7600 iterations is 7%. Does anyone know if there is any problem with the default parameters?

It is a big model, and takes a long time to train. On Kinetics, it usually takes 120,000 iterations with a batch size of ~256 (or about 130-150 epochs). On Charades, it usually takes about 50,000-70,000 iterations (about 150 epochs) to achieve good performance.

This problem is a multi-label classification problem and does require BCEloss. But I don't know why the training convergence is very slow. I use 4*1080Ti at a time, 64 frames input,  per 80 videos update the parameters. Do you have the same problem? Trained for one night, the mAP is only 6, or is there a problem with my environment, or does it require training for a long time? Using pytorch0.3.1

How long you can finish the experiments in your computer?

It is a big model, and takes a long time to train. On Kinetics, it usually takes 120,000 iterations with a batch size of ~256 (or about 130-150 epochs). On Charades, it usually takes about 50,000-70,000 iterations (about 150 epochs) to achieve good performance.

I still have the mentioned problem with training after 100 epochs. The problem is that the values of training loss and validation loss do not decrease any more after 10 epochs. I have tested the pre-trained model on charades which has been provided in the repository to check the validation script and it gives me 31.5 mAP. So, I think there is a problem in the training process which we cannot reproduce the results.

Can you provide more details about the training setup? What is the batch size per-GPU? How many GPUs? What size input? How many frames? Etc.
All these things will impact the training performance, batch norm stats, etc. Our model was trained with 6 videos per GPU and 64 GPUs (effective batch size of 384). Smaller batches can lead to unstable training performance and poor batch norm generalization.

I think it's may be caused by the hyper-parameters settings and the numbers of gpus

Can you provide more details about the training setup? What is the batch size per-GPU? How many GPUs? What size input? How many frames? Etc.
All these things will impact the training performance, batch norm stats, etc. Our model was trained with 6 videos per GPU and 64 GPUs (effective batch size of 384). Smaller batches can lead to unstable training performance and poor batch norm generalization.

Thanks for the reply. I exactly used the default parameters in your code. I used batch_size = 8*5 for running on 8 GPUs and the learning rate and other parameters are also same as your default parameters.

Same for me. I get around 31.5 mAP on the trained Charades model. However after training the model for 64000 iterations with batch_size=4, I get around 10% of mAP.
Is it because of small Batch_size which is hampering the accuracy so much?


Sneaky... not sure if it was intentional or not?

Hi,
In exercise 1 we create a resource group in centralus and deploy a blank arm template. At the end of the exercise we are asked to delete the resource group.
In exercise 2 we deploy a storage account via an arm template and as a side note we are asked to create a resource group. Naturally most people will run the same code from exercise 1. If they do they will get hit with errors when deploying exercise two's arm template as exercise 1 uses "centralus" and exercise 2 uses "eastus" for the location property in the storage account.
Most beginners won't pick this up straight away as the error suggest the storage account name is the issue and not the mismatch in locations.
Thanks
Simon

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 999c1308-1285-7bc6-06aa-a83b7266e0e9
Version Independent ID: cbf8c7f4-d9bd-929f-626f-b65e2f37f896
Content: Tutorial - Add resource to template - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-tutorial-add-resource.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

Thanks for bringing this to our attention. We will investigate it further and update you shortly.

@technologyconsulting - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


feat: add keybind to close test runner

Simple addition that adds a keybind q that can be used to close the test runner.

It works re executing the same command. updating.

Looks great, do you mind fixing the diff for keymaps lua? Maybe it was a crlf -> lf change or similiar?

Looks great, do you mind fixing the diff for keymaps lua? Maybe it was a crlf -> lf change or similiar?

Okay, so yeah, I've been having issues with file format and tested an autocommand that made it go bonkers, should be fixed now.


Provide a way to verify a MAC token request

Parse the header string, verify the signature

Support for MAC Draft 05 (latest, as of March 28, 2022) was added by https://github.com/oauth-xx/oauth2/pull/509 and can be found in master.  Release v2.0.0 will be out soon with this support as well.


feat: Adding icon for Terraform files

Adding Geomancy icon to support terraform files (.tf and .terraform.lock.hcl)
Opened issue #2.
Thanks for the project, i'm really having fun using this extension

Cool, looks good to me! Thank you for your interest and I'm glad you're enjoying it 8) I was surprised that someone hadn't already made something like this.


pkg/trace/testutil: fix RandomTraceChunk

What does this PR do?
Fix RandomTraceChunk

Motivation
https://github.com/DataDog/datadog-agent/pull/23959#discussion_r1533963510

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

APM unit tests pass

/merge


Fix SecurityGroupClient.ListNetworkSecurityGroups()

The unmarshalling code didn't account for the payload being wrapped in a
NetworkSecurityGroups element; this has been fixed.
Thanks you for your contribution to the Azure-SDK-for-Go! We will triage and review it as quickly as we can.
As part of your submission, please make sure that you can make the following assertions:

[ ] I'm not making changes to Auto-Generated files which will just get erased next time there's a release.

If that's what you want to do, consider making a contribution here: https://github.com/Azure/autorest.go

[ ] I've tested my changes, adding unit tests where applicable.
[ ] I've added Apache 2.0 Headers to the top of any new source files.
[ ] I'm submitting this PR to the dev branch, or I'm fixing a bug that warrants its own release and I'm targeting the master branch.
[ ] If I'm targeting the master branch, I've also added a note to CHANGELOG.md.
[ ] I've mentioned any relevant open issues in this PR, making clear the context for the contribution.

Fixes https://github.com/Azure/azure-sdk-for-go/issues/1971

Can one of the admins verify this patch?


Segmentation fault with CoordinateSet in Python

When trying to explicitly build a CoordinateSet object in Python as follows
import molgrid
from openbabel import pybel

obmol = next(pybel.readfile("sdf", "benzene.sdf"))
obmol.addh()

cs = molgrid.CoordinateSet(obmol.OBMol)

I get a segmentation fault:
[1]    13710 segmentation fault (core dumped)  python mg.py

I also tried to pass obmol directly (PyBel molecule) and to pass a molgrid.GninaIndexTyper object as second argument, but always obtained the same error (or just a MemoryError).

OS: Ubuntu 18.04
conda environment:
name: test
channels:
  - conda-forge
  - pytorch
  - open3d-admin
dependencies:
  - python
  - ipython
  - pip
  - numpy
  - scipy
  - openbabel
  - open3d
  - pytorch
  - pip:
    - molgrid

This problem seems to occur only with the pip-installed library. If I compile libmolgrid from source (within a Singularity container), everything seems to work as expected.

The most likely problem is your version of openbabel is different than the one included with the pip package.  Since the OBMol objects aren't compatible, it fails miserably.  There a number of ways this could be dealt with.

Change the install procedure to build from src and/or require a specific version of openbabel.  This would be the conda approach and I'm not sure is possible with pip
Implement version insensitive wrappers when converting between openbabel python objects and openbabel C++ objects (e.g., write out the molecule as a string in python and then read it back in using C++ to get a new object instead of just copying the pointer)
Minimally, check python version versus c++ version and output a warning telling the user they will need to install a certain version of openbabel for things to work.
Do 3, but if the check fails do 2.
Bundle a copy of python openbabel with molgrid to use (e.g. you would import molgrid.openbabel.pybel to get a guaranteed compatible version)

I'm going to settle for 3 for now.  It is possible that there is some other problem besides the versions (e.g. how the library was built vs changes in the source code).

Can you provide the details of your system openbabel (or whatever openbabel package molgrid would be using)?

Thank you for looking into this. I think having a warning after spotting this incompatibility would be great!
I installed openbabel within the conda environment from the conda-forge channel. The version being installed was 3.1.1. The conda environment above was enough for me to reproduce the problem.

I can think of two solutions:
(1) Embed a version of openbabel within molgrid that is guaranteed to be compatible (but may not be compatible with system openbabel, so less than ideal for integrating with pre-existing code).
(2) Configure a conda package that builds molgrid (rather than getting it from pip) against the conda version of openbabel.  Managing these sorts of dependencies is kinda of the point of using conda.
(1) is easy to do.  (2) will have to wait until I have some poor sod to fob it off onto.

Given that (1) is less than ideal, I'd say (2) is the preferred option.
If you have any pointers I could have a look into it? I started looking at conda-forge, but it was unclear to me how to deal with CUDA packages.

Not really, having never done it.  It looks like you need to come up with a build recipe.  Figuring out the dependencies is probably the main difficulty and ensuring that you are building against an appropriate version of openbabel.  I would look at the recipes for other packages.  I would think making torch a dependency would pull in all the CUDA stuff you need.
Incidentally, v0.5.1, which is now in pypi, has openbabel as a submodule.


Add extension type lists for each message type

Please note that while we are transitioning from travis-ci to AWS CodeBuild, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Resolved issues:
resolves https://github.com/awslabs/s2n/issues/1914
Description of changes:
The new extension structs will let us handle extensions using common logic and defined lists of extensions. To take advantage of that, we need lists of extensions!
Call-outs:
Why is supported_versions in the tls12_server_hello_extensions list? When the client starts processing the ServerHello extensions, it potentially doesn't know whether the server wants to use 1.3 or 1.2. We need to make that decision after we process supported_versions, potentially switching which list we are using.
Testing:
This change only includes basic unit tests. We will need to prove that these lists match current behavior, but that will minimally require the send method.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1917 into master will decrease coverage by 0.02%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #1917      +/-   ##
==========================================
- Coverage   80.89%   80.87%   -0.03%
==========================================
  Files         229      230       +1
  Lines       17114    17126      +12
==========================================
+ Hits        13844    13850       +6
- Misses       3270     3276       +6


feat: add FiletypeIs to icons

Resolves #656
Environment
OS: Windows 11
Terminal: wezterm
Tests

[x] Symbolic Links
[ ] Executables

is = "exec" did not recognize .exe files, for both Filetype and Icon rules, is = "link" worked for both.
Conditions
Under Icon rules:
{ name = "*", is = "link", text = "󰿃", fg = "magenta" },
Before

After

Thank you so much for contributing to this feature!
The issue of exec not working on Windows is a known problem, I'm not quite sure which files on Windows are executable. It seems that besides .exe, files like .bat are also executable. What do you think?

The PathExt environmental variable has a list of executable file extensions:
.com .exe .bat .cmd .vbs .vbe .js .jse .wsf .wsh .msc
Maybe .msi files should be added because they are found next to .exe installers.
Examples
7zip and Firefox has .exe and .msi installers.
Adoptiums build of OpenJDK only has .msi installers.


allow converting from/to TwistWithCovarianceStamped

🎉 New feature
Allows converting from/to TwistWithCovarianceStamped
Summary
ros_gz_bridge allows converting gz.msgs.TwistWithCovariance <-> geometry_msgs.msg.TwistWithCovariance.
However the gz.msgs.TwistWithCovariance also contains a header and stamp. So it could be converted to a geometry_msgs.msg.TwistWithCovariance**Stamped**.
This PR implements that conversion. It is now possible to convert either to/from:

geometry_msgs.msg.TwistWithCovariance
geometry_msgs.msg.TwistWithCovarianceStamped.

Note: I also have a backport of that commit to the humble branch, should I open it after this PR is done ?
Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@ahcorde just fixed the conflicts


I can't make a trainer

var cbl = new CBL({
preprocess: function(img) {
img.debugImage("debugPreprocessed");
img.removeLight(230);
img.debugImage("debugPreprocessed");
},
character_set: "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ",
exact_characters_width: 26,
exact_characters_play: 4,
exact_characters: 4,
blob_min_pixels: 300,
blob_max_pixels: 10000,
blob_max_width: 80,
pattern_width: 50,
pattern_height: 50,
allow_console_log: true,
blob_console_debug: true,
perceptive_colorspace: false,
blob_debug: "debugSegmented"
});

This one might be difficult due to the noise, colors, and potentially overlapping letters. I think what you want to play around with is the pixel jump setting (third parameter of colorRegions) to hopefully connect all of the darker grey pixels into single characters.
Here's a good example.
Basically that'll consider pixels to be part of the same blob even if they're n pixels away from each other.
Seems like the other settings you have are heading in the right direction.

Thanks!
Now I have this error when I click on Solve

Without seeing all the code, I'd guess there's probably a reference to a div id that doesn't exist. So if you have blob_debug: "debugSegmented" then you need a

 somewhere. Same for img.debugImage("debugPreprocessed") and cbl.visualizeModel("visualizeModel") and anything else that references a div id.

And if it's when you click solve then it's probably missing one of these:
cbl.solve("captcha").done(function (solution) {
    document.getElementById('solution').value = solution;
});

<input type="text" id="solution">

Sorry, this was the error:
img.debugImage("debugPreprocessed");
It's solved.
Thanks!!


Testing of modules

Test all modules with "Postman" and fix bugs
Acceptance criteria:

compensation-device-service tested.
full-information-service tested.
power-transformers-service tested.
high-volt-cables-service tested.
lighting-information-service tested.
protective-equipment-service tested.
start-information-service tested.

Implements in https://github.com/EllsaG/power-supply-design/pull/57 pull request.


Moving FAQ Download Datasets using Command-line option

Adding my name :)

@beatrizserrano is it possible for you to approve this review?

@beatrizserrano is it possible for you to approve this review?

Yes, we will approve and merge this one right before merging any of your open PRs (https://github.com/galaxyproject/training-material/pull/3333 and https://github.com/galaxyproject/training-material/pull/3334) :)

@beatrizserrano Hey! I have added a new line addressing another FAQ. Please help me review it. Thanks!


Auto-generate documentation from source code

Summary
Thought about this while making docs updates on #81 -- I think it would be neat to explore what it would look like to auto-generate documentation from source code.
Maybe https://typedoc.org/, https://jsdoc.app/, or similar?

I tried several paths in the past but never found something that would work well. But it's been a while, maybe there is something new out there.
I think an ideal solution would be a tool that would pares the index.d.ts file and generate markdown API docs from that, and the output could replace a section in the README.md file. But I'd still want control over what we show and what we don't.
Alternatively we could generate the standard API docs as an addition, not a replacement for the existing documentation.
All this to say, feel free to explore if you are interested in the topic, and we see where we get with it.

Thanks @gr2m! I'll think more about this and when I have cycles, I can investigate and post back with any findings. 💭


Can you help me interpret this memcheck output?

Hi! Thanks for your work on this gem. I'm pursuing a Bison-based GraphQL parser over in https://github.com/rmosolgo/graphql-ruby/pull/4368, so I thought I'd give this library a try. I enabled it on GitHub Actions (example) but I'm not quite sure how to make sense of the output. Here are a few stack traces:

Invalid write of size 8
  vm_exec_core (vm.inc:1028)
  rb_vm_exec (vm.c:1929)
  rb_vm_call0 (vm_eval.c:52)
  rb_vm_call_kw (vm_eval.c:268)
  method_missing (vm_eval.c:874)
  rb_call0 (vm_eval.c:389)
  send_internal (vm_eval.c:1135)
  send_internal_kw (vm_eval.c:1158)
  rb_f_public_send (vm_eval.c:1210)
  vm_call_cfunc_with_frame (vm_insnhelper.c:2514)
  vm_call_cfunc (vm_insnhelper.c:2539)
  vm_sendish (vm_insnhelper.c:4023)
  vm_exec_core (insns.def:801)
  rb_vm_exec (vm.c:1929)
  rb_fiber_start (cont.c:1830)
  fiber_entry (cont.c:694)

Invalid read of size 8
  imemo_type (internal.h:1150)
  rb_vm_make_proc_lambda (vm.c:946)
  rb_vm_make_lambda (vm_core.h:1671)
  rb_mod_define_method (proc.c:2062)
  vm_call_cfunc_with_frame (vm_insnhelper.c:2514)
  vm_call_cfunc (vm_insnhelper.c:2539)
  vm_sendish (vm_insnhelper.c:4023)
  vm_exec_core (insns.def:782)
  rb_vm_exec (vm.c:1929)
  rb_vm_call0 (vm_eval.c:52)
  rb_vm_call_kw (vm_eval.c:268)
  method_missing (vm_eval.c:874)
  rb_call0 (vm_eval.c:389)
  send_internal (vm_eval.c:1135)
  send_internal_kw (vm_eval.c:1158)
  rb_f_public_send (vm_eval.c:1210)
  vm_call_cfunc_with_frame (vm_insnhelper.c:2514)
  vm_call_cfunc (vm_insnhelper.c:2539)
  vm_sendish (vm_insnhelper.c:4023)
  vm_exec_core (insns.def:801)
  rb_vm_exec (vm.c:1929)
  rb_fiber_start (cont.c:1830)
  fiber_entry (cont.c:694)

Invalid read of size 1
  vm_ic_hit_p (vm_insnhelper.c:4131)
  vm_exec_core (insns.def:1042)
  <unknown stack frame>
  rb_vm_exec (vm.c:1929)
  rb_fiber_start (cont.c:1830)
  fiber_entry (cont.c:694)

All the entries in these files point to Ruby internals. Does that mean they're false positives? Or, should I keep digging based on these?
Thanks for taking a look!

Hello! This gem will only filter memory leaks and always output invalid memory accesses (since under normal operation, Ruby should never access invalid memory). I see that you're running on Ruby 2.7, which will be EOL very soon. Can you reproduce these issues on a newer version of Ruby (preferably on 3.2.1).

Hello! This gem will only filter memory leaks and always output invalid memory accesses (since under normal operation, Ruby should never access invalid memory). I see that you're running on Ruby 2.7, which will be EOL very soon. Can you reproduce these issues on a newer version of Ruby (preferably on 3.2.1).

Thanks for checking it out -- I'll keep an eye out about that release!

Ok, I think this is "good"! Here's the output from my last run:
200,014 bytes in 2 blocks are definitely lost in loss record 139,468 of 139,600
  malloc (vg_replace_malloc.c:393)
 *yyparse (parser.c:1768)
 *GraphQL_CParser_Parser_c_parse (graphql_c_parser_ext.c:8)
  vm_call_cfunc_with_frame (vm_insnhelper.c:3268)
  ....

I suppose this constitutes a real problem -- I'll investigate the generated parser code and see if I can make sense of it. Thanks for your help with this!
Here's the build: https://github.com/rmosolgo/graphql-ruby/actions/runs/4556195698/jobs/8036241668?pr=4410


[HOLD for payment 2024-10-29] [$62.50] Android - Chat search - Unable to reach the bottom of chat list when scrolling down

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.48-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Launch New Expensify app
Go to any chat
Send many messages (so that the All list in Search > Chat is scrollable)
Go to Search > Chat
Scroll down the list in All tab

Expected Result:
The bottom of the chat list can be reached
Actual Result:
User is unable to reach the bottom of chat list
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/e1f1e088-8106-4fbe-9757-97809b9536a5
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021844817953624508019
Upwork Job ID:<PHONE_NUMBER>624508019
Last Price Increase: 2024-10-11
Automatic offers:

mkhutornyi | Reviewer | 104433861

Issue OwnerCurrent Issue Owner: @joekaufmanexpensify

@joekaufmanexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Reproduced. A very minor issue, but an issue nonetheless. I think we can fix this, but should be lower budget, as is very minor change.

Thanks for the review @mkhutornyi, assigning @bernhardoj 🚀

PR is ready
cc: @mkhutornyi

PR out on staging

@mkhutornyi could you complete BZ here so we can prep for payment?

Agreed. Skipping regression test makes sense to me here. TY for completing checklist!

All set to issue payment here. We need to pay:

@bernhardoj - $62.50 for PR via NewDot.
@mkhutornyi - $62.50 for C+ via Upwork.

Requested in ND.

TY!

@mkhutornyi $62.50 sent and contract ended

$62.50 approved for @bernhardoj


Issues with negative timestep on Pedestrian Navigation

Simple fix is to clamp the value of Time Scalar to be non-negative (maybe even non-zero?)  in the custom visualisation, or use an unsigned int (and prevent underflow).

This will need fixing in both pedestrian models (and non-public models)


Add option to enable CORS

Normally you can only access the management console from localhost. This adds a new option for it.

Actually after merging it I have realized how dissonant is to have assets/*.xslt and *.xslt. I reverted it in wait of the improvement.


Technical AMA session #4 on Slack

Hi, adding text of Technical AMA Session #4

Thanks ! Keep up the good work


Parse index

Add target_index option to enable partial parse

Codecov Report

Merging #131 into master will increase coverage by 0.56%.
The diff coverage is 93.25%.

@@            Coverage Diff            @@
##           master    #131      +/-   ##
=========================================
+ Coverage   73.24%   73.8%   +0.56%
=========================================
  Files          69      69
  Lines        2586    2642      +56
=========================================
+ Hits         1894    1950      +56
  Misses        692     692

updated based on comment. could you review again?

reflected comments.

LGTM. I'm running the example tests.

Please merge both PRs after you checked it ok.