Introduce StreamProvider

Hi! I'm GitBucket developer. GitBucket is open source Git platform powered by Scala. It looks like as GitHub. It has online git repository browser and online file editor.
I was interested in making GitBucket correspond to Editorconfig. But, current implementation is based on local filesystem's file. So I introduced StreamProvider by this PR. I also wrote JGitProvider in my branch of GitBucket. see here
If this PR is merged and it released to maven, I can make EditorConfig support PR to GitBucket.

Hi @kounoike , in case you look around and decide this project is not lively enough, you may want to look at the alternative Java editorconfig parser called https://github.com/ec4j/ec4 that I wrote together with @angelozerr . In ec4j, we already have an abstraction of a Resource that would allow you to implement the Git backed kind of resource tree you need. We already have ClassPath resource, java.nio.Path resource and a plain string resource. I think especially the ClassPathResource very much resembles what you'd do for Git: https://github.com/ec4j/ec4j/blob/master/core/src/main/java/org/ec4j/core/Resource.java#L77 Feel free to file an issue at https://github.com/ec4j/ec4j/issues if our API is not flexible enough for you.

@ppalaga Thanks! I changed dependency from editorconfig-core to ec4j. I wrote JGitResource and JGitResourcePath in here. It doesn't require ec4j-core changes.

Nice to hear that, @kounoike ! Could you please add GitBucket to "who is using ec4j" section of our README, once your change is merged? https://github.com/ec4j/ec4j#who-is-using-ec4j

@ppalaga Hi! gitbucket/gitbucket#1966 is merged! Thanks for your suggestion and proposal. @takezoe what do you think of this proposal?

No problem at all about adding GitBucket to the list of ec4j users. Thanks for the useful library. :-)


Tiles can be stored as a wide String

save and load would become fasert, and file smaller

What do you mean exactly with "wide string"? Do you have any example?

I'm late here.
I asked Simple framework to serialize the tiles in XML format. The result is a huge file containing abolutly unnecessary data. This is awful and must be refactored.
First, it's useless to store the coordinates of tile as we save all tiles, ordered. Coordinates are implicits.
Second, there are more caracters for markup than for used data.
A solution would be to create our own serializer/deserializer for the Map class (with bytes or strings), as I did for the Atlas Class for the same reasons.
I'm sure you know a super-clean way to do that :)

I have problems to understand how the current serialisation is working. I understand we are using SimpleXML, the BuilderLibrary holding the Builders, but I can't understand, why is it so complex structured. It sounds a little bit naive, but the only thing we need is the create a instance for a object defined in XML(or other format) and while the runtime (if a new unit is spawned in the battlefield).
Did I forgot any requirements?
Another question: should the fileformat being humanreadable(XML/JSON)?
or binary? see
http://wiki.jmonkeyengine.org/doku.php/jme3:advanced:save_and_load
http://wiki.jmonkeyengine.org/doku.php/jme3:contributions:cubes:tools:serialization

This issue only concerns the map serialisation. There is no Builder involved here.
For the records, Builders are working with homemade XML parsing (in model.builders.definitions package). The user write the XML manually and the engine parse this XML into builders, which instanciate objetcs of this type.
The battlefield is serialized and deserialized by SimpleXML and hasn't to be humanreadable. For a battlefield, there is two files : one XML for the map and all components, and one binary for the ground texture atlas.
The XML contains :

the map tiles,
the ramps,
the trinkets,
the engagement (units and buildings).

The problem are the tiles. They occupy 95% of the XML size. Each tile record contains :

tile's coord
level
relief
cliffStyle as a String

The coord is useless (implicit by the order) ! And the cliffShape may be stored as an index, instead of a String.
Optimise this should reduce the weight of the XML (~3 or 4 now) and the loading time (very looong :)).

