No notification of expired sessiom

Earlier there used to be a sound saying to re-enter otp , but since yesterday, it just simple stops tracking and nothing from there.

Also , all the centred are showing "not applicable" beneath them , even if they match the criteria which I put.

Yes even I am experiencing same issues

Is it working for anyone now? I am tracking and it says No vaccines available (which is likely the case). Just want to confirm it is working for folks.
Also not getting a notification for expired session as @jujhar18 mentioned. Keeping a stopwatch timer for 15m by my side for now every time I start tracking here.
@yashwanthm Thanks so much for your efforts in making this superb and simple app 👏🏼

Is it working for anyone now? I am tracking and it says No vaccines available (which is likely the case). Just want to confirm it is working for folks.
Also not getting a notification for expired session as @jujhar18 mentioned. Keeping a stopwatch timer for 15m by my side for now every time I start tracking here.
@yashwanthm Thanks so much for your efforts in making this superb and simple app 👏🏼

Yes..but when box is unchecked. It is showing availability data

@hpk22 What does the check box for 'alternate tracking' mean? Do you recommend I run it with the box unchecked?

Yes, there are users getting confirmations.
On Sun, Jun 6, 2021 at 3:44 PM reelworshipper @.***>
wrote:

@hpk22 https://github.com/hpk22 What does the check box for 'alternate
tracking' mean? Do you recommend I run it with the box unchecked?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yashwanthm/cowin-vaccine-booking/issues/89#issuecomment-855373878,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJ6MSEO2XRBY7RW3XBBATTRNDBBANCNFSM46FSIAYA
.

--
Regards,
Yashwanth Maheshwaram
https://www.linkedin.com/in/myashwanth/

The notification is not working for me also.

@panth Notifications will appear only when there's availability. If you are logged in, booking attempt will happen almost instantly.
@reelworshipper : Alternate tracking is just an option. It's not mandatory to toggle it. If at any point, you think that the availability tracking is not working correctly, then you can switch it. For ex: If you see a telegram notification and the app doesn't pick up availablity, toggling this can help.
There was an slight change in how the data is displayed on UI. It was changed to show only available centres but, that led to users thinking that the app is not tracking. I've restored it to show all the content.
Auto sendng OTPs when tracking active was disabled accidentally. It is now restored.
If you are still expriencing any trouble, just refresh start over. This will load the latest version.

With notification I meant the periodic audio alerts to relogin every 15
mins. That doesn't seem to be working for me
On Mon, 7 Jun, 2021, 2:18 pm Yashwanth Maheshwaram, <
@.***> wrote:

@panth https://github.com/panth Notifications will appear only when
there's availability. If you are logged in, booking attempt will happen
almost instantly.
@reelworshipper https://github.com/reelworshipper : Alternate tracking
is just an option. It's not mandatory to toggle it. If at any point, you
think that the availability tracking is not working correctly, then you can
switch it. For ex: If you see a telegram notification and the app doesn't
pick up availablity, toggling this can help.
There was an slight change in how the data is displayed on UI. It was
changed to show only available centres but, that led to users thinking that
the app is not tracking. I've restored it to show all the content.
Auto sendng OTPs when tracking active was disabled accidentally. It is now
restored.
If you are still expriencing any trouble, just refresh start over. This
will load the latest version.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yashwanthm/cowin-vaccine-booking/issues/89#issuecomment-855738009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPOCEFT26XGNS3V5U2MT6DTRSBW7ANCNFSM46FSIAYA
.

@panth make sure you are on the latest by refreshing. The version yesterday
had some glitches with notifications.
On Mon, Jun 7, 2021, 9:01 PM panth @.***> wrote:

With notification I meant the periodic audio alerts to relogin every 15
mins. That doesn't seem to be working for me
On Mon, 7 Jun, 2021, 2:18 pm Yashwanth Maheshwaram, <
@.***> wrote:

@panth https://github.com/panth Notifications will appear only when
there's availability. If you are logged in, booking attempt will happen
almost instantly.
@reelworshipper https://github.com/reelworshipper : Alternate tracking
is just an option. It's not mandatory to toggle it. If at any point, you
think that the availability tracking is not working correctly, then you
can
switch it. For ex: If you see a telegram notification and the app doesn't
pick up availablity, toggling this can help.
There was an slight change in how the data is displayed on UI. It was
changed to show only available centres but, that led to users thinking
that
the app is not tracking. I've restored it to show all the content.
Auto sendng OTPs when tracking active was disabled accidentally. It is
now
restored.
If you are still expriencing any trouble, just refresh start over. This
will load the latest version.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/yashwanthm/cowin-vaccine-booking/issues/89#issuecomment-855738009
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACPOCEFT26XGNS3V5U2MT6DTRSBW7ANCNFSM46FSIAYA
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/yashwanthm/cowin-vaccine-booking/issues/89#issuecomment-856038265,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJ6MTOWNVIGQZ47O44MC3TRTQ5NANCNFSM46FSIAYA
.


Server Management lost after enable Role-based access control trough WAC

I have enabled RBAC trough WAC in WAC server. The process fail and WAC server isn't manageable through WAC website.
At WAC server side, in ServerManagementExperience this event is registered:
400 - PSRemotingTransportException: Connecting to remote server SERVERNAME failed with the following error message : The client cannot connect to the destination specified in the request. Verify that the service on the destination is running and is accepting requests. Consult the logs and documentation for the WS-Management service running on the destination, most commonly IIS or WinRM. If the destination is the WinRM service, run the following command on the destination to analyze and configure the WinRM service: "winrm quickconfig". For more information, see the about_Remote_Troubleshooting Help topic.
And wirm qc command executed in WAC server returns:
WSManFault
Message = WinRM cannot complete the operation. Verify that the specified computer name is valid, that the computer is accessible over the network, and that a firewall exception for the WinRM service is enabled and allows access from this computer. By default, the WinRM firewall exception for public profiles limits access to remote computers within the same local subnet.
Error number:  -2144108250 0x80338126
WinRM cannot complete the operation. Verify that the specified computer name is valid, that the computer is accessible over the network, and that a firewall exception for the WinRM service is enabled and allows access from this computer. By default, the WinRM firewall exception for public profiles limits access to remote computers within the same local subnet.
WINRM Firewall rules are created and I tried disabling Firewall too but the problem persist. WAC server is accessible and WAC website works perfect with others server.

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

ID: cc4cb072-7157-55b4-dbe1-c5723d2873b3
Version Independent ID: 70a3605d-6d21-5719-b661-6023bad96627
Content: Windows Admin Center Overview
Content Source: WindowsServerDocs/manage/windows-admin-center/overview.md
Service: windows-server-threshold
GitHub Login: @nwashburn-ms
Microsoft Alias: niwashbu

Configuring RBAC will restart the WinRM service. It is possible the service failed to come back online in your case.
Are you able to connect to the server using the traditional Remote Desktop Connection, or via the local console? If so, please check to see if the WinRM service has crashed using services.msc or Get-Service WinRm in PowerShell

@rpsqrd will you please take a look at this?

@sergiomartinezconde Thanks for sending the log details! Can you run rsop.msc and check if the following group policy is set?
Computer Configuration\Administrative Templates\Windows Components\Windows Remote Management (WinRM)\WinRM Service\Disallow WinRM from storing RunAs credentials
This policy is sometimes set when the security baseline policy published by Microsoft is applied to a machine. While JEA (the PowerShell feature underlying WAC RBAC) does not actually store any credentials, WinRM sometimes mistreats the virtual administrator account as storing credentials and therefore gets in an inoperable state once JEA is configured.

Hi rpsqrd
Sorry for the delay in my answer but email notifications were in Junk folder.
You are right, Disallow WinRM from storing RunAs credentials is Enabled by a GPO.
This implies that we can't enable this policy if we want to use JEA?

Hi rpsqrd
There is any way to solve this issue?
Regards

Sorry for not responding earlier. Yes, it looks like you will need to disable the GPO policy that is configuring "Disallow WinRM from storing RunAs credentials" to unblock this. If you want to get back to your original state where RBAC was not configured but the GPO was applied, disable it temporarily, run the following command in PowerShell, then re-enable the policy.
Unregister-PSSessionConfiguration microsoft.sme.powershell -Force

It works!!!!
I have blocked the GPO to that server and I have executed the Powershell command and "Et voilà!". Server could be managed from WAC.
Thanks @rpsqrd  for your help.
Regards

What is the best way to solve the issue please ans to my site  http://www.educationbangla.com/


[SECURITY] Bad UX for lost password / password reset

Upon successful reset of password through lost password email, ChangePasswordForm will redirect you to whatever location happens to be stored in Session::get('BackURL'). This may be desired in some cases but I think it leads to a a bad user experience on two counts

Unpredictability about what page the user will end up on
No feedback / success message for the user may leave them in doubt as to whether the password reset was successful

Think it would be better to ignore the BackURL and instead redirect people back to the change password form every time with a success message. Also think it's a bad idea to ever store a BackURL in Session with no context. If a user starts a process that causes this value to be set but doesn't finish it, then that value could be picked up much later and create an unpredictable/undesirable result, in the same process or a completely unrelated one.
Related: https://github.com/silverstripe/silverstripe-framework/issues/3757

Agreed. The entire password reset function is a bit of a pain from a user point of view

I would not redirect back to the change password form. I personally hate to see the form again, after submitting. It somehow feels like I did something wrong.
Suggestion:
Create a "your password has been reset" page, with a text explaining what happened, and a link saying something like "Go back to where you were before resetting your password", which obviously would point to the BackURL.

If the ChangePasswordForm set a ChangePasswordBackURL the first time it's seen (when it knows the referrer), and redirects them back when it's successfully changed.
The only issue is that it wouldn't always work (e.g. this form is used when the user's password has expired, which is set whenever the user logs in and their password has expired, so you can't take them back to the login form as that's pointless too.

I would not redirect back to the change password form. I personally hate to see the form again, after submitting.

I'm not attached to showing them the form again, the important thing to me is that we give users a success message, not blindly throw them to some location they may or may not want to go to with no indication of whether or not the password change was successful.
@dhensby do you have any thoughts about SemVer on this one? I'd like to fix this in 3.x but is this a broken enough behaviour that we can stop redirecting people and not consider it a breaking change? I guess it could be an opt-in fix with a config variable but seems convoluted.

