PrimeNG 10

Will there be an update for PrimeNG 10?

Ou fork of https://github.com/yelhouti/primeng/releases/tag/10.0.2 has just been released, update should be coming soon

@msegmx 7.0.0 released


TO is mangling SSL server cert on update

Version: 3.0.0-9925.bd61e646.el7
When adding in a new server cert or updating an existing server cert, after pasting in the server cert + intermediate CA Chain into the server cert field and hitting update, the version number increments. TO logs show a POST action with a 200 response and no errors are thrown.
Leave the SSL page and come back to it for that DS, and the server cert field will only contain the server cert and last cert from the intermediate chain. The other certs from the intermediate chain have been removed.
Validated that this mangled cert is pushed to ATS servers and sometimes causes TR to do Null Pointer Exceptions when attempting to process the cert.

This might be due to the behavior of https://golang.org/pkg/crypto/x509/#Certificate.Verify:

Verify attempts to verify c by building one or more chains from c to a certificate in opts.Roots, using certificates in opts.Intermediates if needed. If successful, it returns one or more chains where the first element of the chain is c and the last element is from opts.Roots.

However, based on that wording, I'd expect it to possibly reorder/swap things around but always return a valid chain. Are the intermediates overlapping somehow?

basically it's turning:
Certificate[1]:
Owner: CN=speedtest.staging.xfinity.com, OU=Unified Communications, OU=Hosted by Comcast Corporation, OU=NSO, O=Comcast Corporation, STREET=1 Comcast Center, L=Philadelphia, ST=PA, OID.<IP_ADDRESS>=19103, C=US
Issuer: CN=COMODO RSA Organization Validation Secure Server CA, O=COMODO CA Limited, L=Salford, ST=Greater Manchester, C=GB
Valid from: Wed Feb 27 17:00:00 MST 2019 until: Sat Feb 27 16:59:59 MST 2021
  DNSName: speedtest.staging.xfinity.com
  DNSName: www.speedtest.staging.xfinity.com

Certificate[2]:
Owner: CN=COMODO RSA Organization Validation Secure Server CA, O=COMODO CA Limited, L=Salford, ST=Greater Manchester, C=GB
Issuer: CN=COMODO RSA Certification Authority, O=COMODO CA Limited, L=Salford, ST=Greater Manchester, C=GB
Valid from: Tue Feb 11 17:00:00 MST 2014 until: Sun Feb 11 16:59:59 MST 2029

Certificate[3]:
Owner: CN=COMODO RSA Certification Authority, O=COMODO CA Limited, L=Salford, ST=Greater Manchester, C=GB
Issuer: CN=AddTrust External CA Root, OU=AddTrust External TTP Network, O=AddTrust AB, C=SE
Valid from: Tue May 30 04:48:38 MDT 2000 until: Sat May 30 04:48:38 MDT 2020

Certificate[4]:
Owner: CN=AddTrust External CA Root, OU=AddTrust External TTP Network, O=AddTrust AB, C=SE
Issuer: CN=AddTrust External CA Root, OU=AddTrust External TTP Network, O=AddTrust AB, C=SE
Valid from: Tue May 30 04:48:38 MDT 2000 until: Sat May 30 04:48:38 MDT 2020

into:
Certificate[1]:
Owner: CN=speedtest.staging.xfinity.com, OU=Unified Communications, OU=Hosted by Comcast Corporation, OU=NSO, O=Comcast Corporation, STREET=1 Comcast Center, L=Philadelphia, ST=PA, OID.<IP_ADDRESS>=19103, C=US
Issuer: CN=COMODO RSA Organization Validation Secure Server CA, O=COMODO CA Limited, L=Salford, ST=Greater Manchester, C=GB
Valid from: Wed Feb 27 17:00:00 MST 2019 until: Sat Feb 27 16:59:59 MST 2021
  DNSName: speedtest.staging.xfinity.com
  DNSName: www.speedtest.staging.xfinity.com

Certificate[4]:
Owner: CN=AddTrust External CA Root, OU=AddTrust External TTP Network, O=AddTrust AB, C=SE
Issuer: CN=AddTrust External CA Root, OU=AddTrust External TTP Network, O=AddTrust AB, C=SE
Valid from: Tue May 30 04:48:38 MDT 2000 until: Sat May 30 04:48:38 MDT 2020

I'm inclined to think we shouldn't be manipulating certs, that we should use them in the Router and Caches as the user gives them to us.
We should definitely warn the user, if they give us something we think is invalid or mis-ordered.
But it's always safer to not mangle things, unless we have an extremely high confidence that we're always right. I definitely don't have that confidence here, that the Go libraries always know the proper order (we've seen evidence that some clients require cert order backwards), or support all the valid cert types and algorithms.


replace fishtown references with dbt Labs

There are a number of references to dbt Labs original name, Fishtown Analytics, in the dbt Core and Snowflake guide. This PR cleans up those references and also updates a package reference with a new link and version.

awesome, thanks for the PR!


Bluetooth pairing not possible

web setup runs on a Windows 10/64 machine. A TP Link Bluetooth LE able dongle (type UB400) is attached to the computer via USB, it is shown as working in the hardware manager, I am able to use it to connect to e.g. a bluetooth LE headphone (and other devices such as Sphero robots) without any problems, so I think I have established that the dongle in fact does work.
However, I am stuck at "pair with Vector", that does not work with seven different Vectors I tried. I do not get any error or status message whatsoever when trying to pair. I tried in multiple variations for quite some time now with Vectors 2 m away or directly in front of the dongle.
Nothing seems to get this working.
Basically I do not understand why you use Bluetooth at all. Nearly no desktop computer comes with Bluetooth . It should have been quite sufficient to just enter the robots serial to connect to him on a local network just as e.g. Vector Explorer does. I see abolutely no reason to use this superfluous connection method. I would suggest for further versions to just drop Bluetooth connection and use some other, simpler means that will actually work.
Plus: managing a lot of robots as in my case is a pain, since I need to reconnect and relogin every time. In my opinion this needs to be simplified quite a lot to make it user friendly.

I had to completely purge Chrome (84.0.4147.105) from the (newly installed Win 10/64) system and reinstall  to get it to work.
However, other browsers from the compatibility list do not work. And for people using Vival that might stumble across thisdi: Don't bother, the browser has Web Bluetooth bugs unfixed since early 2019, so theoretically it should work, practically it does not.

As for why they used Bluetooth: it is the only way to communicate with recovery firmware, and they need to use it to get the account details into the robot. After that, the robot settings actually do use similar to the SDK gRPC endpoints.

Then at least an option should be created to stay logged in to known robots. To have to relogin to each and every Vector is a very tedious process especially if you have lots of Vectors like me and if you want to experiment with different firmwares (in the future, when uploading actually works). This looks at the moment as if targeted at end users to do minimal things. But as I already read in comments multiple times: normal end users are highly overwhelmed by the installation and usage.

The software was originally built to bootstrap a new robot. We've been working on expanding the capabilities. I think without stopping to test now that if you just don't log in everything will be fine and an existing robot will work as it's supposed to. We'll investigate querying the BLE connection to see if we can determine if the robot already has a valid session and alter the setup flow accordingly.

Update: it seems the latest Chrome broke vector-web-setup Bluetooth support.

Update: it seems the latest Chrome broke vector-web-setup Bluetooth support.

It still works for me on the latest Canary build.

@grant-olson confirmed, it works ok with https:// when not on localhost. lol it seems so obvious now, sorry.

On pi3b+ and pi4, raspios-buster-armhf-full image (2020-08-20), Chromium with "Experimental Web Platform Features" enabled.
I'm stuck on bluetooth pairing. Vector is in the list of detected devices, but when I click on pair button, instructions and pairing button dissapear. Vector robot stay on ###### screen pairing. No error on console. I retry, Vector is detected as "paired" in bt devices list, click on pair again, but same problem.
Trying with UB400 dongle instead internal bluetooth of the pi do not change anything.
On Shield Portable with app, bt pairing is ok but can't final check because of cloud (can't reach anki).
On windows 7 with UB400 dongle, Chrome, Chromium, or Edge, on localhost:8000, bt pairing doesn't work (nothing append when click on pairing button, no error). I'm able to use other bt devices.
On a windows 8.1, npm install -g vectro-web-setup fail.
npm ERR! Unexpected end of JSON input while parsing near '...","devDependencies" :{'
I'm running out of ideas for initialize my Vector. Please help.

I get the same error on ubuntu focal: Unexpected end of JSON input while parsing near '...","devDependencies" :{' either with successfull npm install or running the code in this repo...

Can you please confirm the version of Node and npm being run?
grant@lord-humungus espressif % node --version
v14.11.0
grant@lord-humungus espressif % npm --version
6.14.8

I've tried many versions but after extensive research it seemed to be due to a network problem. The installation was successful after disabling vpn and firewall on my win8.1/64 machine.
But I'm always stuck on bluetooth pairing. The machine is totally different from my previous test. I really don't understand were is the problem.
I could think that the problem comes from my Vector robot but bt association works fine with the app on my Shield Portable.
Any ideas ?

I'm going to have to spin up a Raspberry Pi and see what's going on.

I was not able to get this to work on a RasPi with RaspiOS or on Ubuntu 20.04 LTS, even after lots of effort. It only did work on Win 10/64. Also other browsers than Chrome were a problem, even if they support Web Bluetooth as of documentation.

node: v10.19.0
npm: 6.14.4

I was not able to get this to work on a RasPi with RaspiOS or on Ubuntu 20.04 LTS, even after lots of effort. It only did work on Win 10/64. Also other browsers than Chrome were a problem, even if they support Web Bluetooth as of documentation.

I remember having some issues with pairing in general. I had to do a mix of turning off and on Bluetooth in settings and CTRL + SHIFT + R to the web setup.

Setup Complete on Pi3b+ and Pi4. Before clicking "pair with vector, click on bt icon in the taskbar and "add new device". Leave this window open without do anything, then click "pair with vector" on the web-setup and pair success.

Unfortunately I can't get passed the blinking environment icon. No fields of any kind show up, just the black page with the Chrome icon in the middle, although I am on latest stable Chrome. It's getting very frustrating...

https://github.com/WebBluetoothCG/web-bluetooth/blob/gh-pages/implementation-status.md

