Design question for npm package

Continued from https://github.com/badges/shields/pull/4756#issuecomment-607420151, where @chris48s wrote:

I think there is a big design question about what the badge-maker package should be which we don't all agree on:
One way of thinking about it is:
Option 1 - Badge-maker should be a stand-alone package. It happens to live in the same repo as shields.io, but in principle it could live in its own repository if we wanted. Its a package with many users, but its biggest and main user is shields.io. Any functionality we depend on should be exposed and documented via the public interface of the package and we consume that functionality in the same way as any other user, eating our own dogfood.
Another way of thinking about it is:
Option 2 - Badge-maker is a package which exposes only a subset of the badge rendering capibilites of shields.io to a wider audience. It lives in and is inseparable from the shields.io repo. The interface we give our users is different to the interface we consume internally.
Option 2 is where we are now. I've been thinking we should be heading towards option 1. I'm not necessarily right about that, but that's definitely the direction I've been thinking in. Given that removing a feature that our own application depends on from the public interface of the package actively reverses that direction of travel, I think its worthwhile to be explicit about the reasons for that.

I agree with you that we should be heading toward option 1.
It has the lowest mental overhead, and also means we can harmonis/ze with the Endpoint badge spec. I think that could create a lot of flexibility for users.
It sounds like the gaps are:

Links
Logos
Possibly JSON, if we remove the JSON support as discussed at https://github.com/badges/shields/pull/4756#issuecomment-597782014

Is that right?

Yep - I'd agree with that assessment. Just to flesh those three points out slightly more. In order to move from using relative imports to being a consumer of our own package, we need:
Links
Exposing links is pretty trivial (although we do need to decide on the public interface for it) its passing through a couple of strings really.
Logos
Logos is slightly more involved as we need to move the logo code from /lib into the package, allow simpleicons to be installed as a peer dependency (if the user wants/needs it) and work out the interface for logos (to deal with simpleicons/custom/base64) in the input object. Doing this would also remove the final remaining relative import to the package requiring anything other than makeBadge:
https://github.com/badges/shields/blob/8621fe42d76b809e9ba4ae844920f8ed0373817a/lib/logos.js#L4
I'd propose pushing this one back to 3.1 it is a feature add, not a BC-break (although its worth thinking about it now)
JSON
If we need JSON for testing, it needs to be a supported output format for users too. We don't want to remove it. Maybe we do want to think about the shape/format of the JSON in the context of the input object as well :thinking:

Exposing links is pretty trivial (although we do need to decide on the public interface for it) its passing through a couple of strings really.

Yea, agree.
Personally I think links are kind of a pointless feature of Shields since in most circumstances they don't work. (i.e. you have to use <object> when embedding the badge, not <img>)
Another option would be to remove them, though that seems like more work than keeping them.

I'd propose pushing this one back to 3.1 it is a feature add, not a BC-break (although its worth thinking about it now)

:+1:

If we need JSON for testing, it needs to be a supported output format for users too. We don't want to remove it. Maybe we do want to think about the shape/format of the JSON in the context of the input object as well 🤔

This one I disagree with. Here's how I'm thinking about it:

We do use JSON for testing – but more importantly, we provide JSON output as a service.
These four things are machine-friendly representations of the contents of a badge. They should be harmonized so they as are close to each other as possible:

The output of coalesceBadge
The input to the library's makeBadge function
The Endpoint badge schema
The JSON badge output schema

If these schemas are brought into close alignment, generating the JSON output badge on the Shields server is JSON.stringify()-ing a really minor transformation on the output of coalesceBadge, and there isn't a need to use a complicated rendering pipeline to do that.
Because the Shields JSON "badge" schema is such a minor/trivial transformation on a small piece of JSON, we don't need to expose that transform so that others can use it for testing. So it's not something the library needs to support.

Yes. This is a good point. We could generate the JSON we need for our own JSON endpoint/testing outside of makeBadge. If we do that, we can still move in the direction of consuming our own package and it doesn't matter if we remove JSON output from the package (and also we don't need to care so much about backwards-compatibility if we change the format a bit, which is also useful). If we plan to do that, we could just drop it while we still don't depend on the public interface of the package. That breaking change is best made in v3.0 and we can work on moving things about for our internal use independently.
Do you have any thoughts on the other question in https://github.com/badges/shields/pull/4756#issuecomment-607420151 about changing the input object format? The change to the input format should also be done for v3.0.
I think given most of the params are optional, the non-nested version makes more sense when you're not using all of them e.g:
{
  labelText: 'build',
  messageText: 'passing',
  messageColor: '#555',
}

Yea, I think flatter is easier. Easier to document, and easier to deal with across various platforms where developers might use this, and a little easier to validate.
labelText, messageText, and messageColor are more explicit which is good. However since we already use label, message, and color in (1) all the service implementations, (2) the Endpoint badge, and (3) the query params, I'd prefer we use those names here too. In particular I see the argument for messageColor over color, although I think it is okay to keep using color since the vast majority of badges, even the static badges, keep the default labelColor.
For link I'm more open to a new API, since (1) very few of the badges use it, (2) it's not in the Endpoint schema, and (3) the current API of specifying the query param twice is really strange. If the default is to link both sides of the badge to the same value, we could use ?link=http://whole-badge, with ?link=http://right&labelLink=http://left to link the label to something different, and ?link=&labelLink=http://left to link the label and not the message (if we think we even need that). We should also decide if we want to keep backward compatibility with the double link query param. This question probably needs a comment period, and an issue of its own. Since we need to make a decision on the package now, three options come to mind:

Pick an API that we think will make sense everywhere (i.e. in services, coalesceBadge, Endpoint, query params) and adopt it first in the library, then bring it into the server codebase, and finally consider what to do about the legacy query param. If we change our mind during those discussions, ship a breaking change.
In the style of the query param (which yields an array) and coalesceBadge, allow makeBadge's link to be either a string, or an array containing one or two values. Revisit and ship a breaking change if necessary.
Include it in the library but as an explicitly unstable feature. React has methods in it which are preceded with unstable_ to indicate they can change in any version. We could name the parameter unstable_link to signal the same thing.

A final thought on link – and the reason I haven't spent any time working on it in the past 3+ years – is that IMO it is a mostly useless feature. We could possibly consider seeking feedback from the community and then removing it.

Just noticed #3370. Should we close it and track here instead?

However since we already use label, message, and color in (1) all the service implementations, (2) the Endpoint badge, and (3) the query params, I'd prefer we use those names here too

Consistency with the endpoint badge/query params is a good point :+1: I'm convinced on label, message and color.

We should also decide if we want to keep backward compatibility with the double link query param. This question probably needs a comment period, and an issue of its own

We could just defer links to 3.1 too. We need it to get rid of relative imports to package internals, but we can't actually do that until we do the logo work as well and adding links is non-breaking. I suggested adding it in 3.0 because it is "just passing 2 strings", but I agree the effort is really in working out how to expose it.
That would make the remaining jobs on #4756 :

Change text[] to label and message, fix message only badge
Remove JSON output format
Update the docs

all of which are minor - I'll have a look at them this evening.

We could possibly consider seeking feedback from the community and then removing it.

My gut instinct is that across the whole of shields.io, links is a very rarely used feature, but its on by default for the github and twitter social badges which do get a decent amount of traffic so removing the feature completely would probably have some friction (I wonder how many actually embed them with the <object> tag though?).

Just noticed #3370. Should we close it and track here instead?

I have found this discussion useful, but its also covered several topics which won't all be worked on at once. Assuming you agree with the points above, I think we close this issue and move the future work to more focussed issues:

What to do about links?
Move JSON output out of the package and into core/server
Keep #3370 open, but update it with relevant info from this thread

💯

Should we rename template to style, too?
I don't think that needs to block the merging of #4756, though it sounds like it should be changed before the release is made.

I've added that to #4756 as well

I've split the remaining future work from this issue into more specific issues


invalidation

the newest update causes problem. The quoted entity  cannot show menu of this plugin.
Thanks for your development.

Could you please provide a little more detail. In what context are you getting this message. Which action?

The sign is ok in old version，but in new version，it can not display truely

张宇考研数学基础30讲【高等数学分册】 (1).md
this is the file which have problem

Please test again in 2.2.0, the most recent update. Thank you for the sample file. If it isn't working, please provide a small, simple file illustrating the problem. This file had a lot of stuff I couldn't read and didn't want to load in my vault.


Mixer has an excessive number of disk writes impacting overall performance

@jnativio commented on Wed Dec 06 2017

Is this a BUG or FEATURE REQUEST?:
Did you review existing epics or issues to identify if this already being worked on? (please try to add the correct labels and epics):
Bug:
Y
What Version of Istio and Kubernetes are you using, where did you get Istio from, Installation details
istioctl version: 0.2.12
kubectl version: v1.7.4-2+eb9172c211dc41

Is Istio Auth enabled or not ?
istio.yaml was installed.
What happened:
I installed istio.yaml 0.2.12  as is initially and then I changed some properties in order to minimize the performance impact on the workload. Even with all known tuning options it was observed a large number of disk writes during the load as well large CPU usage, causing the throughput to be a fraction of what obtained without the mixer.
What you expected to happen:
Less performance impact.
How to reproduce it:
Drive some workload to a microservices app using Istio with and without the mixer to compare the performance impact.
Feature Request:
N
Describe the feature:

cc @jnativio - I added you to the org so you can create/work on issues directly in istio/istio
(istio/issues/issues is for users)

@jnativio, could you give more detailed repro steps & configuration changes you've made?

Here is the document that has all the details of our Istio Performance with some low level data
Istio 0.2.12 Performance Measurements.pdf

Great, thank you!
On Wed, Dec 20, 2017 at 9:34 AM, Surya Duggirala<EMAIL_ADDRESS>wrote:

Here is the document that has all the details of our Istio Performance
with some low level data
Istio 0.2.12 Performance Measurements.pdf
https://github.com/istio/istio/files/1576491/Istio.0.2.12.Performance.Measurements.pdf
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/2025#issuecomment-353130469, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQR8IxDhC9rzrxDaNomkMWUwZyLyA8MLks5tCUU2gaJpZM4Q6YVy
.

@suryadu It looks like the graphana dashboard does not record disk IO, only records CPU/memory. Did I miss something?

@mandarjog  We developed a custom grafana dashboard for performance analysis of Istio where we could get the disk IO as well as shown below. The worker node .130 has only Istio components like Mixer and as you can see the disk IO changes from 7 MB/sec to less than 2 MB/sec when we disabled the mixer all together. We have not dissected the 2 containers (mixer and statsd) disk IO from mixer POD which we will next as we discussed in our WG call today.

Has there been follow-up on this problem?

