Not being abble to start

Please tick all applicable boxes.

[x] I am using Python 3.5.3 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x] The latest master version (release-260419)
[x] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
After running it does an hope it worked mensage
./run.sh: line 10: python3.6: command not found
[INFO] launcher: Starting sanity checks
[INFO] launcher: Checking for Python 3.5+
[INFO] launcher: Checking console encoding
[INFO] launcher: Ensuring we're in the right environment
[INFO] launcher: Required checks passed.
[INFO] launcher: Optional checks passed.
[ERROR] launcher: Error starting bot
Traceback (most recent call last):
File "run.py", line 367, in main
from musicbot import MusicBot
File "/home/pi/MusicBot/musicbot/init.py", line 74, in
from .bot import MusicBot
File "/home/pi/MusicBot/musicbot/bot.py", line 31, in
from .player import MusicPlayer
File "/home/pi/MusicBot/musicbot/player.py", line 17, in
from websockets.exceptions import InvalidState
ModuleNotFoundError: No module named 'websockets'
[INFO] launcher: Attempting to install dependencies...
Defaulting to user installation because normal site-packages is not writeable
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Requirement already satisfied: pynacl in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 1)) (1.4.0)
Requirement already satisfied: discord.py[voice] in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 2)) (1.7.3)
Requirement already satisfied: pip in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 3)) (21.3.1)
Requirement already satisfied: youtube_dl in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 4)) (2021.6.6)
Requirement already satisfied: colorlog in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 5)) (6.5.0)
Requirement already satisfied: cffi>=1.4.1 in /usr/local/lib/python3.7/dist-packages (from pynacl->-r requirements.txt (line 1)) (1.15.0)
Requirement already satisfied: six in /usr/lib/python3/dist-packages (from pynacl->-r requirements.txt (line 1)) (1.12.0)
Requirement already satisfied: aiohttp<3.8.0,>=3.6.0 in /usr/local/lib/python3.7/dist-packages (from discord.py[voice]->-r requirements.txt (line 2)) (3.7.4.post0)
Requirement already satisfied: attrs>=17.3.0 in /usr/local/lib/python3.7/dist-packages (from aiohttp<3.8.0,>=3.6.0->discord.py[voice]->-r requirements.txt (line 2)) (21.2.0)
Requirement already satisfied: chardet<5.0,>=2.0 in /usr/lib/python3/dist-packages (from aiohttp<3.8.0,>=3.6.0->discord.py[voice]->-r requirements.txt (line 2)) (3.0.4)
Requirement already satisfied: async-timeout<4.0,>=3.0 in /usr/local/lib/python3.7/dist-packages (from aiohttp<3.8.0,>=3.6.0->discord.py[voice]->-r requirements.txt (line 2)) (3.0.1)
Requirement already satisfied: multidict<7.0,>=4.5 in /usr/local/lib/python3.7/dist-packages (from aiohttp<3.8.0,>=3.6.0->discord.py[voice]->-r requirements.txt (line 2)) (5.2.0)
Requirement already satisfied: yarl<2.0,>=1.0 in /usr/local/lib/python3.7/dist-packages (from aiohttp<3.8.0,>=3.6.0->discord.py[voice]->-r requirements.txt (line 2)) (1.7.2)
Requirement already satisfied: typing-extensions>=3.6.5 in /usr/local/lib/python3.7/dist-packages (from aiohttp<3.8.0,>=3.6.0->discord.py[voice]->-r requirements.txt (line 2)) (<IP_ADDRESS>)
Requirement already satisfied: pycparser in /usr/local/lib/python3.7/dist-packages (from cffi>=1.4.1->pynacl->-r requirements.txt (line 1)) (2.20)
Requirement already satisfied: idna>=2.0 in /usr/lib/python3/dist-packages (from yarl<2.0,>=1.0->aiohttp<3.8.0,>=3.6.0->discord.py[voice]->-r requirements.txt (line 2)) (2.6)
[INFO] launcher: Ok lets hope it worked
[INFO] launcher: All done.
Steps to reproduce

./run.sh

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
does not have a log folder and not have a file

run py -3.7 -m pip install websockets


Add an option to check build status only once

I'd like to see an option to check the status of the build light only once (RunOnce) (and will submit a pull request for this).

✔


Azure Disk Backup is already GA

Azure Disk Backup is already GA.

Backup for Azure Managed Disk is now generally available

@tksh164 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@tksh164 Thanks for helping us improve the documentation, we appreciate it!
This change looks good so:
#sign-off


[3.3] Fix Cylinder shape collision with margins when using GJK-EPA

In the case of falling back to GJK-EPA algorithm to generate cylinder contact points, margins were never taken into account.
This fixes the depenetration phase for kinematic bodies and allows consistent floor detection for cylinder shapes.
Fixes #47058
3.3 backport of PR #47067
Note: This change doesn't affect the Bullet physics backend or other shapes than cylinder.

Thanks!


Can not combine with Home Asisstant

I can't integrate Lg Thinq. I get this message when i  try enter the profile

Please help me how to integrate Lg Thinq with step by step, i'm new in this sphere.

Same problem happen to me

Can you solve this problem? Have you any ideas brother?

Looking at the image seems that the error occur on LG login page, this is not an integration error and  cannot do nothing. Did you try to login directly from integration config flow instead using URL redirect method?

login directly from integration config flow works. :)

@Umarkhabiliy,
did suggestion provided solve issue also for you? Can we close this?

Looking at the image seems that the error occur on LG login page, this is not an integration error and  cannot do nothing. Did you try to login directly from integration config flow instead using URL redirect method?

Mr I dont understund how to it? Can you share screen step by step?

login directly from integration config flow worked for me. :)

How you did brother?

login directly from integration config flow worked for me. :)

How you did brother?

enter your user and password in the integration without ticking use alternative url method, then submit.

Thanks it worked


WPS not terminating since ESP-IDF 3.1.2 (IDFGH-609)

If WPS in station mode ist started in ESP32 first and after that in the Router (nevertheless if its using PIN or PBC method) WPS never succeeds and is never stopped anymore. If WPS is started at the router first (PBC) it works, though WPS PIN is not possible this way. I tested this with ESP-IDF 3.1.1 and it is working there in all ways so its a bug introduced by 3.1.2.

@Roody75 Please update IDF to 3.1.3. The bug has been fixed.

@Roody75 Would you help share if any updates for the issue? Thanks.

@Roody75  Thanks for reporting the issue. Would you help share if any updates for the issue? Has your issue been fixed? Thanks.

@Roody75 Thanks for reporting, feel free to reopen if you still have the issue, thanks.


SignUp page

This would allow user to sinup by filling form, or using gmail and facebook as well before proceeding to login

@SAITARUN55 would like to work along with my team-mate

No need of this screen....as I already told this in the zoom call not to...only with login instead of both signup and login.


Cross: CD Design Documentation

AS A Developer
I WANT TO have documentation about CD pipelines
SO THAT I can build pipeline what functions as everybody thinks it does
Acceptance Criteria

[x] The document describes in general what CD pipeline does
[ ] The document describes individual steps needed in the pipeline

@keijoraamat to own with support from @yahya130 @decodingahmed

About half-way through, needs details adding on the steps in the pipeline
Please see here: https://github.com/buerokratt/Cross-functional-requirements/pull/79


create new file

create a new file  to test use branch !

add new file ,in second branch !

merge add file


Update dependence

Hi Niklas,
I want to update react 15.0.0. Can you review and merge my my commit. It's very useful for new react app.
Thanks, Khue

👍

This will make it impossible to use a File object as prop?

Hi @nnarhinen ,
I just updated it. Now it can accept string or File as prop.
Thanks, Khue

Closing as react-pdf 1.6.0 is fully updated and this is no longer necessary. Appreciate the help though!


Vague documentation on licensing

The Documentation says "In organizations with Microsoft Defender for Office 365 Plan 2 (add-on licenses or included in subscriptions like Microsoft 365 E5), you can use Attack simulation training in the Microsoft 365 Defender portal..."
Does "you can use Attack simulation" mean that the person setting up and "using" the attack simulation services in Defender needs to have E5, but users that the phishing simulation attack is sent to can have any licenses? i.e. I can still send a simulated email to someone with a simple Exchange Plan 1 or Plan 2 license. Or does that mean that every user I send a simulated attack email to needs to have Plan 2?

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

ID: 59c87455-77af-093a-6fb6-286049aad15a
Version Independent ID: b2690e98-781d-4a27-a9bc-bd621964feb0
Content: Get started using Attack simulation training - Office 365
Content Source: microsoft-365/security/office-365-security/attack-simulation-training-get-started.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@benstegink, my guess is that from a technical standpoint, all you need is one Defender for Office 365 Plan 2 license in the entire organization (likely, for the admin setting it up as you described).
From a compliance/licensing perspective, anyone/everyone involved should be licensed for Defender for Office 365 Plan 2.
There are a few different Defender for Office 365 features that work that way.

According to the licensing terms, if one user is licensed for Defender for Office 365 Plan 2, every user must be licensed for it, which would somehow answer the initial question with "this scenario is non-compliant"

@DMoenks, thanks that's the info I was looking for that explanation on this page where it talks about licensing might be nice as well.  "To use attack simulation, every user and shared mailbox must have a Microsoft Defender for Office 365 Plan 2".
Because it also looks like not only every user, but every shared mailbox as well?! So if I have 100 shared mailbox and add a Defender for Office 365 Plan 2, I need to pay an extra $500/month for those shared mailboxes? Or having to keep track of short term shared mailboxes when a user leaves a company. I can under how this works for user, but also having to licenses shared mailboxes that were previously free as the fluidity that shared mailboxes can have in some companies...

@benstegink, shared mailboxes don't require user licenses, so they shouldn't require licenses for MDO. It's more about the licensing of the users who use those shared mailboxes.

@chrisda, that's always been my thought as well. I was just looking at the licensing terms @DMoenks referenced above and it has the below. Again, going back to the original issue of that it might be good to put some clarification into this. i.e. if all users need to be licensed, the bullet about Shared Mailboxes is just confusing b/c if it's that's user accessing the shared mailboxes need the license, and all users need a license, by default all the licenses will be there and putting shared mailboxes as a bullet just adds confusion imho.
"For Microsoft Defender for Office 365 Plan 2 tenants, licenses must be acquired for users or mailboxes falling under one or more of the following scenarios:

All Exchange Online users on the tenant. This is because Plan 2 features and capabilities protect all users in the tenant.
All shared mailboxes on the tenant.
If Safe Attachments protection for SharePoint, OneDrive for Business, or Teams is turned on, all users that access SharePoint, OneDrive for Business, or Teams.
Any user that uses Microsoft 365 Apps or Teams when Safe Links protections are enabled."

@benstegink, apparently, the Service Description is correct. I'll add a link to it in the What do you need to know before you begin? section.

@chrisda, sounds good to me. That was what I was looking for in the "What you need to do" is which specific users/mailboxes/etc. need to be licensed with that P2 license in order to run an attack simulation. Thanks again!

So a followup question then if Shared Mailboxes need to be licensed, does this also mean you need a license for each Microsoft 365 Group and Teams team as these also have shared mailboxes associated with each of them? The whole thing of having to license shared mailboxes when all of the users using the Shared Mailboxes is baffling, not to mention the administrative overhead of licensing all these mailboxes seems nearly impossible.

I'll just link OfficeDocs-O365ServiceDescriptions/#764 here for reference, mostly being in the same state regarding licensing clarification

@benstegink, I would say: go by what the Service Description says.

Nothing new to report here, so closing.
Thanks for the feedback.


[Suggestion] Add enchantment list and Lore placeholders for item webhooks

When a player publishes an item such as an enchantment book, it just comes through as "Enchanted Book" without any way of telling the actual enchantment applied to it. Please add a placeholder that allows enchantment information to be displayed, as well as lore of the item
Thanks :)

(note: %lore% seems to actually parse, but parses as just an empty string)

Added for latest version


A list of all transactions done by address

Created a rest api endpoint to fetch all signatures for particular address
Created Transaction graphql schema to fetch transactions.

Todo

Need to parse transaction data and display on the UI

I updated the code based on comments. I am able to fetch signatures for transactions and updated the script. I also created a Transaction component which is responsible for rendering a Transaction.
The code is not clean. Need to refactor it.


not having colors when rendering the svg manually

Hi,
Well I'm facing an issue while trying to render the SVG manually, the "stroke" and "fill" attributes are always empty so those 2 lines:
    layer.strokeColor = [path.svgAttributes[@"stroke"] ?: [UIColor blackColor] CGColor];     layer.fillColor   = [path.svgAttributes[@"fill"] ?: [UIColor redColor] CGColor];
always gives the black and the red colour for every SVG.
So if someone can help me it will be nice.

what is the format of your svg's stroke="..." attribute?

Yes it's an hex triplets but when I debug the stroke and the fill values contains "(no summary)".

Please upload the problem svg

It's the svg of the demo project:
https://github.com/pocketsvg/PocketSVG/blob/master/Demos/Sample SVG Files/tiger.svg

Ah, the attributes contain a CGColorRef, not a UIColor. I just tested the tiger svg, and it is working fine with HEAD.

`    NSURL *url = [[NSBundle mainBundle] URLForResource:@"tiger" withExtension:@"svg"];
for(SVGBezierPath *path in [SVGBezierPath pathsFromSVGAtURL:url]) {

    // Create a layer for each path
    CAShapeLayer *layer = [CAShapeLayer layer];
    layer.path = path.CGPath;

    // Set its display properties
    layer.lineWidth   = 4;

    layer.strokeColor = path.svgAttributes[@"stroke"] ?: UIColor.blackColor.CGColor;
    layer.fillColor = path.svgAttributes[@"fill"] ?: UIColor.redColor.CGColor;

    // Add it to the layer hierarchy
    [imgSGV.layer addSublayer:layer];
}`

But now after I copied past your code I'm having a precompilation error :
Incompatible operand types ('id _Nullable' and 'CGColorRef _Nonnull' (aka 'struct CGColor *'))

sorry, (__bridge CGColorRef)

Thank you, it solved the error and I'm having the colours :)


[Setup] Requirements.txt

Create a proper requirements.txt with proper Poetry syntax

Resolved as of 28bddbd.
Subsequent updates to dependencies added via poetry can be updated here with poetry export --format requirements.txt > requirements.txt


Remove misplaced divider in Global Pipelines Page

Fixes #1606
Description

This is how the Global Pipelines page look like now.

How Has This Been Tested?

Go to the Pipelines page using the left Navbar
Check whether you can see the divider between the table and the header

Test Impact

Visual changes.
Request review criteria:

Self checklist (all need to be checked):

[X] The developer has manually tested the changes and verified that the changes work
[X] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[X] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@kywalker-rh can you have a look at the screenshot above? Not sure if this is the ideal space between the header and the table so that the table dropdown and the header dropdown aren't confusing.


Add optional additional_attributes attribute to ldap_object data source

Implements https://github.com/dodevops/terraform-provider-ldap/issues/5

Sorry for the delay, I've fixed the GitHub workflow and it should hopefully work now.


Bot Commands

Is it possible for Nadeko to receive commands from another Bot on the same server. To permit the usage of .give or what have you for currency.

no.


Multiple tsconfig.json files in project

My project has multiple tsconfig.json files, one for my gulpfiles and one for my actual project files. It seems as though neither of the files are recognized, however. I get the "Experimental decorators" error unless the tsconfig.json file is put in the project root.

If you have multiple tsconfig.json in the same folder, only one will be picked up, at a unpredictable order, therefore it is not recommended. Plus tsconfig.json files only covers files in the current folder and its children folder, therefore what you see is the expected behavior.


Tooling to set up cert-manager with external dependencies

Is your feature request related to a problem? Please describe.

Currently when trying to reproduce a bug/verify a bugfix or work on a feature that requires deployment and configuration of an external dependency with cert-manager, a lot of engineer's time is spent figuring out how to set up the external dependency to work with cert-manager. And even if one of the maintainers has knowledge how to configure a particular dependency, this knowledge is not shared with the rest of the maintainers or community.
Describe the solution you'd like
There should be some means how to easily install and cross-configure cert-manager and related external dependencies ( i.e Ingress/Gateway API implementations, Vault etc) and associated cert-manager resources for common scenarios. An example scenario would be to deploy Vault, setup Vault's PKI Secrets Engine, configure auth and create a cert-manager [Cluster]Issuer that works with the configured Vault PKI.
It should be straightforward to change the configuration of any of the tools and resources.
It should be straightforward to update the scripts/tool to allow deploying a new tool.
Describe alternatives you've considered

We have some dependency configuration scripts for e2e tests here. However, arguably, these are too complex to be easily usable/understandable.
Additional context
The more concrete implementation design is still in progress.
Here is the initial design doc https://docs.google.com/document/d/10mm10rmQudssTY71Ou2oqvGDtz70VzosGuKy90Is9-g/edit?usp=sharing
Environment details (remove if not applicable):

Kubernetes version:
Cloud-provider/provisioner:
cert-manager version:
Install method: e.g. helm/static manifests

/kind feature

Hi @irbekrm , I have used cert-manager that was cool, and I have still researched cli tools before.  I would like to apply this feature as my  GSoC 2022. Could you give me some advice？

Hi @lonelyCZ
Thanks for your interest in working on this as  GSoC project, glad to hear you've used cert-manager already!

I would like to apply this feature as my GSoC 2022. Could you give me some advice?

The first thing to do would be to familiarize yourself with the GSoC timeline and contributor docs (that you may have already done)
I'd recommend that you get involved with the cert-manager community, say hi on Slack and see if you have a chance to join some of our meetings, see the details for how to join in our contributing docs
We usually recommend that folks who are interested in working on GSoC also try to work with cert-manager a bit, picking up a good first issue or finding some other way how to contribute is a great way to get started and helps you to understand if the project is suits your interests and helps us to see whether we work well together :). Again, do reach out to us on Slack if you want to contribute and have questions about how to get started!
In terms of deadlines, I understand that 19th of April is the deadline for submitting the proposal.
Between now and then is a period for students to discuss proposal ideas with mentoring organizations. As a mentor I am happy to review a draft proposal and some of cert-manager team may also be able to give advice, do reach out on Slack :)
There is also a discussion board set up in CNCF Github with some interesting discussions https://github.com/cncf/mentoring/discussions/categories/google-summer-of-code-2022
Does this make sense?

Thanks for your reply. :) @irbekrm
I will familiarize the community as soon as possible, and actively participate in the draft proposal. Looking forward to work with community members.

I understand the purpose of this feature is easily to recreate the installation of cert-manager with external dependencies for maintainers or members. There are two opinions:

If there have been a cert-manager environment only lacking external dependencies, we should only deploy external dependencies, so should we separate cert-manger from externally dependent installations. For example, providing a option for script to certain install mode.
Because each external dependency is configured differently, we may need to be flexible in configuring these options, so we can separate the configuration file from the installation file. Such as vault, we provide a default configuration file that users can modify as needed, as shown in the figure below.

If there have been a cert-manager environment only lacking external dependencies, we should only deploy external dependencies, so should we separate cert-manger from externally dependent installations. For example, providing a option for script to certain install mode.

Whatever installation mechanism you choose, it should be able to deploy different combinations of cert-manager + some external tools, i.e cert-manager + Vault, cert-manager + some ingress implementation etc.
It would be nice to consider how the installation mechanism you choose could be made further configurable so that the user can install just the dependencies and not cert-manager or perhaps you implement the mechanism in such a way that it already works for MVP.

Because each external dependency is configured differently, we may need to be flexible in configuring these options, so we can separate the configuration file from the installation file. Such as vault, we provide a default configuration file that users can modify as needed, as shown in the figure below.

This will really depend on what installation mechanism you choose and how much abstraction you build on top. For example, you could have chosen to implement this as a Go CLI and pass each configuration option as a CLI flag and then in your code translate those into the format suitable to configure each dependency. Or you can have configuration templates for each dependency as you say, they are all valid options and each has its pros and cons.
It would be good if your proposal would show how you expect the configuration options can be passed. It can also be good to have a 'Questions' section for things that are not clean yet at this point. You could also add a 'Constraints' section if you can think of any constraints for this particular installation mechanism.

