Build and create tomcat, fix several other issues with management server

Hi guys,
This PR has the following goodies:

Tomcat built by maven project (currently set to 8.0.22);
Java 8 support on tomcat :)
Fixed sluggish page reload times for client webapp;
Fixed some issues and merged differences in the tomcat startup scripts in centos 6 and 7
Fixed issue with cloudstack-setup-management script in centos 7
Fixed duplication of static files in management rpm
Synced sudoers config and fixed error running keytool in ubuntu

Some advantages of packing tomcat (while not getting rid of it completely ;) ):

Control the tomcat version that is actually run across all users by just setting the version on the tomcat module's pom.xml
Remove dependencies on different packages for tomcat in different distros (a change in the OS provided package will break setup, like what just happened with CentOS 7.0 vs 7.1).
Remove necessity of maintaining different configuration files for distros running different versions of tomcat
Extend out of the box functionality for more distros :)

Looks good, I see you've remove tomcat requirement as a package dependency. Is this embedding tomcat so we won't need to install on the distro? I can help test it next week.

Yes, it completely removes the need to have tomcat installed in the distro,
and at the same time won't conflict if there is any other version of tomcat
installed.
Besides that, everything seems to be working fine with Java 8 now.. at
least for management server :)
I've made some basic testing with Centos6 and 7 and Ubuntu 12.04.05, and
all seems fine (didn't update the fedora part yet)
If everything goes smoothly and this gets pulled in, i'll make a bigger
cleanup afterwards like removing any distro-specific files and
config related to tomcat that are scattered troughout /server /client and
/packaging).
BTW Rohit, i noticed you cleaned up awsapi (i had to merge de spec files,
darn) but you forgot to clean up the configs for tomcat, webapps7080
folder, firewall rules for it, etc ;)
RF
On Fri, May 8, 2015 at 4:59 PM, Rohit Yadav<EMAIL_ADDRESS>wrote:

Looks good, I see you've remove tomcat requirement as a package
dependency. Is this embedding tomcat so we won't need to install on the
distro? I can help test it next week.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/cloudstack/pull/238#issuecomment-100262595.

Thanks @rsafonseca, it sounds great. While I think I had removed awsapi related tomcat stuff with commit 58999da it's possible I may have missed few spots, or a recent merge on master could have brought few things back. Can you also help fix the fedora related specs etc, in packaging/{fedora20, fedora21}/ to maintain consistency wrt not using distro provided tomcat?

Yes, I'll take care of that.. i just haven't removed the old tomcat files
of other installers so that this PR wouldn't break building on any OS, as
is.
As soon as i have tested new configs for fedora i'll clean up the rest of
the stuff and do another PR :)
On Fri, May 8, 2015 at 8:07 PM, Rohit Yadav<EMAIL_ADDRESS>wrote:

Thanks @rsafonseca https://github.com/rsafonseca, it sounds great.
While I think I had removed awsapi related tomcat stuff with commit
58999da
https://github.com/apache/cloudstack/commit/58999daafe862ebbe953903ce7f928939c9b85f0
it's possible I may have missed few spots, or a recent merge on master
could have brought few things back. Can you also help fix the fedora
related specs etc, in packaging/{fedora20, fedora21}/ to maintain
consistency wrt not using distro provided tomcat?
—
Reply to this email directly or view it on GitHub
https://github.com/apache/cloudstack/pull/238#issuecomment-100315706.

Hi @bhaisaab
Have you had time to test this? I'm amazed no one else cared to take a look at it, as it's a big change and a big step into supporting java8 and multiple distros :|

@rsafonseca thanks for the updated PR, really like the last update to bring down the mgmt package size. Unfortunately I'm caught up with dayjob and since it's a big change I could only review/test/merge next week or a week after that, unless someone else could help out here.

Hi @bhaisaab :)
I've also made some basic testing with fedora 2x as you asked and got everything working last night (apart from having the cluster servlet port (tcp 9090) clashing with systemd's cockpit.
Just didn't commit the changes because of this last issue, was trying to gather some opinions on changing the port 9090 to some other port.. but no feedback/opinions on that yet...

Rafael, the gzip compression alone indeed does not make the tomcat serve it compressed

tomcat 8.0 allows configuring the default servlet so that it serves compressed if there is a .gz file
Since we had tomcat6 so far, there is a special servlet registered to do this see client/WEB-INF/web.xml
jetty (developer mode) by default uses the .gz compression it must be explicitly turned off if you do not want it

Good catch k0kza, I hadn't noticed that StaticResourceServlet class, but i'll do some benchmarking on that vs tomcat 8's connector built in compression features, but it's still a whole lot faster than it was :)

@rsafonseca sorry, the PR fails to cleanly apply on master. Can you rebase against master and send it again, along with any other fixes. Thanks.

@bhaisaab rebased and sent. No new fixes since yesterday, but i rolled back the change of port 9090, to not leave anyone unhappy and make things easier on existing users :)
Will provide a simple fix for that as soon as i have time, that will apply to all ports used :)

Getting this:
[root@jenkins packaging]# bash package.sh -d centos63
-d centos63 --
7
java-1.x.0-jdk-devel 1.7.0+ not found\n Cannot retrieve version to package\n RPM Build Failed
I have openjdk installed, which always has worked fine.
[root@jenkins packaging]# rpm -qa | grep java | grep devel
java-1.7.0-openjdk-devel-<IP_ADDRESS>-<IP_ADDRESS>.el6_5.x86_64
java-1.6.0-openjdk-devel-<IP_ADDRESS>-<IP_ADDRESS>.el6_5.x86_64

Hey @rsafonseca,
Thanks for improving ACS! ;)
What is the status of this PR? I noticed that the last build failed, but it doesn't seem related to your changes (OVM3 failed).
Could you just describe with more details what you tested? Did you execute any Marvin tests? I saw above that you tested with CentOS 6/7 and Ubuntu 12.04, but what exactly you tested?
Thanks again.
Cheers,
Wilder

Hello @wilderrodrigues :)
I guess the status of this PR is that it was waiting for me to fix the build issues... i was away since last wednesday and just got back yesterday, so i hadn't had time to look into it until last night.
The OVM3 failure in jenkins is indeed unrelated to my changes, it seems it's failing for a few PRs whenever it gets built on specific jenkins servers... probably due do different jenkins configs.
About the testing, i had only run the existing JUnit tests and did some manual testing on the mentioned platforms (service startup/config issues, tomcat performance/startup and java8 supportability for some things that were failing) and did not test the whole platform.. i still didn't have a look at Marvin, i hope i'll have time to start playing with it over the next week :)
@bhaisaab all fixed!

looks like a couple more things got broken since last rebase with master... need to refix :|

@rsafonseca Thanks for updating the PR. I think a rebase is needed. I see a lot of rc/init files removed esp from centos7, fedora20/21 directories? Also, have you build the packages and test them yourself? I'll be able to help test it by building packages later this week or next week.

Hi @wilderrodrigues
I was hoping on reducing the number of changes from this PR to master, so it would be better if you pushed it along with your refactor and i'll just rebase with master, since i don't know when this will get merged :)
During your Libvirt refactor pushes, i was having some trouble with that specific test taking too long to run, it was fixed after i set the timeout (after 5 seconds failure, waits no longer to fail and continues), so this speeds things up unless a connection is not actually established, not the other way around :)
Replacing that with a static final would be nice, but like i said, this is a copy/paste from the xenserver class of the same name, so that change should be applied there also in order to be consistent.
I'd rather not add any more fixes that are unrelated to the tomcat change into this PR :)

Hi @rsafonseca
Cool answer as the PR is hanging for way too long. :)
I will apply the changes to a separate branch and create a PR.
Cheers,
Wilder

Hi @rsafonseca,
It's clear that this was quite some work to pull off, so first of all: thanks! To be honest, I'm not sure on how to test this properly and help you getting this through. If you can give some pointers on how we can test this, that would help. Otherwise maybe others can test it.
The contents of the PR also changed a bit over time. Is the description still covering the changes? Great btw that some stuff ended up in separate PR (that got merged).
More general, could you please squash your commits? There are many commits, lots of merge ones and also some that were more work-in-progress than anything useful.
Regards,
Remi

Hi @remibergsma
First of all. thank you for your comment :)
I've been very busy these past couple of days, so i haven't gotten around to do what you suggested, but i will take care of that over the weekend (squash commits, clean description of fixes), I will also try to get a couple more stuff in as separate pull requests and clean up a couple more things ;)
As for testing this, there is no other way to test this other than manually since marvin doesn't even use tomcat for testing, altough the travis setup scripts download an old version of tomcat into the hard disk the tests do nothing with it :) That should also be cleaned up.
Another handy thing that's really missing in the tests is a test to packaging! In the past weeks a few things got into master that completely broke packaging.. although Rohit fixed most of that already, centos6 packaging is still broken.
As a side note, master also still doesn't work in Centos 7.1 and Fedora 21.. that's all in here hehe ;)
If you guys would like, i can also alter the test scripts to use the same tomcat that will be used in the actual product instead of jetty, which will make for a test scenario closer to reality. This can be done in a way which will easily allow switching between tomcat and jetty in the future :)
What do you think of that?
BR,
Rafael