Yes, and it is also one of the areas for the perf swat team to dig further. @mandarjog is on point for that part.

@douglas-reid has a PR that introduces sampled output to the stdio logger. Is that part of addressing this issue?
We could also put a buffering layer in front of zap. I believe every log statement in zap is pushed all the way out with no intermediary buffering. Adding just a few ks of buffer would go a long way.


Drag multiple items

Is there a possibility to DnD multiple items at once? I could't find it in the documentation ?
If not i file it under feature requests

the feature doesnt exist


locust: error: no such option: -n

Description of issue / feature request
I tried to execute the following command,
"locust -c 10 -r 10 -n 10 --no-web -f locustfile.py"
but, got this error in terminal
"locust: error: no such option: -n"
Environment settings (for bug reports)

OS: Ubuntu 16.04
Python version:py 2.7.12
Locust version: 0.9.0

Hey, I found that -n command is not supported in v0.9.0. I used '-t' command to specify the time period. If you want to use -n command, v0.8.0 is needed.

why -n command is not support, i found this command is very useful

we changed it to time based execution

Adding to what @cgoldberg said, there are a lot of situations where the -n value is not precise too - fundamentally Locust cannot easily guarantee an explicit number of requests.
During startup, when distributed, when shutting down, etc all cause situations where the actual number of requests cannot match -n without significant changes to how Locust worked. This caused questions often why a server would register different numbers of requests, etc.

Please restore this lost functionality


Not able to download the latest Chrome Edge Driver Binary

It looks like the list of BinaryTypes (for possible names) does not include the latest Chromium based Edge. The Driver binary name is msedgedriver.

Fixed by https://github.com/Ardesco/driver-binary-downloader-maven-plugin/commit/cb0278e93317251c42f37f55e4ae914ef8832924


Unable to invoke macros on file private classes

See #3858, by @MakeNowJust. This issue is here to keep track of the problem.
On Crystal 0.20.3, for instance:
private class Foo
end

Foo.new.dup

this code causes an error:
Error in foo.cr:4: instantiating 'Foo#dup()'

Foo.new.dup
        ^~~

in /.../src/reference.cr:40: expanding macro

    {% if @type.abstract? %}
    ^

in macro 'macro_XXX' /.../src/reference.cr:40, line 2:

   1.
>  2.       dup = Foo.allocate
   3.       dup.as(Void*).copy_from(self.as(Void*), instance_sizeof(Foo))
   4.       dup
   5.

This is a simple patch to get Reference#dup work without changing macro: (But we still need to improve the macro system)
From 2b74a5a99eea3e7da6c62646ef609db5e7a6875e Mon Sep 17 00:00:00 2001
From: David Kuo<EMAIL_ADDRESS>Date: Fri, 13 Jan 2017 21:42:00 +0800
Subject: [PATCH] Allow Reference#dup in private type

---
 src/reference.cr | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/reference.cr b/src/reference.cr
index 35e26d2..a09a8e5 100644
--- a/src/reference.cr
+++ b/src/reference.cr
@@ -44,8 +44,8 @@ class Reference
       # but we need to avoid the allocate invocation below
       raise "can't dup {{@type}}"
     {% else %}
-      dup =<EMAIL_ADDRESS>-      dup.as(Void*).copy_from(self.as(Void*), instance_sizeof({{@type}}))
+      dup = self.class.allocate
+      dup.as(Void*).copy_from(self.as(Void*), instance_sizeof(self))
       dup
     {% end %}
   end
--
2.7.4

tested with:
private class Foo
  property a = 1
end

f = Foo.new
p f.a

p f.a = 2
b = f.dup
p b.a

@david50407  Doesn't work with:
class Foo
  property a = 1
end

class Bar < Foo
end

[Foo.new, Bar.new].map &.dup

This needs a compiler fix, there's no way to do it without changing the compiler.

@asterite  oops, I think I know why that fails
Should we create a macro system central-discussing issue like this https://github.com/crystal-lang/crystal/issues/2665 ?


ItemSwapper compatibility

Mod Name
ItemSwapper
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/itemswapper
Describe the bug
Dynamic LVT patch broke
Steps to reproduce
No response
Logs
https://mclo.gs/BPC4eHO
Additional context
Tested by @VladisCrafter. I'm opening an issue so that I don't forget to fix this, seems like an easy bug

Thanks for fixing it! I'm really glad I could help :D


General API comments

Hi all, great job on the APIs documentation.   I used the Python to start off testing along with cURL but then once I got up to speed I built C# code on Windows to access the BMW service calls.   I have been using Charles for URL sniffing which has been very helpful.
I noticed a few things.
In your article where you showed the response JSON from a vehicle status, there are more items that can be retrieved if the car is charging which would be good to include.
I found that the chargingTimeRemaining value added to the current time doesn't often match up with the estimated time to complete charging shown on the i3 dash.   I wonder why.
When retrieving the vehicle status I was wondering why once I got status it didn't seem to change unless I restarted my app.  I guessed that it was a caching issue and although I later figured out how to fix that through code I adopted the same technique BMW did to force each request to be new by appending the time stamp onto the request like this:
https://b2vapi.bmwgroup.us/webapi/v1/user/vehicles/MYVIN/status?deviceTime=2015-12-30T10%3A06%3A13
I included a polling cycle for command executions to look for DELIVERED, PENDING and then exit the wait if an EXECUTED or TIMED_OUT message is returned.
Something funny about door lock.   It works fine but I noticed that it doesn't turn on the car alarm.   If you look outside of your car when doing a DOORLOCK using the API, the doors lock but then the light under the rear-view mirror is not flashing.   If I then press the lock button on my remote, it takes it a step further and turns on the alarm.
I noticed a status from door lock that was odd.  I have seen selective_locked but I can't tell what that means.  I have been trying to create alternative human readable messages to translate BMW messages but don't know what that means.
I tried to create a DOOR_UNLOCK command but that failed.   I think why that is could be because it is not enabled by the car.   In the vehicles information object retrieved it shows this:
"doorUnlock": "NOT_SUPPORTED".   I assume that if an action says NOT-SUPPORTED then the API cannot control it.   Does that sound right?
Has anyone found a way to generate the app's secret token based on using the user's ID and password?  That would be super helpful.
Thanks,
Rick

Hi Rick - thanks for your comments.

In your article where you showed the response JSON from a vehicle status, there are more items that can be retrieved if the car is charging which would be good to include.

Good spot! Can you please add them into the main documentation?  You can edit the doc at https://github.com/edent/BMW-i-Remote/edit/master/README.md

I adopted the same technique BMW did to force each request to be new by appending the time stamp onto the request

Interesting. I didn't bother including that because I wasn't seeing any cached data. I wonder where the cache is occurring - at BMW's end or somewhere else on the network?

Something funny about door lock. It works fine but I noticed that it doesn't turn on the car alarm.
and
I tried to create a DOOR_UNLOCK command but that failed. I think why that is could be because it is not enabled by the car.

I wonder if that's a region specific thing? I know that the UK models don't let you sound the horn via the API. Which region are you in?

Has anyone found a way to generate the app's secret token based on using the user's ID and password? That would be super helpful.

Not yet :-)
Thanks!

I'm in the US region.  Oh by the way that's a neat tidbit too.  When I first started out I was using the UK's URL from your article and that worked for me to authenticate and get the Bearer ID but then stumped me since I got authentication errors for everything else, when I used the US URL like my iPhone iRemote app was using then it worked.
Here is what I have in my vehicle info section:
"countryCode": "V2-US",
"egoVehiclePath": "I01",
"asymEncryptionMethod": "RSA-2048",
"remote360packageEncryption": "ZIP",
"vehicleFinder": "ACTIVATED",
"remote360": "NOT_SUPPORTED",
"hornBlow": "ACTIVATED",
"lightFlash": "ACTIVATED",
"doorLock": "ACTIVATED",
"doorUnlock": "NOT_SUPPORTED",
"climateControl": "NOT_SUPPORTED",
"climateNow": "ACTIVATED",
"chargingControl": "WEEKLY_PLANNER",
"chargeNow": "NOT_SUPPORTED",
"sendPoi": "ACTIVATED",
"rangeMap": "RANGE_POLYGON",
"lastDestinations": "SUPPORTED",
"intermodalRouting": "AVAILABLE",
"climateFunction": "AIRCONDITIONING",
"onlineSearchMode": "MAP",
"smartSolution": "NOT_SUPPORTED",
"supportedChargingModes": ["AC_LOW", "AC_HIGH", "DC"],
"lscType": "I_LSC_IMM"
When you check on your side for lightFlash, does that show as NOT-SUPPORTED?
Regarding caching, it could be that the Python http (requests?) library already takes care of not caching so it doesn't have the issue that the Windows HttpClient class has.
What kind of file format is .md?   I need to find an editor for it.
Oh in the docs you probably don't have an example for the output from /webapi/v1/user/vehicles/ do you?  Its not in your original article.
Thanks.
Rick

Terence, did you see the question above?  I'm curious what the output was from a UK i3 perspective on what shows up as NOT_SUPPORTED in the vehicle info response.

Sorry for the delayed response.

Regarding caching, it could be that the Python http (requests?) library already takes care of not caching so it doesn't have the issue that the Windows HttpClient class has.

Ah! That's probably it.

What kind of file format is .md? I need to find an editor for it.

.md is MarkDown you should be able to edit it with notepad.  I don't use Windows, but I'm sure there's some freeware which will help with the formatting.

Oh in the docs you probably don't have an example for the output from /webapi/v1/user/vehicles/ do you? Its not in your original article.

I don't think so - do feel free to add it.

I'm in the US. Here's my status for doorUnlock
doorUnlock  :  SECRET_KNOWLEDGE


Fix tarball gen for 2.4 branch

Fix https://github.com/Alluxio/alluxio/issues/12764

alluxio-bot, merge this please


be able to suggest new feeds

feature, be able to suggest new feeds and have admin approve them

I haven't implemented any admin functionality yet, so this might take a bit longer to implement.
Some more questions/ideas regarding this:

Would it be enough to send a private message to all users listed in the admin_nicks array with the suggested feed?
Should an admin be able to approve this feed from within the IRC or is manually adding the feed an option?
Do those "admins" have to have  the op status or not?

Maybe we can have a password defined to authenticate the user. I don't have op on the channel.
I don't want to have to reboot, we need a way to add them on the fly.
user should also be able to suggest removals and edits as well (bad url)


Update quick-event extension

Description
I'm from Europe and noticed i keep writing my hour and minute in this format 14u30 or 15h30
This change will preProcess the inputQuery to replace those timeformats with the english 2:30 PM 15:30 PM format
Screencast
/
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for your update @whitespacecode - I'll have to check with the error, it's because the API is to long behind
@mblode could it be an idea to update to the newest version of Raycast API since it's really long behind - are you okay with that?