Thank you! :)

they are all valid options and each has its pros and cons.

Yes, I consider their pros and cons as follows

If we implement this as a Go CLI, it is easy to use, but not easy to extend. Each extension of a dependency requires redevelopment and compilation, which is not conducive to sharing knowledge.
If we implement this as configuration templates, implementing a configuration framework, which facilitates scaling, but we need to design a friendly way to use it.
If we were using Terraform, we might need to ask members to learn a new build language.

I tend to option 2 or option 3 because developing a command-line tool can be complex and difficult to maintain, especially considering that it can install multiple dependencies.
What do you think, or what do you think of Terraform and how many members use this tool?@irbekrm  I think the implementation ideas of scheme 2 and 3 are similar.

Hi @lonelyCZ

Yes, I consider their pros and cons as follows

Good work on considering those, I think you can even include that in the proposal as I think it's always valuable to show that you've considered alternatives 👍🏼
I don't want to influence which option you choose in the proposal 😄 The point of the proposal is for you to choose an implementation mechanism that you think is best and describe how you will solve the problem with it.
Some thoughts though

If we implement this as a Go CLI, it is easy to use, but not easy to extend. Each extension of a dependency requires redevelopment and compilation, which is not conducive to sharing knowledge.

I think if the solution was a CLI, it should be built in such a way that there is some sort of 'framework' that makes adding a new extension easy, i.e adding a new extension should not require touching too many unrelated parts of code (redevelopment). I would argue that having to compile it (most likely this would mean making a new release for the tool) is not a big problem. There could be other maintentance related cons though and agree that it might be more complex than just scripts.

What do you think, or what do you think of Terraform and how many members use this tool

I think all of us are familiar with Terraform

Thanks for your  response @irbekrm
I will write a proposal as soon as possible, including all of my consideration.

Thanks @lonelyCZ
Good luck with the proposal!
To add, the actual GSoC work includes working on a design doc and discussing it with the cert-manager team, so the actual implementation might differ from what you will describe in the proposal. I'd just encourage you to choose the implementation mechanism you think is best suited and describe that.

Hi, @irbekrm , I have submited a proposal, there is a replica at https://docs.google.com/document/d/1wZ5ia0j4-5UVWN24xgj25UaYffPaTlBtpmerr5Uw8Io/edit#, looking forward to your comments, thanks :)
If possible, I would like to discuss the feasibility of the proposal with the community to determine the final practical proposal.

Thanks @lonelyCZ! I will take  a look now.

discuss the feasibility of the proposal with the community to determine the final practical proposal

Once a student gets accepted, there will be a period of time to discuss the final design with the whole team following the same process as we usually do when discussing a design document, so you should not worry about smaller details at this stage I think (The alternative would have been to have the whole of the team involved in working with each candidate on their proposal, which would not be feasible).
Also, CNCF org admin will have the final say about which projects get accepted.
Does this make sense?

Yes, thanks for your reply. @irbekrm During this period, I will try my best to implement some demos to verify the feasibility of the proposal.

Also, CNCF org admin will have the final say about which projects get accepted.

Regardless of the results, we hope to complete the project in a better way and bring more convenience to the community.

Thanks for you comments @irbekrm , I will improve it as soon as possible.

/priority important-soon
Assigned during triage party. Seems like this will be embarked on during the next few months of GSoC.

Hi, @irbekrm. I'm excited to tell you that I have implemented a easy demo that can deploy and setup Cert-manager and Vault with Terraform at this repository https://github.com/lonelyCZ/terraform-cm.
It currently supports the deployment of Cert-manager and Vault, configures the PKI engine for Vault by default, and generates a ClusterIssuer object named vault-issuer. However, the current implementation is simple, lacks some necessary dependency checking, and may not be stable.


🍆 Add Sik | crifferent.de

https://kusama.subscan.io/extrinsic/16789369-2

Thank you for your request!
@SimonKraus
You can check how it will look like:

App screens

Thanks @stepanLav 🤘


Match whole banned words rather than substrings

Currently if a string in the ban list exists anywhere in the title, including as a substring of a longer word, the title is discarded.
Update this to discard titles where the whole word matches exactly, but not as a substring of a longer word.
Example:
BANNED = ['dog']
title = 'my big dog' # should be discarded
title = 'a dogged load shark' # should not discard

This is resolved by commit acc48a8


[Task]: Write Cypress tests for GraphQL API - HTTP2 and H2C type requests

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
No response

Not required atm since adoption is low


bindUiComponent is exported 3 times? not sure if it's a problem...

