Vagrant Ansible provision fails, ERROR! 'import_playbook' is not a valid attribute for a Play

Hello, vccw-team.
I have a vagrant provisioning problem, please help.

Operating System: Mac OS X 10.13.3
Vagrant Version: 2.0.2
Virtual Box Version: 5.2.6 r120293

I bumped VCCW version to v3.16.1 from v3.3.2.
Like this.
% vagrant box add vccw-team/xenial64
    :
==> box: Successfully added box 'vccw-team/xenial64' (v20180107) for 'virtualbox'!
% wp scaffold vccw [my_project] --update

Add or Modify 9 files.

Vagrantfile
provision/default.yml
provision/playbook.yml
provision/playbooks/commands.yml
provision/playbooks/middleware.yml
provision/playbooks/wordpress.yml
provision/playbooks/templates/browse-command.php
provision/playbooks/templates/vccw.sh
provision/playbooks/templates/wp-cli.yml

And vagrant provision, but it is failed.
% vagrant provision==> [my_project]: [vagrant-hostsupdater] Checking for host entries
==> [my_project]: [vagrant-hostsupdater]   found entry for: <IP_ADDRESS> [my_project]
==> [my_project]: Running provisioner: ansible_local...
Vagrant has automatically selected the compatibility mode '2.0'
according to the Ansible version installed (<IP_ADDRESS>).

Alternatively, the compatibility mode can be specified in your Vagrantfile:
https://www.vagrantup.com/docs/provisioning/ansible_common.html#compatibility_mode

    [my_project]: Running ansible-playbook...
ERROR! 'import_playbook' is not a valid attribute for a Play

The error appears to have been in '/vagrant/provision/playbook.yml': line 2, column 3, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

---
- import_playbook: playbooks/commands.yml
  ^ here

Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

Thanks.

I found the issue "Privacy Error #314".
I changed the hostname to *.test from *.dev in site.yml
 #
 # Network Settings
 #
-hostname: [my_project].dev
+hostname: [my_project].test

and vagrant up. (= build a new VM!)
I can't understand Ansible provisioning, but It looks fine to me.
Thanks.


[Question]: How can the user keep his calendar file up to date?

Is there a way to update already downloaded calendar files if f.e. new participants are added to the event?
(Maybe in the iTIP Specification we find a solution)
Or is a redownload necessary for the user to get an updated event?

Possible solutions:

Subscribe on an ics resource route
Downsides:

ics creation on download is cost intensive (creation on update would be necessary)
user has to subscribe by himself using the client account settings
user has to subscribe for every single event
user has to update his subscription depending on the client

Send mail on update event with attached ics or something else


Login page accessible even after login

Current behaviour:

The user can go back to the /login route by the back button or by entering the url.

Expected behaviour:

The user should be redirected to the main page when attempting to go to the login page.

@Yureien I have solved this issue and am putting up a pr. Please do respond As soon as possible for me to work on other projects or I may look for other projects under KWOC if this project is not active due to some reasons.
Thank you.


add ReviewCard test and story

Adds test and story for ReviewCard component. Part of #409 and #433.

Running yarn test --no-cache --runInBand on my machine resulted in the same error from the CI logs.

Whoever merges this PR please squash the commits so we don't have a bunch of fix build test in the log.


Improve the directory structure of docs

Motivation
I'm developing the script to sync the docs from incubator-eventmesh/docs to incubator-eventmesh-site/docs. This pull request improves the directory structure of incubator-eventmesh/docs to make the sync process easier.
Modifications

Move docs/README.md todocs/[en or zh]/intro.md
Remove docs/.vuepress/ and docs/_config.yml
Add _category_.json to allow eventmesh-site to get the name of each directory

Documentation

Does this pull request introduce a new feature? No
If yes, how is the feature documented? Not applicable
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Codecov Report

Merging #516 (2859f3a) into develop (6c222cc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #516   +/-   ##
==========================================
  Coverage      10.42%   10.42%
  Complexity       328      328
==========================================
  Files            241      241
  Lines          11737    11737
  Branches        1001     1001
==========================================
  Hits            1224     1224
  Misses         10411    10411
  Partials         102      102

Continue to review full report at Codecov.

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


Daisydev

@sunny-bak Hi Sunny, we wanted to create a folder for the ATN tutorials easier for people to download. Can you review pr and merge? Thank you.

Hi Daisy,
I am on leave. Please feel free to merge if you think it makes sense!
Thanks,
Sunny
On Thu, Oct 24, 2024, 5:39 AM DaisyShi19 @.***> wrote:

@sunny-bak https://github.com/sunny-bak Hi Sunny, we wanted to create a
folder for the ATN tutorials easier for people to download. Can you review
pr and merge? Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433403370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6YAWI3IKBEGR7ER3ITADZ5ACQ5AVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGQYDGMZXGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi Sunny, hope you're enjoying the time off :)
Can you give me access/authority to merge? Currently it says:
Review required
At least 1 approving review is required by reviewers with write access.
Merging is blocked
Merging can be performed automatically with 1 approving review

The merge is blocked but as an administrator you should be able to continue
to merge.
I'm on the merge page now and want to check if you can still merge it
although it says it.
Could you try? If it doesn't work I will do it.
On Thu, Oct 24, 2024, 6:29 AM DaisyShi19 @.***> wrote:

Hi Sunny, hope you're enjoying the time off :)
Can you give me access/authority to merge? Currently it says:
Review required
At least 1 approving review is required by reviewers with write access.
Merging is blocked
Merging can be performed automatically with 1 approving review
—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433497759,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6YAQJYTB4XQ3HOQQWIT3Z5AIKHAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGQ4TONZVHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks Sunny, and sorry I was away for a bit. I was able to merge early PRs but not this one. I was wondering if it has deleted files. Anyways, it should be good from now.

Hi Daisy,
I merged it for you and also made you the admin of the repo.
I accidentally change the ownership of the total account to you. Can you go
and give me back the ownership? I can't access other repositores
On Thu, Oct 24, 2024, 7:13 AM DaisyShi19 @.***> wrote:

Thanks Sunny, and sorry I was away for a bit. I was able to merge early
PRs but not this one. I was wondering if it has deleted files. Anyways, it
should be good from now.
—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433565423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6YAWHJB4B2C7YRABHECLZ5ANSNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DKNBSGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Go to the account coastwatch-training and go to people and click shospital
and made me the owner.
Mahalo
On Thu, Oct 24, 2024, 7:15 AM Sunny Bak @.***> wrote:

Hi Daisy,
I merged it for you and also made you the admin of the repo.
I accidentally change the ownership of the total account to you. Can you
go and give me back the ownership? I can't access other repositores
On Thu, Oct 24, 2024, 7:13 AM DaisyShi19 @.***> wrote:

Thanks Sunny, and sorry I was away for a bit. I was able to merge early
PRs but not this one. I was wondering if it has deleted files. Anyways, it
should be good from now.
—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433565423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6YAWHJB4B2C7YRABHECLZ5ANSNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DKNBSGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Now you are the administrator of the coastwach tutorial repo and you can
bypass the review process to merge.
On Thu, Oct 24, 2024, 7:16 AM Sunny Bak @.***> wrote:

Go to the account coastwatch-training and go to people and click shospital
and made me the owner.
Mahalo
On Thu, Oct 24, 2024, 7:15 AM Sunny Bak @.***> wrote:

Hi Daisy,
I merged it for you and also made you the admin of the repo.
I accidentally change the ownership of the total account to you. Can you
go and give me back the ownership? I can't access other repositores
On Thu, Oct 24, 2024, 7:13 AM DaisyShi19 @.***>
wrote:

Thanks Sunny, and sorry I was away for a bit. I was able to merge early
PRs but not this one. I was wondering if it has deleted files. Anyways, it
should be good from now.
—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433565423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6YAWHJB4B2C7YRABHECLZ5ANSNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DKNBSGM
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Changed. Can you check if it it correctly done?
On Wed, Oct 23, 2024 at 12:16 PM Sunny Bak Hospital <
@.***> wrote:

Go to the account coastwatch-training and go to people and click shospital
and made me the owner.
Mahalo
On Thu, Oct 24, 2024, 7:15 AM Sunny Bak @.***> wrote:

Hi Daisy,
I merged it for you and also made you the admin of the repo.
I accidentally change the ownership of the total account to you. Can you
go and give me back the ownership? I can't access other repositores
On Thu, Oct 24, 2024, 7:13 AM DaisyShi19 @.***> wrote:

Thanks Sunny, and sorry I was away for a bit. I was able to merge early
PRs but not this one. I was wondering if it has deleted files. Anyways,
it
should be good from now.
—
Reply to this email directly, view it on GitHub
<
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433565423>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAE6YAWHJB4B2C7YRABHECLZ5ANSNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DKNBSGM>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433569111,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANONVQ4V3Q5RVRVJM22SYZ3Z5AN4VAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DSMJRGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you Sunny. I'll only need it till the end of Nov when trainings are
done. And thank you again for helping out while on leave!
On Wed, Oct 23, 2024 at 12:18 PM Sunny Bak Hospital <
@.***> wrote:

Now you are the administrator of the coastwach tutorial repo and you can
bypass the review process to merge.
On Thu, Oct 24, 2024, 7:16 AM Sunny Bak @.***> wrote:

Go to the account coastwatch-training and go to people and click
shospital
and made me the owner.
Mahalo
On Thu, Oct 24, 2024, 7:15 AM Sunny Bak @.***> wrote:

Hi Daisy,
I merged it for you and also made you the admin of the repo.
I accidentally change the ownership of the total account to you. Can
you
go and give me back the ownership? I can't access other repositores
On Thu, Oct 24, 2024, 7:13 AM DaisyShi19 @.***>
wrote:

Thanks Sunny, and sorry I was away for a bit. I was able to merge
early
PRs but not this one. I was wondering if it has deleted files.
Anyways, it
should be good from now.
—
Reply to this email directly, view it on GitHub
<
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433565423>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAE6YAWHJB4B2C7YRABHECLZ5ANSNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DKNBSGM>

.
You are receiving this because you were mentioned.Message ID:
@.***
com>

—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433570845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANONVQZ3FZUAYTI42XR6V73Z5AOBNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3TAOBUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes. Sorry I was still an owner. We can have one or more owners. Now you
and i are owners.
On Thu, Oct 24, 2024, 7:21 AM DaisyShi19 @.***> wrote:

Changed. Can you check if it it correctly done?
On Wed, Oct 23, 2024 at 12:16 PM Sunny Bak Hospital <
@.***> wrote:

Go to the account coastwatch-training and go to people and click
shospital
and made me the owner.
Mahalo
On Thu, Oct 24, 2024, 7:15 AM Sunny Bak @.***> wrote:

Hi Daisy,
I merged it for you and also made you the admin of the repo.
I accidentally change the ownership of the total account to you. Can
you
go and give me back the ownership? I can't access other repositores
On Thu, Oct 24, 2024, 7:13 AM DaisyShi19 @.***> wrote:

Thanks Sunny, and sorry I was away for a bit. I was able to merge
early
PRs but not this one. I was wondering if it has deleted files.
Anyways,
it
should be good from now.
—
Reply to this email directly, view it on GitHub
<

https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433565423>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAE6YAWHJB4B2C7YRABHECLZ5ANSNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DKNBSGM>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433569111>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ANONVQ4V3Q5RVRVJM22SYZ3Z5AN4VAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DSMJRGE>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433576042,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6YASSN4VFW2XFOFGN5YLZ5AONPAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3TMMBUGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

OK sounds good!
On Wed, Oct 23, 2024 at 12:24 PM Sunny Bak Hospital <
@.***> wrote:

Yes. Sorry I was still an owner. We can have one or more owners. Now you
and i are owners.
On Thu, Oct 24, 2024, 7:21 AM DaisyShi19 @.***> wrote:

Changed. Can you check if it it correctly done?
On Wed, Oct 23, 2024 at 12:16 PM Sunny Bak Hospital <
@.***> wrote:

Go to the account coastwatch-training and go to people and click
shospital
and made me the owner.
Mahalo
On Thu, Oct 24, 2024, 7:15 AM Sunny Bak @.***> wrote:

Hi Daisy,
I merged it for you and also made you the admin of the repo.
I accidentally change the ownership of the total account to you. Can
you
go and give me back the ownership? I can't access other repositores
On Thu, Oct 24, 2024, 7:13 AM DaisyShi19 @.***> wrote:

Thanks Sunny, and sorry I was away for a bit. I was able to merge
early
PRs but not this one. I was wondering if it has deleted files.
Anyways,
it
should be good from now.
—
Reply to this email directly, view it on GitHub
<

https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433565423>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAE6YAWHJB4B2C7YRABHECLZ5ANSNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DKNBSGM>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<

https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433569111>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ANONVQ4V3Q5RVRVJM22SYZ3Z5AN4VAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3DSMJRGE>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433576042>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAE6YASSN4VFW2XFOFGN5YLZ5AONPAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU3TMMBUGI>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/coastwatch-training/CoastWatch-Tutorials/pull/72#issuecomment-2433580452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANONVQ2JLO3TY7XRR4BOBE3Z5AOYNAVCNFSM6AAAAABQPUJDP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZTGU4DANBVGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix: Don't clear multiselect on hover if the item is part of the selection

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #11867

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Hovering over multiple selected items won't clear selection
Hovering over not selected items will clear the multi selection & select the new item

Screenshots (optional)
https://user-images.githubusercontent.com/56681014/228270965-5a829742-9e86-456b-a124-9fecd457829c.mp4
Notes
The first commit inverts two conditions and uses return to save some indentation
The second commit is the actual fix

I didn't realize that was handled here, we can check for the hot keys as well e.KeyModifiers == VirtualKeyModifiers.Shift. The basic idea is to return as early as possible. This will make it easier to understand the logic in the future.

Looks good, thanks!


Header (and all fixed elements) only stays fixed on mobile

I'm submitting a ...

[X] bug report
[ ] feature request

Bug Report:
I know this is probably part of the Pusher issues.
Currently, fixed elements on desktop will not stay fixed and will scroll.  Once mobile (menu is gone), all issues are resolved.

Hi @TheDolo, hm, I didn't consider it like an issue, but you're right this needs some research for users that are interested in fixed Header. I think it's an "enhancement", not a "bug". Will try to figure out what's the problem.

Currently, fixed elements on desktop will not stay fixed and will scroll.

That's definitely an issue if the only fixed element allowed on the page is the sidebar!

Oh, I just misunderstood your sentence, ok, I'll investigate this case with fixed elements on the page (desktop), haven't seen this problem before.

This is the culprit in semantic-ui-css module node_modules/semantic-ui-css/semantic.css starting around line:34342
/* Left Visible */

.ui.visible.thin.left.sidebar ~ .fixed,
.ui.visible.thin.left.sidebar ~ .pusher {
  -webkit-transform: translate3d(150px, 0, 0);
  transform: translate3d(150px, 0, 0);
}

pusher and fixed classes don't like the translate3d because of the co-ordinate shift it causes.
I'll look into a fix.

@TheDolo you can use <Rail> and <Portal> components if you need to render something outside the main container. Probably, it can be used as a temporary fix until the better solution. Also, maybe in your case Portal/Rail is the best solution.

Sorry, people.  Not quite had the time yet.

Page structure was improved + react-headroom was added. Now anything can be fixed/statis/relative/etc. Anything supported.

@Metnew it still doesn't work. Header (react-headroom) scrolls on desktop.

Hm, ok, I'll take a look asap

Not sure, but the possible root of the problem:

missing overflow-x in <PageLayout> component
transform:0(in SUI's .pushable) breaks something.

I had a project where this feature works as expected. I'll take a look more detailed later


Update 126-toothpicks.md

This contribution implements a infinite zoom out to render more and more toothpicks. It eventually takes a while to compute ^^.

https://editor.p5js.org/tht-jxny/full/HJXcWqoW4 is the link

https://editor.p5js.org/tht-jxny/full/HJXcWqoW4 is the link


No error thrown on circular ref

I have a case where dbt is not raising any kind of error on a circular ref. The circular ref includes ephemeral models, so that could be the culprit.

@cmcarthur we just fixed something like this in development (soon to be 091). Let's test against that and see if it still doesn't raise


configuration: document candidates for live cluster configuration update

Not all fields in the configuration.yaml file can be updated without restarting the whole cluster, documentation of each option is needed in order to know which ones are candidates for live configuration updates using the CONFIGURE SSNTP command.
(e.g: Networks should not be updated live (mgmt, compute) because these would require reset of the tunnel. )
Current proposal is as follows:
`/etc/ciao/configuration.yaml`
---
configure:
  scheduler:
    storage_uri:       YES / write new config yaml to new location, write configuration{TIMESTAMP}.yaml backup previously to rw
  storage:
    ceph_id:           ??? (unknown on-change behaviour)
  controller:
    compute_port:      YES / test an API call(TBD) with new controller port
    compute_ca:        NO  / certificates should change only on controller reboot
    compute_cert:      NO  / certificates should change only on controller reboot
    identity_user:     NO  / user should change only on controller reboot
    identity_password: NO  / password should change only on controller reboot
  launcher:
    compute_net:       NO  / It would require reset of the tunnel (expensive)
    mgmt_net:          NO  / It would require reset of the tunnel (expensive)
    disk_limit:        YES / change launcher setup of disk_limit
    mem_limit:         YES / change launcher setup of mem_limit
  image_service:
    type:              *UNUSED*
    url:               *UNUSED*
  identity_service:
    type:              YES / test an API call(TBD) with new type
    url:               YES / test an API call(TBD) with new url

Criteria for changes on launcher.mem_limit and launcher.disk_limit is still uncertain and require feedback from other contributors to know if we can change this with no checking of what kind of validation is needed
This is a sub-task of #929 configuration: Add cluster configuration update support

Not working on this anymore, closing...


Added image for building Istio proxy image based on cilium/envoy

Proxy image required by https://github.com/SUSE/avant-garde/issues/1510

How about moving this to SLES 15 SP2 / CaaSP v5 ?

How about moving this to SLES 15 SP2 / CaaSP v5 ?

Moved to SLES 15 SP2 /CaaSP v5


[Modal] Action should be fired only for instance where it was clicked by user

Problem is actual for allowMultiple: true config. Fiddle: https://jsfiddle.net/donnicky/5vpn7wyk/
In the shown form (Person) click Cancel, then click Yes in confirmation dialog and you'll see that onApprove is called for both dialogs, but it is expected to be called for confirmation dialog only. Generally it the same with any action - it is fired for all active dialogs.

BTW, now there is a workaround using selector modal config if set specific selector per dialog.

Hi, your code should look like this JSFiddle
https://jsfiddle.net/gk7j7a72/
There is a related issue though that you probably ran into where with multiple modals the top modal has to be one of the following:

Be initialized last
Be included last inside the modals dimmer
Manually have a higher z-index set in css

I've made a fix which will automatically make additional modals appear on top in next release.


x/review: pending should not list abandoned CLs

Create a change
Abandon the change
git sync on the local client

git pending lists the abandoned CLs, although there isn't much value in listing them.

Likely to be cc'ed to @adg.

Current behaviour is consistent with the fact that pending subcommand is listing all local branches regardless whether they are mailed, submitted or abandoned.
Rather than hiding abandoned CLs in the listing, we should mention that they are abandoned.
$ git-codereview pending
removewiki f697cf2..ee75c4e (all mailed, 246 behind)
+ ee75c4e https://go-review.googlesource.com/32212 (mailed)
	doc: remove the link to the Learn wiki page

	The wiki page seems to be very outdated and poorly maintained. There
	are so many better resources available at this point. We shouldn't
	mention the page particularly from the golang.org docs or make sure
	it is well maintained.

	Change-Id: Iae4a3b2d7990327ca075a3a9514fa1a4dda86577

	Files in this change:
		doc/docs.html

Listing should say rather say
+ ee75c4e https://go-review.googlesource.com/32212 (abandoned)

rather than the CL is mailed.

This issue has been resolved by https://github.com/golang/review/commit/e973841853e6a8257069881107dd934d04049ae8 and should be closed. There was a typo in the commit message (/cc @rakyll) that prevented it from being automatically closed.

Thanks.


Execute kafka-dump-log only as user kafka

kafka-dump-log.sh may create empty index files. Running it as kafka user guarantees that Kafka will not have problems reading those files.

I think this guarantee should be covered by the environment around shipping the script instead of modifying this script for all.


[docs] improve logo in navbar

i can't find anywhere that
main_menu:
  show_logo: true

needs a logo.png image in assets/media/logo.png

i can't find anywhere that
main_menu:
  show_logo: true

needs a logo.png image in assets/media/logo.png

The logo image is documented in Step 3 of the Getting Started guide: http://localhost/docs/getting-started/get-started/


Fixed xhamster ripper not downloading after 30 images

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a bug fix (Fix #1410 )
[ ] a new Ripper
[ ] a refactoring
[ ] a style change/fix
[ ] a new feature

Description
Fixed xhamster ripper not downloading after 30 images.
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[x] I've verified that this change works as intended.

[x] Downloads all relevant content.
[x] Downloads content from multiple pages (as necessary or appropriate).
[x] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[x] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Hey I'm getting the exact same issue, albums stopping at 30. I'm new to Ripme, was wondering whether there something I had to change in the settings or download something in order to fix this problem? Or is it as simple as the admins fixing it and we wait for the update? Cheers

@damienclark64

Or is it as simple as the admins fixing it and we wait for the update?

It's as simple as waiting for an update to be pushed


Fix source member lines from being truncated

Changes
The record length of the source member was not always being respected and there were circumstances when the default record length of 80 characters was being assumed, losing characters between position 81 and 100.
See issue #1235 for full details.
When saving a source member if the record length has not been established then it will be determined before the save. The logic to establish the record length has been extracted into a private function called getRecordLength within extendedContest.ts, as this is used in two places.
Checklist

[x] have tested my change
[ ] updated relevant documentation
[x] Remove any/all console.logs I added
[ ] eslint is not complaining
[x] have added myself to the contributors' list in CONTRIBUTING.md
[ ] for feature PRs: PR only includes one feature enhancement.

@richardm90 Looks good and my testing looks good as well. Fancy fixing this conflict, and then we can merge!

@worksofliam, I've fixed the conflict.


Register project to Scala Steward

@raboof @eed3si9n @mkurz Any objections?

Likely blocked by https://github.com/sbt/sbt-sbom/pull/70

I'd say go for it! I don't think it needs to wait for #70

PR made at https://github.com/VirtusLab/scala-steward-repos/pull/466

Closing since linked PR is merged


Check for proper usage of translation functions

I had read that variables were discouraged from being used in the i18n functions. I assumed this was just to discourage them from being changed. Well, it's actually because some automated i18n tools will tokenize the PHP files to extract the textdomain from PHP without actually evaluating the file. See @markjaquith's blog post Translating WordPress Plugins and Themes: Don’t Get Clever.
So a sniff should be written which makes sure that translation functions have a text domain which is a literal string. The sniff should also ensure that variables are not being used for the text to be translated, or inside of the text to be translated.

I think this was accidentally closed.

Strings that span multiple lines may get highlighted as false positive e.g.:
<?php esc_html_e( 'Long line
of text', 'foo' ); ?>

Both give "The $text arg must be a single string literal, not 'Long line of text'."

@GaryJones interesting. PHP tokenizes that single string literal into two separate T_CONSTANT_ENCAPSED_STRING tokens with content 'Long line and of text'

It seems we need to merge two tokens of type T_CONSTANT_ENCAPSED_STRING that appear next to each other in $argument_tokens.

I guess that merging would need to be done for context args and so on too, not just the string being internationalized.
<?php _x( 'Post', 'Holds the
fence up', 'text-domain' ); ?>

@GaryJones how's #545?

@westonruter #545 tested on my real and made-up code, and the error no longer appears. :+1:


Fix ACAccount error

Fix issue #16 iOS getting several errors around "ACAccount" and "Bad receiver type int'
@katfang

Fixed issue for me.

Thanks for the hard work but we are no longer accepting changes to the legacy samples like this one.


TypeError: 'NoneType' object is not subscriptable

When running some real hospital notes against the cnlpt negation endpoint, I occasionally get a 500 error. This is the server log from one of those calls.
I don't have the text right now, I can work on getting that, if I can clean it safely.
WARNING:Negation_REST_Processor:Received document of len 781 to process with 0 entities
The following columns in the test set don't have a corresponding argument in `CnlpModelForClassification.forward` and have been ignored: text. If text are not expected by `CnlpModelForClassification.forward`,  you can safely ignore this message.
***** Running Prediction *****
  Num examples = 0
  Batch size = 8
INFO:     <IP_ADDRESS>:56486 - "POST /negation/process HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/uvicorn/protocols/http/httptools_impl.py", line 376, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/usr/local/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in __call__
    return await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/fastapi/applications.py", line 208, in __call__
    await super().__call__(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/applications.py", line 112, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc
  File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/usr/local/lib/python3.10/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc
  File "/usr/local/lib/python3.10/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 656, in __call__
    await route.handle(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 259, in handle
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 61, in app
    response = await func(request)
  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 226, in app
    raw_response = await run_endpoint_function(
  File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 159, in run_endpoint_function
    return await dependant.call(**values)
  File "/usr/local/lib/python3.10/site-packages/cnlpt/api/negation_rest.py", line 65, in process
    predictions = output.predictions[0]
TypeError: 'NoneType' object is not subscriptable

Hi Michael, what version of cnlp-transformers do you have installed?

From pip the other day: 0.4.0

This looks like it is entirely due to passing in 0 entities. I can work around this by just not calling cnlpt in that edge case. But maybe it's also nicer if cnlpt did an early exit as well.

Thanks for the feedback! We will add a catch for that and emit a better error.

The fix for this is in #112 and will be released in v0.5.0. Instead of emitting an error we just return an empty list if zero entities are passed in.


onPress not ergonomic

The onPress event only gives back the title of what was pressed, which leads to code like this,
onPress= (e) => {
  const {name} = e.nativeEvent;
  if (name === actions[0].title) {
    navigation.push(MemoryAdd, {item});
  } else if (name === actions[1].title) {
    onDelete();
  }
}

It would be more useful if we got an index
Or better if you look at react-native-swipe-view, each action has a callback
Is there one way you prefer, and do you want a hand implementing this?

Gotcha. Yeah I had a specific reason for this when I was first writing it, but I can see the issue. The callback would work fine too, but I was trying to minimize the amount of event blocks being sent over the bridge.
If you would like to take a stab at changing this how you think would be the most ergonomic that would be awesome.

Closed with #4


automatic utf-8 conversion for all methods, fixes #119

@narek-cliqz

Coverage increased (+0.6%) to 84.856% when pulling 312fc9b3242a59ca85d16010ebb89c9992860c6b on hendrik-cliqz:autowrap-utf8 into c5c8d23576a7718c4187f1b9710f7f155dc1eb83 on cliqz-oss:master.


Simulate holes in the ground

Hi, I want to simulate a hole in the ground (lets say 0.5mx0.5mx0.5m, can be round can be rectangle, doesn't matter) to see how lidar points hit it. I didn't find anything about how to make holes/pits/sinkholes in the ground/road.
any ideas?

I'm also interested in this. My guess is that this isn't the responsibility of carla or even unreal and should instead be done at the 3d modeling level, ex blender. I don't know of any good tools built on this, but i know there are a variety of tools looking at road surface simulation.
https://en.wikipedia.org/wiki/OpenCRG is one.

Hi @jimmyw404 @alinakir
We have some assets that you can spawn on road. It may help. But we don't have holes yet.


Cannot install in Xamarin Android or Windows Phone project

I'm trying to get this library running in a Xamarin Android/Windows Phone project in Visual Studio 2015, but i get the following error:
PM> Install-Package libsodium-net
Attempting to gather dependencies information for package 'libsodium-net.0.8.0' with respect to project 'SocketTestSap.WinPhone', targeting 'WindowsPhone,Version=v8.0'
Attempting to resolve dependencies for package 'libsodium-net.0.8.0' with DependencyBehavior 'Lowest'
Resolving actions to install package 'libsodium-net.0.8.0'
Resolved actions to install package 'libsodium-net.0.8.0'
Install failed. Rolling back...
Package 'Baseclass.Contrib.Nuget.Output 1.0.0' does not exist in project 'SocketTestSap.WinPhone'
Package 'Baseclass.Contrib.Nuget.Output 1.0.0' does not exist in folder    'C:\Users\me\documents\visual studio 2015\Projects\SocketTestSap\packages'
Install-Package : Could not install package 'Baseclass.Contrib.Nuget.Output 1.0.0'. You are trying to install this package
into a project that targets 'WindowsPhone,Version=v8.0', but the package does not contain any assembly references or content
files that are compatible with that framework. For more information, contact the package author.
....

Is there anything that can be done about that?

Upgrade Baseclass.Contrib.Nuget.Output to 2.1 See #114

Thanks for you answer, I've already tried that but unfortunately I got:
Install-Package : Could not install package 'Baseclass.Contrib.Nuget.Output 2.1.0'. You are
trying to install this package into a project that targets 'WindowsPhone,Version=v8.0', but the
package does not contain any assembly references or content files that are compatible with
that framework. For more information, contact the package author.

Does that mean this Baseclass.Contrib.Nuget.Output is not made for mobile phone targets?

Yes i think so.
The nuget package does not contain a wp8 folder inside the lib folder.
Here is the github project https://github.com/baseclass/Contrib.Nuget maybe they can add wp8.
But you can try to copy the libsodium.dlls manually into your bin folder.

Thanks. I've asked the author of Contrib.Nuget about it. Could you give me a few hints on how to manually copy the libsodium.dlls into the project? Should I create a new project, install the libsodium-net NuGet there and then locate and copy the dlls?

You can:

clone and compile this project
download the nuget package and extract its content with 7-zip
download the current stable release on github

I've downloaded the current stable release and tried adding it like this:

In Visual Studio 2015 Solution Explorer right click on "References" inside my project
Select "Add Reference..."
Select "Browse"
Inside the folder where I've downloaded the stable release check libsodium.dll, libsodium-64.dll and Sodium.dll
Click "OK"

This gives me: "A reference to 'C:\temp\libsodium64-dll' could not be added. Please make sure that the file is accessible, and that it is a valid assembly or COM component."
Am I doing it the wrong way?

Only add the Sodium.dll to your references.
And put the libsodium.dll and libsodium64.dll inside your bin folder (this is what Baseclass.Contrib.Nuget.Output do).

Ok, I did that. Adding Sodium.dll did work and I've also copied libsodium.dll and libsodium64.dll to the bin folder. Now I get the following error on compiling:
Exception while loading assemblies: System.IO.FileNotFoundException: Could not load assembly
'System.Security, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. Perhaps it
doesn't exist in the Mono for Android profile?
File name: 'System.Security.dll'

There's no System.Security available under "Reference Manager", "Framework".

I`am not that mono specialist we should ask @fraga :)

According to this thread System.Security does not seem to be available for the mobile target. My C# knowledge is too limited to build that myself :(

Per #96, there are changes needed to the library, and a different build of libsodium is needed to make it work.
This is something that needs to be done, though hasn't happened yet - the focus has been on Windows (desktop) and Linux on Intel. Proper mobile support seems doable (and I think worthwhile), though it's going to take a little work.
With access to the right software & hardware for testing shouldn't be too bad; it will likely require re-working how we call into libsodium, dealing with the lack of System.Security is possible though conditional compile and and eliminating the use of it where possible. Changes like that - not a big deal, but some time.
The other issue will be dealing with a customized build of libsodium, will need to see if there's an efficient way to deal with it, so we don't add too much overhead to the release process.

That would really be great! @adamcaudill let me know if I can assist you with anything.


spawn a separate process for move play by algo player

so that we don't block our server script process

this will also allow to instantiate a better sandboxed algo play


Fix regression of /build_* in gitignore

Commit Message:
This is a regression of 887637c1277134df9bea3ad9ac958e38ca69f6db that should be retained.
This patch re-add it into the gitignore file.
Signed-off-by: DongRyeol Cha<EMAIL_ADDRESS>Additional Description: None
Risk Level: Low
Testing: git status
Docs Changes: None
Release Notes: None

Sorry for my fault. :(


Wrong warning color when using CtrlP

environment
neovim 0.2.0 on Ubuntu 16.04 using xterm/tmux
$ echo $TERM
screen-256color

minimal config:
if isdirectory(expand('~/.vim/bundle/vundle/'))
    set runtimepath+=~/.vim/bundle/vundle/
    call vundle#rc()
    Bundle 'ctrlpvim/ctrlp.vim'
    Bundle 'vim-airline/vim-airline'
    Bundle 'vim-airline/vim-airline-themes'
    Bundle 'w0rp/ale'
endif
let g:airline_theme = 'luna'

Not sure how to get warnings without Ale which doesn't work under Vim 7 which is the only alternative I have available, so I can't test with Vim
actual behavior
The warning color is black on orange when I open a file that has warnings. When I then type Ctrl-P and Esc (to bring up the CtrlP prompt) the color changes to the one from luna.
expected behavior
The color from luna should be used from the start.

can you show a screenshot please?

also in each case, in the window with the wrong colored statusline please show the output of :echo eval(&stl[2:])

:echo eval(&stl[2:])
%{airline#check_mode(1)}%#airline_a#%( %#airline_a_bold#%{airline#util#wrap(airline#parts#mode(),0)}%#airline_a#%{airline#util#append(airline#parts#crypt(),0)}%{airline#util#append(airline#parts#paste(),0)}%{airline#util#append("",0)}%{airline#util#append(airline#parts#spe
ll(),0)}%{airline#util#append("",0)}%{airline#util#append("",0)}%{airline#util#append(airline#parts#iminsert(),0)} %)%#airline_a_to_airline_c#%#airline_c#%( %{getcwd()}  %n: %f %)%=%#airline_c_to_airline_z#%#airline_z#%( %l/%L : %c %)%#airline_z_to_airline_error#%#airline_
error#%( %{airline#util#wrap(airline#extensions#ale#get_error(),0)} %)%#airline_error_to_airline_warning#%#airline_warning#%( %{airline#util#wrap(airline#extensions#ale#get_warning(),0)} %)

The echo eval was enough to change the color, so it's not the ctrl-p changing the color, but only with ctrl-p do I get the wrong color.

and where is the wrong color? the screenshot looks the same both times.

Bottom right, at first it's orange and black font, then it's darkish red and white font.

This seems to be a repaint issue and I just found another one that has the same origin and that I can reproduce without ale and CtrlP:
if isdirectory(expand('~/.vim/bundle/vundle/'))
    set runtimepath+=~/.vim/bundle/vundle/
    call vundle#rc()
    Bundle 'vim-airline/vim-airline'
    Bundle 'vim-airline/vim-airline-themes'
endif
let g:airline_theme = 'luna'

I use export MANPAGER="/bin/sh -c \"col -b | vim -c 'set filetype=man tabstop=8 nomodified nomodifiable nolist nospell nonumber' -\"" in my shell config to have vim as my manpage viewer. So when I do man vim I get the manpage for vim in vim. The problem is that the airline color tells me the file has been modified even though I set nomodified and there is no + indicating a modification:

Just be resizing my xterm the color changes to the one it should be:

does by any chance this patch fix it?
diff --git a/autoload/airline.vim b/autoload/airline.vim
index 1179c2b..f92ffd9 100644
--- a/autoload/airline.vim
+++ b/autoload/airline.vim
@@ -190,7 +190,7 @@ function! airline#check_mode(winnr)
   endif

   let mode_string = join(l:mode)
-  if s:airline_run < 3
+  if 0 && s:airline_run < 3
     " skip this round.
     " When this function is run too early after startup,
     " it forces a redraw by vim which will remove the intro screen.

Yes it does! :-)

Thank you for the fix!


[W.I.P]Start refactoring ProfileToolBar

Start refactoring of ProfileToolBar. Now one ToolBar for 2D and one for 3D

Question shared with @PiRK : This refactoring bring two classes :  Profile2DToolBar and Profile3DToolBar inheriting from ProfileToolBar which became an abstract class. So previous code outside silx should deal with it. Is it bad enough to

do not do this refactoring
Let implicitly ProfileToolBar be 2D abd create a AAbstractToolBar
something else...

But from what I recall other stuff needed to be refactored due to 3D feature.

Some detected issues:
a bottleneck is that we have one plot per dimension. This bring to increase code complexity.
We actually have a ProfileMainWindow which is used but only for 2D profile.
A way to simplify would be to always use the same PlotWidget and calling a setDefault1DInterface and a setDefault2DInterface to switch from 1D and 2D. But user wont be able to tune is own PlotWidget for profile.
We can also always define ProfileMainWindow and a boolean to know if this widget is only used to managed the plot or also for the window show/hide.

not useful for now


how to train a face editor with stylegan?

Hi, does it support train a editor which input an image and add style on it? such as make a face smile and so on?

This repo doesn't have the code to do this, but once you've trained a model it is possible to add code which does this.

Do u know how? Any blogs or tutorials indicates how to do this?


[WIP] Update Podspec to 3.0.0, add AsyncHttpClient dependency

Update the Podspec for the 3.0.0 release and add a dependency on the async-http-client.
DO NOT MERGE: async-http-client pod does not yet exist.  See https://github.com/swift-server/async-http-client/issues/114

 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.


Add sentry

Progress towards #31

Codecov Report

Merging #68 (53a93d9) into master (9d3540e) will increase coverage by 0.53%.
The diff coverage is 99.57%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   91.98%   92.52%   +0.53%
==========================================
  Files          52       53       +1
  Lines        2946     3172     +226
==========================================
+ Hits         2710     2935     +225
- Misses        236      237       +1

Impacted Files
Coverage Δ

pyminion/players.py
93.86% <ø> (-0.03%)
:arrow_down:

pyminion/bots/optimized_bot.py
88.13% <95.00%> (+0.39%)
:arrow_up:

pyminion/__init__.py
100.00% <100.00%> (ø)

pyminion/expansions/base.py
94.13% <100.00%> (+0.63%)
:arrow_up:

tests/test_cards/test_actions/test_sentry.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


How to do inference of Tacotron GST?

I follow the steps in https://nvidia.github.io/OpenSeq2Seq/html/speech-synthesis/tacotron-2-gst.html to train Tacotron GST. But I do not how to do inference. I encounter errors following the steps of https://nvidia.github.io/OpenSeq2Seq/html/speech-synthesis.html#inference
Thanks.

I trained my model on another machine and gotthe same situation. I also read #302 . I think my issue is caused by changing the batch_size to 16 since my 1080 TI do not have enough memory. Is it the reason?  Thanks.

It seems like batch size does seem to make a difference according to #302. Try adding iter_size to the example and set it to 2 to simulate batch size 32.
I would also recommend to train it without a eval set. Use all data as training data

I am afraid that the situation is the same when training with setting iter_size to 2 to simulate batch size 32. I did not use any eval set. I use all the data for training data at the very beginning.

What are requirements for style.wav ? it can contain arbitrary sentence? what is minimum / recommended length of style.wav file?

Ideally style.wav should contain similar wavs to what you conditioned tacotron gst on. It would to interesting to experiment with different wavs and see which parameters are most important.

@blisc should you point me where is checkpoints?


Abdulhakim/Do not allow null or empty actions

This PR checks for Actions in the response and will return 400 if actions are null or empty.
The Bug# 2161553

Potential issue when testing

Potential issue found with this PR. Converted to draft


Cannot set g:airline_powerline_fonts from rc.local

airline_powerline_fonts gets overridden before the vimrc.local is called, which means it always falls back to the non powerline font option.

I had this problem as well. Fix submitted above.


Refactor profile component

Its getting pretty hectic, would be nice to clean up.

a couple of basic ideas:

Awesome


NONMEM License Update is Required for Tests

My preference is that if the installation isn't tested, the Docker image isn't generated.  That way, my qualification tests are as simple as "If the image exists, it's qualified."  For that, the NONMEM license file must be updated prior to starting the SETUP script.
Currently, the NONMEM installation indicates that the license is expired, and no tests are run.
If we add the following lines to the NONMEM Dockerfile, it would update the license file (without requiring a copy operation):
# Preamble
ARG NONMEM_LICENSE_STRING=""
# ... just before running SETUP
  && if [ -n "$NONMEM_LICENSE_STRING" ]; then echo $NONMEM_LICENSE_STRING > /opt/nm/license/nonmem.lic ; fi \
# ... run SETUP and the rest of the Dockerfile

Anything in an ARG of a final build image/stage is retrevable via the image layers history, so best not to put sensive info in that. See 1st comment in #4 for a path forward :)

Definitely true.  I was just trying to improve relative to current state more than thinking of this as a long-term fix.  I'm very interested in the multi-stage builds option you suggested in #4.

This is definitely not the best solution for a public image.  I view it as an improved solution until we have the public image solution fixed.

I feel like this and #4 are same, so will close this issue and we can discuss in #4 to keep the final fix easier to understand/track.


Feat: Adding particles Animation

Describe the feature
I will add a particles animation on the header which makes it more  attractive .
I am attaching the reference of the particles but if you want any other particles let me know
Add ScreenShots

https://github.com/subhadeeproy3902/BloxAI/assets/118350936/cf2cae9b-add9-4d78-863a-13be8ff271b5
Record

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

Too much animation isnt liked by everyone


Add support for fs delete (unlink/unlinkat) events

This PR adds interposing of the unlink and unlinkat system calls, wrapping the calls and generating fs.delete events to notify the user of file removal.
Considerations

We created a new metric type FS_DELETE, since there was no appropriate DELETE type in existence. We now handle that type in doUpdateState() and on the reporting side in doFSMetric().
We decided that there was no need for error handling in the unlink function i.e. we don't need to report an error with unlinking.

Testing

Integration tests for unlink, unlinkat were added to the syscalls tests. This can be tested with the following commands:

cd test/integration
make syscalls # runs in a container

QA instructions can be found here

Questions for Reviewers

In the case of unlinkat, should I pass the dirfd into doDelete(...) and into doUpdateState(...) ?

Closes: https://github.com/criblio/appscope/issues/709

Looks great.  Merging.


Full audit of all field usage

While auditing the model code I found that Stripe stores all currency amounts in cents. We store them in a DecimalField, which is probably related to #247. I don't fully understand why djstripe does that rather than give a way to translate from one to the other.
Additionally, the StripeCurrencyField naming is extremely confusing... because of all these:
currency = StripeCharField(max_length=3, help_text="Three-letter ISO currency code")
These probably should be called StripeCurrencyField rather than just be a charfield, that we may give them a choices for all the currencies.

It looks like djstripe is storing a lot of bigints into ints. This works for most use cases but it's probably easy to get the dashboard to crash djstripe.

The answer is sheer convenience. I'll add a comment to #247.

It looks like djstripe is storing a lot of bigints into ints
For which fields?

Now that django has a bigint field, we can switch over any fields that need it.

I'll do a full audit of all fields at some point and report my findings here.

General

The StripeCurrencyField field type needs discussion, as per original post here.
All currency objects need discussion, see above.
For the sake of simplicity, I will not be going into details on stripe connect functionality.
There are created and modified objects on the StripeObject model. Stripe itself has a created object. These two are not the same - one is created in stripe, the other is created in djstripe. The created field should be renamed djstripe_created, and a new created timestamp should be added.
We need to get our story straight on customer objects. Is a "deleted" customer object deleted in the db or not? This impacts all cascades to customer.
Nullable StripeCharField / StripeTextField: The usual django pattern is to make the field non-nullable but blank=True and store an empty string in place of a null. We should figure out what we want to do there.

Core resources
Balance

The Balance object cannot be tracked (it's not listable/updatable).
The BalanceTransaction object (txn_XXXX) is not tracked, but should be as some of it is referenced in Charge objects.

Charges
Charge object
The following fields are not tracked:

application (connect)
application_fee (connect)
balance_transaction (see Balance)
destination (connect)
dispute (No Dispute model; has disputed instead)
fraud_details (uses fraudulent instead)
invoice
order (Order objects are not implemented)
outcome
receipt_email
receipt_number
review
refunds
source_transfer (connect)
transfer_group (connect)

Extraneous fields: fee, fee_details, source_type, source_stripe_id, disputed, fraudulent.
Other issues:

captured, paid, refunded have a default - why?
on_behalf_of has the name account instead of simply on_behalf_of. It's never correct.
customer is currently set to CASCADE on delete; but if a customer object is closed, the charge remains.
invalid_swipe_data is mlssing from card error codes (failure_code).
rate_limit is present in card error codes, but not officially documented.
STATUS_CHOICES does not have STATUS_PENDING in it for some reason.

Customers
Disputes
The Dispute object (dp_XXXX) is not implemented.
Events
File uploads
Payouts
Refunds
Tokens
Transfers
Legacy transfers
Transfer Reversals


IlmControl: Fix no warning when passing an invalid surface to function input_listener_input_acceptance

In function input_listener_input_acceptance, variable "surface_found" should be initialized to 0.
Only then "if(!surface_found)" will take effect.
Signed-off-by: Au Doan Ngoc<EMAIL_ADDRESS>

Reviewed-by: Tran Ba Khang<EMAIL_ADDRESS>


Feature/format chars

Preview of #237

This adds the feature discussed in hbldh#237:
It is included in bleak.utils:

get_char_xml (function): to parse characteristic metadata
get_char_value (function): to unpack the raw bytes of read_gatt_char into the proper values
(optional) pformat_char_value, map_char_value, dict_char_value, pformat_char_flags: post-processing of the characteristic value

Example: service_explorer.py:
$ python3.8 service_explorer.py
Connected: True
[Service] 180A: Device Information
        [Characteristic] 00002a01-0000-1000-8000-00805f9b34fb: (Handle: 12) (read) | Name: , Value: b'\x00\x03'
        [Characteristic] 00002a29-0000-1000-8000-00805f9b34fb: (Handle: 14) (read) | Name: , Value: b'Espressif Incorporated'
        [Characteristic] 00002a24-0000-1000-8000-00805f9b34fb: (Handle: 16) (read) | Name: , Value: b'esp32'
        [Characteristic] 00002a26-0000-1000-8000-00805f9b34fb: (Handle: 18) (read) | Name: , Value: b'micropython-1.12.0'
[Service] 180F: Battery Service
        [Characteristic] 00002a19-0000-1000-8000-00805f9b34fb: (Handle: 21) (read) | Name: , Value: b'^'
        [Characteristic] 00002a1a-0000-1000-8000-00805f9b34fb: (Handle: 23) (read,notify) | Name: , Value: b'\xbf'
                [Descriptor] 2902: (Handle: 25) | Value: b'\x00\x00'
[Service] 181A: Environmental Sensing
        [Characteristic] 00002a6e-0000-1000-8000-00805f9b34fb: (Handle: 27) (read,notify) | Name: , Value: b'Z\x16'
                [Descriptor] 2902: (Handle: 29) | Value: b'\x00\x00'"""

Example: service_explorer.py with get_char_value and pformat_char_value
    for char in service.characteristics:
                if "read" in char.properties:
                    try:
                        value = bytes(await client.read_gatt_char(char.uuid))
                        value = get_char_value(value, char)
                        pretty_format = pformat_char_value(value,
                                                           one_line=True,
                                                           prnt=False, rtn=True)
                    except Exception as e:
                        print(e)
                        value = str(e).encode()
                        pretty_format = value
                else:
                    value = None
                    pretty_format = value
                log.info(
                    "\t[Characteristic] {0}: (Handle: {1}) ({2}) | Name: {3}, Value: {4} ".format(
                        char.uuid,
                        char.handle,
                        ",".join(char.properties),
                        uuidstr_to_str(char.uuid),
                        pretty_format,
                    )
                )

$ python3.8 service_explorer.py
Connected: True
[Service] 180A: Device Information

        [Characteristic] 00002a01-0000-1000-8000-00805f9b34fb: (Handle: 12) (read) | Name: Appearance, Value: Category: Generic Thermometer
        [Characteristic] 00002a29-0000-1000-8000-00805f9b34fb: (Handle: 14) (read) | Name: Manufacturer Name String, Value: Manufacturer Name: Espressif Incorporated
        [Characteristic] 00002a24-0000-1000-8000-00805f9b34fb: (Handle: 16) (read) | Name: Model Number String, Value: Model Number: esp32
        [Characteristic] 00002a26-0000-1000-8000-00805f9b34fb: (Handle: 18) (read) | Name: Firmware Revision String, Value: Firmware Revision: micropython-1.12.0
[Service] 180F: Battery Service
        [Characteristic] 00002a19-0000-1000-8000-00805f9b34fb: (Handle: 21) (read) | Name: Battery Level, Value: Level: 94 %
        [Characteristic] 00002a1a-0000-1000-8000-00805f9b34fb: (Handle: 23) (read,notify) | Name: Battery Power State, Value: State: {'BitGroup 0': 'Present', 'BitGroup 2': 'Discharging', 'BitGroup 4':
'Charging (Chargeable)', 'BitGroup 6': 'Good Level'}
                [Descriptor] 2902: (Handle: 25) | Value: b'\x00\x00'
[Service] 181A: Environmental Sensing
        [Characteristic] 00002a6e-0000-1000-8000-00805f9b34fb: (Handle: 27) (read,notify) | Name: Temperature, Value: Temperature: 57.22 °C
                [Descriptor] 2902: (Handle: 29) | Value: b'\x00\x00'

This is interesting indeed. Is it possible to trim down the xmls to what you actually need instead of including the entire files? This will grow the size of the bleak installation by many order of magnitudes, and I am reluctant to do so.
I think it would be better to extract this into a separate package (blues_sigspec?) and release it separately on PyPI? Reference to that package could then be added to the documentation for people to install separately.


PLT-1345 Simplify bappTerm / bappType

This PR describes the changes to the partial application of builtins, which allows us to avoid pattern matching on the builtins in functions that do not actually need it (that is most functions that did it).
It includes the addition of a predicate classifying types generated by a signature (SigTy) and a new design of the type of partially applied builtins (BApp).
Because of the changes, there are four proofs that need to be updated (and are not done yet)

Determinism of reduction semantics.
Behavioural equivalence between reduction and CC.
Behavioural equivalence between CC and CK.
Behavioural equivalence between CK and CEK.

The relevant files have been commented out in index.lagda.md so that documentation can be generated anyway if it is merged.

CI seems unhappy and timed out, sometimes just doing a vacuous amend commit and force pushing to the branch resets it.


Fixed issue #731

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!
Providing the issue number/url helps us identify the problem being solved and keep the issues tab clean and tidy

This is not a fix. This is a request to add login and signup pages to the site.
Summarize what this PR fixes:-
Fixes #
Checklist
(Add an x between the brackets to check the items)

[ x ] I have summarize what this PR fixes above
[ x ] I have provided the issue ID/URL above
[ x ] I have not added Bootstrap or jQuery and used correct styling guide.
[ x ] This PR fixes only the issue mentioned above (create separate PRs for separate issues)
[ x ] I was the first one to claim the issue or I created the issue - I created the issue

Actually, I see you linked it in the title. The issue should be linked in the pr template with a description of the changes. There are some conflicts in your pr though and it is unable to merge. I'm going to close this out, but I suggest running git pull upstream master and making the changes again.

Ok thanks for your suggestion
On Tuesday, October 13, 2020, Brittney Postma<EMAIL_ADDRESS>wrote:

Actually, I see you linked it in the title. The issue should be linked in
the pr template with a description of the changes. There are some conflicts
in your pr though and it is unable to merge. I'm going to close this out,
but I suggest running git pull upstream master and making the changes
again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zero-to-mastery/Keiko-Corp/pull/733#issuecomment-707800410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO3PROPJABEMQ757474EADLSKRTRBANCNFSM4SOD3TIA
.


Use npm module 'is-local-host' to determine if a URL is a local host or not

Fixes #9748

Codecov Report

Merging #9751 (4a370ab) into main (c23df7d) will decrease coverage by 0%.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           main   #9751   +/-   ##
====================================
- Coverage    63%     63%   -1%
====================================
  Files       204     204
  Lines      9862    9862
  Branches   1559    1559
====================================
- Hits       6270    6262    -8
- Misses     3093    3099    +6
- Partials    499     501    +2

Impacted Files
Coverage Δ

...rc/platform/common/dataScienceSurveyBanner.node.ts
65% <0%> (-6%)
:arrow_down:

src/platform/errors/errorHandler.node.ts
65% <0%> (-2%)
:arrow_down:

Blech I can't find an npm module that doesn't have node dependencies. I'm just going to close this for now.


fix not saving device_fingerprint on logout

Really simple fix for Cordova (also woudl affect Titanium)
Bug: Upon logging out, the session looses the device_fingerprint_id, which would cause errors for:

all subsequent API requests, because the required field device_fingerprint_id is missing, and
Relaunching the app and calling open, because device_fingerprint_id is missing

So the new procedure for fixing this on Cordova and Titanium will be to merge the PR here, int eh Web SDK. Then, updating the submodule, and running make in each of the other SDKs will update build.js in each project.

seems to work for me :+1:
restarting tests


'numpy.float64' object has no attribute 'values'

'numpy.float64' object has no attribute 'values'
Please Try to run this : https://github.com/AI4Finance-Foundation/FinRL-Tutorials/blob/master/1-Introduction/Stock_NeurIPS2018_Train.ipynb
Error :
/usr/local/lib/python3.8/dist-packages/finrl/meta/env_stock_trading/env_stocktrading.py in _initiate_state(self)
402                 # for multiple stock
403                 state = (
--> 404                     [self.initial_amount]
405                     + self.data.close.values.tolist()
406                     + self.num_stock_shares

+1 for this. I just encountered the same issue running on python 3.8

I get a similar error in the notebook "FinRL_PortfolioAllocation_NeurIPS_2020"

@ambresh1
Sorry this is caused by change of dataframe index when splitting the notebook. It's now fixed. Please go to https://github.com/AI4Finance-Foundation/FinRL-Tutorials/tree/master/1-Introduction/Stock_NeurIPS2018 and run the corresponding notebook.

If anyone solve the problem in "FinRL_PortfolioAllocation_NeurIPS_2020.ipynb", you are welcome to reply to this issue or send a PR.

I'm trying to execute this notebook and got the same error.
https://github.com/AI4Finance-Foundation/FinRL-Tutorials/blob/master/1-Introduction/Stock_NeurIPS2018/Stock_NeurIPS2018_2_Train.ipynb
buy_cost_list = sell_cost_list = [0.001] * stock_dimension
num_stock_shares = [0] * stock_dimension

env_kwargs = {
    "hmax": 100,
    "initial_amount": 1000000,
    "num_stock_shares": num_stock_shares,
    "buy_cost_pct": buy_cost_list,
    "sell_cost_pct": sell_cost_list,
    "state_space": state_space,
    "stock_dim": stock_dimension,
    "tech_indicator_list": INDICATORS,
    "action_space": stock_dimension,
    "reward_scaling": 1e-4
}

e_train_gym = StockTradingEnv(df = train, **env_kwargs)

File [~/Library/Caches/pypoetry/virtualenvs/playground-6H0uwau1-py3.10/lib/python3.10/site-packages/finrl/meta/env_stock_trading/env_stocktrading.py:404](https://file+.vscode-resource.vscode-cdn.net/Users/dan/Git/Alchemist/playground/~/Library/Caches/pypoetry/virtualenvs/playground-6H0uwau1-py3.10/lib/python3.10/site-packages/finrl/meta/env_stock_trading/env_stocktrading.py:404), in StockTradingEnv._initiate_state(self)
    398 if self.initial:
    399     # For Initial State
    400     if len(self.df.tic.unique()) > 1:
    401         # for multiple stock
    402         state = (
    403             [self.initial_amount]
--> 404             + self.data.close.values.tolist()
    405             + self.num_stock_shares
    406             + sum(
    407                 (
    408                     self.data[tech].values.tolist()
    409                     for tech in self.tech_indicator_list
    410                 ),
    411                 [],
    412             )
    413         )  # append initial stocks_share to initial state, instead of all zero
    414     else:
    415         # for single stock
    416         state = (
    417             [self.initial_amount]
    418             + [self.data.close]
    419             + [0] * self.stock_dim
    420             + sum(([self.data[tech]] for tech in self.tech_indicator_list), [])
    421         )


Enable gpu execution

Because
When model=large, Whisper is extremely slow when running on CPU-only.
Done when
When a user runs whisper-wrapper with parameter device=cuda Whisper runs on the machine's GPU instead of CPU.
If it is not possible to choose gpu or cpu as a parameter sent to the web service, it would still be fine (though less desirable) to have a separate app or version with gpu support enabled.
Additional context
Chowda is already configured to use CUDA.

Cool.  Thanks.  We'll see if we can get that working.

This might be a better reference to install helpers on Windows: https://docs.nvidia.com/cuda/wsl-user-guide/index.html

closing as non-issue


Right way to use Postgres

Sorry for disturbing, I'm new with python.
Could you please show me a sample of right way to connect 🐑 with a postgres.
Thank you!

Hi @david-lint
Tomorrow I will publish a project template featuring one of the possible good ways to use Postgres. There is not a single "right way", because some people like to use SQL dbs with an ORM, some people prefer not to use ORMs; some prefer DB first approach, others code first.
I personally prefer code first approach, and I like both using ORMs and using db drivers directly.
The template I will publish tomorrow enables the following things:

database migrations applied automatically by continuous delivery pipelines (implemented using GitHub workflows), using Alembic

collection of telemetries to keep a log of how much time queries take to complete (using OpenCensus and Azure Application Insights), to get reports like the following:

business logic and front-end layer are abstracted from the code that interacts with Postgres (this, in my opinion, is a fundamental feature)

Please let me come back to this issue tomorrow, or on Monday, to elaborate a bit on the project template I will publish.

Note that the telemetries are collected using OpenCensus, therefore the same principle I will share is valid and applies to other scenarios (even without Azure Application Insights).

Hi @david-lint,
As I promised, yesterday I started publishing the demo project I am working on. It is still a work in progress, but you can already see an example of using PostgreSQL with BlackSheep.
I described it here: https://github.com/Neoteroi/BlackSheep/discussions/130#discussioncomment-746498
Hope this helps. 🙂

Also, to add a few more words to the @RobertoPrevato 's answer, you can consider the MVC pattern for your applications.
Regarding async libraries, I can suggest the https://github.com/encode/databases package

Hi @david-lint
Today I published a library to simplify the integration with SQLAlchemy -> https://pypi.org/project/blacksheep-sqlalchemy/
For now it's basic, as it registers the necessary services to:

registers services to the DI container so that db connection, sessions, and the engine, so that they can be injected to request handlers (or dependent services) - there is a full example here
dispose of the SQLAlchemy engine when the application stops

The demo project I described a few days ago is something advanced, handling more details, but if you need to get started with something lighter, you can take a look at the new repository. It also includes integration for ORM Sessions, that can be preferred by many over using a driver directly.
Closing this ticket now, since much information has been provided on this subject. Please let me know if you have any recommendation on integrations with PostgreSQL or other RDBMS.


Can a single nexrender-server work with multiple nexrender-workers?

Hello!
I have a question:
Can a single nexrender-server work with multiple nexrender-workers? For example, when I have one queue on the site-0.com server (nexrender-server is installed) and this queue distributed tasks to several site-1.com, site-2.com, site-3.com servers (with nexrender-worker installed on each of them). For example: http://prntscr.com/pxxyub.
If so, then I want to understand --host http://prntscr.com/py2r5i is my server address where is nexrender-server located? I want to understand this, because I did all my tests on the same machine on which nexrender-server + nexrender-worker were located at the same time and the address was the same, so I got a little confused. I must understand this correctly to rebuild my system.
Thanks in advance!

Hello,
Yes, that is actually the intended way it should work.
And yes, host points to the actual address where your nexrender-server is running.


Fix matrix-project hard dependency

Missed a call to matrix-project compile time dependency before
Closes https://github.com/jenkinsci/slack-plugin/issues/557

@timja GitHub being odd: https://github.com/timja/slack-plugin/commit/cdfc98ece3e656e7634825171d79c90b110760a6 yet PR is not updated :sob:

Jenkins CI being slow :cry:


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2024-11-05)
Features

PaCMAP initial commit (45e2902)

Bug Fixes

handle low row counts (f05aee6)
repository URL (2526720)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.0
:sunflower:


Run cargo fmt in CI

Could also be a separate GH workflow job, either way :shrug:

Great idea! :+1:


[BUG] - toggle.js should be included instead of switch.js when using Switch component

NextUI Version
<EMAIL_ADDRESS>Describe the bug
When I want to import Switch in my project, I add switch.js in tailwind.config.ts:
content: [
    './src/pages/**/*.{js,ts,jsx,tsx,mdx}',
    './src/components/**/*.{js,ts,jsx,tsx,mdx}',
    './src/app/**/*.{js,ts,jsx,tsx,mdx}',
    './node_modules/@nextui-org/theme/dist/components/(button|input|modal|radio|switch).{js,ts,jsx,tsx}'
  ],

Then, I found no UI change when I click Switch component on my page, and all states are correct. I checked node_modules directory, no switch.js there. But toggle.js was found.
I replace switch to toggle, problem was solved. I'm not sure why this happened, someone may check the compile procedure?
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

add following content in tailwind.config.ts:

content: [
    './node_modules/@nextui-org/theme/dist/components/switch.{js,ts,jsx,tsx}'
  ],

add basic Switch example on official docs
no UI change when click Switch on page
replace switch to toggle:

content: [
    './node_modules/@nextui-org/theme/dist/components/toggle.{js,ts,jsx,tsx}'
  ],

Expected behavior
I expected add switch in tailwind.config.ts, but I have to add toggle, without any tips on official docs
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Chrome

We cannot use switch because it is a reserved word in Javascript.


Remove duplicated comment and reformat with two references

this is ferris/rudi enjoying trying to fix docs :slightly_smiling_face:

thanks @DavidVorick all fixed now i believe


chore: adding state and eventing to readme

This PR
Add details about state and eventing handling to readme

Can you use the phrasing from the template here? It's a bit better than what's in JS, IMHO
https://github.com/open-feature/.github/blob/main/templates/READMEs/README.md?plain=1
I specifically mean the phrasing for events, shutdown, etc, the description text.
Your code examples all look good.

Thanks and I updated section descriptions from the template


Release Google.Cloud.Video.LiveStream.V1 version 1.6.0

Changes in this release:
New features

Add IServiceCollection extension methods for client registration where an IServiceProvider is required. (commit 022fab2)

Created release for Google.Cloud.Video.LiveStream.V1-1.6.0


Add websocket server mode

It would be handy to create a JSON-driven websocket mode. A web-based frontend would then be possible without the need for additional middleware.

Merged in 7ccb7de396eb60d7ab5087de6774f1f859679367. All tests pass and all executables build on Linux and Windows.


[general-sdk] split WatsonDeveloperCloud into multiple projects

As the Watson Developer Cloud iOS SDK grows, it becomes increasingly important to maintain stable code, even as new services and versions are released. This can pose a challenge, in particular, to maintain multiple versions of each service (e.g. Tone Analyzer V1 and Tone Analyzer V3).
We believe that the best solution is to create an Xcode project for each of the Watson services instead of a single project for the entire SDK. This will allow developers to pick-and-choose the service versions they would like to use. It also enables developers to pick-and-choose the components of the SDK that they would like to embed in their application (currently, the entire SDK must be embedded into the application even if only a single service is being used).
We want to investigate how to have multiple projects within the same repository, while maintaining the ability for them to be built by the Swift Package Manager, Carthage, and Cocoapods. In particular, we will probably maintain a "WatsonDeveloperCloud" master project with no code but with each project as a dependency, so that consumers still need only enter a single line into their Cartfile (or equivalent) in order to download and build the SDK.

The concerns above may be addressed by using the Swift Package Manager. In particular, SPM automatically creates separate targets based on the directory structure of the repository. I still need to explore it more, but I believe this will allow us to very easily separate the services into separate modules.
In other words, using SPM will "automatically" switch us from a import WatsonDeveloperCloud world to a import ToneAnalyzerV3 world.
The biggest remaining concern is with regards to semantic versioning. It's difficult to properly follow semantic versioning when all SDKs are in one repository--a breaking change to a single service requires the entire SDK version to be incremented.


H3: Do not send Settings that have the default value

Thus minimize exchanged volume. Exchanging default values is useless.

Closed via #643


Is it secure?

Hi there,
am I right that no key derived from user key(s) is used to encrypt the filename? What's the added security then if one may easily decode the file names?
Thanks,
Peter

Hi Peter,
Base4K is an encoding, not an encryption scheme. The difference is this:

Encryption uses a key and the purpose is to prevent anyone from reverting the result back to plaintext without it
Encoding uses no key and the purpose is to transform bytes into different bytes with different attributes. For Base4K, these attributes are that the result can be printed and is as short as possible (in terms of unicode characters). Everyone can revert it back to the original bytes.

Boxcryptor is using Base4K after it encrypted the filename. Because filename encryption results in raw bytes, and these cannot be set as filenames. The filename encryption in Boxcryptor is therefore: Base4K(Encrypt(plaintextFilename, userAESKey)).
We actually dedicated a whole blog article to our filename encryption scheme and the role of Base4K in it - feel free to read it:  https://www.boxcryptor.com/en/blog/post/filename-encryption-with-boxcryptor/
Best regards,
Christian

Thanks, indeed. Is there any reason not to use base64 instead?

Yes, the fact that many cloud storage providers set a character limit on filenames.
Base64 encoded bytes result in very long strings, Base4K ones are very short. With a shorter encoding scheme, we can store more information in one filename, which means that the user can use longer filenames than would be possible with a Base64 encoding.


I wonder if there is a way to check the network status

I added a dozen torrents tasks and each of them has tens of peers but none of them has download speed. Some of them are hot sources and can be downloaded by Thunder in high speed, so I wonder if there is a way to check the network status and find out whether there is something wrong or not. Thanks.

The best way is to download the Ubuntu ISO file
https://releases.ubuntu.com/20.04/ubuntu-20.04.5-desktop-amd64.iso.torrent


Config changes to reflect the default branch from master to main. Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit 7824d7f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


can't create blueprint with 0.52.4

Hi Chris,
I've some trouble creating blueprint towards Apstra 4.2.0 and 4.2.1. I'm using version 0.52.4. The code is your example from: https://registry.terraform.io/providers/Juniper/apstra/latest/docs/resources/blueprint_deployment

# This example instantiates a blueprint from a template,
# allocates IPv4 and ASN resource pools to template roles
# and deploys the template.

# Instantiate a blueprint from a template
resource "apstra_datacenter_blueprint" "instantiation" {
  name        = "terraform commit example"
  template_id = "L2_Virtual_EVPN"
}

# ASN pools, IPv4 pools and switch devices will be allocated using looping
# resources. These three `local` maps are what we'll loop over.
locals {
  asn_pools = {
    spine_asns = ["Private-64512-65534"]
    leaf_asns  = ["Private-4200000000-4294967294"]
  }
  ipv4_pools = {
    spine_loopback_ips  = ["Private-10_0_0_0-8"]
    leaf_loopback_ips   = ["Private-10_0_0_0-8"]
    spine_leaf_link_ips = ["Private-10_0_0_0-8"]
  }
  switches = {
    spine2               = "Juniper_vQFX__AOS-7x10-Spine"
    spine1               = "Juniper_vQFX__AOS-7x10-Spine"
    l2_virtual_001_leaf1 = "Juniper_vQFX__AOS-7x10-Leaf"
    l2_virtual_002_leaf1 = "Juniper_vQFX__AOS-7x10-Leaf"
    l2_virtual_003_leaf1 = "Juniper_vQFX__AOS-7x10-Leaf"
    l2_virtual_004_leaf1 = "Juniper_vQFX__AOS-7x10-Leaf"
  }
}

# Assign interface maps to fabric roles to eliminate build errors so we
# can deploy
resource "apstra_datacenter_device_allocation" "interface_map_assignment" {
  for_each         = local.switches
  blueprint_id     = apstra_datacenter_blueprint.instantiation.id
  node_name        = each.key
  initial_interface_map_id = each.value
}

# Assign ASN pools to fabric roles to eliminate build errors so we
# can deploy
resource "apstra_datacenter_resource_pool_allocation" "asn" {
  for_each     = local.asn_pools
  blueprint_id = apstra_datacenter_blueprint.instantiation.id
  role         = each.key
  pool_ids     = each.value
}

# Assign IPv4 pools to fabric roles to eliminate build errors so we
# can deploy
resource "apstra_datacenter_resource_pool_allocation" "ipv4" {
  for_each     = local.ipv4_pools
  blueprint_id = apstra_datacenter_blueprint.instantiation.id
  role         = each.key
  pool_ids     = each.value
}

# The only required field for deployment is blueprint_id, but we're ensuring
# sensible run order and setting a custom commit message.
resource "apstra_blueprint_deployment" "deploy" {
  blueprint_id = apstra_datacenter_blueprint.instantiation.id

  #ensure that deployment doesn't run before build errors are resolved
  depends_on = [
    apstra_datacenter_device_allocation.interface_map_assignment,
    apstra_datacenter_resource_pool_allocation.asn,
    apstra_datacenter_resource_pool_allocation.ipv4,
  ]

  # Version is replaced using `text/template` method. Only predefined values
  # may be replaced with this syntax. USER is replaced using values from the
  # environment. Any environment variable may be specified this way.
  comment      = "Deployment by Terraform {{.TerraformVersion}}, Apstra provider {{.ProviderVersion}}, User $USER."
}

I get the following error, cutted:

apstra_datacenter_blueprint.instantiation: Creating...
╷
│ Error: Provider returned invalid result object after apply
│
│ After the apply operation, the provider still indicated an unknown value for apstra_datacenter_blueprint.instantiation.access_switch_count. All values must be known after apply, so this is always a bug in the provider and should be reported in the provider's own
│ repository. Terraform will still save the other known object values in the state.
╵
╷
│ Error: Provider returned invalid result object after apply
│
│ After the apply operation, the provider still indicated an unknown value for apstra_datacenter_blueprint.instantiation.build_errors_count. All values must be known after apply, so this is always a bug in the provider and should be reported in the provider's own
│ repository. Terraform will still save the other known object values in the state.
...

hope this help and you can have a look.
Thanks in advance
Viets2k

This kind of thing is expected with Apstra 4.2.1, but shouldn't be happening with 4.2.0. Is it happening with both Apstra versions?

in 4.2.0-236 it's also not working, but error is different:

╷
│ Error: Blueprint has build errors
│
│   with apstra_blueprint_deployment.deploy,
│   on main.tf line 62, in resource "apstra_blueprint_deployment" "deploy":
│   62: resource "apstra_blueprint_deployment" "deploy" {
│
│ Blueprint "512d63c9-c805-45b7-9dbe-05ea59cd988b" has 8 build errors which must be resolved prior to deployment

The blueprint ist create in 4.2.0 and 4.2.1 but it ran into an error.

Ah. This makes sense.
First, the 4.2.0 "Blueprint has build errors" thing:
The apstra_blueprint_deployment resource doesn't really "create" anything, it's more of a "click the 🚀 button on the Uncommitted tab" resource. It causes Apstra to deploy the configuration to your devices.
You'll find the same condition in the GUI: Some detail of the staged configuration is incomplete/incompatible (build error) and requires attention before the staging blueprint can be pushed to your devices.
The underlying issue could come down to any number of things like missing resource pool assignments, no switch assigned to some fabric role, etc...
When the GUI indicates "build errors" (uncommitted -> build errors), it won't let you push the button, and Terraform isn't allowed to push the button either.
Because of the need for explicit dependency mapping via depends_on, it can be tricky to spot these problems: Even if you get all of the resources dialed in to fix the build errors, Terraform doesn't know that deployment has to run last, so you might run the project, get that error, and check the GUI to find all green. This will be because Terraform attempted deploying the project moments before completing it.
Because the "deployment" resource can be tricky to use, especially while you're hammering details of your topology. I recommend you:

set blueprint_mutex_enabled = false in the provider block
skip the apstra_blueprint_deployment resource until the rest of your terraform is consistently rendering no build errors

The 4.2.1 issue boils down to: "some API endpoints have changed and we haven't caught up with them in the Terraform provider". You can see the versions known to work with each release here.

It looks like that should have worked.
Can you share terraform state list and terraform state show apstra_datacenter_resource_pool_allocation.asn["leaf_asns"]?

terraform state list
apstra_datacenter_blueprint.instantiation
apstra_datacenter_device_allocation.interface_map_assignment["l2_virtual_001_leaf1"]
apstra_datacenter_device_allocation.interface_map_assignment["l2_virtual_002_leaf1"]
apstra_datacenter_device_allocation.interface_map_assignment["l2_virtual_003_leaf1"]
apstra_datacenter_device_allocation.interface_map_assignment["l2_virtual_004_leaf1"]
apstra_datacenter_device_allocation.interface_map_assignment["spine1"]
apstra_datacenter_device_allocation.interface_map_assignment["spine2"]
apstra_datacenter_resource_pool_allocation.asn["leaf_asns"]
apstra_datacenter_resource_pool_allocation.asn["spine_asns"]
apstra_datacenter_resource_pool_allocation.ipv4["leaf_loopback_ips"]
apstra_datacenter_resource_pool_allocation.ipv4["spine_leaf_link_ips"]
apstra_datacenter_resource_pool_allocation.ipv4["spine_loopback_ips"]

terraform state show 'apstra_datacenter_resource_pool_allocation.asn["leaf_asns"]'
apstra_datacenter_resource_pool_allocation.asn["leaf_asns"]:
resource "apstra_datacenter_resource_pool_allocation" "asn" {
blueprint_id = "534113df-6749-46ac-b5d6-745146da9fde"
pool_ids     = [
"Private-4200000000-4294967294",
]
role         = "leaf_asns"
}

sorry missed the pre

terraform state show 'apstra_datacenter_resource_pool_allocation.asn["leaf_asns"]'
# apstra_datacenter_resource_pool_allocation.asn["leaf_asns"]:
resource "apstra_datacenter_resource_pool_allocation" "asn" {
    blueprint_id = "534113df-6749-46ac-b5d6-745146da9fde"
    pool_ids     = [
        "Private-4200000000-4294967294",
    ]
    role         = "leaf_asns"
}

Cool.
We should also check whether that ASN pool ID is legit.
You can test that in the Apstra GUI at Resources -> ASN Pools and then clicking on any pool.
Then strip the trailing part of the URL and replace it with this pool ID: Private-4200000000-4294967294

oh shame on me, I accidentially deleted the "Private-4200000000-4294967294" pool before and that's why it failed. If I use the pool "Private-64512-65534" it's working, and it's a bit confusing that the standard ressource using readable strings but everything else random ids.
I'm sorry for wasting your time. Not sure if there should be a warning, pool not found, but other than that it was a human error and you can close it.
Thanks for your support.

Hey, no problem!
I agree about the "friendly" ID strings. Very surprising that "baked in" resources get these IDs while others get random values.
A safer way to proceed here would be to use a data lookup based on a human-readable string, rather than assuming we "just know" the ID:
data "apstra_asn_pool" "private" {
  name = "my pool name"    // <---- human readable name from the web UI - plus we get an error if it doesn't exist
}

locals {
  asn_pools = {
    leaf_asns  = [data.apstra_asn_pool.private.id] // <---- get the pool ID by reference
  }
}

Update on "Not sure if there should be a warning, pool not found":
Apstra engineering has agreed that server side validation of pool assignments is appropriate. We'll have it in a future release. Internally it's being tracked as AOS-42106


Update Command Summary section of Features of the UG

Closing the issue as the UG here is outdated.