Made 'db.url' more complete and some minor refactor

Normalize Ecto URI and Config the same way that ecto does it here
Normalizing the URI allows for:

db.url to be more complete
net.peer.name to be added conditionally
net.peer.port to be added conditionally

Using Ecto.Repo.Supervisor.parse_url/1 to normalize the ecto URI requires that ecto is available at runtime so I added the oldest version of ecto that has what we need (v0.15.0). This seems like a reasonable dependency to add given that these are ecto metrics that are being collected.

This is looking good now, just 2 suggestions from @bryannaegele and some conflicts to resolve @spencerdcarlson

@tsloughter I believe I resolved all conflicts.

@tsloughter any update on this?

@spencerdcarlson there are still a few outstanding requests that have not been addressed, particularly around adding ecto as a dep and the extra processing around URI strings. Once those have been addressed, I can do a final review with final suggestions/changes.


Don`t work when language is jsx

[walter-ribeiro.full-react-snippets]:
Unknown language in contributes.full-react-snippets.language. Provided value: jsx
【setting.json】
{
"editor.snippetSuggestions": "top",
............
}

Hi, do you can provide more information about this, for example when it occurs, it's a VScode error?
what's the type of your file in your IDE?

@qk0728 try to update the extension to 1.4.1
Thanks!

Hi, do you can provide more information about this, for example when it occurs, it's a VScode error?
what's the type of your file in your IDE?

Here is my file type.
====>
And...The current version of extension is 1.4.1.

But......The extension can work well in a TypeScript React file.

try to reinstall because I remove the jsx from contributes in e1fa232fe2dde93e46caf90e1da086fc006c6377


Fix to Get after hook throwing forbidden error for get only condition

Hello. This is working for me and tests are passing.

looks good to me. I leave this here for now since I work on another topic for now (#5). I'll inform you, when they are merged and a new version is out (hopefully this week).
Thanks for the contribution and the nice report in #6 !

This is part of the pre-release: ^0.1.8-0
Thanks for the PR and push in the right direction! I integrated it more consequently. Changed the tests to work with read and find/get. Also the channels now get filtered by action get, which was also read before.
Please try it again with npm i<EMAIL_ADDRESS>and give feedback.
Closing this, as it is part of #5. Thanks again for the PR! Let me know, how it went!


Feature Request: support local config file

Currently, we edit _config.yml file directly to do customization. However, when merge the latest commits, conflicts will occur because of _config.yml changes.
Maybe local config file could be supported, the settings in it could override the default configuration file.
Thanks!

Sorry I still do not understand what the "local config file" means and how to realize it.

@tufu9441 I think he means:
When we do git pull in themes/maupassant/ , without extra operations, the _config.yml in this (remote) repository would overwrite the local _config.yml .


JN-372, DSEC-659 include sonar in Java CI

I'm reworking to separate the different subproject scans so that SonarQube is happy.

mark it down and a link too


Docs improvement: collapsible method needs parameter in form section

Package
filament/filament
Package Version
v1.9
Laravel Version
v8.40
Livewire Version
No response
PHP Version
PHP 7.4
Bug description
Error occured, if no parameter is supplied to collapsible() method in form section ( specified in log output )
Docs : https://filamentadmin.com/docs/1.x/admin/forms#section ( parameter is no given in docs)
But, collapsible method works when have parameter like true.
Steps to reproduce
use collapsible method without any parameter
No response
Relevant log output
ArgumentCountError
Too few arguments to function Filament\Forms\Components\Section::collapsible(), 0 passed in F:\playground\projects\laravel-filament\app\Filament\Resources\CustomerResource.php on line 39 and exactly 1 expected

public function collapsible($collapsible)

    {

        $this->configure(function () use ($collapsible) {

            $this->collapsible = $collapsible;

        });

        return $this;

    }

Hey, I am not actively working on v1 anymore, but would welcome any PRs to solve issues you are having.


Add Command tests

Codecov Report

Merging #157 (fe34e75) into master (2e5f133) will increase coverage by 2.18%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master     #157      +/-   ##
============================================
+ Coverage     67.66%   69.85%   +2.18%
- Complexity      473      492      +19
============================================
  Files            83       84       +1
  Lines          1540     1549       +9
  Branches        172      173       +1
============================================
+ Hits           1042     1082      +40
+ Misses          445      413      -32
- Partials         53       54       +1

Impacted Files
Coverage Δ

...ain/java/foodwhere/logic/commands/RAddCommand.java
91.30% <0.00%> (+91.30%)
:arrow_up:

...ere/model/stall/NameContainsKeywordsPredicate.java
100.00% <ø> (ø)

...where/model/review/NameContainsStallPredicate.java
71.42% <71.42%> (ø)

src/main/java/foodwhere/model/ModelManager.java
95.23% <100.00%> (+8.14%)
:arrow_up:

src/main/java/foodwhere/model/AddressBook.java
87.50% <0.00%> (+4.16%)
:arrow_up:

src/main/java/foodwhere/model/review/Review.java
93.47% <0.00%> (+6.52%)
:arrow_up:

...in/java/foodwhere/logic/commands/RListCommand.java
100.00% <0.00%> (+75.00%)
:arrow_up:

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


Avoid traceback in exit plugin if worker build doesn't exist

Following traceback is raised when worker builds are not available
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/atomic_reactor/plugin.py", line 265, in run
    plugin_response = plugin_instance.run()
  File "/usr/lib/python3.6/site-packages/atomic_reactor/plugins/exit_remove_worker_metadata.py", line 28, in run
    worker_builds = build_result.annotations['worker-builds']
TypeError: 'NoneType' object is not subscriptable

Signed-off-by: Martin Bašti<EMAIL_ADDRESS>Maintainers will complete the following section

[x] Commit messages are descriptive enough
[x] Code coverage from testing does not decrease and new code is covered
n/a JSON/YAML configuration changes are updated in the relevant schema
n/a Changes to metadata also update the documentation for the metadata
n/a Pull request has a link to an osbs-docs PR for user documentation updates
n/a New feature can be disabled from a configuration file

looks good, just unittest

Unittest added


More packages :)

Use official libPNG Repo and bump Version
Update Zlib
Fix OpenTri and update to later commit

Thanks!


docs: fix service name

Nothing major, fixing the service name for Azure data source

cc @Dynatrace-Reinhard-Pilz

Thanks for the reminder!


Fixed instances of e.g. and i.e.

Title
On the title describe
what you've fixed (or created) with this Pull Request (PR).
Summary
Replaced all instances of e.g. and i.e. in .md files with "for example," and "that is,".
Insert a short (one or two sentence) summary here.
Fixes #Issue_Number

Note: The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.
Remove that line if you don't have issues associated with this
PR. Click on the Guidelines for Contributing link above for details.

Details
Explain your changes, and why you made them. If that
information is already available in the issue referenced
above, just referencing the issue is preferred to copying
the text.
This may not be necessary depending on the scope of the PR
changes. For example, "fix typo in introduction.md" is
sufficient to describe that PR.
Suggested Reviewers
@mairaw
If you know who should review this, use '@' to request a review.

@tompratt-AQ,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@tompratt-AQ, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

Addressed the review comments and took all of the suggestions from rpetrusha.


Detection by extension made case-insensitive

This is the implementation of an idea @larsbrinkhoff suggested in #2072.
With this pull request the search by file extensions is case-insensitive.
I wanted to see if it creates any issue with the detection.
It actually solves a few incorrect detections and I think it globally makes sense:
For instance, all .e files in the Rosetta project are detected as Eiffel but most of them are actually E code.
With this pull request the detection results get a lot better.
In general, it looks like lower-case and upper-case extensions are regularly mixed. See #2072, #308, #268 or #1035 for examples.

I've always thought this was weird. @bkeepers this seems like a good idea to me?

:+1:


GoogleAnalytics is deprecated

WARN  deprecated: .Site.GoogleAnalytics was deprecated in Hugo v0.120.0 and will be removed in a future release. Use .Site.Config.Services.GoogleAnalytics.ID instead.

We use new way:


Some f-strings in utils.py are broken (missing the letter "f")

Without this patch, exact_log2(3) will print "{n} is not a power of 2" instead of "3 is not a power of 2" as intended

Actually, care to add a test?

Also, your commit message should be formatted something like
utils: F-strings are missing the letter "f"

Did a forced push with a corrected summary plus a test.
I tested with pdm install && pdm run test and although I am almost totally certain the test is correct and passes, I'm not sure I ran the tests right because I got 10 failures in other areas. I think this test is right though because when I changed "not" to "nfot" in the test string the failure count went up to 11 :P

Did a forced push with a corrected summary plus a test.
I tested with pdm install && pdm run test and although I am almost totally certain the test is correct and passes, I'm not sure I ran the tests right because I got 10 failures in other areas. I think this test is right though because when I changed "not" to "nfot" in the test string the failure count went up to 11 :P

I don't actually see any test?

Apologies, should have got that the first time.


The ability to remove any element from the output

Missing functionality
Fields like:

value_counts_without_nan
value_counts_index_sorted
Are fairly unnecessary for a lot of what I do and really bloat the file and system when used.  Could there be a way to have a list of non-compute or excluded attributes that are just ignored and don;t end up in the final JSON?  Other that that it's great!  I'm just manually deletin them right now, which isn't the worst.

Proposed feature
have an "excluded_fields" element in the config.
Alternatives considered
I've been deleteing them after JSON report generation but before saving down.
Additional context
No response

... I posted something on SO if this is solveable without a feature https://stackoverflow.com/questions/79257442/can-ydata-value-counts-be-disabled-i-dont-need-or-want-them-and-it-creates-un


Dynamic inventory for ansible

Env Details:

Ansible version (ansible --version): 1.9.4
Python version (python --version): 2.7.10
Git commit hash or branch:  master
Cloud Environment: OpenStack
Terraform version (terraform version): 0.6.11

Is there a way to dump the terraform states as a ansible inventory file?

I found this - https://github.com/adammck/terraform-inventory
Is there a reason that you can't use plugins/inventroy/terraform.py --hostfile ?

I tried the above before i posted as that doesn't seem to work.
Basically i am trying to deploy openshift via its' ansible playbook and it needs inventory file to be statically available and the above plugin didn't solve the problem.
I tried with terraform.py plugin and that also doesn't seem to work with https://github.com/openshift/openshift-ansible/playbook/byo  on branch openshift-ansible-3.0.35-1
Though if i could dump the ansible inventory file generated via terraform.py i could make it work with the above playbook

So I think you'd want to open an issue on that repo with guidance on how to use a dynamic inventory there. I don't advise trying to use your terraform from Mantl, things would likely be stomped on imo.

@heckdevice you probably want to take a look at the bare metal instructions for creating an inventory file https://github.com/CiscoCloud/mantl/tree/master/docs/bare-metal#creating-your-inventory

@stevendborrelli @lowwalker  Thanks for the input. I will live for now with manual inventory files, but it would be a nice feature to have as it gives a faster integration with more playbook


backup: implement backup controller for backup, restore and migrate

What type of PR is this?
/kind api-change
/kind feature
What this PR does / why we need it:
implement backup controller for backup, restore and migrate
see #374
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/rerun

/retest

@hzxuzhonghu  PTAL

This PR will be split into multiple smaller PRs because the current one is too large


Mirror Mode

Mirror Mode misses!

@MichaelDiMeo Do you mean Flip Horizontally / Vertically when you say Mirror Mode?

@guoyingtao Yes! I've tried to implement by myself but I had a lot of problems with your code. For example I can't add outlet to the view.

@guoyingtao Anyway, congrats for this amazing tool!


Needs an option to not immediately send the first wave

On a small screen, the game is kind of finicky feeling, making real-time management of towers less fun than intended.

What if you hit the "pause" button immediately after starting the level? Wouldn't that help?

Other TD games I've played either do not send the first wave before you
click a button to do so or are generous with the a 30 second delay before
sending the first wave
On Wed, 25 Sept 2024, 10:09 ochadenas, @.***> wrote:

What if you hit the "pause" button immediately after starting the level?
Wouldn't that help?
—
Reply to this email directly, view it on GitHub
https://github.com/ochadenas/cpudefense/issues/180#issuecomment-2373370131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPIOP44SB3SRZMMMI4DNPTZYJVVDAVCNFSM6AAAAABOZZ6TVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZTGM3TAMJTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I fear that introducing a 30-seconds-delay would generate a storm of unfriendly comments here. Better not try that.
The button would be feasible, but I don't really see the necessity. I really often use the pause button. Just to look around, get an idea of the level layout, and buy the first chips. Because since version 1.27, chips can be bought and sold in paused state.
The new button would only save one tap. On the other hand, on easier levels or if you don't want to think before the first wave, it would require one more tap. So there is not much to be gained.

If it's on by default yes, but I didn't specify it should be (my suggestion
would opt-in)
On Fri, 4 Oct 2024, 14:51 ochadenas, @.***> wrote:

I fear that introducing a 30-seconds-delay would generate a storm of
unfriendly comments here. Better not try that.
The button would be feasible, but I don't really see the necessity. I
really often use the pause button. Just to look around, get an idea of the
level layout, and buy the first chips. Because since version 1.27, chips
can be bought and sold in paused state.
The new button would only save one tap. On the other hand, on easier
levels or if you don't want to think before the first wave, it would
require one more tap. So there is not much to be gained.
—
Reply to this email directly, view it on GitHub
https://github.com/ochadenas/cpudefense/issues/180#issuecomment-2393637456,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPIOPY3KZJWULHZAIJP2ELZZ2FMFAVCNFSM6AAAAABOZZ6TVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJTGYZTONBVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'd like to close this issue. I think that this would only add complexity without any real benefit.

I disagree with your assessment, but I respect your decision. Keep up the good work.

I second this suggestion. The wave would start paused, and the player would need to unpause.
Make this an option and each will be satisfied with their own preferences :)


SelfUpdater

Hi @gouravkhunger, I don't know if you still maintain this app – but it seems to include a SelfUpdater. Which is against the IzzyOnDroid App Inclusion Policy. Is that updater opt-in? Does it make the implications clear (e.g. that it bypasses the security checks performed in the IzzyOnDroid repo)? A quick scan of the code seems to say "no to all": it looks like the check is performed automatically, and that there's no setting to even disable it. Update check is performed without any "conditional".
I don't like setting an ultimatum, but as this repo here didn't see any activity for almost 2 years and I don't know if it is still active, this time I have to: without any response here within a month, the app will be removed from IzzyOnDroid at the end of that period. With a response, it depends of the planned action from your end of course. Thanks for understanding!

Hi @IzzySoft, that part is not a "self updater". It just checks if there are newer app versions available. And shows the option to open the latest github release page:

I agree the repo hasn't been active. I'm sorry I can't give a timeline on when I plan to do any refactors next.

I guess there's no chance to make this opt-in in the near future?

On it. Will probably release something this weekend.

Thanks! :star_struck:

@IzzySoft Released v2.1.2
The setting is now disabled by default:
https://github.com/gouravkhunger/QuotesApp/blob/e9c184d6b90af963a4ac51251154d6535c1309ce/app/src/main/java/com/github/gouravkhunger/quotesapp/store/PreferenceStore.kt#L39-L47
https://github.com/gouravkhunger/QuotesApp/blob/e9c184d6b90af963a4ac51251154d6535c1309ce/app/src/main/java/com/github/gouravkhunger/quotesapp/ui/QuotesActivity.kt#L129

Thanks! Is the toggle also explained as outlined above?

Does it make the implications clear (e.g. that it bypasses the security checks performed in the IzzyOnDroid repo)?

Yes

Thanks! That makes clear where the updates are taken from. But where does that mention the implications (like bypassing the additional scans performed at e.g. IzzyOnDroid)? Maybe you could append something like

[… please update from there], as e.g. additional security measures implemented there would be bypassed.

Always good to be as transparent as possible, to allow well informed decisions :wink:

@IzzySoft I understand that it is good to be transparent. Not denying updating the text. But, a new release for that minor text update doesn't seem enough tbh. I could do that when I implement some new features that are together worth a new release.
I see that the app version on your repo is still 2.1.1 while it's been a few days since 2.1.2 was released. Could you check why it didn't get picked? Thanks!

I could do that when I implement some new features that are together worth a new release.

That must suffice then. Hopefully it's not too far in the future, as I'll get daily warnings until then :see_no_evil:

Could you check why it didn't get picked?

Sure. And easy answer:

2023-07-13 UCM Tags => Static (last release 3/2022, last commit 10/2022)

It was "demoted" to monthly checks due to lack of activity. As there have not been at least 2 new releases within 3 months yet, it wasn't yet "promoted" back. Do you plan more frequent updates again? For the currently latest version, I've just manually triggered an update scan, which btw. yielded:
! repo/com.github.gouravkhunger.quotesapp_5.apk contains signature block blobs: 0x504b4453 (DEPENDENCY_INFO_BLOCK; GOOGLE)

Can easily be solved with a minor addition to your build.gradle:
android {
    dependenciesInfo {
        // Disables dependency metadata when building APKs.
        includeInApk = false
        // Disables dependency metadata when building Android App Bundles.
        includeInBundle = false
    }
}

For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it – and nobody else can even verify what it really contains. More details can be found e.g. here: Ramping up security: additional APK checks are in place with the IzzyOnDroid repo.

@gouravkhunger where do we stand here with the self-updater? It's 3 months now. When I wrote it "must suffice", 6 weeks ago, I didn't expect nothing would happen for more than a month. Any ETA there? I still get the daily warning here :see_no_evil:

a new release for that minor text update doesn't seem enough tbh.

Well, now you have "that minor text update" (did I miss the commit for that?) – and "that minor update to your build.gradle to remove a blob". Maybe that's worth a maintenance release? Considering it would free you from my nagging at well? :stuck_out_tongue_winking_eye:

@IzzySoft My bad! I've been overloaded with work since long. Hope this is better now? I've also added your fix to exclude the blob. Thanks!

Thanks! Yes, that looks excellent! Pulled the update in and, finally, could add the AppUpdater lib to your app's "green list" :star_struck:
And no worries, I know what overloaded means. So I was not complaining, just sending a "friendly nudge" :wink:


Monthly report 202207

I created the report on a branch, markdown is easier to edit on github...

For quarterly board reports, we currently use https://reporter.apache.org/wizard/?sdap and then send out draft to<EMAIL_ADDRESS>for feedback before submission.


immer-reducer works with React useReducer hook too

Just realized that immer-reducer works with the React useReducer hook just fine because it is just a reducer generation lib and no deps with Redux whatsoever.
https://github.com/epeli/immer-reducer#-react-hooks

:tada: This PR is included in version 3.1.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update dependencies

Coverage remained the same at 86.116% when pulling 585abfca93e118b17a51b17a10298fe3f7e1d221 on rel_branch into da9f36fb6e2a1478faaa6b3e84770a3f116f0288 on master.


add Rock the JVM?

@daniel-ciocirlan do you have an RSS feed that could be aggregated here?
I was looking at https://github.com/softinio/scalanews/blob/main/docs/Resources/Blog_Directory.md and trying to think of obvious things that were missing and Rock the JVM was one of the first things that came to mind

Yes! Rock the JVM blog RSS

Thanks both. On it. Will be included in my next PR which is imminent.

resolved by #16


Allow configuration of admin user and password via environment vars

Many official Docker images (and it looks like this Docker image is going to become the official Docker image for couchdb) support the definition of the admin user name and password via environment vars, e.g. MySQL. It would be great if this image would support this as well.

I'd be open to that, even though it is freakishly simple to create an admin user for CouchDB in different ways. local.ini, REST API, Fauxton...
But in order to align with other DBs and make this common configuration task a littler simpler, it is something to consider. :)

I am only writing this because you have the following in the description of the 2.0-dev container description:
…
Admin username: root
Password: 37l7YDQJ
…

Being able to describe credentials via env args would make it incredibly easy to bring up CouchDB for dev purposes (as it would only be a one liner) :).

Oh in the case of 2.0 you can do docker run -it -p 5984:5984 klaemo/couchdb:2.0-dev --with-haproxy --admin=USER:PASS to set the admin user :) Does that help?
Check out docker run -it -p 5984:5984 klaemo/couchdb:2.0-dev --help for all available options.

Yes, that will help. Danke Clemens :)

Sometimes environment vars are your only option. Ie. if you use couchdb in Kitematic.

Thanks for the feedback @jakobrosenberg. It's on my list :)

fixed by #46. will land in the official image soon :)


[YouTube] youtube:skip=translated_subs doesn't work for some videos.

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.05.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Description
The translated_subs argument introduced in version 2022.04.08 only works in some cases.
yt-dlp works as intended in this example:
yt-dlp --ignore-config --write-subs --write-auto-subs --sub-langs all --extractor-args "youtube:skip=translated_subs" "https://www.youtube.com/watch?v=T2kS1gAbxhc"

It downloads 26 subtitles, skipping all the auto-translated ones.
But it doesn't work for this video: https://www.youtube.com/watch?v=bOXCLR3Wric
Verbose log
[debug] Command-line config: ['--verbose', '--ignore-config', '--write-subs', '--write-auto-subs', '--sub-langs', 'all', '--extractor-args', 'youtube:skip=translated_subs', 'https://www.youtube.com/watch?v=bOXCLR3Wric']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.05.18 [b14d52355]
[debug] Python version 3.9.12 (CPython 64bit) - Linux-5.17.9-x86_64-with-glibc2.34
[debug] Checking exe version: ffprobe -bsfs
[debug] Checking exe version: ffmpeg -bsfs
[debug] exe versions: ffmpeg 4.4.1 (setts), ffprobe 4.4.1, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.14.1, brotli-1.0.9, certifi-2021.10.08, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.1
[debug] Proxy map: {'no': '<IP_ADDRESS>,localhost', 'ftp': 'http://<IP_ADDRESS>:1081', 'https': 'http://<IP_ADDRESS>:1081', 'http': 'http://<IP_ADDRESS>:1081', 'all': 'http://<IP_ADDRESS>:1081', 'rsync': 'http://<IP_ADDRESS>:1081'}
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=bOXCLR3Wric
[youtube] bOXCLR3Wric: Downloading webpage
[youtube] bOXCLR3Wric: Downloading android player API JSON
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, codec:vp9.2, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[debug] Downloading subtitles: en, af, sq, am, ar, hy, az, bn, eu, be, bs, bg, my, ca, ceb, zh-Hans, zh-Hant, co, hr, cs, da, nl, en-orig, eo, et, fil, fi, fr, gl, ka, de, el, gu, ht, ha, haw, iw, hi, hmn, hu, is, ig, id, ga, it, ja, jv, kn, kk, km, rw, ko, ku, ky, lo, la, lv, lt, lb, mk, mg, ms, ml, mt, mi, mr, mn, ne, no, ny, or, ps, fa, pl, pt, pa, ro, ru, sm, gd, sr, sn, sd, si, sk, sl, so, st, es, su, sw, sv, tg, ta, tt, te, th, tr, tk, uk, ur, ug, uz, vi, cy, fy, xh, yi, yo, zu
[debug] Default format spec: bestvideo*+bestaudio/best
[info] bOXCLR3Wric: Downloading 1 format(s): 313+251
[info] Writing video subtitles to: The unreasonable effectiveness of complex numbers in discrete math [bOXCLR3Wric].en.vtt
[debug] Invoking http downloader on "https://www.youtube.com/api/timedtext?v=bOXCLR3Wric&caps=asr&xoaf=5&xosf=1&hl=en&ip=<IP_ADDRESS>&ipbits=0&expire=1653592120&sparams=ip%2Cipbits%2Cexpire%2Cv%2Ccaps%2Cxoaf&signature=CE03F84903AC811E4A7EC49EEAA05D7F498B2C3C.ADF5D40DE8FEAA19172D9F174D18C543149E267E&key=yt8&lang=en&fmt=vtt"
[download] Destination: The unreasonable effectiveness of complex numbers in discrete math [bOXCLR3Wric].en.vtt
[download] 100% of 65.30KiB in 00:00
[info] Writing video subtitles to: The unreasonable effectiveness of complex numbers in discrete math [bOXCLR3Wric].af.vtt
[debug] Invoking http downloader on "https://www.youtube.com/api/timedtext?v=bOXCLR3Wric&caps=asr&xoaf=5&xosf=1&hl=en&ip=<IP_ADDRESS>&ipbits=0&expire=1653592120&sparams=ip%2Cipbits%2Cexpire%2Cv%2Ccaps%2Cxoaf&signature=CE03F84903AC811E4A7EC49EEAA05D7F498B2C3C.ADF5D40DE8FEAA19172D9F174D18C543149E267E&key=yt8&kind=asr&lang=en&tlang=af&fmt=vtt"
[download] Destination: The unreasonable effectiveness of complex numbers in discrete math [bOXCLR3Wric].af.vtt
[download] 100% of 354.51KiB in 00:00
[info] Writing video subtitles to: The unreasonable effectiveness of complex numbers in discrete math [bOXCLR3Wric].sq.vtt
[debug] Invoking http downloader on "https://www.youtube.com/api/timedtext?v=bOXCLR3Wric&caps=asr&xoaf=5&xosf=1&hl=en&ip=<IP_ADDRESS>&ipbits=0&expire=1653592120&sparams=ip%2Cipbits%2Cexpire%2Cv%2Ccaps%2Cxoaf&signature=CE03F84903AC811E4A7EC49EEAA05D7F498B2C3C.ADF5D40DE8FEAA19172D9F174D18C543149E267E&key=yt8&kind=asr&lang=en&tlang=sq&fmt=vtt"
[download] Destination: The unreasonable effectiveness of complex numbers in discrete math [bOXCLR3Wric].sq.vtt
[download] 100% of 376.53KiB in 00:00
^C
ERROR: Interrupted by user

skip=translated_subs hides subs that are auto-translated from normal subtitles. These are just auto-generated subs of different languages.

It feels somewhat counterintuitive to me because initially I thought skip=translated_subs would skip everything in Auto-translate:

I wonder if there is a way to achieve that.

There is always only one "original" auto-subs. This is labeled as (Original) (lang ends with -orig) in --list-subs

Thank you for your explanation. My use case is to download all non auto-subs + English (auto-generated). Now I am able to do that with:
yt-dlp --write-subs --sub-langs all "https://www.youtube.com/watch?v=bOXCLR3Wric"
yt-dlp --write-auto-subs --sub-langs en-orig "https://www.youtube.com/watch?v=bOXCLR3Wric"

Don't use --write-auto-subs? All auto-subs are machine translated

The "auto-generated" speech-to-text sub/caption (e.g. English sub of English video) is the special case. Because non-native speakers or accent problems in some videos, It makes it easier for many audiences to watch.
Therefore the "auto-generated speech-to-text sub/caption" could be an exception from "-auto-subs".

[info] Available automatic captions for xxx:
Language        Name
en-orig         English                                            Speech to text, in spoken language
es              Spanish                                            Speech to text
en-es           English from Spanish                               Auto-translated from en subs

[info] Available subtitles for xxx:
Language Name
en       English                 Uploaded by creator

The both "en-orig" and "en".
Because the former contains additional: "sound effects, relevant musical cues, and other relevant audio information", and any missing translation in the latter.
I usually only saw one "auto-generated" sub, which is usually english on youtube web, so what is the "Spanish                                            Speech to text"?

I wrote it to explain

Issues are closed when solved. Which means, #2262 is not yet implemented and running 2 commands is your best workaround for now


Include circuits, data, init_funcs in setup.py

Describe the bug
Installing liionpack using pip install -q git+https://github.com/pybamm-team/liionpack.git@main does not include circuits, data, init_funcs folders in the package

To Reproduce
Steps to reproduce the behaviour:
Run 02 Simulating a bespoke pack using an LTSpice netlist and custom heat transfer.ipynb from examples/notebooks
TO DO
Include these folders in setup.py
For Reference:
https://github.com/pybamm-team/PyBaMM/blob/5fbfec8a46945e99106cc166c283201513066425/setup.py#L136-L160

@TomTranter @wigging what files and extensions from these folders should I include in the package?

We might want to move all of these to a features folder. I'm not sure what to do with init_funcs as it was used for setting intial SoC which we changed how to do but actually for a pack problem where you might want to intialize with a bunch of different values it could be useful. The other two are quite bespoke and used for an example of loading in your own netlist. For now include everything


ipfs.files.write hangs without error message with certain arguments

Opening an issue here per @achingbrain's response to my IRC post.

Is it immediately obvious to anyone what would happen if you passed the string '/[object Object]' as the path in the ipfs.files.write method in js-ipfs? Is it possible it can't handle either spaces or square brackets in the string and is not returning any errors for that condition? Other incorrect things I pass in (like objects) cause error messages, but this (or other examples of strings with spaces and/or square brackets) appear to just hang. I can't guarantee it's IPFS's fault because of the context within ProtoSchool, but this is my working theory at the moment. 😂 For full context and to see what other theories I've experimented with, see the ProtoSchool issue:  https://github.com/ProtoSchool/protoschool.github.io/issues/252

per @achingbrain:

I don't think it's related to the path, I think it's something to do with the file content argument. That is, writing Buffer.from('hello world') to /[object Object] works for me but writing the passed file argument hangs as described in that issue
Could you open an issue on the js-ipfs-mfs repo with a reference to the protoschool issue above please and I'll look into it next week
Weird, adding the file to IPFS via ipfs.add(file) errors out too, I guess it's failing to turn it into a stream

I'm not sure I understand this theory, since I have no trouble writing the same kind of browser file object in general when I provide reasonable strings as the path (from that ProtoSchool lesson).
To be abundantly clear, I don't need to be able to write things to the path /[object Object], I'd just love for it to return an error if it's not going to work. :) (Error messages sent back by IPFS get surfaced as the feedback we show to the user in ProtoSchool unless we've provided a fancier error message for something predictable.)
I also readily admit the possibility that it's a ProtoSchool problem as opposed to an IPFS problem, just need an IPFS pro to help me confirm the source. Thank you!!

@achingbrain and I were able to confirm this was a ProtoSchool issue, not an IPFS issue. :)


[NEW]: Atomic Wallet failed to address security issues found in security review by Least Authority

Date
2022-02-10
Description
Least Authority made a security disclosure to inform the public that they consider the Atomic Wallet to be insufficiently secure in protecting user assets and private data after Atomic Wallet failed to remediate issues found in security review.
Links
https://leastauthority.com/blog/disclosure-of-security-vulnerabilities-in-atomic-wallet/
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
No response
Personal link
No response

Thank you! https://web3isgoinggreat.com?id=2022-02-10-0


Renaming the tmux-pomodoro to pomodoro

Hi,
I downloaded the tar version and bin file has tmux-pomodoro instead of instructions supplied which is pomodoro, chaning the name from tmux-pomodoro to pomodoro worked for me, can you look into it

Plus 1. The README and the app's help-message both refer to an executable named pomodoro, but the real command appears to be bin/tmux-pomodoro.

Thanks for the feedback! I've switched over to using open-pomodoro/openpomodoro-cli (which honestly might have the same issue in the release package), but if you'd like to open a PR changing the release process here, I'd happily merge it (and bestow collaborator privileges for future code review).

That's great to hear, Justin. Might be worth mentioning in the README of this repo that it has been superceded by open-pomodoro/openpomodoro-cli.


New repository: helm-charts

We should create a repo call "helm-charts" or "charts" to provide more openness and visibility to OCM related helm chart(s) that are referenced on our website. For example, https://open-cluster-management.io/getting-started/integration/cluster-proxy
It will be nice if the repo can also host the charts and have CICD to manage the release lifecycle of the chart as well.
If we follow the guide in
https://helm.sh/docs/topics/chart_repository/#github-pages-example
https://helm.sh/docs/howto/chart_releaser_action
we might be able to accomplish this but there is some concerns regarding the frequency of the release.
Before we create repo and start using the above GitHub action, we should investigate a bit more and see if the tooling matches our need. One of the criteria is, if it can reliability replace an existing release with the same version number.

+1

Update: The Helm chart releaser GitHub action doesn't allow for the same version of the chart. At minimal it needs a patch version increment.
But I think regardless of if we end up using the chart releaser or not, we should still have this repo to host the source code of some of our Helm charts to bring more openness/visibility to the community.

I would want to clarify what content should be put in this repo. I think it will include:

helm charts of addons maintained in this repo (subscription/policy/cluster-proxy etc)
helm charts of external addons (submariner/clusternet etc)

With that we could have an addon installation flow:
$ helm install addon-foo
$ clusteradm enable addon-foo --cluster cluster1

I am not sure about the basic installation of ocm. Given it can be done by clusteradm already.

I would want to clarify what content should be put in this repo. I think it will include: ...

I agree with you Qiu Jian.

The convention is to embed the chart with the repo it packages. I am not sure if we need another repo for this


Redpanda kafka connect not populating

Redpanda kafka connect is not populating the debezium kafka connect (dbz-connect:8083)
Try moving the service in the same namespace
Try different configurations
Verify against the docker-compose version

The example docker config works
Got this working by using the separate redpanda helm chart for the console.
This accepts the config values beyond what is generated on just the redpanda chart. That was annoying


Update roadmap.md

Closes #
Proposed Changes

I submit this contribution under Apache-2.0 license.

Information added in PR #2176 by @nunoocx


[Backport 2.4.x] bugfix(#5755): avoiding deadlock between builds with dependencies bui…

…ld order strategy when the builds share enough dependencies

This is backport of #5755 to release-2.4.x

Release Note
Fixed dependencies build order strategy to avoid deadlocks

@lsergio please rebase with release-2.4.x. I have fixed the CI.


Remove missing media type workaround

Once https://github.com/GSA/fedramp-automation/issues/103 has been closed, the workaround for missing media types can be removed in OSCALBackMatter.js. Currently, file extension is being used.
https://github.com/EasyDynamics/oscal-react-library/blob/b34991180dc344729d8de8952222de6b072cef79/src/components/OSCALBackMatter.js#L24-L27

It looks like https://github.com/GSA/fedramp-automation/issues/103 is now closed.


[Event Request] codeunit 5944 "SignServContractDoc"

Please add 'OnAddendumToContractOnAfterPostingDateValidate' publisher into the position below.
procedure AddendumToContract(ServContractHeader: Record "Service Contract Header")
...
if InvoiceNow then begin
PostingDate := InvoiceFrom;
OnAddendumToContractOnAfterPostingDateValidate(PostingDate, InvoiceFrom); // --> NEW EVENT
ServContractLine.Reset();
ServContractLine.SetCurrentKey("Contract Type", "Contract No.", Credited, "New Line");
ServContractLine.SetRange("Contract Type", FromServContractHeader."Contract Type");
ServContractLine.SetRange("Contract No.", FromServContractHeader."Contract No.");
ServContractLine.SetRange("New Line", true);
if ServContractLine.FindSet then
repeat
if (ServContractLine."Contract Expiration Date" <> 0D) and
(ServContractLine."Contract Expiration Date" < InvoiceTo)
then
NoOfMonthsAndMParts := ServContractMgt.NoOfMonthsAndMPartsInPeriod(
InvoiceFrom, ServContractLine."Contract Expiration Date")
else
if (FromServContractHeader."Expiration Date" <> 0D) and
(FromServContractHeader."Expiration Date" < InvoiceTo)
then
NoOfMonthsAndMParts := ServContractMgt.NoOfMonthsAndMPartsInPeriod(
InvoiceFrom, FromServContractHeader."Expiration Date")
else
NoOfMonthsAndMParts :=
ServContractMgt.NoOfMonthsAndMPartsInPeriod(InvoiceFrom, InvoiceTo);
RemainingAmt :=
RemainingAmt +
Round(
ServContractLine."Line Amount" / 12 * NoOfMonthsAndMParts, Currency."Amount Rounding Precision");
until ServContractLine.Next() = 0;
end;
...
[IntegrationEvent(false, false)]
local procedure OnAddendumToContractOnAfterPostingDateValidate(var PostingDate: Date, var InvoiceForm: Date)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Make the website actually look good

Currently the Website does not really look that good. We should change that.
To-Do:

[ ] Login page

[ ] Create a card to mimic traditional login prompts
[ ] "Sign in with 42"

[ ] Home

[ ]  Really necessary? Would link directly to Open Issues, show some stats or maybe announcements from the council, or list possible actions for the user (kind of a sitemap)

[ ] Footer

[ ] Is invisible unless you unintentionally scroll down. I like sticky footers but they're hard to see if they are outside of the viewport

[ ] Issues

[x] Remove outer card as it's the main content of the page, no other elements to distinguish it from (#151)
[x] Maybe move the total votes to the left of the title, more similar to Reddit or HN. Currently you have to follow with the eye all the way to the middle of the page making it harder to scan quickly (#157)
[ ] #158
[x] Move Open/Archived under page Title+Description (#151)
[ ] Button text to "Open an issue" (#164)

[ ] Issue #N

[ ] Main title should be Issue title, with issue number as additional information i.e. here on GitHub (#166)
[ ] Make text size consistent between issue description and comments (#166)
[ ] "Go back" button with less important color and aligned on the left
[ ] Put likes value outside of button or reduce the text inside. Currently the value is hard to scan
[ ] Reduce page width to around 90ch (setting max-width: 90ch already works great, actually could be a good measure for other pages as well)
[ ] Maybe hide detailed description about how you save the likes under a details, with a simple summary like "Upvotes are not anonymous"
[ ] Comments

[ ] Style dates consistently with /issues, remove "on"
[ ] Align date bottom right
[ ] Lighter background color for each card (#166)

[ ] Mark archived issues as such (just a simple badge)
[ ] Link to Student Council answer close to initial Issue description

[ ] Other pages

[ ] Usually the hr after the h1 feels unnecessary and repetitive as there's another line with exactly the same style just above it (#165)

Some vague ideas:

Login page

Create a card to mimic traditional login prompts
"Sign in with 42"

Home

Really necessary? Would link directly to Open Issues, show some stats or maybe announcements from the council, or list possible actions for the user (kind of a sitemap)

Footer

Is invisible unless you unintentionally scroll down. I like sticky footers but they're hard to see if they are outside of the viewport

Issues

Remove outer card as it's the main content of the page, no other elements to distinguish it from
Maybe move the total votes to the left of the title, more similar to Reddit or HN. Currently you have to follow with the eye all the way to the middle of the page making it harder to scan quickly
Open/Archived state is not saved during navigation: opening an issue under Archived and going back, navigates to Open Issues
Move Open/Archived under page Title+Description
Button text to "Open an issue"

Issue #N

Main title should be Issue title, with issue number as additional information i.e. here on GitHub
Make text size consistent between issue description and comments
"Go back" button with less important color and aligned on the left
Put likes value outside of button or reduce the text inside. Currently the value is hard to scan
Reduce page width to around 90ch (setting max-width: 90ch already works great, actually could be a good measure for other pages as well)
Maybe hide detailed description about how you save the likes under a details, with a simple summary like "Upvotes are not anonymous"
Comments

Style dates consistently with /issues, remove "on"
Align date bottom right
Lighter background color for each card

Mark archived issues as such (just a simple badge)
Link to Student Council answer close to initial Issue description

Other pages

Usually the hr after the h1 feels unnecessary and repetitive as there's another line with exactly the same style just above it

Enough for now 😅

Thank you for the many ideas Leo :clap:
I will add them as a To-Do list to the Issue :saluting_face:

Another idea regarding opening issues and possibly adding comments:

textarea should resize automatically to fit all text, avoiding scrollbars i.e. what GitHub and most other modern websites do. Scrollbars are generally not great, especially on mobile, as they conceal text outside of the viewport...


ERROR: Dashboard already exists in space 739

We are using the below Ldeploy script for migrating our Dashboards from one environment to another which was working fine previously and is able to update the dashboard in the destination. but now it is giving the below error.
Error:
ERROR: Dashboard already exists in space 739
Command:
ldeploy content import --env destinationEnv --ini lookerEnvFile --debug --recursive --folders /sourceFolderPath --target-folder destinationFolderPath

We are getting the same, in looker 22.16 v

Unsure I'm able to replicate. I believe this is probably more of an issue with Gazer given the error message. Generally the error message is inclusive of using --force to overwrite. The commands issued by Looker Deployer should always include --force.
Can you provide the version of Looker Deployer you are running?
Can you provide the version of Gazer you are running?
Can you attempt just using gazer to move the dashboard that is erroring to see if Looker Deployer or Gazer issue?
Can you verify if this dashboard has a merged result tile?

@Rutvik1806 @abhirupc88 Could you please let us know if this remains an issue for you?  If so, please provide the following per @AdamMinton's request:

Can you provide the version of Looker Deployer you are running?
Can you provide the version of Gazer you are running?
Can you attempt just using gazer to move the dashboard that is erroring to see if Looker Deployer or Gazer issue?
Can you verify if this dashboard has a merged result tile?

This ticket will be closed at the end of next week if no response is received. Thank you!

@Rutvik1806 please reopen issue with responses to the above if you're able to replicate.


Unable to perform OCR decode.    Error: Get http://bit.ly/ocrimage: dial tcp: lookup bit.ly: no such host
But, i can do a wget of the same URL ( == means Internet connection is working)

That sounds like some sort of strange dns issue.
Try uploading your own test image to your own website (or a github gist or something).
Are you on vagrant?
Also, try running "docker logs --follow " on both the worker and httpd processes.
Closing this, because I think its a network setup issue and not an openocr bug.  But feel free to comment on the ticket.


Make All links working on dashboard tab

This enables buttons for the cards on the dashboard to be clickable and take you to the appropriate filtered list.
PV https://www.pivotaltracker.com/story/show/138640897

@miq-bot add_label enhancement
@miq-bot add_label euwe/no


Command that lets you create a seed script based on the current schema and data in your database

When you start using Prisma with an existing database, maybe just a production one, you have to bootstrap your development environment somehow. Probably you would usually do that by taking your current database, and just create a copy.
But our migrate dev development workflows depends on your being able to simply tear down and recreate your development environment (via reset and seed) - which requires you to have certain things: A Prisma schema of your database schema, a migration history that reflects that schema, and a seed script (that works with your schema) to recreate the data.
For the database schema that is already possible via db pull. You can then baseline your database (to create the base migration to get to the initial state), but for seeding we do not have build in support yet. You have to manually write a seed script and make it create useful data.
It could be great if there was a command or tool that create an initial seed script based on the current data in your database, that you then can sanitize and maintain.
(We investigated and partially implemented something like this internally before for PDP, when we created the templates you can choose when signing up and creating a new project without and existing schema and database. @sid405 did that back in the day, but it might still be around somewhere.)

I think it was called sql2pc, and it took in some SQL and "converted" it into a Prisma Client script. This SQL could come from a DB dump, and I guess that's how you'd be able to generate a seed script. (Doesn't even have to be a whole dump, just a few randomly selected rows even)
These days, you might also want to look into snaplet!

follow


Suspending the VM prior to deleting snapshots to avoid corruption, th…

…en resuming
Description

These changes are related to PR #3194, but include suspending/resuming the VM when doing a VM snapshot as well, when deleting a VM snapshot, as it is performing the same operations via Libvirt. Also, there was an issue with the UI/localization changes in the prior PR, as that PR was altering the Volume snapshot behavior, but was altering the VM snapshot wording. Both have been altered in this PR.
Issuing this in response to the work happening in PR #4029.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@andrijapanicsb a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@weizhouapache does this one makes more sense/checks than the #4033  - i.e. shall we work on this one?

Packaging result: ✔centos7 ✔debian. JID-1172

@ggoodrich-ipp my suggestion there ^^ won't catch all possible conflicts, and won't give us a full mutex, but for the case we are handling now it should do. (cc @weizhouapache @rhtyd again)

as we are in release phase already and #4033 is simpler let's go with that. the logic is essentially the same.

@ggoodrich-ipp can you fix the conflicts, thanks.

@ggoodrich-ipp can you fix the conflicts, thanks.

Conflicts resolved.

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos7 ✖debian. JID-1353

@ggoodrich-ipp could you check if quiescevm is true in the createSnapshot API? Would that be better than doing it by default?

@ggoodrich-ipp could you check if quiescevm is true in the createSnapshot API? Would that be better than doing it by default?

That seems questionable, due to the potential for corruption, in this case. I wouldn't think we would want an option that could lead to corruption.

@DaanHoogland @rhtyd I have made some adjustments to this PR. Thoughts?

@DaanHoogland @rhtyd I have made some adjustments to this PR. Thoughts?

@ggoodrich-ipp can you please resolve conflicts

@ggoodrich-ipp can you please resolve conflicts

@ggoodrich-ipp what issue does this pr solve ?
Delete a vm snapshot twice ?

@ggoodrich-ipp what issue does this pr solve ?
Delete a vm snapshot twice ?

@ggoodrich-ipp what issue does this pr solve ?
Delete a vm snapshot twice ?

Well, I think another PR managed to get in there to suspend the VM, but this PR is specifically around suspending, then resuming the VM when doing a snapshot.

@ggoodrich-ipp what issue does this pr solve ?
Delete a vm snapshot twice ?

Well, I think another PR managed to get in there to suspend the VM, but this PR is specifically around suspending, then resuming the VM when doing a snapshot.

I see @DaanHoogland re-targeted this to 4.15, which must have resolved the conflicts. Are there any other issues with this PR that need to be addressed prior to approval, @weizhouapache @shwstppr ? Let me know, and I'll try to accommodate. Thanks!

I see @DaanHoogland re-targeted this to 4.15, which must have resolved the conflicts. Are there any other issues with this PR that need to be addressed prior to approval, @weizhouapache @shwstppr ? Let me know, and I'll try to accommodate. Thanks!

travis is reporting a checkstyle error but i cannot find it closing

travis is reporting a checkstyle error but i cannot find it closing

re-opening to re-kick travis

re-opening to re-kick travis

@ggoodrich-ipp could you please fix the build error ?

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos7 ✖centos8 ✖debian. JID-2745

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos7 ✖centos8 ✖debian. JID-2759

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos7 ✖centos8 ✖debian. JID-2780

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2783

@blueorangutan test

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3594)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 30418 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4032-t3594-kvm-centos7.zip
Smoke tests completed. 86 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

@weizhouapache have you done any manual testing on this?

Did another round of code review, LGTM


output: to_csv: print header for every set of parsed fields

When parsing multiple invoices it's *incorrect* to:
1. Display header with fields names of the *first* parsed invoice
2. Display content of *all* parsed invoices

It's because parsing dfferent invoices may result in different fields.
In such case header won't make sense for all rows.

To improve handling such situations display new header for every set of
fields that differs from the last one.

This also fixes https://github.com/invoice-x/invoice2data/issues/297

Ping @bosd :)

Me neither, I don't think CSV is a good fit for tree structures anyway. I just wanted to fix that issue and noticed an extra problem with doing so.
Thank you for looking at this pull request!


Add network-plugin option

This commit is to add network-plugin option for the use case of testing
CNI implementation with minikube in CI
Signed-off-by: Tam Mach<EMAIL_ADDRESS>

do you mind rebasing your PR?

Done and sorry for delay response, I am just back from leave.


Added my name Ibimina Hart

resolved conflict

sorry your code is still behind, can you update your code to get the latest changes
git pull upstream main

git push origin main

just updated my code

as you can see the last number in the latest code is 47, while you have 28 in your name.text.
you can update to the latest by pulling from the upstream of the original repository

what of now? I am see 48 names in my name.txt. I don't now what I am doing wrong

recent update

Thanks
On Mon, 8 Aug 2022 at 19:42, Success Chikezie @.***>
wrote:

Merged #152
https://github.com/Oluwasetemi/altschool-opensource-names/pull/152 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/Oluwasetemi/altschool-opensource-names/pull/152#event-7149450497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXCNWCSFZANXHMQB5LJH6G3VYFIILANCNFSM5552NCUA
.
You are receiving this because you authored the thread.Message ID:
<Oluwasetemi/altschool-opensource-names/pull/152/issue_event/7149450497@
github.com>


.travis.yml doesn't run flake8

Having travis (or appveyor) be able to lint my code and any pull requests as part of a contribution is important, and the current templates do not accounts for a flake8 test env.

Hi @miketheman! 👋
Which .travis.yml file are you referring to? The template's or the one of the generated plugin?

👋
The generated one - specifically https://github.com/pytest-dev/cookiecutter-pytest-plugin/blob/7d7bdcda3f73e299e00d79085854e855b2420569/pytest-{{cookiecutter.plugin_name}}/.travis.yml
It appears to use travis' language to infer python version, so it's unclear where a flake8 testenv might be added.

Hi @miketheman! 👋
I just updated the travis config file. Can you please have a look?
Thank you for suggesting this. 🙇

Thanks @hackebrot !


Update things.

Important things.

:-1: This PR should update the minified file too.
We can't expect users of 2012 javascript to have their own build pipelines. How would that integrate with their FTP workflow?

Agreed. @joho if you need help with this, there's a handy section in the readme that documents the build process: https://github.com/markdalgleish/fathom#how-to-build

Fixed. :shipit: ?

Thanks for your support! Keep on jQuerying! :dancer:


How to download firebase cpp sdk older version

I want to download firebase cpp sdk v4.2.0. I can't find it anywhere. How can I download it?

You should be able to download it at: https://dl.google.com/firebase/sdk/cpp/firebase_cpp_sdk_4.2.0.zip
Note you can change that last bit to get different versions.

Thanks @a-maurice for providing the firebase_cpp_sdk 4.2.0 :)


Clean up subscriptions when frames perform navigation.

Instruments desired cleanup brought up by https://github.com/jsonnull/electron-trpc/issues/164

Changes look good! We might want to add a ticket to follow up with some E2E tests for this. I think it would be pretty straight forward to test this. But overall the changes look good.


Fixes the lti captions upload together with the default ingest workflow #2167

The LTI upload used the incorrect flavor for the captions, resulting in paella player not able to find the captions. This aims to fix that.

@flyapen, does it make sense to bring this into 9.x?

@flyapen, does it make sense to bring this into 9.x?

@lkiesow we are running this in 9.x without any issues. Without this the captions upload is unusable so it would make sense to do this!

@lkiesow we are running this in 9.x without any issues. Without this the captions upload is unusable so it would make sense to do this!

Can you then update the pull request to target r/9.x?
I could also do that if you prefer, but you would then need to allow that by checking the “Allow edits and access to secrets by maintainers” at the bottom of the side-bar on the right (you unchecked that when creating the pull request).

Can you then update the pull request to target r/9.x?
I could also do that if you prefer, but you would then need to allow that by checking the “Allow edits and access to secrets by maintainers” at the bottom of the side-bar on the right (you unchecked that when creating the pull request).

@lkiesow updated to 9.x

@lkiesow updated to 9.x


Add output for selected terraform version

Hi, I've just started using this action in my workflow and it works very well. I've had to pass the terraform version installed via tfswitch to $GITHUB_OUTPUT for use with setup-terraform, like so:
      - name: Setup tfswitch
        uses: stv-io/action-tfswitch@b6c5e07f48a6de1934acf5e9c67bd3b21423e9e1 # v1.0.0

      - name: Get Terraform Version
        id: terraform-version
        run: |
          tfswitch --chdir=terraform
          echo "VERSION=$(terraform --version | awk NR==1'{print substr($2, 2)}')" >> "$GITHUB_OUTPUT"

      - name: Setup Terraform
        uses: hashicorp/setup-terraform@633666f66e0061ca3b725c73b2ec20cd13a8fdd1 # v2.0.3
        with:
          terraform_version: ${{ steps.terraform-version.outputs.VERSION }}
          cli_config_credentials_token: ${{ secrets.TFC_API_TOKEN }}

If I omit the echo to $GITHUB_OUTPUT and the terraform_version argument, then Setup Terraform proceeds to install the latest version:

I wondered if this was the expected behaviour, or if action-tfswitch could provide the appropriate version output for setup_terraform?

Hi @chris3ware thanks for your feedback and input!
I wouldn't say this is expected behaviour, and you raise a valid point.
If I understood correctly, you would like to export an output natively by this action, which can be used by subsequent steps, instead of doing it yourself?

Can you test whether the version in the PR solves this issue - https://github.com/stv-io/action-tfswitch/pull/4 and if the docs are relevant?

Hi, thanks for looking into this. I'm not sure if this issue is valid because, if I understand correctly, and I maybe didn't when I raised this issue; the action just installs tfswitch and it must be run in a separate step to obtain the version from a *.tf file. At least for my use case. So the necessary output is going to come from the 'run' step and not the 'setup' step - which I how I have it configured now.
I'm happy to close this from my point of view.

Thanks for your follow up @chris3ware - I would say that either step can output the version, so feel free to choose which step you take the output from, but from the latest PR, you should be able to get it from this action.
I will be closing this issue with your permission then.


[WebToolsE2E][Aspire] Build AspireWithNode solution failed with "The call is ambiguous between the following methods or properties: ..." after updating to Aspire 8.0 preview 2

REGRESSION INFO: Worked fine with Aspire 8.0.0-preview.1.23557.2
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.9 Preview 2
Install Aspire Preview 2 latest build 8.0.0-preview.2.23610.1
Apply NuGet Feeds
Install node from nodejs.org

REPRO STEPS

Clone aspire-samples repo
Checkout preview.2 branch
Update NuGet.config to use the dotnet-tools feed
Update Directory.Packages.props to use aspire version 8.0.0-preview.2.23610.1
Open solution AspireWithNode.sln
Build solution

ACTUAL
Failed to build solution with error: The call is ambiguous between the following methods or properties: 'Aspire.Hosting.NodeAppHostingExtension.AddNodeApp(Aspire.Hosting.IDistributedApplicationBuilder, string, string, string, string[]?)' and 'Aspire.Hosting.NodeAppHostingExtension.AddNodeApp(Aspire.Hosting.IDistributedApplicationBuilder, string, string, string?, string[]?)'	AspireWithNode.AppHost

EXPECTED
Build solution successfully.

@v-cazhang are you sure that you switched to the preview.2 branch of the aspire-samples repo? I cannot reproduce this with preview.2 branch. You'll be able to reproduce this using main branch and if you want to fix it using current bits from main branch, you need to delete a couple of files as shown in this commit.

@balachir, yes, I did. Below are the changes I made:


Machine Learning - Import Models - button to Make Predictions doesn't work

Azure Data Studio Version:
Version: 1.24.0 (user setup)
Commit: eb3c0f5d9df86b9490606aab7fffaa253b0aca4d
Date: 2020-11-10T23:38:09.705Z
VS Code: 1.48.0
Electron: 9.3.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Steps to Reproduce:

Launch ML Extension from a SQL Server 2019 instance.
Click on View or Import MOdels button.
Import a model from a file, into an existing / new model table.
After successfully imported, you should see Imported Model table, which will itemize the list of the models in that table. Each row will have an icon that looks like a line chart for prediction. If you click on this icon, it does nothing.

Expected: Since ONNX runtime is not supported in SQL Server 2019, launching Make Prediction icon after importing a model from the View or Import Model step, should provide a graceful failure that ONNX runtime is not supported..

@llali & @udeeshagautam - can you please triage?

This should be fixed by https://github.com/microsoft/azuredatastudio/pull/13540


Any hope to sse the site up again one day?

Is there any hope to see the Antlr Lab website up again? I've tried running as localhost.. but it's quirky, at best.
I get weird errors like:
1:1 mismatched input 'eval' expecting {'assert', 'eval'}

Probably an encoding issue? I don't know.

sorry... It was up, but somebody gave it so much input it starved everybody else. It's back up after I killed it.

I get weird errors ...
Make sure you go through lexer and parser grammar panels and set them up correctly.


fix(build): moving images to bionic 18.04

Signed-off-by: mayank<EMAIL_ADDRESS>

trivy openebs/cstor-istgt:ci
2020-03-12T12:01:30.746+0530	INFO	Detecting Ubuntu vulnerabilities...

openebs/cstor-istgt:ci (ubuntu 18.04)
=====================================
Total: 124 (UNKNOWN: 0, LOW: 22, MEDIUM: 79, HIGH: 23, CRITICAL: 0)

@mynktl can you push an image from this branch to your docker hub (from travis). You need to setup DNAME and DPASS for your forked repo in travis and test that image, if it is working fine.

Closing this PR since https://github.com/openebs/istgt/pull/317, which fix the same issue, has been merged.


Missing user namespace label user.kubeflow.org/enabled

Namespaces corresponding to Kubeflow users don't get the user.kubeflow.org/enabled label required by default by the Resource Dispatcher to detect the namespaces to inject resources into.
More specifically, the Resource Dispatcher watches for newly created namespaces. If these carry a specific label (configured by the user), it injects the resources required for its related apps to work into the new namespaces. If the Resource Dispatcher is related with MLFlow, for instance, it will have to create PodDefaults and Secrets in the target namespaces to allow workloads running in them to access external services (e.g. MinIO). This label is essentially used to distinguish namespaces belonging to Kubeflow users and is set by default to user.kubeflow.org/enabled: "true".
The Profiles Operator, on the other hand, is the one that controls the labels that will end up on a new namespace, whenever a Kubeflow user Profile is created, through a YAML file pushed to the controller container. The labels currently propagated to the namespace are configured here: https://github.com/canonical/kubeflow-profiles-operator/blob/e46577ac6d7c7d97c5ddc3e6443a1749f006d305/src/templates/namespace-labels.yaml#L15-L19
It looks like the Profiles Operator should also set the user.kubeflow.org/enabled: "true" here, for our services to properly work.

For context, the resource-dispatcher can watch for any label, so we could use an existing one (maybe pipelines.kubeflow.org/enabled).  But adding a new label feels cleaner as its purpose is more clear

re-opening this because it was not fixed in track/1.7


guess_by_extension throws exception on files without extension

When feeding it with files without an extension, an exception is thrown:
>>> e.guess_language_all_methods(code, 'myfile')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\ME\AppData\Local\Programs\Python\Python37-32\lib\site-packages\whats_that_code\election.py", line 35, in guess_language_all_methods
    vote_by_extension = guess_by_extension(file_name=file_name)
  File "C:\Users\ME\AppData\Local\Programs\Python\Python37-32\lib\site-packages\whats_that_code\extension_based.py", line 19, in guess_by_extension
    extensions.append("." + file_name.split(".")[1])
IndexError: list index out of range

Better seems to use os.path.splitext(file)[1] to retrieve the extension.

I pushed my untested fix to https://github.com/matthewdeanmartin/whats_that_code/blob/main/whats_that_code/extension_based.py
I'll test it & push out a release to pypi this weekend.
Thanks for trying out my library!

Works fine now. Thanks for the quick fix.


Issue with new ModalService

There are two issues with the new ModalService
First issue
On first start it only works once, then you'll need to refresh the page in order for it to work fully.
To reproduce:

Clone repository
Set BlazorBootstrap.Demo.Server as startup project
Click on Modals
Click on Service => ModalService
Launch modal
Try to open a second modal
Refresh the page and launch modal
try to launch another modal

Second issue
The modals are hanging behind on your preview site. Take a look on this GIF. I wasn't able to reproduce this on my local machine on both Server Blazor and WebAssembly.

@MarvinKlein1508 Can you share more details on the second issue. I didn't get you the scenario.

@gvreddy04 as you can see in the GIF the styling gets applied to the modal after it has been rendered. It gets more clear when you use different positioning. However this behaviour only applies to your hosted demo site. When I run both Blazor Server and WebAssembly apps on my local machine I didn't have this issue.

In the GIF you can see for example the header changes color after the animation.


fix should not ignore returned Status

Proposed changes
Issue Number: close #xxx

run buildall

run buildall

run buildall

run buildall


SQFormMultiSelect tooltip values & field values sorted differently

This was found by one of the QA testers on one of our projects.
When selecting multiple values in a SQFormMultiSelect drop down, the tooltip that displays when you hover will list your selections in the order they were clicked in, while the field itself will list selected options in order as they appear in the list.
In the above screenshot (taken in the storybook), I selected "Joe, Jessica, Jane, Jack" in that order, and that reflects in the tooltip but not the field itself.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DAWN-508 ⁃ Large txns are not always propagated

I have two nodes -- a producer and a non-producer. I ask the non-producer to make a "large" (in this case, ~20KB) transaction. The non-producer node notifies the producer node of this transaction. The producer node then requests the transaction from the non-producer node. But, what ends up happening is that this request gets all the way to connection::txn_send() on the non-producer and is indeed added to txn_queue. But then txn_queue is never serviced because, I believe, no other txns are being sent by non-producer.
Seen on eos-noon Jan17 2018.

➤ Phil Mesnier commented:
I am taking this issue now. I believe the recently merged patch for issue 567 addresses this issue, but a test is necessary to verify.

Overtaken By Events


Removes unused dependencies

Removes unused dependencies reported by depcheck and double checked to find any usages

This is an automated message from CI:
Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1566
Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.


[BUG] Removing traits from MIO does not remove them from the preview.

Describe the bug
Removing traits from MIO does not remove them from the preview.
Expected behavior
Trait should be removed from preview.
Screenshots

Extension version: v0.12.0

Fixed in v0.12.1


[BUG] Insomnia crashes at Startup

Insomnia Version: latest
Operating System: macOS High Sierra 10.13.6

Details

Insomnia was working perfectly yesterday. Today I turned on my machine and it won't load. I've screenshot the error below. I've uninstalled and re-installed but it didn't fix the problem.

Here the same error:

I just pulled 7.0.2 so no one else gets the update. Working on a fix now. I think this was related to the fact that the last build used GitHub Actions instead of TravisCI which runs in a newer macOS environment.

I just downloaded the app again from the website and now it works fine.

Can you all please verify that 7.0.3 works as well and note your OS version?
Thanks! ❤

working now - thanks for the quick fix!

@gschier Everything is working fine. <3

Awesome! Now I can GET some REST
.
.
.
🚪🚶

Thank you for the fast fix guys! Awesome

Not working for me.
7.0.2 - 7.0.6 via github releases and website https://updates.insomnia.rest/downloads/mac/{{version}} do not work for me.
7.0.1 works via the website and github
Where can I get the Travis CI versions of the latest builds?

@JoshMoreno can you please create a new issue with your specific details?
you get to the
releases on the changelog https://insomnia.rest/changelog/7.0.6


Settings cross over app instances

Type: Bug
I often run more than one instance of code. I have dual monitors at different resolutions when I set the zoom level on one it changes the zoom on another.
In addition I open an instance with a project I am using as reference and I disable thte autosave feature so I notice any accidental changes.  It also disables the autosave on the other instance.
Ubuntu 20.04 all updates
VS Code current (Not Portable Mode)
Single default install
Omnisharp c# current
VS Code version: Code 1.74.3 (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T16:57:40.428Z)
OS version: Linux x64 5.15.0-60-generic
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5 CPU       M 460  @ 2.53GHz (4 x 2233)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
3.64GB (0.83GB free)

Process Argv
--unity-launch --crash-reporter-id 23f14158-8768-43a4-8fe0-205a2237613a

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (2)

Extension
Author (truncated)
Version

vscode-sqlite
ale
0.14.1

csharp
ms-
1.25.4

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
vscccc:30610679
pyindex848:30577860
nodejswelcome1:30587005
2e4cg342:30602488
f6dab269:30613381

You should use Workspace level settings instead of User level settings then.
Should work with autosave.
For zoom level you should search for existing issue tracking handling zoom levels on different monitors.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


RDKB-50186 : [On rbusDirect] Implementation of RBUS Publish API to send Raw Data

Reason for change: Changes made with rawdata API to support both direct mode and normal connection. If normal connection exist, the data will be send through it and if direct connection exist, the data will be send through it.
Test Procedure: Test and verified with rbuscli
Risks: Medium
Priority: P1
Signed-off-by: Gururaaja ESR<EMAIL_ADDRESS>

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: not a problem

Commit: ee3e9ba0cc6b430f5a79d6586eb4703a5abf8252


Reorganize UCX & gRPC apps

Reorg distributed applications into the following:
└── applications
    └── distributed
        ├── grpc
        │   └── grpc_h264_endoscopy_tool_tracking
        └── ucx
            ├── ucx_endoscopy_tool_tracking
            └── ucx_h264_endoscopy_tool_tracking

Looks good to me. Is this ready for merge?

Yes, please.


device: Add SCSI block device support

Recently, the support for block devices (rootfs or device passthrough)
plugged through the VM with virtio-blk have been added.
This commit follows this by allowing block devices plugged through the
VM using virtio-scsi to be supported and properly processed from the
agent.
One thing to notice here is that latest versions of Qemu (>= 2.9) have
some stability issues using virtio-blk, and the use of virtio-scsi is
a good way to work around this current limitation.
Fixes #162
Signed-off-by: Sebastien Boeuf<EMAIL_ADDRESS>

@bergwolf @laijs @amshinde @sameo Please let me know what you think about this. I have it working with a corresponding patch on virtcontainers. I'll fix/add some unit tests to this PR in the meantime.

@laijs PR updated with what we've discussed, PTAL.

@laijs @bergwolf I'll add/fix unit tests if you're okay with those changes.

@laijs @bergwolf PR updated. I have removed Rootfs field, and now MountPoint is the path inside the VM, not the container.

cc @amshinde

@bergwolf @laijs @sameo @amshinde PR is ready, please take a look.

@sboeuf Sure, Will take a look today.

@amshinde PR updated.

@amshinde I have updated the PR, adding more verbose comments about rootfs, storages and devices.

LGTM

@jodh-intel I have updated the function names, let me know if you think this is better now.

lgtm

@jodh-intel could you go ahead and merge it please (since everybody approved the PR) ?

@sboeuf - we're seeing CI failures that might be related:
Stderr: docker: Error response from daemon: OCI runtime create failed: rpc error: code = InvalidArgument desc = Unknown storage driver "": unknown.

Stderr: losetup: /tmp/dd591751879: failed to set up loop device: Device or resource busy

@chavafg - or is this related to the container manager utility dyt?

Problem is, we need virtcontainers to be updated. And virtcontainers needs this PR to land so that we can revendor the agent. We need to add to our CI a way to test a PR with a Depends-on flag.

It sounds like we're in a "chicken and egg" situation here and will have to either force-merge this PR (or https://github.com/containers/virtcontainers/pull/666 according to @sboeuf).
@kata-containers/agent - Can another team member ack the proposal to force-merge this PR please?

@jodh-intel @sboeuf I am ok with force merging this PR. We will be broken till the SCSI PR lands for virtcontainers.

... Wait!
Kidding.
Thanks @amshinde :smile:

Haha ..too late now @jodh-intel :)


Removed the empty class 'com.cloud.deploy.PlannerBase'

Removed the PlannerBase class because it is does not bring contribution to ACS' code.
We changed com.cloud.deploy.FirstFitPlanner, now it doesn’t extends the PlannerBase and implements the com.cloud.deploy.DeploymentPlanner.
We also removed the method com.cloud.deploy.DeploymentPlanner.check(VirtualMachineProfile, DeploymentPlan, DeployDestination, ExcludeList) that was not used anywhere.
Additionally, we removed the “_” from FirstFitPlanner's attributes name, in order to have them in a more standard way.

@pedro-martins that is good that you removed and unused method in com.cloud.deploy.DeploymentPlanner, however you forgot to remove all of the implementations of classes that implement it.
BareMetalPlanner is one of the classes that you forgot to fix, there are others

Reopening PR that was closed by mistake


need feed metadata for Intel XDK integration

The title and description are hardcoded metadata in the Intel XDK feed.
If README.md is updated with a description at the top of the file, that could be manually added into the feed.
An alternative would be to make a separate file that lists the feed metadata (title and description) explicitly.
However the metadata is provided, integrating it into the feed is a manual step - it is not directly ingested from any of the files in the repo.

README now has the first paragraph with a description, is there another edit necessary or other change needed now?  Thanks!

@bwebsterdv - I guess if I combine the first paragraph with the sentence from requirements it would work?  We need to make it clear that an Appmobi account is a prereq to trying to sample.

Here is what I am using, based on discussion with @bwebsterdv (partially cribbed from README, note that I had to insert some formatting to accomodate the placement - we can tweak it as needed):
name: Appmobi Secure Mobile Platform
description:

Appmobi* Secure Mobile Platform by Appmobi

Appmobi Secure Mobile Platform provides a level of security never seen in a mobile platform, enabling the
          development of highly secure Cordova-based applications in minutes versus months. This platform provides
          your development team with configurable, high-performance security options for every app built, including
          app and device level shared keys for encryption and identity verification as well as integration with
          most 3rd party authentication systems such as OAUTH, LDAP, and more.
          Quickly and easily build the most secure apps on the market today, in minutes.

          In order to utilize the Appmobi platform you must have a registered Appmobi PrivateStack platform.
          If you do not have an account or Appmobi PrivateStack server you can Register for a Demo on our website.

Please refer to the plugin documentation link for API details, supported platforms and required variables.