I can, of course, help with the migration 🙂

I can, of course, help with the migration 🙂

Is there a lot to it for updating to the new API?
Do you mean update the package and use/change to the new features?

No shouldn't be that hard - you should be able to run npm install @raycast/eslint-config --save-dev and npx @raycast/migration@latest .


Add HeaderNames, MediaTypes, StatusCodes

This resolves #88
This adds trait/object pairs for HTTP header names, media types, and status codes.
Header Names
A complete list of header names is here, though not sure it's worth adding all of them, so I started with the set defined by Play. I'm open to using a different source.
Media Types
A complete list of media types is here, though not sure it's worth adding all of them, so I started with the set defined by Play. I'm open to using a different source.
Status Codes
This was easy to use the complete list defined here. I did not include any non-standard ones like 418, 420, etc.
The next step would be to integrate these constants with sttp itself. I haven't looked at how much work that would be. Could do it either as part of this PR or a separate PR.

I've updated the code using the constants, maybe you can check if everything looks good? Always good to have another pair of eyes :) https://github.com/softwaremill/sttp/commit/574aee7a322abae646fd7c851e8259c85ec841ae
I suppose what's missing is some mention in docs?

Commit looks good. Agree the docs need a mention (maybe just a single page describing the 3 types of available constants with github links to see the full set of values?) and probably updated examples, e.g. here.
Curious if you have considered scaladocs. I usually use the IDE, but it's sometimes nice to use a browser. Not sure what most other people do.

@jsw do you mean publishing scaladocs? why not ... but where? :) Would you maybe have some time to add the docs?

@adamw Good question. I just discovered javadoc.io.
I can update the main docs.

@jsw if you know how to integrate that into the build, go ahead :)


(HidHide) install autorestarted my machine

Which should 100% not be a thing for obvious reasons. Not sure if installer supports that, but it should.

Hi @Destroy666x  :slightly_smiling_face:
I reproduced your issue, it is indeed something to avoid.
I found a way to tell the installer how to not do that (cf. 7df4c3).
I am going to update the chocolatey package so that this fix will be available to everyone! I'll keep you posted :wink:

Here in the link to the new version of the package with the fix 😉
It is currently being in the review process of the Chocoatey team

Thanks!

@Destroy666x This is done!  :slightly_smiling_face:
The new version of the package hidhide without this issue is not available. :tada:


Update AGP version validation code to support KGP and kotlin build files.

In https://github.com/flutter/flutter/pull/140744 we added the ability to use kotlin in gradle files but did not update all the parsing logic that was previously written assuming groovy.
This bug tracks updating the AGP version grepping and validation logic to support kotlin based gradle builds.

#141400 is related, maybe even it can be considered a subset of this issue

@reidbaker Besides https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/android/gradle_utils.dart, is there any other place where the parsing logic will need to be updated?

Perhaps project.dart as linked in https://github.com/flutter/flutter/issues/141400
@gmackall if ann updates this code will it cause merge conflicts for you?

Perhaps project.dart as linked in #141400
@gmackall if ann updates this code will it cause merge conflicts for you?

Nope!

It looks like parsing/validation logic for build.gradle.kts is already correctly implemented, but parsing/validation logic for settings.gradle.kts has not yet been implemented.

@mossmana If you end up working on this ticket can you make sure that the build.gradle.kts logic has the correct test coverage?

The regex we are currently using may not cover all the cases. I'm currently trying to find the current and prior docs on what is the expected format of the AGP configuration (they have changed with different versions of Gradle).

Current build.gradle for Android: https://docs.gradle.org/current/samples/sample_building_android_apps.html

It looks like parsing/validation logic for build.gradle.kts is already correctly implemented, but parsing/validation logic for settings.gradle.kts has not yet been implemented.

Correction: build.gradle logic is only correct for 6.X gradle

I think what you are seeing are not changes in gradle version support but instead the difference between applying the android tool as a plugin and applying it as a dependency/classpath. The gradle version shouldnt matter and fwiw neither should the AGP/AKP version since they let you apply using either method.

Per https://docs.gradle.org/current/userguide/plugins.html#sec:plugins_block it appears that we should be checking settings.gradle first before build.gradle for the plugin version, which should also include AGP/AKP.


Create ability to show incremental progress on tasks

Done.
The task detail page now shows progress in 10% increments with the ability to set a value. A label shows partial progress (if available) on the task list view as well.


Hystrix stream aggregation

Fixes gh-131

closed in favor of a57c358702147aa356e4246fe96732cfd748d2e0
Thanks for the pull request though.


Don't follow :latest for redis

Hit a weird issue with the latest Redis breaking our automation.
redis.exceptions.ResponseError: DENIED Redis is running in protected mode because protected mode is enabled and no password is set for the default user. In this mode connections are only accepted from the loopback interface. If you want to connect from external computers to Redis you may adopt one of the following solutions: 1) Just disable protected mode sending the command 'CONFIG SET protected-mode no' from the loopback interface by connecting to Redis from the same host the server is running, however MAKE SURE Redis is not publicly accessible from internet if you do so. Use CONFIG REWRITE to make this change permanent. 2) Alternatively you can just disable the protected mode by editing the Redis configuration file, and setting the protected mode option to 'no', and then restarting the server. 3) If you started the server manually just for testing, restart it with the '--protected-mode no' option. 4) Setup a an authentication password for the default user. NOTE: You only need to do one of the above things in order for the server to start accepting connections from the outside.

Signed-off-by: Joe Talerico<EMAIL_ADDRESS>Description
Fixes

All checks have failed, good time to merge.

Hey @jtaleric, I think we can close this one in favor of https://github.com/cloud-bulldozer/benchmark-operator/pull/752, which is based in version 6.2.6

We fixed it locally in our manifests! Glad yall have it fixed too.


[Enhancement] Panel color/transparency in settings

Panel is currently set to a static color.  Making this able to be customized would be really nice.  It would complicate things for the logo ("start" button).

This should be no problem at all. Thanks!

@MrSchism Just commited some changes that will allow you to change both the background and opacity of panel :smile:


Unable to parse comparison vcf file

Hi, I want to compare my vcf file (generated with whatshap) with GIAB SV v.0.6 (SVs >= 50bp, GRCh37) by Truvari. But when I run this command:

truvari bench -c phased_hic.vcf.gz -b HG002_SVs_Tier1_v0.6.vcf.gz -f ref.fa -o ./out

it leads to this error:

2021-06-15 01:07:17,577 [INFO] Running /***/truvari bench -c phased_hic.vcf.gz -b HG002_SVs_Tier1_v0.6.vcf.gz -f ref.fa -o ./out
2021-06-15 01:07:17,577 [INFO] Params:
{
"base": "HG002_SVs_Tier1_v0.6.vcf.gz",
"comp": "phased_hic.vcf.gz",
"output": "./out",
"reference": "ref.fa",
"giabreport": false,
"debug": false,
"prog": false,
"refdist": 500,
"pctsim": 0.7,
"buffer": 0.1,
"pctsize": 0.7,
"pctovl": 0.0,
"typeignore": false,
"gtcomp": false,
"bSample": null,
"cSample": null,
"sizemin": 50,
"sizefilt": 30,
"sizemax": 50000,
"passonly": false,
"no_ref": false,
"includebed": null,
"multimatch": false
}
2021-06-15 01:07:17,674 [INFO] Creating call interval tree for overlap search
2021-06-15 01:07:17,676 [ERROR] Unable to parse comparison vcf file. Please check header definitions
2021-06-15 01:07:17,676 [ERROR] Specific error: "invalid contig 7"

Is there something wrong with the vcf file? Could you please help me solve this problem? And could you please talk more about the how to prepare reference.fasta? Does the reference.fasta indicates the one which I used for generate vcf file with whatshap?
Thank you very much.

Hello,
We've documented common challenges with using VCFs on the wiki with a link to a tool that can help you validate your files.
To answer your question directly, it seems you probably want to check your header lines for their definitions of each ##contig. The reference should be the reference over which your base and comparison calls were made. Since you're using GIAB v0.6, that reference should be hg19.
Have a great day,
~/Adam


Reject malformed modelines

The parsing, at the moment, is very permissive.
Improperly formatted modelines that do not work on Vim should be processed the same way or rejected.
Ideally, syntax errors could be highlighted and noted within the editor to prevent Sublime Text users adopting invalid syntax.

Commit 9bc03b5ef7ddae8da1754acd47af2c69ea8eaa15 added more support for alternative syntax, while commit<PHONE_NUMBER>045106a49dbf845af797a11afb2ad0 added more restrictions. Tests were added, but there could still be some unhandled cases. There is still no syntax highlighting.


Fix to handle phylabel and ifname for network interfaces being different

Am still testing this.. I will test this on kbox tomorrow.. and let u guys know..

I think you should add a check in parseconfig.go to make sure IfName is set for all with port.Ptype

Good point. We should atleast generate an error for this. Better have this check in the cloud as well. Will open a bug for that.

Otherwise you'll be passing empty strings into the functions you've changed to use IfName.


Git fatal ... File is outsite repository

Issue Type: Bug
Today I just upgrade to new version and got this problem. I also face this problem in a build before but it fixed.
This error message display when I tried stag file changes. But it's working in terminal