So I whipped up a change that redirects users back to the form (for now, because it's easy) and displays a confirmation message, but also shows a link to the BackURL if available, which I thought might be a nice compromise.
But then I wondered if this could be a security vulnerability, because the BackURL is posted in via the form so that could be manipulated and maybe cause this page to display some undesirable HTML instead of a valid URL?

As long as you validate that it's a site url, and it's properly encoded, then it's fine.

@jonom you could make the form redirect to that URL after x seconds (if it is valid).
Validate URLs by doing Director::is_site_url()

In terms of SemVer: I think the best process here is to use deduction..
is it a patch: No
is it a new feature: yes
is it a breaking change: maybe?
So, it's definitely not the 3.3 branch, it might be 3 branch if we think that this doesn't "break" an API
I think it's reasonable that a unit test would assert that this step moves a user on to the BackURL, so I think, unfortunately, this is a 4.x change.
What we could do is add this to 3.x with a config option to turn it on and then just enable it in 4
If you PR against 4, please add it to the changelog/upgrade notes

There's already a check against Director::is_site_url() but I don't think that's sufficient at all... it falls back on this function:
public static function is_relative_url($url) {
	return (!Director::is_absolute_url($url));
}

So it looks to me like you can pass any string at all to Director::is_site_url(), and if it's not specifically detected as an external URL, you'll get a :+1:. It's not validating that this is actually a URL we're dealing with and safe for attribute insertion.
Yeah I had a feeling it would be a 4 thing. I'm not sure it's worth doing an opt-in change for 3.
@Firesphere is this overlapping with work you're already doing? I might leave it to you if it is so we're not double handling this issue. have left some thoughts in your dev list post.

It's not validating that this is actually a URL we're dealing with and safe for attribute insertion.

I guess that comes down to casting in the template, then?

Yeah I guess I would just have to make sure it ends up getting escaped properly and still works properly as a link. Probably a way easier problem than I've made it out to be in my head.
I'm going to pause exploring this until I see what @Firesphere comes up with. @dhensby feel free to close this issue if you see it as more of a feature request than a bug.

You're free to comment and/or help. I've created a separate repository, with a framework version minus security (almost).
And a separate Security module here: https://github.com/Firesphere/silverstripe-security
My current implementation, is showing a text saying password is changed and the user is logged in, and can follow a link to go back wherever they came from (backurl from session)
Also done massive restructuring/refactoring, so feedback on there is very welcome :+1:

@Firesphere is that going through an RFC process? Does "Security" include Members?
If security is pulled out, wouldn't the whole admin have to be pulled out too?

Pulling Security out doesn't quite sounds feasible so interested to see how it turns out. Does seem like RFC territory, but I have to say based on RFC-2 that if I'd put my energy in to building a proof of concept I could demonstrate instead of writing and defending a theoretical proposal it would probably be all done ages ago. So it's probably a good idea to do some experimenting first but then maybe write an RFC once you're confident about the general form your solution will take?

@dhensby I am not pulling it out really. I separated it to make it easier to work on and keep track of issues etc. not to be a separate module.
Once done, it can be put back in quite easily.

ah, ok

The password reset form has substantially changed since we introduced the MFA module which now ships by default with the installer. I've recorded what it looks like https://youtu.be/VfTNPzOonfg
I guess we could have a toast notification when you successfully access the CMS after changing your password. But otherwise the flow looks pretty good to me.
@silverstripeux Maybe have a glance at the video and make call on if it's something you would like improved.

Please, no toast notification, they're useless to the end user, it doesn't help them at all, because it's a tiny notification in the corner of the screen, a proper notification is a lot more useful.

For the most part the experience seems good. Yup, missing some sort of notification, but I would think a toast notification is ideal in this situation. The user already knows its worked as they have entered the CMS interface. Toast notifications are used when things happen as the user would expect, e.g. continue, everything is good. I think this is a good example of what they have been designed for IMHO.

I watched the video (thanks @maxime-rainville), it certainly looks prettier than before but I'm not sure it anything has changed apart from a re-skin? No idea if my thoughts on Session::get('BackURL') are relevant today (or were 4 years ago) but I'd personally still like to see some kind of explicit confirmation that the password change was successful.
What happens if you were trying to access a restricted page rather than the CMS? Do you still get redirected there immediately? Depending on the design of the site I'd still argue that users might be left unsure whether the password change worked in some circumstances (particularly since a toast notification not really an option in that context out-of-the-box), so I think my original suggestion of showing a success message with a link you can click to go to the location you were trying to log in to (rather than redirecting immediately) is still a decent idea.

If you're trying to access a restricted page, it seems you still get redirected to that page after resetting your password. That's what it looks like. https://youtu.be/FjgovT_iwX8

@maxime-rainville it's not terrible but if we kept the redirect behaviour I think it would be nice if the password reset form at least told you what was going to happen. Like 'Please enter a new password' could become 'Enter a new password, then we'll take you straight to /admin.'
BTW should that password reset email have some more meat to it? Here's an example from Slack:

@sminnee mentioned the design aspect, would be great to see that cleaned up.

Being lazy I didn't read the all the threads... having a success message prior to moving into the CMS also works.

I left a comment very relevant to this issue here: https://github.com/silverstripe/silverstripe-framework/issues/3757#issuecomment-683987939. TLDR; The result of resetting your password and successfully logging in is an offer to log in as someone else, with no way to get back to the page you were trying to log in to, or the default_login_dest as set by the developer.

Related https://github.com/silverstripe/silverstripe-admin/issues/1066

'Enter a new password, then we'll take you straight to /admin.'

That literally gives away unwanted information. I would personally not like it if I'd see something like that.
It would work for non-admin URLs, but for admin URLs, I would not like to see that.

gives away unwanted information

@Firesphere Can you clarify what you mean by unwanted information - do you mean that it's cryptic / unhelpful? I just think that if we keep the immediate redirect behaviour (no success message displayed to user) then we should inform the user that that is going to happen.
My preferred UX which I outlined here (sorry for the fragmentation) would be to not immediately redirect the user and instead display a success message on a new action like Security/passwordchanged. Then instead of displaying the exact location we could just say 'Continue' or similar vague action on a link that take you to whichever is applicable of BackURL, default_reset_password_dest or default_login_dest.
Example:

@Firesphere Can you clarify what you mean by unwanted information - do you mean that it's cryptic / unhelpful? I just think that if we keep the immediate redirect behaviour (no success message displayed to user) then we should inform the user that that is going to happen.

It gives information that's not wanted, e.g. giving away the next endpoint

Is it sensitive information? I guess I was focused on BackURL and assumed we'd just be displaying the URL they were trying to access in the first place (i.e. already known to them). But I suppose that wouldn't always be the case. Maybe it could just say something like "Enter a new password, then we'll take you straight to your destination" instead (if we stick with the redirect behaviour).

Yeah so putting /admin in the message is a low level security vulnerability in that you're disclosing the admin URL. Our implementation of a configurable admin URL is half baked and doesn't actually work, but we should still aim not to perpetuate these issues.
We can say 'Enter a new password, then we'll take you straight to the CMS.' (or "the admin area" if the CMS module isn't installed), then we're avoiding that problem.

Just discovered there is a Security::$default_reset_password_dest config setting which does work. I still think that showing a success message at /Security/passwordchanged is a good idea, and the Continue button should take you to whichever is not null in this order:

BackURL
default_reset_password_dest
default_login_dest

Edit: oops messed up the order, BackURL should be first 😅

https://github.com/silverstripe/silverstripe-framework/issues/3757#issuecomment-684007503

Yeah so putting /admin in the message is a low level security vulnerability in that you're disclosing the admin URL

Yeah I was assuming that the destination (could be /admin or any other restricted page or URL) was already known to the user because typically you end up at a login screen because you've tried to access that URL. E.g. if you're trying to to access the CMS, the workflow would be user tries to access /admin -> user is redirected to /Security/login.

We can say 'Enter a new password, then we'll take you straight to the CMS.' (or "the admin area" if the CMS module isn't installed), then we're avoiding that problem.