I am having a small problem with the Bluetooth connection dropping while attempting to download the logs to retrieve the QSN. I am able to pair with Vector and as I go through the setup, which appears to be the same setup you would do through the phone app such as log in with your user name and password, WiFi settings and so on. I click the download logs button at the upper right hand side of the window and not long after Vector exits his pairing screen, the Bluetooth connection is lost and when I connect to him again to try downloading the logs, it makes me go back through the user settings again and again and how ever many times I keep trying until I just give up. I have tried to put Vector all over the room, next to the computer, away from the computer, on the floor and on the desk. I checked the signal strength and it is very good with vector sitting on my desk. I have looked everywhere online on troubleshooting this odd problem and I suppose it must just be my problem since I found nothing about this anywhere. I am using a Debian version of Linux not a VM version which is very similar in operation as Ubuntu called Pure OS which developed by a Linux company named Purism and who also built my desktop computer. I am also running a the full version of chrome browser for Linux and not chromium if that helps any at all. Anything that will run on Ubuntu will run on Pure OS. I am exhausted trying to retrieve the QSN to get my copy of OSKR. I could use a some help because I have run out of ideas. I would very much appreciate it. It's probably something simple I am overlooking but too frustrated to see.

I am having a small problem with the Bluetooth connection dropping while attempting to download the logs to retrieve the QSN. I am able to pair with Vector and as I go through the setup, which appears to be the same setup you would do through the phone app such as log in with your user name and password, WiFi settings and so on. I click the download logs button at the upper right hand side of the window and not long after Vector exits his pairing screen, the Bluetooth connection is lost and when I connect to him again to try downloading the logs, it makes me go back through the user settings again and again and how ever many times I keep trying until I just give up. I have tried to put Vector all over the room, next to the computer, away from the computer, on the floor and on the desk. I checked the signal strength and it is very good with vector sitting on my desk. I have looked everywhere online on troubleshooting this odd problem and I suppose it must just be my problem since I found nothing about this anywhere. I am using a Debian version of Linux not a VM version which is very similar in operation as Ubuntu called Pure OS which developed by a Linux company named Purism and who also built my desktop computer. I am also running a the full version of chrome browser for Linux and not chromium if that helps any at all. Anything that will run on Ubuntu will run on Pure OS. I am exhausted trying to retrieve the QSN to get my copy of OSKR. I could use a some help because I have run out of ideas. I would very much appreciate it. It's probably something simple I am overlooking but too frustrated to see.

I am having a small problem with the Bluetooth connection dropping while attempting to download the logs to retrieve the QSN. I am able to pair with Vector and as I go through the setup, which appears to be the same setup you would do through the phone app such as log in with your user name and password, WiFi settings and so on. I click the download logs button at the upper right hand side of the window and not long after Vector exits his pairing screen, the Bluetooth connection is lost and when I connect to him again to try downloading the logs, it makes me go back through the user settings again and again and how ever many times I keep trying until I just give up. I have tried to put Vector all over the room, next to the computer, away from the computer, on the floor and on the desk. I checked the signal strength and it is very good with vector sitting on my desk. I have looked everywhere online on troubleshooting this odd problem and I suppose it must just be my problem since I found nothing about this anywhere. I am using a Debian version of Linux not a VM version which is very similar in operation as Ubuntu called Pure OS which developed by a Linux company named Purism and who also built my desktop computer. I am also running a the full version of chrome browser for Linux and not chromium if that helps any at all. Anything that will run on Ubuntu will run on Pure OS. I am exhausted trying to retrieve the QSN to get my copy of OSKR. I could use a some help because I have run out of ideas. I would very much appreciate it. It's probably something simple I am overlooking but too frustrated to see.

I am having a small problem with the Bluetooth connection dropping while attempting to download the logs to retrieve the QSN. I am able to pair with Vector and as I go through the setup, which appears to be the same setup you would do through the phone app such as log in with your user name and password, WiFi settings and so on. I click the download logs button at the upper right hand side of the window and not long after Vector exits his pairing screen, the Bluetooth connection is lost and when I connect to him again to try downloading the logs, it makes me go back through the user settings again and again and how ever many times I keep trying until I just give up. I have tried to put Vector all over the room, next to the computer, away from the computer, on the floor and on the desk. I checked the signal strength and it is very good with vector sitting on my desk. I have looked everywhere online on troubleshooting this odd problem and I suppose it must just be my problem since I found nothing about this anywhere. I am using a Debian version of Linux not a VM version which is very similar in operation as Ubuntu called Pure OS which developed by a Linux company named Purism and who also built my desktop computer. I am also running a the full version of chrome browser for Linux and not chromium if that helps any at all. Anything that will run on Ubuntu will run on Pure OS. I am exhausted trying to retrieve the QSN to get my copy of OSKR. I could use a some help because I have run out of ideas. I would very much appreciate it. It's probably something simple I am overlooking but too frustrated to see.

So... I believe I have found my problem.  I have new batteries on the way. I read through a few articles and found out when Vector's battery runs low and he start the quest for his charger, in order to save on power he start shutting down unnecessary system to include WiFi and Bluetooth. When Vector's battery goes bad he can still be run on the charger until the battery will no longer take a charge. I checked the CCIS screen to see what his voltage was before his ultimate demise and it was not charging enough to come off the charger with out committing suicide￼. I suspect that because the battery was on the way out even though Vector was on the charger The Bluetooth was probably intermittent. I am hoping a fresh battery will solve the problem. I suppose like all updatable electronics, it's best to have a full charge and/or be plugged up to keep the wireless comm on. Apparently Vector's batteries were so worn out he could not maintain a charge even while on the charger. If anyone else may have had this issue while attempting to use the vector-web-setup this could be a real possibility of connection issues.

So... I believe I have found my problem.  I have new batteries on the way. I read through a few articles and found out when Vector's battery runs low and he start the quest for his charger, in order to save on power he start shutting down unnecessary system to include WiFi and Bluetooth. When Vector's battery goes bad he can still be run on the charger until the battery will no longer take a charge. I checked the CCIS screen to see what his voltage was before his ultimate demise and it was not charging enough to come off the charger with out committing suicide￼. I suspect that because the battery was on the way out even though Vector was on the charger The Bluetooth was probably intermittent. I am hoping a fresh battery will solve the problem. I suppose like all updatable electronics, it's best to have a full charge and/or be plugged up to keep the wireless comm on. Apparently Vector's batteries were so worn out he could not maintain a charge even while on the charger. If anyone else may have had this issue while attempting to use the vector-web-setup this could be a real possibility of connection issues.

Nope. I've had problems with Vectors with very good batteries or units where I just had changed the battery. So this is not battery charge related. Web Bluetooth simply is an experimental tinkering solution that does not even wok on browsers that are compatible with it.
From what I heard even DDL devs recognize this now and work on other solutions: https://github.com/digital-dream-labs/vector-bluetooth

Nope. I've had problems with Vectors with very good batteries or units where I just had changed the battery. So this is not battery charge related. Web Bluetooth simply is an experimental tinkering solution that does not even wok on browsers that are compatible with it.
From what I heard even DDL devs recognize this now and work on other solutions: https://github.com/digital-dream-labs/vector-bluetooth

Cool deal! Thank you for the link. I have been messing with the web Bluetooth trying to get it to work right since October.

Cool deal! Thank you for the link. I have been messing with the web Bluetooth trying to get it to work right since October.

Me too. I have four desktop computers with various OSes in the office plus three laptops plus another desktop in the living room. Web Bluetooth did run halfway reliably on only one of them: A Surface Pro on Win 10/64 Pro with built-in Bluetooth. All other computers with Windows 10 Pro or Linux flavours refused to run Web Bluetooth halfway reliably or at all. Web Bluetooth is nowhere near end user or production ready. I gave up on it after countless hours of intense frustration after the release of Vector Web setup and OSKR.

Me too. I have four desktop computers with various OSes in the office plus three laptops plus another desktop in the living room. Web Bluetooth did run halfway reliably on only one of them: A Surface Pro on Win 10/64 Pro with built-in Bluetooth. All other computers with Windows 10 Pro or Linux flavours refused to run Web Bluetooth halfway reliably or at all. Web Bluetooth is nowhere near end user or production ready. I gave up on it after countless hours of intense frustration after the release of Vector Web setup and OSKR.

That's intense! All those computers and only one to make a connection sound like you had a quite few choice words moments. I ended up having to configure Bluetooth manually with the terminal to get as far as I have then the batteries took a dump on me. I figured the universe was again me. I had hoped it would have been as easy to get the OSKR going as it was to set up the SDK. That was disappointing to discover this was more or less a kicking and screaming kind of set up.

That's intense! All those computers and only one to make a connection sound like you had a quite few choice words moments. I ended up having to configure Bluetooth manually with the terminal to get as far as I have then the batteries took a dump on me. I figured the universe was again me. I had hoped it would have been as easy to get the OSKR going as it was to set up the SDK. That was disappointing to discover this was more or less a kicking and screaming kind of set up.

I cannot blame all of it on Web BT, there also seem to be problems with the BT implementation in Vector Web setup, since I was able to pair a Vector once and when reconnecting it showed up in the browser's list of known and already connected devices, but Vector Web Setup refused to recognize it despite it was clearly shown as connected in the browser's tool pages. I learned things I never wanted to know about Web Bluetooth settings in Chome via chrome://device-log or chrome://bluetooth-internals ... ;)
The thing I learned from this: If DDL releases something, wait at least three months before you try to use it. ;)

I cannot blame all of it on Web BT, there also seem to be problems with the BT implementation in Vector Web setup, since I was able to pair a Vector once and when reconnecting it showed up in the browser's list of known and already connected devices, but Vector Web Setup refused to recognize it despite it was clearly shown as connected in the browser's tool pages. I learned things I never wanted to know about Web Bluetooth settings in Chome via chrome://device-log or chrome://bluetooth-internals ... ;)
The thing I learned from this: If DDL releases something, wait at least three months before you try to use it. ;)


Please add more screenshots!

Your Readme is your main opportunity to sell people on the project. Please include another 3 or 4 screenshots to show what it can do.

cc @sebgoa, replacing the actual pixelated screenshot with a gif or multiple screenshots might be a good idea.

the first pic is actually a screencast. if you click on it plays a movie...
but yes, we will improve this (just back from break)

It's not clear from the text that it links to a video. But screenshots are
even better.
--
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://cncf.io/
tel:+1-415-233-1000
On Thu, Aug 10, 2017 at 3:24 PM, sebgoa<EMAIL_ADDRESS>wrote:

the first pic is actually a screencast. if you click on it plays a movie...
but yes, we will improve this (just back from break)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bitnami/cabin/issues/54#issuecomment-321649037, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC8MBn1mnBtn1d-0HfhV02tJRN6F0WsJks5sW1jTgaJpZM4OqKwb
.

@dankohn can you check now, let me know what you think, thanks

Looks fantastic. Huge improvement, thanks.


Setting minRowCount = rowCount does not hide delete buttons when adding a row

Are you requesting a feature, reporting a bug or asking a question?
Bug
What is the current behavior?
I want to remove the delete buttons from a dynamic matrix, and replace them with new delete buttons, for this I implement the code below:
surveyModel.onMatrixRowRemoved.add(function(sender, options) {
    options.question.minRowCount = options.question.rowCount;
});
surveyModel.onMatrixRowAdded.add(function(sender, options) {
    options.question.minRowCount = options.question.rowCount;
});

It will only work for surveyModel.onMatrixRowRemoved, but not for surveyModel.onMatrixRowAdded.
If I delete a row the delete-buttons do not show, if I add a row then the delete-buttons does show.
To solve this I modified the survey.angular.js file, I added the line: this.fireCallback(this.visibleRowsChangedCallback); to the QuestionMatrixDynamicModel.prototype.addRow function, see below:
/**
 * Creates and add a new row.
 */
QuestionMatrixDynamicModel.prototype.addRow = function () {
    var options = { question: this, canAddRow: this.canAddRow };
    if (!!this.survey) {
        this.survey.matrixBeforeRowAdded(options);
    }
    if (!options.canAddRow)
        return;
    var prevRowCount = this.rowCount;
    this.rowCount = this.rowCount + 1;
    if (!this.isValueEmpty(this.defaultRowValue)) {
        var newValue = this.createNewValue(this.value);
        if (newValue.length == this.rowCount) {
            newValue[newValue.length - 1] = this.defaultRowValue;
            this.value = newValue;
        }
    }
    if (this.data) {
        this.runCellsCondition(this.getDataFilteredValues(), this.getDataFilteredProperties());
    }
    if (this.survey) {
        if (prevRowCount + 1 == this.rowCount)
            this.survey.matrixRowAdded(this);
            this.fireCallback(this.visibleRowsChangedCallback); // Added line !!!!!!
    }
};

What is the expected behavior?
When setting: minRowCount = rowCount, this should hide the delete buttons
How would you reproduce the current behavior (if this is a bug)?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
your_code_here

Specify your

browser:
browser version:
surveyjs platform (angular or react or jquery or knockout or vue):
surveyjs version:

@summumeng I've added the code changes above to the project via the https://github.com/surveyjs/surveyjs/commit/14f91a19635cb86d0d9a205a638c0920691e5fc7. Thank you!

