Please fix it

node_modules/koa-session2/dist/index.js:125
            var ref = _asyncToGenerator(regeneratorRuntime.mark(function _callee(sid) {
                                        ^

ReferenceError: regeneratorRuntime is not defined

@KuTi  Do you use babel-polyfill?

Hi, I get this with v 1.0.4 and before with 1.0.3.
koa next and nodejs 6.2.0.  Trying to set up with redis. Should I install Babel anyway?

Ah, I'm ju not understanding this babel ting; looks like it as to do with yet another javascript spec layer
Which on are you using?

@rimmartin  koa next dependent babel ( async/await ),  so you can use this middleware and koa next with babel-preset-2015 and babel-preset -stage-0  :)

Thank you,
I also tried
npm install --save-dev babel-preset-es2015-node6
npm install --save-dev babel-polyfill

which I thought might make it work with V8 in nodejs 6.2 better.  Still get the error but it may be my environment yet. Adding the preset part.  Nothing more needed from koa session2 :-) Thanks for working on koa v2

:)

This forces us however to add the babelPolyfill even so it may not be required for the rest of the project. Eventually using the transform-runtime instead could be a more modular alternative.

@Krisa  Great! I will rebuilt it soon!  thanks! 👍

great thanks!


ci: bump k8s versions to 1.29

Fixes #N/A
Description
Bumps the default k8s version to 1.29 for CI testing. This does not bump the supported version for the provider.
How was this change tested?
N/A
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.05%) to 82.665%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/version/version.go
1
90.91%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
5007

Relevant Lines:
6057

💛  - Coveralls


ios_unit_test doesn't support flaky=True

Was surprised to find this issue when marking an ios_unit_test as flaky.
At a glance this is due to the flaky kwarg passing through into the library rules which don't support it, because it's not in the list of test args (_IOS_TEST_KWARGS) that are popped; this is a trivial fix which I'll send a patch for, but I was curious if there's some issue at play here, since this one seems likely to have been caught by now.

@zachgray-slack OK to close?


Update service-mesh.md

ref link added for distributed tracing
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description

Issue reference

@holopin-bot @ibandhiya Thank you!


增加两个算子：sqrt和sigmoid

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation and Context

add some operator: sqrt && sigmoid && masked_fill && less
Description

add some operator: sqrt && sigmoid && masked_fill && less
Use cases (Optional)

BC-breaking (Optional)

Checklist
Before PR:

[ ] I have read and followed the workflow indicated in the Contributors.md to create this PR.
[ ] Pre-commit or linting tools indicated in Contributors.md are used to fix the potential lint issues.
[ ] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

[ ] CLA has been signed and all committers have signed the CLA in this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Quieten output on success

You should rebase on main. I fixed (?) an issue that was causing the gradle daemon to die. Probably running out of memory.

My primary point is I'd prefer to discuss that separately, under another
PR.
On June 13, 2021, GitHub @.***> wrote:

@vlsi commented on this pull request.

In gradle/wrapper/gradle-wrapper.properties
<https://github.com/autonomousapps/dependency-analysis-android-gradle-
plugin/pull/402#discussion_r650549427>:

@@ -1,5 +1,6 @@ distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists -
distributionUrl=https://services.gradle.org/distributions/gradle-6.8.3-
all.zip
+distributionSha256Sum=7faa7198769f872826c8ef4f1450f839ec27f0b4d5d1e51bade63667cbccd205

I do not understand what do you mean by "as expected".
 I use -bin distributions for a long time, and I have no issues
drilling down to Gradle sources

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/autonomousapps/dependency-analysis-android-gradle-
plugin/pull/402#discussion_r650549427>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/ABJG5PJ5IGUF54KUM2WLJQTTSTNFZANCNFSM45KADJNQ>.

@autonomousapps , no problem, I can split two commits in two PRs, however, GitHub does not support stacked diffs, so splitting every commit into individual PR is really really painful.
For instance, no way I would build with gradle-all.zip, so I removed gradle-bin commit from here, however, now I skip testing at my side.
Don't get me wrong: I know it is a good practice to keep only related changes into a single PR/commit, however, I believe, it would take you a couple of minutes maximum to commit and even rebase the commits. You could cherry-pick the commits and close the PR. There are plenty of options, and all of them are way better than spending time in discussions here.

I know this is tangential, but

For instance, no way I would build with gradle-all.zip

Why? What makes the all distribution so bad you don't even want it touching your computer?
To the main point, one reason I've been sitting on this is that I thought the CI job failed. In fact, I'm almost certain I told github it was ok to run tests from this, but I just had to do it again 🤷

Thanks for working with me on this, and addressing this long-standing issue.

In principle I approve these changes, but I'd still like to verify behavior locally before merging. Soon (and thanks).

Thanks, I did try to find a test case (or a way to create such), however I failed, so no tests in PR 🤷♂

I deliberately try to avoid testing against output because it was too
volatile in the beginning. So I just have to see it in action. Thanks
for checking.
On June 14, 2021, GitHub @.***> wrote:

Thanks, I did try to find a test case (or a way to create such),
however I failed, so no tests in PR 🤷♂

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/autonomousapps/dependency-analysis-android-gradle-
plugin/pull/402#issuecomment-861201968>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/ABJG5PJYKAKOLZ4NPC44MB3TS3VAVANCNFSM45KADJNQ>.

        val msg = projectHeaderText(projectAdvice.projectPath) + "\n  " + // <-- added indent here as well
          consoleText.replace("\n", "\n  ")

@autonomousapps , what do you think if -  prefix was removed so the output is
> Task :buildHealth
Advice for project :app
  Unused dependencies which should be removed:
    implementation(project(":entities"))
    implementation("com.google.dagger:dagger-android-support:2.24")
    implementation("androidx.core:core-ktx:1.0.1")
    implementation("androidx.appcompat:appcompat:1.1.0-rc01")

  Transitively used dependencies that should be declared directly as indicated:
    implementation("com.google.dagger:dagger:2.24")
    implementation("javax.inject:javax.inject:1")

  Dependencies which could be compile-only:
    compileOnly("androidx.annotation:annotation:1.1.0") (was implementation

Frankly speaking, it is exactly as readable, however, it would be easier to copy-paste (especially Transitively used dependencies that should be declared directly)

Do it, sounds good to me.
On June 20, 2021, GitHub @.***> wrote:

@autonomousapps https://github.com/autonomousapps , what do you
think if - prefix was removed so the output would be

Task :buildHealth Advice for project :app Unused dependencies which
should be removed: implementation(project(":entities"))
implementation("com.google.dagger:dagger-android-support:2.24")
implementation("androidx.core:core-ktx:1.0.1")
implementation("androidx.appcompat:appcompat:1.1.0-rc01") Transitively
used dependencies that should be declared directly as indicated:
implementation("com.google.dagger:dagger:2.24")
implementation("javax.inject:javax.inject:1") Dependencies which could
be compile-only: compileOnly("androidx.annotation:annotation:1.1.0")
(was implementation
Frankly speaking, it is exactly as readable, however, it would be
easier to copy-paste (especially Transitively used dependencies that
should be declared directly)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/autonomousapps/dependency-analysis-android-gradle-
plugin/pull/402#issuecomment-864921910>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/ABJG5PMJFRSY2Y2TIZQOQATTT4HZVANCNFSM45KADJNQ>.

@vlsi just curious if you still want to work on this. No rush of course.

New PR for same issue: https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/pull/471

Closing in favor of the other PR I just merged.


Filter list of nodes on node type

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes
Add the ability to search based on node type as well as the label of the node. This is a temporary fix as other search boxes use the syntax type:inject search term. RED.search API needs a bit more work to make this possible.

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.02%) to 77.303% when pulling aa8f4af3395bd19405c79ba815bfae4aa9c52ae7 on jeancarl:dev into b6fbe7d07db899040c6f50fec3a3a23d026e0fdd on node-red:dev.


adding new subdomain for uai.ac.id

please help to approve @philipto

@tioube First stage of verification passed, official domain for Universitas Al-Azhar Indonesia is student.uai.ac.id as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@philipto thankyou

@tioube Thank you for your patience while waiting for a review and processing your request. Pull request merged.

thanks a lot @philipto


Version bump and CHANGELOG for v2.0.1

Coverage remained the same at 99.346% when pulling 0319690cc977a183562c3a1c910fe77e2f6bc8a3 on DavidS:release-prep into c2987a1b6cd7a9ab104aed8aa7ff61e28ce29cce on mcanevet:master.

Coverage remained the same at 99.346% when pulling 0319690cc977a183562c3a1c910fe77e2f6bc8a3 on DavidS:release-prep into c2987a1b6cd7a9ab104aed8aa7ff61e28ce29cce on mcanevet:master.


Add custom WAF contains rule

Sometimes we want to disqualify some kinds of payloads in our waf. This rule will help in that.

https://staffwww.fullcoll.edu/aclifton/cs133/lecture-17-expression-trees.html


Multiple bindings happen when changing language on settings page in Leaflet application

This is a issue caused by custom WebcComponents (for settings page, page-template component cause the problem)
The template of component is load after slots (this occurs only in production-mode of WebCardinal)

Fixed from https://github.com/webcardinal/webcardinal-core/commit/9ec9977dad4b76b802a3df4f469d8ff738e2dfdc


formatting fix

@4c74356b41 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@4c74356b41 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@4c74356b41 looks like a conflict there, can you resolve?

yeah, so you can review it another month later when it outdated again?

@4c74356b41 I'll check back in on this and happy to review this one for you right away if you can fix the conflict?  we appreciate the contribution, sorry this one sat a while.

@4c74356b41 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mlearned hey, I've taken the latest code from master and formatted it nicely (it doesnt make a lot of sense to do that, since its not horribly broken like it was, but its still properly formatted now, according to standards).

#sign-off


Add e2e test to check the premined balance

Small e2e test to validate that the balance endpoint works.
Steps:

Create a testserver with some dummy premined account.
Use the GetBalance method to query the account.

Solved with commit 96e3f65 👍


Home Screen on Mobile

The mobile look a bit clunky. Maybe we can adjust some of the heights. But if we do not get to this by mid week (before it gets pushed to prod), no biggie.

"Mini-THON" should be on one line
On Sunday, October 25, 2015, Ernie Wilson<EMAIL_ADDRESS>wrote:

The mobile look a bit clunky. Maybe we can adjust some of the heights. But
if we do not get to this by mid week (before it gets pushed to prod), no
biggie.
[image: capture]
https://cloud.githubusercontent.com/assets/6378488/10718874/0cc5da98-7b53-11e5-9708-7cf5c4eb55e2.jpg
—
Reply to this email directly or view it on GitHub
https://github.com/KHS-Minithon/website/issues/34.

--
~Sean

I agree. It's okay for now I think, want to get done all high and medium tickets first. If it gets pushed to prod, it's not the end of the world. But we will address this.

More current:

I will take over this


Persistent Notification Option w/ video/snapshot

Is your feature request related to a problem? Please describe.
I am using the camera option, and would like a persistent notification with video/image of the frames caught by the AI. Similarly to how the mobile app notification is sent.
Describe the solution you'd like
A toggle to turn on/off persistent notifications.
Describe alternatives you've considered
Creating the feature myself by adding to the blueprint has the images never being over written, so with something like this (see below) in the blueprint we just get new notifications with the same image over and over.
 - choose:       - conditions:           - condition: template             value_template: "{{ image != '' or video != '' }}"         sequence:           - alias: "Send instant notification to notify devices"             repeat:               for_each: "{{device_name_map}}"               sequence:                 - action: "notify.{{ repeat.item }}"                   data:                     title: "{{ label }}"                     message: "{{response.response_text}}"                     data:                       video: "{{video if video != '' else None}}"                       image: "{{image if image != '' else None}}"                       entity_id: "{{camera_entity if mode=='Camera' and preview_mode=='Live Preview'}}"                       url: !input tap_navigate #iOS                       clickAction: !input tap_navigate #Android                       tag: "{{tag}}"                       group: "{{group}}"                       interruption-level: passive           - service: persistent_notification.create             data_template:               title: "{{ label }}"               message: "{{ response.response_text }}{% if image %}\n![Camera Image]({{image}}){% endif %}{% if video %}\nVideo: {{ video }}{% endif %}"               notification_id: "{{tag}}"

The reason it will always show the same picture is because images aren't cached and will be loaded every time. Currently only the latest image is stored to save space. I'll see if there's a good workaround because I don't want to change the way images are stored at the moment.
For now, in your automation you can store the snapshot in a custom folder and use a timestamp as its name. This should enable different images for different persistent notifications.
The blueprint also has a lot of issues that need to be addressed first, but if you have a working automation, feel free to share it!

Hey just for clarification sake, you said I could use a custom folder and a timestamp. However the path seems to be baked into the integrations media_handler.py as seen here.
`
async def _save_clip(self, clip_data=None, clip_path=None, image_data=None, image_path=None):
Ensure dir exists
await self.hass.loop.run_in_executor(None, partial(os.makedirs, "/config/www/llmvision", exist_ok=True))
`
and here,
 if expose_images: await self._save_clip(image_data=resized_image, image_path=f"/config/www/llmvision/{image_entity.replace('camera.', '')}.jpg")
Are you suggesting I change this, and the blueprint?

Sorry for the confusion. You'd need to create your own automation and capture a snapshot, store it in a folder and then feed that to the action directly.

Feel free to reopen this at any time if you have any questions! Closing this as unplanned for now, as there are some bugs that need to be fixed first. Once the blueprint actually works well we can still implement persistent notifications.


Will ibm-generative-ai support embedding API?

Context
It looks ibm-generative-ai current does not support embedding model, while the REST API for embedding model is available.
Alternatives
N/A.
Has the feature been requested before?
N/A.
If the feature request is approved, would you be willing to submit a PR?
Yes / No (Help can be provided if you need assistance submitting a PR)

Everything will be supported in the new SDK Version, which will be released next week.

Released in 2.0.0


np.bool is depreciated in latest numpy version, this prevents error

Tiny boring fix :) Latest version of numpy (v1.24) the line np.bool fails as this is depreciated now. The change allows it to work in both v1.24 and older versions.

