Ionic 2 support

Hey there,
I just forked your repo and modified the angular2 implementation so it can work with Ionic2 Applications. See here: https://github.com/ihadeed/ionic2-input-mask
If you are interested in merging this into your repo I can organize it a bit more and submit a PR.
I also noticed some conflicts when running npm install, so I changed package.json a bit.

Sure, I'm interested in merging this. I looked at your fork and left a few comments, but I'm not sure which of those changes fixed Ionic compatibility.
We need the component to be able to work on both Ionic and regular input elements.
Please send a PR!

Well to make it work on Ionic I had to do two things:

Instead of applying the changes directly to "elementRef.nativeElement" I
applied it to the child of that element instead. The reason is that ionic
wraps all inputs with "ion-input" directive. So we have to target it's
child element which will be an input tag.

Move all the initialization code to ngAfterViewInit function. This is
needed because we need for the view to render before we can find the input
tag inside the ion-input tag.

If we want the same codebase to work on both Ionic and Angular, we will
need to do the following:

Do the second step mentioned above and have the init code run inside
ngAfterViewInit

Check if elementRef.nativeElement is an input tag, if not then we should
loop through the children of that element to find an input. This way the
tag can be agnostic and is compatible with input or ion-input

...
I also noticed an issue when using it with a floating ionic2 label. It
looks bad because the guide will overlap with the floating label when the
input is empty and unfocused. It would be nice to be able to toggle the
guides somehow we can resolve this issue with ionic.
On Aug 14, 2016 1:59 PM, "M.K. Safi"<EMAIL_ADDRESS>wrote:

Sure, I'm interested in merging this. I looked at your fork and left a few
comments, but I'm not sure which of those changes fixed Ionic
compatibility.
We need the component to be able to work on both Ionic and regular input
elements.
Please send a PR!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/msafi/text-mask/issues/134#issuecomment-239687173,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANJ8dKie36Uwca9NHWDhChC_U9UAQbGWks5qf1eWgaJpZM4Jj4hL
.

This all sounds good to me.
Regarding the guide, it can be turned off or a custom empty placeholder can be set on the component, something like placeholder=" ".

Well we need the guide to be removed when these conditions are met:

ion-input has floating attribute, if angular doesn't allow an attribute to be shared between two directives we can add our own attribute then.
input value is null or ''
input is not focused

I'm not quite sure how to check if input element is focused or not. The only way I can think of is by having a CSS hack. Add a style to the directive and let it hide the guide of child input:not(:focus)

I'm trying to understand how Text Mask guide might affect Ionic's floating labels. I googled for a demo of Ionic 2 floating labels but couldn't find one. Could you share a link (or a video)?

Here you go!
First time I went to the sign up page, that was a regular input (inline label).
    <ion-item>
      <ion-label>Phone number</ion-label>
      <ion-input [textMask]="{mask: phone_number_mask}" [(ngModel)]="data.phone_number" type="tel"></ion-input>
    </ion-item>

The second time it was a floating input.
    <ion-item>
      <ion-label floating>Phone number</ion-label>
      <ion-input [textMask]="{mask: phone_number_mask}" [(ngModel)]="data.phone_number" type="tel"></ion-input>
    </ion-item>

I'm not sure I'm looking at the right thing here? It's an image?

Sorry, I meant to upload a gif, didn't realize it was a static png. Here you go:

Ah, ok. So, that guide is just a normal placeholder attribute on the input. If you set the placeholder of the input to be a whitespace, i.e. placeholder=" ", it would override Text Mask's default placeholder and this problem should go away.
So, for now we could try to get Text Mask to work with Ionic 2, then see if we can find a way to solve this placeholder problem for users.

Ok sure, I'll submit a clean PR that enhances the Angular 2 functionality to be compatible with Ionic 2.

FYI @msafi since you updated this repo to Angular RC5 I can no longer use the same directive that you have for Angular 2. Ionic 2 still uses RC4 as of now, so I will have to create a new directive that uses RC4.

Could you add some doc to explain how to use it ? I suppose is not exactly same as angular.

@movrack It should be exactly the same as Angular, just instead of input use ion-input.

The compatibility with Ionic 2 doesn't seem to work anymore (since the release or last RC).
If I try to use the textMask directive within an ion-input, I get the following error :

Cannot read property 'selectionStart' of undefined

But it does work perfectly if if put a regular input instead.
Am I missing something ?
If not, any workaround possible ? It would be a shame to not be able to use this great plugin with the native-looking inputs of Ionic.

not working in ionic v2, retunr this error:
@Component({
  templateUrl: 'saida.html',
  directives: [Directive]
})

log:
directives: [Directive] component  does not exist in type 'component'

@victorluissantos please check the docs regarding Angular 2 usage. You have to import the TextMaskModule into your app's module, you no longer need to inject the directive into your components.

import {TextMaskModule} from 'angular2-text-mask'
seems incorrect.
What is the correct import statement ?

@Mapalagama It is correct.
import { TextMaskModule } from 'angular2-text-mask';

@NgModule({
	declarations: [ProjectComponent],
	imports: [IonicModule, TextMaskModule],
	exports: [ProjectComponent]
})
export class ComponentsModule { }