Hi @rsafonseca
I got your PR to build the RPMs and do some manual tests on it.
RPMs were created successfully! I then copied them into a new machine, brand new, and installed common/management RPMs there. When starting the Management server I got the following in the logs:
29-May-2015 03:38:41.413 WARNING [main] org.apache.catalina.startup.ClassLoaderFactory.validateFile Problem with directory [/usr/share/cloudstack-mysql-ha/lib/*jar], exists: [false], isDirectory: [false], canRead: [false]
29-May-2015 03:38:41.567 WARNING [main] org.apache.catalina.startup.Catalina.load Unable to load server configuration from [/usr/share/cloudstack-management/conf/server.xml]
29-May-2015 03:38:41.572 WARNING [main] org.apache.catalina.startup.Catalina.load Unable to load server configuration from [/usr/share/cloudstack-management/conf/server.xml]
29-May-2015 03:38:41.572 SEVERE [main] org.apache.catalina.startup.Catalina.start Cannot start server. Server instance is not configured.
Errors are clear for me: /usr/share/cloudstack-management/conf/server.xml  and /usr/share/cloudstack-mysql-ha/lib/*jar don't exist. Are those some new stuff?
I already asked @remibergsma if he would know about it, but unfortunately he doesn't.
Do you have any hints on those ones? :)
Thanks a lot, dude!
Cheers,
Wilder

Hi @wilderrodrigues
Both the issues you hit are legacy behaviour :)
I'm guessing that you've deployed it in debian, because the server.xml issue is handled properly by the RPM installers, but not by the debian one.
To have it working after install on debian, just run the cloudstack-setup-management script, which i think is the standard way of setting things up, that's why I didn't bother to get the server.xml properly linked on package install in debian yet.
I've been thinking about doing something better (at least that's my opinion, please drop a comment on this) which is setup the management server to ALWAYS run with both HTTP and HTTPS right after setup (using generated cert until user provides one)
About that mysql-ha lib i guess it's one of the optionally packed libraries? i still haven done any testing on those. I just left this bit as it was.
Happy testing :)
RF

Hi @rsafonseca
I built the RPM on CentOS7:
[root@cs5 ~]# uname -a
Linux cs5 3.10.0-229.1.2.el7.x86_64 #1 SMP Fri Mar 27 03:04:26 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
[root@cs5 ~]# cat /etc/centos-release
CentOS Linux release 7.1.1503 (Core)
I will try executing the cloudstack-setup-management script on my centOS as well.
Cheers,
Wilder

Indeed.. it's only setup on centos 6 and not 7, sorry for misleading you there :)
cloudstack-setup-management will link the server.xml in any environment anyway.
The current version on master drops a server.xml in the folder (for Centos7 only), but it doesn't support SSL. I just restored the behaviour to be the same as in other OSs (you get a non-ssl and ssl version and link is handled by the setup scripts).
Configuring SSL by default will remove the need to have a separate server-ssl.xml and server-nonssl.xml and will enable to just drop the server.xml config file in /etc/cloudstack/management .
The behaviour should be consistent across distros
Thanks for taking the time to test it out :)

Hi @rsafonseca
I executed the script then...
lrwxrwxrwx. 1 root root     44 May 29 05:39 server.xml -> /etc/cloudstack/management/server-nonssl.xml
[root@cs5 bin]# systemctl status cloudstack-management
cloudstack-management.service - CloudStack Management Server
Loaded: loaded (/usr/lib/systemd/system/cloudstack-management.service; enabled)
Active: active (running) since Fri 2015-05-29 05:39:16 EDT; 34s ago
Process: 16149 ExecStart=/usr/sbin/cloudstack-management-sysd start (code=exited, status=0/SUCCESS)
Main PID: 16164 (java)
CGroup: /system.slice/cloudstack-management.service
└─16164 /usr/lib/jvm/java/bin/java -Djava.awt.headless=true -Dcom.sun.management.jmxremote=false -Xmx2g -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=/var/log/cloudstack/management/ -XX:PermSize=512M -XX:MaxPermSize=800m -Djava.net.preferIPv4Stack=true -D...
May 29 05:39:16 cs5 systemd[1]: Starting CloudStack Management Server...
May 29 05:39:16 cs5 systemd[1]: Started CloudStack Management Server.
:)
I just have to configure the DB to see how it will behave.
Perhaps if you create a new PR, with the packaging stuff only, it would be easier to test and get it merged onto master. The steps to test the packaging are quite straight forward and we are glad to help on that.
Cheers,
Wilder

This has been stale for a while, here's something to spice it up :)
https://github.com/apache/cloudstack/pull/372
If all in agreement will close this one and we'll go for the new one ;)

@rsafonseca hey, looks like the new #372 one some interesting things (such as embedded jetty); do you want to or have already take the good stuff from this PR to 372? If you think the new one is the best way forward, sure close this one. I'll have some time later this week to help you test packaging related stuff.

Hi @bhaisaab :)
The new https://github.com/apache/cloudstack/pull/372 was built on top of this one, so all the fixes in this are also in the other one.
I do think the other one is the way to go, it includes a few more fixes and cleanup than this one and gives the option to select the servlet engine that gets auto packed into the rpm which i hope will be a popular change. So... closing...


Provide try_inc_by to counters to avoid panics

Users can choice to avoid panics by calling try_inc_by().

Do you think anyone will use this API to replace inc_by?
I think this solves nothing.

Since int counter is not part of spec, then handling overflow feels like an undefined behavior to me. Then it looks OK and convenient to let it overflow when compiling without debug-assertions, which is also the default strategy of rust builtin int types.

@breeswish
I prefer removing try and ignoring the overflow or handling it in debug. Letting the user handle try explicitly help nothing but increases the load.

Any updates?


Changing Uri works but very wierd

I have adresses with version number like https://api.infirma.pl/2.6.0/version.
I have middleware that strips that version number:
$app->add(function (Request $request, Response $response, callable $next) {
    $uri = $request->getUri();
    $path = $uri->getPath();
    $version = file_get_contents(__DIR__ . '/../../VERSION.md');
    if (substr($path, 0, strlen($version) + 1) === '/' . $version) {
        $uri = $uri->withPath(substr($path, strlen($version) + 1));

//        echo ' ';

        return $next($request->withUri($uri), $response);
    }

    return $next($request, $response);
});

With echo ' '; code works. But when I comment it in request stops at 'OPTION' method.
I have no idea  how to fix it.

Can you create a small working project that shows the problem please?


[WIP]: Replace old portico footer with new sectioned one.

This replaces the old footer that has one section with a small list of
items. This expands the footer to have multiple sections.
WARNING: This does not have links; it has filler content.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

Screenshot:

@timabbott, @rishig let's come up with a list for this?

@brockwhittaker the tests are failing due to broken links; I think we should just remove those links for now and add them in later; that way, we can merge this now and then do another pass on the links a bit later.
Visually, I wonder a bit whether the footer would look better with a light-gray background, to set it a bit more apart from the rest of the page, now that it's more substantial.

@brockwhittaker ping on my last feedback.

Updated!

OK, a few requests:

Can you add the template conditionals back?
Can fix the centering issue that we discussed?

I think once those are good, we can merge and then iterate from there.

This is updated, @timabbott.

@timabbott, updated again.

Merged, after a few tweaks to:

Edit the links
Add u18n
Use {{ server_uri }} for the things we really want to go to the main page.  We might want to rename that field to avoid future confusion.

Thanks @brockwhittaker!

@rishig what do you think about changing server_uri to server_root_uri, to make clear it's the URL for the root of the subdomain?  @brockwhittaker do you think that name change would have helped make this more clear?

Nah, I don't necessarily think it increases understanding; or at least the word root doesn't do so, because root to me just means /, not necessarily subdomain.zulip.com vs zulip.com.

OK let's go with root_domain_uri.

OK, merged 126171cb762a4e198baa4793c4fe7939630b073c to fix this.  Thanks folks!


Add support for Zooz Z-Box Hub

[Z-Box Hub](https://zboxhub.com/) uses API version 5. This PR enables scenes to work properly for these devices.
I tested this by making a local patch to the pyfibaro library and verified that the correct REST requests were being sent to my Z-Box and that the scenes operated correctly.

Many thanks for your PR. There is no public information available about the API of the Zooz Z-Box Hub, but it is for sure some software from Fibaro into the box. Let's hope that they keep the compatibility of the API.


fix: add sentry dsn to next env config

:tada: This PR is included in version 1.18.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added ceil and floor functions

These come handy if you do something like:
margin-top: floor(@num/2);
margin-bottom: ceil(@num/2);
Where @num could be an odd number.

thanks, this is in now.


libkbfs: store the full Merkle root in the MD update

Instead of just the seqno, which can be predicted/guessed.
This PR assumes we want to include the merkle root in the MD ID forever.  If this goes in, a future PR (for KBFS-2271) will add an "eventually consistent" caching layer for the merkle root to avoid a server RTT when setting this fields during foreground writes, in most cases.
Issue: KBFS-2296

I had to change the merkle root field to be a pointer, to get the right omitempty behavior.


fix(auth): Allow users to delete account even without org

This patch allows users to delete their accounts even if they don't belong to any organization.
Closes #998

this patch seems to have two things, about deleting the account and about seeing the context.
Are these changes related?

Yes, the CLI when trying to delete first, tries to get the current context of the user https://github.com/chainloop-dev/chainloop/blob/d458117ae53c556aca16e7a3ca8e0dc89287724e/app/cli/cmd/auth_delete_account.go#L34 to later on, delete the account:
https://github.com/chainloop-dev/chainloop/blob/d458117ae53c556aca16e7a3ca8e0dc89287724e/app/cli/cmd/auth_delete_account.go#L47


Temporarily duplicated pieces due to racing updates to currentPosition.

Consider the "Play Random Computer" example from https://www.chessboardjs.com/examples#5001
First, the following bug can be easily reproduced locally by exaggerating the effect with the following changes:

reduce the makeRandomMove delay to 1 (window.setTimeout(makeRandomMove, 1)
increase the move speed to 1000 (in the board config, set moveSpeed: 1000)

As a result of the above, you'll see duplicated pieces (for the duration of the animation) whenever the computer makes a move.
This happens because of these racing currentPosition reads/writes:

The player makes a move (drops the piece) which calls onDrop and schedules the "snap" animation to run after config.snapSpeed (30ms by default). That is, after 30ms, Chessboard will drawPositionInstant() with currentPosition. Usually currentPosition is whatever was on the board when the player dropped the piece and everything is correct in that case.
However, in the example above (or whenever one attempts to implement a computer opponent), when the player drops a piece, onDrop is called which: calculates the new move (can be faster than 30ms), calls board.position() (potentially queued with setTimeout), and starts an animation for the computer move. Note that board.position() updates the currentPosition immediately.

As a result, if board.position() is called in the 30ms snapSpeed time window after player's move, the player gets to see two versions of the piece moved by the computer:

one because of the animation moving the piece (this is expected)
one because when snap animation finishes, it sees the new position that happens only after the computer move but the snap animation callback intended to only draw the position that happened after the player's move (i.e. before the computer gets to move their piece).

I attached a screenshot. I played 1.d4 and the computer from your example replied 2...,g4 and now the black g-pawn is duplicated: one is on g4 and the other one is moving from g2 to g4).

As mentioned at the start of this post, you can easily reproduce it locally by changing  makeRandomMove delay to 1 and config.moveSpeed to 1000 in your example at
https://www.chessboardjs.com/examples#5001
One workaround is to set config.snapSpeed to 0 and disallowing dragging pieces while computer move is being animated.

I have the same problem

I added delay whenever the computer makes a move, it is fix duplicated pieces.


Document (and automate?) release process

Write some documentation on how to release a new version.
Maybe use something like semantic release to automate the whole process in the first place?

#84 adds semantic-release

Release process is fully automated with semantic-release.


Add CIFuzz

OSS-Fuzz was wondering if wuffs would use CIFuzz, a new feature being rolled out
in OSS-Fuzz. It allows for pull requests to be fuzzed in CI using fuzz targets enabled
in OSS-Fuzz. Documentation for this feature can be found at https://google.github.io/oss-fuzz/getting-started/continuous-integration/
Let us know if you have any questions!

Thanks!


Publish image on docker hub

Would you be willing to do this? You already publish an image for your openvpn exporter

Hi,
Yes, I agree, it will be nice to have it on DockerHub. We already have some images on there but before we add any more we would like to set up a CI pipeline on our side for open source images like this one. Now things are a little bit unorganized and hard to keep track.
I will leave this issue as open for now. We will be working on this tomorrow or next week so updates will come shortly.

Hey @Ivaylo-Korakov, Any updates on where is this at?

@Ivaylo-Korakov I would recommend just start using the docker hub builder to start with.
I have shared libraries for jenkins if you want those instead


Fixing Oculus Deployment Issue, whitespace error

Overview
When deploying to Oculus the 618 error will still pop up, this adds that code to the csc to fix that issue.
Fixed some whitespace that was introduced accidentally.

/azp run

/azp run


Fixes reference to undefined

Fixes https://github.com/webdriverio/wdio-sync/issues/51

@jmnsf could you please add unit tests for is$$Result?

@christian-bromann I would, normally, but the function isn't exported. Also, I would wager that an array of undefineds might be symptom of a bug elsewhere.
Honestly this is more of a hotfix since the exception is breaking my test suite through no fault of my own. Worse, it's a package that's depended on by wdio-mocha-framework which makes it a bit of a pain to pull into fix through git installs...
Feel free to edit this PR and add tests that handle this, but I would really appreciate a fast merge on what is essentially a defensive programming patch.

but the function isn't exported

Let's export it into a helper file then.

I would really appreciate a fast merge on what is essentially a defensive programming patch.

I agree with you .. however I see a value in having this covered in unit tests. I can take care of it but you will have to wait until I have time to work on that

@christian-bromann then if you don't get to it in the meantime, I'll have a look later today

@christian-bromann Done. Had to update the rewrite plugin because of a bug.

@christian-bromann ready

Will release this version with updated framework adapter later today

Awesome, thank you ;)

@christian-bromann hey, could we have this published soon, we're stuck on older package versions until then :)


Develop plan for metrics reporting

[ ] Collect list of metrics desired for the Invenio dashboard
[ ] Determine which metrics can be obtained using existing REST API

Initial list of metrics:

Total deposits
No. deposits tracked over time
No. views per deposit
No. downloads per deposit
Average views per deposit + over time
Average downloads per deposit + over time
List of top deposits by download
No. deposits by member
No. deposits by identified institution (ROR)
Average no. deposits per user + over time
No. deposits removed over time
List of users with most deposits
Eventually moderation statistics here too

Just a quick note: it seems that "stats" also consider "versions."
"stats": {
                "all_versions": {

                    "unique_views": 0,

                    "views": 0,

                    "unique_downloads": 0,

                    "downloads": 0,

                    "data_volume": 0.0

                },

                "this_version": {

                    "unique_views": 0,

                    "views": 0,

                    "unique_downloads": 0,

                    "downloads": 0,

                    "data_volume": 0.0

                }

           }"

Should we take that into consideration?

Cassie's summary on invenio stats.


[Feature] Ignore First Comment

Is your feature request related to a problem? Please describe.
As many companies do, my company insists on us having copyright license headers at the tops of all of our files. This breaks the functionality of the tool since the license header is an HTML comment just like the tool is looking for to define the parameters for the file.
Describe the solution you'd like
It would be nice to have an option to skip the first  HTML comment in the file which would accommodate license headers.
Describe alternatives you've considered
I can only think to remove the license headers to make it work, which is obviously not ideal.
Additional context
It's pretty straight forward 🤷♀ This is a great tool though!

Just today I finished an internal script which takes care of publishing my documentation to our internal Confluence upon push into main.
My markdown files do not contain the space nor the parent information and start directly with h1.
All the mark-stuff is put at the front before the file is published.
Maybe this is an option for you as well? Creating a publish script which takes care of putting all the configuration stuff at the top of the file?

Just today I finished an internal script which takes care of publishing my documentation to our internal Confluence upon push into main.
My markdown files do not contain the space nor the parent information and start directly with h1.
All the mark-stuff is put at the front before the file is published.
Maybe this is an option for you as well? Creating a publish script which takes care of putting all the configuration stuff at the top of the file?

Hmm yes, that could be an option! This would still be a nice feature to have though :) But thanks for sharing the idea!

Easiest would be to introduce a new header like:
<!--# this will be ignored by mark -->

or, more specific to your use case
<!-- Copyright: …

it likely can be solved with a more generic solution like: https://github.com/kovetskiy/mark/issues/317

The generic solution is cool but unfortunately wouldn't work in my specific case, I don't think. We have automation that looks for the license header at the tops of our files. And that same automation will inject it at the top of it isn't found there. That's why a simple "ignore the very first comment you find" would have been nice, since I think it's quite common for license headers to be one big html comment block:
<!--

    Copyright (c) 2011-present blahblahblha
    blahblahblah
    ...

-->

But I can see how the generic version would be sufficient for most.


Is the react-charts free?

Is the react-charts free? Kindly confirm and add a note of pricing term in readme.me file, if any. Thanks a lot.

Yes It is, it has an MIT license

Due to a massive overhaul and rewrite of React Charts to TypeScript and a massively upgraded public API, this issue is being bulk closed for triage. The latest version can be tested via the react-charts@beta tag. Documentation is in the process of being upgraded, but thankfully, everything is fully typed now, so you can explore the API that way for now.
If, after testing or playing with the v3 beta, you believe your issue is still not addressed or relevant, feel free to open a new issue.


drivers: adc: ads101x: add driver for TI ADS101x

The ADS1014 and ADS1015 devices (ADS101x) are precision, low-power,
12-bit, I2C compatible, analog-to-digital converters (ADCs).
The ADS101x devices incorporate a low-drift voltage reference and an
oscillator. The ADS1014 and ADS1015 also incorporate a programmable
gain amplifier (PGA) and a digital comparator. These features, along
with a wide operating supply range, make the ADS101x well suited for
power and space-constrained, sensor measurement applications.
Signed-off-by: Matija Tudan<EMAIL_ADDRESS>

I've actually already got a pull request I'm working that will include these devices (as well as others within it's family).
Can you take a look at https://github.com/zephyrproject-rtos/zephyr/pull/40477

Hey @XenuIsWatching, sorry I missed that. In that case, I think we can close this PR. :)

Would you be able to try out https://github.com/zephyrproject-rtos/zephyr/pull/40477 on your build/board?


How about using the kotlin to develop flutter?

Is it possible to develop flutter with kotlin in the future?

build castles in the air

Hi @yangwuan55
this doesn't appear to be an issue with Flutter
and I struggle to see it as a proposal for improvements,
there are several reasons why Flutter uses Dart instead of
Kotlin, JS, Java etc...
and have been extensively discussed on many platforms,
here's one.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
thank you


vagrant executor support

Why are these changes needed?
in same case for example kernel driver,  install kernel driver is risky for host.
vagrant executor can be more safe than docker executor in this case.
Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Can you please add tests and fix the failing pre-commit check?
You can get started with pre-commit here: https://pre-commit.com/#install


cross-env NODE_ENV=development node server Fails

Error says: 'cross-env' is not recognized as an internal or external command, operable program or batch file.
npm-debug.log:
0 info it worked if it ends with ok
1 verbose cli [ 'C:\Program Files\nodejs\node.exe',
1 verbose cli   'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js',
1 verbose cli   'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info prestart<EMAIL_ADDRESS>6 info start<EMAIL_ADDRESS>7 verbose unsafe-perm in lifecycle true
8 info<EMAIL_ADDRESS>Failed to exec start script
9 verbose stack Error<EMAIL_ADDRESS>start: cross-env NODE_ENV=development node server
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter. (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:217:16)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at EventEmitter.emit (events.js:172:7)
9 verbose stack     at ChildProcess. (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at ChildProcess.emit (events.js:172:7)
9 verbose stack     at maybeClose (internal/child_process.js:827:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
10 verbose pkgid<EMAIL_ADDRESS>11 verbose cwd C:\Users\chcarter\Desktop\react-boilerplate
12 error Windows_NT 10.0.10586
13 error argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "start"
14 error node v4.4.5
15 error npm  v2.15.5
16 error code ELIFECYCLE
17 error<EMAIL_ADDRESS>start: cross-env NODE_ENV=development node server
17 error Exit status 1
18 error Failed at the<EMAIL_ADDRESS>start script 'cross-env NODE_ENV=development node server'.
18 error This is most likely a problem with the react-boilerplate package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     cross-env NODE_ENV=development node server
18 error You can get information on how to open an issue for this project with:
18 error     npm bugs react-boilerplate
18 error Or if that isn't available, you can get their info via:
18 error
18 error     npm owner ls react-boilerplate
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]

did you install cross-env with npm?

'cross-env' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "dev"
npm ERR! node v6.11.0
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>dev: `cross-env NODE_ENV=development npm run webpack-dev-server -- -d --colors --hot`
npm ERR! Exit status 1
npm ERR!

I've done npm install, and my npm/node versions are mentioned above? Any advice?

Put && before 'node server' (2nd command)
On 08-Aug-2017 10:32 AM, "Mohammad Ali Raza"<EMAIL_ADDRESS>wrote:

'cross-env' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "dev"
npm ERR! node v6.11.0
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>dev: cross-env NODE_ENV=development npm run webpack-dev-server -- -d --colors --hot
npm ERR! Exit status 1
npm ERR!
I've done npm install, and my npm/node versions are mentioned above? Any
advice?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/react-boilerplate/react-boilerplate/issues/513#issuecomment-320850729,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANpzLNgODGrqGjxzlVuaOljxtr5wN-tgks5sV-vygaJpZM4IyK5O
.


db_dialect undefined

The docs on the Sequelize migration setup refer to a key called "db_dialect".
const app = require('../../src/app');
const env = process.env.NODE_ENV || 'development';

module.exports = {
  [env]: {
    url: app.get('db_url'),
    dialect: app.get('db_dialect'),
    migrationStorageTableName: '_migrations'
  }
};

When I run things as documented I get the following:
root@7fd93358ce33:/opt/api# sequelize db:migrate

Sequelize [Node: 6.9.5, CLI: 2.5.1, ORM: 3.30.2]

Loaded configuration file "migrations/config/config.js".
Using environment "development".
Error: The dialect undefined is not supported. Supported dialects: mariadb, mssql, mysql, postgres, and sqlite.

I ended up adding a key in my default.conf for the db_dialect and then it worked.  I also changed the db_url key to the actual key name of the DB url that I have in default.conf (postgres).
Things worked afterwards - just wondering if I'm doing it right?

Sounds about right. Can you update the document accordingly in a pull request?


Cell statusbar API

Fix #105809
Much better implementation of the cell statusbar API.
Open questions/todos

Do extensions need to show items that are only visible when the cell is active? Should I just call the providers again when the cell becomes active?
Need to debounce statusbar updates when scrolling?
Any other cases when providers should be called? The only case when the provider is canceled is when the item scrolls out of view during an update, is that right?
The service is kind of pointless here, would it be better to just expose a list of providers, basically a provider registry?
Get other internal things onto this API, and remove statusMessage metadata
Fix bugs like https://github.com/microsoft/vscode/issues/115099 and https://github.com/microsoft/vscode/issues/112772

Need to debounce statusbar updates when scrolling?

It would be great if we debounce it a bit as while scrolling / cell resizing (as they have dynamic height), a cell can be created, disposed and then re-created within a short period.

a cell can be created, disposed and then re-created within a short period.

What do you mean?
The downside of delaying it is that you will see cells without status bar items briefly, then the status bar items will blink into existence, and I found it annoying when testing.

Fixed the issue with dupe items showing up, I thought a cancel token would be canceled on dispose by default.

The service is kind of pointless here, would it be better to just expose a list of providers, basically a provider registry?

Services are always better than singletons, tho that's all they are: better singletons


Change design. Fixes #74

Hi @fedealconada ,
Thanks for taking the time to also work on the design stuff :)
I see that this PR still includes the merged commit from the previous commit. This should be fixed by syncing your repo and rebasing your dev branch:
# Add Etcher remote
$ git remote add upstream git@github.com:resin-io/resin-etcher.git

# Incorporate latest changes into your fork
$ git checkout master
$ git pull upstream master

# Go to your dev branch and incorporate the latest changes there
$ git checkout dev
$ git rebase master

# Update PR
$ git push --force

Let me know if you run into any issue!


Sections

Showcase and useful sections

@sabanyelkenci  now the main is merged branch is up to date but the previous part are gone. It is almost empty page now?

@sabanyelkenci now the main is merged branch is up to date but the previous part are gone. It is almost empty page now?

Hi @erkamguresen. We can review the code together this evening. I tried to use bootstrap but as you know I am not so experienced with bootstrap.

Hi @sabanyelkenci there are some tutorials that help to make it clear about bootstrap
I mentioned them in your navbar PR

https://youtu.be/1v_XcJsGtyA

https://www.youtube.com/watch?v=4sosXZsdy-s


fix(webpack): adds less-loader

adds less-loader for Less
adds dependencies for less-loader

@gastaldi pls test if it helps with ngx-forge 0.1.9

These changes looks good to me

we don't use less so this change should be added to fabric8-analytics-dependency-editor instead


v0.3.0-alpha.16 inoperable with likely RegExp bug affecting imports

Instructions
Please fill out the template below to the best of your ability and include a label indicating which tool/service you were working with when you encountered the problem.
Problem
All tests fail with:
[Error Code: 1101] cadence runtime error Execution failed:\n' +         'error: [Error Code: 1054] location (./MetadataViews.cdc) is not a valid location: expecting an AddressLocation, but other location types are passed
This error occurs when calling any transaction.
Could be an issue with RegExp not matching properly.
Steps to Reproduce
Ensure version is<EMAIL_ADDRESS>Run any test that involves a transaction that uses imports.
Acceptance Criteria

Context
Testing of project

Hello,
I'm having the same issue when sending transactions. It seems that the framework don't succeed in resolving imports.

Flow CLI: 0.41.0
node: v18.9.0
flow-js-testing<EMAIL_ADDRESS>

Same problem here.
The issues seems to occur only for the latest version: v0.3.0-alpha.16
I also tested with v0.3.0-alpha.15 and that works fine.

@gregsantos could this be related to what was seen in #182?

PR #190 will fix this (exact same fix as #186 but with executeScript instead of sendTransaction this time).

Update to the latest version v0.3.0-alpha.17, which fixes the issue.


Add XCM SDK docs

Description
This PR adds the new XCM SDK docs! 🎉  These are versioned docs, so the docs for v0-v2 are all available in one central place.
For mkdocs, versioning is made possible through mike, which works by creating a new Git commit on the gh-pages branch every time a new version of the docs is deployed using mike deploy commands (see mkdocs/README.md). When deploying a particular version, previously-deployed docs for that version are overwritten, but docs for other versions remain untouched. So we'll need to make sure that the gh-pages branch is always open and never manually modified.
🔁 The current workflow can be summarized as:

Make changes and open a PR to be merged into the main branch
Run the mike deploy commands locally, which will push up changes automatically to the gh-pages branch of your fork
Open a PR to merge your fork changes into the gh-pages branch

There may be a better way to automate or handle this, but at least with this flow, we'll be able to dictate whether changes result in updating the latest version or publishing a new version.
✅ For initial setup:

Go to the settings page of the repo
Under code and automation, click Pages
For the Build and deploy settings, the source should be set to Deploy from a branch and the Branch should be set to gh-pages

Anytime we merge new version changes to the gh-pages branch, it will automatically deploy and publish those changes to https://moonbeam-foundation.github.io/xcm-sdk/ 🚀
👀  To see how it currently looks, you can checkout my test page: https://eshaben.github.io/xcm-sdk-mkdocs/latest/
Checklist

[ ] If this requires a documentation change, I have created a PR in [moonbeam-docs] (https://github.com/moonbeam-foundation/moonbeam-docs) repository.
[x] If this requires it, I have updated the Readme
[ ] If necessary, I have updated the examples
[ ] I have verified if I need to create/update unit tests
[ ] I have verified if I need to create/update acceptance tests
[ ] If necessary, I have run acceptance tests on this branch in CI

thanks for the review @ekenigs! applied your feedback 🙂
I'll need to go back through and redeploy v1,  but I'd like to do that after I get a review of the changes I just pushed, so I only have to redeploy once.


Add CI job to push Docker image to Docker Hub

Fixes #31

@ehildenb or @F-WRunTime, please add DOCKERHUB_PASSWORD to secrets.
https://hub.docker.com/r/runtimeverificationinc/kontrol might also need to be set up for docker image push to go through.

@F-WRunTime, please add DOCKERHUB_PASSWORD to secrets.
https://hub.docker.com/r/runtimeverificationinc/kontrol might also need to be set up for docker image push to go through.

done

@tothtamas28 we also need to be:

Publishing the Nix binaries to the public nix cache on master pushes.
Cutting a GitHub release with the version tag (so that we can do kup install kontrol --version VERSION_TAG).

@tothtamas28 we also need to be:

Publishing the Nix binaries to the public nix cache on master pushes.
Cutting a GitHub release with the version tag (so that we can do kup install kontrol --version VERSION_TAG).

Addressing the credentials needed for this. CACHIX_PUBLIC_TOKEN is available for pushing to the public cachix binary store.

we also need to be

Opened issues:

https://github.com/runtimeverification/kontrol/issues/36
https://github.com/runtimeverification/kontrol/issues/37


Upgrade ExoPlayer to 2.18.1 on v6.0.0-alpha.1

Feature Request
Google released a major ExoPlayer version r2.18.0 and a patch r2.18.1
Between both versions, there are some DRM improvements like (https://github.com/google/ExoPlayer/issues/10274) and several stability fixes
Why it is needed
To avoid big refactors when upgrading many versions it's better to upgrade as soon as a major version drops
Several stability and bugfixes for all projects depending on this one
Possible implementation
I'll handle the upgrade, basically, it's upgrading the dependencies and fixing some Track-related deprecation problems

I also looking for updated version of ExpoPlayer in react-native-video to have a try with latest<EMAIL_ADDRESS>to support both audio and video media with full features and controls.

Me too. A lot of us are using both repos (react-native-video) and (react-native-track-player), with react native 0.68 (or newer) based projects. There are many issues also on rn-track-player repo (see rntp#1678, rntp#1694).
It's a common need have this package works on the same version.
Hope this version gap will be filled shortly, not sure if this point resolution is planned to be resolved with the stable version 6.0 of this package.

PR : https://github.com/react-native-video/react-native-video/pull/2846
Thank you !

will be available in 6.0.0-alpha.2


Update CHANGEME.md

yolo

Ahoy, @Heidet! To earn the YOLO badge, please follow these steps. Let me know if you have any questions.

Seems to be a duplicate of #929.


Simplify Dict code, and eliminate calls to Debug.crash

removing redundant bindings
inlining/simplifying case branches
removing an unused, and unexported, function
turn a potentially crashing auxiliary function into one that returns Maybe

Replaces https://github.com/elm-lang/core/pull/254.

This looks good to me! When you say "potentially crashing" do you mean "given arbitrary inputs, it can crash" or do you mean "as it is used in this library, it can crash"? If it is the former, is it best to have a silent failure/bug or to immediately say something to the user?

I meant that the function contained a call to crash, so with arbitrary inputs it could crash. But it is called only inside this library, and for the calls in the library an invariant holds that the crashing case cannot occur.
As long as the function remains internal, I think the current use with withDefault at the use site of max in the rem function is the best way to go. If the function were to become used more widespread, then probably changing its return type from Maybe to Result would be best. Then the Err case of the result could carry the current error message string. Even in such a setting, using the Result type would certainly be better than a call to crash?
(Maybe the best idea is to turn that Maybe into Result right away, and add a call to Result.toMaybe at the use site in my refactoring.)

I merged the master branch into this PR to make sure it still builds. It does, and is still an improvement IMO.

Just a walkthrough of the changes in this pull request (since the comments above do not anymore reflect exactly what happened here overall):

The min function was deleted. It wasn't used in the module, nor exported.
The max function was changed from having type Dict k v -> (k, v) and throwing a Debug.crash in one case to having type Result String (k, v). The function is only used inside the module and present, and its use there is unaffected by this "change" (see below). If the function ever gets exported from the module, signalling error through Result String is certainly better than just crashing. (The consumer of the result might still decide to crash on receiving the Err value, of course.)
In the rem function, there was a pattern matching on (l, r) vs. (RBNode cl kl vl ll rl, RBNode cr kr vr lr rr) and later bindings of l and r to exactly RBNode cl kl vl ll rl and RBNode cr kr vr lr rr. Those bindings were clearly superfluous, one can as well pass the original l and r on to where they are further needed. So the bindings were deleted.
What remains was a call of max on the l from the previous bullet point.  This, I replaced by withDefault (kl, vl) (toMaybe (max rl)). This is justified by the definition of max, which has the cases:

max (RBNode _ key value _ (RBEmpty _)) = Ok (key, value)  -- previously, simply (key, value)
max (RBNode _ _ _ _ right)             = max right
max (RBEmpty _)                        = Err "..."        -- previously, Debug.crash "..."

That clearly makes the above a semantics-preserving transformation. (Because max (RBNode cl kl vl ll rl) will first look at whether rl is an RBEmpty, in which case (kl,vl) are to be returned, just as will happen in the new call withDefault (kl, vl) (toMaybe (max rl)) in that case. If rl is not an RBEmpty, then both the original call and the new call will return what max rl returns -- which is known to never hit an RBEmpty if rl itself was not an RBEmpty.)
(Note: Just for the explanation above, I have abbreviated all RBEmpty_elm_builtin and RBNode_elm_builtin to RBEmpty and RBNode.)

Do you have any idea what this costs in performance? Perhaps it's not smart to be worrying about this right now, but I have a sense that we will be wanting to make Dict and Array as fast as possible, and it will sometimes be worth it to do weird stuff within the module itself. I guess my concern here is that we are maybe paying quite a lot for this change but not getting a ton back.
I need to get https://github.com/JoeyEremondi/elm-benchmark set up for Joey's ongoing compiler work, so maybe it'd be a good time to start setting up some benchmarks for data structures like this. I'd be curious to see how much we gain by removing a ton of closures from the compiled code. So my vote would be to do some exploration in this direction before going with something that could be quite a bit slower.

Well, part of my change actually removes work: the superfluous reconstruction of trees l and r in those bindings I removed.
Other than that, you are concerned about the Ok constructor? I have no idea whether/what impact that may have on performance.

Closing this. May make a smaller pull request with just the obvious improvement (and leaving the elimination of min to dead code detection).


Ability for service to respond to .finished events

Is your feature request related to a problem? Please describe.
My services triggers due to an evaluation.finished. My service is technology specific so doesn't live in the shipyard file.
Describe the solution you'd like
My custom services needs to response to a .finished event and I need it's workflow shown in the Keptn's bridge.
Note that my do work phase relies on the output of evaluation.finished so I cannot just respond to the evaluation.triggered instead.
Alternatives
I've tried listening for evaluation.triggered and sending my-service.started. This works and I see my-service.started in the bridge. But then the evaluation.finished fires and Keptn automatically finishes my-service. This means that I never get to see the message or result or my my-service.finished in the bridge.
I have important information in this output which needs to be presented to the user in the bridge.
Additional context
Workflow. Key here is that my-service.started and my-service.finished appears under the sequence trace inside the evaluation workflow to show that something
evaluation.finished
                               >> my-service.started
                                   >> do work
                                       >> my-service.finished
                                           >> Important information sent in the message which must be displayed in the bridge

Can you give us some insights into what your services is doing? does it have to send a start & finished event or can it simply do its task without reporting back, e.g: for notification integrations (such as slack) - we also dont send back start/finished as it is not necessary

This particular service creates a JIRA ticket so the return value (that needs to be presented in the bridge) is the hyperlink to the ticket value.
Furthermore, if the bridge & sequence list is the "source of truth" for everything that happens in Keptn then I believe we should see that a service has executed due to a response. Otherwise, if I was new to the a project and looked at the bridge, how would I even know that a slack message had been sent (for example)? Perhaps that is visualised elsewhere, but as it is, you'd need to know what's listening to what to have that "full picture".

Hi @agardnerIT, your use case can be solved by having your integration listening to both evaluation.triggered and evaluation.finished. When it receives the .triggered events, it answers with a .started event and waits until it receives the evaluation.finished event from the lighthouse-service to answer with its .finished event on its own that contains the JIRA ticket URL.
Feel free to reach out to me over our public slack if you need further help


Release a new version

There has been a bunch of updates, but the most recent release (0.2) is from august 2015.
A new release would be awesome! ⛵

The most recent release is https://www.npmjs.com/package/@mapbox/polyline, from December 2016, and has all the changes in the repository as of right now.

Thanks for answering @jfirebaugh! Much appreciated!
What do you think about adding timestamps + version details to the changelog?
Plus a bump to the version number (according to the cited npm page the lastest version is still 0.2, but that one is from august 2015 according to the git history in this repo).


Update hide column about IoT packages in csv files

Updates：

Update hide column about IoT packages in CSV files by email CSV Updates.
Update the display name of the package  Azure.Iot.DeviceUpdate and  Azure.Iot.ModelsRepository  in four languages.

Questions:
In this PR, I changed hide to true about package azure-iot-deviceupdate in java according to the email. But, I wonder if the package azure-iot-deviceupdate in java needs to be hide? This package in the other three languages is not hidden.
@jongio for notification.

I added elena  @tzhanl
LGTM

@elraikhm, @weshaggard -  I have updated according to your comments. If there is no problem, please help merge it.

@elhorton I'm going ahead and merging this PR as it looks reasonable to me and these PR's are hard to keep updated. If there are other changes we can file another PR.


"Buffer.from is not a function" when using ESM package

Hey, I'm trying to use web3 from ESM
import Web3 from "https://esm.sh/web3"

export default function Home() {
  return null;
}

But when loading the page I'm getting this error
TypeError: Buffer.from is not a function
    at varintEncode (file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:31889:23)
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:31900:37
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:69:9
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:31930:33
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:69:9
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:34701:18
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:69:9
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:34738:23
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:69:9
    at file:///.../.aleph/development.es2018/-/cdn.esm.sh/v11/web3@1.3.1/es2018/web3.development.js:34833:23

@hazae41 thanks, i will fix it ASAP.

@hazae41 i will depoly the bugfix tmw

Thanks!

@hazae41 deployed, please try deno cache --reload https://esm.sh/web3

Hey, it seems https://cdn.esm.sh/ is down

i'm not sure what happened, i will try to restart the server.

@hazae41 fixed

I have the following error when using aleph dev (after aleph upgrade)

error: Uncaught (in promise) ReferenceError: Cannot access 'fromFileUrl14' before initialization
at https://cdn.esm.sh/v12/_node_fs.js:363:1903

@hazae41 sorry i didn't check the deno bundle output, i just reverted the fs polyfill, pelase try again, thanks
aleph dev --reload

aleph dev --reload doesn't seem to work

It just shows the same error

please refresh the deno cache, v0.3 will share cache with deno so reload option will work fine.
deno cache --reload<EMAIL_ADDRESS>

It works! Thanks!

@hazae41 thank you


Added support for Docker, GitHub Actions and graceful shutdown

Related #624 (missing setup login and secrets of registry), #627 (missing cleanup TravisCI)


Project Not Found Error

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior:
Hello whenever, I enabled Ftrack Leecher and Processor for ftrack.
Ayon Server report Project Not Found Error for project that are not linked to AYON at all.
Please advise.

Expected Behavior:
No response
Version
1.0.0
What platform you are running on?
Windows, Linux / Ubuntu
Steps To Reproduce:
To Reproduce just connect Ayon with Ftrack and start the services.
I'm using Ayon-Ftrack Addon version 1.2.0, which is the latest version at the moment.
Are there any labels you wish to add?

[X] I have added the relevant labels to the bug report.

Relevant log output:
DEBUG      anonymous       [GET] /projects/XXXXXX : Project XXXXXX not found

Additional context:
No response

Hello @iLLiCiTiT, sorry for the late reply.
Since those project are not linked to AYON, there shouldn't be any code running or querying AYON about those project. Which means that the DEBUG statement shouldn't print out.


Upload File command should show an error if upload fails

Your environment
SuiteCloud Extension for Visual Studio Code v1.0.3
Describe the bug
If the Upload File command fails (for example due to a syntax error in the file), the VS Code extension still displays the message, "'Upload File' has been successfully executed").
To Reproduce
Create a suitescript file with a syntax error, and then try run the command SuiteCloud: Upload File
Actual Behavior
The file fails to upload, however, this is not reflected in the message.

Expected Behavior
It's expected that if the file upload failed, an error message should be shown instead to inform the user of such.

This is not a bug. The pop-up just informs that the command execution has finished successfully. See details for more information.
But I understand this can be misleading. We will see if this can be enhanced.


Remove-PSDrive don't work

Steps to reproduce
New-PSDrive -Name "J" -PSProvider FileSystem -Root "\\ip\temp" -Persist
Remove-PSDrive -Name "J"

Expected behavior
Delete "J"

Actual behavior
Remove "J" don't work.

Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      5.1.17134.112
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17134.112
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Just to be clear!
This Github repository is for PowerShell Core only. Any Windows PowerShell issue should be address thru the UserVoice forum: https://windowsserver.uservoice.com/forums/301869-powershell
As is stated in the PowerShell Core landing page: https://github.com/PowerShell/PowerShell#windows-powershell-vs-powershell-core
Now, I would recommend to test the issue on a later Windows 10 build (> 16299) or Insider edition as sometimes seen that Windows PowerShell issues has been corrected on later builds.
:)

Now, I did test both Windows PowerShell and PowerShell Core.
I can't replicate the condition.  I see no issues in either PowerShell.
Please, make sure the shared folder is accessible in order for the cmdlet to work.

PS [15] > $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.1.0-preview.3
PSEdition                      Core
GitCommitId                    v6.1.0-preview.3
OS                             Microsoft Windows 10.0.17686
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@MaximoTrinidad  Thanks your information.

Can we reopen this as it IS an issue, it's just missing a one peice of information. Add '-scope Global' on the 'New-PSDrive' command and the Remove-PSdrive command will not work.

@VortexUK
Better off opening a new issues and complete the required information, like which version of PowerShell Core you are working and which system is running on: Windows, Linux, or Mac.
Your comment doesn't related to the original issue posted initially. Please, include sample code to try to reproduce the issue you're experiencing and what  the expected results.
Also, just a friendly  reminder, that all Windows PowerShell issues are not addressed in this repository.  This is only for PowerShell Core.
Thanks!


ENTSO-e not showing as integration

I've added the repository via HACS, downloaded the component to custom_components, restarted HomeAssistant, but can't find the integration when trying to add it.
Is it possible to configure this integration via the UI?

Yes, this integration is in HA.
After installing it through HACS and restart HA, you should normally find it when adding an integration 'ENTSO-e Transparency Platform.'
Could you find it in HACS?

Apparently browser cache indeed was the issue. So sorry for that!


HAWQ-558. The query running on random table does not push resource manager to request resource from YARN as maimum possible resource consumption

+1

+1


Language for Labels

I tried this ontology: http://www.ontologydesignpatterns.org/ont/dul/DUL.owl
with both English and Italian labels...and it seems to include both in the output:

Can  we specify the default language?
And/or display both with small "lang" tags?

I have the same issue with my own ontolgy. Would be wonderful if PyLODE would be fully multilingual.
Thank you for providing this wonderful tool, by the way.


⚠ PrivateHD has degraded performance

In 018533d, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1644 ms

Resolved: PrivateHD performance has improved in 6ffca1f.


Dump webpack stats to a subdirectory

Make it easier to ignore but restore the output of stats for the web task.
Finishes: #145204839

          1 Warning

:warning:
Please provide a linked tracker story for this PR in the description

  Generated by :no_entry_sign: danger

ping @jayzes
Want to merge this one?


Dryscrape to login via Facebook.

I'm trying to use Dryscrape to login via Facebook. But I get these error.
Traceback (most recent call last):
  File "/Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/webkit_server.py", line 420, in __init__
    self._port = int(re.search(b"port: (\d+)", output).group(1))
AttributeError: 'NoneType' object has no attribute 'group'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "facebook_scraper.py", line 40, in <module>
    sess = dryscrape.Session(base_url = 'https://www.facebook.com')
  File "/Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/dryscrape/session.py", line 22, in __init__
    self.driver = driver or DefaultDriver()
  File "/Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/dryscrape/driver/webkit.py", line 30, in __init__
    super(Driver, self).__init__(**kw)
  File "/Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/webkit_server.py", line 230, in __init__
    self.conn = connection or ServerConnection()
  File "/Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/webkit_server.py", line 507, in __init__
    self._sock = (server or get_default_server()).connect()
  File "/Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/webkit_server.py", line 450, in get_default_server
    _default_server = Server()
  File "/Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/webkit_server.py", line 427, in __init__
    raise WebkitServerError("webkit-server failed to start. Output:\n" + err)
webkit_server.WebkitServerError: webkit-server failed to start. Output:
dyld: Library not loaded: @rpath/./libQtWebKit.4.dylib
  Referenced from: /Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/webkit_server
  Reason: image not found

Here's the code I'm using.
import dryscrape

# make sure you have xvfb installed
dryscrape.start_xvfb()

# set up a web scraping session
sess = dryscrape.Session(base_url = 'https://www.facebook.com')

# we don't need images
sess.set_attribute('auto_load_images', False)

# visit homepage and search for a term
sess.visit('/')
q = sess.at_xpath('//*[@id="email"]')
q.set('email')
q = sess.at_xpath('//*[@id="pass"]')
q.set("password")
login_button = sess.at_xpath('//*[@id="u_0_x"]')
login_button.click()

# save a screenshot of the web page
sess.render('facebook.png')
print("Screenshot written to 'facebook.png'")

The login field has an id of u_0_v not u_0_x like you have in your code.
Try this:
login_button = sess.at_xpath('//*[@id="u_0_v"]')

It sounds like you have a problem with your Qt installation. I just used Homebrew: brew install qt and it worked. You potentially have to install dryscrape after this.
What @trendsetter37 said might also be a problem, especially if the ID is randomized, in which case you need a different XPath expression to select the button.

@niklasb Yea I have ran into that before with randomized id's on login pages. However, I checked Facebook's login button and it was static as far as I could tell.

@trendsetter37 Thanks for the reply. I uninstall dryscrape using pip uninstall dryscrape and ran brew install qt and reinstall dryscrape again but I still see the same issue.

Hi @noppanit. I haven't seen this error before. Which version of Mac OS X are you using? And which version of Qt was installed by Homebrew?

@niklasb  My OSX version is 10.10.5 and Qt is qt-4.8.7_1

hi @noppanit,
Were you able to solve this? I'm having the same issue as well.

@KickingHorse no I wasn't able to solve it. I just gave up. It looks like Facebook doesn't like scraping.

Does the libQtWebKit.4.dylib file even exist on your computers @KickingHorse  @noppanit ?

It does. The problem I was having was my code would work from the command line but the IDEs I was using (PyCharm and Spyder) kept complaining about the error above :
dyld: Library not loaded: @rpath/./libQtWebKit.4.dylib
  Referenced from: /Users/noppanit/.virtualenvs/envpy3/lib/python3.4/site-packages/webkit_server
  Reason: image not found

Took me forever to figure it out, but eventually had to do something like this :
import os
os.environ.putenv('DYLD_FALLBACK_LIBRARY_PATH', '/Users/name/anaconda/lib/')

This ultimately fixed the problem for me.
This is where the file libQtWebKit.4.dylib was actually stored after installing from PyCharm.
I'm not sure where brew -install qt put these files as I was never able to find them.

I just tryed to run a simple dryscrape.Session(url) and got the same error.
Just as @noppanit said, I got:
dyld: Library not loaded: /usr/local/opt/qt/lib/QtWebKit.framework/Versions/4/QtWebKit
I am on a Mac OS similar to @noppanit`s but i don`t have Homebrew installed, I am trying to use Mac Ports only.
When installing qt4-mac from mac ports, it does not install qt under /usr/local/opt but rather /opt/local/libexec
So I created a symlink in /usr/local/opt to libexec:
ln -s /opt/local/libexec/qt4 /usr/local/opt/qt
But dryscrape does not find the dyld file cause there is no such path.
Once I created the symlink, I searched for the missing /usr/local/opt/qt/lib/QtWebKit.framework/Versions/4/QtWebKit
$ ls  /usr/local/opt/qt/lib/QtWebKit.framework
ls: /usr/local/opt/qt/lib/QtWebKit.framework: No such file or directory

But there`s none as I said:
$ ls /usr/local/opt/qt/lib | grep 'QtWebKit'
libQtWebKit.4.9.7.dylib
libQtWebKit.4.9.dylib
libQtWebKit.4.dylib
libQtWebKit.dylib
libQtWebKit.prl

Actually, there is a way to symlink it correctly but it is necessary to symlink each Qt 4 framework separately.

/usr/local/opt/qt/lib $:sudo ln -s /opt/local/libexec/qt4/Library/Frameworks/QtCore.framework/ QtCore.framework
/usr/local/opt/qt/lib $:sudo ln -s /opt/local/libexec/qt4/Library/Framework/QtNetwork.framework/ QtNetwork.framework
/usr/local/opt/qt/lib $:sudo ln -s /opt/local/libexec/qt4/Library/Framework/QtNetwork.framework/ QtNetwork.framework

This way it`s not necessary to download homebrew, which is nice cause there are some who use Fink or MacPorts and would rather avoid installing several package managers.

currectly, the login_button = sess.at_xpath('//*[@id="u_0_l"]'), but I think you could try q.form().submit() instead of finding the login button


First /users script

I needed this for something so I made it. You'll notice there are some extra commits which cancel each other out.
I had some unrelated changes that I had pushed, but decided that I wanted to separate into different pull requests.

Found an error.


Tool returns 0 on failure, should return a non-zero code

Hi
Thanks for this Artifactory CLI, it's really useful!
However, I find that when I get an error, the CLI returns exit code 0. This is wrong, because when running it from a shell script or build tool, the operation will appear to have succeeded, even though it didn't.
Specific error was:
[Thread 0] Artifactory response: 404 Not Found
Uploaded 0 artifacts to Artifactory.
Failed uploading 1 artifacts to Artifactory.
$ echo $?
0

Thanks for this feedback @ssssam !
We will handle this soon.

Version 1.2.0 was released and includes this fix.


Feature/951

#951

This is ready for review at branch feature/951 at http://localhost:3030/tree
Note: I did not include an example of the selected state but it is available. Just add aria-selected="true" or class ```.is-selectedto the approriate__row` element. The coding to implement this for demo-able purposes alone is substantial and since it is not present in the existing component I skipped it.

@xak
The only issue I see here is the text doesn't seem to be centered vertically in the row. Theres less padding on above than below the text.

@LeoT7508 Fixed. Pull the changes and check again.


Can't set objstore option for thanos.

What did you do?
Thanos latest version supports objstore.config options instead of s3.xxx (these are removed)
So when I use set thanos v0.2.0, storage configuration is not applied properly.
--objstore.config-file=/etc/thanos/thanos-s3.yaml
https://github.com/improbable-eng/thanos/blob/master/cmd/thanos/flags.go
func regCommonObjStoreFlags(cmd *kingpin.CmdClause, suffix string) *pathOrContent {
	fileFlagName := fmt.Sprintf("objstore%s.config-file", suffix)
	bucketConfFile := cmd.Flag(fileFlagName, fmt.Sprintf("Path to YAML file that contains object store%s configuration.", suffix)).
		PlaceHolder("<bucket.config-yaml-path>").String()

	bucketConf := cmd.Flag(fmt.Sprintf("objstore%s.config", suffix), fmt.Sprintf("Alternative to '%s' flag. Object store%s configuration in YAML.", fileFlagName, suffix)).
		PlaceHolder("<bucket.config-yaml>").String()

	return &pathOrContent{
		name: fmt.Sprintf("objstore%s.config", suffix),

		path:    bucketConfFile,
		content: bucketConf,
	}
}

I can't find any related code in prometheus-operator.
What did you expect to see?
I want to add objstore.config and objstore.config.file options.
What did you see instead? Under which circumstances?
Environment

Prometheus Operator version:
insert image tag or Git SHA here

Kubernetes version information:
insert output of kubectl version here

Kubernetes cluster kind:
insert how you created your cluster: kops, bootkube, tectonic-installer, etc.

Manifests:

insert manifests relevant to the issue

Prometheus Operator Logs:

insert Prometheus Operator logs relevant to the issue here

Related code
https://github.com/coreos/prometheus-operator/blob/0ea64587a152727e072986dbe6cd60a06b57a545/pkg/prometheus/statefulset.go#L702-L732
Errror(s) seen within sidecar when trying to use v0.2.0 of  Thanos...
Error parsing commandline arguments: unknown long flag '--s3.bucket'
thanos: error: unknown long flag '--s3.bucket'

Looking at the thanos sidecar we can see the operator is attempting to pass in s3 bucket details via args
thanos-sidecar:
    Container ID:  docker://6e19519c626edbfcc6354c31935828f88e78ab5dff4cb7552d88b556edf44570
    Image:         improbable/thanos:v0.2.0
    Image ID:      docker-pullable://improbable/thanos@sha256:7555f2255dbfa7791cb9630a4633d8e22c1fba34071eec13954a2f44ec203ad1
    Ports:         10902/TCP, 10901/TCP, 10900/TCP
    Host Ports:    0/TCP, 0/TCP, 0/TCP
    Args:
      sidecar
      --prometheus.url=http://<IP_ADDRESS>:9090
      --tsdb.path=/prometheus
      --cluster.address=[$(POD_IP)]:10900
      --grpc-address=[$(POD_IP)]:10901
      --cluster.peers=thanos-peers:10900
      --s3.bucket=test
      --s3.endpoint=minio-service:9000
      --s3.insecure
      --s3.signature-version2
    State:          Waiting
      Reason:       CrashLoopBackOff
    Last State:     Terminated
      Reason:       Error
      Exit Code:    1
      Started:      Tue, 18 Dec 2018 15:21:25 -0500
      Finished:     Tue, 18 Dec 2018 15:21:25 -0500
    Ready:          False
    Restart Count:  17
    Environment:
      POD_IP:   (v1:status.podIP)
    Mounts:
      /prometheus from prometheus-prometheus-operator-prometheus-db (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from prometheus-operator-prometheus-token-bp7jx (ro)

However, with v0.2.0 and newer for Thanos, a ConfigSet with the s3 parameters should be setup in K8's and then used with a volumeMount via the --objstore.config-file=/your/file.yaml used via the volumeMount provided by a creates ConfigSet.

I made PR for this.
https://github.com/coreos/prometheus-operator/pull/2226#issuecomment-447338905


SeamlessM4T  Using audio files to implement translation  ！Can you support it?

Translate other languages via audio files like this！

Hi, please try the newest update, I added a basic implementation for testing. Not all of the languages in the list are supported, and I don't have the S2TT, T2TT and ASR yet, I just added the S2ST functionality.
https://github.com/rsxdalv/tts-generation-webui/pull/284

您好，请尝试最新的更新，我添加了一个基本的测试实现。并非列表中的所有语言都受支持，而且我还没有 S2TT、T2TT 和 ASR，我只是添加了 S2ST 功能。
第284章

thank u!You can refer to this https://replicate.com/adirik/seamless-expressive. They can recognize speech and translate the same timbre, just like heygen. They seem to use seamless.   For example: Upload the audio of an American who can translate and speak Chinese with the same accent/tone

Interesting, the results on replicate seem better than what I saw myself so far. Thank you! This gives more research to do.


Site becomes unresponsive after leaving it open for a long time

Describe the bug
I read (=did quite some nativating) most of the official site (https://quartz.jzhao.xyz) and then left it open for a day in a background tab. Then my notebook went into standby mode (don't know if that's relevant).
Coming back today I noticed that that tab now was unusably slow for the first two clicks and then stopped working alltogether. It only displayed a spinner in the center for an hour before I finally closed it.
My Firefox instance was a ~7GB memory usage at that point in time. Closing the site freed most of it.
To Reproduce

Open official site (with Firefox?)
Navigate around
Put PC to sleep mode
Turn on again a day later
Try navigating again

Expected behavior
Should not become slower with time.
Desktop (please complete the following information):
Whatever builds the official site ;-)

Hmm, do you know what page you are navigating through? Bit surprising that this is the case? I too also have a few quartz docs opening in the background sometimes and didn't notice and memory spike

Especially clicking the browser "back" button adds a lot to memory usage.
Starting at 70MB RAM and staying there when nothing is clicked, after a minute of clicking links in the navigation mar (left side) and using the browser back and forward buttons, it's already at 500MB. This happens in Chrome as well:

You should double check your extensions, I'm unable to reproduce this on my machine with Chrome or Firefox

Took me about ~1min of rapidly clicking random pages and using the browser nav to get an incognito (=no extensions) tab there as well.

I can't reproduce this, you'll need to give more detailed reproduction steps and include more information about what version of Chrome/FF/etc you are testing on

Closing for now, this seems to be specific to your setup and neither me nor Aaron can reproduce it

Sure, here's a screencap with network and console open for starters, using latest Chrome incognito. Version 121.0.6167.185 (Official Build) (64-bit) (cohort: Stable). Happens in FireFox all the same, though.
In the video I'm kind of speed-running the issue by clicking very often in a very short timeframe, but as stated in the opening post in this thread, it also happens gradually over time.
Had to cut the vid in 2 parts because of GH file size limits:
https://github.com/jackyzha0/quartz/assets/1475672/902c202b-2c0c-45ae-a51f-987acba4b175
https://github.com/jackyzha0/quartz/assets/1475672/ba23259b-786f-4f9b-9e94-abe546b9ff03


[移除]：

Hotspot Name
Late Misty Seal
Hotspot b58 Address
112bARePsaXbHTS9Fdw2JAVkszoAhyuB65o6WzU6wB9vaKhmG9M8
Discord Handle
No response
Hotspot Manufacturer
PantherX
Removal Reason
My machine used to use vpn, but later vpn was withdrawn. At present, the network cable is added to the machine! Request to delete the blacklist
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information

Looks like a fake to me. First HS is connected to the power outlet somewhere near the ground, no window visible. On the next photo its on a chair, near the window but nothing more than the antenna is visible? Is it even connected to the poweroutlet there? :D
Looks really suspicious to me.


[WIP] Update cni plugin configuration for calico v2.1

Include k8s_api_root option in cni plugin configuration. This will probably resolve all issues we currently have with kube2dns and k8s dashboard containers.
Please do not merge until we double check it.

Busy deploying it keeps timing out on waiting for api server up

If it doesn't work, we can revert it


Integration with Citizens

The goal with this is to provide a "Guide" functionality, to replace the particle trail that helps players navigate a calculated path today. The integration plugin will likely require a new type of interface that extracts what the particle trail-based JourneySession can do. Then, the Citizens integration can implement a derived abstract class, like a "Guide session", that provides functionality about how to manipulate a guide entity, which would call NPC methods in Citizens.
So most functionality would be in Journey, but just the lowest level interface would be implemented by the incoming JourneyCitizens plugin. Then, users can choose what kind of display to send users, particle trail or other registered ones, like "citizens" as JourneyCitizens would call it, in a flag. Like other flags, it can be set to a default in the config and players may be given permission to use the flag if admins want. Permission should also probably be per-option... More details to come

Added in v1.2


Space inserted after ,@ in Paredit for Elisp

When I type an unquote-splicing followed by an open-paren, like:
,@(

Paredit inserts an additional space before the open-paren, which I have to go and delete.

Irrelevant now that we don't use Paredit.


Command line option to specify "speed" appears to be limited to integers?

I'm trying to use the command line "-speed" option, on Windows 10 (DJV 2.0.8), to set the frame rate based on the image sequence being loaded. However, it appears if I specify a floating point frame rate (e.g. "-speed 23.98") it gets truncated to an integer (once loaded, DVJ shows a "custom" frame rate of 23).
Am I doing something wrong? Is it possible to support floating point frame rates on the command line, please?
Thanks!

Hi, thanks for the bug report, this should be fixed now and will be included with the next release. It won't accept arbitrary floating point frame rates, but will recognize common frame rates like 23.98.

Hi, that's great, thank you. Thanks for the rapid turn around!


cudaPackages_12_1.cudatoolkit: Fix build failure over missing QT dependencies

Description of changes
Dependencies on libgstreamer and libcurl were added starting in cudatoolkit 12.1.
This PR compensates for the resulting autoPatchelf failures.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This addresses the following build failure for cudaPackages_12_1.cudatoolkit on nixpkgs-23.05 and nixpkgs-unstable-2023-06-17:
error: auto-patchelf could not satisfy dependency libcurl.so.4 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/target-linux-x64/CollectX/clx
error: auto-patchelf could not satisfy dependency libcurl.so.4 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/target-linux-x64/CollectX/libclx_api.so
error: auto-patchelf could not satisfy dependency libgstaudio-1.0.so.0 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/host-linux-x64/libQt6Multimedia.so.6
error: auto-patchelf could not satisfy dependency libgstvideo-1.0.so.0 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/host-linux-x64/libQt6Multimedia.so.6
error: auto-patchelf could not satisfy dependency libgstpbutils-1.0.so.0 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/host-linux-x64/libQt6Multimedia.so.6
error: auto-patchelf could not satisfy dependency libgstallocators-1.0.so.0 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/host-linux-x64/libQt6Multimedia.so.6
error: auto-patchelf could not satisfy dependency libgstapp-1.0.so.0 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/host-linux-x64/libQt6Multimedia.so.6
error: auto-patchelf could not satisfy dependency libgstbase-1.0.so.0 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/host-linux-x64/libQt6Multimedia.so.6
error: auto-patchelf could not satisfy dependency libgstreamer-1.0.so.0 wanted by /nix/store/l99hfvpf0ll98j5bvmygkzs3wyadf54m-cudatoolkit-12.1.1/host-linux-x64/libQt6Multimedia.so.6
auto-patchelf failed to find all the required dependencies.

The change introduces a dependency on three packages:

curlMinimal
gst_all_1.gstreamer
gst_all_1.gst-plugins-base

[ ] Later we need to look into discarding the copies of qt6&c libraries vendored by nvidia. If we choose so, we can patch the cudatoolkit to link against the nix-built libraries instead, e.g.:

❯ nix-locate libQt6Multimedia.so.6
qt6.qtmultimedia.out                                  0 s /nix/store/2biw72z3w0wqf1b56ps041hhcmpndp6n-qtmultimedia-6.4.2/lib/libQt6Multimedia.so.6

Since this fixes a build-break, and I'm rather new at this to boot, I'd rather attempt that as a second pass- or have someone more familiar with Nix do so @SomeoneSerge; is that acceptable to you?

Result of nixpkgs-review pr 238341 --extra-nixpkgs-config '{ config.cudaSupport = true; }' run on x86_64-linux 1

[x] ❯ NIXPKGS_ALLOW_UNFREE=1 nix build --impure github:Dessix/nixpkgs/dev/dessix/fix-cuda-12.1-build#cudaPackages_12.cudatoolkit
[ ] ❯ NIXPKGS_ALLOW_UNFREE=1 nix build --impure github:Dessix/nixpkgs/dev/dessix/fix-cuda-12.1-build#cudaPackages_12.nsight_compute

> error: auto-patchelf could not satisfy dependency libQt6WlShellIntegration.so.6 wanted by /nix/store/lvmazmjq3cibdmb8y2204gj5wbajh9is-nsight_compute-2<IP_ADDRESS>/nsight-compute/2022.4.1/host/linux-desktop-glibc_2_11_3-x64/Plugins/wayland-shell-integration/libwl-shell-plugin.so

🤔 let's merge, and then fix cudaPackages_12.nsight_compute when we get to removing the vendored stuff?

The commit message better begin with the attribute name

Fixed in a rebase atop latest HEAD.

Thank you for the contribution @Dessix!
@SomeoneSerge would you create an issue for getting rid of vendored stuff and for fixing nsight-compute, just so they don’t get lost?


I got an error when running YoloV4 in anaconda

Traceback (most recent call last):
File "detect_video.py", line 178, in
app.run(main)
File "C:\Users\danie\anaconda3\envs\yolov4-cpu\lib\site-packages\absl\app.py", line 308, in run
_run_main(main, args)
File "C:\Users\danie\anaconda3\envs\yolov4-cpu\lib\site-packages\absl\app.py", line 254, in _run_main
sys.exit(main(argv))
File "detect_video.py", line 160, in main
image = utils.draw_bbox(frame, pred_bbox, FLAGS.info, allowed_classes=allowed_classes, read_plate=FLAGS.plate)
File "D:\YoloV4\yolov4-custom-functions\core\utils.py", line 249, in draw_bbox
cv2.rectangle(image, c1, c2, bbox_color, bbox_thick)
cv2.error: OpenCV(4.6.0) :-1: error: (-5:Bad argument) in function 'rectangle'

Overload resolution failed:

Can't parse 'pt1'. Sequence item with index 0 has a wrong type
Can't parse 'pt1'. Sequence item with index 0 has a wrong type
Can't parse 'rec'. Expected sequence length 4, got 2
Can't parse 'rec'. Expected sequence length 4, got 2

Viseron is sadly not officially supported on Windows

@roflcoopter
Okay thanks


Assignment11 - 6ITA - Vaishnavi

For #630
📋 Description
All 5 questions - done
📸 Screenshots

@sagariyavaishnavi Squash commits please!


Getting rid of CRF++

At some point, we could use only Wapiti since it provides only advantages as compared to CRF++. This would simply also the maintenance given that we need otherwise to manage two set of JNI libraries and models.

Wapiti should be now completely integrated and the default CRF engine with commit d2318a30825385638759e457703484ac041f02bd


SMTP example is not working with hotmail.  (IDFGH-5522)

under examples/protocols/smtp_client
the example code is working fine with gmail.
However, it is not working with hotmail.

Perhaps this data is different for gmail and hotmail.

@jerrytry1 please try with certificate replaced as suggested by @nopnop2002

openssl s_client -showcerts -connect smtp.googlemail.com:587 -starttls smtp
openssl s_client -showcerts -connect smtp-mail.outlook.com:587 -starttls smtp

Thanks for sharing updates, feel free to reopen.


MGMT-10623: Add OCP 4.11 nightly images

Description
This change should mainly affect integration env, having nightly-built images of OCP 4.11 (no fc/rc release has been promoted to date).
I also took the liberty to add 4.10 ARM image which seems missing.
How was this code tested?
oc process --local -f deploy/template.yaml -pIMAGE_TAG=latest seems to work.
For some reason I couldn't make RHCOS_VERSIONS have a minimal value (without line-breaks), even when using value: <- but I think it's not a big issue.
Assignees
/cc @carbonin
/cc @danielerez
Links
MGMT-10623
Checklist

[x] Title and description added to both, commit and PR
[x] Relevant issues have been associated
[x] Reviewers have been listed
[x] This change does not require a documentation update (docstring, docs, README, etc)
[ ] Does this change include unit tests (note that code changes require unit tests)

Why does this have a hold?
While checking configuration of 4.11 images in staging, I found out we can't really use those 4.11 nightly OCP releases with staging pull-secret. It means we can push those ISO images, but either way cannot use any of it for a full-installation flow. I'm considering waiting for the fc-0 release instead of having those problematic releases
Prod pull-secret works just fine with those nightly releases btw

/unhold
4.11.0-fc.0 is available now, so no real reason to hold this one
PTAL @carbonin @danielerez

/refresh


[BUG] client not work with windows forms - stuck when request

windows forms
net 7

Are you using the async methods? It's a known issue that the requests hang when called from UI framework threads using the non async versions.


Failure to scale up using rack and zone based node groups on EKS

Pods of the statefulset get stuck in Pending.
Events:
Type     Reason             Age                 From                Message

Warning  FailedScheduling   12s (x8 over 10m)   default-scheduler   0/12 nodes are available: 1 node(s) had volume node affinity conflict, 2 Insufficient memory, 2 node(s) didn't match node selector, 9 Insufficient cpu.
Normal   NotTriggerScaleUp  10s (x18 over 10m)  cluster-autoscaler  pod didn't trigger scale-up (it wouldn't fit if a new node is added): 1 node(s) had volume node affinity conflict, 2 node(s) didn't match node selector
Looking at the pvc:
Volumes:
9afc3e6fc2c83f5b74b95be4c5d7c0-opt-aerospike-data:
Type:       PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
ClaimName:  9afc3e6fc2c83f5b74b95be4c5d7c0-opt-aerospike-data-aerocluster-test-2-0
ReadOnly:   false
Details on the pvc show that it is tied to us-west-2c:
apiVersion: v1
kind: PersistentVolume
metadata:
annotations:
kubernetes.io/createdby: aws-ebs-dynamic-provisioner
pv.kubernetes.io/bound-by-controller: "yes"
pv.kubernetes.io/provisioned-by: kubernetes.io/aws-ebs
creationTimestamp: "2021-09-15T04:58:31Z"
finalizers:

kubernetes.io/pv-protection
labels:
failure-domain.beta.kubernetes.io/region: us-west-2
failure-domain.beta.kubernetes.io/zone: us-west-2c

but rack id 2 is supposed to be us-west-2b:
- id: 2
zone: us-west-2b
aerospikeConfig:
service:
proto-fd-max: 18000
storage:
filesystemVolumePolicy:
initMethod: deleteFiles
cascadeDelete: true
blockVolumePolicy:
cascadeDelete: true
volumes:
- storageClass: ssd
path: /opt/aerospike
volumeMode: filesystem
sizeInGB: 3
Looking at the actual pod of aerocluster-test-2-0 confirms that the pod itself was tied to the right zone, us-west-2b:
spec:
affinity:
nodeAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
nodeSelectorTerms:
- matchExpressions:
- key: failure-domain.beta.kubernetes.io/zone
operator: In
values:
- us-west-2b
I did some checking in the code for the statefulset controller, and I see where the Rack zone setting is picked up and applied, but can not find equivalent for the pvc.
The behavior being exhibited seems to match issues described here, under Problems section:
https://github.com/kubernetes/community/blob/master/contributors/design-proposals/storage/volume-topology-scheduling.md

Hi @eb-koddi, Sorry for the late response. PVC does not have any scheduling policy fields. We should use WaitForFirstConsumer as volumeBindingMode in storageClass to ensure that PVC follows Pod's scheduling policies.
https://kubernetes.io/docs/concepts/storage/storage-classes/#volume-binding-mode

Closing as answered.