I noted here (https://github.com/RainCat1998/obsidian-custom-attachment-location/issues/88) that the obsidian-custom-attachment-location plugin settings does not display the current value of jpegQuality, it always shows 0.1.
I confirmed the data is correctly saved to data.json, so this is a display issue. I am not smart enough to figure out where the problem is, but seems to be in this library maybe.
Is it a problem that this bindUiComponent function is exported 3 times with differing parameters?
https://github.com/mnaoumov/obsidian-dev-utils/blob/967ca31067898feda3935945e4e4454f77a4a92b/src/obsidian/Plugin/UIComponent.ts#L99
https://github.com/mnaoumov/obsidian-dev-utils/blob/967ca31067898feda3935945e4e4454f77a4a92b/src/obsidian/Plugin/UIComponent.ts#L129
https://github.com/mnaoumov/obsidian-dev-utils/blob/967ca31067898feda3935945e4e4454f77a4a92b/src/obsidian/Plugin/UIComponent.ts#L159

No, it is not an issue. That's Function Overloads


Admin can view list of ALL Users.

include name, email address, mailing address, phone number, billing information, etc.

A summary of the problem or requirement that this task solves
Admin user has the ability to views all users as necessary
Clear requirements for how to consider each task or issue "done"
Testing is complete verifying capability.


Viral sequence database is not accessible.

The link
https://zenodo.org/records/10435588/files/VirID_rRNA_db.fasta.bz2?download=1&preview=1
is not publicly accessible, Zenodo giving the message:
You do not have sufficient permissions to view this page.

We change the link, you can download the database at: https://drive.google.com/file/d/1PDbuIG1ZcMRWrQ8nhE50p33B2J1IposQ/view?usp=sharing

Great. Thanks.
On Mon, 8 Jul 2024 at 12:23 AM, YangZiyue @.***> wrote:

We change the link, you can download the database at:
https://drive.google.com/file/d/1PDbuIG1ZcMRWrQ8nhE50p33B2J1IposQ/view?usp=sharing
—
Reply to this email directly, view it on GitHub
https://github.com/ZiyueYang01/VirID/issues/2#issuecomment-2212468018,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABN2PC5KCDME6CV2K2RKRATZLFFPXAVCNFSM6AAAAABKMM5CBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJSGQ3DQMBRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Btw, for shell-based systems, Google drive can be an awkward service from which to obtain files. I really don't want to DL it to my local machine and then push it back to an HPC over piddly Australian broadband.
The easiest tool for this task is the Python based gdown
Where, I got your file using the following.
gdown --fuzzy "https://drive.google.com/file/d/1PDbuIG1ZcMRWrQ8nhE50p33B2J1IposQ/view?usp=sharing"

Fuzzy is required, since you've provided a sharing link rather than the direct file ID. The quotes are also needed to prevent the shell parsing the URL.


docs: update zenith build from source instructions

The Zenith is now merged, it can now fetch from main branch.

Thank you Thana!


Support for async/await

Support for Swift 5.5's new async/await would be amazing.

Yep, I would love that, too. Now it’s official: async/await backward compatible down to iOS 13: https://developer.apple.com/documentation/xcode-release-notes/xcode-13_2-release-notes

This should be as simple as wrapping the send function with withCheckedThrowingContinuation. Don’t think it applies to the websockets.

Correction; We can implement AsyncSequence for web sockets: https://obscuredpixels.com/awaiting-websockets-in-swiftui
But this is not as trivial as the send functions so I would make this a separate issue.

Hi everyone 👋 ,
I am happy to co-work on this feature now that the new version is out. Let me know if anyone wants to give it a try!

@maticzav Sounds interesting! I would like to give it a try 🙏
Tho, seems a lot changed since the initial discussion had kicked off. Please correct me if I'm wrong, but it seems that introducing async/await in the current version's context means extending the following functions with async analogous:
public class Client: GraphQLClient, ObservableObject {
    // ...
    public func query(
        _ args: ExecutionArgs,
        request: URLRequest? = nil,
        policy: Operation.Policy = .cacheFirst
    ) -> Source {
       // ...
    }

    // ...
    public func mutate(
        _ args: ExecutionArgs,
        request: URLRequest? = nil,
        policy: Operation.Policy = .cacheFirst
    ) -> Source {
    // ...
    }

    // ...
    public func subscribe(
        _ args: ExecutionArgs,
        request: URLRequest? = nil,
        policy: Operation.Policy = .cacheFirst
    ) -> Source {
    // ...
    }
}

Also I must note that Combine provides a binding to async/await out-of-box:
let publisher = remoteDataPublisher(forURLs: urls)

for try await data in publisher.values { // AsyncThrowingSequence
    ...
}

(Source ☝  Async sequences, streams, and Combine | Swift by Sundell)
That said, it seems that we can extend the Client's API by introducing one-shot throwing async versions of query and mutate. As for subscription we can provide a version that returns AsyncThrowingStream as out of convenience  🤔
Please let me know what do you think about it 🙇

@elfenlaid, you are right! Let me know if you are still interested, your comment must have slipped through the cracks of my inbox ☹

@maticzav yeap, on it then 🙇

I'm so sorry, but I'll park the ticket for now 🙇

for my needs I just created an extension
import struct Foundation.URLRequest
import SwiftGraphQL
import SwiftGraphQLClient

public extension GraphQLClient {
    func queryAsync<T, TypeLock>(
        _ selection: Selection<T, TypeLock>,
        as operationName: String? = nil,
        request: URLRequest? = nil,
        policy: Operation.Policy = .cacheFirst
    ) -> AsyncThrowingStream<DecodedOperationResult<T>, any Error> where TypeLock: GraphQLHttpOperation {
        query(selection, as: operationName, request: request, policy: policy)
            .values
            .eraseToThrowingStream()
    }

    func mutateAsync<T, TypeLock>(
        _ selection: Selection<T, TypeLock>,
        as operationName: String? = nil,
        request: URLRequest? = nil,
        policy: Operation.Policy = .cacheFirst
    ) -> AsyncThrowingStream<DecodedOperationResult<T>, Error> where TypeLock: GraphQLHttpOperation {
        mutate(selection, as: operationName, request: request, policy: policy)
            .values
            .eraseToThrowingStream()
    }

    func subscribeAsync<T, TypeLock>(
        to selection: Selection<T, TypeLock>,
        as operationName: String? = nil,
        request: URLRequest? = nil,
        policy: Operation.Policy = .cacheFirst
    ) -> AsyncThrowingStream<DecodedOperationResult<T>, Error> where TypeLock: GraphQLWebSocketOperation {
        subscribe(to: selection, as: operationName, request: request, policy: policy)
            .values
            .eraseToThrowingStream()
    }
}

For the record, and in case its useful, somehow I thought eraseToThrowingStream() is part of Swift but I had it in already imported swift-concurrency-extra packaged:
https://github.com/pointfreeco/swift-concurrency-extras/blob/main/Sources/ConcurrencyExtras/AsyncThrowingStream.swift
--
Also note that while in case of mutations arguably cached results should not be used, for queries with policy set to .cacheAndNetwork there may be two values returned, see https://github.com/maticzav/swift-graphql/pull/105#discussion_r1348624662


Support for Windows 11

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Problem
Unable to access the Explorer in Windows 11
Desired Solution
Full support and access for Windows 11
Alternatives and Workarounds
No response
Additional Context
No response

Hi @aljbishop , as far as we have seen there is no issues related to trying to use Storage Explorer on Windows 11. Can you elaborate on what issues you are seeing? Have you tried reinstalling from https://storageexplorer.com ? Are you seeing any errors when trying to launch Storage Explorer?

Closing due to lack of response. If you are still having problems using Storage Explorer on Windows 11, please open a new issue with more specifics. Thanks!


Evaluate and add option --generate-json-number-as

We are adding an option in this PR for json type integer:

FEATURE: Add new option for specifying .NET type to express Json integers: --generate-json-integer-as = int | long | biginteger | auto with a default of int. #158

This is to evaluate what options to have and implement similarly for the json type number.

Close the issue as it is done and merged.


claim bypass/block interaction on 1.12.2

GD : 1.5.3 DEV
LuckPerms : 5.2.39
Not sure if this might be related to the particular mod of this block
in a claim of my own:
1.enable cfdebug
2. right click extra utils 2 screen, no gui access, works as intented.
https://i.imgur.com/wo5EqO8.png
Now the bypass.

with cfdebug still on, enable claim mode
right click block to interact.

I tried it in someone's elses claim without cfdebug and was able to interact with a block. you can interact with the block while in claim mode.
https://i.imgur.com/IpPjI9p.png
Excellent plugin, just wanted to report this.

Mods.BlockIdsRequiringAccessTrust: [1, 2, 3]
Any block IDs you list here will require /accesstrust from the owner before a player can interact with them (usually right click). I've heard several mods introduce some kind of "machines" which you can activate by right clicking them. Those would be good candidates for adding here.
(from the gd wiki)
apparently this isn't an issue. Thank you for the hard work.

Unable to reproduce. Make sure you read https://github.com/bloodmc/GriefDefender/wiki/Permissions carefully


Wemo - Use returned device name instead of model_name

model_name in the discovery info is based off the description which is not always correct. Change this to use the output of repr()

I have submitted an issue to pywemo identifying that model_name is not always correct
https://github.com/pavoni/pywemo/issues/89


Fixing example code for BinaryFocalCrossentropy in losses.py file

The example mentioned for the Keras API binarycrossentropy was failing with the error AttributeError: 'list' object has no attribute 'shape'.
>>> # Example 2: (batch_size = 2, number of samples = 4)
>>> y_true = [[0, 1], [0, 0]]
>>> y_pred = [[-18.6, 0.51], [2.94, -12.8]]
>>> # Using default 'auto'/'sum_over_batch_size' reduction type.
>>> loss = keras.losses.BinaryFocalCrossentropy(
        gamma=3, from_logits=True)
>>> loss(y_true, y_pred)

It works if we change the input from list to arrays.
>>> # Example 2: (batch_size = 2, number of samples = 4)
>>> y_true = np.array([[0, 1], [0, 0]])
>>> y_pred = np.array([[-18.6, 0.51], [2.94, -12.8]])
>>> # Using default 'auto'/'sum_over_batch_size' reduction type.
>>> loss = keras.losses.BinaryFocalCrossentropy(
        gamma=3, from_logits=True)
>>> loss(y_true, y_pred)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 59.98%. Comparing base (96e07ec) to head (a8641cc).

:exclamation:  There is a different number of reports uploaded between BASE (96e07ec) and HEAD (a8641cc). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (96e07ec)
HEAD (a8641cc)

keras
4
1

keras-torch
1
0

keras-tensorflow
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20492       +/-   ##
===========================================
- Coverage   82.08%   59.98%   -22.10%
===========================================
  Files         515      515
  Lines       47543    47543
  Branches     7455     7455
===========================================
- Hits        39024    28519    -10505
- Misses       6709    17270    +10561
+ Partials     1810     1754       -56

Flag
Coverage Δ

keras
59.98% <ø> (-21.95%)
:arrow_down:

keras-jax
?

keras-numpy
59.98% <ø> (-0.01%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


$auth.currentUser doesn't update when subscribe to new data

In my application at some page I subscribe to new date like this: (angular 1.3.5)
  if (this.userId) {
    return Meteor.users.find({_id: this.userId},
                             {fields: {'other': 1, 'things': 1}});
  } else {
    this.ready();
  }
});```

but $auth.currentUser doesn't get it while $root.currentUser does

Can you be more explicit on the workflow ?

@ThomsCass
The default fields the server publish for currentUser are "id" and "profile", on a page in my application
I call for another subscription to publish another date about currentUser like I said before.
On that page where I call the new subscription, "$root.currentUser" gets the new fields, but "$auth.currentUser" doesn't

@jamalx31 it's a big hard to get to the bottom of it without a reproduction.
Can you create a small reproduction of that problem in a separate repo and link to it?
Thank you

Hi @Urigo
https://github.com/jamalx31/meteor-rep
in this repo you can reproduce the bug, just run it with meteor and you will see it

@jamalx31 You can update your app to use<EMAIL_ADDRESS>and replacing $auth.currentUser with vm.currentUser or currentUser if you're using it with $scope.

@kamilkisiela @Urigo
it seems like $rootScope.currentUser and $scope.currentUser are not the same object
in $scope.currentUser I miss some of the fields I publish for current user!

released 1.0.3.  please re-open if it's not fixed for you


[WIP/DNM] CVE: Fix Receiver malicious tenant

If running as root or with enough privileges, receiver can create a
directory outside of the configured TenantHeader.
This commit fixes it up by sanitizing the user input and explicity not
allowing such behavior.

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Signed-off-by: Daniel Mellado<EMAIL_ADDRESS>

@GiedriusS how about putting that for now in the receive package?

Failure on the tests seems totally unrealted


Cannot find Subnets

Hi,
I'm new to using the aws-alb-ingress-controller.
I am probably doing something wrong but it seems that version
894847497797.dkr.ecr.us-west-2.amazonaws.com/aws-alb-ingress-controller:v1.0.0
is not respecting subnet values or the subnet tags for Internet facing ALB.
(please note: the internal ALB working perfectly)
I get the following error:
E1126 07:36:59.033715       1 :0] kubebuilder/controller "msg"="Reconciler error" "error"="failed to build LoadBalancer configuration due to retrieval of subnets failed to resolve 2qualified subnets. Subnets must contain the kubernetes.io/cluster/\u003ccluster name\u003etag with a value of shared or owned and the kubernetes.io/role/internal-elb tag signifyingit should be used for ALBs Additionally, there must be at least 2 subnets with unique availability zones as required by ALBs. Either tag subnets to meet this requirement or use the subnets annotation on the ingress resource to explicitly call out what subnets to use for ALB creation. The subnets that did resolve were []"  "Controller"="alb-ingress-controller" "Request"={"Namespace":"default","Name":"grafana-ingress"}
However I have set the subnet values in the ingress yaml
apiVersion: extensions/v1beta1 kind: Ingress metadata:   name: "grafana-ingress"   namespace: "default"   annotations:     alb.ingress.kubernetes.io/scheme: internet-facing     alb.ingress.kubernetes.io/success-codes: 302     alb.ingress.kubernetes.io/ssl-policy: ELBSecurityPolicy-2016-08     kubernetes.io/ingress.class: "alb"     alb.ingress.kubernetes.io/subnets: subnet-xxxx,subnet-xxxxx   labels:     app: grafana-ingress spec:   rules:     - http:         paths:           - path: /*             backend:               serviceName: "grafana-ingress"               servicePort: 80
And I also set the tags on both of my 2 public subnets to
kubernetes.io/cluster/terraform-eks-demo | shared
kubernetes.io/role/elb | 1
i'm pretty sure, i am doing something wrong, so apologies if the answer is painfully obvious

Hi, it's an mistake in our docs.
You should specify alb.ingress.kubernetes.io/success-codes: 302 instead of alb.ingress.kubernetes.io/success-codes: 302.
Docs have been updated, sorry for the trouble caused 😄

haha, its works like magic!
thanks for getting back to me.
:D haha i spent time on implementing reverse proxy using oauth2_proxy
https://github.com/bitly/oauth2_proxy
and using ELB.
But i will revert back to aws-alb-ingress-controller.
and i will script the oidc setup using the AWS CLI, since you currently do not support it.
but thats a different ticket
https://github.com/kubernetes-sigs/aws-alb-ingress-controller/issues/592
anyways thank you @M00nF1sh


[core] Add additional warnings when attaching ref to function elements

add propType validator (similar to elementTypeAcceptingRef)
use a consistent API docs warning
fix some dialog demos not trapping focus properly (due to function transition components)

Closes #15515

Nice 👌


Read output files from real-time calculations

In order to make the density movie we need to read and store in json files the following output files given by Yambo:
o.external_field
o.carriers

YamboOut fails to read 'external_field'

Maybe one can expand the class YamboRTDB() for that purpose (my branch 'occupations').


minor: metrics 接口免登录

Codecov Report

:exclamation: No coverage uploaded for pull request base (V2.2.9-rc@8bd91ef). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             V2.2.9-rc     #631   +/-   ##
============================================
  Coverage             ?   71.07%
============================================
  Files                ?      292
  Lines                ?    21230
  Branches             ?        0
============================================
  Hits                 ?    15090
  Misses               ?     6140
  Partials             ?        0

Continue to review full report at Codecov.

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


set digit length to 10

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

Doesn't that duplicate the existing @Digits annotation? The intent seems to be the same at least.

Doesn't that duplicate the existing @Digits annotation? The intent seems to be the same at least.

@digits sets maximum digit limit not minimum.

I see. But not all telephone numbers have 10 digits, so @Digits seems like the right tool to me still. It's not really a big deal.

closing this PR


Passined prop as object rather than type for row

fixes #8953

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@2728300). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #9176   +/-   ##
==========================================
  Coverage           ?   52.11%
  Complexity         ?    35989
==========================================
  Files              ?     2522
  Lines              ?   197616
  Branches           ?    35383
==========================================
  Hits               ?   102993
  Misses             ?    79637
  Partials           ?    14986

Continue to review full report at Codecov.

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

Thank you very much!
Luigi


database is not working correctly

so there is an error in reading the data in the database

my log looks quite similar, I will try to find out the trigger

Same issue here, any updates?

Not from me, unless someome who knows how to work on the program has this issue i dont think we will get anywhere with it


[5.2] Add support for checking if an array has multiple keys in Arr::has()

I don't think this breaks BC because it converts non-array keys to single element arrays.

Maybe 5.3 is the best place for this?

I assumed 5.2 would be the branch following the contribution guide that says.

Minor features that are fully backwards compatible with the current Laravel release may be sent to the latest stable branch.

Right, but this does technically change the method signature, and 5.3 is essentially the current stable branch now as we near the 5.3.0 release.

Ah fair enough, do you want to me to remake the PR targeting 5.3?

To avoid ambiguity, I'd suggest hasAny and hasAll methods.

@vlakoff Disagree, lots of other "has" methods in Laravel can accept more than one key: https://github.com/laravel/framework/blob/5.2/src/Illuminate/Http/Request.php#L282-L299

Request::has() checks if all keys are present, Str::contains() checks if any key is present... I could provide a few other examples.
It really depends on the developer's point of view, whereas the methods I suggested remove any ambiguity.

Sounds like a better solution. I'll leave this PR open for the time being

Please send to 5.3.


Feature Request: untruncated normal in variance scaling initializer

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): N/A
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): N/A
TensorFlow installed from (source or binary): N/A
TensorFlow version (use command below): N/A
Python version: N/A
Bazel version (if compiling from source):N/A
GCC/Compiler version (if compiling from source):N/A
CUDA/cuDNN version:N/A
GPU model and memory:N/A
Exact command to reproduce:N/A

Currently the "normal" mode of variance_scaling_initializer actually produces truncated normal distribution.
Despite that I saw Google suggest in several places that a truncated normal might be better than a true normal distribution, truncated normal is NOT what is used in the initialization literature such as:
[1] Glorot, Xavier, and Yoshua Bengio. "Understanding the difficulty of training deep feedforward neural networks." In Proceedings of the thirteenth international conference on artificial intelligence and statistics, pp. 249-256. 2010.
[2] He, Kaiming, Xiangyu Zhang, Shaoqing Ren, and Jian Sun. "Delving deep into rectifiers: Surpassing human-level performance on imagenet classification." In Proceedings of the IEEE international conference on computer vision, pp. 1026-1034. 2015.
I found that this difference is affecting some of my experiments. To be more consistent with literature, I would request a new mode in variance_scaling_initializer that produces untruncated normal distribution. I'm OK to implement it If someone can give it a good name.

(For API review.) We agree this should be fixed. I think the best path forward is:

We change the legal values in https://github.com/tensorflow/tensorflow/blob/23c218785eac5bfe737eec4f8081fd0ef8e0684d/tensorflow/python/ops/init_ops.py#L443
if distribution not in {"normal", "uniform"}:
to "normal", "uniform", "truncated_normal", "untruncated_normal"

We change the default here: https://github.com/tensorflow/tensorflow/blob/23c218785eac5bfe737eec4f8081fd0ef8e0684d/tensorflow/python/ops/init_ops.py#L435
from "normal" to "truncated_normal" (matching current behavior, but clearer)

We change the implementation here:
https://github.com/tensorflow/tensorflow/blob/23c218785eac5bfe737eec4f8081fd0ef8e0684d/tensorflow/python/ops/init_ops.py#L465
to treat "normal" & "truncated_normal" as the same (required for backwards-compatibility) and add a new implementation for "untruncated_normal"

In the documentation here:
https://github.com/tensorflow/tensorflow/blob/23c218785eac5bfe737eec4f8081fd0ef8e0684d/tensorflow/python/ops/init_ops.py#L421
update the text to document the options as "truncated_normal", "untruncated_normal", and "uniform", and that "normal" is a deprecated alias for "truncated_normal"

I will make a note that at 2.0, when we can break backwards compatibility, we should change "normal" to be an alias for "untruncated_normal".

Ok. We cannot change the behavior right now, but how about the following:

We add a "

Would that solve your problem? If so, please send a PR and reference this issue. Thank you!

That's exactly the solution I was hoping for.


Cannot add a new column to incremental model that is also a unique_key

If I create an incremental model, and then add a column which is also added to the unique_key, the model fails to build with the error :
DB::Exception: Missing columns: 'col_3' while processing query: 'SELECT col_1, col_2 FROM dbt_clickhouse_2760_test_schema_change_1721300786653.schema_change_append WHERE (col_1, col_3) NOT IN ((SELECT col_1, col_3 FROM dbt_clickhouse_2760_test_schema_change_1721300786653.schema_change_append__dbt_new_data) AS _subquery18)', required columns: 'col_1' 'col_2' 'col_3', maybe you meant: 'col_1' or 'col_2'. (UNKNOWN_IDENTIFIER) (version <IP_ADDRESS> (official build))

It seems to be failing when trying to insert the records from the old table that are not in the new_data table.
See related PR for unit test that demonstrates the issue:  https://github.com/the4thamigo-uk/dbt-clickhouse/pull/1/files
I wonder though, how this could be fixed?
One way would be to modify the query as indicated :
SELECT   col_1,   col_2
FROM
  dbt_clickhouse_2760_test_schema_change_1721300786653.schema_change_append
WHERE
    (
      col_1,
      col_3  <--- REMOVE THE NEW UNIQUE KEY COLUMN
    ) NOT IN (
    (
      SELECT
        col_1,
        col_3   <--- REMOVE THE NEW UNIQUE KEY COLUMN
      FROM
        dbt_clickhouse_2760_test_schema_change_1721300786653.schema_change_append__dbt_new_data
    ) AS _subquery18
  )

Do you think this approach is viable/correct?
Additionally: I notice that for schema changes the strategy switches to the legacy approach, is there scope to implement this using delete+insert?

Had a go at a fix for this https://github.com/ClickHouse/dbt-clickhouse/pull/323/files#diff-80b4c82d42e5f077f4339f050f2c48986cb755188be3f6fc0836aa3c512262f4R157-R175

Fixed by #332


automate notebook doc-making

Looks good! The one catch is that we'll need to add something in the documentation about installing pandoc if users wish to build the notebooks locally with sphinx (as pandoc is a haskell library, so we can't bundle it into pip requirements)
Without pandoc in my rail conda environment, running make html from the docs/ directory of this PR results in an interrupted run and:
Notebook error:
PandocMissing in examples/core_examples/FileIO_DataStore.ipynb:
Pandoc wasn't found.
Please check that pandoc is installed:
https://pandoc.org/installing.html

Running conda install pandoc allows make html to run as expected.
Proposed solutions include:

explaining this in the documentation, in a section relevant to devs trying to build notebooks (or documentation in general) locally
we could also include a comment in our Makefile, in the hopes that anyone encountering trouble with make commands will read that
we could include pandoc in our other conda packages in the standard rail install instructions, but I hesitate to add that kind of overhead for such a relatively niche use case

(This should not be an issue on ReadTheDocs, as pandoc should be automatically included. If I recall, there is a toggle in Advanced Settings where you can remove this, but unless we run into issues we can assume we don't have that set.)


add documentation with Sphinx using a theme provided by Read the Docs.

Current coverage is 7.02%

Merging #17 into master will not affect coverage as of 4272334

@@            master    #17   diff @@
=====================================
  Files           40     40
  Stmts         2748   2748
  Branches         0      0
  Methods          0      0
=====================================
  Hit            193    193
  Partial          0      0
  Missed        2555   2555

Review entire Coverage Diff as of 4272334

Powered by Codecov. Updated on successful CI builds.


Bash on Ubuntu on Windows - bind unix socket failed.

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [版本 10.0.15063]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
I want to bind socket in my program , compile and run it, it complains "bind: Operation not permitted".
But the same program run ok on Ubuntu.

main.cpp content is:
#include <stdio.h>
#include <stdlib.h>
#include <errno.h>
#include <string.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <sys/un.h>
#include <unistd.h>

#define SOCK_PATH "echo_socket"

int main(void)
{
	int s, s2, t, len;
	struct sockaddr_un local, remote;
	char str[100];

	if ((s = socket(AF_UNIX, SOCK_STREAM, 0)) == -1) {
		perror("socket");
		exit(1);
	}

	local.sun_family = AF_UNIX;
	strcpy(local.sun_path, SOCK_PATH);
	unlink(local.sun_path);
	len = strlen(local.sun_path) + sizeof(local.sun_family);
	if (bind(s, (struct sockaddr *)&local, len) == -1) {
	    perror("bind");
		exit(1);
	}
	return 0;
}

CMakeLists.txt content is:
cmake_minimum_required (VERSION 2.6)
project (main)
add_executable(main main.cpp)

commands:
cd socket-test
cmake .
make
./main

What's wrong / what should be happening instead:
It should not complains "bind: Operation not permitted".

Strace of the failing command, if applicable:  (If <cmd> is failing, then run strace -o strace.txt -ff <cmd>, and post the strace.txt output here)
See attached files.

strace.txt

@sinojelly - I don't have cmake, so compiled it with g++ and ran it; it made a file named echo_socket but print no messages, even bind: Operation not permitted or socket or bind. I inserted printf () before return 0 and that was printed. I mean I could not reproduce yours.
And then... I compiled test code from http://osr507doc.sco.com/es/netguide/dusockT.code_samples.html . It looked working well ether.

@sinojelly - Thanks for reporting the issue and the detailed repro steps. I was able to repro the issue. The problem seems to be in the bind operation of unix sockets in DrvFs i.e /mnt/<dir>. I will look further to see what is going on. But, this should work fine in LxFs i.e anywhere in your home directory. So, if you cd ~, copy the program there and run it, it should run fine. This is just a workaround, and, I understand that this is a bug nonetheless.
@sunjoong - Thanks for chiming in. The reason you were not able tor repro it is probably because you were trying it in 'LxFS. If you try it under /mnt/<c, d, etc..>`, you should be able to repro it. Also, just FYI, that unix domain sockets have nothing to do with Anti-virus or firewalls.