@ihadeed
My actions for the mask to work ion-input
npm i angular2-text-mask
edit app.module.ts
import { TextMaskModule } from 'angular2-text-mask';
@NgModule({
imports: [
TextMaskModule
],
next i install your module
npm i angular2-text-mask --save
edite home.ts
import { Directive } from 'ionic2-input-mask';
export class HomePage {
public myModel = ''
public mask = ['(', /[1-9]/, /\d/, /\d/, ')', ' ', /\d/, /\d/, /\d/, '-', /\d/, /\d/, /\d/, /\d/]
}
and edit home.html


addComponents supports multiple arguments, requires does not

#.addComponent
https://github.com/craftyjs/Crafty/blob/b73ba75c27a214a3686bd453b2c76e2dc6bab61c/src/core/core.js#L293-L298
#.requires
https://github.com/craftyjs/Crafty/blob/b73ba75c27a214a3686bd453b2c76e2dc6bab61c/src/core/core.js#L426-L428
shouldn't it be this.addComponent.apply(this, arguments) ?

Seems reasonable to me.

Fixed in #1150


internal/fixed harvest.bat

Phases
MM
MB
MR
JM
Total

ETA
0.2
0
0
0
0

Developing
0
0
0
0
0

Review
0
0
0
0
0

Total
-
-
-
-
0

ETA est.

0

ETA cust.
-
-
-
-
0

Problem description
changed harvest.bat

@MajoBerger could we close this PR?

I don't know. I was never using harvesting... We could ignore it possibly.

Harvesting works, I'm closing it.


Adding Aiven Terraform provider v1 to v2 migration guide

Subj 👆

It looks like preview link is the description is not correct one. Here is a proper link - https://deploy-preview-343--developer-aiven-io-preview.netlify.app/docs/tools/terraform/v1v2.html

Linter complains about the following – /home/runner/work/devportal/devportal/docs/tools/cli/service.rst:2: WARNING: Duplicate explicit target name: "columns"..
Does it cause the failure? Because this article is coming from master and already published.

Don't worry, I can take a look when I review

I removed TF v1 section and added more explanation in case TF < 1 is used and if TF >= 1 is used. I hope it is clearer now.
Thanks.

I'll merge this one since it's been thru some rounds of review already and I would like to communicate this guide to a customer.
I've created an issue to track any further changes which needs to be done to this guide.
Ty.

Could you please wait for the review stage? I've spent a couple of hours on this today. Also if you have a deadline, that would have been useful to know a bit sooner.

Could you please wait for the review stage? I've spent a couple of hours on this today. Also if you have a deadline, that would have been useful to know a bit sooner.

Sure. I will wait then.
Ty.

Could you please both take another look? Particularly the last section has changed ... but also I switched up the order of the steps so that upgrading Terraform was one section, and upgrading the provider was separate (since some people might need to do only one of those) - hopefully this won't break anything and our old provider would work on new Terraform also! Let me know if I broke anything but I'm also adding my own approval so either of you can go ahead and merge.

Thanks a lot for improving the article. I double checked the flow and had to switch up few things but it is all good and works 💪


Give content-tagger an email-alert-api token

I think I'll need someone else to actually create the token and add it to the secrets(?) repo.

Trello card

I think I'll need someone else to actually create the token and add it to the secrets(?) repo.

Yep, let me know if you're stuck with this.


Is this repo beeing kept alive?

+1

Yes!  New package coming soon, thank you for everyone in the community that has helped keep this library alive for 9 years!


HBASE-24285 Move to hbase-thirdparty-3.3.0

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

_ Other Tests _

+1 :green_heart:
asflicense
0m 17s
The patch does not generate ASF License warnings.

19m 37s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1605

Optional Tests
dupname asflicense hadoopcheck xml

uname
Linux b45e24068e9b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 3781ac9dac

Max. process+thread count
76 (vs. ulimit of 12500)

modules
C: . U: .

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 26s
the patch passed

+1 :green_heart:
compile
3m  2s
the patch passed

+1 :green_heart:
javac
3m  2s
the patch passed

+1 :green_heart:
shadedjars
5m 39s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 16s
root in the patch failed.

_ Other Tests _

-1 :x:
unit
187m 35s
root in the patch failed.

217m 39s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1605

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 99d23b76b31d 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 3781ac9dac

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/testReport/

Max. process+thread count
3158 (vs. ulimit of 12500)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 53s
master passed

+1 :green_heart:
compile
2m 13s
master passed

+1 :green_heart:
shadedjars
4m 59s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m  1s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 21s
the patch passed

+1 :green_heart:
compile
2m 15s
the patch passed

+1 :green_heart:
javac
2m 15s
the patch passed

+1 :green_heart:
shadedjars
4m 53s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 55s
the patch passed

_ Other Tests _

-1 :x:
unit
274m 20s
root in the patch failed.

303m 10s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1605

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1510baf83011 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 3781ac9dac

Default Java
1.8.0_232

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/testReport/

Max. process+thread count
5531 (vs. ulimit of 12500)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

The backup failures are regular flakies (backup seems abandoned). Reran the build to see if failures at all related.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
Docker mode activated.

_ Other Tests _

+1 :green_heart:
asflicense
0m 18s
The patch does not generate ASF License warnings.

18m 40s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1605

Optional Tests
dupname asflicense hadoopcheck xml

uname
Linux f2014f85dc17 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 13f36d9db3

Max. process+thread count
76 (vs. ulimit of 12500)

modules
C: . U: .

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 42s
the patch passed

+1 :green_heart:
compile
3m  9s
the patch passed

+1 :green_heart:
javac
3m  9s
the patch passed

+1 :green_heart:
shadedjars
5m 58s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 23s
root in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
261m 39s
root in the patch passed.

293m 51s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1605

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 069505b46a88 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 13f36d9db3

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/2/testReport/

Max. process+thread count
4220 (vs. ulimit of 12500)

modules
C: . U: .

 U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1605/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

I think #1611 can then be closed, or?

Sure, I'll cherry-pick this and close #1611.


Add validator launchpad

It would be nice to have a way of providing a launchpad https://launchpad.ethereum.org/en/

not needed


make install with DESTDIR fails

I found that using make install DESTDIR=/tmp/ng to install built ngraph in a location of my choice leads to build failure. If I use make install, as it is suggested in INSTALL.md, then the build is fine. Details below.
 -DNGRAPH_USE_PREBUILT_LLVM=1
make -j4 install DESTDIR=/tmp/ng
--
Quick analysis suggests that DESTDIR conflicts with how mkldnn install is done.
Snippet from build log below telling that mkldnn is installed under DESTDIR, but build looks
for mkldnn.hpp under build/third_party/mkldnn/include/mkldnn.hpp.
-- Installing: /tmp/ng/home/nhasabni/intel/nervana/jai_ngraph/build/third-party/mkldnn/include/mkldnn.h
-- Installing: /tmp/ng/home/nhasabni/intel/nervana/jai_ngraph/build/third-party/mkldnn/include/mkldnn_types.h
-- Installing: /tmp/ng/home/nhasabni/intel/nervana/jai_ngraph/build/third-party/mkldnn/include/mkldnn.hpp

Full build log attached: ng.build.log

I have a similar issue when trying to make install using INSTALL_PREFIX :
cmake .. -DCMAKE_INSTALL_PREFIX=/tmp/ngraph -DNGRAPH_INSTALL_PREFIX=/tmp/ngraph
make install
...
Install the project...
-- Install configuration: "Release"
-- Installing: /usr/local/lib/libmklml_intel.so
CMake Error at cmake_install.cmake:36 (file):
file INSTALL cannot copy file
"/home/wtambellini/repos/ngraph/build/mkldnn/src/external/mkl/lib/libmklml_intel.so"
to "/usr/local/lib/libmklml_intel.so".
looks like mklml still tries to install into the default /usr/local I guess.

You should only need to use -DCMAKE_INSTALL_PREFIX=/tmp/ngraph. I tested this on Ubuntu 16.04
Install the project...
-- Install configuration: ""
-- Installing: /tmp/ngraph/lib
-- Installing: /tmp/ngraph/lib/libiomp5.so
-- Installing: /tmp/ngraph/lib/libmkldnn.so
-- Installing: /tmp/ngraph/lib/libmkldnn.so.0
-- Installing: /tmp/ngraph/lib/libmkldnn.so.0.14.0
-- Installing: /tmp/ngraph/lib/libmklml_intel.so
...


Dice animation to determine who goes first

Have speed stat which is your speed / your speed + opponent speed, which boosts dice roll value, whoever has higher roll value goes first

Hmm, let’s not do the dice animation, kind of goes against aesthetic


Homebrew isn't saving

Error Code
Error: Internal Server Error

require<.superagent</Request/<@https://homebrewery.naturalcrit.com/homebrew/bundle.js:76355:19
require<[9]</Emitter.prototype.emit@https://homebrewery.naturalcrit.com/homebrew/bundle.js:1966:20
require<.superagent</Request.prototype._end/xhr.onreadystatechange@https://homebrewery.naturalcrit.com/homebrew/bundle.js:76664:10

{
  "status": 500,
  "method": "PUT",
  "url": "/api/update/GNNdAK9ekiCI"
}

### Your description of what happened:

Homebrew isn't saving.

I'm assuming this is fixed by now, but the error was you had an old version of the Homebrewery site that is missing a recent update. If reloading the page didn't work, try Hard Reloading the page. You can do this by pressing CTRL + SHIFT + R at the same time while at the Homebrewery.  Alternatively, you can open up the Chrome Developer Console (Right-click on the page and choose "inspect"). This will add some options to the refresh button if you right-click it. Again, choose "Hard Reload".

That should force Chrome to check for an update to the Homebrewery. Let us know if that doesn't work.

Thank you for your help
On Mon, 23 Jan 2023, 21:17 Trevor Buckner, @.***> wrote:

Closed #2590 https://github.com/naturalcrit/homebrewery/issues/2590 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/naturalcrit/homebrewery/issues/2590#event-8338331975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5D7CA3NETNCYB4B5FPMME3WT3YPRANCNFSM6AAAAAATT4JT44
.
You are receiving this because you authored the thread.Message ID:
@.***>


Changing to a custom font doesn't work reliably

Windows Terminal version
1.21.2701.0
Windows build number
10.0.22631.4317
Other Software
No response
Steps to reproduce

Install custom font like "Fira Mono" or "IBM Plex Mono"
Try to configure it in Windows Terminal

Expected Behavior
I expect the font to be changed
Actual Behavior
Windows Temrinal claims in cannot find the font when pressing the save button. This also happens if I change the font directly in the JSON, as soon as the config file is saved Terminal says it cannot find the font.
Although, after I trigger something (I did not find out yet what it is, maybe it's just randomly, it suddenly works until i restart terminal again)

Here is a video demonstrating the issue: https://streamable.com/81sj2v

Verify Font Installation: Ensure the font is correctly installed on your system and accessible.
Clear Terminal Cache: Clear any cache or reset settings for the terminal.
Look for Updates: Check if there are any updates to Windows Terminal that might address this bug.

Verify Font Installation: Ensure the font is correctly installed on your system and accessible.

Other applications don't have a problem accessing them

Clear Terminal Cache: Clear any cache or reset settings for the terminal.

Where can I do that?

Look for Updates: Check if there are any updates to Windows Terminal that might address this bug.

It's already the newest one that winget has available. I also tried to downgrade without luck.

This issue was previously reported in #17987 and has subsequently been fixed. We're just waiting to release it now, hopefully soon! /dup #17987

This issue was previously reported in #17987 and has subsequently been fixed. We're just waiting to release it now, hopefully soon! /dup #17987

Are you sure? The Video looks very different from mine.

I apologize for the misunderstanding - we had multiple duplicates of the other issue come in over the last few days. I missed that you modified the settings.json file as well.

I have same issue with 1.22.240926002-preview

I have the same issue on windows 11 with 1.21.2701.0

I have the same issue , I need Nerd font

Also experiencing the same issue. Curiously the issue fixes itself if I simply open Settings i.e. the missing font error stops popping up and the font starts working in new Terminal tabs.


fix: @modern-js/runtime/model effects type error (close: #1002)

PR Details
参考https://github.com/modern-js-dev/modern.js/pull/62, 修复https://github.com/modern-js-dev/modern.js/issues/1002

Description
修复@modern-js/runtime/model 中effets类型推断错误的问题

Related Issue
https://github.com/modern-js-dev/modern.js/pull/62/files

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] I have updated changeset
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

 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.


Update version of multiapps plugin to 2.1.3

Thanks, @boyan-velinov!

Hi @boyan-velinov we are creating a new v7 plugin interface and are developing new methods which call the v3 API. We noticed that you use the AccessToken method and wonder what your use case is? If we didn't support it on v7, what would you use instead? Thanks!

Hi @abbyachau
Thanks for notifying us!
We use AccessToken from our MTA CF CLI plugin, as you noticed. Then it is passed to Multiapps Controller  backend. Multiapps Controller contains all of the business logic and operates on behalf of the user with that token. All consequent calls from Multiapps Controller to Cloud Controller are done with that token, e.g creating services, creating apps and so on.

Thank you @boyan-velinov  appreciate the feedback. We'll reach out if we have any follow-up questions, thanks again!


On start plugin creates div with class popper that break app layout

v.2.0.0-beta.2
Describe the bug
On start plugin creates div with class popper that break app layout.
Screenshot

OS
Windows
Obsidian Version - v1.0.3

Potentially related to https://github.com/liamcain/obsidian-calendar-plugin/issues/291 (Window shifts upwards when Calendar is displayed)

Here is how to fix it:
https://github.com/liamcain/obsidian-calendar-plugin/issues/270#issuecomment-1595584717


--removeProtractor Flag not working

What I did:

npm i -g @angular/cli@latest (=> 11.2.11)
ng new wdio-test (strict & routing yes, styling format scss)
ng add @wdio/schematics --yes --removeProtractor

Expected:

protractor dependency removed from package.json
e2e folder removed
e2e property in angular.json configured to not use protractor anymore

Actual:

protractor dependency still in package.json
e2e folder still existing
angular.json is untouched
ENOENT: no such file or directory, open '[...]\wdio-test\wdio.conf.js'
(probably its own issue, as a wdio.conf.ts is created)

same here


Error when formatting correlation data

When preparing data for new project with format_corr_data.pl script, got errors pertaining to lines 28 and 29:
"Experimental keys on scalar is now forbidden at ../TEA/import_project/format_corr_data.pl line 28."

Closed by PR #76


DRY on password management; move eng sec policy to eng sec page

:books: Documentation preview :books:: https://civicactions-handbook--1298.org.readthedocs.build/en/1298/

Password Policy is more cleanly separated from Password Management, and SSH and Server management policies have been moved into Engineering > Security and Compliance -- this latter page needs some Engineering attention.

https://guidebook.civicactions.com/en/latest/practice-areas/engineering/security-compliance/ needs work, and this PR just adds to the problem; the next step will be to create a branch/PR to clean up this page.

Overview of Security Policy section changes:

Password managers and two-factor authentication

unique text integrated into common-practices-tools/security/README.md -- duplicate text removed

Some password exceptions

removed (if you want this, consider a move to practice-areas/engineering/security-compliance.md)

Private keys

moved to practice-areas/engineering/security-compliance.md

Server and site security

moved to practice-areas/engineering/security-compliance.md

The removal of sections that are only of interest to engineers should not require that everyone re-read/re-sign acceptance of the policy.
This was basically a tightening and clean-up of duplicated text. Next step will be to go over the practice-areas/engineering/security-compliance.md doc, tighten it up and ensure it contains policies that are general to all our engineers and devops teams. Tia.


switch to use smart pointer in parser

fix issue #523.  The unnecessary null pointer checks mainly exist in include/parser. I replace the raw pointers with unique_ptr to avoid explicit delete operations in destructor.

Coverage decreased (-0.009%) to 72.914% when pulling a1a46f1516da80dc10388d69d0a72964aa2d86a0 on lixupeng:smart_pointer into dcac90d9d5db592cf50b3613c2022a7371afa482 on cmu-db:master.

Coverage decreased (-0.007%) to 72.916% when pulling 92a690bfc9c24b6b7c7c50edfcbb82332001776a on lixupeng:smart_pointer into dcac90d9d5db592cf50b3613c2022a7371afa482 on cmu-db:master.

Coverage decreased (-72.4%) to 0.0% when pulling d6e957cc1246dc1c0e0f3ec25e7bdd1ecfe560d3 on lixupeng:smart_pointer into 7faab8dc6938f16861be3031965e2ccaa03fc669 on cmu-db:master.


Add hexo-generator-json-content to plugins list

I created a plugin to generate a JSON file with contents of pages and posts and some site meta data.
I think it's useful for AJAX search or API content. So I added it to the site plugins list.

looks cool

I suppose this helps theme creators too.
Thanks.

Thanks, some updates coming soon!
I liked Hexo and hope to contribute more.


Update README.md

Hello,
Thank you for the comprehensive collection of SAM-related papers you've compiled. We have recently made our code available for our AAAI 2024 paper. You can find our project page at https://lwpyh.github.io/GenSAM/. We have also submitted a pull request to have our paper included in your collection. Could you please consider approving our request?
Thank you.

@lwpyh Thanks for your interesting work. I have merged.


dataFormat adds  when dataFormat is used

Ellipsis disappears when dataFormat is used and I noticed  is added to the text when using the dataFormat.  Is there way to remove this? or any other way to add ellipsis?
I'm just trying to show 'N/A' when description field is null, is there any other way to do this other then dataFormat?  Thanks!
formatDescription(cell, row) {
return row.description === null ? 'N/A' : row.description;
}

Description

For a quick workaround I added this CSS in my project:
.react-bs-table table td div {
  overflow: hidden;
  text-overflow: ellipsis;
}


If a public folder exists in the app, the build fails.

2015-03-30T13:43:22.91+0200 [STG/7]      OUT -----> Downloaded app package (2.0M)
2015-03-30T13:43:23.66+0200 [STG/0]      ERR Cloning into '/tmp/buildpacks/staticfile-buildpack'...
2015-03-30T13:43:25.89+0200 [STG/0]      ERR grep: Staticfile: No such file or directory
2015-03-30T13:43:25.89+0200 [STG/0]      OUT -----> Using root folder
2015-03-30T13:43:25.89+0200 [STG/0]      OUT -----> Copying project files into public/
2015-03-30T13:43:26.03+0200 [STG/0]      ERR mv: cannot move `/tmp/cache/public' to `./public': Directory not empty
2015-03-30T13:43:26.04+0200 [STG/0]      OUT Staging failed: Buildpack compilation step failed
2015-03-30T13:43:26.21+0200 [API/1]      ERR Encountered error: App staging failed in the buildpack compile phase
2015-03-30T13:49:

Which is a real bummer since all our apps have a public folder. -.-
Issue might be caused by https://github.com/cloudfoundry-incubator/staticfile-buildpack/commit/79584924099f957227311254d3d8b73355ec27e6

We need another test/fixtures example app that fails :/
Sorry
On Mon, Mar 30, 2015 at 7:52 AM, Max<EMAIL_ADDRESS>wrote:

2015-03-30T13:43:22.91+0200 [STG/7]      OUT -----> Downloaded app package (2.0M)
2015-03-30T13:43:23.66+0200 [STG/0]      ERR Cloning into '/tmp/buildpacks/staticfile-buildpack'...
2015-03-30T13:43:25.89+0200 [STG/0]      ERR grep: Staticfile: No such file or directory
2015-03-30T13:43:25.89+0200 [STG/0]      OUT -----> Using root folder
2015-03-30T13:43:25.89+0200 [STG/0]      OUT -----> Copying project files into public/
2015-03-30T13:43:26.03+0200 [STG/0]      ERR mv: cannot move `/tmp/cache/public' to `./public': Directory not empty
2015-03-30T13:43:26.04+0200 [STG/0]      OUT Staging failed: Buildpack compilation step failed
2015-03-30T13:43:26.21+0200 [API/1]      ERR Encountered error: App staging failed in the buildpack compile phase
2015-03-30T13:49:

Which is a real bummer since all our apps have a public folder. -.-
Issue might be caused by https://github.com/cloudfoundry-incubator/staticfile-buildpack/commit/79584924099f957227311254d3d8b73355ec27e6
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry-incubator/staticfile-buildpack/issues/28

I mean, sorry for the bug after merging the PRs, another test app might have spotted this issue.
But if you have a moment, can you create a failing test app?
You can run through all the test apps using tests/test.sh I think from memory
On Mon, Mar 30, 2015 at 7:52 AM, Max<EMAIL_ADDRESS>wrote:

2015-03-30T13:43:22.91+0200 [STG/7]      OUT -----> Downloaded app package (2.0M)
2015-03-30T13:43:23.66+0200 [STG/0]      ERR Cloning into '/tmp/buildpacks/staticfile-buildpack'...
2015-03-30T13:43:25.89+0200 [STG/0]      ERR grep: Staticfile: No such file or directory
2015-03-30T13:43:25.89+0200 [STG/0]      OUT -----> Using root folder
2015-03-30T13:43:25.89+0200 [STG/0]      OUT -----> Copying project files into public/
2015-03-30T13:43:26.03+0200 [STG/0]      ERR mv: cannot move `/tmp/cache/public' to `./public': Directory not empty
2015-03-30T13:43:26.04+0200 [STG/0]      OUT Staging failed: Buildpack compilation step failed
2015-03-30T13:43:26.21+0200 [API/1]      ERR Encountered error: App staging failed in the buildpack compile phase
2015-03-30T13:49:

Which is a real bummer since all our apps have a public folder. -.-
Issue might be caused by https://github.com/cloudfoundry-incubator/staticfile-buildpack/commit/79584924099f957227311254d3d8b73355ec27e6
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry-incubator/staticfile-buildpack/issues/28

Ohm, on any of your test apps, just add a folder named public to the root and it will fail. I'm not sure how to provide you with a test app.

Sorry Max I'm suggesting it to you/anyone who wants to investigate a solution before I get to it one day.
On Mon, Mar 30, 2015 at 8:53 AM, Max<EMAIL_ADDRESS>wrote:

Ohm, on any of your test apps, just add a folder named public to the root and it will fail. I'm not sure how to provide you with a test app.
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry-incubator/staticfile-buildpack/issues/28#issuecomment-87732610

Solution in progress https://github.com/cloudfoundry-incubator/staticfile-buildpack/pull/29 but I'm not thrilled about it.

Released in v0.5.1 https://github.com/cloudfoundry-incubator/staticfile-buildpack/releases/tag/v0.5.1

Thank you for your quick response and fix.  Unfortunately we had not much time to investigate, the things we tried did't work out better,  so I cannot provide a better solution.
Quite a breaking change for users with a custom nginx.
Needs to be declared in the changelog, if not already there.

The final fix doesn't require a breaking change. V0.5.1 should work for everyone and you :)
On Wed, Apr 1, 2015 at 2:34 PM, Max<EMAIL_ADDRESS>wrote:

Thank you for your quick response and fix.  Unfortunately we had not much time to investigate, the things we tried did't work out better,  so I cannot provide a better solution.
Quite a breaking change for users with a custom nginx.
Needs to be declared in the changelog, if not already there.
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry-incubator/staticfile-buildpack/issues/28#issuecomment-88637556


Fetch about page images properly

Apparently I really didn't test this very well...
... turns out I was looking at the built/assets folder instead of the client/public folder
... and this wasn't working at all
:sob:

This was sad.


rhythmbox not working with xfce

Issue description
rhythmbox spits out a lot of warnings and then segfaults after a few seconds.
Steps to reproduce

nix-env -iA nixos.rhythmbox
enable 'start gnome runtime environment' in xfce sessionsettings>advanced
nix-env -iA nixos.gnome3.gnome_settings_daemon
because i thought it would remove most warnings and think that this belongs to the runtime environment of gnome, what the xfce setting would require. i have no idea if xfce found it, though. (where can i find xfce errors and warnings?)
start rhythmbox
it spits out:

$ rhythmbox

** (rhythmbox:5321): WARNING **: Error retrieving accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown: The name org.a11y.Bus was not provided by any .service files

(rhythmbox:5321): Gtk-WARNING **: Failed to register client: GDBus.Error:org.freedesktop.DBus.Error.UnknownMethod: Method "RegisterClient" with signature "ss" on interface "org.xfce.Session.Manager" doesn't exist

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

(rhythmbox:5321): Rhythmbox-WARNING **: Unable to grab media player keys: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.gnome.SettingsDaemon was not provided by any .service files

(rhythmbox:5321): Rhythmbox-WARNING **: Unable to release media player keys: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.gnome.SettingsDaemon was not provided by any .service files

(rhythmbox:5321): Gtk-WARNING **: Drawing a gadget with negative dimensions. Did you forget to allocate a size? (node grid owner RBHeader)
Segmentation fault

Side questions
if this problem stems from missing packages, shouldn't the rhythmbox nix expression contain the dependency?
amarok seems to have a similar problem: #165588. -> Am I (and others) experiencing edge cases or is nixos not that stable yet?
since we have a declarative system, is it preferred to include the system and user manifest in a bug report?
Technical details

System: NixOS: `17.03.1931.8b8977f9ff (Gorilla)
Nix version: nix-env (Nix) 1.11.14
Nixpkgs version: "17.03.1931.8b8977f9ff"
Sandboxing enabled: build-use-sandbox = false

try install nix-env -i dconf

this solved it. dconf should be added is dependency!

Same happens with plank, it doesn't crash, but I can't change any settings:
 plank
[WARN 19:27:49.048173] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.071265] [Preferences:192] '/nix/store/5j6v0x75v1xf2jwdj87asib17vwhawh5-plank-0.11.4/share/plank/themes/Default/dock.theme' is read-only!
[WARN 19:27:49.080071] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.102659] Failed to get running applications: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.ayatana.bamf was not provided by any .service files
[WARN 19:27:49.109905] (Services/Matcher.c:296):plank_matcher_active_launchers: runtime check failed: (_tmp3_ != NULL)
[WARN 19:27:49.112197] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.135351] Failed to register favorites: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.ayatana.bamf was not provided by any .service files
[CRITICAL 19:27:49.141041] [DockWindow:297] Retry #1 update_size_and_position() to force requested values!
[WARN 19:27:49.144278] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.176453] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.207841] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.239478] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.270318] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.301985] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.332796] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.364520] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.395908] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.425076] Failed to get running applications: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.ayatana.bamf was not provided by any .service files
[WARN 19:27:49.428007] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.432232] (Services/Matcher.c:296):plank_matcher_active_launchers: runtime check failed: (_tmp3_ != NULL)
[WARN 19:27:49.459436] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.466961] [Preferences:192] '/nix/store/5j6v0x75v1xf2jwdj87asib17vwhawh5-plank-0.11.4/share/plank/themes/Default/dock.theme' is read-only!
[WARN 19:27:49.491344] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.523334] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.555442] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.586505] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.618447] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files
[WARN 19:27:49.650067] [dconf] failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

Installing gnome3.dconf fixed it for me!

gnome3.dconf is a system service that needs to be added to services.dbus.packages by your desktop environment. If you create your own environment, you will need to add it yourself (adding programs.dconf.enable = true; to configuration.nix should work). Nix does not contain a mechanism for representing a dependency on a service.


fix: fix data commitment ranges overlap

Overview

Fixes https://github.com/celestiaorg/celestia-app/issues/1204
Now, the ranges will be as follows, with a data commitment window of 400:

0-399
400-799
800-1199
...

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Codecov Report

Merging #1217 (ee62787) into main (f4f6a3a) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1217      +/-   ##
==========================================
- Coverage   48.08%   48.03%   -0.06%
==========================================
  Files          72       72
  Lines        4082     4087       +5
==========================================
  Hits         1963     1963
- Misses       1947     1952       +5
  Partials      172      172

Impacted Files
Coverage Δ

app/process_proposal.go
0.00% <0.00%> (ø)

x/blob/types/blob_tx.go
18.57% <0.00%> (+1.90%)
:arrow_up:

x/qgb/keeper/keeper_data_commitment.go
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

also, do we need the qgb-integration PR as well as this one? or can we just use this one?

also, do we need the qgb-integration PR as well as this one? or can we just use this one?

After today's sync, we can close the other PR, will do so shortly


Installation fails on ubuntu 14.

Hi there
I've hit a bug whilst attempting to install pivpn.
I'm using the https://app.vagrantup.com/bento/boxes/ubuntu-14.04 as a base image so if you use vagrant you should be able to replicate, here is my vagrantfile
Vagrant.configure("2") do |config|
  config.vm.box = "bento/ubuntu-14.04"
  config.vm.provider "vmware_workstation" do |v|
    v.gui = true
  end
  config.vm.box_check_update = false
end

I've started the vm, ran a apt update & apt upgrade and then when I go
::: Your system is up to date! Continuing with PiVPN installation...
Can't call method "set" on an undefined value at /usr/share/perl5/Debconf/FrontEnd.pm line 126,  to call the script this happens
root@vagrant:~# curl -L https://install.pivpn.io | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   178  100   178    0     0    302      0 --:--:-- --:--:-- --:--:--   302
100 53505  100 53505    0     0  71569      0 --:--:-- --:--:-- --:--:-- 71569
:::
::: You are root.
::: Verifying free disk space...
:::
::: Checking apt-get for upgraded packages.... done!
:::
::: Your system is up to date! Continuing with PiVPN installation...
Can't call method "set" on an undefined value at /usr/share/perl5/Debconf/FrontEnd.pm line 126, <GEN0> line 5.```

I could try a fresh install using the iso if you think it might be the box, but I use them for a lot of other tasks.

Ok .. Before I test this and check this .. why do you want Ubuntu 14?
Any special requirement?
PiVPN should work on ubuntu 14 ... however... I consider this to be kind of a waste of time since its already quite old release =/
however don't take me wrong... if you really need that ... we can try to work it out!! Just be aware that I don't have much time to help you out.

Hi @cfcolaco Cheers for coming back to me
I'm not tied to that version so don't want to waste your time, I think I only choose that because that's what it said on the homepage
FLEXIBLE ::: ...the installer will work just as well on an Ubuntu Server running Trusty Tahr 14.04
Let me quickly stand up a v16 box and try that.

OK here is the result from 16.04
root@vagrant:~# uname -a
Linux vagrant 4.4.0-87-generic #110-Ubuntu SMP Tue Jul 18 12:55:35 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

root@vagrant:~# cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.3 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.3 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

root@vagrant:~# curl -L https://install.pivpn.io | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   178  100   178    0     0    201      0 --:--:-- --:--:-- --:--:--   201
100 53505  100 53505    0     0  42691      0  0:00:01  0:00:01 --:--:--  156k
:::
::: You are root.
::: Verifying free disk space...
:::
::: Checking apt-get for upgraded packages.... done!
:::
::: Your system is up to date! Continuing with PiVPN installation...
Can't call method "set" on an undefined value at /usr/share/perl5/Debconf/FrontEnd.pm line 126, <GEN0> line 5.

Hmm!! odd thing ... I tested it months ago in ubuntu 16.04 ... well .. will take a look at it when as soon as I have the time for it ...
can you try run apt-get update && apt-get upgrade before running PiVPN?

Hi @cfcolaco
I tried running the update/upgrade but ended up in the same place.
I took a punt at lunchtime to see if I can see what's going on.  I think  'debconf-apt-progress' is broken in ubuntu at the moment as the error looks to be when installing the dependenacy's as I get the same perl error when running the command manually.
I've amended that function to not use that when installing on ubuntu and that worked for me.
I'll send over a pr for it, no problem if you don't use it, all good practice.
Gary

cool @username-is-already-taken2 thank you for the help! feel free to make the pull request ...  unfortunately, right now I cannot try to replicate this =/

@username-is-already-taken2 I have been looking into the packages ... the debconf-apt-progress package is part of the debconf package ....are you sure that you are not missing any deps from debconf??
how deep went you to check the origin of the issue apart from just tell pivpn to not use the "broken package" ?? right now I don't have tests machine to try and run this and check if it is an Ubuntu issue .. or an "your ubuntu" issue .... are there another issues related to this one around?

I'm using the vagrant ubuntu box's being generated as part of the bento  project so they are basic scripted installs.
I have virtual box on my laptop, I'll create a new vm and do a minimal server install using the 16.04 iso from ubuntu's website.  Just to rule out if its something inside that bento build.  The bento builds are open source and you can see each step but you never know at least that will rull it out

@cfcolaco - I'm very sorry, it must be clearly something in the build,  I've just completed a fresh install and it worked first time.
I'm really sorry to have troubled you with this.  I'll see if I can workout whats missing and feed that back to them

@username-is-already-taken2 that's cool!! no worries about that!! thank you a lot for your time!!! if you would like to keep helping out and contributing ... we have plenty of work to do around here! =p

closing issue since it is not pivpn related.
@username-is-already-taken2 , feel free to reopen if you find that you need something from our side!
Thank you for spending time with the project!


REST api bad pratices

Please:

use nouns instead of action
use kebab-case for urls
plural name for ressources
/resources/:resourceId/subresources => /collections/:id/items

Hey, thanks for you suggestion!
I have a couple of Qs

use kebab-case for urls

Will do!

plural name for ressources

so rename itemList to items ? My brain process was avoiding s in url because people can make typos easily.

use nouns instead of action

ELI5 pls? 🥺

so rename itemList to items ? My brain process was avoiding s in url because people can make typos easily.

Models are represented in singular
Ressource are always in plurals

use nouns instead of action

ELI5 pls? 🥺

Wow, I wouldn't hand over the task of writing an Api to a 5 year old 😅
I was referring to the REST part that should be Stateless:

collection/:id => /collections/:id
collectionByIssuer/:issuer => /collections?issuer=:issuer
collectionByOwner/:owner =>  /collections?owner=:owner
eventByAddress/:address =>  /events?address=:address
eventByInteraction/:interaction => /events?interaction=:interaction or => /items/:id/events?interaction=:interaction
eventByNftId/:id => /items/:id/events
item/:id => /items/:id
itemByCollection/:id => /collections/:id/items
itemByCollectionList/:ids => ?
itemByIssuer/:issuer => /items?issuer=:issuer
itemByCid/:id => /items?cid=:cid
itemByOwner/:owner => /items?owner=:owner
itemCollectedBy/:address => /items?owner=:address
itemSoldBy/:address => ?

it should be human readable & easy to extend.
Bonus:
It would be top if you could create a Swagger doc for this api specs

I have just a couple of problems:

I currently cant parse queries like /collections/:id/items
when Is it as query param I can't validate if fields are correct
I need to rethink how the REST magic is built (now it's build on the top of client query.

I would like to help you but I don't have enough knowledge on TS backend (only basic express experience)
Like how do you handle different endpoints query? are you wrapping gql with a rest api?

Other option I have in mind it to fork
https://github.com/unjs/ungh
and write proper api 😅

Code looks simpler & maintenable. I agree with that.

Like how do you handle different endpoints query? are you wrapping gql with a rest api?

basically what rest this version does it that expect path like /{chain}/{call}/{id}
Then I check the call against the map I have defined (let's assign the result to fn)
Last thing I do is make const query = fn(id)
Find endpoint based on the chain and wrap it together

Code looks simpler & maintenable. I agree with that.

Decided to deprecate this one for the sake of #29

Decided to deprecate this one for the sake of #29

wdym? new api version? new routes? nitro stack?

wdym? new api version? new routes? nitro stack?

Like plan to remove the ask function for uniquery and would rather develop hosted version (nitro/hono) based on the client implementation.


Ask Dependabot to bump actions

I noticed some of our GitHub action workflows are throwing deprecation warnings...
Let's have Dependbot open PR's to bump them whenever they're outdated.
We could also enable it for watching our Python deps, but that may be more controversial, so that can be done as a follow-on PR if there's interest.

Codecov Report
Base: 94.67% // Head: 94.33% // Decreases project coverage by -0.34% :warning:

Coverage data is based on head (abb0a81) compared to base (0c56441).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #693      +/-   ##
==========================================
- Coverage   94.67%   94.33%   -0.34%
==========================================
  Files          57       57
  Lines        8339     8339
==========================================
- Hits         7895     7867      -28
- Misses        444      472      +28

Impacted Files
Coverage Δ

kazoo/tests/test_cache.py
55.10% <0.00%> (-4.09%)
:arrow_down:

kazoo/tests/test_eventlet_handler.py
87.43% <0.00%> (-2.19%)
:arrow_down:

kazoo/protocol/connection.py
95.45% <0.00%> (-1.45%)
:arrow_down:

kazoo/testing/harness.py
97.52% <0.00%> (-0.83%)
:arrow_down:

kazoo/tests/test_gevent_handler.py
78.87% <0.00%> (-0.71%)
:arrow_down:

kazoo/handlers/utils.py
93.60% <0.00%> (-0.46%)
:arrow_down:

kazoo/client.py
97.92% <0.00%> (-0.16%)
:arrow_down:

kazoo/tests/test_client.py
98.70% <0.00%> (-0.10%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Request for details about sfm_shape_3448.bin

Thanks alot for providing the library!! Could you please give some information about how this model was prepared. How many face scan does it has ?
Or some link which describes this Model.
That would be a great help.

Hi!
We just published a paper with all the details in it:
http://www.patrikhuber.ch/files/3DMM_Framework_VISAPP_2016.pdf

Thanks alot !!


Split WebClient into blocking / async versions?

An idea came to mind (inspired by one @adriancole made about async vs not being global decisions rather than call-site ones), would it make sense to split WebClient into these three versions?

BlockingWebClient, always returns AggregatedHttpResponse
AsyncWebClient, always returns CompletableFuture<AggregatedHttpResponse>. ClientOptions to specify whether to pool or not, and whether to use the newly added ClientRequestContext.root().contextAwareExecutor() when aggregating.
Streaming WebClient, always returns HttpResponse

I guess users either always use blocking, or always use async, so it can be annoying to always have to call e.g., .aggregate().join(). And the above naming happens to be backwards compatible :) It's similar to gRPC's client stubs.
I think a similar concept could be applied to decorators too, but I guess it could be harder to reason about. A clean API doesn't come to mind for them yet.

Sounds good to me all in all.
AsyncWebClient doesn't sound nice to me though.. AggregatingWebClient? :thinking:

These new clients could be created via two ways - via WebClient or Clients:
WebClient client = ...;
BlockingWebClient blockingClient = client.withBlocking();
AsyncWebClient asyncClient = client.withAggregation();

// or
BlockingWebClient blockingClient = Clients.newClient(...., BlockingWebClient.class);

BlocingWebClient has been added in #4021.

RestClient added in #4263 fully supports the requirements of AsyncClient.


SYCL CI: Manually build oneDPL

Alternative to #1118. Uses the same approach as in https://github.com/kokkos/kokkos/pull/7112 to avoid errors such as
--2024-06-28 16:01:41--  (try: 2)  https://registrationcenter-download.intel.com/akdlm/irc_nas/19133/l_oneDPL_p_2<IP_ADDRESS>35.sh

Connecting to registrationcenter-download.intel.com (registrationcenter-download.intel.com)|<IP_ADDRESS>|:443...
connected.

HTTP request sent, awaiting response...
No data received.

when building the SYCL image used in the CI.

The warnings are spurious.


Player detector infinite range

Descripe
The config option does not work, getPlayersInRange ignores this setting and is limitless
https://cdn.discordapp.com/attachments/779400553813377074/882808581085347850/unknown.png
CC Script
https://pastebin.com/raw/7hZBx8RD
Steps to reproduce
Load up the provided CC script and observe how it bypasses the config settings
Multiplayer?
Yes
Version
0.7.1r (Latest)
Minecraft, Forge and maybe other related mods versions
Forge 36.2.2 Minecraft 1.16.5
Screenshots or Videos
No response
Crashlog/log
No response

Fixed in 0.7.7r


zform validation: Migrate validation logic to zod.

This PR follows the work done by @xoldyckk in PR #25109, thanks to him it was near completion I just rebased the PR with some minor changes.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @Lalit3716 and @xoldyckk!


Feature/Bug MCA Reborn

Describe your idea in depth.
Try adding MCA reborn [Feature], i tried used with MCA Reborn but it causes a crash, without showing the crash message. [bug]

I haven't tested it yet, but we use the mod Numismatic Overhaul, maybe this mod causes the crash with MCA Reborn

Oh, ok, I will check.

MCA Reborn seems to be incompatible with Numismatic Overhaul.
Thx for reporting.


Login using SSO credentials during deploy

This is more of a question / feature suggestion. Our tenant is behind Okta + SSO. Is there a way to use pnp/action-cli-login using SSO credentials? Thanks.

Hey @michaelsoriano 👋
I've moved your issue to the main repo as your ask is more related to updating the functionality of the m365 login command which we use in the action-cli-login command to support your scenario.
Just to confirm, you would like to login to your Microsoft 365 tenant using a non-interactive flow that uses SSO?

Closing due to lack of reply

Hey @michaelsoriano 👋
I've moved your issue to the main repo as your ask is more related to updating the functionality of the m365 login command which we use in the action-cli-login command to support your scenario.
Just to confirm, you would like to login to your Microsoft 365 tenant using a non-interactive flow that uses SSO?

Sorry for the late response. Yes - non-interactive flow would be awesome.


deepspeed on gpus

Describe the bug
Importing deepspeed on gpus (tested on juwels) for FLOP profiling leads to "deepspeed.ops.op_builder.builder.MissingCUDAException: CUDA_HOME does not exist, unable to compile CUDA op(s)". Are we enforcing that deepspeed uses cpu accelerator?
@timurcarstensen could you look into this and try to reproduce this issue on juwels? Ideally we should not depend on CUDA_HOME environment variable.

Did you also test this on META?

Nope, only juwels, I think CUDA_HOME is a flag on meta, so should likely be fine there. I am wondering if we can prevent deepspeed from accessing this flag (which is triggered when using a gpu)?


Minor update

@tsholmes Minor fix here -- Twitter's documentation we received had an error in it where they said to pass in the order_id as a string wrapped in an array. Their rep corrected this to me via email and said it should just be a string.
Also removed toString() after toFixed() :)

Current coverage is 100% (diff: 100%)

Merging #10 into master will not change coverage

@@           master   #10   diff @@
===================================
  Files           1     1
  Lines          98    98
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits           98    98
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update c4950e8...23229c2


[BUG] agent status should be more verbose by default

Agent Environment
Agent 7.38.2 - Commit: ba442fd - Serialization version: v5.0.23 - Go version: go1.17.11
Describe what happened:
agent status command print is insufficient when global logLevel is WARN or lower (less verbosity)
Describe what you expected:
agent status or any other command used for troubleshooting agent/check issues, should ignore global logLevel configuration and print by default enough information to help to troubleshoot.
We reduced decrease recently the default agent verbosity from default INFO to WARN, however, I was not expecting that when triggering the command to troubleshoot agent status, this will be taken into consideration. In addition, agent status --help doesn't show any possibility to increase the verbosity (e.g.: -vvv).
Steps to reproduce the issue:

Configure a logLevel to WARN or lower (less verbosity)
You can check current global logLevel with:

root@datadog-ldbsx:/# env | grep DD_LOG_LEVEL
DD_LOG_LEVEL=WARN

Run agent status

root@datadog-ldbsx:/# agent status
2023-04-28 07:56:35 UTC | CORE | WARN | (pkg/util/log/log.go:591 in func1) | Deactivating Autoconfig will disable most components. It's recommended to use autoconfig_exclude_features and autoconfig_include_features to activate/deactivate features selectively
2023-04-28 07:56:35 UTC | CORE | WARN | (pkg/secrets/secrets.go:50 in Init) | Agent configuration relax permissions constraint on the secret backend cmd, Group can read and exec

Only WARN level messages are printed.

Run DD_LOG_LEVEL=INFO agent status
It prints:

root@datadog-ldbsx:/# DD_LOG_LEVEL=INFO agent status | head -15
2023-04-28 07:56:53 UTC | CORE | WARN | (pkg/util/log/log.go:591 in func1) | Deactivating Autoconfig will disable most components. It's recommended to use autoconfig_exclude_features and autoconfig_include_features to activate/deactivate features selectively
2023-04-28 07:56:53 UTC | CORE | INFO | (cmd/system-probe/config/config.go:118 in Merge) | no config exists at /etc/datadog-agent/system-probe.yaml, ignoring...
2023-04-28 07:56:53 UTC | CORE | WARN | (pkg/secrets/secrets.go:50 in Init) | Agent configuration relax permissions constraint on the secret backend cmd, Group can read and exec
2023-04-28 07:56:53 UTC | CORE | INFO | (cmd/agent/app/status.go:125 in requestStatus) | Getting the status from the agent.
2023-04-28 07:56:55 UTC | CORE | INFO | (cmd/agent/app/status.go:163 in requestStatus) |
===============
Agent (v7.38.2)
===============

  Status date: 2023-04-28 07:56:53.355 UTC<PHONE_NUMBER>355)
  Agent start: 2023-04-26 10:48:04.21 UTC<PHONE_NUMBER>210)
  Pid: 96643
  Go Version: go1.17.11
  Python Version: 3.8.13
  Build arch: amd64

Additional environment details (Operating System, Cloud provider, etc):
Confirmed with GKE/Kubernetes Integration, probably happen with the other platforms.

Hi @dlouvier,
We discussed this within the team and as anticipated in my earlier comment we are generally leaning towards having a separately hard-coded level for agent status log output (WARN) with a configurable argument --log-level to decouple the main config from the CLI while still retaining consistency in expected UX.
We'll keep this ticket open and update it as we work on this.

Hello @dlouvier, I think we initially misunderstood your comment, the issue you report is that when setting DD_LOG_LEVEL to warn, the status isn't displayed anymore.
This is due to the status being displayed as a log, it was the case specifically in version 7.38 and it was fixed in 7.39.0 (see https://github.com/DataDog/datadog-agent/pull/13117), so this issue will be solved by updating to any more recent version of the datadog-agent.
Feel free to open the issue again if you have further comments.


Website Standards: Gatsby update needed

Mike Abbink's update request:

Hi Shix, Who can update the Web Standards site to the latest Gatsby theme which should bring in the latest switcher too. The current one is months old.

@shixiedesign created an upgrade bot to auto create PRs with gatsby upgrades when available. First successful PR generated by the bot: https://github.ibm.com/webstandards/standards-website/pull/18


Add example that uses finch-oauth2

All we need to do is to implement DataHandler (e.g., using in-memory hash map) and provide two endpoints: the one that does auth and the one that issues tokens.
As an example: here is the handler I implemented for one of the first Finch applications using both hash-maps and Postgres database.

As @ImLiar noticed, there is also a bug in the current implementation when JSON decoders are picked up instead of a simple ones (see on Gitter). I wonder if we should distinct decoders used to parse params, headers  from JSON ones.

I'd like to work on this.

Thanks @ilya-murzinov!


Add Py37 to tox?

.travis.yml runs the tests on Python3.7 - is there any good reason not to add py37 to tox.ini too? Seems to work locally on my Core i7 Ubuntu PC...

I don't see why not. @waveform80?


[Enhancement] Add functionality to legal-hold command

Summary
The Code42 console does not offer sufficient options for querying legal hold. Technical Services utilizes and maintains a script that allows customer to report on legal hold objects and memberships. A number of these reporting options are already available through the legal-hold cli command; however, some are not.
Paul Hirst, the developer of the script, and I identified the current gaps and separated them into two phases.
Phase I:

Devices on Legal Hold
Legal Hold Storage by Organization

Phase II:

Transactions leveraging LegalHoldEvents endpoint (not currently on roadmap for Audit Log functionality)
Allow show to show details for all matters, not just one specified matter

Proposed API
code42cli legal-hold show <matter_id> —-include-devices
—include-devices  View all devices associated with legal hold custodians

code42cli legal-hold show <matter_id> —-include-org-storage
—include-org-storage   View total legal hold storage by organization

code42cli legal-hold events —-begin ‘2020-01-01 00:00’ —end ‘2020-06-01 00:00'
Usage: Fetch legal hold events
--begin  The beginning date range in which to look for events
--end    The end of the date range in which to look for events

Intended Use Case
Environment reporting

Phase I merged in with PR#192  |  Looking to start the conversation on Phase II
LegalHoldEvents
This information is visible in console through the Legal Hold Timeline, but it is neither queryable or exportable. Because this is already visible in console, this has been deprioritized for Audit Log.
The prosed addition is such:
code42cli legal-hold events —-begin ‘2020-01-01 00:00’ —end ‘2020-06-01 00:00'
Usage: Fetch legal hold events
-b, --begin  The beginning date range in which to look for events
-e, --end    The end of the date range in which to look for events
-t, --type   [HoldCreated, HoldDeactivated, HoldReactivated, MembershipCreated, MembershipDeactivated, MembershipReactivated, Restore]
         Limit events to those with the given event type

Show details for all matters
This was discussed in PR#192 - the show command is intended to show details on one object while list show the details about many objects. We have added functionality to show all devices on legal hold in the devices command, but there still remains a gap where a customer cannot report on all users on legal hold. Do you have a suggestion for where this would fit?

Ok. The LegalHoldEvent api requires a very particular ISO date format (e.g. 2021-02-19T10:00:00.000-05:00). Is there any existing date handling in py42 / cli that takes user input to match specific formats?
As for the users, customers care about username, first name, last name.

In Py42 user can specify date in string format yyyy-MM-dd, as a date object or even as an epoch timestamp. There are conversion functions in util module.

PR #264 address the events portion of Phase II.
The remaining item is: Allow show to show details for all matters, not just one specified matter
The details that we wish to show are the user memberships. Issue #259 will add a users command, which may be a reasonable place for legal hold membership reporting (similar to running devices list --include-legal-hold-membership).
If others agree, then we can close this Issue.


Fix inverted bool value for pragma case_sensitive_like

Summary
I think the instruction for enabling case-sensitive LIKE uses the wrong value in the current docs.

You can easily make the LIKE operator case-sensitive by using the following pragma statement:

Current:
PRAGMA case_sensitive_like = 0

Should be:
PRAGMA case_sensitive_like = 1

https://sqlite.org/pragma.html#pragma_case_sensitive_like

Oops. 😏 Off by one error?

Likely particles from cosmic rays 😉 Cheers!


Not seeing any issues on Firefox/Ubuntu:

I should’ve mentioned that this is in Firefox and Safari, but not in Chrome. I’m also seeing it in @quincylvania’s screenshots: compare #798 and #817.
You seem to be getting the Overpass font, which isn’t a default font on any operating system. Until #768 lands, we should be concerned about any regression when using default fonts, even if on some systems it has never been quite right and on others it’s always been fine.

This does not seem to be a problem in Chrome/Firefox on #768. I'm thinking we can close this as resolved with that PR.


Use Alpine Linux as base image (closes #1)

This reduces the resulting image size to only ~29MB (from ~150MB).

This has now been merged into the new "pipewire" image:
https://github.com/hifiberry/extension_spotifyd/commit/1d6713cd99e3c29e26deecbc77b250f85bb8296a


Target mongo:2.6

On a side note, 2.8 doesn't actually exist anymore. "MongoDB changed from 2.8.x and called it 3.0.x instead"

Haha, whoops. Shows what I know!

It probably happened in between picking the latest image and now. :wink:


searching documentation in getakka.net in google gives blank page with error message

Hi,
for example if you searching for HOCON configuration you get a result in google  the following url:
getakka.net/docs/concepts/configuration
when accessing the url you getting a blank page with this message:
"the resource you are looking for has been removed, had its name changed, or is temporarily unavailable"
I suppose you changed the documentation site but  google's crawlers still finding it.
this is the screenshot from the search result:

We just updated the entire website last night: http://getakka.net/ - some of these URLs have been moved

thank you for your fast reply.
Are you plan to update the landing page message with some link to the new URLs or update the crawlers?
very hard to search this way for documentation.


SQOOP-3423 Let user pass password to connect Hive when it set to LDAP…

… authentication

LGTM, ship it!


Expose TestConsole in artifact

Here's a first draft of the idea - give the ability to create a console for testing capabilities. Maybe it belongs to a distinct module, I don't know, but I imagine it's worth seeing what we can do with it, so here's an initial PR :)

I'm :+1: with it. If it was a bigger project I'd push for having TestConsole in a different module but since it's a very small one I think it's fine to have it in core.

Rebased, also made the input action customizable - let me know when you have a moment if this looks like the right way to go

I made some adjustments, think it's good now :D

I sent you an invite, once accepted you'll be able to merge PRs :)

Nice, thanks! I'll see about the formatting thing.


💡[Need Feedback] - Native Typescript Support Strategy in core library

Describe the Feature Request
As user of react-grid-layout:

I would like types to be colocated close to the actual source code

That is, have the types from react-grid-layout defined in this same repo as opposed to installing a 3rd party dependency @types/react-grid-layout
In the long run, this reduces operational burden from maintainers and members of the community who would like to improve the types. Contributing to or enhancing the types will no longer require forking and maintaining 2 packages, waiting for 1 to get merged, having things be out of sync etc. The easier the process is, the easier it is to contribute; this would help minimize friction points for contributing

As a potential contributor of react-grid-layout codebase:

having the types colocated with the code will allow me more quickly familiarize myself with the codebase
i no longer need to refer to 2 repositories @types/react-grid-layout for verifying types etc
over the long run:

reduces bugs due to type safety
increases productivity & saves us time since IDE/editor's have more information available

reduces friction points for contributing

Proposed Initial Implementation Strategy
The main goal is to have the code working as before with very minimal friction

consumers of the library will not notice any new changes & this will not require changes to @types/react-grid-layout
the code will be compiled in loose mode, expect to see lots of any's in complex areas of the codebase

0. create a new branch for typescript.
1. Rename .js files to .ts / tsx
This script below demonstrates how relatively straightforward converting all JS files to typescript can be. This will handle renaming the file with git mv so git doesn't track this file as a deletion or entirely new file.
# rename all JSX files in src/ to TSX (reactjs example)
find src -name '*.jsx' -exec bash -c 'git mv "$0" "${0%.jsx}.tsx"' "{}" \;

# rename all JS files in src/ to TS
find src -name '*.js' -exec bash -c 'git mv "$0" "${0%.js}.ts"' "{}" \;

# rename all JSX files in tests/ to TSX
find tests -name '*.jsx' -exec bash -c 'git mv "$0" "${0%.jsx}.tsx"' "{}" \;

# rename all JSX files in tests/__snapshots/ to TSX
find tests -name '*.jsx.snap' -exec bash -c 'git mv "$0" "${0%.jsx.snap}.tsx.snap"' "{}" \;

# rename all JS files in tests/ to TS
find tests -name '*.js' -exec bash -c 'git mv "$0" "${0%.js}.ts"' "{}" \;

we will likely need to update some of the build system's devDependency to support typescript

update test suite to support reading typescript/JSX files
update webpack to support reading typescript files

test that the UI works
ensure all tests pass as before

Other Considerations

There are other codemod tools that exist (ex: airbnb/ts-migrate), but they tend to be more complicated and require a certain folder layout, whereas this bash script is simple and is project agnostic

@STRML - what do you think? I didn't want to start writing any code without input from the community and/or waste folk's time

Hi. I think converting the repo to TS is a good idea, and would support it. However, there are two things you may be underestimating:

Keeping Flow types available, either on https://github.com/flow-typed/flow-typed, by converting automatically from TS definitions, or by hand
Converting from Flow to TS will take some manual work aside from just renaming the files.

That said, if you're wanting to tackle that, I would absolutely support it and spend time reviewing.

https://transform.tools/flow-to-typescript-declaration
@cliffordfajardo @STRML this could be simpler with tools like this to help you guys out converting it to type declaration.

Spent the day porting this lib in a fork.

TypeScript
Functional React
Lerna monorepo, where one package is the lib, other is examples (with vite build rather than webpack)
React 18
Removal of all prop-types things.

Tests are not working at the moment, but the code is fully functional (by playing around with all examples).
Could perhaps do this as a PR, to back-propagate changes, else it could perhaps help with some work.

That sounds great. I'd rather it be a TS lib given Flow has lost the battle.

Hey! Is this a closed ticket or can we perhaps expect a TS version sometime? I just figured @types/react-grid-layout isn't 100% accurate. At least the resizeHandle type seems wrong...

Hey! Is this a closed ticket or can we perhaps expect a TS version sometime? I just figured @types/react-grid-layout isn't 100% accurate. At least the resizeHandle type seems wrong...

See my comment, where I present my work porting this lib into a functional Typescript project.
Regarding the issue I think it was automatically closed by being stale

@timoisalive See my comment, where I present my work porting this lib into a functional Typescript project.

@marcus-pousette That's awesome, I'm just about to check it out!
It would however be nice if your work could be merged, so we could stay on the main library instead of a fork, so keeping my fingers crossed...
Thanks for your effort though! 👏

Thanks. Yeap I hope/wish it can be merged back to!
There are no real breaking changes except that almost every single file in the project has been changed. During the migration I did not take pay a lot of attention of making the test suite work, but to fix critical bugs that I had for my end use-case. If this fork will be merged back there are two things needed

Rewriting the test suite so it is up to date (and also improved testing more edge cases that were previously missed).
Bump major version

I'm happy to merge it if it is quality and doesn't break users.
On Tue, Jun 27, 2023 at 3:07 AM Marcus Pousette @.***>
wrote:

Thanks. Yeap I hope/wish it can be merged back to!
There are no real breaking changes except that almost every single file in
the project has been changed. During the migration I did not take pay a lot
of attention of making the test suite work, but to fix critical bugs that I
had for my end use-case. If this fork will be merged back there are two
things needed

Rewriting the test suite so it is up to date (and also improved
testing more edge cases that were previously missed).
Bump major version

—
Reply to this email directly, view it on GitHub
https://github.com/react-grid-layout/react-grid-layout/issues/1683#issuecomment-1608919113,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJEKP6H4KFALYT3PX4MEGLXNKBBXANCNFSM5PYBOJRQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Is there a config file to enable/disable features?

if I dont want the users to edit prompts, see history etc, is there any config file or something where I can enable/disable these features?

We already have a couple of settings in the config such as hide_cot and default_expand_messages. We are going to add more of those in the next releases!


warning triggers when checking todoist-plugin task inside a callout

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
When I check a tasks inside the Todoist-Plugin, which is inside a call out on my daily note, it shouldn't interfere with the task plugin by displaying the callout warning. Or at least I should be able to deactivate this warning myself.
Current behaviour
When I check a task inside the  querry that is inside one of the callouts of my daily note, this happens :

My setup is as follows :
> [!SUMMARY]+ Inbox
> ```todoist
> name: Inbox
> filter: "#Inbox"
> ```

Steps to reproduce
You just need to add the todoist query of the todoist-plugin inside callout and check of a task. It triggers the tasks plugin warning.
Which Operating Systems are you using?

[X] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[X] Windows

Obsidian Version
1.4.6 (105)
Tasks Plugin Version
3.10.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
I guess we should at least be able to deactivate the warning when we hear about it. I have read the documentation and the conditions for an issue with that are really specific and not for my use case. Hence the warning should have a toggle somewhere I guess.

Hi, there is already an existing request to turn off that warning so please do subscribe to that and up-vote it:

#1010

The Todoist aspect of this issue itself is not something that I personally would investigate alone, for two reasons:

It involves another plugin: see Bug involving another plugin
It needs a Todoist account and token to reproduce the bug.

If you can talk to the author of the Todoist plugin and find out if there is any way that other plugins can identify whether a task is in a Todoist code block, that would be helpful.
Once we know that, there's a conversation to be had about whether Tasks should just ignore and tasks that are in a Todoist code block, as in not do any checks for recurring tasks, and not add a Done Date.

Thanks for the answer. I was pretty sure there was already an issue that I'd seen before but I couldn't put my eyes on it, hence I created this one. Anyway, I contacted the dev of the todoist-plugin here.

I'm doing some housekeeping on the Issues list.
The request to disable the warning is already covered in another request.
So I'm closing this as 'Not planned'.
If anyone comes back with actionable info on how Tasks can detect whether a task is being toggled inside a Todoist block then I will re-open this and see if anyone volunteers to do the work.


Need shouldCause

Hi
shouldThrow very simple and useful.
need specify inner exception (cause exception) like assertj  (hasCause, hasCauseInstanceOf ...)
shouldCause is proper name^^

You can get the exception from shouldThrow and then add a further assertion.

Thanks.
Coding like this
val ex = RuntimeException("boom") val ce = shouldThrow<CompletionException> {                     val future = completableFutureOf<String>(ex)                     completedOf(future) } ce.message?.let { it should(have) substring "boom" } ce.cause should(be) an RuntimeException::class ce.cause shouldEqual ex


fix: added userOpHash to executeUserOp interface

oh I didnt see this before, but just added this yesterday - thanks anyways


Fixed issue related to Router hash

Fixed issue related to Router can't find the route by hash with query

Closing as discussed.


How to match the devices UUID with owners

I can see in the code it's only uploading the UUID and timestamp .
So in case MOH need to talk to UUID owners , how they will match the UUIDs with owners ?

Hi, please take a look at the spec: https://github.com/Co-Epi/CEN


Update

This step is now deprecated, use https://github.com/bitrise-io/steps-certificate-and-profile-installer instead


Upgrade the use of pydantic version 1 to 2

It would be great to upgrade the use of pydantic version 1 to version 2 since there are a lot of syntax changes. However, let's wait for autodoc_pydantic to upgrade first for documentation sake. Looks like lots of progress is being made! See https://github.com/mansenfranzen/autodoc_pydantic/issues/146#issuecomment-1646775697

You can now test the new autodoc_pydantic version supporting pydantic v2 via pip install autodoc_pydantic==2.0.0. The dev docs already showcase v2 support (see here). Migration guide is available here.
Any feedback is more than appreciated. The current autodoc_pydantic version 2.0.0 on pypi remains yanked for one week until final release.

Thanks @mansenfranzen for letting me know! I really appreciate your hardwork in maintaining the autodoc_pydantic library! 🍾


Reduce build size

The build is currently approximately 80kb now, which should be reduced.
We can take the following steps:

Update to Webpack 2 to enable tree shaking
Split components from the plugin so these can be imported separately
See which dependencies can be removed/optimized

I think that switching to webpack 2 is perfect.
Also removing the two component and make them a separate npm package that depends on this package is a good idea: they are not mandatory and they can be imported only if needed.

We're at 63kb now #18

@jonaskuiler awesome!

we reached 17kb! 🎉 🎉


fortify dplyr::transmute

I will tackle this.

Dear @cbeleites,
Initially, I wasn't sure if a user should be able to transmute the $spc column. That is, should one be able to ever update the spectra with transmute? At the moment, I'm assuming transmuting/mutating the $spc column is allowed (suppose the user wanted to multiply all the spectra by a factor of 2).
Well, using dplyr::transmute(.data@data, spc = spc*2) results in an error...but .data@data$spc <- .data@data$spc*2 works as expected. So, I came up with a solution that makes use of quosures (requires rlang), eval, and paste liberally.  I've never done any of this before, so it does feel weird. Anyways, please lmk if you see any logical or design errors...I'M ALL EARS.

Additional Notes:
Similar to select.R, I transmute.R won't always return a hyperSpec object (e.g., transmute(flu, c) is a data frame)
Additional Qs:
Q1: Is transmute(flu, spc2 = spc) still a hyperSpec object? Or should we just treat that as a data frame?
-- currently returns a hyperSpec object with no labels for spc2
Q2: Should transmute(flu, spc = spc *0) or transmute(flu, spc = spc *-1) be allowed or guarded against?
-- currently allowing this
Best,
EO

Yes, transmuteing the spectra matrix should be allowed, and Q2 arithmetic operations on the spectra matrix are perfectly fine and needed frequently.
E.g. `transmute

dplyr::transmute() and $spc: I suspect that is due to dplyr::transmute() not working with columns that contain a whole matrix.
I think we have to expect more matrix columns than just $spc: Partial Least Squares regression is a widely used model in chemometrics. package pls also uses matrices in columns, e.g. a so-called multi-analyte regression may be written plsr(y ~ spc, ...) with $y being again a matrix with each analyte in a column.

Q2, yes that would still be a hyperSpec object, we need to make sure there is a $spc column with

additional consideration: we need to decide how to deal with labels.

add a labels parameter so labels can be updated immediately, or
prescribe that people should call spc %>% transmute () %>% setLabels() setLabels()would be a new additional name forhyperSpec::labels<-()` (which is not very convenient to call as a setter since it must be quoted.

Opinions, please?

I think I like the design of:
spc %>% transmute(setLabels = T)

I guess we'll rather need
spc %>%
   transmute (r = sqrt(x^2 + y^2), phi = atan(y/x),
      labels = list (r = "r / μm", phi = expression (phi)
   )

or
spc %>%
   transmute (r = sqrt(x^2 + y^2), phi = atan(y/x)) %>%
   setLabels (r = "r / μm", phi = expression (phi))

r = sqrt(x^2 + y^2), phi = atan(y/x)

So, before we set the labels we'll end up with the columns r and phi...but where do we get the correct labels from?... Oh I see you're saying that the user will have to setLabels themselves.

@cbeleites or @ximeg could either of you help me understand the difference between
setLabels (r = "r / μm", phi = expression (phi)) and
setLabels (r = "r / μm", phi = phi)

Have a look at ? plotmath and try the various versions e.g. for the xlab argument of a plot.

"phi" is a string, and the axis label will be phi spelled in latin letters
expression (phi) will produce a φ (Greek phi character)
phi looks for a variable called phi, and will evaluate to the content of that variable (or an error if no such variable exists)
expression ("phi") looks the same as "phi", but it allows combination with further plotmath, e.g. expression (frac("phi",2*pi)) which yields a fraction with enumerator "phi" and denominator 2π.

@eoduniyi

Q1: Is transmute(flu, spc2 = spc) still a hyperSpec object? Or should we just treat that as a data frame?
-- currently returns a hyperSpec object with no labels for spc2

According ot definition (correct me if I'm wrong), the hyperSpec can contain only one spectra matrix and it must be present and must be named spc and not something else. hyperSpec also can have an arbitrary number of extra 1D columns. AFAK it cannot contain two spectra matrices or have nested data.frames stored in the extra columns.

Therefore

it should be illegal to rename the spc matrix, like in  transmute(flu, spc2 = spc)  or rename(flu, spc2=spc). So transmute(flu, spc2 = spc) may return a data.frame, but definitely not a hyperSpec object
it should be illegal to add second data matrix or data.frame with e.g. mutate(spc2 = spc + 1)

There must be exactly one maxtrix $spc (but that can be empty, i.e. nrow rows x 0 columns).
Other columns in general can contain matrices as well. One example would be preparing a multi-analyte data set for PLS regression: that would have e.g. concentrations in a matrix $c of nc columns, and the spectra in $spc. PLS regression can then be done by pls::plsr (c ~ spc, data = hyperSpec_object, ...)

@cbeleites Thanks for the explanations! I think this is something worth mentioning in the documentation, and I would also prefer to have the diagram updated. What about something like this (changes in red)?

mutate and transmute for hypeSpec objects have been implemented. Going to close this now.


Error reading IPAM reservations

Describe the bug
The IPAM provider started producing these errors in the plan for our sub vending pipeline
Error: Error Reading AzureIpam Reservation
│
│   with module.sub_vending["sub-ghc-prd-01.yml"].module.ipam_reservation_aue_vnet_2[0].azureipam_reservation.new_resv,
│   on modules/ipam/main.tf line 2, in resource "azureipam_reservation" "new_resv":
│    2: resource "azureipam_reservation" "new_resv" {
│
│ Could not read AzureIpam Reservation with id Fj6YaRZkT78WRAQVQjmewr:
│ Reservation not found: Fj6YaRZkT78WRAQVQjmewr

There are 14 of these errors in the plan now but these reservations were made between 1 and 9 days ago. There are also 14 reservations which aren't erroring but I assume that's because they have a dependency on the other reservations so weren't checked.
The following is from TF trace log
2024-10-04T11:16:31.745+1000 [ERROR] vertex "module.sub_vending["sub-ghc-prd-01.yml"].module.ipam_reservation_aue_vnet_2[0].azureipam_reservation.new_resv" error: Error Reading AzureIpam Reservation
2024-10-04T11:16:31.745+1000 [TRACE] vertex "module.sub_vending["sub-ghc-prd-01.yml"].module.ipam_reservation_aue_vnet_2[0].azureipam_reservation.new_resv": visit complete, with errors
The reservations are removed from IPAM once the vnet is tagged but I don't know why this is suddenly happening. The plan has been fine for 9 days despite being run multiple times. The IPAM provider was added to our vending process on the 25th September.
To Reproduce
Unknown
Expected behavior
No errors thrown in the plan
Desktop (please complete the following information):

Terraform version - 1.9.6
azureipam solution version 3.4.0
XtratusCloud/azureipam 2.0.0

Hmmmm, dumped the reservations from IPAM and they seem to be all there. More investigation required on my side.

Yep - token error causing the issue.
Resolved now - apologies.


Upgrade to Elm 0.19

The actual change to 0.19 was pretty minor, but elm-upgrade re-formatted some parts of the file. I also inlined a few of your utility functions from your other packages in order to avoid being blocked by Elm 0.19 compatibility on those packages.
I didn't bother upgrading the 'Advanced' example because it had too many dependencies. Maybe delete it?

FYI: Elm 0.19 is out now
If you don't have plans to update this, just let me know so that I can find an alternative. Thanks 😄

Hey! I've been on vacation, but I'm in the middle of updating all my Elm libs. Should be up in a day or two.

@krisajenkins close on this one? anything I can assist with?


nuxt issue when downloading package

Getting this error when downloading last version of the package: Pre-transform error: Missing "./dist/app/compat/capi" specifier in "nuxt" package

Hi @SantiagoLM01 did you read this?
https://github.com/nuxt-modules/sitemap/releases/tag/v5.0.0
Can you provide more information?

Yes, I read all that and still geting the same error.
I have a nuxt 3 application with nitro. I wanted to install sitemaps and found your package, but as soon as i install it and run npm run dev i get the errors Pre-transform error: Missing "./dist/app/compat/capi" specifier in "nuxt" package various times.

I'd recommend deleting the lock file + node_modules + .nuxt folder and re-installing. It's likely you have an old version of one of the core dependencies that the module is using.


Fix magic method arithmetic for MultiPointerTensor w/ TF objects

Currently, magic method arithmetic doesn't work for a MultiPointerTensor to multiple TF objects. For example:
x = tf.constant(2.).send(alice, bob)
print(tf.add(x, x).get())  # ==> List[Tensor(4), Tensor(4)]
y = x + x  # Raises AttributeError: 'MultiPointerTensor' object has no attribute 'add'

This is almost certainly due to torch-specific code left in MultiPointerTensor. Once a PR is made into PySyft moving these methods off MultiPointerTensor, and the script above is passing, this issue can be closed.

Not sure if anyone is working on this but I think that the problem can be solved by changing the MultiPointerTensor class to inherit from the ObjectPointer class as well (similarly to the way the PointerTensor class is defined).
From
class MultiPointerTensor(AbstractTensor):
to class MultiPointerTensor(ObjectPointer, AbstractTensor)
A quick change in syft/generic/pointers/multi_pointer.py did the trick.