Thanks. now there is better understanding.
Is it correct, that the user are writing some blueprints/templates... to describe the units/entities? If yes, this must be as easy as possible and naturally humanreadable.
in the tiles aspect I am 100% with you.
if we split the both things. In the map/battlefield there must only a reference for the unit. For example "a command like that "please insert an unit with id "humanId" at position X,Y (and some parameters like faction) (see https://github.com/methusalah/OpenRTS/issues/29)

Yes, data directory hold XML written bu user. I tried to make it as easy as possible to write. I have to implement #29 to make it more efficient and less verbose.
battlefield may be entirely binary as the user won't have to read it. And yes, units are called by there definition ID only. hee is an example of the SimpleXML output :

humantank
1

10.081957817077637
4.811057806015015
-0.6632389533061867

0.48741929772002046

There are a lot of charaters for so few usefull data, but this is even worst with the tiles. Don't hesitate to open a .bf file in assets.maps to see more.

OK.and in last please look athttps://github.com/methusalah/OpenRTS/issues/19 and answer it. Than I have the bundle complete and I would fix iz ;-)

Serialization/deserialization is much fatser with Jackson. I close for now.

map02 is not migrate to json. I will do it today. Till now I will reopen it.
Another question is, Should we migrate the entities (units,weapons and so on) to JSON too?

Entities could be in json as well, but it's much less urgent scince there is no performance issue.

The entityeditor (I am working on it now) will generate the new XMLs. The new one will be identical with the current one. Later we can change the generator to generate JSON. This will be easy.

but your are right, we can close this issue now. I report the status at https://github.com/methusalah/OpenRTS/issues/19


add Kurtosis as a project that uses Starlark

Kurtosis is developer tool for Platform Engineers and DevOps teams to define and spin up complex distributed systems for dev, test, and prod. Kurtosis uses Starlark as the DSL for describing how the environment should be stitched together.

poking in here, i've opened a new PR (#269 ) that accomplishes this goal with the proper CLA <-> github username mapping. this can be closed @leeederek


DSL enhancement proposal

As discussed on Piter Py #5 in person. Here is the proposal discussed with @proofit404 .
Things which we do here:

Wipe any realization from a story_suite decorated class
Use a story suite as a context container. Everything mentioned as steps arguments or results is a context member
Allow defining some context members explicitly as class attributes, see PurchaseProduct.payment_allowed
Define all steps call signatures and a context member to store its result in
Allow validating context members consistency using annotations
Allow validating realization methods signatures in runtime using DI, see dry-python/dependencies
Allow validating realization methods signatures in import time using a story suite class as a metaclass of the realization class similar to ABCMeta

import decimal
import stories

@stories.story_suite
class PurchaseProduct:
    payment_allowed: bool

    @stories.story
    def purchase(self, user: User, product: Product, shipment_details: ShipmentDetails):
        self.create_order()
        self.calculate_price()
        self.request_payment()
        self.process_payment() if self.payment_allowed else self.reject_payment()
        self.notify_user()

    @stories.step
    def create_order(
        self, user: User, product: Product
    ) -> stories.Success["order", Order]:
        ...

    @stories.step
    def calculate_price(
        self, order: Order
    ) -> stories.Success["price", decimal.Decimal]:
        ...

    @stories.step
    def request_payment(
        self, user: User, order: Order
    ) -> stories.Success["payment_allowed", bool]:
        ...

    @stories.step
    def process_payment(
        self, user: User, price: decimal.Decimal
    ) -> stories.Success["payment_result", PaymentResult]:
        ...

    @stories.step
    def reject_payment(
        self, user: User
    ) -> stories.Success["payment_result", PaymentResult]:
        ...

In this form, something like PurchaseProduct.__ctx__.schema returning the collected schema of the suite context could make sense.

Yes, schema is a useful thing. But I wonder for what in this case? Any usage examples?

I see it could be useful for tests or debugging purposes. I'm doubt it will be needed in runtime in that fashion.

I'm trying to figure out desired api. And it is hard without actual usecases.


Id Field with (guid) won't be passed to the database

Can anyone tell me what I'm doing wrong, I've got a error telling me that a id filed is null saving, having searched previous records I've overrides the BeforeSave and saw It has value, can«t seem to figure this one out :(
@volkanceylan  Any tip ?
Sample Override:

Error I've Got:

MasterDetailBehavior won't let you use arbitrary ID's in client records, as it might be a injection attempt, so on insert it will clear those IDs. You should write a behavior for detail record and set guid there.
Anyway i'd add a SQL table default to that field.

@volkanceylan thanks that makes sence, will do the changes accordingly.


UAT: landing page after login

Here's what I see after login in. In most other pages, the "Good Morning, " appears in a larger bold font. We may want to retain consistency.

This issue is fixed and working as expected.
Ref :-


Interfacing OCS2 with CoppeliaSim and other physical engine based simulations

While the provided dummy tests are very useful for studying mpc policy behaviors and weight tuning, they aren't rigorous validations since the generated states are based on the same dynamic models. Currently, we are trying to interface OCS2 with CoppeliaSim which so happens to simulate our robot behaviors well. The naive thought was to simply replace dummy_test with CoppeliaSim's ROS interface (this node will publish /mpc_observation and subscribe to /mpc_policy). We soon realized that by excluding dummy_test node, we also disable MRT ROS interface, which supposes to run along side with MPC (handling reset and timing control).  Therefore, we wonder:

Is MRT interface absolutely necessary or it can be replaced if a third party robot simulator handles the time control? (MPC computes traj based on simulation time)
How do we enable state-based and feedback policy setting for the MPC solver and MRT? What are the pros and cons compared to time-based?
Is it possible to provide a DummyXXbotNode.cpp or a MRT_ROS_Interface.cpp temple that allows people to take state observations from other ROS node (i.e., CoppeliaSim) while still having MRT to regulate the timing?

(Sorry, just started using OCS2, I hope the questions make sense. )

Sorry for the delay in getting to your question. You are right that at the moment all examples use this DummyLoopNode structure and we have no examples yet that show how to best integrate with other simulations (or the robot).
Integrating your own physics
There are a few ways to go, depending on your needs.

Stay with the dummy loop setup, but provide a different dynamics. In this scenario, you would create a separate system dynamics object that you want to use as a simulator. (These dynamics should derive from ocs2::ControlledSystemBase) From this system dynamics, you can create a "TimeTriggeredRollout" and pass that rollout to the MRT. You would replace the default rollout with your custom rollout in this line:
https://github.com/leggedrobotics/ocs2/blob/b037d819c9a02a674de9badb628b32646ce11d6a/ocs2_robotic_examples/ocs2_cartpole_ros/src/DummyCartpoleNode.cpp#L63

Integrate a "MRT_ROS_Interface" into your own simulation loop. In this case you would follow the example until here:
https://github.com/leggedrobotics/ocs2/blob/b037d819c9a02a674de9badb628b32646ce11d6a/ocs2_robotic_examples/ocs2_cartpole_ros/src/DummyCartpoleNode.cpp#L64
You can embedd this MRT object into your own simulation loop, and it will handle the ROS communication with the MPC loop for you. You would then be calling setCurrentObservation(), updatePolicy(), etc. Your own simulation loop will probably look similar to the implementation of the Dummy loop, so I guess this is what you have current done: Adapting the dummy loop to your situation.

A third way is to use the ROS independent interface:
https://github.com/leggedrobotics/ocs2/blob/b037d819c9a02a674de9badb628b32646ce11d6a/ocs2_mpc/include/ocs2_mpc/MPC_MRT_Interface.h#L49
This interface has both the MPC and the MRT in one, giving you full control over both components. This is what we use when deploying the algorithms on hardware. I won't go into more detail for now. Just so you know it exists.

How to use the feedback policy
This can be done from the config file:
https://github.com/leggedrobotics/ocs2/blob/b037d819c9a02a674de9badb628b32646ce11d6a/ocs2_robotic_examples/ocs2_cartpole/config/mpc/task.info#L37
Set that one the "true". From there the MPC node should switch to publishing also the feedback matrices, and the MRT node will use them during the call to evaluatePolicy()
The MPC fails
A few things come to mind here

The manually implemented dynamics in OCS2 is wrong. [I will look into this]
Is your simulation using the same conventions and units as the OCS2 model? It might be worth it to look at the model errors between the MPC model and your simulation in isolation to rule such mistakes.
Is the simulation loop synchronized with the MPC loop? In other words: is the simulation halted while a new MPC optimization is done? If not, a large delay in the pipeline can cause problems.

@rubengrandia Thank you for your in-depth response!
Updates:
We have recently resolved the MPC failure issue. Your guesses were right, we identify a bug in our pseudo MRT_ROS_Interface (we named it CoppeliaCartpoleNode.cpp, the equivalent to DummyCartpoleNode.cpp) that caused synchronization issue between CoppeliaSim and OCS2. Now both the physic engine simulated cart-pole and the cart-pole with hardcoded dynamic work with OCS2 (with the default physical parameters, i.e., cart mass: 2.0 kg, pole mass: 0.2 kg, pole length: 1.0 m)! (Here is a brief demo for the physic engine simulated cart-pole)
https://www.youtube.com/watch?v=kOZuncIXxXw
The pure feed forward mode didn't work so well with this physic engine simulated cart-pole (probably due to model discrepancy). But switching the control mode to linear + feed-forward controller (controllerType: 2) resolved the issue.
Current issues:
Now since the interface seems to work, we are moving on to adapt our own robot dynamic / physic with OCS2 MPC.

First, we leave the dynamic model (the EOM) of the cart pole the same, but changing the physical parameters to cart mass: 10.5 kg, pole mass: 40 kg, pole length: 0.3405 m in both the task.info and the CoppeliaSim.
We tuned the control weight matrix in task info to 0.01. The dummy test works fine with this setting.
However, OCS2 MPC failed to perform the swing up test / small angle balancing in CoppeliaSim.

We will provide more detail and information on this topic soon.
Follow-up questions

It seems like learning MPC_MRT_Interface.h is the key to the correct implementation and deployment of OCS2 MPC. To better understand it, we figured the double_integrator is a good place to start. Is there any other source / example that worth looking into?
Can you share some info on what kind of hardware you used to host the OCS2 MPC for your robot? Is it some kinds of real-time computer? Or just regular ROS but assuming it is updating fast enough to be considered real-time?
The error "Integration terminated since the maximum number of function calls is reached. State at termination time " still occasionally occur when the control task fails. We suspect that it is due to some robot states making the non-linear dynamic model unsolvable to the MPC. What is your insight?

Thanks again! We will provide further update soon!

Good to see it working for the default case!
I checked the dynamics implementation. That seems all fine. How are you defining the inertia of the pole in your simulation?
In OCS2 it is assumed that the center of mass is located at the middle of the pole (L/2), and that the Inertia around the center of mass of the pole is that of a pole with uniform mass distribution (I = m / 12 * L^2).
You might want to increase the terminal cost "Q_final" as well. Given that you say it already fails for small angles.

For the MPC_MRT_Interface you can start here:
https://github.com/leggedrobotics/ocs2/blob/b037d819c9a02a674de9badb628b32646ce11d6a/ocs2_robotic_examples/ocs2_double_integrator/test/DoubleIntegratorNoRosIntegrationTest.cpp#L86
After that, there is the test "asynchronousTracking" in the same file.

We mostly use regular Intel (laptop series) CPUs with Ubuntu + RT kernel patch, so soft real-time. With a multithreaded setup as in the "asynchronousTracking" example, you can have one thread executing the latest MPC policy and one creating running a new optimization. This way the optimization is not blocking the update of the commands.

The maximum integration reached error is indeed usually a sign that the current problem is 'hard'. The continuous time DDP algorithm takes an adaptive stepsize both in the forward and backward pass. If the problem becomes very stiff, this stepsize will go to zero, and we abort after reaching the maximum number of steps. We have a few other algorithms in the toolbox that you could try, but I would stick with this one for now.

@rubengrandia Thank you! This is super helpful!
Updates:
We fixed the issue with the heavier physical parameters.

CoppliaSim uses a dt of 50 ms by default. So, running the simulation in real-time results in a update frequency of ~20 Hz. Therefore, we set the MPC and MRT desired frequency to 20 Hz to match with the CoppeliaSim's update rate. (Unfortunately, we cannot easily change the dt setting as the simulation needs to hook up with some VR applications and HMIs)
It turns out, the simulation will miss a few steps of observation / policy exchanges at the start of the simulation. This issue remained undetected as the previous set of physical parameters result in a system dynamic that has a larger time constant. As the mass of the pole increases and the length of the pole decreases, the time constant becomes more critical and less tolerance to control delay.
Simply setting the MPC and MRT desired frequency to a higher number (100 Hz) this problem disappears.

Here is an updated demo:
https://www.youtube.com/watch?v=Xn6ilTbPa9Y


[BUG] Set-PnPTenantSite -ConditionalAccessPolicy Does Not Work

Reporting an Issue or Missing Feature
I am reporting a bug
Expected behavior
Set-PnPTenantSite -ConditionalAccessPolicy BlockAccess should change ConditionalAccessPolicy property of the site
Actual behavior
Set-PnPTenantSite -ConditionalAccessPolicy BlockAccess runs just fine, produces no error, however ConditionalAccessPolicy of the remains unchanged

Reference: https://github.com/pnp/powershell/discussions/220
Steps to reproduce behavior
Connect-PnPOnline
$PnPConnection = Get-PnPConnection
Set-PnPTenantSite -ConditionalAccessPolicy BlockAccess -Connection $pnpconnection -Identity $PnPConnection.url
Get-PnPTenantSite $PnPConnection.url | fl *conditional*

ConditionalAccessPolicy : AllowFullAccess

What is the version of the Cmdlet module you are running?
1.2.14-nightly
Get-Command Connect-PnPOnline

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Cmdlet          Connect-PnPOnline                                  1.2.14     PnP.PowerShell

Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

#220

We simply consume the APIs provided by us. The Set-SPOSite cmdlet for instance is doing exactly the same internal calls as Set-PnPTenantSite. Have you tried with Set-SPOSite? If that fails then it seems to be server side. From the documentation about conditional access policies I see that you require to have an Azure AD Premium P1 license. Do you have that in place?
https://docs.microsoft.com/en-us/sharepoint/control-access-from-unmanaged-devices (first note)
and
https://docs.microsoft.com/en-us/azure/active-directory/conditional-access/overview#license-requirements

There seems Set-PnPTenantSite is ignoring ConditionalAccessPolicy if we just set the policy. We cannot set to any policy as it is being ignored in the below code.
https://github.com/pnp/powershell/blob/dev/src/Commands/Admin/SetTenantSite.cs
It is checking for below parameters as well and then checking for the ConditionalAccessPolicy but again if the value is AllowLimitedAccess, it is being ignored.
BlockDownloadOfNonViewableFiles
AllowDownloadingNonWebViewableFiles
LimitedAccessFileType
AllowEditing
I see only BlockAccess works along with one of the above parameters.
Thank you!

@YuriySamorodov  - closing this. This issue has been fixed @chinthamahesh ( Thank you ! ) and is available in the latest nightly and will be there in the next major release . Please reopen if it happens again,


anagram: Regenerate Tests

This is my first PR to Exercism! 😄 I followed the instructions in the README and I hope I did it right.

Thank you @Thrillberg Thrillberg, looks good.


ExtractorError: YouTube said: Unable to extract video data

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.07.28
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
youtube-dl -v P5NB1H5pCHk
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'P5NB1H5pCHk']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.07.28
[debug] Python version 2.7.16 (CPython) - Linux-4.19.0-9-amd64-x86_64-with-debian-10.4
[debug] exe versions: ffmpeg 4.1.6-1, ffprobe 4.1.6-1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] P5NB1H5pCHk: Downloading webpage
ERROR: P5NB1H5pCHk: YouTube said: Unable to extract video data
Traceback (most recent call last):
File "/home/theo/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 797, in extract_info
ie_result = ie.extract(url)
File "/home/theo/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/home/theo/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1893, in _real_extract
'YouTube said: %s' % unavailable_message, expected=True, video_id=video_id)
ExtractorError: P5NB1H5pCHk: YouTube said: Unable to extract video data
youtube-dl -v 4wZPsFWfi3k
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'4wZPsFWfi3k']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.07.28
[debug] Python version 2.7.16 (CPython) - Linux-4.19.0-9-amd64-x86_64-with-debian-10.4
[debug] exe versions: ffmpeg 4.1.6-1, ffprobe 4.1.6-1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 4wZPsFWfi3k: Downloading webpage
ERROR: 4wZPsFWfi3k: YouTube said: Unable to extract video data
Traceback (most recent call last):
File "/home/theo/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 797, in extract_info
ie_result = ie.extract(url)
File "/home/theo/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/home/theo/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1893, in _real_extract
'YouTube said: %s' % unavailable_message, expected=True, video_id=video_id)
ExtractorError: 4wZPsFWfi3k: YouTube said: Unable to extract video data
Description
It seems that ytdl can't download restricted videos anymore (the ones behind a "sign in to confirm your age" wall).

I'm seeing the same issue with latest version.