@sinojelly - I did not notice that's happened in /mnt/<dir>. Another workaround; changing SOCK_PATH  definition to #define SOCK_PATH "/tmp/echo_socket" could avoid that, you know.
@sunilmut - Thanks. I ran it within LxFS. I had not been able to belive it if someone said unix domain socket could be influenced with Anti-virus, before I saw my Anti-virus block to access file and/or directory named "ext". BTW, this is not about this issue but about other issue of last week... my Anti-virus vendor look like not to receive my report; I sent it to my provider but they look like not to send it to their contractor, i.e., the vendor, yet.

The problem seems to be in the bind operation of unix sockets in DrvFs i.e /mnt/<dir>

Marking this as a bug is cool, but are you really considering supporting this? I mean that would be balls, but... really?? You've got the same problem with say block/character device inodes (mknod(1)). Or named pipes (mkfifo(1)).  Support would would be interesting to be sure, but you gotta ask, what's the use case... That is, absent actually supporting those things in win32-land.

@therealkenc - The underlying issue here is that DrvFs does not has support for metadata. There are other scenarios (some, you have listed) that are blocked behind the same problem. If we want to provide compatibility for running Linux apps on DrvFS, this will be needed.

Awesome. I didn't know that was even on the table. Notwithstanding no-promises on timeframe caveats of course.