I would rather have a 'friendly name' than a URL (e.g. "the CMS" as you suggested) but a restricted URL on a SilverStripe app could be anything and isn't necessarily going to be easily resolvable to a nice name as is the case with the CMS. Also, if you did give away the name then presumably that's a form of the same security vulnerability being discussed - namely revealing that the endpoint does exist (and by the way, here's a friendly title for it...)
Ultimately for simplicity probably just saying something vague like "we'll take you straight to the URL you were trying to access" (preparing users to expect a redirect) would be fine. It's a moot point though if we do the better fix which (as @Firesphere originally suggested) is to ditch the redirect and instead display a confirmation message with a link to whatever the redirect destination would have been.


Transpiling from SQL to PRQL

This is a longer-term idea — but now we're using sqlparser to write SQL, we could also use it to read SQL and produce PRQL.
We'd need to write:

a fairly simple compiler from sqlparser AST to PRQL AST
Display implementations which convert PRQL AST to PRQL code

This would be useful for people transitioning from SQL. At least initially, the main users are likely to be engineers who are frustrated by SQL because they've written a lot of it.
We wouldn't need to support all of SQL for it to be useful — there's a fairly small subset that's used a lot — and possibly a larger subset which we could implement with s-strings.
We also wouldn't need to automatically do things like variable "creation" (e.g. replacing salary + payroll_tax + benefits_cost with gross_cost), which would be much more difficult (but great if we could!).

Update: this is currently not possible yet, but we are not far off.
The Display trait for prql's AST Node is done, we just need to parse SQL and convert it to AST, which should't be a major problem.

A SQL -> PRQL transpiler would  REALLY ease adoption for PRQL.
Even if it had some disclaimers, it lowers a lot of barriers if a new user is trying to learn/adopt PRQL and could copy/paste a bunch of SQL queries.

Using Apache Calcite for parsing queries and turning them into an AST could be an idea.
Calcite is written in Java, though, and not in Rust.

We'd likely use sqlparser-rs, which we currently use to represent SQL AST, and it builds the SQL string.


Enables Travis Cache

Enabling Travis cache should make builds faster (if there's no changes in gems versions).
Tell me if there's something more to do! :smile:

Thanks for this. Currently I don't use this myself anymore. Not using ruby right now. But if you have ideas for things to add, feel free to add issues and let's discuss 👍

@bitboxer Ok! I'm gonna create a new issue, if you could, assign it to me! Thanks!


Fix manhole middleware tests.

These tests suddenly run under Travis with Twisted 16.0 (yay!), but are very broken (eep!).

This version passes tests locally for me on Twisted 16.0. Will need to see what happens with Twisted 13.2 in Travis.

The tests are skipped for Twisted 13.2 on Travis as they always were and now pass on Twisted 16.0. Given that the tests were completely broken (on all Twisted versions before), these feels like it's already a major improvement.
Ready for review.

Tests now run & pass for Twisted 13.2 too.

@rudigiesler @KaitCrawford Would you mind reviewing?

:+1:


CI: packaging: improve datadog-package tool fetching

What does this PR do?
This PR attemps to fix sporadic failures of the OCI packaging job by reducing the amount of things to fetch, and retrying on error

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Fix quiries typo in docs.

Noticed when reading through the documentation two instances of queries spelled incorrectly :+1:

Thanks!


Item Conduits fail to honor can extract from slot with Reconstructor

Issue Description:
Item Conduits cause a dupe bug with the Reconstructor mod, as they will always extract an item even if the item isn't fully repaired.
Ref: smbarbour/Reconstructor#7
What happens:
Items get duped every tick
What you expected to happen:
No duping
Steps to reproduce:

Place and power a Reconstructor
Put an extracting item conduit on the bottom and connect it to another inventory
Put a damaged item in the Reconstructor.
Observe item getting duped

Affected Versions (Do not use "latest"):

EnderIO: 5.2.59
EnderCore: 0.5.76
Minecraft: 1.12.2
Forge: <IP_ADDRESS>54
SpongeForge? no
Optifine? no
Single Player (not tested on server)

Your most recent log file where the issue was present:
Logs currently unavailable. I'll get one as soon as I can.

see the original issue

You may have missed the code leading up to that: https://github.com/SleepyTrousers/EnderIO/blob/c09dc117660de265a2619fb42134744d49924fec/enderio-conduits/src/main/java/crazypants/enderio/conduits/conduit/item/NetworkedInventory.java#L151-L174

The point is that the actual result of the extract should be used to insert into the destination instead of the result of the simulation.  The Forge InvWrapper does not use the same calls for simulation and the actual extract.  This is correctly handled by every other item transport system.


Did you try the amplify env pull --restore command?

          Did you try the `amplify env pull` --restore command?

Originally posted by @kaustavghosh06 in https://github.com/aws-amplify/amplify-cli/issues/1523#issuecomment-496645977
This did not work for me.

Hey @CloudInnovator, could you open the issue using our templates as it allows us to collect information that would assist in troubleshooting the issue.


Disable tap handler by default, except in mobile Safari

Atm it is still needed to simulate taphold in iOS
Fix #7262, close #7255.
To maintainers: consider alternative #7026.
More context: #6980.

@johnd0e Why is this one showing as "draft"? I think we should merge ASAP.

@IvanSanchez
Ok

Thanks for the fix! It would be great, if a minor release of Leaflet with this fix could also be available in the near future...

Unfortunately this fix is not ultimate, see https://github.com/Leaflet/Leaflet/issues/7255#issuecomment-691109158


Add cucumber-rust to related-tools.md

Hello,
This PR adds cucumber-rust crate in the list of 3rd party tools!
Thanks

Hi @spontoreau,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for this generous offer we hope you will:

✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber


script to save / restore system config

simple CLI script to save / restore / factory reset all the settings.
Its just tarballing the etc directory but should add some saftey checks to ensure config is good on restore.

needs documentation update

documentation updated
http://fmad.io/en.fmadio10-manual.html#config-save


Prevent people from accidentally using Python 2.

I'll let @paultag make the final decision but I think the intent is for this library to work w/ Python 2

Yeah, I was thinking this API wrapper could be useful to both Python 2 and 3 users -- did you find any bugs in Python 2 (I'll admit I didn't write this in a Python 2 virtualenv)
I'll close this, let's weed out Python 2 issues!


Postprocessing error when embeding metadata after already embedding thumbnail

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2021.10.10. (update instructions)
[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. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
Also See - https://www.reddit.com/r/youtubedl/comments/qbae5l/comment/hhe0hg1/
Verbose log
[opus @ 0000014cc384c180] Unsupported codec id in stream 1
Could not write header for output file #0 (incorrect codec parameters ?): Invalid argument
Error initializing output stream 0:1 --
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (copy)

ERROR: Postprocessing:   Stream #0:1 -> #0:1 (copy)
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 3236, in process_info
  File "yt_dlp\YoutubeDL.py", line 3406, in post_process
  File "yt_dlp\YoutubeDL.py", line 3388, in run_all_pps
  File "yt_dlp\YoutubeDL.py", line 3367, in run_pp
  File "yt_dlp\postprocessor\common.py", line 25, in run
  File "yt_dlp\postprocessor\common.py", line 126, in wrapper
  File "yt_dlp\postprocessor\ffmpeg.py", line 692, in run
  File "yt_dlp\postprocessor\ffmpeg.py", line 313, in run_ffmpeg_multiple_files
  File "yt_dlp\postprocessor\ffmpeg.py", line 351, in real_run_ffmpeg
yt_dlp.postprocessor.ffmpeg.FFmpegPostProcessorError:   Stream #0:1 -> #0:1 (copy)

@pukkandan I think it is fixed

Well, we didn't change anything. Though, if you are no longer affected, feel free to close the issue

No problem still exists

Closing in favor of #11487


Update component readmes

This should fix the pre-commit pipeline.

I didn't even notice that they were added :sleeping:
Already fixed.
Fixes #539


Clean up AzureCI matrix

The CI matrix will probably need some further adjustment, as will general build configuration, but not for this PR as it seems out of scope.

Originally posted by @stuartarchibald in https://github.com/numba/numba/issues/8822#issuecomment-1478350653
#8822 PR may have increased the number of CI configuration too much. We should consider reducing it and perhaps remove testing of older python versions.

Currently looks like this:
$ python -c 'import numba.tests.support; numba.tests.support.print_azure_matrix()'
NumPy | Python | Count
-----------------------
 1.21 |  3.8   |   4
 1.22 |  3.8   |   4
 1.22 |  3.9   |   1
 1.23 |  3.8   |   2
 1.23 |  3.9   |   2
 1.23 |  3.10  |   1
 1.24 |  3.10  |   3
 1.24 |  3.11  |   3
 1.24 |  3.8   |   1
 1.24 |  3.9   |   1


Use vLLM's native Outlines integration for outlines.serve.serve

Now that Outlines was integrated in vLLM we can remove the server code from Outlines and instead import vLLM's code for outlines.serve.serve, which should simplify the codebase and ensure that the integration does not break unexpectedly in the future.

I guess this only includes the server code, as you mention, and not the outlines.integrations.vllm code, right? As the new vLLM guided_decoding module that you're referring to only really supports OpenAI server usage, not "vanilla usage" of the vllm.LLM class.

@saattrupdan the logit processors provided in outlines.integrations.vllm can be replaced with those in https://github.com/vllm-project/vllm/blob/main/vllm/model_executor/guided_decoding/outlines_logits_processors.py.

@saattrupdan the logit processors provided in outlines.integrations.vllm can be replaced with those in https://github.com/vllm-project/vllm/blob/main/vllm/model_executor/guided_decoding/outlines_logits_processors.py.

Gotcha! So solving this issue amounts to deprecating the vLLM logits processors in Outlines and changing to the ones from vLLM directly, right?

I think so, yes.


Add the ability to mock the RECONNECT message

https://dev.twitch.tv/docs/irc/commands#reconnect-twitch-commands
This may be more targeted towards Twitch IRC library creators than bot creators, but it's a useful response to test nonetheless

@pajlada Sorry I didn't see this sooner! I'll pull it down and give it a peek this afternoon, then hopefully get it merged! 😁

@pajlada Sorry I didn't see this sooner! I'll pull it down and give it a peek this afternoon, then hopefully get it merged! 😁

@pajlada The only issue I have right now is that the RECONNECT message is currently handled the same way normal messages are, rather than the way that system messages are. For example, if you send a /ping command, it's handled specially by the handlePINGMessage helper. With the way that the reconnect is being handled in your PR, the response gets pushed through the renderMessage helper unnecessarily, causing a superfluous @ to be appended to the response.
I'd prefer that we move the handling of the reconnects into a method on the connection (connection.sendReconnect, similar to connection. sendUnknownCommand). Something like this:
sendReconnect = () => {
	incrementStat('reconnects')
	this.send('RECONNECT')
}

That means we'll also have to add special handling for the reconnect message. We can do that in handlePRIVMessage, right after line 45 where we parse the message:
if (command.toLowerCase().trim() === 'reconnect') {
	return connection.sendReconnect('RECONNECT')
}

@pajlada The only issue I have right now is that the RECONNECT message is currently handled the same way normal messages are, rather than the way that system messages are. For example, if you send a /ping command, it's handled specially by the handlePINGMessage helper. With the way that the reconnect is being handled in your PR, the response gets pushed through the renderMessage helper unnecessarily, causing a superfluous @ to be appended to the response.
I'd prefer that we move the handling of the reconnects into a method on the connection (connection.sendReconnect, similar to connection. sendUnknownCommand). Something like this:
sendReconnect = () => {
	incrementStat('reconnects')
	this.send('RECONNECT')
}

That means we'll also have to add special handling for the reconnect message. We can do that in handlePRIVMessage, right after line 45 where we parse the message:
if (command.toLowerCase().trim() === 'reconnect') {
	return connection.sendReconnect('RECONNECT')
}

@trezy Thanks for your feedback! I implemented the code changes as you suggested and the tests still work! :slightly_smiling_face:

@trezy Thanks for your feedback! I implemented the code changes as you suggested and the tests still work! :slightly_smiling_face:


Build tf_core_kernels on With VS2015 stops with "fatal error C1060: compiler is out of heap space"

System information

Build tf_core_kernels.vcxproj
Windows 10 Enterprise Build 1607
TensorFlow installed from source
TensorFlow version Release 1.4.0 from github
VS2015 Enterprise 14.0.25431 Update 3

Cannot build tf_core_kernels.vcxproj (generated with CMake) because of a fatal error C1060: compiler is out of heap space.
Source code / logs
1>------ Build started: Project: tf_core_kernels, Configuration: RelWithDebInfo x64 ------
1>  training_ops.cc
1>e:\software\tensorflow\tensorflow\contrib\cmake\build\external\eigen_archive\unsupported\eigen\cxx11\src/Tensor/TensorBase.h(245): fatal error C1060: compiler is out of heap space
1>  INTERNAL COMPILER ERROR in 'C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\x86_amd64\CL.exe'

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Have I written custom code: No
OS Platform and Distribution: Windows 10 Enterprise Build 1607
Bazel version: N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce:
Followed instructions here:
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake
and then:
MSBuild /p:Configuration=RelWithDebInfo ALL_BUILD.vcxproj

I hit this out of heap space error before as well, this prones to happen when you don't have enough RAM.
A workaround is to remove all /MPcompile flag to make cl.exe create less processes and hence use less memory (works for me).

https://github.com/tensorflow/tensorflow/search?l=CMake&q=%2FMP&type=&utf8=✓
https://msdn.microsoft.com/en-us/library/bb385193.aspx

If may because of two things:

you don't have enough memory.

you are using 32 bits toolchains.
Please upgrade cmake to 3.10.x and add "-T host=x64" to cmake command line args.

In the install instruction from tensorflow, it's already been mentioned.
Building TensorFlow from source can use a lot of RAM. If your system is memory-constrained, limit Bazel's RAM usage with: --local_resources 2048,.5,1.0.


Method move_and_collide always returns null

Godot version:
v3.2.2.rc.custom_build.36a30f681
OS/device including version:
Windows 10
Issue description:
Method move_and_collide returns null when KineticBody2D collides with RigidBody2D
and should return instance of KinematicCollision
Minimal reproduction project:
GodotCollisionBug.zip
Remember to enable Debug->Visible Collision Shapes

I just noticed that when I set infinite_inertia = false than it works.

This is by design. As per the documentation (for move_and_slide()):

If infinite_inertia is true, body will be able to push RigidBody2D nodes, but it won’t also detect any collisions with them. If false, it will interact with RigidBody2D nodes like with StaticBody2D.


What procedures do we need for adding or modifying Pythia datasets?

We should enumerate our procedures for how to add new datasets to this repository (or elsewhere).  In particular, related to infrastructure, we should take a close look at whether these procedures can be automated or made "push button" in any way.

Not sure if it helps, but we've been experimenting with this in Fatiando here: https://github.com/fatiando/data (with data guidelines here https://github.com/fatiando/data/blob/main/CONTRIBUTING.md)
The types of data we use often need to be heavily edited and standardized since there isn't really a widely used format. So we have the data repository for curating and processing. The distribution is then handled by a different package that pulls from the curated data: https://www.fatiando.org/ensaio

Thanks, @leouieda!  Nice to see you here in Project Pythia!  😄  This is good information.  Thanks for sharing it.

As part of triaging today, we revisited this. It was determined that our actions over the past two years have led in the direction of pythia-datasets not being intended for public use and instead just as a tool to serve data specifically in Foundations. We instead advocate for Cookbook contributors to rely on publicly available and accessible datasets or very small per-cookbook data subsets as needed.
It's not been an issue, but it might be worth codifying this sentiment in the README.


Update Navigationbar Item

This pull request allow you to modify your navbar item title and button witout replace all your view

Any chance to get this merged? It's unchanged in over a month...

Well I worked on it last month i will update it in few days

move to this PR
https://github.com/facebook/react-native/pull/2526


Stream button covering other buttons or spotify pause button

These are the problems that I have with the theme but overall its really nice.
Edit: Typo lol

The stream button will be displayed here now

Thanks for the fix though it covers the game activity toggle plugin that not everyone use. Can I ask how to move it a bit to the left?

 For the custom status overlap, this should be fixed as of now. For the BetterDiscord version, DevilBro did not add something to know whether you have this plugins or not, not even to differentiate it from the other user buttons.

Update : After I talked to DevilBro, he's finally adding a class for it, and I'm for now using a messy but working way, thanks to his support. Since the problem is fixed, I'm closing this issue.


Broken Documentation links

The 3 documentation links in the readme file are not working (404 errors)
Official Agents Documentation - https://aka.ms/AgentsFramework
.NET Documentation - https://aka.ms/Agents-net-docs
JavaScript Documentation - https://aka.ms/agents-js-docs

Thanks @dacmcbibs! I've just updated the links + will update the placeholders once they become available


Disable low speed roll

Set roll angle and roll rate to zero below a critical speed of 1 m/s.
Fixes #169
This cannot be tested until #208 is fixed.

logs.zip
test0.pb.cobs: rider stars at zero velocity
test1.pb.cobs: rider starts at non-zero velocity

From looking at the test0:

velocity is not being logged
handlebar inertia should probably use the true encoder angle and rate instead of steer angle and rate from the bicycle observer
roll rate quick grows away from zero at 5 seconds

velocity is not being logged

It is now - ee74639, b60aba6

handlebar inertia should probably use the true encoder angle and rate instead of steer angle and rate from the bicycle observer

Appears better after fixing the inertia value -  7487f7e

roll rate quick grows away from zero at 5 seconds

Likely related to incorrect inertia torque calculation

There is some motor "jerkiness" when we reach the drive speed limit due to saturation:

Tested yesterday. Need to decide if this should be merged or https://github.com/oliverlee/phobos/pull/224.


[Suggestion] Rooty dirt quark grass color compatibility

When using quark, the rooty dirt has the vanilla grass color.
https://imgur.com/a/ZFSeK

It has the wrong color because quark broke it with it's "greener grass" option.  Turn the option off or deal with the color being wrong.  Not my problem.

Quark is hugely popular and has some neat features.  I'm relenting on my decision not to support it.  I'll see what it takes to fix this minor issue.

Quark greener grass compatibility has been added in the latest code push.


Update investor todo list

I think we need a migration that gets and updates the user data because some bullet points are removed from the TODO list

@bart-at-mobicage added migration


Security: Security test plan based on new security consideration

S.No
Use Case
Results
Comments/Defects

1
Flood   ICMP type 8 packets towards BMC from multiple IPs, check BMC behavior

ICMP   flooding, DoS attack

2
Send   ICMP type 18 message netmask request

Security   feature

3
Send   ICMP type 13 timestamp request

Security feature

4
Inititate   POST method without Auth Token

5
Inititate   PUT method without Auth Token

6
Inititate   DELETE method without Auth Token

Stability   of authentication module

7
Flood   POST method without Auth Token

8
Flood   PUT method without Auth Token

9
Flood   DELETE method without Auth Token

Stability   of authentication module

10
Make   10000 wrong login attempts to REST inetrface, in a fast interval without even   waiting for response

11
Make   10000 wrong login attempts to Redfish sessions, in a fast interval without   even waiting for response

12
Make   10000 wrong login attempts to SSH, in a fast interval without even waiting   for response

13
Make   10000 wrong login attempts to IPMI, in a fast interval without even waiting   for response

Reducing   attack surface

14
Check   enabling and disabling feature, when service is disabled its equivalent port   should be closed

15
Try   enabling and disabling feature 100 times, when service is disabled its   equivalent port should be closed

16
Create   max allowed REST connection sessions, check these sessons are auto cleaned   after timeout value if there is no traffic

Security   feature

17
Create   max allowed Redfish connection sessions, check these sessons are auto cleaned   after timeout value if there is no traffic

18
Create   max allowed GUI sessions, check these sessons are auto cleaned after timeout   value if there is no traffic

19
Create   max allowed SSH sessions, check these sessons are auto cleaned after timeout   value if there is no traffic

20
Create   max allowed connection sessions, check if throws an error on new connection   request

21
Create   max allowed REST connection sessions, keep trying establishing sessions

22
Create   max allowed Redfish connection sessions, keep trying establishing sessions

23
Create   max allowed GUI connection sessions, keep trying establishing sessions

24
Create   max allowed SSH connection sessions, keep trying establishing sessions

25
Create   max allowed connection sessions, check if throws an error on new connection   request

page   refresh will consume lot of resources from server and lead to DoS

26
Open   http connections, keep refreshing the home page

27
Send   continous traffic towards redfish port 443

28
Send   continous traffic towards IPMI port 623

29
Send   continous traffic towards SSH port 22

30
Send   continous traffic towards SOL port 2200

31
Make   10000 wrong login attempts to Rest when LDAP enabled

32
Make   10000 wrong login attempts to GUI when LDAP enabled

Derived from @joseph-reynolds  https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/22106

This looks like a good start.
I wonder if we can actually do all of the "flooding / DoS" tests.  We may need multiple systems attacking the BMC to cause problems.  But, I think that having a single system attacking the BMC by accident is a foreesable situation, so I am okay with limiting the test cases to those scenarios.
We have existing test cases to check SSH ciphers.  Seee https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-test-automation/+/21137.  We should also specify test cases to check which HTTPS and IPMI RAKP ciphers are offered.  I've added a reminder to add this to the Network Threat Model doc - https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/22106 (patch set 6 line 77).

@prkatti1 can u look at @joseph-reynolds  input there above. .Thanks


ustack is not readable

What reproduces the bug?
hello,  I have a question of bpftrace:
 bpftrace -e 'uprobe:./log-server:io.copyBuffer /comm == "log-server"/ { printf("%s %d %s %s\n", comm, pid,ustack,kstack); }' 2>&1
Attaching 1 probe...
log-server 55235
        0x4b5f00
        0x6f8d05
        0x6e274d
        0x9b1139
        0xb19279
        0xb12345
        0xb13706
        0xb37079
        0xb366ce
        0xb363fd

the above stack is not readable ，how to change the command so that the stack is readable.
bpftrace --info
System
  OS: Linux 4.18.0-147.mt20200626.413.el8_1.x86_64 #1 SMP Fri Jun 26 10:27:19 CST 2020
  Arch: x86_64

Build
  version: v0.13.0
  LLVM: 12.0.0
  ORC: v2
  foreach_sym: yes
  unsafe uprobe: no
  bfd: yes
  bpf_attach_kfunc: yes
  bcc_usdt_addsem: yes
  bcc bpf_attach_uprobe refcount: yes
  libbpf: yes
  libbpf btf dump: yes
  libbpf btf dump type decl: yes

Kernel helpers
  probe_read: yes
  probe_read_str: yes
  probe_read_user: no
  probe_read_user_str: no
  probe_read_kernel: no
  probe_read_kernel_str: no
  get_current_cgroup_id: yes
  send_signal: no
  override_return: no
  get_boot_ns: no
  dpath: no

Kernel features
  Instruction limit: 131072
  Loop support: no
  btf (depends on Build:libbpf): no
  map batch (depends on Build:libbpf): no
  uprobe refcount (depends on Build:bcc bpf_attach_uprobe refcount): no

Map types
  hash: yes
  percpu hash: yes
  array: yes
  percpu array: yes
  stack_trace: yes
  perf_event_array: yes

Probe types
  kprobe: yes
  tracepoint: yes
  perf_event: yes
  kfunc: no
  iter:task: no
  iter:task_file: no

Do you have debug symbols voor log-server?

Do you have debug symbols voor log-server?

Yes， by using bpftrace -l 'uprobe:log-server:*' ,  many symbols are listed in the screen

The output seems a bit strange. You're printing ustack and kstack but what we see on the top seems like only the ustack. Did you crop the output?

This looks like a duplicate of #2284. Consider two programs, gotest.go:
package main

import (
    "fmt"
    "io"
    "os"
    "strings"
)

func main() {
    src := strings.NewReader("GfG\n")
    dst := os.Stdout
    buffer := make([]byte, 1)
    bytes, err := io.CopyBuffer(dst, src, buffer)
    if err != nil {
        panic(err)
    }
    fmt.Printf("The number of bytes are: %d\n", bytes)
}

and gotest-sleep.go:
package main

import (
    "fmt"
    "io"
    "os"
    "strings"
    "time"
)

func main() {
    src := strings.NewReader("GfG\n")
    dst := os.Stdout
    buffer := make([]byte, 1)
    bytes, err := io.CopyBuffer(dst, src, buffer)
    if err != nil {
        panic(err)
    }
    fmt.Printf("The number of bytes are: %d\n", bytes)
    time.Sleep(1 * time.Second)
}

With these, I get:
# bpftrace -e 'uprobe:/tmp/gotest/gotest:io.copyBuffer { print(func);  }'
Attaching 1 probe...
0x473980
# bpftrace -e 'uprobe:/tmp/gotest/gotest-sleep:io.copyBuffer { print(func);  }'
Attaching 1 probe...
io.copyBuffer

Merging #2386 should fix your issue if either ASLR is disabled or log-server is running at the time of attaching the probe and BPFTRACE_CACHE_USER_SYMBOLS environmental variable is set to PER_PID.

ping @bitzhuxb, could you give it a try with #2386?

Closing due to inactivity. Feel free to re-open if this is still a problem.


Podinfo test PR

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

test

rejected

closing


gardenlet: invalid memory address or nil pointer dereference

/area quality
/kind bug
/priority 3
What happened:
time="2021-07-20T09:56:18Z" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=garden-it/tm9gf-vdv task="Waiting until shoot worker nodes have been reconciled"
time="2021-07-20T09:56:18Z" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=garden-it/tm9gf-vdv task="Waiting until vpn-shoot LoadBalancer is ready"
time="2021-07-20T09:56:18Z" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=garden-it/tma99-d7p task="Waiting until shoot worker nodes have been reconciled"
time="2021-07-20T09:56:18Z" level=info msg=Succeeded flow="Shoot cluster reconciliation" operation=reconcile shoot=garden-it/tma99-d7p task="Waiting until vpn-shoot LoadBalancer is ready"
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1f346fd]

goroutine 386650 [running]:
github.com/gardener/gardener/pkg/operation/botanist.(*Botanist).SetNginxIngressAddress(0xc01d8b0a50, 0xc02f3ed860, 0x47, 0x286f800, 0xc00093d5a0)
	/go/src/github.com/gardener/gardener/pkg/operation/botanist/addons.go:203 +0x7d
github.com/gardener/gardener/pkg/operation/botanist.(*Botanist).WaitUntilNginxIngressServiceIsReady(0xc01d8b0a50, 0x2854958, 0xc00005a030, 0xf4d8885, 0xc003f76d00)
	/go/src/github.com/gardener/gardener/pkg/operation/botanist/waiter.go:41 +0x156
github.com/gardener/gardener/pkg/utils/flow.(*execution).runNode.func1(0xc02646d730, 0x258649d, 0x31, 0x2854958, 0xc00005a030)
	/go/src/github.com/gardener/gardener/pkg/utils/flow/flow.go:223 +0x1cf
created by github.com/gardener/gardener/pkg/utils/flow.(*execution).runNode
	/go/src/github.com/gardener/gardener/pkg/utils/flow/flow.go:218 +0x14f

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Gardener version: v1.27.2, v1.28.0-dev-489e985691671faac0a73bc9a4a4cf62c2acbcc2
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Others:

Fixed by #4481
/close


Unable to get a list of jobs in a namespace

I want to retrieve a list of jobs so I can see what jobs are current running. With kubectil, I can:
kubectl get jobs --namespace dev
However, when I try the following, it fails:
api = pykube.HTTPClient(pykube.KubeConfig.from_file("/Users/Dave/.kube/config"))
jobs = pykube.Job.objects(api).filter(namespace="dev").get()
I get requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https:///apis/batch/v1/namespaces/dev/jobs
If I use:
kubectl get jobs --namespace=dev -v=8
Then I can see that kubectl is actually referencing /apis/extensions/v1beta1/namespaces/dev/jobs.
How can I get extensions/v1beta1 used instead of v1?

I was having this problem after checking the kubeconfig file, my code is the same:
        api = pykube.HTTPClient(pykube.KubeConfig.from_file("/home/balancin/.kube/config"))
        jobs = pykube.Job.objects(api).filter(namespace="balancin")

May you test your kubectl connection with direct configfile with this parameter in your command:
--kubeconfig=YOUR_CONFIG_PATH
You can debug the kubectl output too with this parameter:
--v=11
May you find something using direct context and cluster in your kubeconfig:
kubectl --v=11 get pods --namespace=balancin --cluster=default


Add static method Schema.from

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/impress/commit/9188762f4cc3664d9fd8c8d16d12ff1e80e4e2f3

Landed in https://github.com/metarhia/impress/commit/9188762f4cc3664d9fd8c8d16d12ff1e80e4e2f3


Disable editing in the modified window (right side) and jump to first diff in diff editor

Hi,
Is there an official way to disable editing on the modified window (right side) in the diff editor?  In the source, I can see you can disable/enable editing on the original (left side).  Right now, I'm do a cheap hack to remove editing in the modified window, and was wondering if there is a supported/proper way to do this.
Second question. Is there a way to jump/scroll to the first diff, in the diff editor?
Thanks

I think this is the way diffEditor.getModifiedEditor().updateOptions({readOnly:true})
Try it in the playground
var originalModel = monaco.editor.createModel("just some text\n\nHello World\n\nSome more text", "text/plain");
var modifiedModel = monaco.editor.createModel("just some Text\n\nHello World\n\nSome more changes", "text/plain");

var diffEditor = monaco.editor.createDiffEditor(document.getElementById("container"));
diffEditor.setModel({
	original: originalModel,
	modified: modifiedModel
});

diffEditor.getModifiedEditor().updateOptions({readOnly:true})

For the second question, did you see Navigating a Diff example ?

Thanks for the info. Didn't know about the playground and diff nav example.  I just used
https://github.com/Microsoft/monaco-editor-samples/blob/master/sample-diff-editor/index.html
as a starting point.  Would have save some time if I looked there first.
Cheers

You may want to use alwaysRevealFirst option to auto jump to the first diff:
var emptyLines = '\n'
for(var l=0;l<10;l++){
    emptyLines+=emptyLines
}

var originalModel = monaco.editor.createModel("just some text"+emptyLines+"Hello World"+emptyLines+"Some more text", "text/plain");
var modifiedModel = monaco.editor.createModel("just some text"+emptyLines+"hello World"+emptyLines+"some more text", "text/plain", "text/plain");

var diffEditor = monaco.editor.createDiffEditor(document.getElementById("container"));
diffEditor.setModel({
	original: originalModel,
	modified: modifiedModel
});

var navi = monaco.editor.createDiffNavigator(diffEditor, {
	followsCaret: true, // resets the navigator state when the user selects something in the editor
	ignoreCharChanges: true, // jump from line to line
    alwaysRevealFirst: true // jump to first diff
});

Thanks.  Everything works like a charm!
https://gitsense.com/insight?c=bitbucket:gitsense/contexts:gs_contexts::default.ccf#b=github:Microsoft/vscode:master&dr=1month&dl=javascript&dp=gulpfile.js
It's pretty crazy that right out of the box you get something this powerful.
Cheers


[Bug] Links created via ShareX endpoint have no author

To Reproduce
Steps to reproduce the behavior:

Use the sharex endpoint to create a link
The author is n/a

Expected behavior
The link is created with the author's username

Fixed in 7809b09


Fix duplicated entries in script editor menu

Hey guys i fixed the issue #43707, on 3.2 branch, it seems is already fixed on master.

That bug was from a merge conflict resolution gone wrong in #40461, CC @pouleyKetchoupp.
We be good to review both the regression commit and this fix to make sure that the final list of entries is correct.

@dakennedyd Could you amend the commit log to be more explicit about what it's doing? Outside GitHub, an issue number is not sufficient information on what a bug is/was. See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#format-your-commit-messages-with-readability-in-mind
A good commit message would be e.g.:
Fix duplicates entries in script editor menu

This was a regression from #40461.

Fixes #43707.

Thanks!

Thanks for the help guys :)


How do you backtest a custom strategy?

Hi!
First of all, awesome work!!
I'm struggling with backtesting a custom strategy.
I think I don't really understand how your backtesting works. I mean, how do you set your TP or SL for a strategy?
Let's say I want to test an RSI strategy, selling with the RSI over 70 and buying with de RSI under 30, and I want to try with a 0.5% profit point under (or over) the entry (order) point.
Is there a way to try this?
Thanks!

Hi!
I'm glad to hear that you found the project useful! I tried to produce a quick example based on your description on repl.it:
<EMAIL_ADDRESS>The current backtest doesn't support the profit points, but certainly would be nice to add that. I just need to understand how that works definitely.
The default RSI strategy rsi2Strategy uses sell over 90, and buy under 10, but a new one can be defined by the user easily:{
const myRsiStrategy: StrategyFunction = (asset: Asset): Action[] => {
  const indicator = customRsi(2, asset.closings);

  const actions = new Array<Action>(indicator.length);
  for (let i = 0; i < actions.length; i++) {
    if (indicator[i] < 30) {
      actions[i] = Action.BUY;
    } else if (indicator[i] > 70) {
      actions[i] = Action.SELL;
    } else {
      actions[i] = Action.HOLD;
    }
  }

  return actions;
};

Once you have that, you can easily use it with the backtest function:
const myRsiStrategyInfo: StrategyInfo = {
  name: "My RSI 70-30 Strategy",
  strategy: myRsiStrategy,
};

const results: StrategyResult[] = backtest(sp500, [myRsiStrategyInfo]);
console.table(results);

I have a working example at the repl.it link. Please take a look. If you can help me with how the profit margin will work there, I will be happy to add that here.
Best regards,
-onur

Hi! Thanks for the answer!!
Yep, I imagined the current approach do not consider the Take Profit points. It's currently not that hard, but I need to fully understand your code first.
I think the correct approach to Take Profit points should be something like, if the percentage you put as Take Profit it's inside the next 2 candles, then it'a a success, always keeping in mind that the TP point should be implemented with a SL point too.
For example, something like this:
SL: 5% (configurable and optional)
TP: 1% (configurable and optional)
If the next 1- 2 candles contains the TP value and it doesnt contains the SL value, then is a success by the percentage value.
If it contains the SL, then it's a loss equivalent to the SL percentage value.

This way you can actually backtest a complete strategy, giving more or less space to SL and TP points to reach the sweet spot.


0.42.0: Style/ParallelAssignment autocorrection can break code

Style/ParallelAssignment autocorrection can break code if there is a dependency between assigned objects.
Test case
class Range
  def initialize(first, length)
    @first = first
    @length = length
  end

  def first
    @first
  end

  def first= value
    @first = value
  end

  def last
    @first + @length
  end

  def last= value
    @length += value - last
  end

  def normalize_parallel! # correct way
    if @length < 0
      self.first, self.last = self.last, self.first
    end
  end

  def normalize_series! # incorrect way
    if @length < 0
      self.first = self.last
      self.last = self.first
    end
  end

  def to_s
    "Range[#{first}..#{last}]"
  end
end

a = Range.new(10, -5)
puts "a1 = #{a}" # Range[10..5]
a.normalize_parallel! # correct way
puts "a2 = #{a}" # Range[5..10] (correct)

b = Range.new(10, -5)
puts "b1 = #{b}" # Range[10..5]
b.normalize_series! # incorrect way
puts "b2 = #{b}" # Range[5..5] (incorrect)

Expected behavior
normalize_parallel! remains unchanged.
Actual behavior
normalize_parallel! gets rewritten to the same as normalize_series!.
Steps to reproduce the problem

Put test case in a test.rb file
Run rubocop -a test.rb

RuboCop version
0.42.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-darwin14)

This is a more general case of #3286.


optimize getTXByHash

currently we will retry 20 times, because we need to wait for next block to be able to find it, in this case we will need to wait 20 seconds if tx is not found.
Can think about ways to optimize this, maybe we can cache those "sent but not mined TX" in rpc-adaptor

you can check the tx pool to see if it is there or not and if is not in txpool nor recently mined block, it is not there

that sounds like a good idea, by saying txpool do you mean pending extrinsics?

yes


Add model references to Daikin 1100 codes (FTXSBVMB and FTXSCVMB)

Add missing model names / references to codes 1100.

How do you convert a code from Broadlink to Xiaomi format?


Default setting of llama.cpp and settings required for go-llama

Getting oomed. not sure what other settings i need to do:
llama.cpp that works:
./main -m models/speechless-llama2-13b.Q3_K_S.gguf -n 3400 -c 1920 -p "write a blog about llama:" -e --temp 0.3 --mirostat 2
oomed reference:
https://github.com/go-skynet/go-llama.cpp/discussions/237
sorry asking here because waited for days without reply.

@mudler @deadprogram @mauromorales @jrc2139 @soleblaze
any help here? sorry been waiting for days over this.


modify extractAltFee to be smarter

see code comments for motivation behind this work
additionally references https://github.com/penumbra-zone/web/issues/1569 bug

still need to add changeset, bump idb version, and reach consensus on https://github.com/penumbra-zone/web/pull/1577#discussion_r1696602433.

still need to add changeset, bump idb version, and reach consensus on https://github.com/penumbra-zone/web/pull/1577#discussion_r1696602433.

high-level flow:
1. Fetch all alternative gas prices from the GasPrices table.
2. Extract assetId from the transaction request and check that:

The balance for the asset is positive.
The asset is a valid fee token.

3. If the above checks fail, fall back to using another asset from the GasPrices table.


Allow semantic tokens for strings to be disabled

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/7111
Pretty straightforward change, but open to any suggestions if there's a more recommended testing strategy than what I went with.

bors r+
Thanks! I'll adjust config name in a follow-up PR

It feels like this is a weird layer to implement and test it, but..

The layer I think is OK -- this is a work-around for the client impl, so it belongs to the lsp layer, and not to the ide.
Agree that it doesn't make sense to spend an integration test here, I'll adjust that as well (don't want to make the PR enter a round of reviers, it's been waiting way too long already)

bors r+


Sky tests

I strongly recommend you to get familiar with https://www.python.org/dev/peps/pep-0008/ - a style guide for python code (especially import section) :)


Add Milestone.StartDate to Issue

Following #94, Milestone.StartDate was missing from Issue struct.

Thanks again for the PR!


Added phantomsObeyHostileMobCap rule

Phantoms will no longer spawn around a player if their hostile mobcap is full.
Closes #2

Just realised the ChunkManagerHelper class is probably not needed now. That was required for the lunaar port.

Just realised the ChunkManagerHelper class is probably not needed now. That was required for the lunaar port.

Nevermind, only some method usages are no longer needed.

Just realised the ChunkManagerHelper class is probably not needed now. That was required for the lunaar port.

Nevermind, only some method usages are no longer needed.

Removed.


I cannot execute the game when I have the dll in the bin-64 folder

As mentioned in the title, I have downloaded the files and copied the d3d9.dll to the bin64 folder. After doing that I tried to run the game and I get an error message and the game does not start. Am I doing something wrong?
The error message is huge but if it is necessary I can copy and paste it here. I would like to note that I don't have any other addons.
The part that reffers to the crash is this one.
--> Crash <--
Assertion: Failed to load Coherent DLL: 0x7e
File: ......\Engine\ArenaWeb\CoherentUi\AwCoh.cpp(50)
App: Gw2-64.exe
Pid: 12588
Cmdline:
BaseAddr: 00007FF77B000000
ProgramId: 101
Build: 85056
When: 2017-12-21T21:34:31Z 2017-12-21T22:34:31+01:00
Uptime:   0 days  0:00:03
Flags: 0
Thanks

Here the same error. Is there any solution yet?

See the Q&A answer regarding the "Coherent DLL". I can do nothing more for this particular problem, unfortunately.

--> Crash <--
Assertion: Failed to load file.
File: ......\Services\LoadHandle\LoadHandle.cpp(160)
App: Gw2-64.exe
Pid: 224
Cmdline: -allowInstall
BaseAddr: 00007FF628010000
ProgramId: 101
Build: 90061
When: 2018-06-27T15:06:25Z 2018-06-28T01:06:25+10:00
Uptime:   0 days  0:00:01
Flags: 0


Adding title and aria-label attributes to step buttons.

This PR adds aria-label and title attribute parameters to the steps' previous and next buttons. It also adds title and aria-label attributes to the tab list for steps.

@enchev, this PR uses AriaLabel and not Label.


telemetry missing if project_name is set

Hi people,
I maintain a Rails+sidekiq project where I got a PR that adds opentelemetry support: https://github.com/voxpupuli/vox-pupuli-tasks/pull/272. The following config works fine:
# config/application.rb
OpenTelemetry::SDK.configure do |c|
  require 'pg'
  c.use 'OpenTelemetry::Instrumentation::PG'
  c.use 'OpenTelemetry::Instrumentation::Rails'
  c.use 'OpenTelemetry::Instrumentation::Sidekiq'

  c.add_span_processor(
    OpenTelemetry::SDK::Trace::Export::BatchSpanProcessor.new(
      OpenTelemetry::Exporter::Jaeger::AgentExporter.new(host: '<IP_ADDRESS>', port: 6831)
    )
  )
end

This works fine. jaeger is running from docker with the following docker-compose file:
  jaeger:
    image: jaegertracing/all-in-one:1.22
    container_name: jaeger
    ports:
      - "5775:5775/udp"
      - "5778:5778"
      - "6831:6831/udp"
      - "6832:6832/udp"
      - "9411:9411"
      - "14268:14268"
      - "14250:14250"
      - "16686:16686"
    environment:
      COLLECTOR_ZIPKIN_HOST_PORT: ":9411"

In the jaeger UI, my rails app is listed as unknown_service:

I also verified that data is coming with tcpdump -i lo port 6831

Now I saw in your README.md that I can set a version/service name:
OpenTelemetry::SDK.configure do |c|
  require 'pg'
  c.use 'OpenTelemetry::Instrumentation::PG'
  c.use 'OpenTelemetry::Instrumentation::Rails'
  c.use 'OpenTelemetry::Instrumentation::Sidekiq'

  c.add_span_processor(
    OpenTelemetry::SDK::Trace::Export::BatchSpanProcessor.new(
      OpenTelemetry::Exporter::Jaeger::AgentExporter.new(host: '<IP_ADDRESS>', port: 6831)
    )
  )
  c.service_name = 'vox-pupuli-tasks'
  c.service_version = VERSION
end

If I set service_name and/or service_version and restart the app + jaeger, the service doesn't appear in the UI. I did the tcpdump again and didn't saw any packets. Please let me know if you've any clue/need more information.

@bastelfreak Hey there, thanks for reaching out on this. I don't have the jaeger ui running but quickly tried to recreate this using the Jaeger::CollectorExporter, but was unable to. See below screenshots, the service name is correctly exported.
I'lll try to carve out some time to see if I can reproduce with the AgentExporter although the logic of the Jaeger Encoder ought to be the same. Will update here when I have a chance to do so, should be able to in the next few days.
In the interim, one other idea I wanted to suggest was the use of the OTEL_RESOURCE_ATTRIBUTES environment variable to control service.name and service.version resource attributes (docs): https://github.com/open-telemetry/opentelemetry-ruby/blob/a9bcbda1ffca0e781e174da7bdaeb1a83394c595/sdk/lib/opentelemetry/sdk/resources/resource.rb#L44
If you try setting the env var like OTEL_RESOURCE_ATTRIBUTES=service.name=vox-pupuli-tasks,service.version=<VERSION>, does that resolve the issue for you? Just trying to narrow down why this may not be working as expected for you.
Config:
Output:

Just as a note: seeing similar problems. Rack and ActiveModelSerializers instrumentation seem to be fine,Rails is missing sofar.

Closing this issue as we were not able to reproduce this.  If some one can confirm this is still an issue and provide a reproduction we can re open the issue and investigate further.


Investigate the usefulness of Profile-guided optimization

https://go.dev/doc/pgo
https://go.dev/blog/pgo

Thinking we'd do a profile for the Go library by itself, and another profile for the Terraform Provider.
Although profiling tests runs is not likely to be a good long-term source of profiling data for this, it's a cheap-and-easy place to start.

Implemented using tests for the profiling.


(GH-131) Add Get-BTHeader function

Fixes #131
This commit adds the Get-BTHeader function which is used to query, and filter,
existing active and scheduled toasts for their unique headers. This is useful
for managing state across PowerShell scripts with toasts using the same Header.
This commit also includes tests and help text for the new function and any
private helper functions.

 This is awesome.


replace tags with distinct values

i'm openning a new pr to replace distinct tags with your respective values, because i was expressed myself baddly at another pr.

More details is required...
Input, output, specific, etc.

This feature was already supported by this update.
Therefore, it seems that this PR is meaningless...
And also, this could not be merged because it has many problems. See https://github.com/anseki/pre-proc/pull/1#issuecomment-309384174

And also, that method in this PR may return invalid value.
Anyways, thank you for the your contribution 😄


[ECS] Remove remote docs directory to avoid piling up deleted gif in remote repository

Ref: https://github.com/symplify/symplify/pull/3955

Thank you @zingimmick

:+1:

Thank you @zingimmick

There still seems to be a problem, the remote directory is deleted, not overwritten.

@zingimmick could you verify? probably need to copy back the meta files after downgraded, ref
https://github.com/rectorphp/rector-src/blob/f7e1035828745f5cb95634709f7970a9a246ca77/.github/workflows/build_scoped_rector.yaml#L87-L92

@samsonasik  This problem is caused by the absence of the docs directory in the target-repository directory.

copy back the meta files after downgraded

I'll try it.


[WIP] Sync Build CRD Creation

Update the operator to sync the CRD spec for config.Build (not the actual cluster instance - just the CRD definition).

/assign @deads2k
/cc @bparees

@smarterclayton imho this is a ridiculous amount of code to replace "drop buildcrd yaml file into manifests dir, let the CVO sort it out".
Do we really think this is necessary?  If so, why?

I don’t see value in syncing CRDs in the operator vs having cvo sync it unless you are trying to be clever with CRDs which is probably bad.
For instance, I recently removed cleverness in CVO that merges CRDs.  CVO reconciles CRDs every few minutes

@bparees updated, assuming CVO will take care of the rest.

/lgtm
Thanks, please go back and remove it from openshift/api so there's no confusion.

@bparees the YAML for the conifg.Build CRD was never merged to master in api. I closed the pull which added it (see comment ref above).

/retest

ah, cool.


can't install FAR

The install option in the SKIM is greyed out, at the bottom saying the game is not installed.
If I extract from the FAR zip file (not sure which version I should use here) the game run as normal without the yellow text, pressing ctrl+shift+backspace doesnt bring the FAR menu either. but after playing for a while, I'll get the MGS sound and the game crash

The SKIM install the original mod, not the cracked.
https://github.com/marcussacana/FAR/releases/download/far_070/FAR_0_7_0_14.7z
this is the lastest version.
Try SELECT + START from the joystick or CTRL + ALT + SHIFT + BACKSPACE

thnx for your attention. I got to change the resolution to properly fit my monitor, so i guess that's something... still can't show the FAR menu tho. from what i heard there's some settings I should change in there that'll make the game run smoother and more stable. as of right now, I still get the occasional crashes with the MGS alert sound

If you hear the MGS alert the FAR is already installed and running, you can manually setup at dinput8.ini

sorry, 1 more thing. can you show me where do i edit in dinput8 to change global illumiation quality, under lighting?

Edit at FAR.ini


Shortcuts for date widget

Widget is hard to use, lets add
Last 30days
Last 90 days
Last 6 months
Last 2 yrs

We should use the click-ui widget.


-y option no longer valid in homebrew-cask?

Not sure what is wrong here, but something changed either in Homebrew or Homebrew-cask that made it so the -y option isnot accepted in brew cask anymore. I don't know if this is a bug or intentional yet and no one has raised an issue on either github, so I am writting in a || to try and error trap a bit for both cases.

Anyone with a better grasp on Homebrew know why this is now failing to find the tlmgr from basictex?
+brew cask install basictex
==> Downloading http://mirror.ctan.org/systems/mac/mactex/mactex-basictex-20170523.pkg
==> Verifying checksum for Cask basictex
==> Installing Cask basictex
==> Running installer for basictex; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are ignored.
==> installer: Package name is BasicTeX-2017
==> installer: Installing at base path /
==> installer: The install was successful.
🍺  basictex was successfully installed!
+export PATH=/usr/texbin:/Users/travis/anaconda/bin:/Users/travis/.rvm/gems/ruby-2.0.0-p643/bin:/Users/travis/.rvm/gems/ruby-2.0.0-p643@global/bin:/Users/travis/.rvm/rubies/ruby-2.0.0-p643/bin:/Users/travis/.rvm/bin:/Users/travis/bin:/Users/travis/.local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/bin
+PATH=/usr/texbin:/Users/travis/anaconda/bin:/Users/travis/.rvm/gems/ruby-2.0.0-p643/bin:/Users/travis/.rvm/gems/ruby-2.0.0-p643@global/bin:/Users/travis/.rvm/rubies/ruby-2.0.0-p643/bin:/Users/travis/.rvm/bin:/Users/travis/bin:/Users/travis/.local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/bin
+sudo tlmgr update --self
sudo: tlmgr: command not found

The problem is that it was working 4 days ago, and the basictex recipe did not change. I think it is actually a homebrew or homebrew-cask change, but I've got an idea to work around it.

Okay, so it looks like this will go through finally. It feels really hack-y, but I don't see an easier fix. Does this work-around look like its a decent solution? It should still work as the basictex version gets updated from year to year, unless they change the install path, or homebrew(-cask) does something else to break what we had working.

Failing due to time outs of builds, but the problem this PR fixes is addressed. I'm going to merge this to get the builds running again, we can fix anything I broke at a later time


Use images to illustrate block

We need to add possibility to use images to illustrate a block (not just icon library).
An icon library is not sufficient to illustrate different types of blocks.
I think we should offer the possibility to upload an image (preferably SVG) to use as an illustration.

+1 here, I have different kind of text block, choosing 5 differents icons to illustrate them is nonsense whereas a simple screenshot would be easier to understand

Woops, i forgot to close this Issue. It's actually possible by using the icon_path option in your block:
'icon_path' => 'https://yoursite.com/img/icon.png', // custom icon


Feat/graphql faucet gateway

This PR switches Faucet gateway from REST to GraphQL protocol, following specifications in https://github.com/okp4/ui/issues/248

Codecov Report

Merging #259 (9d8143f) into main (a366ba9) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #259      +/-   ##
==========================================
+ Coverage   97.54%   97.57%   +0.02%
==========================================
  Files          54       55       +1
  Lines         529      535       +6
  Branches       73       74       +1
==========================================
+ Hits          516      522       +6
  Misses         12       12
  Partials        1        1

Impacted Files
Coverage Δ

...ters/faucet/secondary/graphql/HTTPFaucetGateway.ts
100.00% <100.00%> (ø)

.../faucet/secondary/graphql/InMemoryFaucetGateway.ts
100.00% <100.00%> (ø)

src/adapters/faucet/secondary/graphql/client.ts
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

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


Type hints are not validation

Might be good to validate incoming payloads, so that they don't break the code in weird ways if they don't conform to what we expect.

Would running a static analyser during the submit process solve this?

I was thinking more along the lines of a ser/de and validation library like marshmallow or pydantic. Especially leaning towards the latter, since it works with data classes and type hints.

I just ran into this. A static analyser is indeed insufficient. We should probably also make the error make it clear that this is caused by an issue with the template

Yeah, it's rather annoying. The service can also do a bit more validation before calling the handler, too. But it would be good to have in any case, e.g. if a payload structure is changed in future. There's a few other things on my plate, but I've made progress using pydantic on this.


Add a handler for requeuing jobs

Currently it's possible to requeue the same job when it's already encoding.
It's not likely you'll run into it unless two timelines share the same file or the same timeline is requeued while it's still processing for some reason (maybe found new files or the first attempt exited early or something).
Anyway, the current behaviour is a little weird. I believe Proxima will ask you if you want to link the existing file even though it's not finished. Of course it will fail and you'll be asked if you want to requeue it. This assumes that the file is corrupt and needs re-encoding. Not that the file may already be encoding. We can check all Proxima jobs and see if any share the same source path with ones we'd like to queue.

Looks like it'll be a little complicated to use Celery event snapshots.
Another method would be to watch all existing paths asynchronously in another process while running the other handlers.
If any have grown, we remove them from the media list. Adding async this late in the game might be pretty messy though.


Put TL;DR command into a shell script file

This gets harder to copy without error as more edge cases need to be covered. Put it in a bootstrap script and change the readme to call that instead.

Done.


Remove redundancy in find teammates page

After selecting few skills for finding teammates, there is redundancy in teammates.

There should not be repitition in results . Also if there are same teammate under mentee and mentor then also only one should be shown.

I want to work on this issue under GSSoC22.

I want to work on this issue under GSSoC22.

I want to work on this issue under GSSoC22.

I want to work on this issue under GSSoC22.

Assigned @aashay-03


✨ [Feature request]: Build dashboard for react-play

Is your feature request related to a problem? Please describe.
Currently, there is no dashboard that shows the statistics of activity across react-play website vise as well individual play vise.
Describe the solution you'd like
As part of this new feature, we will be introducing a dashboard that gives statistics on activities

can you please assign me to this issue

I believe #218 is related to the same.

I believe ✨ [Feature request]: Play statistics: View Count #218 is related to the same.

Nah.. they are different..
I think, this issue is going to be like a creator's profile where the other one will show a view count per play in the play list page. @shrilakshmishastry What you think?

Seems like it's a part of what we are building now. We have currently planned to build a dashboard at individual play level, react-play level view statistics and GitHub activities
Wireframe of same is below

Ok @shrilakshmishastry I misunderstood then!
One question... How would user get into this dashboard? What will be the user journey?

@shrilakshmishastry & @vasantisuthar, if you need a hand with this issue, let me know!


mb 4704 request reweigh

Description
This PR adds a GHC API endpoint to allow a TOO to request a reweigh on a shipment.
Some assumptions I made for the validations:

We should only allow a reweigh request if the shipment is either Approved or Diversion Requested.
A reweigh request is denied if a reweigh already exists for this shipment

Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?
Setup
Add any steps or code to run in this section to help others prepare to run your code:
echo "Code goes here"

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
this article explains more about the approach used.

Screenshots
If this PR makes visible UI changes, an image of the finished UI can help reviewers and casual
observers understand the context of the changes. A before image is optional and
can be included at the submitter's discretion.
Consider using an animated image to show an entire workflow instead of using multiple images. You may want to use GIPHY CAPTURE for this! 📸
Please frame screenshots to show enough useful context but also highlight the affected regions.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 37c55fd865307312be855bc7a20c1aaa0d39f549

Thanks for the thorough review! I think I addressed everything.


docs: Readme tweaks :)