I've tried disable all extension but still got this problem. Does someone also got this problem like me ?
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T09:03:20.462Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4800MQ CPU @ 2.70GHz (8 x 2694)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.81GB (7.59GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Yes.

/duplicate

Thank @gjsjohnmurray for reference


Error 500 on submit login

Hello, I’m encountering some issues after the wizard configuration.
I used docker-compose and followed the steps in the documentation.
I received a 500 error after submitting the login form and was able to retrieve this error log from the container volumes.
[2024-07-06T12:07:30.785418+00:00] request.CRITICAL: Uncaught PHP Exception Symfony\Component\Filesystem\Exception\IOException: "Cannot rename "/tmp/url_matching_routes.phpgv5MnI" to "/mosparo-data/var/cache/prod/url_matching_routes.php": rename(/tmp/url_matching_routes.phpgv5MnI,/mosparo-data/var/cache/prod/url_matching_routes.php): Permission denied" at Filesystem.php line 312 {"exception":"[object] (Symfony\\Component\\Filesystem\\Exception\\IOException(code: 0): Cannot rename \"/tmp/url_matching_routes.phpgv5MnI\" to \"/mosparo-data/var/cache/prod/url_matching_routes.php\": rename(/tmp/url_matching_routes.phpgv5MnI,/mosparo-data/var/cache/prod/url_matching_routes.php): Permission denied at /mosparo/vendor/symfony/filesystem/Filesystem.php:312)"} []

Hi @ghpaini
Thank you very much for your issue.
The error message generally means that mosparo inside the Docker image cannot write to the Docker volume.
If you used the provided docker-compose.yml file, it should be no problem at all. But can you verify that the volume exists correctly? You can use docker volume ls to list all volumes and docker volume inspect <name> to see all the volume details.
Additionally, you can use docker exec -it 'mosparo-docker-mosparo_web-1' /bin/bash (adjust the name mosparo-docker-mosparo_web-1 of the Docker container if needed) to get a shell inside the Docker container. You can use ls -la /mosparo-data/var/cache/ in the container. What's the result of this command?
Kind regards,
zepich

Hi @zepich
Thank you for the support 😄.
I was only able to get the error log through the mosparo_db_data volume, see the results of docker volume ls:
DRIVER    VOLUME NAME
local     mosparo_db_data
local     mosparo_mosparo_data
And this is the result of  ls -la /mosparo-data/var/cache/:
drwxr-xr-x 3 www-data www-data 4096 Jul  6 14:04 .
drwxr-xr-x 4 www-data www-data 4096 Jul  5 20:33 ..
drwxr-xr-x 8 www-data www-data 4096 Jul  6 14:04 prod

Hi @ghpaini
Thank you for the outputs.
Have you tried to log in only once or multiple times with the same error?
Do you have enough free space on your server?
I can see that the cache directory was created. I guess it was when you installed mosparo. So technically, there should be no reason for the login not working.
Kind regards,
zepich

I've tried only a couple of times.
I’m running MOSPARO on my machine, only to try out the application.
The cache directory was created when I first opened the localhost:8080 on my browser.
I’ll wait until Monday to try and run Mosparo on another machine with more free space (I have about 20 GBs), but I don’t think that’s the cause of my issue here.
If it’s worth mentioning, I am running the images on my Windows computer.
I cleaned everything (images, containers and volumes) and tried to run again, but got the same 500 error after login. Although
this time the error was SSL related:
[2024-07-06T16:42:14.874336+00:00] request.CRITICAL: Uncaught PHP Exception Symfony\Component\HttpClient\Exception\TransportException: "SSL certificate problem: self-signed certificate in certificate chain for "https://updates.mosparo.io/v/1.2.json"." at ErrorChunk.php line 56 {"exception":"[object] (Symfony\\Component\\HttpClient\\Exception\\TransportException(code: 0): SSL certificate problem: self-signed certificate in certificate chain for \"https://updates.mosparo.io/v/1.2.json\". at /mosparo/vendor/symfony/http-client/Chunk/ErrorChunk.php:56)\n[previous exception] [object] (Symfony\\Component\\HttpClient\\Exception\\TransportException(code: 0): SSL certificate problem: self-signed certificate in certificate chain for \"https://updates.mosparo.io/v/1.2.json\". at /mosparo/vendor/symfony/http-client/Response/CurlResponse.php:315)"} []

Hi @ghpaini
Thank you very much for your answer and the details.
No, if you have 20 GB of free space, that should be way more than enough, so that's not the problem (but sometimes, no space left creates very interesting issues, so I wanted to make sure that that's not the problem in your case).
Thank you very much for the information about Windows. I've just tested it on a Windows machine. I installed Docker Desktop (with WSL), restarted the machine, and used the Docker compose file we provided in our documentation to create the containers. Everything worked without any problem.
Are any files inside the var/cache/prod directory?
Regarding the SSL issue: Do you use any tool to block outgoing communication (Antivirus/Firewall)? Can you test with ping updates.mosparo.io to which IP address it resolves? There should be no such error since we're using valid certificates.
Kind regards,
zepich

Hi @zepich
There is some files in var/cache/prod:
drwxr-xr-x   8 www-data www-data  4096 Jul  6 16:41 .
drwxr-xr-x   3 www-data www-data  4096 Jul  6 16:41 ..
drwxr-xr-x   2 www-data www-data 36864 Jul  6 16:41 Container1N2uDDh
-rw-r--r--   1 www-data www-data   875 Jul  6 16:41 Mosparo_KernelProdContainer.php
-rw-r--r--   1 www-data www-data     0 Jul  6 16:41 Mosparo_KernelProdContainer.php.lock
-rw-r--r--   1 www-data www-data 90643 Jul  6 16:41 Mosparo_KernelProdContainer.php.meta
-rw-r--r--   1 www-data www-data 61907 Jul  6 16:41 Mosparo_KernelProdContainer.preload.php
drwxr-xr-x   3 www-data www-data  4096 Jul  6 16:41 Symfony
-rw-r--r--   1 www-data www-data    98 Jul  6 16:41 annotations.map
drwxr-xr-x   3 www-data www-data  4096 Jul  6 16:41 doctrine
drwxr-xr-x   4 www-data www-data  4096 Jul  6 16:41 pools
-rw-r--r--   1 www-data www-data   100 Jul  6 16:41 serialization.php
drwxr-xr-x   2 www-data www-data 12288 Jul  6 16:41 translations
drwxr-xr-x 110 www-data www-data  4096 Jul  6 16:41 twig
-rw-r--r--   1 www-data www-data 40024 Jul  6 16:41 url_generating_routes.php
-rw-r--r--   1 www-data www-data  9346 Jul  6 16:41 url_generating_routes.php.meta
-rw-r--r--   1 www-data www-data 25491 Jul  6 16:41 url_matching_routes.php
-rw-r--r--   1 www-data www-data  9346 Jul  6 16:41 url_matching_routes.php.meta
-rw-r--r--   1 www-data www-data  5968 Jul  6 16:41 validation.php
-rw-r--r--   1 www-data www-data   745 Jul  6 16:41 webpack_encore.cache.php.
The resolution for updates.mosparo.io was mosparo.io [<IP_ADDRESS>], but the ping received no response.
Is it mandatory to have access to mosparo.io domains? Because, I really don’t think I can change anything in the firewall of my computer here. I’ll give it a try in another Windows environment that I have, to validate if it’s some kind of Firewall or Antivirus block.
I forgot to mention some changes that I have made to the configuration files. My Web container wasn’t showing any errors at all. It was only some “display_error” flags that might explain why I am not getting the first error anymore.

Hi @ghpaini
Thank you very much for your response.
No, there is no need to have access to the mosparo.io domains. You can turn off the automatic update check by setting the following environment variable:
MOSPARO_AUTOMATIC_UPDATE_CHECK_ENABLED=0
You have to add it to the Docker compose file like this:
services:
  db:
    # We recommend the mariadb image
    image: mariadb:10.10
    # If you want to use MySQL, uncomment the following line (and comment the one above)
    #image: mysql:8.0.27
    command: '--default-authentication-plugin=mysql_native_password'
    volumes:
      - db_data:/var/lib/mysql
    restart: always
    environment:
      - MYSQL_ROOT_PASSWORD=mosparo_root_pw
      - MYSQL_DATABASE=mosparo
      - MYSQL_USER=mosparo
      - MYSQL_PASSWORD=mosparo_password
    expose:
      - 3306
      - 33060
  mosparo_web:
    image: mosparo/mosparo:latest
    ports:
      - <IP_ADDRESS>:8080:80
    restart: always
    environment:
      - MOSPARO_ENABLE_WEBSERVER=1
      - MOSPARO_ENABLE_CRON=0
      - MOSPARO_AUTOMATIC_UPDATE_CHECK_ENABLED=0  # Disable update check
    volumes:
      - mosparo_data:/mosparo-data
  mosparo_cron:
    image: mosparo/mosparo:latest
    restart: always
    environment:
      - MOSPARO_ENABLE_WEBSERVER=0
      - MOSPARO_ENABLE_CRON=1
      - MOSPARO_AUTOMATIC_UPDATE_CHECK_ENABLED=0  # Disable update check
    volumes:
      - mosparo_data:/mosparo-data
volumes:
  db_data:
  mosparo_data:

I will check that one server, but generally, it should not return wrong SSL certificate.
Can you check with this Docker compose file if you still have the first error?
Kind regards,
zepich

It worked!! 👍👍
After another clean setup, the login worked and now I can see the project screen.
I'll try the 'default' version of the docker-compose.yml in another environment.
For now, thanks for your support and I'll let you know if I run through any problems. 🔥
Kind regards,
ghpaini.

Hi @ghpaini
Awesome! Great to hear! Thank you for letting me know!
Kind regards,
zepich

I close this issue. If you are still encouraging these problems, please post here or open a new one.


Typo: omitt --> omit.

I see you've previously signed the CLA, so merging. Thanks, @charleswhchan

Thanks @beckettsean


library version up

aws-lambda-java-events : latest version is 1.3.0

Thank you! @takashima0411

@takashima0411 version 0.7.0 with your change is available in Maven Central.


How to implement Forgot Password

How to implement Forgot Password option in my next-js project. I can't find any solution in readme docs or full example please help me to solve this... I am using Email, Password authentication in my project
Please give me some resources

Hi @Darpan-favfly you can build your own forgot password system.
You have the user schema with the password of each user stored in your sanity project, the only requirement are to hash the passwords with argon2 when you save the new password.
For example too simple path:

Get the email in a field and submit the form
Check if the user is real with a sanity query *[_type == 'user' && email == $email][0]
Store a unique token with expiration time in the user schema or another schema or in another db like a redis with some relation to user
Send an email with one link like https://fake-app.com/recover-password?token=$forgotPasswordToken
In that route you can find the token and know if it's related to user
Hash the new password with argon2 and replace the old one with a mutation

I can't put things like this in the lib, because they don't make sense. this lib just store and finds data, you are open to do whatever you want with it.


Make legacy librealsense compatible with recent kernel

This should make it work with 4.16+ kernel
Tested with a R200, not sure about the behavior with other cams

This seems much nicer than what I did in #3749. That was basically just a desperate hack because we needed it to work ASAP. I'd much rather merge this than #3749.

Hi,
I have been playing with the modified code, it works fine with R200 and SR300 but it does not work with the ZR300 camera... I have made some modifications to uvc-v4l2.cpp in order for this camera to work.
Modifications:
To add support for ZR300 the same condition "if(sub->is_metastream)" has to be added after the line 781, otherwise it uses one of the metastreams and throws the following error: VIDIOC_S_FMT: 22, and after the line 822 as otherwise it will not add correctly the Fish_eye camera.
Thanks!

Hi,
I have been playing with the modified code, it works fine with R200 and SR300 but it does not work with the ZR300 camera... I have made some modifications to uvc-v4l2.cpp in order for this camera to work.
Modifications:
To add support for ZR300 the same condition "if(sub->is_metastream)" has to be added after the line 781, otherwise it uses one of the metastreams and throws the following error: VIDIOC_S_FMT: 22, and after the line 822 as otherwise it will not add correctly the Fish_eye camera.
Thanks!