Thanks!


Feature request: multicolumn buttonbar

Ok, so my map is getting crowded with controls, but limited screen height is starting to bite. One way to fix would be a two column buttonbar.

I can't put time into new features right now, but I'd be open to accepting patches :+1:


New version: JuliaVariables v0.1.4

Registering package: JuliaVariables
Repository: https://github.com/thautwarm/JuliaVariables.jl
Created by: @thautwarm
Version: v0.1.4
Commit: 94d49da824e8801c41c00d03bb5356766a303a87
Reviewed by: @thautwarm
Reference: https://github.com/thautwarm/JuliaVariables.jl/commit/94d49da824e8801c41c00d03bb5356766a303a87#commitcomment-35275556

Is skipping 0.1.3 intentional?

Please don't make consecutive sets of version PRs like this. There's no reason for it. If you have additional changes that you want to publish before a version PR is merged, you can update the existing PR by retriggering with the same version number. When I encounter sets of versions like this in the future, I'm going to pick one at random, merge it and then close all the rest.


Add gitea manager reload-templates command

This can be useful to update custom templates in production mode, when they are updated frequently and a full Gitea restart each time is disruptive.

This might be a bit obscure, I'm not sure how generally useful this functionality is. But our templates display things like release status and ongoing projects, and it's nice to be able to update those without risking e.g. data loss when submitting comments or failed git repository access to update those.

Maybe we also need a template load test sub command to do a test before real reload.

Maybe we also need a template load test sub command to do a test before real reload.

I see the original PR doesn't have so, and I do not think there is a real case which really requires "testing".  If there is any error, the old templates are still used.
Do you think this is a must?  If no, I would suggest to keep things simple. @lunny

Maybe we also need a template load test sub command to do a test before real reload.

I see the original PR doesn't have so, and I do not think there is a real case which really requires "testing". If there is any error, the old templates are still used.
Do you think this is a must? If no, I would suggest to keep things simple (no test). @lunny

If the reload failure will not break the old templates I agree we don't need the test.

I removed the --test flag now. When there is an error on reload, the old templates are kept. It was already implemented to work this way for auto reloading in dev mode.


Google Not Using IDFA Anymore

Google announced today (https://www.blog.google/products/ads-commerce/preparing-developers-and-advertisers-for-policy-updates/) that they will no longer use the IDFA, so the App Tracking Transparency prompt will not be shown on iOS 14.
I am not sure how this affects the open PR that added support, but I wanted to open an issue to start a discussion on what changes may be needed.
Let me know how I can help, love this library.

Hey
I am working on the iOS 14 support at the moment.
I am having trouble to get the pod to validate for the current updates, the new arm64 slice is missing on AdMob for release
https://github.com/CocoaPods/CocoaPods/issues/10198
https://github.com/CocoaPods/CocoaPods/issues/10104
Having issues with the suggested fixes, will have to downgrade to xCode 11 if I cannot fix it the next few days. After that I will finish of the iOS 14 tracking.
It's super frustrating.

Hey
I am working on the iOS 14 support at the moment.
I am having trouble to get the pod to validate for the current updates, the new arm64 slice is missing on AdMob for release
https://github.com/CocoaPods/CocoaPods/issues/10198
https://github.com/CocoaPods/CocoaPods/issues/10104
Having issues with the suggested fixes, will have to downgrade to xCode 11 if I cannot fix it the next few days. After that I will finish of the iOS 14 tracking.
It's super frustrating.

Frustrating for sure!

Frustrating for sure!

Hey,
thanks again for your kind comments. I managed to fix the cocoa pods issues I was facing. Will start with the Ad tracking this weekend. I will keep you updated.

Hey,
thanks again for your kind comments. I managed to fix the cocoa pods issues I was facing. Will start with the Ad tracking this weekend. I will keep you updated.

Thank you for being so responsive!

Thank you for being so responsive!

Hey,
you are very welcome. I started yesterday with the ad tracking. I replaced the deprecated GDPR consent SDK with the new UMP SDK. This new SDK also handles the iOS 14 app tracking. There is alot to digest here https://developers.google.com/admob/ump/ios/quick-start
This will all be part of the next major version of SwiftyAds and will hopefully be ready very soon. You can follow the progress on the developer branch.

Hey,
you are very welcome. I started yesterday with the ad tracking. I replaced the deprecated GDPR consent SDK with the new UMP SDK. This new SDK also handles the iOS 14 app tracking. There is alot to digest here https://developers.google.com/admob/ump/ios/quick-start
This will all be part of the next major version of SwiftyAds and will hopefully be ready very soon. You can follow the progress on the developer branch.

Hey, small update on this.

Replaced deprecated Consent SDK with new UMP SDK which handles both GDPR and ATT alerts using Google Funding Choices
Updated to AdMob SDK 8.0
Just need to to do a bit more testing and refinement

This will be part of SwiftyAds v12 which I will release latest next weekend.

Hey,
SwiftyAds 12 has just been released :). Please feel free to open another issue you are having problems.

@crashoverride777 Thank you so much.
Quick question. Given Google is not using the IDFA, why in your screenshots is it asking to be tracked? I also assume this means when submitting updates of this version, we do not need to select that we use the IDFA. I also assume our App Privacy labels can also remove the track you for ads.

Hey,
Thanks for your comment.
This is the consent form provided by Google UMP, there is still tracking involved.
In regards to the IDFA setting on iTunes Connect, I be honest I am not sure, haven't yet updated my apps that use SwiftyAds.Maybe you can google this, which I will do once I update my apps.
I don't understand your question about the privacy label?

Sorry,
Let me rephrase entirely.
Is it possible to use your library and opt-out of IDFA usage so that I do not have to show the "allow tracking" prompt?


[RAC-5256]Deploy vNode failure should be captured in test scripts

1. Backgroud
There should be 3 vNodes deployed.
But there are only 2 vNodes deployed and the shell step didn't exit with error code.
Error: Duplicate name 'vmslave16-Rinjin1' on target. Use --overwrite option to delete it. Completed with errors
Why do the jenkins can not capture the ERROR ?
2.  Analysis
The key cause is the command: "expect" in the method execWithTimeout()  of shareMethod.sh.
expect -c 'set timeout 90; spawn -noecho /bin/sh -c "ovftool --noSSLVerify --diskMode= --datastore=Isilon-Cluster-NFS  --name='\''vmslave16-Rinjin1'\'' --net:'\''Mgmt - longej=vmslave16-switch'\'' '\''/home/****/isofarm/OVA/vRinjin-Haswell.ova'\''<EMAIL_ADDRESS>expect timeout { exit 1 } eof { exit 0 }'

if ovftool executes timeout, return 1.
if ovftool occurrs other error, such as "Duplicate name 'vmslave16-Rinjin1' on target.", the command expect returns 0. Therefore the Jenkins can not capture this error.(That is unexpected.)

Generally, the normal behavior of execWithTimeout() is

if ovftool executes timeout, return 1.
else return  what the ovftool returns.

@PengTian0 @PengTian0 @anhou @iceiilin @changev

:+1:


Default behavior of Stacking notifications isn't the same as the fcm

Expected Behaviour
Non-collapsible messages
Actual Behaviour
Collapsible messages
Reproduce Scenario (including but not limited to)
Use this plugin
Steps to Reproduce
..
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
All
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Any
Cordova CLI version and cordova platform version
doesn't matters
Plugin version
cordova plugin version | grep phonegap-plugin-push   1.6.0

Sample Push Data Payload
var message = {
  to: "/topics/cars",
  priority: 'high',
  data: {
    message: 'supercar',
    sound: 'default',
    title: 'Cars',
    priority: 2
  }
};