Btw, I'd not put everything under details. It makes you expand everything and isn't searchable, and you also cannot link to any section if you want to direct someone on chat.


TVP varchar(max) fails with ISQLServerDataRecord, works with SQLServerDataTable

Driver version
7.1.1.jre10-preview AND 6.2.2.jre8 (assuming all other versions too)
SQL Server version
Microsoft SQL Server 2014 (SP2-CU11) (KB4077063) - 12.0.5579.0 (X64)
Feb 21 2018 12:19:47
Copyright (c) Microsoft Corporation
Developer Edition (64-bit) on Windows NT 6.3  (Build 9600: )
Client Operating System
Ubuntu 18.04
JAVA/JVM version
10.0.2+13-1ubuntu<IP_ADDRESS>
Table schema
create type data.DgTestType AS table (
  fieldA VARCHAR(MAX) NULL
)
grant exec on type :: data.DgTestType to public

create procedure data.SelFromType @tbl data.DgTestType READONLY
AS
BEGIN
  SELECT * FROM @tbl
end

Problem description
Error thrown when trying to use TVP, with string column > 4000 characters.  Using SQLServerDataTable works, but ISQLServerDataRecord fails.  The code snippet below shows the usages.  This relates to #250
Error message is The incoming tabular data stream (TDS) remote procedure call (RPC) protocol stream is incorrect. Table-valued parameter 0 (""), row 1, column 1: Data type 0xE7 has an invalid data length or metadata length.
Reproduction code
Code is in Kotlin, but is nearly identical to java version.
fun main(args: Array<String>) {
    val ds = devCustCustodyData()

    val paramString = "abcd".repeat(1001)

    val table = SQLServerDataTable()
    table.addColumnMetadata("fieldA", Types.LONGVARCHAR)
    table.addRow(paramString)
    table.tvpName = "data.DgTestType"

    ds.connection.use { conn ->
        conn.prepareStatement("exec data.SelFromType ?").use { pstmt ->
            pstmt.unwrap(SQLServerPreparedStatement::class.java).use { msStmt ->
                msStmt.setStructured(1, "data.DgTestType", MyRow(listOf(paramString)))  // <---- ISQLServerDataRecord fails
                //msStmt.setStructured(1, "data.DgTestType", table)  // <---- SQLServerDataTable works
                msStmt.executeQuery().use { rs ->
                    while (rs.next())
                        println(rs.getString(1).length)
                }
            }
        }
    }

}