Good to know, please PR

I pushed a small changes which makes multiple R200 works on my side; and also did (I think) the modification that @marcmateumateus suggested for the ZR300 (but without testing it)

Add label F200 because it definitely worked for me!  I didn't need to make any changes on this branch to compile / use it on linux, kernel 4.18.16 :heart:  Thank you very much for figuring this stuff out @doisyg!

Hi, I can confirm that it works with ZR300, sorry for not PRing, i'm kind of new to GitHub and i do not know how...
Thanks @doisyg for the changes.
Add label ZR300 and SR300 as i tested both of them.

Add label ZR300 and SR300 as i tested both of them.

Good to know!

@dorodnic What do you think about merging / realeasing a new version of the legacy driver? Even better releasing a new deb / ros package for librealsense

It would probably make sense to remove the #include <iostream> and std::cout now?

@doisyg - I’m for it. Got some r200s to test home with but didn’t have time yet.
Not sure regarding debians, but will take a look
(We can update the rosdeb branch, but from there I’m not sure what was the flow and if it is still working)
Sent with GitHawk

I am sure @SteveMacenski will be happy to see this merged and released too.

Tested on <IP_ADDRESS> with R200 and LR200, both work perfectly fine with the patch.
@doisyg please consider removing cout since its quite intrusive for any user app, otherwise it's good to merge

@doisyg please consider removing cout since its quite intrusive for any user app.

Done

Also, I still need to check the ZR300 issue...

I think @marcmateumateus confirmed that the issue is now solved:

Hi, I can confirm that it works with ZR300

Thank you!


Enable Hackage-friendly stack.yaml settings

This helps ensure that the cabal meta-data is more accurate and so that install-plans don't bitrot that easily over time.
See also https://docs.haskellstack.org/en/stable/yaml_configuration/?highlight=pvp-bounds#pvp-bounds

Thank you 😃
LGTM, merging this. 💯


Change the display images.

There are memes in the homescreen slideshow, remove all of them and add dummy images for now or previous even posters.

@kpbot claim


Update README.md self checkout

please verify the updated clone commands for the new repo are accurate.

@riacheruvu  please review it and merge if ok


ws.wait_until_identified times out causing NotIdentifiedError

Hello, I'm trying to run sample_request.py from the samples, but I can't seem to get a connection to my obs websocket.
When I run sample_request.py this is what I see in the obs logs:
00:55:04.121: ------------------------------------------------
00:55:04.404: [obs-websocket] WSServer::start: Not locked to IPv4 bindings
00:55:04.407: [obs-websocket] server started successfully on port 4444
00:55:04.407: [obs-websocket] IO thread started.
00:55:11.581: [obs-websocket] new client connection from [::ffff:<IP_ADDRESS>]:57954
00:56:51.609: [obs-websocket] Websocket connection with client 'Unknown' closed (disconnected). Code is 1006, reason is: 'End of File'

I tried increasing the timeout parameter to wait_until_identified but that only caused a delay before repeating the same behavior.
Any pointers on how to debug this?
System info:
Ubuntu 20.04, using obs-websocket-4.9.1 and simpleobsws installed via pip install which gave me version 1.3.1

And immediately after posting this I read the release notes here: https://github.com/IRLToolkit/simpleobsws/releases/tag/1.2.1 which say releases 1.1/2/3 are not compatible with obs-websockets-4.X  It looks like obs-websocket-5.0.0 is in pre-release alpha, is there a particular version I should grab?

The protocol between 4.x and 5.x is completely different. Either upgrade to 5.0.0 and use 5.x requests, or use 4.9.1 with simpleobsws <1.0.0


Fix issues with venv object for tox>=3.8.0 (on py36)

I don't know if the tests work, but the code is working properly on my machine now, python 3.6.8.

Ran into what I believe is the same issue on Python 3.7.3:
... snip ...
  File "/Users/tom/.local/share/virtualenvs/tools-YTG8DW9i/lib/python3.7/site-packages/tox_pipenv/plugin.py", line 164, in tox_runenvreport
    pipfile_path = _clone_pipfile(venv)
  File "/Users/tom/.local/share/virtualenvs/tools-YTG8DW9i/lib/python3.7/site-packages/tox_pipenv/plugin.py", line 22, in _clone_pipfile
    root_pipfile_path = venv.session.config.toxinidir.join("Pipfile")
AttributeError: 'VirtualEnv' object has no attribute 'session'

Haven't had a chance to verify @chiqomar's attempted fix yet.

@tonybaloney @AlmogCohen Any chance this can be reviewed and moved forward? I tested the fix and works fine (minus the line 37 change in plugin.py - the original works for me with tox 3.14)

Superseded by #66


Split upx run from the default make target

May be a silly change, but since upx takes a while to run, it's nice to take it out of the default target.

--split-objs is also super slow. You should just use stack build if you'd not building a release.


⚠ Bofrost.at has degraded performance

In a66c77a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1746 ms

Resolved: Bofrost.at performance has improved in b02f517.


ATV.at: fix extractor using the new jwt JSON Web Token

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Fix the ATV.at extractor as the site changed
Instead of using the external python module jwt from the python-jwt
package implement the web token creation is done following RFC7519.
The page uses a simmilar player as pro7 ans sat1, but different.
Fixes: https://github.com/yt-dlp/yt-dlp/issues/988