I'm running Docker on Windows (by connecting the linux version to the windows daemon), and one of the containers I'm trying to run needs to create a unix socket. It's running into the same error, so I'm assuming the containers are being run on DrvFs. Is there a workaround or way for me to set the containers to run in a way that it can create unix sockets?


No chat input history

Describe the bug
Just realise you couldn't click the up/down key to cycle through the last input message/command
To Reproduce
Steps to reproduce the behavior:
Enter a command for ex /help
Then press enter and then the up arrow nothing get copied
Expected behavior
Normally when you press enter then the up arrow it would automaticly copy the last entry, here /help

I haven't gotten down to the bottom of this yet, but I found if you hit Alt + Up/Down arrow, it will cycle. I'm not sure if this is a Prat feature or something Blizzard have added.

Amazing that you realise Alt was working, it's barelly an issue now

Will still need a clear explanation of why this is.

Does Up/Down arrow using default Blizzard chat cycle the history? Or is it Alt + Up/Down arrow
I vaguely recall Prat having a setting for whether you need to Alt + Up/Down or just Up/Down arrow.

Default UI requires pressing Alt to scroll through previously sent messages. Many Addons remove the requirement to hold Alt.

Looks like EditBox:SetAltArrowKeyMode(enable) might help here.
Unfortunately, it seems to be broken and only enables Left / Right navigation, Up / Down still requires the Alt key.

Thanks for the report all. I've opened #96 specifically for this new feature so I'm closing this now!


meson: 0.61.2 -> 0.62.2

Description of changes
This updates meson to the latest version, which was necessary for my upcoming PR to bump kmscon to the latest version. Several patches were removed due to being upstreamed.
HOWEVER: This definitely needs feedback, because running the nixpkgs review command below appears to fail, with the error message:
error: Function called without required argument "lzma" at /home/infinidoge/.cache/nixpkgs-review/rev-d55fcf6acf96287efc540014758f67fddbd4dc69/nixpkgs/pkgs/development/libraries/avro-c/default.nix:1, did you mean "lima", "czmq" or "emma"?

(Before that there was a very long list of packages)
I'm unsure what caused this, because the avro-c/default.nix mentions nothing about meson, but thought it worth noting here.
Due to the issue above, notifying maintainers: @jtojnar @AndersonTorres @brandonedens

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I think I may know the issue,
It might be possible that for some reason, this latest meson update conflicts with the fortify hardening, (kmscon and wayland-protocols, which both used meson, threw the same error about FORTIFY_SOURCE)
The issue there being that it seems like it is trying to compile with fortify without including the optimization level.
In both cases of kmscon and wayland-protocols, adding hardeningDisable = [ "fortify" ]; resolves the issue, but doing that tree-wide for meson-built software seems unwise.

lzma is an alias, replacing it with xz should fix the evaluation error.

This would need to go into staging and we already have https://github.com/NixOS/nixpkgs/pull/171069.

Please test the other PR and comment on any issues you see.

Ah, it seems I didn't see that PR, will retarget accordingly.


Remove static import statement that keep failing sporadically

@ao2017 do we have any idea why this is failing in this manner? A SO post or something?

@ao2017 do we have any idea why this is failing in this manner? A SO post or something?

@ao2017 do we have any idea why this is failing in this manner? A SO post or something?

SO post ???  I am not sure, this PR simply removes the static import statement

@ao2017 do we have any idea why this is failing in this manner? A SO post or something?

SO post ???  I am not sure, this PR simply removes the static import statement

Which failure in particular is this intending to resolve?

Which failure in particular is this intending to resolve?

Which failure in particular is this intending to resolve?

Failing to find distributions(), its occurs sporadically during compilation.

Which failure in particular is this intending to resolve?

Failing to find distributions(), its occurs sporadically during compilation.

Which failure in particular is this intending to resolve?

Failing to find distributions(), its occurs sporadically during compilation.

I'm still getting the error on my local against this branch.

Which failure in particular is this intending to resolve?

Failing to find distributions(), its occurs sporadically during compilation.

I'm still getting the error on my local against this branch.


Using limit and offset on findAll generates a syntax error

"sequelize": "^4.37.4",
"sequelize-typescript": "^0.6.3",
Trying to call this function:
export async function getData({limit=50, offset=0}){
	return await Data.findAll({limit, offset});
}

this works just fine with the defaults but if I try to pass in a offset=4 then I get a syntax error:
Executing (default): SELECT `id`, `log` FROM `Data` AS `Data` LIMIT '4', 50;

  SequelizeDatabaseError: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ''4', 50' at line 1
      at Query.formatError (C:\Users\Leonardo\Documents\movie-website\node_modules\sequelize\lib\dialects\mysql\query.js:247:16)
      at Query.handler [as onResult] (C:\Users\Leonardo\Documents\movie-website\node_modules\sequelize\lib\dialects\mysql\query.js:68:23)
      at Query.Command.execute (C:\Users\Leonardo\Documents\movie-website\node_modules\mysql2\lib\commands\command.js:30:12)
      at Connection.handlePacket (C:\Users\Leonardo\Documents\movie-website\node_modules\mysql2\lib\connection.js:502:28)
      at PacketParser.onPacket (C:\Users\Leonardo\Documents\movie-website\node_modules\mysql2\lib\connection.js:81:16)
      at PacketParser.executeStart (C:\Users\Leonardo\Documents\movie-website\node_modules\mysql2\lib\packet_parser.js:77:14)
      at Socket.<anonymous> (C:\Users\Leonardo\Documents\movie-website\node_modules\mysql2\lib\connection.js:89:29)
      at Socket.emit (events.js:182:13)
      at addChunk (_stream_readable.js:283:12)
      at readableAddChunk (_stream_readable.js:264:11)
      at Socket.Readable.push (_stream_readable.js:219:10)
      at TCP.onread (net.js:635:20)

@ljmerza Are you sure you've put 4(typeof 4 === 'number') in there? Looks like getData  get a '4'(typeof '4' === 'string') passed to it.

Yes that was exactly it thank you.


VerifyCode returns false / ensureCorrectTime issue

Hello,
I was trying to implement your library into my project.
The QR Code as well as the Code are being generated properly but I can't verify the Code.
After looking at the code I thought about, that the Time could be an issue. After trying the code
try {         $tfa->ensureCorrectTime();         echo 'Your hosts time seems to be correct / within margin';     } catch (RobThree\Auth\TwoFactorAuthException $ex) {         echo 'Warning: Your hosts time seems to be off: ' . $ex->getMessage();     }
I get the following error
Uncaught Exception: Connection refused
extern/lib/Providers/Time/NTPTimeProvider.php Zeile 40
Backtrace
#0 extern/lib/TwoFactorAuth.php(157): RobThree\Auth\Providers\Time\NTPTimeProvider->getTime()
#1 extern/2fa.php(39): RobThree\Auth\TwoFactorAuth->ensureCorrectTime()
#2 {main}
NTPTimeProvider.php
 /* Receive response and close socket */             if (socket_recv($sock, $recv, 48, MSG_WAITALL) === false)                 throw new \Exception(socket_strerror(socket_last_error($sock)));             socket_close($sock);
So it's probably an Time issue I can't verify.
Also do I set the discrepancy in the TwoFactorAuth.php
like this public function verifyCode($secret, $code, $discrepancy = 4, $time = null, &$timeslice = 0)
or in my file with
$Ergebnis= $tfa->verifyCode($secret, $_POST['verification'], $discrepancy);
Many regards
Lukas