class MyRow(values: List<String>) : ISQLServerDataRecord {
    private val iterator = values.iterator()
    private var current: String = ""

    override fun next(): Boolean {
        if (iterator.hasNext()) {
            current = iterator.next()
            return true
        }
        return false
    }

    override fun getRowData(): Array<Any> {
        return arrayOf(current)
    }

    override fun getColumnMetaData(i: Int): SQLServerMetaData {
        if (i == 1)
            return SQLServerMetaData("fieldA", Types.LONGVARCHAR)
        throw IllegalArgumentException("unexpected column index $i")
    }

    override fun getColumnCount(): Int = 1
}

fun devCustCustodyData(): HikariDataSource {
    val windowsUrl = """jdbc:sqlserver://dev-cust-db;database=CustodyData;integratedSecurity=true"""
    val linuxUrl = """jdbc:sqlserver://dev-cust-db;database=CustodyData;integratedSecurity=true;authenticationScheme=JavaKerberos"""
    val url = linuxUrl

    val config = HikariConfig()
    config.dataSourceClassName = "com.microsoft.sqlserver.jdbc.SQLServerDataSource"
    config.addDataSourceProperty("url", url)
    return HikariDataSource(config)
}

hi @dangets,
Thank you for providing the repro code we will look into this and update you on our findings.