This causes a lot of issues because we are trying to follow the official fcm docs that basically says

A non-collapsible message denotes that each individual message is delivered to the device. A non-collapsible message delivers some useful content, as opposed to a "ping" to the mobile app to contact the server to fetch data. Messages are non-collapsible by default except for notification messages, which are always collapsible.

I think the default behavior must to be changed

@mazyvan sounds valid. I'll look at it in the 2.0.0 branch as that would be a change to behaviour which should only be done during a major version bump.

@mazyvan the docs that you link to with regards to collapsible & non-collapsible messages deal with the server side of FCM not the client side.
A collapsible message is a message that may be replaced by a new message if it has yet to be delivered to the device.
Basically, FCM will attempt to deliver all non-collapsible messages but if you send a collapsible message with some data and then send a second collapsible message with some new data and the first has not been delivered to the device then only the 2nd message will be delivered.


docs:  Scaffold user docs

What does this PR do and why?
This PR adds the initial scaffold to the user documentation, providing the initial layout so other contributors know where to add their documentation.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other pull requests.
How to set up and validate locally

Start the docs in a terminal within your irida-next repo with:

cd docs-site
pnpm build
npm run serve

Ensure the docs and provisional headers look correct.

PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

[x] I have evaluated the PR acceptance checklist for this PR.

Quick comment about adding a couple of pages:
Can we add pages for Bot Accounts underneath http://localhost:4000/irida-next/docs/user/organization/groups/ and http://localhost:4000/irida-next/docs/user/project/projects/

Purpose of bot accounts
Type of bot accounts
How to manage (add/remove) bot accounts, as well as, adding/removing memberships.
View/Generate/Revoke personal access tokens for bot accounts

Note: A group bot that is created in a group can be added as a member to other projects/groups (global). However, a project bot can only be associated to the project it was created within

Added in a937584499656ea7b900ad230860ac8e80fb8bee. Let me know if this isn't what you envisioned!


backup: allow excluding online-only cloud files

What does this PR change? What problem does it solve?

Allow excluding online-only cloud files (such as OneDrive Files On-Demand) with a switch --exclude-cloud-files
Assuming we have the following file test.txt in OneDrive, but not available on the device

Attempting to backup the file with the --exclude-cloud-files switch will not attempt to download the file locally, instead the file will be excluded.
restic -r test --verbose backup C:\Users\username\OneDrive\test.txt --exclude-cloud-files

open repository
repository e5f7dfde opened (version 2, compression level auto)
created new cache in C:\Users\username\AppData\Local\restic
no parent snapshot found, will read all files
load index files
[0:00]          0 index files loaded
start scan on [C:\Users\username\OneDrive\test.txt]
start backup on [C:\Users\username\OneDrive\test.txt]
scan finished in 0.003s: 0 files, 0 B

Files:           0 new,     0 changed,     0 unmodified
Dirs:            4 new,     0 changed,     0 unmodified
Data Blobs:      0 new
Tree Blobs:      5 new
Added to the repository: 2.425 KiB (1.841 KiB stored)

processed 0 files, 0 B in 0:00
snapshot 5ef1ff96 saved

2024/08/08 22:39:41 archiver/tree.go:298	archiver.NewTree	1	result:
/C, root "C:\\", path "", meta "C:\\"
    /Users, root "", path "", meta "C:\\Users"
        /username, root "", path "", meta "C:\\Users\\username"
            /OneDrive, root "", path "", meta "C:\\Users\\username\\OneDrive"
                /test.txt, root "", path "C:\\Users\\username\\OneDrive\\test.txt", meta ""

...

2024/08/08 22:39:44 archiver/archiver.go:420	archiver.(*Archiver).save	40	/C/Users/username/OneDrive/test.txt target "C:\\Users\\username\\OneDrive\\test.txt", previous <nil>
2024/08/08 22:39:44 restic/exclude.go:404	main.collectRejectFuncs.rejectCloudFiles.func1	40	rejecting online-only cloud file C:\Users\username\OneDrive\test.txt
2024/08/08 22:39:44 archiver/archiver.go:443	archiver.(*Archiver).save	40	C:\Users\username\OneDrive\test.txt is excluded

Was the change previously discussed in an issue or on the forum?

Closes #3697
Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[x] I have added tests for all code changes.
[x] I have added documentation for relevant changes (in the manual).
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[x] I'm done! This pull request is ready for review.

@MichaelEischer I'm not really sure how to add tests for this one. It would require either manually setting the file attributes or somehow setting up OneDrive on Windows?

@m-wild I really love that PR. That's exactly what I'm searching for more than a year at #4155
I'll happily make some tests with some beta builds if needed.

There's already an option --one-file-system. Can this feature use that option, instead of introducing a new one? Or does it already have a meaning on Windows?

There's already an option --one-file-system. Can this feature use that option, instead of introducing a new one? Or does it already have a meaning on Windows?

--one-file-system currently has no effect on Windows. However, Windows does support junctions which can be used to mount other filesystems within a filesystem. My expectation would be that --one-file-system once it has an effect, addresses exactly this scenario.
Cloud files feel somewhat different: they are placeholders for files that can basically show up everywhere in the filesystem. I wouldn't really expect that --one-file-system excludes those files.

@m-wild Any news on this one perhaps ?

If someone is interested in taking over this PR, please just create a new PR, then I'll close this one.

@m-wild @MichaelEischer I'll happily sponsor whoever wants to pick this up ;)

@m-wild Do you still want to work on this ?

@m-wild Do you still want to work on this ?

Planning to pick it up over the Xmas break

Ho ho ho, it's a Christmas miracle, a PR ready to review.

Ho ho ho, it's a Christmas miracle, a PR ready to review.

Ho ho ho, ping me whenever this is merged so I can sponsor you a bit ;)


No way to "Disable Forced Refresh" MCM Prompt

There is a prompt that is constantly spammed

Neither GPT nor Google knows what im talking about. A search through the folders of all MCM related mods yielded nothing for a fix. Checking ever MCM related settings menu also yielded no results. So ive gotta assume this is either a problem with the pack or perhaps wabbajack?
I7 13700k
64gb DDR4 3600mhz @16cl
RTX 4070
(Latest Build of the pack)

Update to the newest build, and to save yourself problems join the discord.


XPRINTER XP-5811H not detected during scan

Hi
I tried to implement this librarry to my app but my printer is not detected.  Though i can see the device in search using my phone settings. Tried also to install other bluetooth printer app and it is working there but not in flutter.
can anyone help me with this?

@bryangiray what flutter platform do u use? is it desktop / mobile?
is this printer XPRINTER XP-5811H already connected on your router wifi?
the printer usually require to setting SSID & Password of your wifi first


Precaching idb fix

R: @jeffposnick @philipwalton
More changes to ensure idb works.
In IndexedDB version 1, we used two object stores 'workbox-precaching' and 'precaching-details-models'. In IndexedDB version 2 we only use 'precaching-details-models'. To ensure we are upgrading to a clean working state, this PR removes any old object stores and creates a new one.

So I fixed the integration tests, but the underlyinig indexedDB mock isn't correct, so I've raised a PR against phil's version of the repo here: https://github.com/philipwalton/shelving-mock-indexeddb/pull/1

Coverage increased (+0.05%) to 84.736% when pulling c2c0d993bd52678173fc3761d69f73a5af4f74f6 on precaching-idb-fix into 08d323ef98998827c162b988c8e9fe4854cd0a0b on v3.


Added single-byte integer value setting service

I have added a single byte integer value setting service.
The service is enabled only if the corresponding parameter can be set.
Error handling works better with https://github.com/scottyphillips/pychonet/pull/51.

Sounds very interesting, I’ll to clear this backlog this weekends What is your plans with this one @nao-pon san?

Thanks! 👍


[Error] Unable to copy to storage host

Im having trouble uploading my projects to modulus since changing to meteor build image.
modulus deploy --node-version 4
resulting in this
Deploying Meteor version 1.4.1.
Compressing project...
10.8 MB written
Uploading project...
Upload progress [===================] 100%
[Error] Unable to copy to storage host
Previously it ended like this:
Initializing Meteor environment.
Using<EMAIL_ADDRESS>Node version 4 specified.
Demeteorizing application...
[✓] myproject running at myproject-0000.onmodulus.net

@timmakken Sorry for the trouble. The problem that you were experiencing was with our storage servers. That issue is now resolved. If it continues to happen you can open a support request and we will assist you from there.


[Feature Request]: CATE for instrument variable based models

Describe the feature you want to propose or implement
Thanks for the great work! I am just using this package for heterogeneous treatment effect using instrument variables (DoubleMLPLIV). For example, estimate price elasticity for different categories using a supplier-side instrument.
But it seems that this repo only provides this function for non-iv models like DoubleMLPLR.
Propose a possible solution or implementation
I came up with several ideas to modify the data to achieve CATE for each category, like

split data into subsets and estimate the ATE for each.

Interact the treatment/instrument with category dummy and estimate the effect of constructed treatments using constructed instruments.

Did you consider alternatives to the proposed solution. If yes, please describe
No response
Comments, context or references
But I think these are not neat solution to this problem. Do you think these features have a chance to be implemented in the future? Thanks again!

Hello @Tao-Han,
thank you for the suggestion. I agree that this would be a nice additional feature.
We might add this in the future, but this might take while as we are currently quite occupied.


Issue 14383: attach ddoc unittests to module declaration

Fixes: https://issues.dlang.org/show_bug.cgi?id=14383
Turns out that the ddoc code already handles generating docs for module unittests; all that was missing was that the parser wasn't attaching these unittests to the module declaration. Once this is done everything works as expected.

ping @andralex @MartinNowak
Autotester has been green for a while. Is this PR a go or a no-go?

Thanks, just wondering how this https://github.com/biozic/d2sqlite3/commit/a977288c1f8472dd88f05af317a98dd35c90d619 works.

Auto-merge toggled on


Add window buttons and title

In all the operating systems we have buttons (minimize, maximize, close) and also the title.
We want to add the buttons + title in this app also.

Photo Credit: GeeksForKeeks

Hey @thtauhid  ,I want to work on this, could you please assign it to me? Thank you!

@praptisharma28 can't wait to see what exciting design you implement.
Best of luck!

Hi @praptisharma28 have you had a chance to dive into the issue yet? Need any help?

hey could I work on this?

@kirontoo assigned.

Join our Discord server to discuss ideas, ask questions, and collaborate with others.


for [ordered_json], the member function [parse] do not support keeping order of the input

std::string input;// input json
{// I want to keep order,but failed
    ordered_json json = json::parse(input);
    ....
}
{// another way to cover this problem, but ugly...
    std::stringstream str(input);
    ordered_json jsonInput;
    str >> jsonInput;
    ....
}
hope someday, nlohmann/json can support this

Can you try calling ordered_json::parse?