How unique? For just the playlist currently downloaded, or for the whole site?
On September 19, 2021 5:01:13 AM GMT+02:00, pukkandan @.> wrote:
@. commented on this pull request.

     return {

       'id': part_id,

       'title': entry.get('title'),

       'duration': int_or_none(entry.get('duration')),

       'formats': formats

       'id': str(clip_id + 1),  # start counting at 1 for the clips

actually, is clip_id even unique? The id field (along with extractor) must be a unique identifier of each video
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/yt-dlp/yt-dlp/pull/1012#discussion_r711673418
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

for the whole site. The download-archive works based on id.
Also, if a video can be downloaded through multiple URLs, all of them should give the id

I've got the following in the content variable
{
  "id":          "v-ce9cgn1e70n5",
  "tvShowId":    "745842",
  "tvShowTitle": "Bauer sucht Frau",
  "seasonId":    "s-cap22abg9hs9",
  "upc":         "v-ce9cgn1e70n5",
  "splitId":     "v-ce9cgn1e70n5-1"
}

I think the splitId field would be the best match. Would the other fields be useful for other extracted attributes?

only splitId and title seem usefull. But I believe u r already getting the title

rebased on current master and squashed commits


[1.11.x] Fixed #28441 -- Fixed GEOS version parsing with a commit hash at the end.

Less invasive than https://github.com/django/django/pull/8817 so as to minimize the chance of a problem on a stable branch.

@timgraham I'm using the 1.11.4 and I got the same error

@mrkaspa it will be part of 1.11.5 which should be released in the first week of September.

Great


fix(elements|ino-radio-group): trigger valueChange upon clicking

Closes #886

Works like a charm, good job 💪
But I gave you some improvements for the storybook entry
@MariaLStefan
Thank you for your detailed review. It's been some time since I last wrote stories and I genuinely appreciate the insightful feedback with references to other components 👌


add time after date shortcuts

https://github.com/giovannicoppola/AlfreDo/discussions/15

added in version 0.4


support on ppc64, ppc64le and s390x

I just did a test...i build keydb,last release on s390x, ppc64 and ppc64le... the keydb program execute but... the server just dont start... i dont get any results/prints on screen..

It's possible to get a free access to a s390x machine, on this link: https://linuxone.cloud.marist.edu/


Minor typo

[On this sentence, I think it should be safe instead of save: "so it is save to assign updates from multiple channels to a device."]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d23ac821-9cb8-8855-598f-d1327df04fac
Version Independent ID: 78f2127a-0c6b-d236-a415-6215c582ce8b
Content: Change the Microsoft 365 Apps update channel for devices in your organization - Deploy Office
Content Source: DeployOffice/change-update-channels.md
Service: o365-proplus-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

@LindaCicero Thanks for catching that and letting us know. We've fixed the issue and it should be published live later this afternoon.


Memory Requirements - Reg

Hi,
I have few questions regarding NuScenes dataset.

For LIDAR-only Nuscenes data excluding radar, camera info, is it enough to download metadata file and lidar blobs only files of each part or do we require to download all the data?
How much RAM and GPU memory is needed to work on 1000 scenes of NuScenes dataset i.e., to train 700 scenes of nuscenes dataset?

Hi, I have few questions regarding NuScenes dataset.

For LIDAR-only Nuscenes data excluding radar, camera info, is it enough to download metadata file and lidar blobs only files of each part or do we require to download all the data?
How much RAM and GPU memory is needed to work on 1000 scenes of NuScenes dataset i.e., to train 700 scenes of nuscenes dataset?

wechat<PHONE_NUMBER>6 we can comunicate

wechat is not available in playstore. Can u pl text here?

wechat is not available in playstore. Can u pl text here?

sure，we can go to<EMAIL_ADDRESS>

Have solve the problems? I have a  GPU A6000 48G ,but when i train the cam branch ,it is out of memory


combine actions in a layer

I'm not sure whether this should be it's own issue it's somehow related to #64
Let's assume I draw this brain and then I want to rotate the brain. How can I do this?
This seems to be a big probably which is currently in my head.
A different example might make this clearer;
I want to draw a graph with nodes and edges and animate some stuff and then I want to scale it down and move it to a corner of the animation just to still show it and draw something else.
We somehow need to be able to combine actions into a layer than can be accessed later. I neither know a good interface for it nor how to implement it so this is just one idea:
Action(1:100, draw_node(1); subactions=[SubAction(1:20, appear(:fade)], layer=:graph)
Action(1:100, draw_node(2); subactions=[SubAction(1:20, appear(:fade)], layer=:graph)
Action(101:200, layer(:graph), Translate(200, 0)

And it would pick the latest version of that layer then which I think makes sense. Any other ideas just for the user interface @TheCedarPrince ?

I was actually thinking something similar - especially in regards to scaling. As of right now, I put this under milestone v0.2.0 as I think this would take a bit of time to implement. I like the syntax you described. Here are some options that I can think of:
Example 1

graph = Layer([Action(1:100, draw_node(1); subactions=[SubAction(1:20, appear(:fade)], layer=:graph),
                        Action(1:100, draw_node(2); subactions=[SubAction(1:20, appear(:fade)], layer=:graph)])

javis(video, [
                    graph,
                    Action(101:200, graph, Translate(200, 0))
                    ])

With this idea, I could see defining a layer becoming very confusing within the general javis function. Therefore, perhaps a better user interface might be to create a Layer object that one could create outside the javis function wherever they want. This might enable a cleaner solution as one could easily read what all is happening in the Layer object. Inside javis, the Layer object would be seen and know it would be a set of Actions to be executed. This is not dissimilar to the SubAction that you implemented earlier. Could maybe make Layer be a subtype of AbstractAction perhaps?
Example 2
This takes place inside a Javis function

Collection(1:100, :graph)
Collection(:same, :graph, draw_node(1); subactions=[SubAction(1:20, appear(:fade)])
Collection(:same, :graph, draw_node(2); subactions=[SubAction(1:20, appear(:fade)])
CollectionAction(Rel(100), :graph, Translate(200, 0)

This is similar to your original idea that you had but rather than calling this system a "layer", I think the better syntax might be "Collection". The reason why I suggest this is that a layer to me suggests everything on that current frame. However, a collection refers to a subset of what is present in a frame. A Collection could be a subtype of AbstractAction which could "know" to identify specific actions as part of a subset of the present frame.
Example 3
graph = Collection([Action(1:100, draw_node(1); subactions=[SubAction(1:20, appear(:fade)], collection=:graph),
                        Action(1:100, draw_node(2); subactions=[SubAction(1:20, appear(:fade)], collection=:graph)])

javis(video, [
                    graph,
                    CollectionAction(Rel(100), graph, Translate(200, 0))
                    ])

I think this is my favorite syntax idea as it captures concepts from both prior examples. Here, again, I posit that the syntax should be Collection instead of Layer as Collection can refer to all objects in a given frame or some of the objects in a given frame. The collection kwarg also reflects this idea by saying that this is a collection of objects from the current frame. I think the CollectionAction can also provide additional utility as it allows special actions to be taken upon Collections.
Conclusion
These are just some thoughts, but let me know what you think. @Wikunia

Thanks for your thoughts! Some more :wink:
I'm not sure I understand your confusion (okay I'm confused by your comment :smile: ) on Layer vs Collection.
In Photoshop as an example you have several layers for one "frame" (the image). And currently we have a Luxor @layer for each action.
I would like to add that in my way you can define it outside or inside javis i.e:
Action(1:100, draw_node(1); subactions=[SubAction(1:20, appear(:fade)], layer=:graph)
Action(1:100, draw_node(2); subactions=[SubAction(1:20, appear(:fade)], layer=:graph)
Action(101:200, layer(:graph), Translate(200, 0)

can be written as
graph = [Action(1:100, draw_node(1); subactions=[SubAction(1:20, appear(:fade)], layer=:graph),
              Action(1:100, draw_node(2); subactions=[SubAction(1:20, appear(:fade)], layer=:graph)]

inside javis:
graph...,
Action(101:200, layer(:graph), Translate(200, 0)

I was thinking about something like example 2 as well but I don't like the many symbols we already support in the first 2 positions.
About Example 1 and 3. We don't need the extra kwarg there, do we?

A new idea:
video = Video(500, 500)
layer = Layer()
draw_circle = Action(1:10, (args...) -> circle())
appear_sub = SubAction(1:5, appear(:fade))
draw_circle += appear_sub
layer += draw_circle
render(video)

maybe it makes sense in general to only have some very small things in v0.2.x and release v0.3 with these breaking changes quite quickly without having a lot of v0.2.x changes. What do you think @TheCedarPrince

Hey @Wikunia , thanks for adding more thoughts.
So, after thinking about this syntax for a bit, I feel like some of it is obfuscating while other parts of it I very much like.
Parts I Think We Should Definitely Implement

I really like the idea of how you defined Layer() similar to creating a Video object. And I really appreciated the syntax of layer += draw_circle. I think that makes sense for Layer syntax. Could you also stack layers on top of each other? Such that one Layer becomes an object in another Layer? The reason I ask for the conversion of a Layer into a generic object inside of another Layer is so that we could perform transformations on the "child" Layer hosted inside of the "parent" Layer. I'd love to have syntax like this:

layer1 = Layer()
layer2 = Layer()
...
layer_action = Action(1:10, layer1)
layer_action += SubAction(1:5, scale(1, 0.1))
layer2 += layer_action

Essentially, I am trying to figure out the best way to also handle the syntax of combining multiple layers together and mutating an entire layer at once.

I think the idea of creating assignment for individual Action and SubAction objects is a great idea. I think to an extent we have already been doing this by storing lists of Action objects into a single variable (like Tutorial 6). I frankly could see the generic syntax of ideas like draw_circle = Action(1:10, (args...) -> circle()) and appear_sub = SubAction(1:5, appear(:fade)) being the foundation of solving the convenience function idea (#130). I feel like we could build upon this generic syntax pretty easily for future functionality.

How are you thinking about implementing a Layer object currently?

Parts I Think Could Be Problematic

Building off of 2. in the first section, one issue that I see being problematic is how to handle function returns for Action objects such as draw_circle. With the above syntax, it is not clear to me how to reimplement Tutorial 1 using this syntax for example as I do not see how we could make id syntax work while also being confusing. If we stick with id syntax, I could see easily getting confused with having an Action stored into a variable, and having another id name represent the return of the function in the Action that was assigned to a variable. It feels like it could get messy very fast.

Could we clear a Layer object? The reason I ask this is that I would like to be able to render more than one video per Julia script. The rationale for this is if someone wants to batch the creation of animations for various phenomena, I would not want to have them run the script, tweak variables, rerun the script ad nauseam.

Parts I Am Confused About

What happened to the javis function? I miss the javis function! It's like our slogan. :joy:

Regarding render, I don't understand why we can't pass a layer directly to the function. I am not sure if I like this syntax very much if it is to replace the javis function as I do not like not having direct control about what goes into the render function.

Was the lack of a BackgroundAction intentional or not considered for this example?

Conclusion
These were just some of my working thoughts on the proposed syntax. I really like the direction you are going with this! Especially the += operator syntax I think can be very powerful. Was that coined by @xiaodaigh in #179?

+= operator syntax

As mentioned elsewhere R's ggplot 2 uses + to compose layers. Seems like a good an intuitive way

Regarding the id problem: I would remove the symbol variant then and one can write:
red_ball = Action(1:10, (args...)->draw_circle("red"))
blue_ball =  Action(1:10, (args...)->draw_circle("blue"))
blue_ball += SubAction(1:10, Rotation(0, 2*pi, red_ball)

I'm not a fan of using layer_action = Action(1:10, layer1) why not layer1 += ?
If you want several videos in one julia script you just need to define a new Video this has nothing to do with Layers.
I forgot the BackgroundAction but also thinking that we might want to have a default one.

Just a small update. We decided against using + as we mutate the object which is normally not done with +.
For adding an action to an object we use act! now. We need something similar for adding an object to a layer.

The full discussion is freely viewable here if you want to check it out @xiaodaigh : https://julialang.zulipchat.com/#narrow/stream/253420-javis/topic/Taking.20Object.20Addition.20Seriously/near/213057763
Thanks for your thoughts as they were super valuable!

shouldn't mutate but crest a bew object with stuff added

If we create a ball and move it we don't create a new one but instead move the old one therefore the object should be mutated.

Good point


Merge volatile module, store persistent and volatile module in root dir

What is changed and how it works?
Cherry-pick volatile branch, store persistent and volatile module to independent directories.
Check List
Tests

Unit test
Stress test

Side effects
No

Yes, this is a todo task. Also, as c API can't be isolated by namespace, we may need to do some rename work.
Yes, like Status and common types.

Just 2 questions:

Will we use different namespaces for persistent and volatile, in the future? Otherwise, these 2 modules may not be compiled into one app.
Also, will there be some common parts being extracted from both modules, e.g., header files?

Yes, this is a todo task. Also, as c API can't be isolated by namespace, we may need to do some rename work.
Yes, like Status and common types.


Publish a few trailing versions of the documentation on ReadTheDocs

After an issue here and a brief discussion on Slack, it's become clear that we should start keeping a few of the older versions of the documentation up on ReadTheDocs. Perhaps 3-5 versions back from the current one? This would mean changing the docs from a "Stable/Latest" approach to a versioned approach based on more tags. Should be very easy to configure.

A few older versions have been built on RTD. The number and choice of tagged versions available will need to be managed manually.

Latest continues to point to master branch, and stable points to last tagged version.


[CodingStandard] complete README and few checkers

ok, I think that we make pretty deep analysis with @SpacePossum ;P
going sleep ;)

Thanks you guys :heart: , me too.
I have some ideas how to make improve this process thanks to you  :+1:
GN

Thanks for feedback, buys.
I've added few changes and made it work for normal and static properties.
I merged this, because it drags too long and I couldn't remember my work weeks ago :(
Saying that, any additional feedback is appreciated.

Nice to see this merged, I would say your fixer is good for what you want to do, so it nice work done : D
Let this be a two way street, if you want me to have a second look on something for refinement or whatever, just ping me :+1:

Thanks for your affirmation, it means a lot to me from you.
Thanks for your feedback offer as well. I will definitely use it. I'd rather wait for next fixer, so it motivates me to do it sooner :)


Detect and fix misconfigurations and security anti-patterns in Ferdi with electronegativity

Is your feature request related to a problem? Please describe.
I recently discovered the library https://github.com/doyensec/electronegativity and think it would be a great idea to fix reported issues.
There are good explanations of the individual security issues that can arise and how to run it here: https://github.com/doyensec/electronegativity/wiki
Apart from making the application more secure, it is also a great learning experience about Electron security.

Nice find @mhatvan - please also see: https://github.com/marketplace/actions/electronegativity - maybe we can incorporate this into the builds?

Yeah, saw that too!
I would suggest first testing out the lib, if it turns out to be great help for us, we can use it in the builds.
Whenever I find time I will testdrive this lib ;)


"Update to this version instead" is a relative link

From my gmail "show original"
## Version **2.4.1** just got published.=20
[Update to this version instead =F0=9F=9A=80](/halkeye/proxy-s3-google-oaut=
h/compare/greenkeeper%2Fdebug-2.4.1?expand=3D1)=20

Update to this version instead <g-emoji alias=3D"rocket" fa=
llback-src=3D"https://assets-cdn.github.com/images/icons/emoji/unicode/1f68=
0.png" ios-version=3D"6.0">=F0=9F=9A=80</g-emoji>

Thank you for reporting this, we are going to roll out a fix soon.

This fix is live since Dec 16, but we forgot to close the issue 🙈
Thanks for reporting it :)


Write a guide for debugging (not even WIP)

This isn't particularly coherent so far but I figured it's worth starting something.
The goal would be that when something breaks, you know what tool you need to reach for and how to use it to decrease the search space.
Somewhat related might be problem prevention.
At least the following should be covered/TODO:

search the manuals for keywords such as debug
nix evaluation

tracing nix expressions, the various lib tracing functions, etc

breakpointhook ( + check the manuals)

list reference: https://www.youtube.com/watch?v=ULqoCjANK-I

seq and laziness
printing some things seems to quit the interpreter, I'm not sure whats going on there?
NIX_DEBUG
https://nixos.wiki/wiki/Nixpkgs/Create_and_debug_packages + check wiki for other materials
recursive enableDebugging? (on dependents / deps?) (additionally: would be nice to have homogeneous debug enable infrastructure...but thats not a documentation problem.. i.e. a magic enableDebugging that works on everything)
something to help with noticing failing dlopen calls or somesuch
testing expressions ( https://github.com/deliciouslytyped/nix-expr-testlib ? )
Edit:
probably strace , though scope creep to general system debugging? on the other hand nix usage is very broad so...

Relevant papercut thread / forum posts:

https://discourse.nixos.org/t/the-papercut-thread-post-your-small-annoyances-confusions-here/3100/40?u=joepie91
https://discourse.nixos.org/t/the-papercut-thread-post-your-small-annoyances-confusions-here/3100/16?u=joepie91
https://discourse.nixos.org/t/debugging-broken-builds/3138

More:

https://discourse.nixos.org/t/the-papercut-thread-post-your-small-annoyances-confusions-here/3100/58?u=joepie91

Also this factoid from the bot:
Use NIX_COUNT_CALLS=1 and/or NIX_SHOW_STATS=1 to profile Nix evaluation

NIX_SHOW_STATS_PATH=stats.json will also make recent versions of nix to print those stats to a file, rather then stderr
then you can jq < stats.json '"heap size: \(.gc.heapSize / 1024/1024/1024), total bytes: \(.gc.totalBytes / 1024/1024/1024), cputime: \(.cpuTime)"' -r to pretty-print it like:
heap size: 16.133785247802734, total bytes: 38.90941274166107, cputime: 199.245


[AIRFLOW-3174] Refine Docstring for SQL Operators & Hooks

JIRA
https://issues.apache.org/jira/browse/AIRFLOW-3174
Description

[x] Here are some details about my PR, including screenshots of any UI changes:

These SQL operators (based on DbApiHook) receive sql and parameters (optional) as arguments. But parameters were not documented. The contents I added is based on the docstring in https://github.com/apache/incubator-airflow/blob/master/airflow/hooks/dbapi_hook.py#L88

Fixed two minor style errors which would cause incorrect documentation rendering

Added docstring for autocommit parameter in relevant operators. The content is based on https://db.apache.org/derby/docs/10.9/devguide/cdevconcepts29416.html and https://www.postgresql.org/docs/9.1/static/ecpg-commands.html

Adjusted order or docstring items to make sure they are consistent with the order of arguments.

Hi @Fokko , I accidentally screwed my branch and could not push to my earlier PR https://github.com/apache/incubator-airflow/pull/4023 . So I had to open this new PR.....
The comments you raised earlier (adjust order for consistency, add docstring for autocommit) are addressed. Kindly have a check.
I also addressed two minor style errors in two Hook files.
Thanks.

BTW, given it's only docstring change, I had [ci skip] in my commit message to skip CI tests.
I had already run Flake8 check locally to make sure no Flake8 error.

Thanks @Fokko for merging :-)


allowUserOptions breaks option selection via keyboard

If the component has allowUserOptions set and the user has entered some input, matching options are no longer selectable via the arrow keys. Instead, the cursor only jumps to the beginning or end of the input, depending on which key was pressed.
This is easily reproducible for me (Firefox 107.0.1) on the demo page with the "Slot Components" example:

Focus the multiselect component
Hit up/down arrow; options are selectable
Type a character, for example "g"
The options narrow down
Hit up/down arrow; options are no longer selectable via the keyboard

Good catch! Thanks for reporting.
Fix and new release incoming.


test for crops single embedding

Also, I think we should use add the folder with images to DVC and commit .dvc file here.

closing in favor of #93


All Greedy Algorithms

Expected:
All greedy algorithms by using prograaming language java ,c ,c++
plz assign me this under Gssoc 2023

hey @aaryahjolia  i am intrested to contribute this project


TNO-239: User filter / sorting / styling

Putting up as draft while I polish up here

server side sorting on user table
server side filtering on user table
add ellipsis on columns
minor styling

@Fosol updated again... might be the one 🤔


feat: Add argo events addon module

What does this PR do?
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

Resolves Argo Events #197

More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[x] Yes, I ran pre-commit run -a with this PR

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

We are not accepting more generic helm charts that do not require any aws metadata, we added a variable helm_releases you can pass as many helm charts you want as a map take a look at the example here https://github.com/aws-ia/terraform-aws-eks-blueprints-addons/blob/main/tests/complete/main.tf#L189C3-L219


Add 'Draft Release 0.1 - 11/17/2021' into new branch

given the changes since November, and final pending PR, renaming issue

no longer relevant


Add example for MediaStream?

Hi,
Could you add an example using mediaStream, capturing Window or the desktop.

I haven't a plan to make it now.
And if I make it, I don't use the native library but use electron.


Set service.name for OpenTelemetry spans

Currently we are sending telemetry spans with an empty service name from both the KubeArchive sink and API, we need to add its name. Probably kubearchive.<component> is the right move for the name, so kubearchive.sink and kubearchive.api, and eventually kubearchive.operator.
Notes

The kubearchive.api string we are sending to the otelgin library its not setting the service name.

I agree on that naming convention. The otel configuration that is being used looks for an env var OTEL_SERVICE_NAME and sets the service name to the value of the env var. The easy way to add this would be to add after this line to the kubearchive.v1.otel.env template. kubearchive.<component> can be created with {{ $.Release.name }}.{{ .name }}


Fix VirtualizedList with initialScrollIndex not rendering all elements when data is updated

Summary
Fixes https://github.com/facebook/react-native/issues/33529 (note that I reproduced the bug on iOS too).
The bug comes from the fact that we were using this._scrollMetrics.offset to determine if the initial scroll was done. But sometimes it equals 0 even after the initial scroll is done, for example when the content does not fill the list. So I replaced it with this._hasDoneInitialScroll.
I believe that this._hasDoneInitialScroll was not used in the first place because it was introduced later (3 years ago vs 5 years ago for the original code).
The replacement correctly fixes the broken test case and the example given in the issue.
Then I had to update two test cases (rename the first and remove the second), that shows explicitly the broken behavior:
we have to simulate the initial scroll for the content to be adjusted, so when the content does not fill the view and the scroll cannot be executed, the content is not adjusted.
Changelog
[General] [Fix] - Fix VirtualizedList with initialScrollIndex not rendering all elements when data is updated
Test Plan

I added a broken test case based on the issue
I tested with the RNTesterApp using the code example given in the issue

Thanks for making this change. I think it makes sense to me.
One thing to mention here -- I tried to reproduce this with your example, but got error at VirtualizedList.js where the this._highestMeasuredFrameIndex value is still zero. I believe that only got updated when onCellLayout is triggered, which is not happening when I load the playground code.

@ryancat Sure, I'll check if that's also the case for me and I'll try to give you an update soon.

I rebased onto master to resolve conflicts.

(Rebased again after commit was reverted)

Is this still a problem or something? I have a horizontal FlatList, each item has the width of the screen (carousel style) and with active pagination. When I declare a value for initialScrollIndex all next items are only rendered when scrolling/paging is done


fix(client): allow wrapping items in namespace-list

Not sure if this is the right fix, but this attempts to fix https://github.com/weaveworks/scope/issues/3408

I'd have also liked to fix the issue with the footer floating on top of the namespace-list, but that seems more of a UX issue rather than UI

Closed with deference to https://github.com/weaveworks/scope/pull/3117


Live Share always crashes 15 minutes after starting a share.

Issue Type: Bug
Hello,
My Live Share extension consistently crashes about 15 minutes into launching VS code. I receive the error "Extension host terminated unexpectedly" and I must restart VS code in order to get the extension back.
Extension version: 1.0.1783
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:57:51.166Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz (4 x 2700)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
15, 10, 8

Memory (System)
8.00GB (0.16GB free)

Process Argv

Screen Reader
no

VM
0%

Hi thanks for reaching out. Sounds like it might be  causing by VS code. Did you tried to update your vs code ? if it  doesn't work and still you think it is caused by Live Share your logs may provide us with helpful information. You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue.

Closing due to age. Please feel free to reactivate with more details so we can repro the problem!


Capture and disseminate GCEP curation process reasons

At point of publishing, capture reasons for the curation process.
See mockup here: https://docs.google.com/presentation/d/1uD_V_78xKg4JZdz9_DoJeEpTNTlw7ECpUTwdNyWP_gs/edit?usp=sharing

relates to #318


Add RHEL 8.8

Problem
Red Hat Enterprise Linux 8.8 is missing causing this error: The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'redhat8.8'. Red Hat 8.8 can be used for db2 deployments.
Solution
Define RedHat 8.8 in repos and os-packages

@KimForss @hdamecharla please review this PR. We would like to test the DB2 deployments with RHEL 8.8.

approved

Could you merge it as well?


cert-manager webhook failure

Describe the bug
We seem to have a race condition with the cert-manager webhook. Writing it here so we know about it.
To Reproduce
Steps to reproduce the behavior:

Create a brand new vanilla Kubernetes cluster, or re-use an old one with ./scripts/clean-sc.sh.
Create a new configuration (e.g., ./bin/ck8s init, edit relevant fields).
Apply ./bin/ck8s apply sc.

Expected behavior
Apply should work successfully.
Actual behavior
FAILED RELEASES:
NAME
issuers
opendistro-es
in ./01-applications.yaml: 2 errors:
err 0: failed processing release issuers: command "/home/cklein/bin/helm" exited with non-zero status:

PATH:
  /home/cklein/bin/helm

ARGS:
  0: helm (4 bytes)
  1: upgrade (7 bytes)
  2: --install (9 bytes)
  3: --reset-values (14 bytes)
  4: issuers (7 bytes)
  5: charts/issuers (14 bytes)
  6: --version (9 bytes)
  7: 0.1.0 (5 bytes)
  8: --timeout (9 bytes)
  9: 600s (4 bytes)
  10: --namespace (11 bytes)
  11: cert-manager (12 bytes)
  12: --values (8 bytes)
  13: /tmp/helmfile395826681/cert-manager-issuers-values-55f8745cd5 (61 bytes)
  14: --history-max (13 bytes)
  15: 10 (2 bytes)

ERROR:
  exit status 1

EXIT STATUS
  1

STDERR:
  Error: Internal error occurred: failed calling webhook "webhook.cert-manager.io": Post "https://cert-manager-webhook.cert-manager.svc:443/mutate?timeout=10s": x509: certificate signed by unknown authority

COMBINED OUTPUT:
  Release "issuers" does not exist. Installing it now.
   Installing it now.
  Error: Internal error occurred: failed calling webhook "webhook.cert-manager.io": Post "https://cert-manager-webhook.cert-manager.svc:443/mutate?timeout=10s": x509: certificate signed by unknown authority
exit status 1

Version (add all relevant versions):

Compliant kubernetes apps version v0.13.0

Did it work when you ran apply again?

@cristiklein please validate if #446 solved this issue

Will redeploy the demo cluster with v0.16.0 is out and verify this.

@cristiklein can this be closed?


0.1.0a74

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

It looks like the 'tiled-base' output doesn't have any tests.
It looks like the 'tiled-array' output doesn't have any tests.
It looks like the 'tiled-sparse' output doesn't have any tests.
It looks like the 'tiled-dataframe' output doesn't have any tests.
It looks like the 'tiled-xarray' output doesn't have any tests.
It looks like the 'tiled-compression' output doesn't have any tests.
It looks like the 'tiled-minimal-base' output doesn't have any tests.

@conda-forge-admin, please rerender


Lookup targets with IP address or CIDR

What version of fleet are you using (fleet version --full)?
fleet - version 2.0.1
branch:       master
revision:     d01727dff26683ba29424fbdb7e58e403ab6210d
build date:   2018-11-26T19:21:48Z
build user:   zwass
go version:   go1.11.2
What operating system are you using?
CentOS Linux release 7.5.1804 (Core)
What did you expect to see?
When I select targets I expect to search them with IP address or CIDR, but I can lookup only with hostnames which is pretty uncomfortable. Also, IP address lookup is suggested on input placeholder.

IP Address input is possible:

As for CIDR, this is a great idea, but initially looks nontrivial to implement so I won't give any promises on its implementation.


feat: Made the packages public

In order to let third parties use the package of this library, we moved the pkg folder out of the locked internal folder

That's too open.
I would just share the Input, add some hooks to handle progress or json output, and having a public write that will call inner method.
I've got some idea about this. I need a bit of time to work on it.


All futures

Implementation of #12
Example usage:
let elg = MultiThreadedEventLoopGroup(numberOfThreads: 1)
let results = try elg.next().whenAllComplete(
    someFunctionThatReturnsIntFuture(),
    someFunctionThatReturnsIntFuture()
).wait()

// results = [Result<Int, Error>]
// results.count == 2

Other than the things I mentioned above, looks good to me :)

@tanner0101 Do you have any thoughts on this?


Installing GPU version for python

I have searched all the available issues about GPU-supported python installation of media pipe. Every issue is being referred to the same source which has got no clear solution.
How do I install GPU supported version of python API?

Hi @Zumbalamambo, Could you please elaborate your query with complete details and info about the Platform(OS).
Thank you!


Corrected laravel name

May be you @FiloSpaTeam  should try the https://github.com/phpvim/phpcd.vim . A new phpcomplete extended version based on the PHP's Reflection.


Fix social links in mobile view

Description
Social icons in mobile view were broken because component is + <script setup> requires an special handling in vite:
https://forum.vuejs.org/t/set-dynamic-component-in-script-setup-using-the-component-name/126584
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

Visual context

Please provide any relevant visual context for UI changes or additions. This could be static screenshots or a loom screencast.
Checklist:

[ ] I have performed a self-review of my own code
[ ] I have requested at least 1 review (If the PR is significant enough, use best judgement here)
[ ] I have commented my code where relevant, particularly in hard-to-understand areas
[ ] If package-lock.json has changes, it was intentional.
[ ] The base of this PR is master if hotfix, develop if not

@agualis Wait if you want to make this a hotfix, you'll have to start again, branch off master and cherry-pick the commit from here.


update cmake to properly allow includes to be inside of src

Currently, to include classes of DogTales you have to call to them like as follow:
#include <src/player.hpp>

Which is not only inconvenient but also pretty dirty for use in the project and breaks many common conventions with how a project is structured in C++.
I suggest we separate the includes so they either work like as follows:
// Either any of these style of includes
#include <player.hpp>
#include <dogtales/player.hpp> // This is my personal favorite
#include <game/player.hpp>

I'd also like to push a separation of includes and implementations where headers go into a include folder and implementation files go into a src folder, but this is of lesser importance than the above proposal.
Here is an example of how the project would look like in directories and how we would separate everything with cmake.
DogTales/
│
├── CMakeLists.txt
│
├── include/
│   └── dogtales/ (headers go here)
│       └── player.hpp
│
└── src/ (implementation goes here)
    └── player.cpp

The includes would look like as follows with the above project structure after properly being configured with cmake:
#include <dogtales/player.hpp>

The src/x.hpp part was deliberate, because just x.hpp could clash with a system / third party header. Agree that dog_tales/x.hpp is ideal. FYI that won't need any changes in CMake, just to rename src to dog_tales and fixup includes in code.

The src/x.hpp part was deliberate, because just x.hpp could clash with a system / third party header. Agree that dog_tales/x.hpp is ideal. FYI that won't need any changes in CMake, just to rename src to dog_tales and fixup includes in code.

Yeah I am aware if we just rename src then no changes are required in the cmake. :P
Also personally I'm not picky of having everything in a singular root file. Though I do still stand by that we should rename it away from src as its an extremely generic name. I'm fine with just naming it dog_tales though. ^^

Closed via #32


No substitution when a var is assigned to itself

Variable expansion of an externally declared variable does not appear to occur when assigned back to the same variable name e.g. given the declaration of GITHUB_TOKEN, in my .env file:
GITHUB_TOKEN="$GITHUB_TOKEN"

...then show envFromFile from within the sbt session reveals:
[info] 	Map(GITHUB_TOKEN -> $GITHUB_TOKEN)

i.e. the token is not substituted where GITHUB_TOKEN has been declared prior to invoking sbt.
I'm not certain, but I think the following line is causing a problem:
      VariableExpansion.expandAllVars(sys.env ++ environment, environment)

i.e. the fact that environment is appended to sys.env.

Thanks for raising, and apologies for my slow response. I don't spend much time on this these days, but if you care to submit a PR we'd be happy to bring it in.
P.S. interesting background, and hello fellow Aussie :)

Thanks @mefellows ! I’ll try and get a PR in!


Added CMake Build Support

Hi,
Currently I've added the ability to build and install the C++ component. I've some rudimentary work in place to build the python components as well, though I'll be adding the missing bits in a while.
This PR aims to address #292

Can you tell me how to run cmake?
It complains about missing DIRECTORY?
I think on Mac, I will recommend using homebrew to install dependencies
On windows, it will be useful.
Thanks!
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-05-30 14:19 GMT+02:00 Amit Prakash Ambasta<EMAIL_ADDRESS>
@ambasta commented on this pull request.
In ortools/algorithms/CMakeLists.txt
https://github.com/google/or-tools/pull/397#discussion_r119082661:

@@ -0,0 +1,10 @@
+ENABLE_LANGUAGE(CXX)
+# ENABLE_LANGUAGE(Python)
+SET(ROOT_PROJECT ${PROJECT_NAME})
+PROJECT(${PROJECT_NAME}_algorithms)

+FILE(GLOB ${PROJECT_NAME}_SOURCES "*.cc")
+LIST(REMOVE_ITEM ${PROJECT_NAME}_SOURCES "${CMAKE_CURRENT_SOURCE_DIR}/hungarian_test.cc")
Sure. Currently trying to allow for ExternalProject_Add for mac/windows.
Also trying to figure out a way to install the python module. I had
actually submitted the PR for review before merge, but I this was really
quick. I'll be updating the CMake build over the next few days to address
all the issues!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/pull/397#discussion_r119082661, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKj17bp5iFF4LLJGkkVoKiuT-GCrWB5uks5r_AlngaJpZM4NqJAS
.

mkdir build && cd build && cmake .. -DBUILD_CXX=ON && make && make install

CMake Error at CMakeLists.txt:77 (INSTALL):
install TARGETS given no LIBRARY DESTINATION for shared library target
"ortools".
??
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-05-30 14:37 GMT+02:00 Amit Prakash Ambasta<EMAIL_ADDRESS>
mkdir build && cd build && cmake .. -DBUILD_CXX=ON && make && make install
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/pull/397#issuecomment-304864893, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKj17QQMyx3njxFt_6wbi-CG_56Zg_boks5r_A2agaJpZM4NqJAS
.

Right, I'm not sure where to install files on non-linux architectures. I'll submit a patch by EoD to set appropriate directories for Windows/Mac

I made some inroad
/Applications/CMake.app/Contents/bin/cmake -DBUILD_CXX=ON
-DCMAKE_INSTALL_LIBDIR=/Users/lperron/Work/or-tools-dist/lib
-DCMAKE_INSTALL_BINDIR=/Users/lperron/Work/or-tools-dist/bin
-DCMAKE_INSTALL_INCLUDEDIR=/Users/lperron/Work/or-tools-dist/include ..
finished OK on mac.
But compilation fails:
[  0%] Building CXX object
CMakeFiles/ortoolsProto.dir/ortools/bop/bop_parameters.pb.cc.o
clang: *error: *no input files
/bin/sh: -I/usr/local/Cellar/cgl/0.59.6/include/cgl/coin: No such file or
directory
/bin/sh: -I/usr/local/Cellar/clp/1.16.7/include/clp/coin: No such file or
directory
/bin/sh: -I/usr/local/Cellar/osi/0.107.5/include/osi/coin: No such file or
directory
/bin/sh: -I/usr/local/Cellar/coinutils/2.10.10/include/coinutils/coin: No
such file or directory
/bin/sh: -I/usr/local/Cellar/coinutils/2.10.10/include/coinutils/coin: No
such file or directory
make[2]: ***
[CMakeFiles/ortoolsProto.dir/ortools/bop/bop_parameters.pb.cc.o] Error 127
make[1]: *** [CMakeFiles/ortoolsProto.dir/all] Error 2
make: *** [all] Error 2
Please note that all missing include directories do exist.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2017-05-30 14:47 GMT+02:00 Amit Prakash Ambasta<EMAIL_ADDRESS>
Right, I'm not sure where to install files on non-linux architectures.
I'll submit a patch by EoD to set appropriate directories for Windows/Mac
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/pull/397#issuecomment-304867273, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKj17Wb2IlrOpilk3xKtTfFF878l0-D0ks5r_A_TgaJpZM4NqJAS
.

Hi @lperron
You're correct about the missing directories existing. However, currently, the cmake project expects the user to have specified the dependencies. I'm enabling ExternalProject_Add in a new set of patches. For now I'm also removing the Python bits so I can work on them separately and not muddle the patches!