I can confirm that this is also happening with with HW4IM87DAdg

Seems like a mass issue, as issues #26148 #26123 #26103  is having very similar issues.

Both videos above worked for me - only after I fed it my YouTube cookies.
I came here after noticing a similar problem with _Uelrsz_VfY (even though video is not age restricted and plays in private mode just fine). It seems like YT is requiring a session cookie to dl certain videos (maybe to do with country restrictions as this video got song matched).
As more videos have this issue I think it would be a nice enhancement for YTDL to give the user some hand-holding as far as getting cookies.txt set up and automatically using it when needed instead of adding --cookies c:\path\to\youtube-dl\cookies.txt every time.

Yeah, like a youtube-dl --first or youtube-dl --cookie-update that sort of walks you through it would be nice.

Hey there!
Same here PrJ5Sx1Dtc8 . So is the (missing) cookie actually the issue here?
Can someone confirm?

PR #26100 fixes the age-gate check, but it's not merged into upstream yet. You can apply the patch locally.

I ask because I tried two approaches to provide cookies for a logged-in user and both didn't work (providing text file and --add-header option).
I assume I got something wrong in the format then...

Happens with yJj9q-67ivA too.

Another example with this error: https://www.youtube.com/watch?v=oL276M79mYE .

I am having the same issue as #26299.
here is what my software gave me as the issue
ERROR: Unable to download webpage: HTTP Error 429: Too Many Requests (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
I am using a current updated version of Windows 10.
I  am running youtube-dl version 2020.07.28
I've checked that all provided URLs are alive and playable in a browser
I've checked that all URLs and arguments with special characters are properly quoted or escaped
None of the videos are age restricted

Edit: Got it working with the browser plugin and the --cookies option.

This only works sporadically for me, very often I still get the "Unable to extract video data" error. I'm not sure what triggers the error.

Thank you will try that

Thank you will try that

Well like I said it only works for me occasionally. Hopefully a more fundamental solution can be found for the "unable to extract video data" error.

I don't know if it's helpful, but if you're not connected to an account on youtube, you can still see a restricted video by changing in the url "watch?v=" with "embed". Unfortunately doing this with youtube-dl doesn't work.

Is there an easy way for those without programming experience to apply the patch locally? And how do I create/find the cookies.txt file?

If you're running it in Python, there's no programming necessary, just change this line in this file: https://github.com/ytdl-org/youtube-dl/pull/26100/files
If you're running it from an .exe file (as recommended on Windows), I'm not sure what to recommend, I haven't looked into the compilation process myself

I'm running it from the .exe in Windows 10. I have Python 3.7.4 but wouldn't know how to use it for this without step-by-step instructions. If someone wishes to walk me through that, my email's in my profile. If not, thank you for your time and I'll wait for it to be merged.

Is there an easy way for those without programming experience to apply the patch locally? And how do I create/find the cookies.txt file?

A cookies.txt file can be created with the "cookies.txt" extension.
Do this while being logged in to YouTube.
For Firefox:
https://addons.mozilla.org/en-US/firefox/addon/cookies-txt/
For Chrome:
https://chrome.google.com/webstore/detail/cookiestxt/njabckikapfpffapmjgojcnbfjonfjfg
Then run youtube-dl like this:
youtube-dl.exe --cookies C:\Users\tezJR\Downloads\cookies-txt https://www.youtube.com/watch?v=$VIDEO-ID
(adjust that command to where your cookies.txt file is actually located and replace $VIDEO-ID with  the ID of your restricted video).

That worked! Thank you very much, nicolaasjan!

That worked! Thank you very much, nicolaasjan!

No problem.
I have sent you a mail with instructions to compile youtube-dl.exe yourself.
It's not that complicated when you have Python installed.

What's happening with youtube-dl team? No commits in a long time, and no fix for this obvious bug that has been reported many times?

I ask because I tried two approaches to provide cookies for a logged-in user and both didn't work (providing text file and --add-header option).
I assume I got something wrong in the format then...
Edit: Got it working with the browser plugin and the --cookies option. And now it downloads the video with an age-check.

I ask because I tried two approaches to provide cookies for a logged-in user and both didn't work (providing text file and --add-header option).
I assume I got something wrong in the format then...
Edit: Got it working with the browser plugin and the --cookies option. And now it downloads the video with an age-check.

Hey @Xan-Kun , i did the same thing but every time I try to fetch URLs, the cookies.txt file gets override!! Is it expected behaviour? Also, I am running those commands in AWS hosted ubuntu machine. Please help.

Is there an easy way for those without programming experience to apply the patch locally? And how do I create/find the cookies.txt file?

A cookies.txt file can be created with the "cookies.txt" extension.
Do this while being logged in to YouTube.
For Firefox:
https://addons.mozilla.org/en-US/firefox/addon/cookies-txt/
For Chrome:
https://chrome.google.com/webstore/detail/cookiestxt/njabckikapfpffapmjgojcnbfjonfjfg
Then run youtube-dl like this:
youtube-dl.exe --cookies C:\Users\tezJR\Downloads\cookies-txt https://www.youtube.com/watch?v=$VIDEO-ID
(adjust that command to where your cookies.txt file is actually located and replace $VIDEO-ID with the ID of your restricted video).

Hey @nicolaasjan , i did the same thing but every time I try to fetch URLs, the cookies.txt file gets override!! Is it expected behaviour? Also, I am running those commands in AWS hosted ubuntu machine. Please help.

Is there an easy way for those without programming experience to apply the patch locally? And how do I create/find the cookies.txt file?

A cookies.txt file can be created with the "cookies.txt" extension.
Do this while being logged in to YouTube.
For Firefox:
https://addons.mozilla.org/en-US/firefox/addon/cookies-txt/
For Chrome:
https://chrome.google.com/webstore/detail/cookiestxt/njabckikapfpffapmjgojcnbfjonfjfg
Then run youtube-dl like this:
youtube-dl.exe --cookies C:\Users\tezJR\Downloads\cookies-txt https://www.youtube.com/watch?v=$VIDEO-ID
(adjust that command to where your cookies.txt file is actually located and replace $VIDEO-ID with the ID of your restricted video).

Hey @nicolaasjan , i did the same thing but every time I try to fetch URLs, the cookies.txt file gets override!! Is it expected behaviour? Also, I am running those commands in AWS hosted ubuntu machine. Please help.

I read somewhere, that youtube-dl changes the expiration date of the cookies in it, so that it keeps working as intended.
In the header of the file it says:
# This file is generated by youtube-dl.  Do not edit.
The expiration dates are in Unix Timestamp format, so<PHONE_NUMBER> means 10/27/2020 @ 8:51am (UTC)
You can convert these values online here:
https://www.unixtimestamp.com/index.php

@nicolaasjan said:
I read somewhere, that youtube-dl changes the expiration date of the cookies in it, so that it keeps working as intended.

I tested and confirmed it's being very heavily rewritten.

This worked for me (via reddit):
add -f http-1080-0 to the arguments, like youtube-dl -f http-1080-0 <url>
(I didn't even have to use cookies!)

Hey @Xan-Kun , i did the same thing but every time I try to fetch URLs, the cookies.txt file gets override!! Is it expected behaviour? Also, I am running those commands in AWS hosted ubuntu machine. Please help.

@upex Hi there. Yes it does! I just checked and didn't realise it before, but when I start the download, the cookies file gets overwritten.

Perhaps try to specify quality? That worked for me but I'm not
entirely sure if I ever actually tested it with no args. See if that
works for you.
On Mon, Sep 14, 2020 at 7:29 AM Xan-Kun<EMAIL_ADDRESS>wrote:

Hey @Xan-Kun https://github.com/Xan-Kun , i did the same thing but
every time I try to fetch URLs, the cookies.txt file gets override!! Is it
expected behaviour? Also, I am running those commands in AWS hosted ubuntu
machine. Please help.
@upex https://github.com/upex Hi there. Yes it does! I just checked and
didn't realise it before, but when I start the download, the cookies file
gets overwritten.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/26152#issuecomment-691993021,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF5DB4HIQ3KRYJKCXNU5XKLSFX5BZANCNFSM4PLAN72A
.

To me: removing YouTube-dl downloaded using sudo apt install youtube-dl and installing it using pip3 install youtube-dl solved the issue!

To me: removing YouTube-dl downloaded using sudo apt install youtube-dl and installing it using pip3 install youtube-dl solved the issue!
FOR LINUX USERS ONLY!!!

This works on MacOS for me.
"""
$ brew uninstall youtube-dl
Uninstalling /usr/local/Cellar/youtube-dl/2<IP_ADDRESS>... (2,203 files, 16.4MB)
$ pip3 install youtube-dl
Collecting youtube-dl
Downloading youtube_dl-2020.12.12-py2.py3-none-any.whl (1.8 MB)
|████████████████████████████████| 1.8 MB 1.1 MB/s
Installing collected packages: youtube-dl
Successfully installed youtube-dl-2020.12.12
"""

Try installing it with snap instead if apt or apt-get
sudo snap install youtube-dl

Try installing it with snap instead if apt or apt-get
sudo snap install youtube-dl

To me: removing YouTube-dl downloaded using sudo apt install youtube-dl and installing it using pip3 install youtube-dl solved the issue!
FOR LINUX USERS ONLY!!!

I get following error:

-bash: /usr/bin/youtube-dl: No such file or directory

Can anyone help me to resolve it?

To me: removing YouTube-dl downloaded using sudo apt install youtube-dl and installing it using pip3 install youtube-dl solved the issue!
FOR LINUX USERS ONLY!!!

I get following error:

-bash: /usr/bin/youtube-dl: No such file or directory

Can anyone help me to resolve it?

To me: removing YouTube-dl downloaded using sudo apt install youtube-dl and installing it using pip3 install youtube-dl solved the issue!
FOR LINUX USERS ONLY!!!

I get following error:

-bash: /usr/bin/youtube-dl: No such file or directory

Can anyone help me to resolve it?

Hi @SatyamGaba,
Did you try running python3 -m youtube_dl (i.e. running reinstalled python's modul youtube-dl)?
Note also: youtube_dl and not youtube-dl in the above command.
Explained:
I suspect that your specific problem is that when you uninstalled youtube-dl using apt, it also deleted the binary /usr/bin/youtube-dl and therefore comand $ youtube-dl in bash will return your-mentioned error. When you 'reinstalled' it with pip, it stayed inside python's (private?) packages without creating yotube-dl executable. But that's no problem, just run it via python as said.


PCB Design

Hi,
I'm creating a PCB for this that I plan to make available, but I have a question.
When you don't have a variable resistor, what values do you use for R7 and R8? The same as R6? (Based on the radio?)
Thanks,
Andrew (JJ1OKA)

Hi Vaelen.
You have one error in your schematic. R9 is in the wrong place. It should be between base of Q1 and ground. The emitter of Q1 must go directly to ground, zero ohms. Q1 is the PTT transistor and must pull the PTT line directly to ground. The purpose of R8 is to limit the current into Q1 to a safe value for the Pico CPU pin. The purpose of R9 is to make sure Q1 turns off when the Pico CPU pin goes low. The purpose of R6 is to make the audio out pin compatible with handie-talkie keying circuits. The value changes with radio brand. If connecting the circuit to a mobile radio that needs a separate PTT line, then R6 should be removed. It is really better if R4+R5 is a pot instead of fixed values. It is far easier to turn a pot than it is to change resistors to set the audio level going into the radio transmitter. You can always replace R5 with a pot, probably anything in the 1K to 10K range, using one outside leg plus the center leg to replace R5. But I think I'd recommend the circuit that K4APR used instead of R3/C3.
Doug.

Nope, I was thinking of some other schematic. K4APR didn't include a circuit and probably used the existing design. I was concerned that R3/C3 makes a very "stiff" pull-down on the Pico CPU PWM output and I'm not sure what the output level and signal harmonics will look like at the audio OUT pin. I was thinking of a different RC filter configuration, but if this works, use it.....
Doug.

I'm pretty sure you were correct that I made a mistake on the schematic. I never could get it working quite right and I ended up giving up on it and going with a different solution.
Here is what I had: https://oshwlab.com/andrew_3060/picotnc-tnc-board
If someone wants to collaborate on this using EasyEDA, please let me know and I will add you as a contributor on the project.


KServe does not catch Caikit runtime status correctly when subsprocess (tgis) have issues

When I create a ServingRuntime+InferenceService with some incorrect parameters, Caikit cannot load the model.
{"channel": "MODEL-LOADER", "exception": null, "level": "error", "log_code": "<RUN62912924E>", "message": "load failed when processing path: /mnt/models/flan-t5-small-caikit with error: RuntimeError('TGIS failed to boot up with the model. See logs for details')", "model_id": "flan-t5-small-caikit", "num_indent": 0, "thread_id":<PHONE_NUMBER>53792, "timestamp": "2023-09-21T19:39:45.781105"}

This part is expected. However, the InferenceService still shows the model as Loaded, which is unexpected:
  modelStatus:
    copies:
      failedCopies: 0
      totalCopies: 1
    states:
      activeModelState: Loaded
      targetModelState: Loaded
    transitionStatus: UpToDate

InferenceService status
predictor deployment logs
RH-internal slack discussion

I see that many text-generation process remains <defunct>  when the serving runtime is running after it hit this error:
sh-5.1$ ps aux
USER         PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
1000850+       1  0.5  0.0 19143780 611752 ?     Ssl  09:02   0:07 python3 -m caikit.runtime.grpc_server
1000850+     401  0.0  0.0      0     0 ?        Z    09:03   0:01 [text-generation] <defunct>
1000850+     403  0.0  0.0      0     0 ?        Z    09:03   0:01 [text-generation] <defunct>
1000850+     405  0.0  0.0      0     0 ?        Z    09:03   0:00 [text-generation] <defunct>

this means that Python caikit.runtime.grpc_server process does not wait() for the text-generation. This explains why it doesn't detect that text-generation failed ...

This should be handled by the container splitting (this sprint)

When running caikit+tgis in this configuration, this is no longer an issue.

@dtrifiro what about when running caikit+tgis in the single container? 🤔

Single container architecture was only a stopgap to the current architecture

Verified in RHOAI 2.5 RC4, now if I cause a failure while loading a model (i.e. give the wrong path to the deployment modal) the InferenceService will set the modelStatus to FailedToLoad with relevant error messages.


Layer.get_value_3d() progress

This is an issue to track the progress of adding get_value_3d() implementations to our different layer types and modes within each layer.

[ ] Image
[x] Labels (#3074)
[x] Points (#3508)
[ ] Vectors (removed since it doesn't have a 2D implementation)
[x] Shapes (WIP @ #3117)
[ ] Surface

cc @kevinyamauchi

Image I think can be checked off by
https://github.com/napari/napari/pull/7126


Add docs for SlaveContext and slave/unit ids

Motivated by #30

Pull Request Test Coverage Report for Build 155

0 of 0   changed or added relevant lines in 0 files are covered.
19 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 75.753%

Files with Coverage Reduction
New Missed Lines
%

slave.rs
19
57.81%

Totals

Change from base Build 153:
0.0%

Covered Lines:
1409

Relevant Lines:
1860

💛  - Coveralls


updated readme for venture

forgot to update it, shouldn't be anything useful

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


Refactor static classes under UiFramework

See NextVersion.md.
fixes iTwin/appui#89
Todo:

[x] Extract interfaces that can be kept beta so we dont expose directly the internal type.

Element in /framework folder are the public interfaces of the previous static manager classes.
Every Manager classes are no longer extending the internal class as this exposes the internal type as a public...
So every Manager classes are now directly calling the internal counterpart "manually", tests were added to ensure this, the Manager classes MUST NOT do any work or validation, ALL the logic must remain in the internal class.
That adds a lot of code, but I could not see a good way around it. And, it's not that bad conceptually, I guess.

I didn't comment on each instance separately in the PR, but we should probably:

Get rid of internal properties on interfaces
Implement new Framework interfaces by the internal corresponding class.

ui-test-app doesnt compile:
src/frontend/tools/PopupTestPanel.tsx:8:10 - error TS2305: Module '"@itwin/core-react"' has no exported member 'BadgeType'.

@mergifyio backport release/3.6.x

ui-test-app doesnt compile: src/frontend/tools/PopupTestPanel.tsx:8:10 - error TS2305: Module '"@itwin/core-react"' has no exported member 'BadgeType'.

And I forgot about those having a different behavior again, test apps are completely broken, I'll fix today.

All test apps fixed, removed SyncUiEventDispatcher from the refactor as the name events was clashing and felt wrong, we will revisit this system and see if we want to refactor it later.
2 layout related method on FronstageManager were removed in favor of their clones already existing in ContentLayoutManager:
FrontstageManager.setActiveLayout => UiFramework.content.layouts.setActive
FrontstageManager.setActiveContentGroup => UiFramework.content.layouts.setActiveContentGroup
deprecation notices were updated to 3.7 (instead of 3.6)
NextVersion.md was reset because we are now in 4.0 branch, I'll put back the NextVersion.md in the 3.7 branch when it is ported over.

@mergifyio backport release/3.7.x


How to use GMSGeometryContainsLocation for GeoJsonParser

I have drawn some polygons on the google map using geojson. Now whenever the user long presses on the map, I want to check whether the long press was in the polygon or not. But unable to convert GMUFeature to GMSPath to use in GMSGeometryContainsLocation. Could anybody suggest me how to do this?

@mallikarjuna0308 you can access the geometry object of the GMUFeature and get the path that way. Note that you will have to check the type of the GMUGeometry to ensure it contains a GMSPath.
e.g.
if let polygon = feature.geometry as? GMUPolygon {
    let paths = polygon.paths
    // Do something with paths here
}


Typo in the paper?

I'm reading the paper ，I‘m not sure if the expression is right?
1）m' instead of  m'[S] ?

2）Commit number is nz(m), while Prove is nz(m)-1？

Thank you for the corrections. You are right, and we'll update these in the next eprint update.

3）2l instead of 3l ？

No, actually 3l -- for a rather subtle reason in the proof.

Eprint updated https://eprint.iacr.org/2020/419. Thanks!


[RPI3] install on Archlinux-ARM

Could be nice have instruction about how to build/install in Archlinux-ARM.
The Wiki page isn´t working.

Could be nice have instruction about how to build/install in Archlinux-ARM.
The Wiki page isn´t working.

We are currently working on the new FAQ / WIKI Section

including:

Installation guide for the wallets (Different Operating Systems);
Wallet specific information about the functions and such;
The information and installation guide for the MasterNodes and The Regular staking process;

I'll make sure to follup up on this request and close the issue as soon as we have delivered the required information on our new WiKi / FAQ Web section!

Nice, a CLI wallet is important, my rpi3 run full time (online the hole year) and don´t have GUI. Could be nice have rpicore on it.

Nice, a CLI wallet is important, my rpi3 run full time (online the hole year) and don´t have GUI. Could be nice have rpicore on it

The v040 wallets will be released today or tomorrow, these are compiled (or builded) with our Gitian server and comes with properly signed packages / installers.
That means that the following will be available in the new release:

rpicoind (headless deamon / which doesn't accept any RPC requests anymore).
rpicoin-cli (the command-line as available within the GUI version)
rpicoin-tx
test_rpicoin_qt
test_rpicoin
for example: ./rpicoin-cli getinfo or ./rpicoin-cli stop (to stop the daemon from running).

WINDOWS

Windows installer
Windows 32bits & 64bits binaries

LINUX

aarch64
- arm (Raspberry Pi or Other ARM based kernels)
i686-pc
riscv64
x86_x64 (Ubuntu / Debian etc..)

MacOS / OSX

This will be a .dmg file that needs to be placed in the ~/Applications/ directory of your OS.
(More information about this build, will be there soon).

@semeion
As soon as the new release is available on Github - I will make sure to notify you about that!
Cheers!

Great news, very nice. I will try use it for use, but it is important provide some info about how to compile to armv7l, armv7h, ..., because my rpi3b is using Archlinux-ARM to armv7 (32 bits). I am not sure if it run on 32 bits OS version.

Great news, very nice. I will try use it for use, but it is important provide some info about how to compile to armv7l, armv7h, ..., because my rpi3b is using Archlinux-ARM to armv7 (32 bits). I am not sure if it run on 32 bits OS version.

We have recently published our new v0.4.0 wallets, we have the binaries for the Raspberry Pi which you can download directly from Github
Instructions
You can try the following from the terminal on your raspberry pi
sudo apt-get update && sudo apt-get upgrade
sudo apt-get install git
(It migt be necessary to create a swapfile / partition depending on if you wish to use the GUI version or just the daemon)
Now download RPICoin version 0.4.0 (latest). When released, newer versions can be found at github.com/rpicoin/rpicore/releases
Raspbery Pi uses arm-linux-gnueabihf file for 32 bit. Command below:
wget https://github.com/rpicoin/rpicore/releases/download/v0.4.0/rpicoin-0.4.0-arm-linux-gnueabihf.tar.gz
 tar -xvzf rpicoin-0.4.0-arm-linux-gnueabihf.tar.gz --strip 2
mkdir ~/.rpicoin
nano ~/.rpicoin/rpicoin.conf
Here add the following:
rpcuser=ahardtoguessrandomusername
rpcpassword=hard-long-very-longhjo4002nfchar-password
rpcallowip=<IP_ADDRESS>
addnode=seed1.rpicoin.com
addnode=seed2.rpicoin.com
addnode=explorer.rpicoin.com
maxconnections=32

Try launching the rpicoind (daemon) and see if this parses any error in the console:
./rpicoin-0-4-0/bin/rpicoind -daemon
/rpicoin-0.4.0/bin/rpicoin-cli getinfo
Checks the synchronization status and provides other info about current staking status. and block height.
If that works you can try launching the QT (GUI) version of the wallet with the following command:
cd ~/rpicoin-0.4.0/bin/
./rpicoin-qt
Let me know the outcome of this, otherwise we'll have to cross-compile the build!

Oh great news! Thanks for reply!
The above instructions teach how to install pre-compiled image right?
Archlinux-ARM have the AUR to build that kind of package. Using the AUR the user build from source using a single PKGBUILD file and make a system wide installation with dependencies control.
I will check the sources and try create a PKGBUILD file to build it locally and publish on AUR. On this way everybody using Archlinux could build and install the package using a single command. I have no time right now, but i will try it later.

I will close this issue as I assume it's resolved, if you need any further help you can re-open a issue! Thanks.


Switch to pigz to support multi-core tar command

pigz is a multi-core version of tar, which makes the docker build process much faster.

I'm not familiar with pigz. Is the resulting artifact equivalent to a gzip artifact? Does a user need pigz installed to tar -xzf the resulting files? Does that command even still work?

A couple of tests to show equivalence for consumers:
With gzip:
$ cat txt
oink
$ ls
txt
$ pigz txt
$ ls
txt.gz
$ gunzip txt.gz
$ ls
txt
$ cat txt
oink

With tar:
$ tar --remove-files --use-compress-program=pigz -cf out.tar.gz txt
$ ls
out.tar.gz
$ tar -zxf out.tar.gz
$ cat txt
oink

https://zlib.net/pigz/
It is gzip format but with the parallel multi-core feature. The traditional gzip command uses only one core but modern CPU/vCPU has more than one.

Have all containers been tested? Do some of them need pigz included in their Dockerfiles?
Should bazel_configure.py be updated to check for pigz if it is an external dependency?
Do any of the website pages need to be updated? For instance brew install pigz for someone compiling on their Mac.

let me try out all docker builds tomorrow.

Docker build testing result:
ubuntu18.04 - success
ubuntu20.04 - success
ubuntu16.04 - success
debian9 - fail - python3.6 not supported (unrelated to pigz)
debian10 - fail - protobuf issue (unrelated to pigz)
centos7 - success


audit: audit python-internal

Check that python-internal is only used for a limited number of formulae

Check that it is used as a build dependency only

[x] Have you followed the guidelines in our Contributing document?

[x] Have you checked to ensure there aren't other open Pull Requests for the same change?

[x] Have you added an explanation of what your changes do and why you'd like us to include them?

[x] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew typecheck with your changes locally?

[x] Have you successfully run brew tests with your changes locally?

[ ] Have you successfully run brew man locally and committed any changes?

Review period will end on 2021-01-26 at 00:00:00 UTC.

Review period will end on 2021-01-26 at 00:00:00 UTC.

Is this a Linux-only formula? What's it used for?

Is this a Linux-only formula? What's it used for?

@MikeMcQuaid https://github.com/Homebrew/homebrew-core/pull/69508

@MikeMcQuaid https://github.com/Homebrew/homebrew-core/pull/69508

Closing as we will use a requirement, see https://github.com/Homebrew/homebrew-core/pull/69508

Closing as we will use a requirement, see https://github.com/Homebrew/homebrew-core/pull/69508


DXP 7.1 support

I downloaded and installed DXP 7.1, we are running java 8. I tried to install the open id app version 0.5.2. I get this error:
2018-08-30 14:26:41.880 ERROR [fileinstall-/usr/local/liferay-ce-portal-7.1.0-ga1/osgi/marketplace][org_apache_felix_fileinstall:97] Failed to install artifact: /usr/local/liferay-ce-portal-7.1.0-ga1/osgi/marketplace/OpenID-Connect-plugin.lpkg
com.liferay.portal.lpkg.deployer.LPKGVerifyException: com.liferay.portal.lpkg.deployer.LPKGVerifyException: LPKG validation failed with {[missing requirement nl.finalist.liferay.oidc; version=0.5.2; type=osgi.bundle [caused by: Unable to resolve nl.finalist.liferay.oidc version=0.5.2: missing requirement com.liferay.portal.kernel.model; version=[1.0.0,2.0.0)]], [missing requirement nl.finalist.liferay.oidc.portalsettings; version=0.5.2; type=osgi.bundle [caused by: Unable to resolve nl.finalist.liferay.oidc.portalsettings version=0.5.2: missing requirement com.liferay.portal.kernel.model; version=[1.0.0,2.0.0)]]}
at com.liferay.portal.lpkg.deployer.internal.DefaultLPKGVerifier.verify(DefaultLPKGVerifier.java:127)
at com.liferay.portal.lpkg.deployer.internal.DefaultLPKGDeployer.deploy(DefaultLPKGDeployer.java:118)
at com.liferay.portal.lpkg.deployer.internal.LPKGArtifactInstaller.install(LPKGArtifactInstaller.java:125)
at org.apache.felix.fileinstall.internal.DirectoryWatcher.install(DirectoryWatcher.java:937)
at org.apache.felix.fileinstall.internal.DirectoryWatcher.install(DirectoryWatcher.java:871)
at org.apache.felix.fileinstall.internal.DirectoryWatcher.doProcess(DirectoryWatcher.java:485)
at org.apache.felix.fileinstall.internal.DirectoryWatcher.process(DirectoryWatcher.java:361)
at org.apache.felix.fileinstall.internal.DirectoryWatcher.run(DirectoryWatcher.java:312)
Caused by: com.liferay.portal.lpkg.deployer.LPKGVerifyException: LPKG validation failed with {[missing requirement nl.finalist.liferay.oidc; version=0.5.2; type=osgi.bundle [caused by: Unable to resolve nl.finalist.liferay.oidc version=0.5.2: missing requirement com.liferay.portal.kernel.model; version=[1.0.0,2.0.0)]], [missing requirement nl.finalist.liferay.oidc.portalsettings; version=0.5.2; type=osgi.bundle [caused by: Unable to resolve nl.finalist.liferay.oidc.portalsettings version=0.5.2: missing requirement com.liferay.portal.kernel.model; version=[1.0.0,2.0.0)]]}
at com.liferay.portal.lpkg.deployer.internal.LPKGIndexValidator.validate(LPKGIndexValidator.java:265)
at com.liferay.portal.lpkg.deployer.internal.DefaultLPKGVerifier.verify(DefaultLPKGVerifier.java:93)
... 7 more

Hi,
Thanks for trying out the plugin in 7.1.
Apparently the dependency definitions are not ready yet for 7.1, so far it has only been tested (and running in prod) in 6.2 and DXP 7.0.
I'll mark this issue as an enhancement and will try to find some time in the near future to add 7.1 support.

No explicit plans, but it's still on the radar.

For what is worth, this diff will build both DXP modules that can be deployed to Liferay 7.1 to active state:
diff --git a/dxp-oidc-filter/pom.xml b/dxp-oidc-filter/pom.xml
index a3fdb73..21040ae 100644
--- a/dxp-oidc-filter/pom.xml
+++ b/dxp-oidc-filter/pom.xml
@@ -38,7 +38,7 @@
                <dependency>
                        <groupId>com.liferay.portal</groupId>
                        <artifactId>com.liferay.portal.kernel</artifactId>
-                       <version>2.0.0</version>
+                       <version>3.0.0</version>
                        <scope>provided</scope>
                </dependency>
         <dependency>
diff --git a/dxp-oidc-portalsettings-web/pom.xml b/dxp-oidc-portalsettings-web/pom.xml
index 59851ff..fabf298 100644
--- a/dxp-oidc-portalsettings-web/pom.xml
+++ b/dxp-oidc-portalsettings-web/pom.xml
@@ -15,7 +15,7 @@
         <dependency>
             <groupId>com.liferay</groupId>
             <artifactId>com.liferay.frontend.taglib</artifactId>
-            <version>2.0.0</version>
+            <version>3.0.0</version>
             <scope>provided</scope>
         </dependency>

@@ -28,13 +28,13 @@
         <dependency>
             <groupId>com.liferay.portal</groupId>
             <artifactId>com.liferay.util.taglib</artifactId>
-            <version>2.0.0</version>
+            <version>3.0.0</version>
             <scope>provided</scope>
         </dependency>
         <dependency>
             <groupId>com.liferay.portal</groupId>
             <artifactId>com.liferay.portal.kernel</artifactId>
-            <version>2.0.0</version>
+            <version>3.0.0</version>
             <scope>provided</scope>
         </dependency>
         <dependency>

Thanks for this initial work, Michael.
I'll try and see whether we can rework that to a version range that gets
properly translated into OSGi Import-Package statements with ranges, so
that both 7.0 and 7.1 will be satisfied...
On Thu, 6 Dec 2018 at 17:29, Michael Chen<EMAIL_ADDRESS>wrote:

For what is worth, this diff will build both DXP modules that can be
deployed to Liferay 7.1 to active state:
diff --git a/dxp-oidc-filter/pom.xml b/dxp-oidc-filter/pom.xml
index a3fdb73..21040ae 100644
--- a/dxp-oidc-filter/pom.xml
+++ b/dxp-oidc-filter/pom.xml
@@ -38,7 +38,7 @@

com.liferay.portal
com.liferay.portal.kernel

                  <version>2.0.0</version>

                  <version>3.0.0</version>
                  <scope>provided</scope>
          </dependency>
   <dependency>

diff --git a/dxp-oidc-portalsettings-web/pom.xml b/dxp-oidc-portalsettings-web/pom.xml
index 59851ff..fabf298 100644
--- a/dxp-oidc-portalsettings-web/pom.xml
+++ b/dxp-oidc-portalsettings-web/pom.xml
@@ -15,7 +15,7 @@

com.liferay
com.liferay.frontend.taglib

       <version>2.0.0</version>

       <version>3.0.0</version>
       <scope>provided</scope>
   </dependency>

@@ -28,13 +28,13 @@

com.liferay.portal
com.liferay.util.taglib

       <version>2.0.0</version>

       <version>3.0.0</version>
       <scope>provided</scope>
   </dependency>
   <dependency>
       <groupId>com.liferay.portal</groupId>
       <artifactId>com.liferay.portal.kernel</artifactId>

       <version>2.0.0</version>

       <version>3.0.0</version>
       <scope>provided</scope>
   </dependency>
   <dependency>

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/finalist/liferay-oidc-plugin/issues/32#issuecomment-444935036,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABG69vIPEP2Mab-z5X5eHabZR0Yk9sOpks5u2UYHgaJpZM4WToCr
.

--
http://www.finalist.nl

Big thanks for developing this plugin! Are there any plans for supporting Liferay 7.1 and/or 7.2?


Do not block tokio runtime when syncing on-chain

One patch addresses production and the other addresses the tests.
This could help with #807, but I'm not convinced. [edit]: Indeed it did not help, the same test still hangs on this very PR.

LGTM, but given that it does not fix the actual issue, do we still want to merge that?

I would merge both patches as I think what we are currently doing is incorrect.
But specifically c9c9bb85ee97063d771940bc790fda5d394bb0d2 is a fix for production, not for the linked ticket.

bors r+

bors r+


Feature: Email validation

The email provided to reset password is not checked on valid format

I will leave this for @siemensikkema as he has better overview of what can be re-used for this new piece of functionality.


performance data request

New to this testing system.
I want to test the performance (timing) of multiple sized data segments of hipMemCpyAsync for different Rocm versions..  I think your testing has something like this.  How?

I see that the memory tests are run for basic types.  stdout can be processed.

Hi @srinathv, apologies for the lack of response. Do you still need assistance with this ticket? Thanks!

@srinathv Closing ticket due to lack of activity.  Please feel free to re-open ticket if you still need assistance. Thanks!


1-click serverless cloud deployments

What problem or use case are you trying to solve?
I'm exploring ways to easily deploy OpenDevin on my personal cloud (not locally), and automate the deployment process.
I'm using AWS, and prefer using the AWS Copilot CLI to deploy on a serverless stack – App Runner or ECS + Fargate.
Describe the UX of the solution you'd like

Deploy OpenDevin into my personal cloud account (AWS/Azure/GCP) form a single click / command. This requires programmatically deploying both front and backend to a cloud compute, preferably serverless (like on AWS Fargate).
[Optional] If I make local changes to the OpenDevin src code then I should be able 1-command redeploy (e.g., copilot deploy) it to my cloud, without pushing the code to the Github repo.

I tried deploying the Dockerfiles in OpenDevin/containers on AWS and ran into a bunch of errors like below:
06-19-2024 08:27:28 PM Starting OpenDevin...
06-19-2024 08:27:28 PM SANDBOX_USER_ID is not set

In Getting Started I see that the run command needs the local user Id, password, etc. but my local variables won't apply when the container is being run in a serverless environment.
Do you have thoughts on the technical implementation?
Ideally the Dockerfile could be self sufficient and not require environment variables to run. This will allow it to be easily deployed to any serverless compute.
We can modify the app to not require user Id / password. If I understand correctly the user/pass is needed for making file system changes in the host. Options:

Have a fully isolated cloud workspace within the container, which is not linked to the host's file system (i.e., no Docker Volume). I'm okay if the workspace is deleted along with the container.
Alternatively, for long term storage put the workspace in S3 and not on the host.

Describe alternatives you've considered

Ditch the Copilot CLI and write a custom script / CDK to build, deploy and run the docker containers on a self-managed EC2 instance. I will lose the serverless benefits.
Create a script that uses AWS Secrets Manager to get the ssh user/pass from a serverless host (Fargate), and run that script from Docker ENTRYPOINT. This seems complex and unstable.

In Getting Started I see that the run command needs the local user Id

It just needs the current user_id?

In Getting Started I see that the run command needs the local user Id

It just needs the current user_id?

Yes I mean this approach should work fine if I run it locally on Mac, or a server like EC2. But I'm not sure if commands like id -u work in a serverless environment (say AWS Fargate) because the server/host is abstracted.

But I'm not sure if commands like id -u work in a serverless environment (say AWS Fargate)

Could you check it?

I tried deploying on AppRunner (serverless) first but it doesn't allow ssh.
I'm exploring the alternatives above, and will use Fargate next. There are some ways to ssh into Fargate, although not recommended.
Another potential issue I see is that serverless environments may not support mounting the Docker socket (/var/run/docker.sock).
For now is it safe to say that the current setup of OpenDevin is not directly compatible with serverless?

@arko7n At the moment, OpenDevin cannot run on serverless environments. As you said, it requires access to the docker.sock in order to create the sandbox container, which is impossible to do within serverless environments (or SaaS~) like Google Cloud Run, AWS Fargate, CF's Workers, and so on.

Could LocalBox work?
https://github.com/OpenDevin/OpenDevin/blob/26fc3c886adf74614abfa14857273d8178dcb481/opendevin/runtime/docker/local_box.py#L12-L28


Vendor dependencies, update k8s to 1.16.3

Towards giantswarm/giantswarm#7675

While you are here can you also add:
[[constraint]]
  name = "github.com/Masterminds/semver"
  version = "=3.0.2"

I would create a PR but I don't want to introduce conflicts.


JsonPath filters not working

I added this test case to org.springframework.cloud.stream.module.filter.FilterProcessorIntegrationTests:
	@WebIntegrationTest("expression=#jsonPath(payload.a)>5")
	public static class UsingJsonPathIntegrationTests extends FilterProcessorIntegrationTests {

		@Test
		public void test() throws InterruptedException {
			channels.input().send(new GenericMessage<Object>("{\"a\": 3}"));
			channels.input().send(new GenericMessage<Object>("{\"a\": 5}"));
			channels.input().send(new GenericMessage<Object>("{\"a\": 8}"));
			assertThat(collector.forChannel(channels.output()), receivesPayloadThat(is("{\"a\": 8")));
			assertThat(collector.forChannel(channels.output()).poll(10, MILLISECONDS), is(nullValue(Message.class)));
		}
	}

The test fails with:
org.springframework.messaging.MessageHandlingException: nested exception is org.springframework.expression.spel.SpelEvaluationException: EL1022E:(pos 0): The function 'jsonPath' mapped to an object of type 'class org.springframework.expression.TypedValue' which cannot be invoked

You should add
<dependency>
	<groupId>com.jayway.jsonpath</groupId>
	<artifactId>json-path</artifactId>
</dependency>

into filter-processor pom and try again.

Better:
org.springframework.messaging.MessageHandlingException: nested exception is org.springframework.expression.AccessException: Exception while trying to deserialize String

Investigating if I did something wrong with the path expression.  Will update once I know more.  At a minimum it's a good test case to add.

after adding the dependency as listed above then one change to the jsonPath syntax (thanks @artembilan !) and one change in the assert expectation, it's passing for me:
	@WebIntegrationTest("expression=#jsonPath(payload,'a')>5")
	public static class UsingJsonPathIntegrationTests extends FilterProcessorIntegrationTests {

		@Test
		public void test() throws InterruptedException {
			channels.input().send(new GenericMessage<Object>("{\"a\": 3}"));
			channels.input().send(new GenericMessage<Object>("{\"a\": 5}"));
			channels.input().send(new GenericMessage<Object>("{\"a\": 8}"));
			assertThat(collector.forChannel(channels.output()), receivesPayloadThat(is("{\"a\": 8}")));
			assertThat(collector.forChannel(channels.output()).poll(10, MILLISECONDS), is(nullValue(Message.class)));
		}
	}

Excellent, thanks for the help!  Hopefully with this positive example I can get it working in my other case as well.

This is now supported in app-starters via: spring-cloud/spring-cloud-stream-app-starters#37


Extension truncates list of Github repos

When configuring deployment from Github, the extension is truncating the full list of repos for any Github account.
Environment
MacOS: 10.13.3
Extension: 0.7.0
VS Code: 1.21.1
Steps to reproduce

Right-click and select "Configure deployment source"
Select "Github"
Select a Github account from the list

Expected Behaviour
All repos for the account are shown
Actual Behaviour
Only the first 30 repos for the account are shown. Any other repos are not shown and are impossible to select.
Video: https://www.screencast.com/t/iIXgHdYHGk7

Fixed in https://github.com/Microsoft/vscode-azuretools/pull/170
Package updated in https://github.com/Microsoft/vscode-azurefunctions/pull/345


Invalid redirect_ur. This value must match a URL regisitered with the API Key

I integrate login with linkedin using ng-cordova-oauth, but I got error that: Invalid redirect_ur. This value must match a URL regisitered with the API Key.
I tired fix it but I can't
Please help me resolve this.
Thanks

By default the lib use as redirect_uri http://localhost/callback you can register this url as your redirect url on linkedin app config or you can pass this propery as a parameter to use the one you already register there, the lib allows you that.

Hi matheusrocha89 and nraboy
I tried config Authorized Redirect URLs: http://localhost/callback (I used http://localhost:8088/signin-linkedin before) but I can run my app. Could you please help me resolve this.

I don't think providers will let you specify a port.  If you added http://localhost/callback to LinkedIn as the callback and it's not working, I think you have other problems in your LinkedIn dashboard.
Regards,

@nraboy, thanks for your support, I fixed that issue. But I got an issue different is after authentication account with Linkedin sucessful then my app display a blank page before continue run my app.
Could you please help me resolve that.
Thanks and Regards

Not really sure what you're talking about.  Pictures and logs please.


setDate in viewMonthAgenda broken / unexpected behavior (vue)

Hi i was trying to set a date on the Month Agenda view but it behaves unexpectedly.
Context:

Browser [chrome latest]
Schedule-X [v1.46.1]
Schedule-X/vue [v1.43.3]

To Reproduce
Steps to reproduce the behavior:

Go to https://stackblitz.com/edit/github-iaect9?file=package.json,components%2Fapp-calendar.vue
Click on 'Set Date'
View does not update correctly (only the value in the input changes)

Expected behavior
View updates correctly to specified date
Screenshots

Thanks a lot for reporting!
looks to me, like there’s just a missing useEffect hook listening on the selected date.


Better outlining for solid geometry

I was optimistic that #8646 would be sufficient for outlining solid geometry (e.g. untextured buildings), but it's not quite. It's pretty good (not great) when the volumes are entirely convex, but outlines in concave parts of the model get swallowed. For example:

There should be an outline at the red arrow, but there's not because the outline completely fails the depth test against the two adjacent faces.
I think the best solution to this problem is to bias the depth of the outlines toward the camera so that they are visible. But we can't bias them too much, or outlines on back faces will show through the front faces.
WebGL's built-in polygonOffset doesn't quite work here, for two reasons:

polygonOffset can only bias triangle depth, not line depth. Biasing triangle depth is not ideal because it can interfere with pickPosition. Biasing outlines can too, but they're a far less important part of the scene.
Fixed-function polygonOffset doesn't work with log depth, so we have to implement it ourselves. (partially done, perhaps poorly, as part of #8600)

Some people advocate using the stencil buffer to render outlines. But no approach I've seen or can think of is able to avoid showing outlines "through" solid geometry when multiple outlined faces are rendered in a single draw call.
So here's my half-baked idea for properly biasing outlines:

Render a line (i.e. GL_LINES primitive) along the edge of each outlined face. An edge connecting two faces will be rendered twice. One draw call, lots of line primitives.
Include the face normal as a vertex attribute. This is easy because we need the normals for rendering the faces anyway, and we're likely re-using the same vertex data for the outlines.
In the fragment shader for an outline:

Find the ray to a diagonally-adjacent screen-space pixel, e.g. 1 pixel away from the center of the screen (I think) in both the X and Y directions from the current fragment.
Find the intersection of the ray with the plane defined by the fragment eye space position and the eye space face normal.
Subtract the Z coordinates of the original fragment eye space position and the ray-plane intersection computed above. This is the change in eye-space depth for a one pixel change along the face. Biasing the depth closer to the camera by this value should guarantee that the outline passes the depth test against fragments from the solid face.

I haven't implemented this or completely convinced myself that it will work, yet.

I've implemented the above, and it's pretty good! Except when faces are nearly edge-on with the camera, in which case it's a bit too eager:

In the solid-outlines branch.

Also I think it should be possible to do this computation in the vertex shader instead of the fragment shader to improve performance, but I'm going to get it working well in the fragment shader before I try that.

Ok after spending way too much time on this, I've gotten it pretty good, but as expected it's impossible to completely avoid artifacts with this approach of rendering separate lines with a depth bias.
For example:

Note the "tails" sticking into the solid geometry, and the generally rubbish look here due to lack of anti-aliasing.
We can fix both problems with a single-pass technique as described in these papers:
Two Methods for Antialiased Wireframe Drawing with Hidden Line Removal
Fast and versatile texture-based wireframe rendering
Both should offer much better performance than rendering GL_LINES in a separate command, and look nicely anti-aliased, too. However, they're both tricky (and less efficient) to implement without geometry shaders, which sadly WebGL lacks.

Artifacts go the other way too, where the line isn't solid when viewed on an angle:

(Note that these screenshots are on a 4K monitor and being blown up to a normal DPI, so they do exaggerate the problem somewhat)

I'm very interested if anyone has any ideas or knows of any other good papers for outline rendering.


perf: optimize utils/color.ts

hsl2rgb performance increased by ~10%
toRGB performance increased by ~50%
performance measurements were gathered using my own tests with random colors

Codecov Report

Merging #708 (81607f1) into main (bb0341d) will decrease coverage by 0.00%.
The diff coverage is 70.37%.

@@            Coverage Diff             @@
##             main     #708      +/-   ##
==========================================
- Coverage   88.13%   88.13%   -0.01%
==========================================
  Files          57       57
  Lines        4873     4872       -1
  Branches     1039     1040       +1
==========================================
- Hits         4295     4294       -1
+ Misses        313      312       -1
- Partials      265      266       +1

Impacted Files
Coverage Δ

src/utils/color.ts
78.18% <70.37%> (-0.39%)
:arrow_down:

Continue to review full report at Codecov.

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


function "__shfl_down(double, unsigned int, int)" has already been defined

Here is one more issue when building CSR5 with CUDA:
nvcc -O3  -w -m64 -gencode=arch=compute_52,code=sm_52 -gencode=arch=compute_52,code=compute_52 main.cu -o spmv -I/usr/local/cuda-6.5/include -I/home/oberhuber/NVIDIA_CUDA-10.1_Samples/common/inc -L/usr/local/cuda-6.5/lib64 -lcudart -D VALUE_TYPE=float -D NUM_RUN=1000
detail/cuda/utils_cuda.h(71): error: function "__shfl_down(double, unsigned int, int)" has already been defined

detail/cuda/utils_cuda.h(80): error: function "__shfl_up(double, unsigned int, int)" has already been defined

detail/cuda/utils_cuda.h(89): error: function "__shfl_xor(double, int, int)" has already been defined

detail/cuda/utils_cuda.h(340): error: function "atomicAdd(double *, double)" has already been defined

4 errors detected in the compilation of "/tmp/tmpxft_0036955f_00000000-4_main.cpp4.ii".

Tomas O.

I met the same errors , is there any way to solve errors?

I met the same errors , is there any way to solve errors?


Version 5.4.0

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.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

requirements: run: packaging >= 20 should not contain a space between relational operator and the version, i.e. packaging >=20

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub actions workflow below for more details. You can also ping conda-forge/core for further assistance or you can try rerendering locally.
The following suggestions might help debug any issues:

Is the recipe/{meta.yaml,recipe.yaml} file valid?
If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnnings.
Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11504701641.

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/meta.yaml) and found it was in an excellent condition.


uuid

Hi there
I see a uuid in the chat, what can I do?

If you are referring to the Discord user ID number seen in Minecraft chat when someone mentions someone, which looks like <@!112398217398213>, that is normal. Perhaps he could make it show the @Name instead but you would have to suggest that to him. If you are referring to something else you will have to provide more detail for him to help you.

I'm working on this at this very moment :)

Fixed in 2ad5755d06ce2be0c82528171af892fd0535009d, released in v2.4.0


Declare a BitVector in a struct

Hello,
Is it possible to declare a BitVector of a certain dimension in a struct?
Like [bool; 5] for a 5 bit value. I think I could just declare it as BitVector and that should work, but I also would like to enforce the length of the vector. Is this possible?
Thanks

Currently there is no array type with static sizing in the crate. The best way to accomplish this is with a Rust value such as u8 and a method which produces a &BitSlice reference over it.
I don't have a good concept for statically sized slices yet.

I'm going to close this as "impossible in the current language"; bitvec operates by storing information in pointers, because the type system does not allow me to encode information in it. The current unstable implementation of const generics looks like I will be able to make types such as BitSlice<O, T, const START, const LEN> that correctly manage direct memory locations, but until that stabilizes, bitvec has no way of representing these types.


strategy: 'override'

Description
Hi,
Will there be a :ui prop for strategy:‘override’ as in version 2?
merry christmas,
gregor

No this hasn't been and will not be implemented in Nuxt UI v3. However, once this PR https://github.com/nuxt/ui/pull/2938 is merged you will be able to disable tailwind-merge entirely.

Is there a way to remove a class?
If I want to remove ms-2 I have to write ms-[unset] or is there a remove:ms-2 thing :)

No you have to override it this way or write ms-0.


Make gem work with Ruby < 2.4 and >= 2.4

Compatibility changes
Update Travis to test with all 2.x versions
Minor code cleanups (inherit from StandardError instead of Exception, use string interpolation instead of concatenation)

Thanks a lot @citizen428 !!

Coverage remained the same at 100.0% when pulling 49add50da650e1aa0ef307c250f6f559eddd5c78 on citizen428:master into e66b0c34217e8c3cfab70f1d4e6abf26f876555f on manorie:master.

Coverage remained the same at 100.0% when pulling 49add50da650e1aa0ef307c250f6f559eddd5c78 on citizen428:master into e66b0c34217e8c3cfab70f1d4e6abf26f876555f on manorie:master.


Add Vale rules for capitalization

Description
We have a lot of unnecessary capitalization in our docs, see this example. Let's check if Vale can help.

@michaelbeaumont this is getting fixed now that vale is back in play no?

Yeah, it's still a case by case process. Maybe Vale can do disallow by default or something. I'll look into this.

Added error level for headings at least: https://github.com/kumahq/kuma-website/pull/1992


Staging

I couldn't use notify-keyspace-events without changing sed a little.

Thanks for this. Can you please rebase against master?

I believe that should do it. let me know if i did my rebase incorrectly. ill fix in a timely manner this time :)


"FMDBTransformer" value transformer don't declare in MTLFMDBSerializing?

I think the declare shoud be there

Sorry but I don't understand what you mean..

"FMDBTransformerForKey:" method has not't be declared in "MTLFMDBSerializing" protocol, then others could't known they can use transformer to transform the value fetched from FMDB when they just see the header file.


Dryad not merging with dps unit

See https://github.com/AxelBjork/Rush-Royale-Bot/issues/84#issuecomment-1650321735

After testing I noticed it indeed does not merge if you run Dryad wihout HQ.
Because I had this line with 'and':
https://github.com/niels-db/Rush-Royale-Bot/blob/6f52fc5ec487b21091729b06468f97b47058178f/Src/bot_core.py#L313C10-L313C10
Should be 'or' instead

After testing I noticed it indeed does not merge if you run Dryad wihout HQ. Because I had this line with 'and': https://github.com/niels-db/Rush-Royale-Bot/blob/6f52fc5ec487b21091729b06468f97b47058178f/Src/bot_core.py#L313C10-L313C10
Should be 'or' instead.
Let me know if that fixes it.

seems to fixed it awesome work!

hello how do I add this into the scrip I no -1 info on how to do any of this stuff with script I got lucky with being able to get the bot to work on the game

hello how do I add this into the scrip I no -1 info on how to do any of this stuff with script I got lucky with being able to get the bot to work on the game

@Piet-Jacobs @Newb420


Add GeoPandas/Fiona support

This will allow users to open any vector dataset supported by GeoPandas/Fiona and display it on the map. It can also be converted to ee.FeatureCollection if needed.
Fiona supported drivers:
https://github.com/Toblerity/Fiona/blob/master/fiona/drvsupport.py
geopandas.read_file
https://geopandas.org/docs/reference/api/geopandas.read_file.html#geopandas.read_file

This feature has been implemented. Users can now use geemap to load any geopandas-supported vector dataset and display attributes without coding.
https://user-images.githubusercontent.com/5016453/117090472-66345400-ad26-11eb-8ef7-687238b7ff0c.mp4


C# - completely remove RestSharp.Serializers.SystemTextJson

this has not been updated since Version="106.15.0" from 2021 and we are not actively using it anyway.
remove it from all c# sources and .csproj files

please test after removing "using RestSharp.Serializers;" statements


Support transformation for types with type annotation

Issue
Fixes: https://github.com/awslabs/aws-sdk-js-codemod/issues/549
Description
Support transformation for types with type annotation
Testing
CI

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Some intermediate state breaks when TSQualifiedName is replaced with TSTypeReference.


Switch webpack to rollup

Bug or Feature Request?
Build enhancement
Description
I just found out that webpack 4 by default produces UMD builds that cannot be required in node, nor automatically mocked with jest. The workarounds are quite hacky, so I am thinking about switching to rollup. Rollup seems to be used by most libraries nowadays.

It's an interesting thought, I'm aware Rollup is apparently better for libraries than Webpack, although I have never actually used Rollup before.
In terms of requiring in Node, I'm not sure what the use case for that might be? Do you mean in terms on unit testing? My first thought was isomorphic React apps, however that's not possible with this library anyway, as Highcharts can't be rendered on the server side.
I'd like to do it bit more reading on Rollup before we go ahead. I finally feel like I have a decent grasp of Webpack concepts after struggling with it initially. So I want to make sure I understand exactly what is going on, beyond the magic.

No worries, it's not urgent matter, and perhaps webpack will fix the problems.
Highcharts can't be used in isomorphic app, but it certainly can be rendered server side as our tests are already doing that. Simple use case for that would be charts included in emails.
I made a quick test with rollup, and it produced 10kB bigger bundle. I guess it was because it includes bigger parts of lodash.
I think the webpack is pretty well optimized in this project.
Let's close this issue for now?

@JustFly1984 react-jsx-highcharts is not importing whole of lodash. Instead it is using tree shakable lodash-es.
There are lodash functions that should be replaced, like find and findIndex, but the eslint plugin suggests replacing them with code that does not work on IE11. With IE11 the replacements are sometimes a bit more complex than simple one-liners.
find for array could be replaced with the following snippet:
function find(arr, condition) {
  if(!Array.isArray(arr)) return;
  return typeof arr.find === function ? arr.find(condition): arr.filter( condition )[0];
}

The problem with rollup is, that although lodash-webpack-plugin should work with rollup, it is not documented and I could not get it to work.

Closing this, as it can wait.


fix35982: allow BigIntLiteral to parse as PropertyName for literal object and indices

Fixes #35982
This PR allows more valid bigint uses in JS to parse and emit correctly. We will parse (and correctly emit) the following cases, but give the following errors:
{ ({1n: 123}); };
    ~~ 1538: A `bigint` literal may not be used as a property name.

[1,2,3][1n];
        ~~ 2538: Type `bigint` cannot be used as an index type.

const arr = [1,2,3];
const { 0n: f } = arr;
       ~~ 2538: Type `bigint` cannot be used as an index type.

class K {
    4n = 0;
    ~~ 1538: A `bigint` literal may not be used as a property name.
}

@typescript-bot test it

@typescript-bot test it

@typescript-bot user test this


[Metrics Advisor] update some files and generated files to core v2

Update Core v2 for generated file and the convenience layer
Updated the api review file
Fixes #15811

API changes have been detected in @azure/ai-metrics-advisor. You can review API changes here

@KarishmaGhiya CI is failing and there are some conflicts too. Could you please fix it?

@sarangan12 Everytime I resolve conflicts I see new ones appear that happen from the commits a year ago. Why this is happening is a mystery.

@KarishmaGhiya Maybe your local copy became stale???? Or you are not merging it right. Maybe you can try with a fresh copy.

There are some 15 tests failing now. Fixed the root cause for others. The ones that are failing now are all paginated requests - and are failing due to one of the following reasons -

Bearer token authentication is not permitted for non-TLS protected (non-https) URLs
TypeError [ERR_INVALID_URL]: Invalid URL: /https://endpoint:443/metricsadvisor/v1.0/enrichment/anomalyDetection/configurations/26ece682-80a6-4415-89a2-05903dd9a640/anomalies/query?$maxpagesize=2&$skip=2 => In this error there is a preceding \ - don't know where it is getting added from.
Any idea how to fix these ? @jeremymeng  @sarangan12

API changes have been detected in @azure/ai-metrics-advisor. You can review API changes here
API changes
+ export interface RestResponse {
+     _response: FullOperationResponse;
+     [key: string]: any;
+ }


Different color from original gruvbox?

I noticed that this extension use #292828 instead of #282828. Is this intended? I could help to fix this if this is not intended

No it's intended. As I said in README, this is a modified version of gruvbox.
There are some design considerations that are not convenient to explain here, but the use of #282828 in vim and the use of #292828 in vscode are well thought out.

Sorry I still don't get it why use different color in vscode. I looked at https://github.com/sainnhe/gruvbox-material, and even though that repo is the same "gruvbox-material" it uses #282828 (CMIIW)

As I said, it's a bit hard to explain this design. In short, in high-contrast workbench style, these fading colors can work nicely together.
https://github.com/sainnhe/gruvbox-material-vscode/blob/40ad566b2101c1c0fb5d20890c6356c0528951b1/src/palette/dark/background/medium.ts#L2-L4

This is an open source project, if you are not satisfied with this design, it's pretty easy to fork this repo and maintain your own branch, see CONTRIBUTING.md.


Issue 128 introduce poetry

Use poetry for building
Closes #128
Description

Motivation and Context
Makes it easier to manage dependencies and venvs in a reproducable manner.
Testing

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Checklist

[ ] Local run of ./build.sh succeeds
[ ] Code changes have been run against the repository's pre-commit hooks
[ ] Commit messages follow Conventional Commit Specification
[ ] I have read the README document
[ ] I have added tests to cover my changes
[ ] I have run all unit tests using pytest test/unit and they are passing.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hey @bneijt , thanks for getting started on this. One small question -- would it be possible for us to retain the setuptools build implementation while adding support for poetry? I ask because our build system is currently using setuptool and after looking into this a bit more, migrating to poetry would require some rework that we don't have bandwidth for at the moment. What are your thoughts?

I'm not sure how specific your build requirements are. Are you talking about:
custom_cmds = {
    "bdist_wheel": BDistWheelCommand,
    "unit_test": UnitTestCommand,
    "integration_test": IntegrationTestCommand,
}

If so, those commands are part of a code pipeline somewhere you can't update/change (outside of github) than I guess we would have to find a work-around.
One work around I could find doing a quick google was having Poetry generate a setup.py, but that would require you to be able to inject a command before the custom commands are required.
Could you give some insights into how these commands are being called in the pipeline? Maybe we can use a simple script to support it.
If that is to much trouble, than feel free to let me know and we can archive this pull request for a later date.
Thank you for responding!

I put the setup.py (al be it stripped) back in. You should be able to now still run python setup.py integration_tests if you need that for a pipeline outside of github.
Please let me know which commands the off-github pipeline runs and I can see if we can support those. Please consider publishing or sharing parts of the pipeline configuration to allow me to effectively judge if this is something we could hook in. If that is not possible, then I think we should close this PR and write a summary in the issue that this will be picked up at a later date and should not be worked on atm.

Hey @bneijt ,

Could you give some insights into how these commands are being called in the pipeline?

All the mentioned commands in custom_cmds are used in our CI pipeline using something similar to python setup.py $MY_CMD as you wrote above.

If that is not possible, then I think we should close this PR and write a summary in the issue that this will be picked up at a later date and should not be worked on atm.

Given that we cannot share our pipeline configuration I think this course of action would be best for now. My apologies for the time and energy you've invested in looking into this, I should have done better due diligence before giving you the go ahead to start work on this.

closing this out as we won't be supporting poetry in the short term


Is there a way to turn off the log 'No subscribers registered for event class xxx.xxx'?

Is there a way to turn off the log 'No subscribers registered for event class xxx.xxx'?

The documentation describes two different options.

Create your own EventBus instance and configure it

EventBus eventBus = EventBus.builder().logNoSubscriberMessages(false)
    .sendNoSubscriberEvent(false).build();

Configure the default instance prior first usage

EventBus.builder().logNoSubscriberMessages(false).installDefaultEventBus();

problem solved, Thanks a lot !


Update checkout amount appropriately

Fixes #1537

Unfortunate that the dependencies array causes prettier to update the styling of the function signature – causing additional indentation of the entire function body and an unnecessary large diff. Only the dependencies array are changed in reality.

@birkjernstrom Does this mean the bug was caused because of a missing dependency in the hook array?
If so, it could be nice to add the react-hooks/exhaustive-deps ESLint rule to avoid this in the future.

Thanks for merging & shipping @zegl :)
@frankie567 Yes, that's what caused it. Oh, nice I didn't know about that lint rule. I think that would be nice to have. However, I think it likely would complain in a few areas today even though it doesn't create a bug at all times, e.g certain object attributes being used for values to API calls (issue.id) which doesn't change, but the broader object itself might causing the effect to run again. Of course, this can be solved and perhaps it's worth it, but likely would require some review and refactoring to get right upon introducing such a linting flag.

@birkjernstrom I've PR this in #1540. I don't think we need to overthink it, it's a warning by default that helps us to not forget dependencies. If it's not sensible in a specific context, we can line-disable it or simply... Ignore it.

@frankie567 Nice and agreed. Provided we simply get a warning vs. error it feels like a no-brainer and nice addition and sanity check for the future.


Update algorithms for field layout

Add concept of computing field offsets seperate from computing the entire type sizes/shapes
Make field algorithm callable with just a DefType and not have it require a MetadataType
Allow an override for HasGCStaticBase

PTAL @MichalStrehovsky

@dotnet-bot test Windows_NT Debug please

@dotnet-bot test Windows_NT Debug please

LGTM


Using the data in your own (new) template

Hi,
I was wondering HOW I can parse all kinds of data. Since I can't find the correct stuff in the source.
Like I want to make my own acp, but I just can't parse the correct data. I want to be able to make it possible to get visits, site activity (pageviews and visitors), bounce rate, averange time on the site, etc... How can I do this and get this data?

It will be  great if we can use our own template!