The first issue sounds like the server isn't able to reach time.google.com. You can try a ping or something to see if it should be reachable, try a different NTP server or even an different TimeProvider.
About your second question: the TwoFactorAuth class has no $discrepancy constructor argument? It does have a $period argument, but those two things are totally different. The $period determines how long a code is valid (30 seconds by default) and the $discrepancy determines how many periods the client device may be "off" time-wise. So for a $discrepancy of 4 the code will be valid 4 + 4 (both ways) * 30 seconds = 4 minutes. Or: your client device may be "off" by 2 minutes in either direction.
I would like to point out that all of this is documented in the README.

So I deleted the NTPTimeProvider in the TwoFactorAuth.php although the Google Server responds to a Ping.
After that I got a response that the host time seems to be correct.
I still just get false as a response although I'm copying the generated code into my form, so I can't really figure it out why it doesn't work.

are you able to manually verify the time on your computer/server is correct for your timezone? If you're on mac or linux you could compare the output of the date command in a terminal with https://time.is. Windows has a sync button somewhere for it.

And if that doesn't help, please show us your code or at least tell us the configuration (period, digits, algorithm) and which authenticator app you're using.

So apparently the Server time is about 4 minutes off.
I tried to solve it by setting the discrepancy to 10 but still not working.
The code can be found below.
require_once 'loader.php';
Loader::register('lib','RobThree\\Auth');

use \RobThree\Auth\TwoFactorAuth;

$tfa = new TwoFactorAuth('Service Portal)');

$secret = $tfa->createSecret(160);

echo 'getQRCodeImageAsDataUri('SAP', $secret) . '">
'.chunk_split($secret, 4, ' ');
$code = $tfa->getCode($secret);
echo $code; // Displaying code for testing reasons

<form action="?verify=1" method="POST">

		<input type="number" name="verification">
		<button class="regBtn" type="submit">Senden</button>

</form>

if (isset($_GET['verify'])) {
            $result = $tfa->verifyCode($secret, $_POST['verification'], $discrepancy);
		    	if ($result) {
				die('<meta http-equiv="refresh" content="0; URL=dashboard">');
		    	} else {
		    		echo "Es ist leider ein Fehler aufgetreten, bitte versuchen Sie es noch einmal.";
		    	}
 }

Using Google Authenticator with standard configuration (except $discrepancy = 10)

Google Authenticator won't handle anything but 80 bits / 30 seconds / sha1 (e.g. default configuration). So instead of $tfa->createSecret(160); try $tfa->createSecret();

$_POST['verification'] contains the right information,
Also not working with 80.
$result = $tfa->verifyCode($secret, $_POST['verification'], $discrepancy); I do here, but I don't know if this is correct.

Does the demo work for you?

Yes, it shows OK as a result

Then can't you try and figure out what the difference between the two is and narrow it down?
require_once 'loader.php';
Loader::register('../lib','RobThree\\Auth');

use \RobThree\Auth\TwoFactorAuth;

$tfa = new TwoFactorAuth('MyApp');

$secret = $tfa->createSecret(160);
$code = $tfa->getCode($secret);
$result = $tfa->verifyCode($secret, $code) === true;

This is the relevant code from the demo. There must be some difference in the code you're using. Also try adding error_reporting(E_ALL); ini_set("display_errors", 1); as very first line if you don't have it already.

So Error reporting displayed Notice Undefined variable: discrepancy
in Line 113 $result = $tfa->verifyCode($secret, $_POST['verification'], $discrepancy);

So I checked everything and apparently it's an POST Error because when I check using the $code Variable everything it's working perfectly fine. But I checked and the code I'm posting is the same.

I'm sorry but I can't help you with that. The problem is obviously in your code and not in TwoFactorAuth.
Also, judging from the Notice Undefined variable: discrepancy inLine 113 the code you've posted here is not the code you're actually using so that makes it hard for us to debug the issue.
For a quick'n'dirty check to see if TwoFactorAuth works correctly you could simply replace $discrepancy on the aforementioned line 113 with a constant (like 3 or 4 or ...)

So yes I got the Error resolved with the $discrepancy.
What exactly expects the $code?
I'm entering a string, so this should be fine, if the code works if I take the $code variable because they are the same.

I kindly request you refer to the documentation and/or demo. This question is answered there.

Okay figured it out, thanks!

@luggesexe: It would be nice if you posted what the issue was so that others can learn from it too.

I called ?verify=1. That generated a new secret so it was always false.
Figured it out by creating a new page where I checked the two belonging POST Variables.


Add St Peters for registration

@Zxkq1 Please provide us with:
- the school official website URL
- the school street address, including city and country
- an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
- a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

On Mon, 23 May 2022 at 08:07, Philip Torchinsky @.***>
wrote:

@Zxkq1 https://github.com/Zxkq1 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/14212#issuecomment-1134266442,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURRGKQPOIYZKUDRHIWOIRDVLMVDHANCNFSM5WNDDXVQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

https://www.st-peters.bournemouth.sch.uk
St Peter’s School, St Catherine’s Road, Bournemouth BH6 4AH
Unfortunately I cannot show the IT course since they do not list the GCSE
Options on the website (I don’t think).
I have my own personal email given by the school for the school (
@.***)

@Zxkq1  Thank you for the clarifications.
Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

On Mon, 23 May 2022 at 08:22, Philip Torchinsky @.***>
wrote:

@Zxkq1 https://github.com/Zxkq1 Thank you for the clarifications.
Please provide us with a proof, which shows that the school recognizes the
domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you
by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that
students use this email domain
A reference to a recognized journal where an article was published
by someone from the school, with explicit affiliation to the school, and
e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that
the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be
verified independently.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/14212#issuecomment-1134281497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURRGKVDJ6L4764R4WMEZIDVLMW3FANCNFSM5WNDDXVQ
.
You are receiving this because you were mentioned.

You can probably find some information here:
https://www.st-peters.bournemouth.sch.uk/attachments/download.asp?file=466&type=pdf
Under the subtitle: ICT Acceptable Use Policy.

Message ID: @.***>

@Zxkq1 Regrettably, I did not find stpetersca.com mentioned anywhere in the document. Would you mind providing us with another proof? Thank you!

On Mon, 23 May 2022 at 08:54, Philip Torchinsky @.***>
wrote:

@Zxkq1 https://github.com/Zxkq1 Regrettably, I did not find
stpetersca.com mentioned anywhere in the document. Would you mind
providing us with another proof? Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/14212#issuecomment-1134312518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURRGKXTYZQEIWSTTDRGYDTVLM2UTANCNFSM5WNDDXVQ
.
You are receiving this because you were mentioned.

Unfortunately there isn’t much more I can do because the email domain is
different. Is there a way to accept without it? I have my own school email
but I’m guessing that’s not enough.

Message ID: @.***>

On Mon, 23 May 2022 at 13:19, Zak Toms @.***> wrote:

On Mon, 23 May 2022 at 08:54, Philip Torchinsky @.***>
wrote:

@Zxkq1 https://github.com/Zxkq1 Regrettably, I did not find
stpetersca.com mentioned anywhere in the document. Would you mind
providing us with another proof? Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/14212#issuecomment-1134312518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURRGKXTYZQEIWSTTDRGYDTVLM2UTANCNFSM5WNDDXVQ
.
You are receiving this because you were mentioned.

Unfortunately there isn’t much more I can do because the email domain is
different. Is there a way to accept without it? I have my own school email
but I’m guessing that’s not enough.

Is there anything I can do? I really really want this.

Message ID: @.***>

Please; is there anything? These are some of the best programming software I've ever seen.

@Zxkq1 Would you please ask your teacher to confirm in writing that students at your school use the domain stpetersca.com for their emails, and this domain is available to the students and teachers of the school only? If there are other people who may have an email address ending with @stpetersca.com, please let me know.
You can upload a copy of the note from your teacher to this request. Please use your browser to do it, DO NOT send an attachment by email. GitHub strips out all attachments sent by email.
Thank you.

On Wed, 25 May 2022 at 21:54, Philip Torchinsky @.***>
wrote:

@Zxkq1 https://github.com/Zxkq1 Would you please ask your teacher to
confirm in writing that students at your school use the domain
stpetersca.com for their emails, and this domain is available to the
students and teachers of the school only? If there are other people who may
have an email address ending with @stpetersca.com, please let me know.
You can upload a copy of the note from your teacher to this request.
Please use your browser to do it, DO NOT send an attachment by email.
GitHub strips out all attachments sent by email.
Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/14212#issuecomment-1137838735,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURRGKXLQFJAQ5OPWS2DJ3DVL2HPRANCNFSM5WNDDXVQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yep, so do I just ask them to send me an email of proof that we can use it?
Or just a hand written letter?

@Zxkq1 An email is OK if the teacher agrees to expose their email address in the public comment on GitHub. Alternatively, the teacher can send the email directly to<EMAIL_ADDRESS>with Subject: ATTN Philip Torchinsky. The email has to say that stpetersca.com domain belongs to your school.

I've just sent an email asking them to send you it, hopefully it will be alright.

@Zxkq1 Do you have any update on the matter?

On Fri, 3 Jun 2022 at 12:15, Philip Torchinsky @.***>
wrote:

@Zxkq1 https://github.com/Zxkq1 Do you have any update on the matter?
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/14212#issuecomment-1145860046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURRGKUBZDM5V3SDIGTENHLVNHSOFANCNFSM5WNDDXVQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yep, the teacher will send over the email some time soon, if it doesn’t
happen I’ll remind him. Is there a way to add it beforehand or do you have
to specifically wait for the confirmation?

@Zxkq1 Pull request merged. Please remind the teacher to send the message. I will monitor the domain usage, and if there will be some issues before the official confirmation letter from your teacher, I'll let you know. I believe everything has to be OK. Thank you.

On Mon, 23 May 2022 at 08:31, Zak Toms @.***> wrote:

On Mon, 23 May 2022 at 08:22, Philip Torchinsky @.***>
wrote:

@Zxkq1 https://github.com/Zxkq1 Thank you for the clarifications.
Please provide us with a proof, which shows that the school recognizes
the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you
by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that
students use this email domain
A reference to a recognized journal where an article was published
by someone from the school, with explicit affiliation to the school, and
e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying
that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be
verified independently.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/14212#issuecomment-1134281497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURRGKVDJ6L4764R4WMEZIDVLMW3FANCNFSM5WNDDXVQ
.
You are receiving this because you were mentioned.

You can probably find some information here:
https://www.st-peters.bournemouth.sch.uk/attachments/download.asp?file=466&type=pdf
Under the subtitle: ICT Acceptable Use Policy.

Or this

Message ID: @.***>


add CI check for valid import formatting

🚀 Feature
add a CI checker for valid import formatting - three section: build-in, third party, and this package
note that all "unused" imports in init shall be ignored and they are meant such a way...
Motivation
We shall follow pep8 recomendation

@Borda I'm up for adding this, since I've already done this for my own project based on PL :wink:


IGNITE_PLUGINS_PATH shouldn't be required

#772 introduced IGNITE_PLUGIN_PATH and required it. However, I don't really think this should be required -- we should infer the path if not found. This results in a bad DX (IMO) when first spinning up an Ignite app.

It was only in place so we can make everything work while in dev mode.  The packages will all live on npm.

Just a heads up, this message only appears when the package isn't found on npm.
The resolution order for ignite add something goes like this:

look for a directory called something (can be relative or absolute)
look for a directory called something in any path found on IGNITE_PLUGIN_PATH
look on npm

If all these fail you see that message.
Once we publish ignite-ir-app, we won't see this error anymore.

I knew that.


Open DataSource without locking it

This allow to keep the datasource file writable by the application generating it while parsing.
(Sorry for the online editing of github screwing file encoding)

Good call.  I'll get this merged in, along with a contributor file.  The latest by the end of the week hopefully.


tests: add ips midstream exception policy tests - v2

Add tests for the exception policy for midstream streams (https://redmine.openinfosecfoundation.org/issues/5468)
Previous PR: #907
exception-policy-midstream-01: tests pass-flow behavior with midstream SYNACK
exception-policy-midstream-02: tests drop-flow behavior with midstream SYNACK
exception-policy-midstream-03: tests the default behavior with midstream SYNACK
exception-policy-midstream-04: tests the pass-flow behavior with midstream ACK
exception-policy-midstream-05: tests the bypass behavior with midstream SYNACK
exception-policy-midstream-06: tests drop-flow behavior with midstream ACK
exception-policy-midstream-07: tests bypass behavior with midstream ACK
Tests 06 and 07 are currently failing, I think this is related to the behavior described in
https://redmine.openinfosecfoundation.org/issues/5510, still to be investigated.

Thanks, I think I still need to better wrap this whole concept in my head...

Replaced by: https://github.com/OISF/suricata-verify/pull/921


Use C99 types in decompiler

I suppose this part was written when C/C++ compilers were not as good as today. These days, even MSVC suppports C99 standard, thus lets reduce the code churn and easier future porting to new platforms.

What's the progress of this?

@Escapingbug I am closing this in favor of https://github.com/NationalSecurityAgency/ghidra/pull/2609
I really hope it will be merged soon.


Add ToStdio functionality to emitter

Have emitter.NewDir and emitter.NewFile return a new Emitter interface, which
is a zbuf.WriteCloser along with a function that returns true if the emitter is
writing to stdio (i.e. stdout or stderr). This can be used to write command
line functionality that changes depending on where zng records are being output
to.

I added this because I wanted to make brimcap analyze write stats updates to stderr if records are not getting sent to stdout. I think this is useful information.

Going another direction with this. Closing.


Multiple CPUs

Is it possible to train an agent with multiple cores?

hi,
dopamine currently doesn't support that, although i've heard of people who
have modified it to be able to do so (i'm not aware of the details, though).
On Sun, May 5, 2024 at 7:52 AM Sanyam Vyas @.***> wrote:

Is it possible to train an agent with multiple cores?
—
Reply to this email directly, view it on GitHub
https://github.com/google/dopamine/issues/220, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3CCMO5G7HX6P7JESYR3MTZAYMQDAVCNFSM6AAAAABHHU7JPSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TSNBYHEZDQNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Timelines to make unitycatalog to feature complete for production usecases

Is your feature request related to a problem? Please describe.
As of now current version do not support production scale as it uses in memory database(H2) for persistence.
Another feature is front end component for unity catalog(similar to databricks version)
Describe the solution you would like
Please fill the gaps between opensource and databricks version.

Thanks @krisnaru - could you please create a separate item for the front-end UI so we can track this separately as there are currently two initiatives that we can track with it.
As for the database, yes - H2 is a great lightweight Java database so it’s a solid way to start.  We will start reviewing database options in the near future so will keep this issue open to continue to track its progress.

Feel free to track the project milestones for details on features and expected completion dates!


Should fix so arrays can be updated programatically.

Right now, if you try to update the values of the sliders programatically - like updating the value with a function - it works for the single value sliders. However, with more values in an array, it seems the change is not detected and the visual slider is not updated. This should fix the problem.
I'm not quite sure of the procedure for helping out here, so I just made this request. If you need anything from me, for this change, please let me know. Or if I'm just plain wrong about something here :-)

s-sasaki-0529, i forgot to add you as reviewer, and I can't edit the PR it seems, so I'm gonna shamelessly tag you here. Sorry!

@jesperlauridsen Shipped v0.1.5


fix: 将 anyproxy <IP_ADDRESS>:8001 端口作为单独的代理（比如 omega）的时候报错

see #419 #415 #291 #258 #206 #203

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi，我没能理解这个bug是怎么产生的。看关联的issue也没明白....是怎么配置代理导致的？

一个尝试性修复： https://github.com/alibaba/anyproxy/commit/68188c9cd6e2db5e9e474af81e9fba6e90dc596a


BUG: quantile (approxHistogram) not produce correct result

In my product system, the quantile=0.5 result is very far from real value. (even if I set the resolution to a relative high value) for e.g.:
ingest:
                        {
                            "type": "approxHistogramFold",
                            "name": "hist_fold_api_duration",
                            "fieldName": "api_duration",
                            "resolution": 10000,
                            "lowerLimit": 0,
                            "upperLimit": 60000
                        }

query
{
    "queryType": "timeseries",
    "dataSource": "xxx",
    "granularity": "all",
    "intervals": [
        "2017-02-06T00:00:00+08:00/2017-02-07T00:00:00+08:00"
    ],
    "aggregations": [
        {
            "type": "approxHistogramFold",
            "name": "hist_fold",
            "fieldName": "hist_fold_api_duration",
            "resolution": 10000,
            "numBuckets": 100
        }
    ],
    "postAggregations": [
        {
            "type": "quantile",
            "name": "myResult",
            "fieldName": "hist_fold",
            "probability": 0.5
        }
    ],
    "context": {}
}

expected real value: 190
druid result: 1167.81 (I tried multiple resolution in ingest time, druid result is near this value)
I attach the api_duration dataset, in case someone can reproduce or fix the bug.
api_duration.txt

I see this behavior as well in both druid 0.10.0 and 0.10.1. In my dataset, I get 0.95/0.98 quantile values exceeding that of max (whether from doubleMax or max on the approxHistogramField).
In my particular case, the mem fields were returning odd quantile values (but cpu and disk were fine).
sample-dataset.csv.txt
sample-index.json.txt


Error: Couldn't find preset "es2015"

I followed the install instructions, although sourcing from a TypeScript file
var NotificationBadge = require('react-notification-badge')
and I'm getting the following error:
Error: Couldn't find preset "es2015" relative to directory "C:\\Repos\\foo\\node_modules\\react-notification-badge" while parsing file: C:\Repos\foo\node_modules\react-notification-badge\index.js

Any ideas? Other modules sourced the same way work just fine.
I'm transpiling TypeScript to javascript with browserify/watchify, using the following NPM task:
watchify src/main.ts -v -t [ brfs ] -p [ tsify ] --debug -t babelify -x react -x react-dom -x react-router -x underscore -o scripts/bundle.js

Hi @anjdreas
Please try this.
http://babeljs.io/docs/plugins/preset-es2015/
Sorry I'm not familliar to TypeScript,
But In my case, I create package.json like below. And it works fine.
{
  "name": "test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "start": "watchify index.js -o main.js -dv",
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "MIT",
  "dependencies": {
    "react": "^0.14.3",
    "react-dom": "^0.14.3",
    "react-notification-badge": "^1.0.0"
  },
  "devDependencies": {
    "babel-preset-es2015": "^6.1.18",
    "babel-preset-react": "^6.1.18",
    "babelify": "^7.2.0",
    "browserify": "^12.0.1",
    "watchify": "^3.6.1"
  },
  "browserify": {
    "transform": [
      [
        "babelify",
        {
          "compact": false,
          "presets": [
            "react",
            "es2015"
          ]
        }
      ]
    ]
  }
}

Thanks, I'll give it a try.

Hi @anjdreas
I also pushed transpiled code from v1.1.0.
Please try it.


Initial stab at #251

Needs rebase onto rust

My intent with this PR was that it address #251 (the global imports one, which Jordan has now anyway) but the PR contains the changes in the branch on which this was based. So I'll close this, and address the comments in https://github.com/mozilla/mentat/pull/249


timeout issue

I have the same problem since today although everything was working correctly before it was impossible to enter my server

Try reinstalling or reverting back to default configs. Not able to reproduce this


Fix for issue #548

Added list of GI bill approved schools to home page

Hi Patrick,
Thanks so much for your PR. Before we merge this do you mind using the file config/code_schools.yml to generate this list? It might even make sense to convert the VA approved schools portion on /code_schools to a partial to DRY things up

Hi @patrickmollohan - have you had a chance to update this PR?

Closing due to inactivity. Please feel to resubmit!


[docs] Fix wrong migration

A regression from https://github.com/mui/material-ui/pull/42498
I think this case was my manual adjustment mistake. I missed the theme.breakpoints.up. The codemod improvement will come in a separate PR.

[x] I have followed (at least) the PR section of the contributing guide.

Can you do screenshots of what you are trying to fix? It would help with the review.