Thanks,
I still have a question: I do not seem to be able to get this commit, what I did was:
In the package.json file I set for dependencies:
"dependencies": {
"@types/knockout": "3.4.46",
"package": "git+https://github.com/surveyjs/surveyjs.git#14f91a1"
}
When I do an npm install in the node_modules\survey-angular directory the reply is:
<EMAIL_ADDRESS>C:\ionic\testionicsurveyjs\myApp\node_modules\survey-angul
ar
`--<EMAIL_ADDRESS>extraneous (git+https://github.com/surveyjs/surveyjs.git#14
f91a19635cb86d0d9a205a638c0920691e5fc7)
Which does not seem to update the survey.angular.js file.

The fix will be available in the next patch update, so it will be 1.0.43.

Thanks!


Update Composer dependencies (2021-11-24-00-12)

Visual regression test passed!
View the visual regression test report


Gem doesn't support simultaneous deployment to more than one autoscale group.

When deploying to a second autoscale group, this occurs:

I think this is because the deploy script for one autoscale group is attempting to delete the launch configuration for the other autoscale group. I think that this could happen with the AMIs too.

I'm open to ideas on how to cleanly tag AMIs/LCs so that the deployment knows which ones to remove at the end of the run. Right now we simply check that the name contains "ELBAS" and the environment. I suppose we could tag them somehow with the name of the autoscale group instead.

I'm open to ideas on how to cleanly tag AMIs/LCs so that the deployment knows which ones to remove at the end of the run.

Great.

I suppose we could tag them somehow with the name of the autoscale group instead.

I had the same idea. I think we need to update these lines:

https://github.com/turinginc/capistrano-elbas/blob/master/lib/elbas/launch_configuration.rb#L32
https://github.com/turinginc/capistrano-elbas/blob/master/lib/elbas/launch_configuration.rb#L54
https://github.com/turinginc/capistrano-elbas/blob/master/lib/elbas/ami.rb#L32

I'm trying to get something working at the moment.

Name is probably fine, we probably just need to change the deployed_with_elbas? method to look for whatever identifier is decided on. I think the cleanest thing would be to remove the overridden version of deployed_with_elbas? in LaunchConfiguration, and instead just change the one in AWSResource to work for both AMI and the LC.
I remember changing the AWSResource version to use tags instead of the name, can't remember why I had to override for the LC... I think it might be because LCs don't support tagging? I'd have to look again, it's been awhile. It would be ideal to just check a tag on either resource though so that it is consistent for both resources. And instead of tagging the AMI with just Deployed-With=ELBAS, we can tag with Deployed-With=.
:+1:

Agreed.
How about something along the lines of this:
module Elbas
  class AWSResource

    private

      def deployed_with_elbas?(resource)
        resource.tags['Deployed-with'] == 'ELBAS' && resource.tags['Autoscale-group-name'] == autoscale_group_name
      end

  end

  class LaunchConfiguration < AWSResource

    private

      def name
        timestamp "ELBAS-#{environment}-#{autoscale_group_name}-LC"
      end

      def deployed_with_elbas?(lc)
        lc.name =~ /ELBAS-#{environment}-#{autoscale_group_name}/
      end

  end

  class AMI < AWSResource
    include Taggable

    def self.create(&block)
      ami = new
      ami.cleanup do
        ami.save
        ami.tag 'Deployed-with' => 'ELBAS'
        ami.tag 'Autoscale-group-name' => autoscale_group_name
        yield ami
      end
    end

  end
end

Some quick comments:

I just tested this and it works.
I'm not sure why you implemented the LC elbas detection mechanism in a different way either, but I'm assuming that you had some reason so I haven't attempted to unify the mechanisms here.
I added a separate tag for the autoscale group name because this seemed like a separate data point and it could be useful to get all resources that have been deployed with ELBAS at some point (and this would allow doing so without regex/string parsing).
If this looks good, I can make the changes properly in the forked repo and submit a PR.

Do you have any feedback on the above?

Looks good. Thanks. I will look into unifying it later. I was going to hop into this project again soon anyway so I can update it to aws-sdk 2.

Awesome. That all sounds great.


At every page load reload the chai-form

I will change this to determine the final url based on current environment.


[Feat]: Adding a new ai tool

What feature?
the tool helps us to create images using prompts
Add screenshots
Add screenshots
Code of Conduct

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

go ahead and add it


Add let-traced

I don't think this will work with our current gen-rewrites-trace!-calls approach. More evidence for a function-based approach being the right way?


Add OpalOPC

OpalOPC is a vulnerability scanner developed by myself. It targets the OT protocol OPC UA.
It is free for non-commercial use.

Hi, thank you for the PR! The content you submitted has been merged and added to the database. As the pages are generated from the database, you will notice some differences between your submission and the final pages that include it, but the content stays the same.


Add BMWI Scraper

Thank you for contributing!
The extraction of questions looks good. However, there are some issues with the extraction of answers:

Some answers just contain an empty string (e.g. the answer to the question 'Welche arbeitsrechtlichen Fragen können sich im Zusammenhang mit dem Coronavirus für Arbeitnehmer und Arbeitgeber ergeben?')
I would strip the answer strings to remove whitespace at the beginning and the end of the answers

The answer string already got stripped, the stripped version just wasn't stored! :-D
The answers are structured in at least three different formats but I think with this latest version of the scraper it works quite well.

Looking good now! :)


ytop: Add "ytop" to ytop's error message

Right now, running nixos-rebuild gives an obscure error:
$ nixos-rebuild switch
building Nix...
building the system configuration...
error: Abandoned by upstream. Consider switching to bottom instead
(use '--show-trace' to show detailed location information)

(And --show-trace adds no useful information.)
The error message doesn't indicate that ytop is what's causing the problem.
By adding ytop to the error message, configurations that still reference ytop will be easier to debug and fix.

Actually in that file there are some other occurrances like this, maybe we can fix them all?

Sorry, this is a mistake on my end. I had assumed that aliasing a package to throw an error would also give the attribute that failed to evaluate.

No worries, @berbiche — it's a reasonable assumption and the stack-trace feature really could use some improvement. I'll see if I can fix the others @xaverdh.

No worries, @berbiche — it's a reasonable assumption and the stack-trace feature really could use some improvement. I'll see if I can fix the others @xaverdh.

Something like https://github.com/NixOS/nixpkgs/pull/101174 ?

Still as this seems a common and easy enough mistake to make, we should probably think about some form of automation.

Linking the RFC mentionned on IRC: https://github.com/NixOS/rfcs/pull/33

Opened #101210 to automatically add attr names, so this mistake can't be made manually any more.


Add an 'All Attributes' constructor to ColorScheme

Is your feature request related to a problem? Please describe.
I have started trying to upgrade TerminalGuiDesigner to use the latest nuget package.  A lot has changed since 2.0.0-pre.250!
One thing I am trying to tackle is the new init only properties.
Code DOM does not support initializer syntax (see this stack overflow post).  Post is old but I have checked latest prerelease nuget packages and still do not see any option for object initializers.

Describe the solution you'd like
Can we add a constructor that takes the relevant properties as parameters?
Example of current code produced by designer:
     this.dialogBackground = new Terminal.Gui.ColorScheme();
     this.dialogBackground.Normal = new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray);
     this.dialogBackground.HotNormal = new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray);
     this.dialogBackground.Focus = new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray);
     this.dialogBackground.HotFocus = new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray);
     this.dialogBackground.Disabled = new Terminal.Gui.Attribute(Terminal.Gui.Color.Black, Terminal.Gui.Color.Black);

I would then instead generate code like:
     this.dialogBackground = new Terminal.Gui.ColorScheme(
 new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray),
new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray),
new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray),
new Terminal.Gui.Attribute(Terminal.Gui.Color.White, Terminal.Gui.Color.DarkGray),
new Terminal.Gui.Attribute(Terminal.Gui.Color.Black, Terminal.Gui.Color.Black));

Code DOM does not support initializer syntax

Yeah, it's fun with analyzers and code generation in general, since those have to be netstandard2.0-compatible.
You can work around it by including the missing interfaces and attributes, though. I've done exactly that in the analyzers and code generators I'm writing for TG right now. Most that are attributes are just metadata that may or may not even end up in the compiled assembly.
It sucks even outside the context of TG, but I'm feeling it here as well.
In general, any type that has an init-only property really should also have the required keyword set, should probably have a formal constructor or factory method available that ensures they are set, and any such constructors/methods should also have the [SetsRequiredMembers] attribute on them.


Error with withdraw/deposit transactions after initiating hold transa…

Description
Describe the changes made and why they were made.
FINERACT-1705.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[x] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@vidakovic I assume we don't want to merge this for 1.8.0 rather you wanna cherry pick them from develop?

This is to be sent  to develop and not to 1.8.0
this PR can be closed.


Vim help files in HTML format are missing from the ftp server

Steps to reproduce

go to http://ftp.vim.org/vim/doc/
search for the vim91html.zip file

Expected behaviour
The html version of the Vim help exists in the ftp server.
In this case the http://ftp.vim.org/vim/doc/vim91html.zip is expected to exist.
Version of Vim
9.1
Environment
It is not environment dependent.
Logs and stack traces
No response

Quoting from the release announcement

The Vim FTP Server has been retired. It is no longer used to distribute artifacts to users. If you used it in the past, please update to using the git repository instead

The HTML Zip file archive is available here: https://www.vim.org/downloads/vim91html.zip (reachable from https://www.vim.org/download.php)


Native types in macros works for list but does not work for int

#1511 by @mkrizek added a test that looks like this (formatting mine):
{%- macro x() -%}
  {{- [1,2] -}}
{%- endmacro -%}
{{- x()[1] -}}

This shows that rendering [1,2] in the macro returns an actual list from the macro, not a string.
According to the first example on https://jinja.palletsprojects.com/en/3.1.x/nativetypes/ I would expect this to work as well:
{%- macro x(a,b) -%}
  {{- a + b -}}
{%- endmacro -%}
{{- x(1,2) > 3 -}}

I would expect the macro to return an int, but apparently it returns a string.
Environment:

Python version: 3.11.6
Jinja version: 3.1.3

but apparently it returns a string.

The expression x(1,2) > 3 is a boolean expression and as such should return a boolean which indeed it does:
>>> from jinja2.nativetypes import NativeEnvironment
>>> r = NativeEnvironment().from_string("{%- macro x(a,b) -%}{{- a + b -}}{%- endmacro -%}{{- x(1,2) > 3 -}}").render()
>>> r, type(r)
(False, <class 'bool'>)

If the macro returned a string there would be an error like TypeError: '>' not supported between instances of 'str' and 'int'. Can you clarify why you think the macro returns a string?
Or maybe https://github.com/pallets/jinja/issues/1701 is related to your issue?

Hm, indeed, that code works when I try it in another Jinja2 playground. I was using it in the Template editor in Home Assistant, where it does in fact give the error you mention:

So there must be something special about the environment there.


JSON dependency not downloading automatically when installing

After installing ruby and gems, we run a simple gem i pact-mock_service which seems to download just fine and it's able to use the command, but it crashes since it's missing the json dependency.
It doesn't seem to download it automatically or it simply fails silently since the json dependency needs the dev kit to be installed as well.

Are you on windows?

I'm not, but other developers are :)
On Wed Jan 07 2015 at 15:23:26 Beth Skurrie<EMAIL_ADDRESS>wrote:

Are you on windows?
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-68977395
.

json is a dependency in my Gemfile.lock, can you check yours?
    json (1.8.1)
...
    pact-support (0.1.3)
      awesome_print (~> 1.1)
      find_a_port (~> 1.0.1)
      json
      rack-test (~> 0.6.2)
      randexp (~> 0.1.7)
      rspec (>= 2.14)
      term-ansicolor (~> 1.0)
      thor

My workmate mentioned that he'd had problems with JSON on windows, and needed the dev-kit. He was using windows to create a standalone ruby executable for pact, and gave me these instructions to create it:
https://github.com/bethesque/pact-mock_service/wiki/Building-a-Windows-standalone-executable
See if that helps? I've never done Ruby on windows, so I'd just be googling the problem.

I can see that dependency too, but I think the issue is that it's failing
silently if you don't have the dev kit installed on windows.  Either way,
it's a bit confusing and we were scratching our heads as to why it wasn't
working.  It would be nice to have windows specific directions in the
readme file or a link to it.
For our purposes, we couldn't get it running on windows using ruby 2.x
without instability.  We also had to downgrade ruby gems to 2.0.3 to work
with native extensions, then had to install the dev kit manually to be able
to install json and get it working.  Here's part of our README for our
project on how to get it up and running.  Just for good measure, I added
the step of manually installing json so it doesn't silently fail.

You will also need to install Ruby 1.9.x ([windows install](
http://rubyinstaller.org/downloads/) with the appropriate Dev Kit (windows
only, on the same page as the ruby download) or `apt-get install
ruby1.9.3`) and [RubyGems](https://rubygems.org/pages/download).  After
Ruby has been installed, you must setup the DevKit; run the install package
and extract it wherever is needed, then run:

ruby dk.rb init
ruby dk.rb install

After the install of RubyGems, it must be downgraded to a previous version
for compatibility reasons (don't ask):

gem update --system 2.0.3
Finally, you can install the mock service for pact.  Run the following
command to install it globally:

gem i json pact-mock_service

Hope this helps.
On Wed Jan 07 2015 at 15:33:16 Beth Skurrie<EMAIL_ADDRESS>wrote:

json is a dependency in my Gemfile.lock, can you check yours?
json (1.8.1)

...
pact-support (0.1.3)
awesome_print (~> 1.1)
find_a_port (~> 1.0.1)
json
rack-test (~> 0.6.2)
randexp (~> 0.1.7)
rspec (>= 2.14)
term-ansicolor (~> 1.0)
thor
My workmate mentioned that he'd had problems with JSON on windows, and
needed the dev-kit. He was using windows to create a standalone ruby
executable for pact, and gave me these instructions to create it:
https://github.com/bethesque/pact-mock_service/wiki/Building-a-Windows-standalone-executable
See if that helps? I've never done Ruby on windows, so I'd just be
googling the problem.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-68977895
.

Good work. Sorry I couldn't be more help. You're probably the first people to run it on windows! Seeing as the wiki is open to everyone, would you like to add a wiki page called "Installing the pact-mock_service gem on Windows"?

Sure thing.
On Wed Jan 07 2015 at 15:47:21 Beth Skurrie<EMAIL_ADDRESS>wrote:

Good work. Sorry I couldn't be more help. You're probably the first people
to run it on windows! Seeing as the wiki is open to everyone, would you
like to add a wiki page called "Installing the pact-mock_service gem on
Windows"?
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-68978542
.

Btw, are you interested in building executable, as per those instructions? If you send it to me, I can upload it to the releases page, and it should mean you don't have to do the whole Ruby setup again.

I'm sorry, I'm new to Ruby; does the gem automatically build the executable
for us?  Would it just be in the gems directory?
On Wed Jan 07 2015 at 15:49:53 Beth Skurrie<EMAIL_ADDRESS>wrote:

Btw, are you interested in building executable, as per those instructions?
If you send it to me, I can upload it to the releases page, and it should
mean you don't have to do the whole Ruby setup again.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-68978674
.

Wiki page added. I recommend you add a link to it from the README:
https://github.com/bethesque/pact-mock_service/wiki/Installing-the-pact-mock_service-gem-on-Windows
On Wed Jan 07 2015 at 15:51:47 Michel Boudreau<EMAIL_ADDRESS>wrote:

I'm sorry, I'm new to Ruby; does the gem automatically build the
executable for us?  Would it just be in the gems directory?
On Wed Jan 07 2015 at 15:49:53 Beth Skurrie<EMAIL_ADDRESS>wrote:

Btw, are you interested in building executable, as per those
instructions? If you send it to me, I can upload it to the releases page,
and it should mean you don't have to do the whole Ruby setup again.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-68978674
.

Thank you! Done.
The gem does not automatically build an executable. The instructions I sent were how to build the executable, so that it packages Ruby and all the gems, and you don't have to install Ruby separately. One of my workmates did it, but I've lost the package, and I don't have a windows machine.

Gotcha. Gimme a moment to do that and I'll send you the package.
On Wed, Jan 7, 2015, 4:03 PM Beth Skurrie<EMAIL_ADDRESS>wrote:

Thank you! Done.
The gem does not automatically build an executable. The instructions I
sent were how to build the executable, so that it packages Ruby and all the
gems, and you don't have to install Ruby separately. One of my workmates
did it, but I've lost the package, and I don't have a windows machine.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-68979351
.

Alright, it's been packaged.  You can find pact.exe and pact-mock-service.exe on my google drive:
pact: https://drive.google.com/file/d/0B5HCBC5lmHhzc1BVbkYwZHM2UmM/view?usp=sharing
pact-mock-service: https://drive.google.com/file/d/0B5HCBC5lmHhzVGtLOUNpeUNCbVE/view?usp=sharing
Cheers and thanks for all the help.

That's fantastic, I'll put them on the releases page, thanks so much.

No problem.  Thanks for the work on pact :)
I actually got pact running from within grunt for our e2e testing process.
I'll create another wiki page to explain the process.  Ruby has given us a
lot of issues with node though and I was thinking of maybe spending some
time creating a node version of the mock server to easier integration :)
On Fri Jan 09 2015 at 10:36:43 Beth Skurrie<EMAIL_ADDRESS>wrote:

That's fantastic, I'll put them on the releases page, thanks so much.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-69270305
.

That has been considered often, however, it is a non-trivial amount of work.

I know, but I'm hoping that it would popularize pact even more :)
On Fri Jan 09 2015 at 11:25:08 Beth Skurrie<EMAIL_ADDRESS>wrote:

That has been considered often, however, it is a non-trivial amount of
work.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-69274956
.

Well, hope on the pact-dev channel and have a chat about it if you want to give it a go, and I'll point you to the specs and test cases.

Will do when I'm ready :)
On Fri Jan 09 2015 at 11:35:11 Beth Skurrie<EMAIL_ADDRESS>wrote:

Well, hope on the pact-dev channel and have a chat about it if you want to
give it a go, and I'll point you to the specs and test cases.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact-mock_service/issues/8#issuecomment-69275925
.


add fetchAppCred that will try for env level setting and not fail on     missing (instead returning defaultVal)

LGTM


unhandled_keypress: fix inconsistent behaviour of ctrl/meta right

this changed the ctrl/meta right behaviour to be consistent with ctrl/meta left
also mark xo.py executable for convenience
close #24

Does that include incrementing the version number?

Sorry that took so long, I was trying to wrap my head around those instructions. I think I've got it right now?

Yes, this is great! Sorry if the instructions were confusing!


How to add padding?

New Issue Checklist
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[x] I have looked at the Documentation
[x] I have read the F.A.Q.
[x] I have filled out this issue template.

Issue Info

Info
Value

Platform
e.g. ios

Platform Version
e.g. 10.0

SnapKit Version
e.g. 0.19.0

Integration Method
e.g. cocoapods

Issue Description
Hi,
How to add a padding value in my case if i dont know the width?
btExchange.snp.makeConstraints {
            $0.top.equalTo(productContainer.snp.bottom).offset(16)
            $0.centerX.equalToSuperview()
            $0.height.equalTo(40)
}

I think snapkit not provided this. Maybe can using this.
extension UILabel {
    var textWidth: CGFloat? {
        guard let myText = self.text else { return nil }
        guard let myFont = self.font else { return  nil }

        let rect = CGSize(width: CGFloat.greatestFiniteMagnitude, height: CGFloat.greatestFiniteMagnitude)
        let labelSize = myText.boundingRect(with: rect, options: .usesLineFragmentOrigin, attributes: [NSAttributedString.Key.font: myFont], context: nil)
        return ceil(labelSize.width)
    }
}

btExchange.snp.makeConstraints {
    $0.top.equalTo(productContainer.snp.bottom).offset(16)
    $0.centerX.equalToSuperview()
    $0.height.equalTo(40)
    if let width = btExchange.titleLabel?.textWidth {
        let padding: CGFloat = 8.0
        $0.width.equalTo(width+padding)
    }
}

@paulo-coutinho

You should be able to use the edges along side an inset these sorts of questions are more Auto Layout usage and should be directed towards platforms better suited for that like Reddit or Stack Overflow


Controllers specs

Create a Request spec file for all your controllers.

each action check:

If response status was correct.
If a correct template was rendered.
If the response body includes correct placeholder text.

Thanks!


Remove shared memory logic

Hi @jdpedrie @dwsupplee , per our discussion, this PR is to remove shared memory logic entirely from our library.

@jdpedrie Thanks for the review. The tests that will be affected is located in tests/grpc-gcp/fpm/, other tests are not affected. I think we can keep those tests for now since they can still be used to test cacheItemPool.

@jdpedrie Hi John, I have released v0.1.4 with this change merged. Do you mind re-enable the channel pooling functionality for spanner? Or should we make the changes accordingly?

I just opened a PR to put the config file back: https://github.com/googleapis/google-cloud-php/pull/1677


Add libc tag to the node package.json

From npm 10 libc filtering is supported.
In order to avoid installing both musl package and gnu package to linux users, adding this tag will solve the issue for npm 10 users.
For npm<10 users it'll not affect anything and we will stay with the current logic.

Relevant just for the next version of NPM which is not stable or supported yet, need to be checked when it will be relevant.


testing simple_example

[ ] https://threejs.org/docs/#manual/en/buildTools/Testing-with-NPM

✔ ~/workshop/potential-robot [test|✔]                                                                           [7/566]
16:01 $ npm test

><EMAIL_ADDRESS>test
> mocha --reporter list

(node:7916) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)

/home/alex/workshop/potential-robot/test/colors.js:1
export const Color = {
^^^^^^

SyntaxError: Unexpected token 'export'

16:04 $ npm test

><EMAIL_ADDRESS>test
> mocha --reporter list

(node:8222) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)

/home/alex/workshop/potential-robot/test/simple.js:1
import * as helios from "@hyperionbt/helios"
^^^^^^

SyntaxError: Cannot use import statement outside a module

https://stackoverflow.com/questions/62488898/node-js-syntaxerror-cannot-use-import-statement-outside-a-module

✔ ~/workshop/potential-robot [test|✔]
09:38 $ npm test

><EMAIL_ADDRESS>test
> mocha --reporter list

Error: ENOENT: no such file or directory, open './simple.helios'
    at Object.openSync (node:fs:600:3)
    at Object.readFileSync (node:fs:468:35)
    at file:///home/alex/workshop/potential-robot/test/simple.js:5:21
    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:438:15)
    at async formattedImport (/home/alex/workshop/potential-robot/node_modules/mocha/lib/nodejs/esm-utils.js:9:14)
    at async exports.requireOrImport (/home/alex/workshop/potential-robot/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
    at async exports.loadFilesAsync (/home/alex/workshop/potential-robot/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
    at async singleRun (/home/alex/workshop/potential-robot/node_modules/mocha/lib/cli/run-helpers.js:125:3)
    at async exports.handler (/home/alex/workshop/potential-robot/node_modules/mocha/lib/cli/run.js:370:5)

https://search.brave.com/search?q=how+to+import+modules+in+javascript+tests%3F&source=desktop

https://github.com/aleeusgr/potential-robot/issues/2

https://mochajs.org/#run-cycle-overview

https://mochajs.org/#asynchronous-code

https://stackoverflow.com/questions/59175043/error-enoent-no-such-file-or-directory-open-test-testfiles-shadab-jpeg-t

https://stackoverflow.com/questions/41060173/how-do-you-read-from-a-file-on-mocha-prior-to-running-tests

https://mochajs.org/#examples

https://mochajs.org/#running-mocha-in-the-browser


num_ch_utilize = config[dataset]['num_ch_utilize']

Thank you very much for sharing this code. But I don't quite understand some variable settings in it. I hope to get your answers and help. What is the meaning of “num_ch_utilize = config[dataset]['num_ch_utilize']" ?

Hi~
It was made to select the number of channels to be used when using multi-channel.
If you are satisfied with using the code, please click only one star.~
Have a nice one~

Still in the process of editing~

Thank you very much for your reply in your busy schedule. I have clicked a star to show my gratitude and admiration.


Support for Consul datacenters

Currently spring-cloud-consul assumes all services in Consul live within the same datacenter. Consul however has support for multiple datacenters and it would be very useful to lookup services using spring-cloud-consul that live in another datacenter.
This PR adds support for multiple datacenters. Since by default the Consul client only looks for services in it's own datacenter is is necessary to specify the datacenter explicitly when querying for a service that lives outside the current datacenter. Therefore I've introduced the spring.cloud.consul.discovery.server-list-query-datacenters property. This is a map of serviceIds -> datacenters, in the same way the existing property spring.cloud.consul.discovery.server-list-query-tags maps serviceIds to tags.
This allows one to specify in your spring boot config:
endpoint-stores.url=https://stores/api/products
spring.cloud.consul.discovery.server-list-query-datacenters.stores=dc-west

I've also added some documentation so users know how to use it. FWIW I'm currently using a patched version of spring-cloud-consul with this PR in our testing environment and everything works properly.

@rkettelerij Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@rkettelerij Thank you for signing the Contributor License Agreement!

Note: the failing tests in CircelCI are not related to this issue, master has failing tests.

@spencergibb cloud you perhaps have a look at this PR?

@spencergibb you mean rename serverListQueryDatacenters to datacenters?

@spencergibb I've updated the PR with your review comment. Renamed serverListQueryDatacenters to just datacenters

Squashed and merged with a bit of polish via 6d788d1582c9adc76defd8b8529b767b58d15577


spelling fixes

patch contains some spelling fixes ( just in comments ) as found by a bot ( https://github.com/ka7/misspell_fixer )

Cheers!


Discussion: Protocol Spec & Definitions

Let's establish some definitions. Spitballing here, feel free to
Q: What is "The Umbra Protocol"?
A: The Umbra protocol is a set of rules which define an interoperable shadow address standard.
Q: What is in The Umbra Protocol
A: Off the top of my head, it would have to define:

Which encryption algorithms are used
ENS Text record key
Standard message to be signed and stored in the ENS record (salt?)
How to format and encrypt announcement data
A smart contract where encrypted announcements are published
Rules (encoded in the smart contract) on announcing, for example, should an announcement require a small fee payment to prevent spam?

Am I missing anything here for the core protocol?
This library should make it very easy to interact with the protocol, meaning it should provide easy helper functionality for:

Publishing a signed message to ENS at the appropriate Text record
Checking if a given ENS name has published an Umbra PK
Sending a payment to given ENS name, steps:

Find the Text record
Recover public key
Generate random number
User random number to create a new address
Submit payment to Umbra contract

Checking for payments made to a given ENS name the user controls

Scan & attempt to decrypt all announcements
Identify applicable announcements and extract the random data used
Generate the address + private key

What am I missing. Is this all making sense?
Now that I'm thinking about thisI'm actually wondering if we should rename (again!) this repo, as it doesn't constitute the protocol, but instead represents a library for interacting with a protocol. Like you wouldn't hame a library for interacting with IMAP, IMAP. Or if you did, it would make conversations very difficult. Maybe something like umbra-core?

Am I missing anything here for the core protocol?

The only other thing I can think of is specifying how the funds are sent. I'm not sure if this should be part of the main protocol, but it does change how funds are accessed so it might be worth including.
I think there's two main approaches.

Send funds directly to the computed EOA address
If we assume the sender calls a function on the Umbra contract call to atomically publish the random number r and transfer funds, then we can either:

Cheaply deploy a minimal proxy contract during that transaction, send funds to that contract, and the contract only lets the funds be withdrawn by the computed address.
Use CREATE2 to send funds to some precomputed address and deploy the contract only upon withdrawal of funds.

The benefits of the second set of options are:

It removes the need for the recipient to have ETH to withdraw, and they'd use the GSN so gas costs can just be paid for using whatever token the recipient was paid in.
It allows recipients to specify what token they want to be paid in, so the owner can pay in any token and the receiver can have it automatically converted to DAI, ETH, etc.

What am I missing. Is this all making sense?

Makes sense! Probably good to write up a spec somewhere for it (where? maybe as an EIP?). The "Sending a payment to given ENS name" is just missing the step to encrypt and send the random number, though that's kind of implied by step v.
At first I was thinking we don't need a contract, and can just send r to some arbitrary address. But a contract would probably be good because we can atomically log the r as an event or save it in storage, then send the transaction, so it mitigates the issue of the random number not getting delivered to the recipient.

Now that I'm thinking about thisI'm actually wondering if we should rename (again!) this repo

Yea that's probably a good idea. Either umbra-core or umbra-js is good with me. No strong preference here!

If we assume the sender calls a function on the Umbra contract call to atomically publish the random number r and transfer funds, then we can either

Yes, this is what I was thinking. But I had not considered paying to a contract in either of the forms you list below. I had simply thought the umbra protocol contract would forward the funds to the generated EOA. I'm not opposed to the use of a contract recipient, I just want to understand what the benefits are.

Cheaply deploy a minimal proxy contract during that transaction, send funds to that contract, and the contract only lets the funds be withdrawn by the computed address.

As a baseline, what does this option provide over simply paying the computed EOA? Just trying to understand why we'd even consider it.

Use CREATE2 to send funds to some precomputed address and deploy the contract only upon withdrawal of funds.

This is interesting. Can you expand on how this would allow us to use GSN? My understanding with GSN is someone still has to pay the gas. Who would that be in our usecase?
Also, with regards to token swaps between payer and receiver, the idea would be the contract deployed upon withdrawl would use Uniswap or something similar to do the conversion? I guess one issue with that is the exchange rate can float significantly between when the payer make a payment and whenever the payee withdraws.
Also, I'm wondering if this functionality would be part of the core protocol, or a layer up. Is there someway the protocol can be flexible to allow uses like this, but without building them into the base itself?

I think there are two main benefits to supporting the ability to send to contracts:

Receiver can customize rules, such as Uniswap (yea, exchange rate could fluctuate though, haven't thought about the details much—just kind of brainstorming) or entering a chosen destination address when calling the contract's withdraw function. Withdrawal address is nice because of the reason below.
Receiving to a contract enables GSN support. The "someone" that has to pay for gas would just be the recipient—GSN is commonly used in cases where a company pays for gas on behalf of its users. But you can also use it so that users can pay gas costs in tokens.

This also improves UX because if a user uses Umbra frequently, they will have many different receiving addresses. If they are EOAs, it is tedious to fund each EOA with ETH for gas to withdraw funds, then afterwards withdrawing leftover ETH/dust. With GSN + create2, they can use a meta-tx sent from the stealth EOA to withdraw straight to an arbitrary address, such as an exchange account, cold storage address, etc.

Also, I'm wondering if this functionality would be part of the core protocol, or a layer up. Is there someway the protocol can be flexible to allow uses like this, but without building them into the base itself?

What about something like this, which would be part of the protocol: When a user first links their pubic key to their ENS address, they also have the option to link bytecode to that address. This is the bytecode used for their receiving contract. Then in our umbra contract, the sender calls a function similar to this:
function send(/* inputs here */) external {
  // 1. Save encrypted random value
  // Do stuff...
  // 2. Look up public key and bytecode associated with specified ENS address
  // Do stuff...
  // 3. Transfer the funds
  if (bytecode.length > 0) {
    // If bytecode specified by recipient, use create2 and send to deterministic address
  } else {
    // Otherwise, use transferFrom to send funds directly to computed EOA
}


Add keymaps the top-level Preferences menu

Add keymaps the top-level Preferences menu

@sandy081 FYI: I've added a Keymaps entry to the Preferences menu, the command palette and the extension viewlet's context menu. When a keymap extension is installed and another keymap extension is still enabled the user is asked if the old keymap extension should be disabled.

Thanks for letting me know. I will update it in our standup today.

@chrmarti Keymaps action is not showing any key map extensions. Is it because the keymaps category is not yet available in Market place? And are there plans to get this available in this milestone?

@chrmarti I talked about this addition in the stand up today and there was some feedback from the team.
Why can't be implemented / shown as how themes are shown?
What are the recommended key maps?
@Microsoft/vscode FYI

IMO, this should be implemented the same way as the themes. Keymaps should not be mutually exclusive, in terms of extension enablement: it should just be that the user picks with some UI like the theme switcher.
Also, we want to go away from curated lists of extensions. Introducing a keymapExtensionTips section in our product.json is the wrong way to go. Isn't there a category or keyword we can search for in the Marketplace to find all keymaps? The theme picker presents an option to find more themes in the Marketplace and queries it with category:themes. Shouldn't we do the same here?

There is a category:keymaps, but that list can quickly grow beyond what we want to offer a new user. That's why we use a curated list.
We've discussed the picker as an alternative, but went the Extension viewlet's route to start with.

Curated list will make it hard for newcomers to be seen. The problem of having too many extensions is solved by sorting the results by popularity, which is the default anyway.
Extension disablement is a heavy-weight action: it requires a workbench reload. At the same time, I don't think it should be used as an implementation detail for what is basically an option picker. We should be consistent with icons and themes here instead of introducing a new concept.

:+1: for not introducing another curated list. Sorting all "keymap" extensions by popularity will likely yield very similar results.

Removing the curated list depends on having a better sorting criteria for extensions (#16156). On using a picker instead of the extensions viewlet without having to reload the workbench: This can't currently be done with extensions and would either need that as a depending feature or be done without extensions which would make everything baked-in which isn't the direction we want to take. Closing for November.


Two changes in waiter.state for readability

Changes proposed in this PR

Inline the update function used by an update-in
Refactor a concat expression to remove a confusing cond-> and use when instead.

Why are we making these changes?
I was seriously confused by both of these expressions. I think the updated versions are more readable.

Awaiting green build


Fails to execute in Gradle (NoSuchMethodError in commons-io) (reopen #2509)

Describe the bug
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] Caused by: org.owasp.dependencycheck.exception.ExceptionCollection: One or more exceptions occurred during analysis:
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   InitializationException: Unexpected Exception
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]           caused by NoSuchMethodError: org.apache.commons.io.IOUtils.length([Ljava/lang/Object;)I
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   InitializationException: Unexpected Exception
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]           caused by NoSuchMethodError: org.apache.commons.io.IOUtils.length([Ljava/lang/Object;)I
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   InitializationException: Unexpected Exception
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]           caused by NoSuchMethodError: org.apache.commons.io.IOUtils.length([Ljava/lang/Object;)I
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   at org.owasp.dependencycheck.Engine.analyzeDependencies(Engine.java:662)
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   at org.owasp.dependencycheck.Engine$analyzeDependencies$0.call(Unknown Source)
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   at org.owasp.dependencycheck.gradle.tasks.AbstractAnalyze.analyze(AbstractAnalyze.groovy:90)
2021-06-23T11:47:02.002-0700 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter]   ... 120 more

This reopens #2509 because of 39f6a0b770a78ba3a0fbca9fa87e25968e429515 which re-replaced the Guava calls with IOUtils calls. This was previously fied in #2587 but the commit referenced above is a regression.
Version of dependency-check used
The problem occurs using version 6.2.2 of the gradle plugin with Gradle 6.8.3
Expected behavior
Don't crash. :grin:
Additional context
See PR #2587 for the fix.

Alternatively reimplement the method from ioutils within this library, if one wants to avoid calling guava?

@jeremylong looks very old, fixed in #2509  The dependency clashes with other plugins are an ongoing 'joy' though :)


[DevEx]: Replace vue-cli with vite

Improvement Summary
Vite is the current build tool recommended for vue development. We should replace the deprecated vue-cli with vite.
Detailed Description
This relates to the vue3 update as well. We will need to unravel the prerequisites to sequence this correctly with the typescript and vue3 todos. See  #193 and #114

Done :)


Add functionality for change color dynamically

Add functionality to change color dynamically for badge just like their data while creating the dynamic badge

#1525 Adds this functionality 😄

It could be nice to support in the standard json badge as well. I can't remember if the code in #1525 is doing that.

Thanks @paulmelnikow for adding this feature


Add Option button should shift downward for adding a new option in Bundle Product.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
MASTER
Issue Description
When we create a bundle product, there we can see Add option Button, Once we add Options and we need to create a new Option, there we need to scroll up and then we select the add option button. It would be better if the Add Option button shifts downward.
Preconditions
1. Simple product should be created so that to add the product to bundle options.
2. At least one option should be added, so that we can add new option.
Steps to reproduce

Go to Admin -> Create a Bundle Product -> Add Options.
After adding First option, now create another option.
Here we need to scroll up and then we select the add option button.

Expected Result
It would be much better if the Add Option Button shifts downward after we add a new option. So that if we need to create another option so we would not scroll up for the button, in bottom we can see the button we can add directly.
Actual Result
Please check the video for a better understanding of the enhancement what we can do in it.
Video
https://user-images.githubusercontent.com/92721837/199908038-f1e70d31-9080-434d-b694-6a480c81426c.mp4
Screenshots
Please check the screenshot where we can add the button.

Additional context
No response

@ashishkumar-webkul Issue Fixed with this PR #7084

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot below for future reference.


Restore manylinux-compatible PGO builds

Change Summary
Looks like omitting manylinux option is not the same as 'auto', cc @messense
Related issue number
Fix #1065
Checklist

[ ] Unit tests for the changes exist
[ ] Documentation reflects the changes where applicable
[ ] Pydantic tests pass with this pydantic-core (except for expected changes)
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Yep this looks better: https://github.com/pydantic/pydantic-core/actions/runs/6810149804/job/18517896939?pr=1068#step:13:18
please review


DirectDruidClient: Fix division by zero.

Introduced in #3954 when some floating math was changed to
integer math. This patch restores the old math.

👍

LGTM!


Make CRC for NVS data entries as a default y selection

Is your enhancement proposal related to a problem? Please describe.
NVS backed now supports CRC for data entries as well https://github.com/zephyrproject-rtos/zephyr/pull/73436
Describe the solution you'd like
NVS backend now has CRC enabled for metadata (as per older implementation), but CRC for actual data is disabled by default. It now makes sense to enable CRC for data entries as a default, instead of leaving this to user (end app developers would have high change of forgetting this).
Also, in general, it could be considered as a good practice that NVS does not blindly load corrupted data in the application states and prevent misbehavior that would be difficult to debug in the field. If someone does not need this CRC protection for data entries, then they need to make conscious choice to disable this manually.
Thus, recommending, https://github.com/RICCIARDI-Adrien/zephyr/blob/9184e09100ba00f813e707190f6d23d262bb8dc6/subsys/fs/nvs/Kconfig#L32 to be set to y as a default.

@omkar3141 thank you for creating this RFC. Sadly this cannot be done as it would break the NVS system for existing users.
I am also not convinced that this is the best approach in general: It is straight forward to add a crc to critical settings data by correctly using the settings system. This allows better control and conserves flash space. Other systems e.g. littlefs also don't provide protection of the data, only the metadata is protected.


Library doesn't encount for extra needed time inside expectation callback

If there are expectations to wait for that need extra time for example with enzymes update function jest will throw an error especially in combination wraped with act, since it exceeds the default timeout of 5000ms defined by jest with the following error message:
Async callback was not invoked within the 5000ms timeout specified by jest.setTimeout

Did you change your mind here? I wasn't sure what you meant.

@lgandecki
i guess the OP was complaining that the library precalculates the number of maxTries without taking into account the variable amount of time each expectation() execution might take..
that means the timeout to reject the waitForExpect might not turn out to be the one specified..


eBPF tutorial is out of date - Hooks and arguments

The Hooks and arguments section is out of date. It still says the context data is in https://github.com/microsoft/ebpf-for-windows/blob/main/libs/api_common/windows_platform_common.cpp which is no longer true.

@jsturtevant was this addressed in PR #1414?  If so, can this be closed now?

Clearing triaged tag since this didn't mark "help wanted" or get assigned to anyone, but might be fixed in 1414 so good to review/update in triage meeting.


Fix caching of target dir for fuzz project

Currently caching does not work for fuzz job which can be verified by looking at Github workflow outputs.
The main cause of the issue is the config workspaces: fuzz -> target. The fuzz target does not use a proper workspace but a separate crate. There is a pending issue discussing that.
The rust-cache action is executed in the root of the project and is not able to locate fuzz directory by workspace name.
I decided to list the directories to cache instead of relying on target dir detection which seems to work.
I also fixed the path fuzz/corpus to i18n-helpers/fuzz/corpus.
I adjusted the caching condition save-if to cache only on main builds. I think this makes sense because we don't need to cache PRs which might not get merged or get updated multiple times during review.

@sakex, I believe we forgot this in #86.

Oh sorry I missed this and thanks for the fix


Support custom guzzle client

Hi @malc0mn  and @VoganWong what do you think about this implementation?
It's backwards compatible, allows for custom guzzle client, and doesn't include composer.phar

@malc0mn missing params added, thanks! does it otherwise look good to merge + release?

I personally see no further issue with it, so go ahead I'd say :)

Great, merged and tagged v1.4.0. Thank you!


Alias gdtr to git diff-tree

gdt cannot be used for git diff-tree because this alias is being overriding later for git difftool so there is a conflict in names.
I propose to do renaming for git diff-tree alias but not for git difftool alias because:

gdt alias may already been used by someone for git difftol.
We already have a gmt for git mergetool so having a gdt for git difftool is less surprising.

cc @Stibbons

@marshall-lee we recently pushed PR #2790 that made a lot of changes to the git plugin, one of them being the removal of the duplicate gdt alias, in favor of the git diff-tree version.
If you think we should change that you need to rebase this PR against the current master and make the change you want to see.

Also, @Stibbons is not longer maintaining the Git plugin, I am.
Anyway, @mcornella put it perfectly, just follow what he wrote.

+1

Things are more complicated than I thought. There's no sense in this PR anymore.


Spark driver crashes with sigbus 0x7

Hi
My spark driver crashes every few days with sigbus 0x7 error code. I took intermittent directory usage of /tmp it is hardly 10 mb.
Spark operator it 0.8.0
And spark version is 2.4.0
fatal error has been detected by the Java Runtime Environment:#
SIGBUS (0x7) at pc=0x00007fe179e1df4e, pid=15, tid=0x00007fe15ee19ae8#
JRE version: OpenJDK Runtime Environment (8.0_181-b13) (build 1.8.0_181-b13)
Java VM: OpenJDK 64-Bit Server VM (25.181-b13 mixed mode linux-amd64 compressed oops)
Derivative: IcedTea 3.9.0
Distribution: Custom build (Tue Oct 23 11:27:22 UTC 2018)
Problematic frame:
V  [libjvm.so+0x5a0f4e]  JVM_handle_linux_signal+0x318cb#
Core dump written. Default location: /opt/abc/analytics/core or core.15#
An error report file with more information is saved as:
/opt/abc/analytics/hs_err_pid15.log#
If you would like to submit a bug report, please include
instructions on how to reproduce the bug and visit:
http://icedtea.classpath.org/bugzilla

I suggest you post the question to the Spark user mailing list instead of here, where you will more likely get help or answers you are seeking.


Never closing Goroutines

Goroutines triggerd are never closed, this is a wanted behaviour atm, workflows should run until end. It would however not be a good idea to add a stop channel so that we actually can trigger
the wg.Done() and close the goroutines and channels.

Each flow now has a stopchannel that can be used to cancel properly


hg installed by mercurial is a missing dependency

On a clean install of go1.6.2, running make depend results in the following error:
[ERROR]	Update failed for bitbucket.org/pkg/inflect: hg is not installed
Manually installing mercurial installs hg. make depend should probably install this.

It doesn't install git or subversion either? That's just a Golang development dependency and it depends on what platform you are on how you install it. The purpose of make depend is to install the Go library dependencies only.


One of Text's characters is just half rendered on Android

Description
Test case : https://snack.expo.dev/vyBu3Fm-d
The text "已售5.15万哈哈哈哈哈哈哈" is expected to be rendered on the screen with some of the text characters, but we got one of the characters is half rendered as the following image shows on Android. On iOS devices, it can work around with ellipsizeMode = {'wordWrapping'} .
React Native Version
0.72.1
Output of npx react-native info
System:
OS: macOS 13.3.1
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 29.83 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 16.13.1 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 9.6.7 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK:
API Levels: 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33
Build Tools: 23.0.1, 25.0.3, 26.0.1, 26.0.3, 28.0.3, 29.0.0, 29.0.2, 29.0.3, 30.0.0, 30.0.2, 30.0.3, 31.0.0, 33.0.0
System Images: android-18 | Intel x86 Atom, android-18 | Google APIs ARM EABI v7a, android-22 | Google APIs ARM EABI v7a, android-30 | Google Play Intel x86 Atom
Android NDK: 22.1.7171670
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9619390
Xcode: 14.3/14E222b - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_291 - /usr/bin/javac
Python: 2.7.5 - /Users/lzq/Library/Android/sdk/ndk-bundle/prebuilt/darwin-x86_64/bin/python
npmPackages:
@react-native-community/cli: 4.14.0 => 4.14.0
react: 16.11.0 => 16.11.0
react-native: Not Found
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Run the code example below
Snack, code example, screenshot, or link to a repository
https://snack.expo.dev/vyBu3Fm-d

still exist in the most recent releases.

Yes, it still exists in the most recent releases.

⚠	Newer Version of React Native is Available!
ℹ	You are on a supported minor version, but it looks like there's a newer patch available - 0.72.1. Please upgrade to the highest patch for your minor or latest and verify if the issue persists (alternatively, create a new project and repro the issue in it). If it does not repro, please let us know so we can close out this issue. This helps us ensure we are looking at issues that still exist in the most recent releases.

Yes, it still exists in the most recent releases.

I've tested the issue with the pr #37667. The issue doesn't occurred on Android. The effect now is equal to that on iOS with ellipsizeMode = {'wordWrapping'}.  I doubt that we get a new problem about consistency of 'clip'. I'm not certain for no iOS devices on hand now.

I believe that the PR was reverted


Unable to match Text wrapped in Text

Unable to match a touchable text wrapped in text
Description
In order to have a long string of text separated with touchable text, one need to wrap Text within Text as below. But I am unable to match the touchable text.

Steps to Reproduce
This is the render code
  <Text>
    {over21}

    <Text onPress={termsAndCondition}>{tnc}</Text>

    <Text>{and}</Text>

    <Text onPress={privacyPolicy}>{privacy}</Text>

    <Text>{other}</Text>
  </Text>

I then try to matching using the following but unable to get a match
await element(by.text({privacy})).tap()

When I inspect it using Xcode View Hierarchy tool I only see a single RCTText element.

Node, Device, Xcode and macOS Versions
Node: 8.6.0
Device: Macbook Pro, iPhone 7
Xcode: 8.3.3
macOS: 10.12.6

@hiaw just add testID and use id matcher

@isnifer Thanks for the tip. Tried that and didn't work.

You should be tapping on the switch element, not the text. Add an identifier to the switch and .tap() that.

@LeoNatan You suggestion was not related to my issue. Yes there is a Switch next to the Text of interest but I'm asking about the Text, not the Switch. It was not cool that you closed the issue without getting feedback.

On iOS, tapping on text does not trigger switches. Is this a custom functionality that you have added?

I'm not trying to tap the Switch, I can do that no problem.
I'm trying to tap Terms and Condition and Privacy Policy Text. And I couldn't as describe in the issue.
Thanks.

React Native’s render mechanism probably optimizes your view, so the end result rendered in native doesn’t have the same view hierarchy as you defined in JSX.
Try playing with your JSX view hierarchy and watch what is generated in native

@hiaw Don't take closing the issue personally. You problem was not clear to me from your issue. I tend to close issues quickly. After all, reopening an issue is a click away.

@rotemmiz Thanks for the comment. As mentioned before In order to have a long string of text separated with touchable text, one need to wrap Text within Text. So yes you're right that it optimizes my view and so it appears as a single RCTText in native.

Whatever you do in JSX is not noticable for Deto, as it ONLY asserts on the native view hierarchy.

Whatever you do in JSX is not noticable for Deto, as it ONLY asserts on the native view hierarchy.

Hello,
We are moving support questions to Stack Overflow. Please ask a question there with the detox tag. This section is only meant for Detox issues and enhancement requests.
Thanks

If anyone is still looking at this, with react native 0.54.2, detox ^7.3.2, you can do something like
<Text style={styles.helloText}>agree to
      <TouchableOpacity style={{ marginTop: -2 }} testID="termsBtn">
          <Text style={styles.helloText}>Terms</Text>
      </TouchableOpacity>
</Text>

then got this to work
await element(by.id('termsBtn')).tap();

@monmonja - I did the same. It definitely messes up the text layout tho, unless you style the touchable with margins to match the line height etc...


. #333 - Initial implementation for using gofabric8 docker-env as a fallback.

Not yet done: Call of 'fabric8:install' in case there is now gofabric8 in the path. Currently the goal bails out.
Requires 0.15-SNAPSHOT of d-m-p
Also still open: gofabric8 docker-env asks for user + password if not connected, but this shouldnt be the case (an empty string should be printed). See  https://github.com/fabric8io/gofabric8/issues/175 for this issue.

lgtm [merge]

lgtm

@rhuss would love a release once the snapshot of d-m-p is removed!

@jstrachan will do it today


Display the score for all Landers in that session (co-op)

Maybe each lander saves their high score in the lander table so it can be saved/fetched communicated quickly.


Add !ug as an alias for !unignore

Motivation: IRC crops long lines at a certain number of characters. If you add a too long pattern with !ig, it is impossible to remove it again because the line with !unig would be two characters too long. This alias works around that.
Cf. #218

On a related note, we should probably also make sure that the patterns in the igsets are short enough that they can be removed using !ug if necessary.

Just saw that the CI failed. I don't think that's related to my change in any way, especially considering that #286 failed with a very similar log.


Problem during npm install on prepare-hooks

Hello,
npm install fails with:
npm ERR<EMAIL_ADDRESS>preinstall: npm run prepare-hooks -s && npm run setup-hooks -s
npm ERR! Exit status 1
Full log
npm-debug.log.txt
Would be glad if someone could fix it.
Best Regards,
Krzysztof Grajek

I have this problem on npm 3.93 and 3.10.5 on Windows 10. In regular windows shell and on cygwin shell.

On ubuntu npm version 1.3.10 it worked fine for me.

On windows shell it gives: 'chmod' is not recognized as an internal or external command, few lines before that error.
After adding fake chmod.bat somewhere in PATH it changes to:
'.' is not recognized as an internal or external command,

Did anyone succeed install it on Windows?

Hi @kgrajek ,
I managed to solve it by npm install via windows ubuntu bash (read how to install it here),
then, you can go back to windows cmd, run 'npm rebuild node-sass'
and then you can build successfully the dep'.
consider you need to install on the ubuntu bash environment node, npm, gulp and bower as well.
Hope it helped,
Daniel.

Hi @592da,
thanks for your workaround tip!
Unfortunatelly it is not the way for me because it is a part of corporate project so I cannot use solution which will work only on my computer :(
Best Regards,
Krzysztof


Fails with poetry-core 1.6.0

Getting the following error:
  ModuleNotFoundError

  No module named 'poetry.core.toml'

  at ~/.lydev/poetry/venv/lib/python3.8/site-packages/poetry_source_env/plugin.py:7 in <module>
        3│ from os.path import expandvars
        4│
        5│ from cleo.io.io import IO
        6│ from dict_deep import deep_get
    →   7│ from poetry.core.toml.file import TOMLFile
        8│ from poetry.plugins.plugin import Plugin
        9│ from poetry.poetry import Poetry
       10│ from poetry.repositories.legacy_repository import LegacyRepository
       11│ from pydantic import BaseSettings, validate_arguments

That module seems to have been deleted in poetry-core 1.6.0, bundled with poetry 1.5.0: https://github.com/python-poetry/poetry-core/pull/483

Downgrading to poetry 1.4, I'm getting
Only one repository can be the default

Hi, thanks for reporting. I'l look into this when I get a chance. 👍🏾

What do you mean by this?

Running poetry self add poetry-source-env fetches poetry-source-env from files.pythonhosted.com, not the repos configured in env vars.
This makes sense, because poetry doesnt know to use the env vars before installing poetry-source-env. But I'm just pointing this out as a problem poetry-source-env might want to take a look at; e.g. in the original poetry issue, someone pointed out a solution using ~/.config/poetry/pyproject.toml

Running poetry self add poetry-source-env fetches poetry-source-env from files.pythonhosted.com, not the repos configured in env vars.

As far as I can tell, Poetry does not load plugins when executing poetry self commands, so what you're asking is unfortunately impossible. If and when python-poetry/poetry#5958 is resolved Poetry will hopefully be able to do this on its own.

poetry-source-env 1.6.0 has been released, requiring Poetry 1.5.0 or later. You may need to invoke Poetry with the --no-plugins option in order to get around failures that could arise from temporary incompatibilities between Poetry and poetry-source-env as you update both.
If your issues persist, feel free to leave another comment here.

what you're asking is impossible

That's not true, this is one way you can do it:
https://github.com/python-poetry/poetry/issues/5958#issuecomment-1479183720
The README should mention this step before running poetry self add poetry-source-env, in case someone wanting to use poetry-source-env can't access the internet at all

That's not true, this is one way you can do it:
https://github.com/python-poetry/poetry/issues/5958#issuecomment-1479183720

Minor clarification: it's impossible for poetry-source-env to accomplish.
I appreciate your suggestion, though — I'll add that information to the README.


JOSS review: Add a summary of current available sites that can be queried

Given this is a live package where users can add sites etc, can the authors report the total available sites that can be queried by this package? Ideally there should be a function that just reports this. Authors can also add this to a table in the README.
https://github.com/openjournals/joss-reviews/issues/5561

get_stats() function has been created

@sedv8808 Could you add a line in the documentation for this new function? Just what it does and how it can be used


remove micromatch, use metalsmith.match

Fixes #22

@RishiKumarRay Thank you, the package.json changes are ok. You forgot to edit lib/index.js though
https://github.com/metalsmith/remove/blob/master/lib/index.js#L1 needs to be removed
https://github.com/metalsmith/remove/blob/master/lib/index.js#L30 needs to be changed to metalsmith.match(patterns, input)
To keep the git history clean, you can make these changes locally, then use (so you can reuse the existing commit):
git commit --amend
git push --force origin master

@webketje please review now

@RishiKumarRay index.js changes are not there? Did you forget to run git add lib/index.js before doing the other 2 commands?

@webketje yeah sorry i forgot to add please check now

@RishiKumarRay

https://github.com/metalsmith/remove/blob/master/lib/index.js#L1 needs to be removed (not replaced)

Also just noticed you forgot to increment the metalsmith peerDependency in package.json to ^2.4.1

@webketje actually i had updated but forgot to save that file but now seems everything is there. Thanks for sticking with my silly mistakes and letting me know

metalsmith.match is a core method, so there is no need to have "metalsmith.match": "^1.1.1", in the dependencies, it can be removed. see https://www.metalsmith.io/api/#Metalsmith+match)
It's also a good practice to run npm test before you commit to make sure that everything is working as intended.

"dependencies": {
"debug": "^4.3.3"
}
do i need to remove this too ? along with that , Also i ran npm test , all test are failing

@RishiKumarRay You forgot to assign the result to const matches =  ...: https://github.com/RishiKumarRay/remove/blob/master/lib/index.js#L29

@webketje i have done this changes but the npm test is still failing

@RishiKumarRay I'm sorry, I forgot to add that metalsmith needed to be updated in devDependencies as well.
I cloned your fork, and it works with these changes:
diff --git a/lib/index.js b/lib/index.js
index 5f7ba5d..8b07869 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -26,7 +26,7 @@ function plugin(options) {
     options = options || {}
     const patterns = options.patterns || []

-    metalsmith.match(patterns, input)
+    const matches = metalsmith.match(patterns)
     debug('Marked %s files to remove', matches.length)

     matches.forEach((filename) => {
diff --git a/package.json b/package.json
index a8e7b40..98b5dc5 100644
--- a/package.json
+++ b/package.json
@@ -41,7 +41,7 @@
     "coveralls": "^3.1.1",
     "eslint": "^8.4.1",
     "eslint-config-prettier": "^8.3.0",
-    "metalsmith": "^2.3.0",
+    "metalsmith": "^2.4.1",
     "mocha": "^7.2.0",
     "nodemon": "^2.0.15",
     "nyc": "^15.1.0",
@@ -55,7 +55,6 @@
     "access": "public"
   },
   "dependencies": {
-    "metalsmith.match": "^1.1.1",
     "debug": "^4.3.3"
   }
 }

@webketje actually i had updated but forgot to save that file but now seems everything is there. Thanks for sticking with my silly mistakes and letting me know

Given that the issue is labeled good first issue, the feature being added is as important as: you hopefully learning something about Git, NPM, Node, Metalsmith, and open-source conventions so you can contribute better, later 😄

@webketje i have updated as stated also ran npm install then npm test but 4 tests are still failing , i don't know if this CI is going to run or not

@RishiKumarRay If the tests don't run locally, they will fail in CI.  You missed at line: https://github.com/RishiKumarRay/remove/blob/master/lib/index.js#L29. Please look attentively at the diff in my previous comment, you will see that your code is different.

@webketje did it failed again ? it passed locally

Ah yes

/home/travis/build/metalsmith/remove/lib/index.js
2:9  error  'normalize' is assigned a value but never used  no-unused-vars

That require('path') line must be removed

Ah yes

/home/travis/build/metalsmith/remove/lib/index.js
2:9  error  'normalize' is assigned a value but never used  no-unused-vars

That require('path') line must be removed

removed whole line or only path word ?

Yes you can remove that line

@webketje finally it passed :)

@webketje i was looking at the -0.7% decrease in code coverage is it due this extra line ? line 58 in screenshot

@RishiKumarRay No, nothing to worry about. It's because the total # of lines of code decreased.  I will merge this PR as soon as I'm ready to publish a new version


Fix unreferenced asset name on demo project: "sunburn" -> "Sunburn"

The “sunburn” image on demo project could not refer from any code. So I was fixing the asset name from “sunburn” to “Sunburn”. I only changed the directory name.

Dang, don't know how I let this through haha :)
Thank you for the pull request!


Android Basics: Fragments Navigation component

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-fragments-navigation-component?authuser=1&continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-2%3Fauthuser%3D1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-fragments-navigation-component#3
In which task and step of the codelab can this issue be found?
4 "Create Fragments and layout Files"
Describe the problem
Text says "Update tools:context in fragment_letter_list.xml to .LetterListFragment", but in the code below it is ".WordListFragment"
Steps to reproduce?

Go to the codelab
Scroll to the bottom
See the error

Additional information
Screenshot:

@Firreit
Thank you for reaching out to us.
This should be fixed now.