For sure! I can rewrite in Java if it would be more helpful.

hi @dangets
I was able to make this work by specifying the precision of the column in SQLServerMetaData in getColumnMetaData(). In this case that is the length of the string in the row you are adding. In the case of SQLServerDataTable, the addRow() method fills in the precison for SQLServerMetaData when adding the string so getColumnMetaData() needs to do the same when constructing the SQLServerMetaData.
Please see https://static.javadoc.io/com.microsoft.sqlserver/mssql-jdbc/6.1.6.jre8-preview/com/microsoft/sqlserver/jdbc/SQLServerMetaData.html. constructor for SQLServerMetaData which also includes precision:
public SQLServerMetaData(String columnName,
                         int sqlType,
                         int precision,
                         int scale)

creates a new SQLServerMetaData

Parameters:
columnName - the name of the column
sqlType - the SQL type of the column
precision - the precision of the column
scale - the scale of the column

So in your example, if you change to:
return new SQLServerMetaData("fieldA", Types.LONGVARCHAR, getRowData().length , `0);

Then both methods would produce the same results.
Hope this helps, let me know if you have more questions.

The workaround does help, thank you.  It still seems like this could/should be something handled by the driver code itself though, and would be more friendly & consistent with the SQLServerDataTable approach.
Otherwise, I would ask that this edge case be documented somewhere like https://docs.microsoft.com/en-us/sql/connect/jdbc/using-table-valued-parameters?view=sql-server-2017 or https://docs.microsoft.com/en-us/sql/connect/jdbc/using-advanced-data-types?view=sql-server-2017 or

hi @dangets,
I agree it's a bit non-intuitive, we will look into improving and/or documenting this case. Thank you for your patience.

Hi @dangets ,
Thank you for bringing this issue to our attention.
We decided to add a new constructor to let the users specify the string length: SQLServerMetaData(String columnName, int sqlType, int length). This constructor should be used for string values of length greater than 4000.


Drop Exasol table from Spark

Is this supported?
I saw that there is an option to run general query, but it seems that it is limited to select statements.
I would like to be able to drop existing table and create new one from Spark.

Hey @nikoki,
Yes, it is possible in the context of saving the Spark dataframe. You can use the overwrite mode when saving the dataframe.
df
     .write
     .mode("overwrite")
     .option("host", "<IP_ADDRESS>")
     ...
     .option("table", "RETAIL.ADJUSTED_SALES")
     .format("exasol")
     .save()

However, if you want to run single SQL statements, you can use the regular JDBC commands from programming language. Unfortunately, it is not possible to run SQL statements from Spark Exasol connector.

Hi @morazow,
Thanks for the quick response!
I fond that if I try to override the existing Exasol table, like you suggested, sometimes it happens that the command hangs indefinitely. I suspect that happens if the DataFrame schema is different from the schema of the table already written to Exasol - but I am not 100% sure that is the cause.
To overcome the problem I wanted to drop the table before writing the new one and avoid the problem.
Do you have any thoughts?

Hello @nikoki,
Okay, I understand.
Again when saving it is possible to create table if it does not exist,
df
     .write
     .mode("overwrite")
     .option("host", "<IP_ADDRESS>")
     ...
     .option("create_table", "true")
     .option("table", "RETAIL.ADJUSTED_SALES")
     .format("exasol")
     .save()

but the table should be dropped separately outside of the connector. At the moment, it is not possible with the connector.
For the quick test, could you please drop the table using SQL client (DBVisualizer or DBeaver), and run the Spark dataframe save using the create_table option set to true?

That is how I use it right now :)
I was hoping I could do it using connection.
Thank you!

Great!
Then, I am going to keep this issue open. Adding this feature with an additional option should be feasible.
We will update here once it is included into the connector.
Thanks for your feedback!

Hey @morazow
Is there an ETA for this?
Thanks!

Hey @nikoki ,
Sorry, we could not address this issue yet.
We planning to work on it coming weeks.

Hello @nikoki,
This issue is automatically closed with pull request merge.
I wanted to inform you that the latest version, 0.3.1, includes the drop_table option when saving the Spark dataframe.
Please give it a try!
The maven central does not list connector yet (due to sync with sonatype). But you can start using it already by including it to dependency list.
"com.exasol" %% "spark-connector" % "0.3.1"


"ignore-comments" does not completely ignore commented lines

Because both Docker and just use Go-style templates, it is necessary to escape any occurrences of {{ to {{{{ in various recipes that need to use Docker's --format option.
I put a comment in the recipe with the same command without escaping the {{, with the intention of being able to easily copy the command from the justfile and paste it (in a "ready-to-go" state) in to an arbitrary shell for debugging, systems without just, etc.
For example:
set ignore-comments

ls-mounts:
    # docker inspect --format '{{range .Mounts}}{{if .Name}}{{.Name}}{{else}}{{.Source}}{{end}}{{"\n"}}{{end}}' $(docker ps --all --quiet) | grep -v '^$' | sort -u
    docker inspect --format '{{{{range .Mounts}}{{{{if .Name}}{{{{.Name}}{{{{else}}{{{{.Source}}{{{{end}}{{{{"\n"}}{{{{end}}' $(docker ps --all --quiet) | grep -v '^$' | sort -u

However, the {{ in the commented line causes just to fail — I have to escape them even though they are in the comment, which defeats my intended purpose.
Any suggestions?
P.S. Thanks for just!

Thanks for the issue! This is definitely annoying, but it would be pretty hard to solve, and might not be worth the complexity. Whether or not ignore-comments is turned on can only be determined after parsing, so the justfile must be parsable, which includes comments, and unmatched {{ and invalid syntax in {{…}} are both parse errors. So ignore-comments is only able to ignore errors that happen after parsing, like unknown variables.
I'll leave this open. A solution might be to defer parsing recipe bodies, but I honestly think the code for that might be so ugly that it won't be worth it.

Would it be possible to have an option to print a recipe "as it it would be executed"?
That might actually be better, because then it doesn't mean maintaining a copy of the command in a comment.
For example, just --print ls-mounts could print the "unescaped" commands in the recipe (perhaps optionally along with any comments), ready for copy-and-pasting?

Yah, i think just --dry-run might be what you want, give that a shot.

Ah! I don't know how I missed --dry-run. 🤦♂
I do indeed think that will be an OK work-around given the previously-explained difficulties of trying to completely ignore comments.
Thanks for the super-quick responses! (I'll leave it up to you if you want to keep this issue open, but completely understand if you want to consider it a "won't fix".)

Nice, glad that works! I think I'm going to close this. It would be nice to fix it, but the fix would be to delay parsing recipe bodies and store them in an unparsed form, and then parse them later on, which would be super gross.