sorry, just forgive my stupid....
---Original---
From: "Niels<EMAIL_ADDRESS>Date: 2020/12/19 17:06:23
To<EMAIL_ADDRESS>Cc: "sinbobo"<1405783418<EMAIL_ADDRESS>Subject: Re: [nlohmann/json] for [ordered_json], the member function [parse] do not support keeping order of the input (#2539)
Can you try calling ordered_json::parse?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Error after Transform All T4

After click Transform T4 I get this error:

What should I do ?
Fyi, I was Add one C# project, so i have 3 projects now, (script, web and report), but i have no idea this error cause of my new project(report) or not ?
Thank a lot as usual

Your web project has ref to script project or had. See references and clean bin directory.

clean script/bin/debug and clean web/bin with same result.
where i should start find references?

Your web project -> references

same error, but since I removed reporting project, the problem solved.
my reporting project working find...i'll find again whats wrong with my reporting project.
Thank volkan for all your guide and fast response.


0.9.0 on Windows: KeyError: "Unable to open object (object 'anno1' doesn't exist)"

Our test CI job on Windows started to fail with release of anndata 0.9.0:
https://github.com/quiltdata/quilt/actions/runs/4666597638/jobs/8261488713
Linux jobs are still fine.
I can try to provide minimal example for repro if it's needed.

@sir-sigurd , looks like this problem https://github.com/scverse/anndata/issues/956
It was fixed here https://github.com/scverse/anndata/pull/958, but i don't see that this PR was merged in 0.9.0 https://github.com/scverse/anndata/tree/0.9.0
@ivirshup, can we have a 0.9.1release with this PR included, because now the release is broken on Windows.

Just started a backport

Fixed by: https://github.com/scverse/anndata/pull/973


LicenseRDFAGenerator still treating args[0] as a spreadsheet?

This code dates back to at least 76f2cb861.  Are we still treating the first argument as a spreadsheet?  I'm having trouble squaring that with our passing the XML repo path as the first argument.  Maybe the comments are just stale?

Closing this issue since the code has moved to the LicenseListPublisher repository.  Opened similar issue #4


PR: Create empty widget

add versioneer
add basic methods
add empty widget

add versioneer

I think we should remove versionner. We're not using it in spyder-notebook and spyder-terminal because it adds too much complexity to the release process (in my opinion).

I think we should remove versionner.

I'm not familiar with the release process, I added because @goanpeca say me to do it, also spyder-plugin-cookiecutter add it by default

also spyder-plugin-cookiecutter adds it by default

I know, but please remove it for now. We still have how useful it is (I think it's not that useful. @goanpeca tried to use it in loghub but he had to revert to the old way of setting the release version).


Tests are failing on master

If https://github.com/Palindrom/palindrom-client/pull/63 gets merge tests will be failing on master. I will fix them.

Tests were failing on master even before #63. See:

Do you know how to fix that?

Ah, sorry, I did not notice that :(

No problem at all.

The tests are now passing, thanks for fixing!


fixes: #365;

fix: Correct encoding of DateTime and Enum fields in toJson(), ensuring proper JSON response generation
Resolves #365
Example
Old Output of Generated Model Class
The old output of the generated toJson() method was as follows:
Map<String, dynamic> toJson() => {
       'role': role,
       'created_at': createdAt,
       'updated_at': updatedAt,
     };

New Output  of Generated Model Class
The toJson() method correctly encodes DateTime fields using toIso8601String() and Enum fields using their string representations.
The expected output of the generated toJson() method is:
  Map<String, dynamic> toJson() => {
        'role': role?.name,
        'created_at': createdAt?.toIso8601String(),
        'updated_at': updatedAt?.toIso8601String(),
      };

For more info check issue #365

Thx


TypeError: crypto.randomUUID is not a function

It was reproduced on gem ver. 0.3.2. On 0.3.1 works ok.
It seems to be related to changes in https://github.com/josefarias/hotwire_combobox/pull/191

Bug description
To Reproduce

Start typing in input field
In browser console Uncaught (in promise) TypeError: crypto.randomUUID is not a function

Expected behavior
No errors in console, finds suggested values for input fields.
Screenshots

Gem version: v0.3.2
Rails 8.0.1 with importmaps pipeline
Firefox 133.0.3
macOS

Related: https://github.com/josefarias/hotwire_combobox/discussions/223


[Release-1.28] - Update kine for multiple fixes

Backport fix for Update kine to pull in fix for failed compaction causing the next iteration to be skipped

#11269

Validated using latest commit on release-1.28 branch following https://github.com/k3s-io/k3s/issues/11269#issuecomment-2471861675
ubuntu@ip-172-31-12-199:~$ k3s -v
k3s version v1.28.15+k3s-46fbfb7e (46fbfb7e)
go version go1.22.8
ubuntu@ip-172-31-12-199:~$ journalctl -u k3s |grep locked
Nov 13 22:01:03 ip-172-31-12-199 k3s[74724]: time="2024-11-13T22:01:03Z" level=info msg="Bootstrap key locked for initial create"
ubuntu@ip-172-31-12-199:~$


Test ADAPT installation

The API endpoint is http://freme.adaptcentre.ie/freme
Please add some comments how to fix the problems if possible, e.g.
e-Link test fails
There are no e-Link templates in the database, so install the e-Link templates from this database dump: http:/...
Please add the issues and the fixes here in this issue.

API endpoint is now http://freme.adaptcentre.ie/current

These three tests fail on this API endpoint: http://freme.adaptcentre.ie/current.

e-link example API call
filter knowledgebase example
xslt converter - tei2temp-html example

For all three calls the respective data is missing. You have to load it into your freme mysql database.
This is explained here: https://api-dev.freme-project.eu/doc/knowledge-base/freme-for-sysadmins/initialising-freme.html
There are init-templates that you can use to load the data into the database:

templates:
https://raw.githubusercontent.com/freme-project/freme-packages/master/broker-local/src/main/resources/init-template.sql

sparqlconverters:
https://raw.githubusercontent.com/freme-project/freme-packages/master/broker-local/src/main/resources/init-sparqlconverter.sql

xsltconverters:
https://raw.githubusercontent.com/freme-project/freme-packages/master/broker-local/src/main/resources/init-xsltconverter.mysql.sql

With the following calls you can check if the templates, filters and xsltconverters are available:
curl -X GET "http://freme.adaptcentre.ie/current/e-link/templates"
curl -X GET  "http://freme.adaptcentre.ie/current/toolbox/convert/manage"
curl -X GET "http://freme.adaptcentre.ie/current/toolbox/xslt-converter/manage"

all tests pass now


send ext.flutter.debugDumpApp; ext.flutter.reassemble

Some leftovers from a previous PR:

expose debugDumpApp() as a service protocol extension and wire it up to the flutter run UI

and:

after we successfully restart an isolate, send ext.flutter.reassemble to refresh the UI

You should also consider debugDumpRenderTree().
LGTM.

You should also consider debugDumpRenderTree().

Added; we may need to change the UI of flutter run around a bit if we add many more options.


Data seal of approval

Apply for http://www.datasealofapproval.org/en/assessment/

Data seal of approval was discontinued in 2018: https://www.coretrustseal.org/about/history/data-seal-of-approval-synopsis-2008-2018


Fixed crash on danger init

Fixes
In order to expose an environment variable, go to:
/Users/felixkrause/Developer/danger/lib/danger/commands/init.rb:234:in `circle_token': undefined local variable or method `repo_slug' for #<Danger::Init:0x007f9cfcdfc208> (NameError)
	from /Users/felixkrause/Developer/danger/lib/danger/commands/init.rb:159:in `setup_danger_ci'

This is non-trivial, should have a changelog entry


[Relay][Frontend] Span Filling TFLite

Set tensor name as the source name of span during the conversion of TFLite model
Add structural_equal comparisons with and without set_span to the existing test cases
Add span test cases for frequent conversions

Hi community,
This is a part of the RFC, TVM Explorer Infrastructure. After this PR, the source layer information of TFLite will be tagged to the converted relay. Feel free to give us any comment. Thank you! :D
For your reference, here is the related links.
PreRFC in forum
RFC in git
Tracking issue
@haowhsu-quic, @zack-ch


[BUG] Cluster provisioning with non-built-in node drivers fail randomly

Rancher Server Setup

Rancher version: v2.7.3
Installation option (Docker install/Helm Chart):

Helm on RKE2

Proxy/Cert Details:

No proxy set

Information about the Cluster

Kubernetes version: v1.25.9+rke2r1
Cluster Type (Local/Downstream):

Local

Describe the bug

Provisioning of non-built-in node driver clusters fail sporadically and reproducible due to 404 not found error.
Rancher machine cannot download driver from rancher assets
Rancher machine pod logs:
Downloading driver from https://rancher-demo-27.devops.lab/assets/docker-machine-driver-otc
2023-08-03T10:57:38.816931346Z Doing /etc/rancher/ssl
2023-08-03T10:57:40.744198155Z downloaded file docker-machine-driver-otc failed sha256 checksum
2023-08-03T10:57:40.747065253Z download of driver from https://rancher-demo-27.devops.lab/assets/docker-machine-driver-otc failed

It seems the non-built-in node drivers are not downloaded on all pods of rancher.
This results in 404 not found error while downloading the driver from rancher into the cluster provisioner
Missing driver can be seen the in rancher pods:
Pod 1 (contains all drivers needed)
Rancher pod 1 # ls -la  /usr/share/rancher/ui/assets | grep docker-machine
-rwxr-xr-x 1 root root 11685574 Aug  3 10:32 docker-machine-driver-cloudscale
-rwxr-xr-x 1 root root 39034880 Apr 24 15:52 docker-machine-driver-harvester
-rwxr-xr-x 1 root root  8654688 Apr 24 15:52 docker-machine-driver-linode
-rwxr-xr-x 1 root root  8749056 Aug  3 07:58 docker-machine-driver-nutanix
-rwxr-xr-x 1 root root 10715282 Aug  3 10:06 docker-machine-driver-otc

Pod 2 (non-built-in drivers missing)
Rancher pod 2 # ls -la  /usr/share/rancher/ui/assets | grep docker-machine
-rwxr-xr-x 1 root root 39034880 Apr 24 15:52 docker-machine-driver-harvester
-rwxr-xr-x 1 root root  8654688 Apr 24 15:52 docker-machine-driver-linode

Running the curl command to download the driver multiple times result in sporadic 404 errors:
~ curl -k https://your.rancher.domain/assets/docker-machine-driver-linode
Warning: Binary output can mess up your terminal. Use "--output -" to tell
Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.
~ curl -k https://your.rancher.domain/assets/docker-machine-driver-cloudscale
404 page not found
~ curl -k https://your.rancher.domain/assets/docker-machine-driver-cloudscale
404 page not found
~ curl -k https://your.rancher.domain/assets/docker-machine-driver-cloudscale
Warning: Binary output can mess up your terminal. Use "--output -" to tell
Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.
~ curl -k https://your.rancher.domain/assets/docker-machine-driver-cloudscale
404 page not found
~ curl -k https://your.rancher.domain/assets/docker-machine-driver-cloudscale
404 page not found

To Reproduce

Use a rancher which has multiple rancher pod replicas
Enable a non-built-in driver like cloudscale, nutanix or telekom cloud
Provision a cloud using one of this node drivers
See the provisioning failing as the driver download fails from your.rancher.domain/assets/...

To Reproduce on cli

Use a rancher which has multiple rancher pod replicas
Enable a non-built-in driver like cloudscale, nutanix or telekom cloud
run curl https://your.rancher.domain/assets/docker-machine-driver-cloudscale
There will sporadically appear an 404 not found Error

Result
Provisioning of nodes fails sporadically as the driver cannot be downloaded from the rancher pods.
Expected Result
Provisioning (and driver download) should work 100% of the time
Additional context
To provision clusters using non-built-in node drivers, Rancher seems to download the driver to the rancher pods
Then it provides the driver binary for the provisioner under the subfolder /assets/docker-machine-driver-...
Workaround
A workaround can be to scale down the rancher deployment to 1 replica.
This is not what you want for an production environment.

Closing as duplicate of https://github.com/rancher/rancher/issues/38481.


Remove separate wskdeploy executable from ow-utils

wskdeploy is now distributed as a subcommand of wsk (wsk project),
so no need to include a wskdeploy executable.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/OpenWhisk-Playground/88/
Test FAILed.
Test FAILured.


Implementing People Card Component

Is your feature request related to a problem? Please describe.
When searching for a new user, a people card component needs to be designed that showcases the information of the appropriate user(s).
Describe the solution you'd like
The People card component would have a design specified in the Figma mockups. This component would display the relevant user's name, handle, bio and provides an option to follow or unfollow the user.
Describe alternatives you've considered
N/A
Additional context

This issue was brought to light, approved and allocated  in the frontend team meeting on the 24th March 2022
Here is a mockup of the People card component.

Looks good to me!


fix punctuation typos

Signed-off-by: Jie Luo<EMAIL_ADDRESS>
Proposed changes
I think the doc should in the same style, and the curly quote is not formally in standard English documents.

Thanks, @luojie233 :)


Pgdn/Pgup

Saw erming/shout#668 and realised that would be a good feature to have.
We could either pull in the PR that guy created, or write it a different way if people don't like how he did it. But I think it's something that would be useful (and I've thought a few times that it is a pain them not working).

Sigh, nevermind. Duplicate of #263


Can't figure out how to order queries

I have a relatively simple 1 model query but I can't figure out how to get the ordering in. Closest that I have gotten is
courses = (Course.where{venue_id.in(accessible_venues.each{ |v| v.id} ) }).order{{course =>start_date.asc}}
Gave me an invalid sql statement in the console:
SELECT "courses".* FROM "courses" WHERE "courses"."venue_id" IN (88, 90, 91, 93, 95, 96, 99) ORDER BY "course"."start_date" ASC
The table name is courses so the logical thing to do is make courses plural in the order part:
courses = (Course.where{venue_id.in(accessible_venues.each{ |v| v.id} ) }).order{{courses =>start_date.asc}}
this gave
ArgumentError: Unable to get table for #ActiveRecord::Relation:0xba1cf24
from /home/mark/.rvm/gems/ruby-1.9.2-p290/gems/squeel-0.9.3/lib/squeel/adapters/active_record/context.rb:59:in `get_table'
I'm obviously doing something stupid but there is no documentation that I could find on order{}.
Any help greatly appreciated
Regards
dukha
P.S. In the config file I have  config.load_core_extensions :hash, :symbol

This worked
courses = (Course.where{venue_id.in(accessible_venues.each{ |v| v.id} ) }).order{{:courses => :start_date}}
as did
courses = (Course.where{venue_id.in(accessible_venues.each{ |v| v.id} ) }).order(:courses => :start_date)
I just can't get the desc or asc in
Regards
dukha

Hi Guys
I just don't get it. Still working on the same query but a different part
d = Date.today
courses = (Course.where{(venue_id.in(av.each{ |v| v.id} )) & (end_date <= d)  })
The SQL that I get is like this
SELECT "courses".* FROM "courses"  WHERE (("courses"."venue_id" IN (88, 90, 91, 93, 95, 96, 99) AND 1))
PGError: ERROR:  argument of AND must be type boolean, not type integer
So squeel substituted 1 for end_date <= d. What should I be doing here??
Regards
Dukha

Hi I think it should be:
d = Date.today
courses = Course.where{ (venue_id.in(av.each{ |v| v.id} )) & (end_date <= my{d}) }

Hi tanelj
Many thanks for the help. It worked. What is my{}? I couldn't find it in the squeel doc or anywhere else. I must have missed something basic somewhere
Kind regards
Mark

@dukha Squeel uses instance_eval magic inside the block helpers for referencing keypaths. If you need to access a variable in your enclosing scope, you have to use my to get it.
You can disable this behavior by providing a block variable:
courses = Course.where{|c| ( s.venue_id << accessible_venues.map(&:id) ) & ( s.end_date <= d ) }

To answer your original question, you could do this:
Course.order{ start_date.asc }


Update travis ci configuration

Add sudo: false to tests run on travis' new container based environment.
Add newer version of go.

@lantins thanks for that. I removed go1.1 and go1.2 since few people use that version any more.

Cool, I'd of done the same :)


Add missing dropapp menues to seed

use this diff view: https://github.com/boxwise/boxtribute/pull/1389/files/9bff4d06909219981c9aad23af94958ab988c02e..1a3477b5887196f7e2371795464c3582666b8678
Question

on production, the action_permission column of cms_functions is always NULL. Why does it exist then? Are action permissions taken into account from the JWT and compared to what is listed in constants.php in the menusToActions mapping?

 Dropapp: staging is missing menues

@pylipp in response to

on production, the action_permission column of cms_functions is always NULL. Why does it exist then? Are action permissions taken into account from the JWT and compared to what is listed in constants.php in the menusToActions mapping?

action_permissions are not checked in dropapp. Authorization happens through usergroups and usergroups_functions and usergroups_camps table. We created this column to maybe transfer to action based permission checking in the future, but it did not happen until now.

 Dropapp: add menue forwarding to Statviz Dashboard


Roll Flutter Engine from d993e0d55fc6 to ac34d3e63998 (7 revisions)

https://github.com/flutter/engine/compare/d993e0d55fc6...ac34d3e63998
2022-08-31<EMAIL_ADDRESS>[fuchsia] Workflow scripts. (flutter/engine#35822)
2022-08-31<EMAIL_ADDRESS>Reland: Adds a reusable FragmentShader (flutter/engine#35846)
2022-08-31<EMAIL_ADDRESS>roll CanvasKit to 0.36.1 (attempt 2) (flutter/engine#35839)
2022-08-31<EMAIL_ADDRESS>Roll Dart SDK from 5a2737b71877 to fec63626f078 (2 revisions) (flutter/engine#35844)
2022-08-31<EMAIL_ADDRESS>[Impeller] Compute shader support (flutter/engine#35750)
2022-08-31<EMAIL_ADDRESS>Roll Skia from e0b87738eca7 to 45fe3b5a6710 (9 revisions) (flutter/engine#35841)
2022-08-31<EMAIL_ADDRESS>Roll Clang from 41d4067d0b7c to 039b969b32b6 (flutter/engine#35840)
Also rolling transitive DEPS:
flutter/web/canvaskit_bundle from yrsfF-vTvu4j to 8wh6J7ZXGCgo
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Disable react/wrap-multilines

This prevents standard from whining about needing parentheses around multi-line JSX (react/wrap-multilines), such as in this case which currently fails:
let Hello = React.createClass({
  render () {
    return

Hello {this.props.name}

  }
})

See #4.

@cesarandreu @jprichardson @dcousens You guys cool with this?

My argument for removing this is that this takes too many lines. In this trivial example, we can remove 4 lines by omitting the parentheses:
let Hello = React.createClass({
  render () {
    return (

        {this.prop.users.map(this.renderUser)}

    )
  },

  renderUser (user) {
    return (

Hello {user.name}

    )
  }
})

I'm alright with it. Would like to know what the others think.

Released as 1.0.4 and standard 5.0.2.

I'm OK with this.


IndexError: index 676 is out of bounds for dimension 0 with size 676

| load 'model' from 'checkpoints/audio2motion_vae/model_ckpt_steps_400000.ckpt', strict=True
| WARN: checkpoints/motion2video_nerf/may_torso/lm3d_radnerf_torso.yaml not exist.
| load 'model' from 'checkpoints/motion2video_nerf/may_torso/model_ckpt_steps_250000.ckpt', strict=True
trainval: Smooth head trajectory (rotation and translation) with a window size of 7
/data/zssy-digital-human/projects/gpp/tasks/radnerfs/dataset_utils.py:263: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
self.lm68s = torch.tensor(self.lm2ds[:, index_lm68_from_lm478, :])
Extracted wav file (16khz) from data/raw/val_wavs/8-27s.wav to data/raw/val_wavs/8-27s_16k.wav.
Loading the HuBERT Model...
/data/home/yaokj5/anaconda3/envs/geneface/lib/python3.9/site-packages/torch/_utils.py:776: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
return self.fget.get(instance, owner)()
Loading the Wav2Vec2 Processor...
Traceback (most recent call last):
File "/data/zssy-digital-human/projects/gpp/inference/genefacepp_infer.py", line 542, in
GeneFace2Infer.example_run(inp)
File "/data/zssy-digital-human/projects/gpp/inference/genefacepp_infer.py", line 490, in example_run
infer_instance.infer_once(inp)
File "/data/zssy-digital-human/projects/gpp/inference/genefacepp_infer.py", line 180, in infer_once
out_name = self.forward_system(samples, inp)
File "/data/home/yaokj5/anaconda3/envs/geneface/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/data/zssy-digital-human/projects/gpp/inference/genefacepp_infer.py", line 475, in forward_system
self.forward_audio2secc(batch, inp)
File "/data/home/yaokj5/anaconda3/envs/geneface/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/data/zssy-digital-human/projects/gpp/inference/genefacepp_infer.py", line 384, in forward_audio2secc
cano_lm3d = inject_blink_to_lm68(cano_lm3d)
File "/data/zssy-digital-human/projects/gpp/inference/genefacepp_infer.py", line 103, in inject_blink_to_lm68
lm68[idx, 36:48] = lm68[idx, 36:48] * (1-blink_factor) + closed_eye_lm68[idx, 36:48] * blink_factor
IndexError: index 676 is out of bounds for dimension 0 with size 676

It seems like a error caused by index out of bounds. Can you provide more details? Since the code should have convert the audio to 16k and video to 25 fps.

@lokvke, could you please attempt it using an audio file longer than 10 seconds? In my testing, it consistently fails when the provided audio is less than 8 seconds.

Hey @yerfor. I tried running with longer audio clips as well. For the same audio clip, I tried the full length (around 1min 30s) and a 59s segment, both failed with a similar error, just the index value mention was different (but the same between multiple runs). It seems like it worked for a sample that was around 40s long. All samples were encoded to 16kHz successfully and as far as I can tell, the error seems to happen in the exact same line. Is there any other detail I can provide for this to help debug this issue ?

Hi, I have similar error but caused in
    samples = self.prepare_batch_from_inp(inp)

I trained with my own videos following the docs.
The shape of self.dataset.pose does not match with get_hubert(self.wav16k_name) and caused IndexError.
But I am not sure how to fix that.

Hi, I have the same problem.  Always IndexError appears and same error with different lengths of drive audio.
How can I solve this problem?

in the inject_blink_to_lm68 function, when the generated video contatins 676 frames, T=676.  So when i=675, j=1, the idx=676(out of index),  here is my solution:   idx = i % (i + j)  （ps: blinking result seems not very natural）

in the inject_blink_to_lm68 function, when the generated video contatins 676 frames, T=676. So when i=675, j=1, the idx=676(out of index), here is my solution:
idx = i % (i + j)
（ps: the blinking result seems not very natural）

in the inject_blink_to_lm68 function, when the generated video contatins 676 frames, T=676. So when i=675, j=1, the idx=676(out of index), here is my solution:
idx = i % (i + j)
（ps: the blinking result seems not very natural）

in the inject_blink_to_lm68 function, when the generated video contatins 676 frames, T=676. So when i=675, j=1, the idx=676(out of index), here is my solution:
idx = i % (i + j)
（ps: the blinking result seems not very natural）

Hi, thanks for your comment. I will update the mentioned modification in the latest commit.
As for the blinking results, the blink motion is controlled by the hard-coded blink_factor_lst = np.array([0.1, 0.5, 0.7, 1.0, 0.7, 0.5, 0.1]) # * 0.9 in the inject_blink_to_lm68 function.  Maybe you can try different values to improve the naturalness of eye blink.

in the inject_blink_to_lm68 function, when the generated video contatins 676 frames, T=676. So when i=675, j=1, the idx=676(out of index), here is my solution:
idx = i % (i + j)
（ps: the blinking result seems not very natural）

Hi, thanks for your comment. I will update the mentioned modification in the latest commit.
As for the blinking results, the blink motion is controlled by the hard-coded blink_factor_lst = np.array([0.1, 0.5, 0.7, 1.0, 0.7, 0.5, 0.1]) # * 0.9 in the inject_blink_to_lm68 function.  Maybe you can try different values to improve the naturalness of eye blink.

is this issue fixed now??


Near 100% probabilities

Hi,
I am trying to categorize images into 4 groups. Its is currently doing a good job of differentiating between the categories.
['Baking_Soda', 'Diet_Coca_Cola', 'Frosted_Flakes', 'Honey_Nut_Cheerios']
Relevant Code:
https://gist.github.com/achainan/9241f695b9e72d3667cccec07b7f4faa
https://gist.github.com/achainan/cb92f8018337b679d315d28ff5ee685f
I am doing this by fine-tuning the VGG16 model using Keras 1.2.2
All my probabilities are coming out extremely high even when I test it on images that are not of the kind that I desire. Aka this is the result of testing against an image of a black table:
Diet_Coca_Cola 1.0
Frosted_Flakes 1.0
Honey_Nut_Cheerios 0.0
None 0.0
Baking_Soda 0.0

Things I have tried:

softmax vs sigmoid: As I thought that sigmoid would not dampen the probabilities of the others. This covered it: https://github.com/fchollet/keras/issues/741 Currently trying again with sigmoid.

binary_crossentropy vs categorical_crossentropy: since nothing else was working.

fewer and more enums: 5, 10, 50 (perhaps I was overfitting)

Without and with transfer learning: After reading this: http://cs231n.github.io/transfer-learning/

Tried implementing EarlyStopping: Since I thought make it could be because of overfitting.

callbacks=[
  callbacks.EarlyStopping(monitor='val_loss', min_delta=0, patience=0, verbose=1),
])

Adding and removing a 'None' class with zero images: since I wanted none to be an option and images of a black desk was triggering other classes at near 100% probabilities. (This was a guess and I haven't seen much documentation on how to implement a background category. Any advice on the proper implementation would be appreciated.)

As you might notice I am using a very small number of images:
nb_train_samples = 188
nb_val_samples = 137

Ps. I am pretty new to ML and am using coremltools to convert the models.
Any advice or help would be much appreciated.
Something similar I found: https://github.com/fchollet/keras/issues/1878

Hi,
It looks like you have a very small training dataset. Have you tried to use a smaller model? Like you have pointed out, many things can go wrong if a very deep model is used on a tiny dataset.  It is possible that the model is simply lingering around a small area of the very large parameter space and so gives weird outputs.

@StrawhatPeter Thanks for the advice. Think I'm asking about how to deal with the none of the above case. Aka asked here: https://stackoverflow.com/questions/43578715/how-best-to-deal-with-none-of-the-above-in-image-classification
I am getting for any image I put in I get results like this:
[1,0,0,0], [0,1,0,0]
I want:
 [.9,.4,.4,.1], [.2,.4,.3,.2]

Think I got it. Created a bg class and put in random images of the table I am placing the objects on. How does f-rcnn choose the images to put in the background folder?


Packing issue on several containers

Hi,
I am trying to pack boxes into two containers. But the problem is that all boxes are packed separately for each container. I expected that second container will be filled only if first container is full is it right or not? Is this desired behavior or something wrong is with my code?

       //Input
        List<Box> b_result = new List<Box>(); //Output of packed boxes
        List<Box> b0 = new List<Box>();
        List<Box> b1 = new List<Box>();

        DA.GetDataList(0, b0);//Gui to get containers
        DA.GetDataList(1, b1);//Gui to get items

        //Create a list of Container objects, which describes the dimensions of the containers:
        List<Container> containers = new List<Container>();

        for (int i = 0; i < b0.Count; i++) {
            containers.Add(new Container(i, Convert.ToDecimal(b0[i].X.Length), Convert.ToDecimal(b0[i].Z.Length), Convert.ToDecimal(b0[i].Y.Length)));
        }

        //Create a list of items to pack:
        List<Item> itemsToPack = new List<Item>();

        for (int i = 0; i < b1.Count; i++) {
            itemsToPack.Add(new Item(i, Convert.ToDecimal(b1[i].X.Length), Convert.ToDecimal(b1[i].Z.Length), Convert.ToDecimal(b1[i].Y.Length),1));
        }

        //Create a list of algorithm IDs corresponding to the algorithms you would like to use. (Currently EB-AFIT is the only algorithm implemented.) Algorithm IDs are listed in the AlgorithmType enum.
        List<int> algorithms = new List<int>();
        algorithms.Add((int)AlgorithmType.EB_AFIT);

        //Call the Pack method on your container list, item list, and algorithm list:
        List<ContainerPackingResult> result = PackingService.Pack(containers, itemsToPack, algorithms);

        //Extracting values
        for(int i = 0; i < result.Count; i++) {

            for (int j = 0; j < result[i].AlgorithmPackingResults.Count; j++) {

                for(int k = 0; k< result[i].AlgorithmPackingResults[j].PackedItems.Count; k++) {
                    Point3d origin = new Point3d(
                        Convert.ToDouble(result[i].AlgorithmPackingResults[j].PackedItems[k].CoordX),
                        Convert.ToDouble(result[i].AlgorithmPackingResults[j].PackedItems[k].CoordY),
                        Convert.ToDouble(result[i].AlgorithmPackingResults[j].PackedItems[k].CoordZ)
                        );
                    origin += b0[result[i].ContainerID].BoundingBox.Min;

                    double x_ = Convert.ToDouble(result[i].AlgorithmPackingResults[j].PackedItems[k].PackDimX);
                    double y_ = Convert.ToDouble(result[i].AlgorithmPackingResults[j].PackedItems[k].PackDimY) ;
                    double z_ = Convert.ToDouble(result[i].AlgorithmPackingResults[j].PackedItems[k].PackDimZ) ;

                    b_result.Add(new Box(
                        new Plane(origin, Vector3d.ZAxis),
                        new Interval(0, x_),
                        new Interval(0, y_),
                        new Interval(0, z_)
                        ));

                }

            }
        }

Hi, there! Yes, the way the service is written, it will try to pack all the items into all of the containers--it is not packing one and then taking the leftovers and packing them in the next. To do what you are trying to do, I suggest taking the pack results from round one, find the "best" result, see which items are in the unpacked list for that result, and then run that subset through a second time, etc. Keep peeling off the best sets in each round and go again with the leftovers. Does that make sense?

Hello, I would like to render the boxes in my scene, the thing is that I dont know what does the coordinates mean, are they the center of the box? if yes we will need to add the value of the boundingBox/2 to the origin!! .. , and i have an issue with the rotation values, they are around 1 2 3 and it doesnt make sense to me..


Issues w/ creating Firewall rules

I keep getting a strange issue with this library and trying to create firewall rules.  Some of the time, the code below works, but most of the time, I get a 400 bad request error with code "10202" filter at index 0.
I don't see where there are any issues with my expression and don't understand why this sometimes works but I get an error also with the same exact code being run.
require __DIR__ . '/vendor/autoload.php';

$key     = new Cloudflare\API\Auth\APIKey('***', '***');
$adapter = new Cloudflare\API\Adapter\Guzzle($key);
$zones   = new Cloudflare\API\Endpoints\Zones($adapter);
$fw      = new Cloudflare\API\Endpoints\Firewall($adapter);

$sites = $zones->listZones('', '', 1, 300, '', '', 'all');

foreach ( $sites->result as $site ) :
	$site_id = $site->id;
	$site_name = $site->name;

	$expression = 'ip.geoip.country ne {"US" "MX" "CA"}';
	$config = new Cloudflare\API\Configurations\FirewallRuleOptions();
	$config->setActionBlock();
	$result = $fw->createFirewallRule(
        $site_id,
        $expression,
        $config,
        'Country Block'
    );

endforeach;

I found out the error was telling me that the filter already exists.  However, I don't see anywhere in this library to clear out old filters or anything.  Is their an end-point to do so?

I found the same issue. Firewall create and delete rules correctly but doesn't update the rule.

Yeah, this library doesn't seem to support that. What you need to do is add a new function to Firewall.php that uses the filters endpoint, something like
`public function updateFirewallFilter(
string $zoneID,
string $filterID,
string $expression
): \stdClass {
$filter = array(
'id' => $filterID,
'expression' => $expression,
'paused' => false
);
    $response = $this->adapter->put('zones/' . $zoneID . '/filters/' . $filterID, $filter);
    $body = json_decode($response->getBody());
    return $body->result;
}`


Wrong path for ruby.exe for ChefDK 0.3.5 on Windows for Several Commands

fauxhai.bat, rubocop.bat, ruby-parse.bat, and ruby-rewrite.bat all contain the wrong path for ruby.exe.
They have c:\opscode\chefdk\bin\ruby.exe as the path, when it should be c:\opscode\chefdk\embedded\bin\ruby.exe.

@mattstratton the bundle exec issue should be fixed when we ship with a newer rubygems
https://github.com/opscode/omnibus-chef/commit/91398be2bee49079484cd8fb768348e0acdb47a6
See the upstream fix

@jdmundrawala thanks for pointing me to the upstream fix. This had bitten me a few times already and led me to applying really weird workarounds which I hope can be removed with the next ChefDK release :-)

Is this the same issue as this one, or should I open a new one?   (Win7 ChefDK 0.3.5-1)
PS C:\Users\IBM_ADMIN\cookbooks\testing-stack> chef exec rake clean
bundle exec chef-client -z destroy_all.rb
C:/opscode/chefdk/embedded/lib/ruby/site_ruby/2.0.0/rubygems/dependency.rb:313:in `to_specs': Could not find 'bundler' (

= 0) among 9 total gem(s) (Gem::LoadError)
Checked in 'GEM_PATH=C:/Users/IBM_ADMIN/.chefdk/gem/ruby/2.0.0:C:/opscode/chefdk/embedded/lib/ruby/gems/2.0.0;C:/Users/I
BM_ADMIN/.chefdk/gem/ruby/2.0.0', execute gem env for more information
from C:/opscode/chefdk/embedded/lib/ruby/site_ruby/2.0.0/rubygems/dependency.rb:322:in to_spec'         from C:/opscode/chefdk/embedded/lib/ruby/site_ruby/2.0.0/rubygems/core_ext/kernel_gem.rb:58:in gem'
from C:/opscode/chefdk/embedded/bin/bundle:22:in `'
rake aborted!

And how I tried to debug this a bit.....
PS C:\Users\IBM_ADMIN\cookbooks\testing-stack> chef exec gem env
RubyGems Environment:

RUBYGEMS VERSION: 2.4.1
RUBY VERSION: 2.0.0 (2014-02-24 patchlevel 451) [i386-mingw32]
INSTALLATION DIRECTORY: C:/Users/IBM_ADMIN/.chefdk/gem/ruby/2.0.0
RUBY EXECUTABLE: C:/opscode/chefdk/embedded/bin/ruby.exe
EXECUTABLE DIRECTORY: C:/Users/IBM_ADMIN/.chefdk/gem/ruby/2.0.0/bin
SPEC CACHE DIRECTORY: C:/Users/IBM_ADMIN/.gem/specs
SYSTEM CONFIGURATION DIRECTORY: C:/ProgramData
RUBYGEMS PLATFORMS:

ruby
x86-mingw32

GEM PATHS:

C:/Users/IBM_ADMIN/.chefdk/gem/ruby/2.0.0
C:/Users/IBM_ADMIN/.chefdk/gem/ruby/2.0.0:C:/opscode/chefdk/embedded/lib/ruby/gems/2.0.0

Checkout the GEM_PATH.  It has a colon instead of semi-colon between the paths.  Anyone know where these paths get put together so I can hack in fix?

I believe rubygems requires that format

Even in first path that failed?  I see a colon for between first two and then a semi colon between last two
GEM_PATH=C:/Users/IBM_ADMIN/.chefdk/gem/ruby/2.0.0:C:/opscode/chefdk/embedded/lib/ruby/gems/2.0.0;C:/Users/I BM_ADMIN/.chefdk/gem/ruby/2.0.0

oh, i see, ill look into it

https://github.com/opscode/chef-dk/commit/b86e11c1d0e730eb20f643b2cfc17f8edf620979#diff-fe326f6d5ce6ed8533e649afa178a0b2
I think I had run into this before a changed it. Should be resolved in chefdk 0.3.6

@jdmundrawala  thx!

@jdmundrawala  BTW I quick grep, shows there might be other places this type of thing needs to be fixed:
test-kitchen-1.2.1/features/step_definitions/gem_steps.rb:  ENV['GEM_PATH'] = [gem_home.to_s, ENV['GEM_PATH']].join(':')
ronn-0.7.3/Rakefile:  ENV['RUBYLIB'] = $LOAD_PATH.join(':')

@jdmundrawala ok, ran into the initial issue stated above, wrong ruby path in the .bat files.    After appling the upstream patch, is there a way to re-run the bat file creation?   Or do I have to fix each of these by hand right now?   (0.3.6 is when?)

which bat files don't work. Can you paste them one here

based upon the upstream patch touching the rubygems-update-2.4.1/lib/rubygems/installer.rb batch file generation, my rake.bat file for example looks like this:
@ECHO OFF
IF NOT "%~f0" == "~f0" GOTO :WinNT
@"C:\Users\IBM_ADMIN\.chefdk\gem\ruby\2.0.0\bin\ruby.exe" "C:/Users/IBM_ADMIN/.chefdk/gem/ruby/2.0.0/bin/rake" %1 %2 %3 %4 %5 %6 %7 %8 %9
GOTO :EOF
:WinNT
@"%~dp0ruby.exe" "%~dpn0" %*

which has the wrong ruby.exe path, should come from the C:\opscode\chefdk\embedded\bin path instead.    so, if there was a way to re-run the gem installer, it would update my .bat files.  But since there's not that many I care about, I won't worry about this for now. thx for the replies.

I think only the last line should run. Where is rake.bat located?

I have two of them, one in    /.chefdk/gem/ruby/2.0.0/bin   and the other in c:/opscode/chefdk/embedded/bin.     I'll update that last line accordingly and give that a go. thx.

Just a report that 0.3.6 ChefDK fixed both these issues, THX!   now I hit the ssh chef provisioning issue: https://github.com/opscode/chef-provisioning/pull/267   but patch seems to be work.

Closing as fixed in ChefDK 0.3.6+


NeoJSONWriter doesn't know how to write LeUnknownSnippet

The page GemStone session registry and manager UI in the GT Book contains instances of LeRobocoderMetamodelSnippet that are not loaded in the image, so they revert to LeUnknownSnippet when the book is loaded. If you now modify this page and try to save the image, NeoJSONWriter will raise an exception since LeUnknownSnippet is not found in the NeoJSONMapper.
There is a dictionary built as follows:
LeJsonV4 new newWriter.

It looks for all the implementations of #leJsonV4Name.
So the fix is to implement LeUnknownSnippet class>>#leJsonV4Name.


Section Acknowledgment/References section implemented

This resolves
#28
#29
In addition, DocumentTitle and DocumentCategory (formally Document Type) have been implemented.
Due to some commit failure last week, PR #33 and #34 have been completly worked into this PR.

@sustefil: Please quickly go through and merge if accepted.


Only build manifest when building in VMR in general

No need to build anything but the SDK manifest asset when building in VMR mode.

@joperezr Please merge and flow into installer next time you get a chance.


add link to app as about url

I always struggle to find the link for the deployed version of this app. Can we please add it as website url in the about section? As we did for rx-angular pointing to the docs.

good point!

done!


Bridges(Cut-Edge) and Articulation Point(Cut-Vertex) In Graph using C++

Hey! @tarun26091999  @rudrakshi99  @shreyanspoddar  @nakul-19 @plazzy99
I am GSSOC'21 Participant and I would like to implement following Graph Algorithm in C++ which are inter-related:

Algorithm to Find Bridges(Cut Edge) in Graph : A Bridge is defined as an edge in the graph whose removal either makes a graph disconnected or increase number of disconnected components of graph.

Algorithm to Find Articulation Point(Cut Vertex):  An Articulation Point is defined as a vertex whose removal of self and related edge makes a graph disconnected. In a Graph if an edge (node1,node2) is a bridge then either node1 or node2 is an articulation point. In a condition if there are no bridge , Even then if a cycle exist it is also an indication of probable existence of an Articulation point.

I can contribute clean and well-commented code so that it gives readability to all users. Please assign me this issue.

both are different problems, you need to raise separate issues.


[[FIX]] Accept new RegExp flags introduced by ES6

This resolves gh-2361.
Commit message:

ECMAScript 6 introduces two additional flags for regular expressions:
"u" and "y". Extend the lexer to accept these when linting ES6 code.
Existing code attempts to compile regular expression literals as a
mechanism to ensure validity. Update this code to ignore regular
expression flags in order to ensure consistent behavior when JSHint is
executed in ES5 and ES6 environments.

Coverage increased (+0.0%) to 96.53% when pulling cdb439f15b10b952fcd2be7ea0b3b30eb88efd39 on jugglinmike:2361-es6-regexp-flags into 290280c29d47baad06d8958ef4a90807f4a67e40 on jshint:master.

I feel like @mathiasbynens should maybe take a look

Sounds good to me!
I just pushed an additional commit to explicitly ensure that the flags aren't repeated (something we got "for free" when passing the flags to the RegExp constructor). Because we're no longer relying on the environment's regular expression parser to verify the flags, we'll need to be especially careful about false-negative regressions. Certainly these won't break any builds, but that's no reason to accept less strict enforcement.

Coverage increased (+0.0%) to 96.53% when pulling 4dddb54892454d791133e0dad777da581837b87b on jugglinmike:2361-es6-regexp-flags into 290280c29d47baad06d8958ef4a90807f4a67e40 on jshint:master.

@caitp I've worked with @mathiasbynens to improve esprima's RegExp parser (marginally) and back-ported the latest version to this patch. Mind giving it another look?

Coverage increased (+0.06%) to 96.58% when pulling 7453ce3b524f0249df68362c814d49b2d42e44cc on jugglinmike:2361-es6-regexp-flags into 290280c29d47baad06d8958ef4a90807f4a67e40 on jshint:master.

Coverage increased (+0.009%) to 97.763% when pulling d33431e4aadd5ee19e4d2b126291c82a5dbdd122 on jugglinmike:2361-es6-regexp-flags into 0938dc70f65726973f05cdab8a6772e9ade47080 on jshint:master.

I've rebased this work (including @nicolo-ribaudo's suggested warning
improvement) and force-pushed the result. It should be good to go!
Note that we merged support for the y flag in gh-2999 (opened in the time
since this pull request was first submitted), so this patch now only concerns
the u flag.

(The original version of this branch is available for posterity here: https://github.com/jugglinmike/jshint/tree/2361-es6-regexp-flags-orig)

Mind taking a look at this, Cait?

This looks good to me, are we still waiting for @caitp or should I go ahead and merge?

Hey @rwaldron,
I found this while working on support for ES2018 named capture groups.
Years later, I can see that it was woefully incomplete. While it correctly interprets the new Unicode escape sequences, there are a whole bunch of early errors that we need to re-implement. I've pushed up a fixup! commit which introduces those.
While working, I found a simplification that while not technically necessary, would make the code (and the review) easier. I've implemented that as a dedicated commit to help keep the changeset for the feature addition more focused.
Since the test suite has changed since I first submitted this patch (it now requires the expected column number of all errors in addition to the expected line number), I needed to rebase. Given the age of the pull request, I doubt this is very disruptive, but in the interest of transparency, I've preserved the previous version of this branch (itself a rebase) as 2361-es6-regexp-flags-2.


Add support for mergeCells, shrink to fit

See https://github.com/box/spout/pull/738

https://github.com/box/spout/pull/729 - fix shrink to fit + add test shrinkToFit was not handled in StyleMerger so it was overwritten by the default cell style StyleManager didn't add the property to the xml if shrinkToFit was used without alignment or text wrap. Unit test
https://github.com/box/spout/pull/529 - tests for column width, row height and cell merge + minor refact Tests Added 'addOption' to OptionsManagerInterface Moved 'setColumnWidths' and 'mergeCells' methods to Xlsx Writer implementation since the actual feature only works for Xlsx at the moment

Thank you @quamis and @ignaczistvan


ERROR Error: Could not find a valid element for given id or className.

I am trying to integrate this with my existing angular application. I am showing this editor into the popup. At the first time, it is working fine. But if I close the popup and try to open it again then it is giving the following error.
Error: Could not find a valid element for given id or className.
Angular -> 7
Angular Material -> 7
angular-email-editor -> 0.7.0
My observations,
At the first time component is able to add id to div. But when I reopen the popup again then id is not applied to div. Due to this, I am getting the above error.
Is there any solution to this issue?

@kaustubhkhare19 this has been merged. Thank you! Feel free to close the issue if it looks good.

I got this error again when using version 0.8.0 with Angular 11.
It working fine when downgrade to v0.7.1

Same behavior/error in here.
angular-email-editor: ^0.7.1
angular/material: ^12.2.5
angular/cli: ^12.2.5
Any solution?


Add a bumpformat option for the bumpslopes exporter

This option prevents situations where the bumpslopes exporter fails to detect an input bump format as an height map. The typical situation is a 3 channels RGB height map we would expect to be monochromatic (R=G=B for all pixels) but having subtle color variations introduced by a texture publication process (typically a precision or rounding error). In such situation the exporter will automatically interpret the bump map as a false normal map, resulting in a corrupted texture output.
The bumpformat option allows the user to bypass the automatic detection and manually specify the input bump format.

The approach LGTM, though it's failing the CI tests on Travis. Do some tests need to be updated?

An initialization was missing for one test case. Should be better now!

CI passes. LGTM. Merging.


Fix cache error in CI

If cache is desired, we should look into this in the future and do it
properly.

#1155 can be rebased once this change is in

Ah, that was added in #1152. It seemed to be working at the time. After merging it I forgot all about and never connected it with the current error. In any event, yes, let's back this out. Personally, I could care less about whether cache is enabled. If someone does care about it, then they need to provide a working solution that is not so brittle.


(AzureCXP) Fixing ARM Template

Fixing the ARM Template typo based on customer feedback
https://github.com/MicrosoftDocs/azure-docs/issues/50590
Do want to call out that with the previous doc state when we copy into Visual Studio blank template , it is correcting itself.

@BharathNimmala-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


fix(katib): Add user header to address #414

Added a patch to katib-ui deployment
[x] Verified on GKE 1.23 with 1.7.0-rc.2 manifests

Reference: https://github.com/kubeflow/katib/issues/2126

Great job, Ablai! Your work looks good to me.


inv

@rultor release, tag is 0.0.23

@rultor release, tag is 0.0.23

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.23

@yegor256 Done! FYI, the full log is here (took me 8min)

@rultor release, tag is 0.0.24

@rultor release, tag is 0.0.24

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.24

@yegor256 Done! FYI, the full log is here (took me 8min)


Cant't choose new language for kata

I interesting by kata "The boolean order" and want to solve it with Haskell. But i can't choose it in languages list.  Is it possible to add Haskell in languages list?

Yes, you can translate to Haskell after solving in a language that's already available.


Go 1.10 does not automatically installs libraries

What version of Go are you using (go version)?
go version go1.10 windows/amd64
Does this issue reproduce with the latest release?
1.10 is the latest at this moment
What operating system and processor architecture are you using (go env)?
GOARCH=amd64
GOOS=windows
What did you do?
I have small test program with two small dummy libraries.
Go program in the src has path github.com/user/hello
package main

import (
    "fmt"

    // Libraries bellow are not automatically generated in pkg
    "github.com/user/stringutil"
    "github.com/user/tttt"
)

func main() {
    fmt.Printf(stringutil.Reverse("\nHello, world.\n"))
    fmt.Println(add(85,tttt.Tttt(9)))
}

add() function is in separate file, also in hello directory:
package main

func add(x int, y int) int {
    return x + y
}

When I don't have the above libraries compiled and just use go install hello program is installed and works but the pkg directory is not created and no stringutil.a and tttt.a.
If I want to have libraries generated I have to use go install on every library, in which case pkg is generated and libraries in it.
this is not behaviour according to this documentation:

Whenever the go tool installs a package or binary, it also installs whatever dependencies it has. So when you install the hello program
$ go install github.com/user/hello
the stringutil package will be installed as well, automatically.

Note that I have just installed the latest version (1.10) and just start using it as is writing only one dummy program and two dummy libraries one of which is copied from Go documentation.
What did you expect to see?
Dependencies installed automatically in the pkg
What did you see instead?
No pkg and no installed dependencies

The behavior has since been changed so that to install the dependencies you need to pass the -i flag.
https://golang.org/cmd/go/#hdr-Compile_and_install_packages_and_dependencies

The -i flag installs the dependencies of the named packages as well.

Do you have a link to where you got the documentation?

Got it from here
https://tour.golang.org/concurrency/11
Link to the documentation is on this page in the

To learn how to organize and work with Go code, watch this screencast or read How to Write Go Code.

section.
"How to Write Go Code" is the link to it.

With the -i flag works as it should.

Fixed in +105655 @bcmills

I'm still seeing install without -i switch in the web documentation linked above.

@toni-rmc
Fix has not been approved and merged yet


feat(graphql): add index directive support for rds datasource

Adds @index directive support for RDS datasource. Import workflow detects the indexes and adds @index on the field.

Customer can edit a generated index and add a queryField attribute which will create a GraphQL query with the given name.

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1219 (bb2fdb7) into feature/rds-support (a08589a) will increase coverage by 0.02%.
The diff coverage is 90.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                   Coverage Diff                   @@
##           feature/rds-support    #1219      +/-   ##
=======================================================
+ Coverage                63.52%   63.55%   +0.02%
=======================================================
  Files                      301      301
  Lines                    18987    19012      +25
  Branches                  4585     4589       +4
=======================================================
+ Hits                     12062    12083      +21
- Misses                    6311     6315       +4
  Partials                   614      614

Impacted Files
Coverage Δ

...rmer/src/resolvers/generators/rds-vtl-generator.ts
93.54% <0.00%> (-6.46%)
:arrow_down:

...src/resolvers/generators/dynamodb-vtl-generator.ts
92.85% <88.88%> (-7.15%)
:arrow_down:

...aphql-index-transformer/src/resolvers/resolvers.ts
95.00% <96.42%> (-0.02%)
:arrow_down:

...transformer/src/graphql-primary-key-transformer.ts
97.97% <100.00%> (-0.10%)
:arrow_down:

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


Better result reporting

At the moment import_data() is a monolithic function with no easy customization points. In order to provide better feedback to users, I would like to extend it to report the total number of rows in each result type (NEW, UPDATE, SKIP, DELETE), including the total number of skipped rows even when report_skipped == False
There is no easy way to do this without copying the whole import_data() function into the subclass because there is no way to override Result class used, or the handling of individual rows.
I could achieve my requirements by making a slight change to import_data() to count the total number of each result type.
If it is desirable, I could also:

replace the result = Result() with result = self.get_result_class()() to allow people to customize the Result if necessary
move the inner part of the for row in dataset.dict: to a new import_row() method that would allow people to customize what happens before and after the row import is attempted
add a report_error() and a report_row_result() to hold the code that appends the results to the result, once again providing an additional extension point for people that need additional reporting

@rhunwicks that looks sensible to me,  it wouldn't break anything and it would be easier to customise importing process


DO NOT MERGE: test external-dns v0.14.2

/retest
Persistent?

/retest
level=fatal msg=failed to fetch Cluster Infrastructure Variables: failed to fetch dependency of "Cluster Infrastructure Variables": failed to generate asset "Platform Provisioning Check": api.ci.azure.devcluster.openshift.com CNAME record already exists in ci-op-sdmqlkdq-77071 and might be in use by another cluster, please remove it to continue

https://github.com/kubernetes-sigs/external-dns/pull/3301 added search parameters which now force to specify the view. Without a view the following error is produced during the attempt to create a record:
time="2024-08-05T22:27:09Z" level=info msg="Creating CNAME record named 'xxx'
2024/08/05 22:27:09 CreateObject request error: 'WAPI request error: 404('404 Not Found')
Contents:
{ "Error": "AdmConDataNotFoundError: View  not found",
  "code": "Client.Ibap.Data.NotFound",
  "text": "View  not found"
}

Closed in favor of https://github.com/openshift/external-dns-operator/pull/228.


Separate client/server transports

This PR improves the client/server communication interfaces by supporting optional separate Addresses for clients and servers. Each server can specify a client Address and Transport through which clients can communicate with the server. This will potentially allow for different types of transports and protocols in the future. The addition of client addresses necessitated some significant cleanup of the configuration system. In order to ensure client addresses are propagated, servers submit a JoinRequest to the leader which logs and replicates it as a normal configuration change. This means configuration changes now take place absent the actual quorum size changing as they're used to essentially identify nodes.
This PR should be reviewed by going through the commits sequentially. They should be easy to understand. Commits towards the end are mostly bug fixes.

BTW this also merges #64 so we should get that merged first.

The failing tests here are new. Have to take a look at those.

I managed to look through all the commits and overall I am fine with the proposal. Since this is a big commit it was hard to cross check every piece of logic. But overall this seems reasonable to me.
From my side:  :+1:
Sorry for the delay in reviewing this. Holidays and the large commit size are my excuses :)

Oh I know no worries. I didn't expect this to get reviewed very quickly. Since this is probably the last big PR and I don't want to make it any bigger, I think I'll merge it some time today after some more testing so we can move on to snapshots and smaller commits to clean up any remaining items. Once this and whatever comes of the snapshot PR are merged there will be a feature freeze until after a 1.0 release, so should be small commits - bug fixes and structural changes - from here on.

On Nov 30, 2015, at 1:32 PM, Madan Jampani<EMAIL_ADDRESS>wrote:
I managed to look through all the commits and overall I am fine with the proposal. Since this is a big commit it was hard to cross check every piece of logic. But overall this seems reasonable to me.
From my side:
Sorry for the delay in reviewing this. Holidays and the large commit size are my excuses :)
—
Reply to this email directly or view it on GitHub.