fix(unocss): update filetypes

update filetypes of unocss lsp
Based on:
https://github.com/unocss/unocss/blob/35297359bf61917bda499db86e3728a7ebd05d6c/packages/vscode/src/autocomplete.ts#L12
https://github.com/neovim/nvim-lspconfig/blob/dd329912c8d446240584a2dbcd3802af3a19105a/lua/lspconfig/server_configurations/tailwindcss.lua

I am confused by similar behavior. Although this improves the out-of-the-box capabilities, it also bloats the table here. Many filetypes are not even in our built-in filetype support list. Maybe we should provide an append interface to achieve similar behavior instead of bloating it here.

@glepnir Thaks
So how can I do this.
Actually it is okay to configure by myself.
Or just add astro is fine for me.

Yes. Most of the time we recommend that users modify it locally. However, if the file type is supported by the built-in, it may be acceptable.

OK, so in this PR, I just add the ft which are listed in the official vscode extension

@glepnir Please check this out.

needs a rebase since https://github.com/neovim/nvim-lspconfig/pull/3330

Rebased done!

try moving the file to lua/lspconfig/configs/unocss.lua

@justinmk Hi! I just updated


Community: Update and fix ZenGuardTool docs and add ZenguardTool to init files

Thank you for contributing to LangChain!

[x] PR title: "community: update docs and add tool to init.py"

[x] PR message:

Description: Fixed some errors and comments in the docs and added our ZenGuardTool and additional classes to init.py for easy access when importing
Question: when will you update the langchain-community package in pypi to make our tool available?

[x] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Thank you for review!

Question: when will you update the langchain-community package in pypi to make our tool available?

Should be released now, in langchain-community==0.2.6


timed out waiting for interface matching the management IP (<IP_ADDRESS>) of network Calico，windows node

Expected Behavior

allocate ip ok
Current Behavior

firewall is closed in windows
root@k8s-master01:~# kubectl get node
NAME              STATUS   ROLES           AGE   VERSION
k8s-master01      Ready    control-plane   15d   v1.31.0
k8s-worker01      Ready    <none>          15d   v1.31.0
k8s-worker02      Ready    <none>          15d   v1.31.0
win-f1njfokn3tm   Ready    <none>          27m   v1.31.0

  Warning  FailedCreatePodSandBox  4s    kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "cf8199556fa7cd64da0971c6dfa030be43b42d0e9c20a2ccc8fb9c4831bc395b": plugin type="calico" name="Calico" failed (add): timed out waiting for interface matching the management IP (<IP_ADDRESS>) of network Calico

apiVersion: v1
kind: Pod
metadata:
  name: windows-pod
spec:
  containers:
  - name: windows-container
    image: mcr.microsoft.com/windows/servercore:ltsc2019
    command: ["powershell.exe", "-Command", "while($true) { Write-Host 'Hello, Windows!' -ForegroundColor Green; Start-Sleep -Seconds 1 }"]
  nodeSelector:
    kubernetes.io/os: windows
  os:
    name: windows

Possible Solution

Steps to Reproduce (for bugs)

1.install windows node
2.apply the yaml above
3.
4.
Context

Your Environment

Calico version v3.28.0
Orchestrator version (e.g. kubernetes, mesos, rkt):k8s 1.31.0
Operating System and version: ubuntu2404
Link to your project (optional):

do we need other cni plugins

cni error
2024-09-17 16:40:08.573 [INFO][5812] dataplane_windows.go 431: Found existing HNS network [&{Id:C84DDB4C-022B-421A-89D5-6FBC2F5C347C Name:Calico Type:L2Bridge NetworkAdapterName: SourceMac: Policies:[] MacPools:[{StartMacAddress:00-15-5D-51-70-00 EndMacAddress:00-15-5D-51-7F-FF}] Subnets:[{AddressPrefix:<IP_ADDRESS>/26 GatewayAddress:<IP_ADDRESS> Policies:[]}] DNSSuffix: DNSServerList: DNSServerCompartment:0 ManagementIP:<IP_ADDRESS> AutomaticDNS:false}] ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0"
2024-09-17 16:40:08.860 [WARNING][5812] dataplane_windows.go 639: Waiting for interface matching management IP... ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0" error=no index found for interface "" ip="<IP_ADDRESS>" network="Calico"
2024-09-17 16:40:10.158 [WARNING][5812] dataplane_windows.go 639: Waiting for interface matching management IP... ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0" error=no index found for interface "" ip="<IP_ADDRESS>" network="Calico"
2024-09-17 16:40:11.599 [WARNING][5812] dataplane_windows.go 639: Waiting for interface matching management IP... ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0" error=no index found for interface "" ip="<IP_ADDRESS>" network="Calico"
2024-09-17 16:40:12.888 [WARNING][5812] dataplane_windows.go 639: Waiting for interface matching management IP... ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0" error=no index found for interface "" ip="<IP_ADDRESS>" network="Calico"
2024-09-17 16:40:14.176 [WARNING][5812] dataplane_windows.go 639: Waiting for interface matching management IP... ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0" error=no index found for interface "" ip="<IP_ADDRESS>" network="Calico"
2024-09-17 16:40:15.498 [WARNING][5812] dataplane_windows.go 639: Waiting for interface matching management IP... ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0" error=no index found for interface "" ip="<IP_ADDRESS>" network="Calico"
2024-09-17 16:40:16.798 [WARNING][5812] dataplane_windows.go 639: Waiting for interface matching management IP... ContainerID="364be85717e1fed8d4c159e46ae0f42bf5d3cd6acfd456d49c9cab8130bfeb4c" Namespace="default" Pod="windows-pod" WorkloadEndpoint="win--f1njfokn3tm-k8s-windows--pod-eth0" error=no index found for interface "" ip="<IP_ADDRESS>" network="Calico"

cni config
{
			  "name": "Calico",
			  "cniVersion": "0.3.1",
			  "plugins": [{"DNS":{"Nameservers":["<IP_ADDRESS>"],"Search":["svc.cluster.local"]},"capabilities":{"dns":true},"datastore_type":"kubernetes","ipam":{"subnet":"usePodCidr","type":"calico-ipam"},"kubernetes":{"k8s_api_root":"https://<IP_ADDRESS>:6443","kubeconfig":"c:/etc/cni/net.d/calico-kubeconfig"},"log_file_max_age":30,"log_file_max_count":10,"log_file_max_size":100,"log_file_path":"c:/var/log/calico/cni/cni.log","log_level":"Info","mode":"windows-bgp","mtu":0,"name":"Calico","nodename":"win-f1njfokn3tm","nodename_file":"c:/var/run/calico/nodename","nodename_file_optional":true,"policies":[{"Name":"EndpointPolicy","Value":{"ExceptionList":["<IP_ADDRESS>/12"],"Type":"OutBoundNAT"}},{"Name":"EndpointPolicy","Value":{"DestinationPrefix":"<IP_ADDRESS>/12","NeedEncap":true,"Type":"SDNROUTE"}}],"policy":{"type":"k8s"},"type":"calico","vxlan_mac_prefix":"0E-2A","vxlan_vni":4096,"windows_loopback_DSR":true,"windows_use_single_network":true}]
			}

@13567436138 from your screenshot I can see you're using a non-English version of windows, any chance the interface name has non-English characters? If so, there currently is an issue with supporting those, it was fixed in https://github.com/projectcalico/calico/pull/9062 and will be released on Calico v3.29.0, but for now the work around is to config windows to use English...

When will v3.29.0 release


from Finder import finder - cannot import name 'finder'

So this command copied from real_data.ipynb does not work.
But from Finder import Finder does work, if that helps (this was the spelling for the class in Finder.py)

I think you forgot to include the command which does not work (?)

Hi Andreas,
Good to be in touch.
Sorry, I should have in included it again, not just in the title - The command that did not work was
from Finder import finder
But using from Finder import Finder instead fixes this, and then I was able to run the algorithm on my own data, which is interesting, thanks.
Can you explain how FINDER compares with OPTICS (e.g. in scikit-learn)?
Best wishes,
Alistair
From: Andreas Nold @.>
Sent: 04 January 2023 22:24
To: NoldAndreas/FINDER @.>
Cc: Alistair Curd @.>; Author @.>
Subject: Re: [NoldAndreas/FINDER] from Finder import finder - cannot import name 'finder' (Issue #4)
I think you forgot to include the command which does not work (?)

Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FNoldAndreas%2FFINDER%2Fissues%2F4%23issuecomment-1371498985&data=05|01|a.curd%40leeds.ac.uk|b1155157cc2744ccc7b108daeea255ab|bdeaeda8c81d45ce863e5232a535b7cb|1|0|638084678234329018|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Z4Y1SW08Q5qbJd6FJz3qvuTQaxlxNHKeOcG5uKY7J14%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK5D4DTDNOQUOFT3VZPIOB3WQXZ6ZANCNFSM6AAAAAATRAUMIU&data=05|01|a.curd%40leeds.ac.uk|b1155157cc2744ccc7b108daeea255ab|bdeaeda8c81d45ce863e5232a535b7cb|1|0|638084678234329018|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=UImznB%2BxVraAVsR11vDl%2Fi%2FSnDyPsUYEyXV8oXXCdTs%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

I did not find any command in our code where "from Finder import finder" was used with lower case. Can you check again?
Concerning the difference to OPTICS: In essence, OPTICS dynamically adapts the clustering criteria to the density of the local neighbourhood of the points. This means it can detect variations with dense and less dense regions of the same image. Here, we assume that the conditions/ the background noise is uniform across the image, and propose a method to determine suitable global parameters for a given image.

Thanks. I can't find that command in the notebooks now, either. Maybe it was something I created or altered, sorry if so!
From: Andreas Nold @.>
Sent: 06 January 2023 16:18
To: NoldAndreas/FINDER @.>
Cc: Alistair Curd @.>; Author @.>
Subject: Re: [NoldAndreas/FINDER] from Finder import finder - cannot import name 'finder' (Issue #4)
I did not find any command in our code where "from Finder import finder" was used with lower case. Can you check again?
Concerning the difference to OPTICS: In essence, OPTICS dynamically adapts the clustering criteria to the density of the local neighbourhood of the points. This means it can detect variations with dense and less dense regions of the same image. Here, we assume that the conditions/ the background noise is uniform across the image, and propose a method to determine suitable global parameters for a given image.

Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FNoldAndreas%2FFINDER%2Fissues%2F4%23issuecomment-1373856482&data=05|01|a.curd%40leeds.ac.uk|fe872a74940b4515879d08daf001a3d5|bdeaeda8c81d45ce863e5232a535b7cb|1|0|638086187081470811|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Ik2%2BF7waNSSfBl9xHQJigWSWQ%2FbK2%2BqNCEnJ1msgO1w%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK5D4DRDAJG6UGMU5ALSHODWRBAVBANCNFSM6AAAAAATRAUMIU&data=05|01|a.curd%40leeds.ac.uk|fe872a74940b4515879d08daf001a3d5|bdeaeda8c81d45ce863e5232a535b7cb|1|0|638086187081470811|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=HqpVlKd6LXBKUp%2F3SozkzX%2BQ58fis95lgtRD5dx4%2Ffg%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


Linux: meson.build:367:22: ERROR: C++ shared or static library 'soci_sqlite3' not found

I am building this HDA and have had some trouble when building it on Linux x64.
I am totally new to the whole C++ build process and fiddled with C++ about 10+ years ago.
I installed all the packages and updated the triplet + the vcpkg home.
I run the following command: ./buildall.sh on Linux.
Compiler stderr:

Library drogon found: YES
Library trantor found: YES
Library jsoncpp found: YES
Library ssl found: YES
Library crypto found: YES
Library brotlicommon-static found: YES
Library brotlienc-static found: YES
Library brotlidec-static found: YES
Library cares found: YES
Library z found: YES
Library fmt found: YES
Library sqlite3 found: YES
Library soci_core found: YES

meson.build:367:22: ERROR: C++ shared or static library 'soci_sqlite3' not found

ln 367 of meson.build
libsoci_sqlite3 = cpp.find_library(
    soci_sqlite3_name,
    dirs: [vcpkg_root / 'packages/soci_' + triplet + '/lib'],
)

The packages in that directory are:

Solution: :heavy_check_mark:
I had to rebuild the vcpkg for some odd reason and that solved the problem, but now I am having a Criterion library not found.
vcpkg install soci[sqlite3] --recurse rebuilt everything and fixed the issue.

The criterion error:
meson.build:390:19: ERROR: C++ shared or static library 'criterion' not found

ln 390 of meson.build
# Criterion test library

libcriterion_name = 'criterion'
libcriterion = cpp.find_library(
    libcriterion_name,
    dirs: [criterion_libs],
)

The ninja build error:
[14/14] Linking target demo_web_server
...
c.o): in function `drogon::utils::getUuid[abi:cxx11]()':
Utilities.cc:(.text+0x905): undefined reference to `uuid_generate'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

The PR has been created @brakmic https://github.com/brakmic/HDA_with_Cpp/pull/5

Many thanks for your patches! I merged them.


Assignment 5-B vision

@wvictor14 Here's where I'll write my vision for Assignment 5-B.
I have a few pieces of the vision, based on today's discussions, but I'll elaborate more later.

Assignment should have three options: (1) a "default" option that works with stringr and regex, (2) if they are keen on further developing their R package (started in Assignment 2-B), they can do so, and (3) if they are keen on developing their shiny app (started in Assignment 3-B), they can do so.
Options for further developing the R package:

They could simply just add more functionality.
Impressive would be if they make an S3 object oriented package.
Contribute to another R package, like broom.

Maybe a fourth option would be inspiring to students: (4) contribute to an open source analysis or R package or shiny app.
Maybe this can generally be about contributing to the R community. Maybe they can be graded on their impact somehow. Other examples:

Present a topic in a community group like R Ladies or the UBC study group.
Contribute to Stack Overflow's R topics.

The one unfortunate thing about this is that, if we do decide to add this option, it would have sure been nice to have this available to the students at the start of the course so that they can work on it throughout 545B! There's always next year...

OK, here's a summary of where we're at.
Options

Do an assignment we give you about character data.
Improve your R package from Assignment 2-B.
Improve your shiny app from Assignment 3-B.
Contribute to the open source R community.

Ideas for the Rubrics
Option 2 + 3
This time, we'll be looking at design.

Describe the purpose of your app/package. Rubric based on usefulness and depth of purpose. ("depth" means that you need to seek out something more than the modest additions required by the original assignment -- but don't make this too ambitious so that you can't get it done in time!).
Design: is your app/package designed in a user-friendly way? Or are there some rough patches that interrupts the user's flow?

Option 4
Make 3 additions to the R community:

A post on a Stack Overflow R question = 1 addition
Reporting a bug or improvement request to an R package as a GitHub Issue in a clear and reproducible way = 1 addition
Submitting a pull request to improve an R-related resource = 1 addition

Too small: something like fixing some typos in the R4DS book doesn't count
Too much: something like giving a complicated function like lm a facelift is far too much
Just right: adding a condition to throw an error message when a function ought to, and justifying it in the pull request.

Delivering a presentation/seminar in a public/UBC R group = 3 additions (record it so we can see and evaluate your effectiveness of communication)

What now?

@vincenzocoia drafts a vision for a character data assignment. Perhaps drawing inspiration from the R4DS strings chapter.
@wvictor14 Can you provide some feedback on Options 2-4, specifically my ideas for the rubrics? Some things to watch out for: does anything look overly simplistic to you? Overly demanding? Not grade-able? Please do suggest improvements if you see any.
@wvictor14 A more high-level question -- is making an assignment like this overly ambitious? Should we dial it down a notch this year? Sometimes I just need a reality check.

Option 2 + 3
This time, we'll be looking at design.
Describe the purpose of your app/package. Rubric based on usefulness and depth of purpose. ("depth" means that you need to seek out something more than the modest additions required by the original assignment -- but don't make this too ambitious so that you can't get it done in time!).
Design: is your app/package designed in a user-friendly way? Or are there some rough patches that interrupts the user's flow?

For the R package option at least, maybe we can require an informative README that describes and demonstrates their package. Things to include in this README would be like

examples of package core functions execution
examples of package demo execution
some benchmark results

and we can evaluate them based on 1. the quality of the additional functions and/or code changes to existing functions, and 2. quality of the README
Some other ideas:

What if we make them go through the process of what an update to an existing R package is like. For example, they could post an issue about an existing problem or challenge with the package, like missing functionality /features, bugs /errors, missing documentation. Then, they must create a PR (maybe multiple depending on number of unique issues) that address those issues. Lastly update the version.
They can version/tag assignment 2b as v0.1.0 and then this assignment as v0.2.0. And we can evaluate their changes as well as how well they document these (major changes described in NEWS.md?).
Maybe require turning their package into github pages if they haven't?

For the app, I feel like we could leave the idea of what is an "improvement" open-ended and let them decide? Then have somewhere where they can explain why it's an improvement and how they did so. Maybe in a PR, or in a tab in the app itself, or the readme on the github repo...

We can make the evaluation more weighted towards effort and thoughtfulness than execution
For marking, what if they keep the shiny app for assignment 3b deployed, but also deploy this updated 4b version so we can easily compare in side-by-side browser tabs (just easy for us to mark)

I like option 4, and it makes sense. But no one did bonus question 1.4 in assignment 1b (create a PR on an existing public R package). I feel like very few would feel able to do this option. I think many might just be intimidated by the idea? I feel contributing to "big name" packages like from tidyverse is a bit scary, maybe you can comment on this in class? But I think it's fine still to give them this option. Maybe mention it in next class, so that they have more time to search and find an R package to contribute / presentation opportunities?

Options 2 and 3: I wholeheartedly agree with all of those points, and think we should include them.
Option 4: I think you're right, nobody will do this without some sort of encouragement in class. Perhaps let's leave this for a future year.
@wvictor14 Are you able to bundle this info together into something that will eventually be student-facing? I'm still working on ideas for the strings assignment, but that will be done early this week.

OK I will try to make instructions and a rubric for options 2-4, and post something later this week!

Haven't read through this issue in detail yet, but one (basic) requirement for both the app and the package would be to implement feedback we have given them. Since the package was Assignment 2 and the app was Assignment 3, I think they should be able to get all the feedback in time for Assignment 5.

YEah that's a good idea @dy-lin , thanks for the suggestion. I don't think it should be a requirement though because some will not receive much or any feedback on their app. I can make it an option and provide alternatives though.

Yeah! The requirements I meant more like the feedback that requires fixing? (Eg inaccuracies)-- like for A2, for some reason a lot of students stated that their package is on CRAN, or wrong installation code (install.packages() instead of devtools::install_github())

Yeah I noticed that too! I'll put that down as an example

I don't think it should be a requirement though

There's something that doesn't sit right with me by not requiring implementation of feedback -- not good if they make an effective package/app with some fundamental flaws unchanged since last time.
But then, marks set aside for addressing feedback isn't adequate either, because not everyone will have things they need to change.
How about this: set aside a few marks for the package/app fundamentals according to the requirements of the original assignment. Yes, it's duplicating criteria from a past assignment, but it (1) allows for the possibility of someone making a new app/package compared to their previous, (2) allows them to implement feedback, and (3) emphasizes the importance of having a fundamentally sound package/app.

OK, sound good! Aiming to get something out tomorrow...

Hi @vincenzocoia , to clarify, you're working on option 1, the strings part of this assignment? So I will not touch that part and just focus on the other options 2-4?

So I will not touch that part and just focus on the other options 2-4?

I think so. I suspect I'll just take some open-ended questions from R4DS.


Avoid creating an empty if block

Version: web version at https://retdec.com/service/api
File: https://github.com/rfalke/decompiler-subjects/tree/master/from_holdec/condition_structure/ia32_elf
Output:
// Address range: 0x8048437 - 0x8048465
int32_t test_1_blocks_1_targets_direct_variant_0(void) {
    // 0x8048437
    if (g2 == 0) {
        // 0x8048437
        // branch -> 0x8048439
    } else {
        // 0x8048457
        puts("then/else block 3");
        // branch -> 0x8048439
    }
    // 0x8048439
    puts("return block");
    return 0;
}

Expected:
The condition should be inverted and there should only be the then-block without else.

Output by the current master, after the new LLVM IR to BIR convertor became the default convertor in llvmir2hll:
// Address range: 0x8048437 - 0x8048466
int32_t test_1_blocks_1_targets_direct_variant_0(void) {
    // 0x8048437
    if (g4 != 0) {
        // 0x8048457
        puts("then/else block 3");
    }
    // 0x8048439
    puts("return block");
    return 0;
}

I can't say empty if bodies are never going to be generated, but the current output (all the functions) look quite ok at the moment - nice test binary by the way. If you come across another issue, open an issue :-)
I added regression test (https://github.com/avast-tl/retdec-regression-tests/commit/82b373552d7c37a68cd177c8811c1df479c6b7a4) for the reported problem, but unfortunately I did not came up with a way to test all the control-flow structures in all the functions. The test itself is quite ugly - hard-coded string to check for. I don't think we have a better mechanism to check for things like this, and it would be a lot of work anyway, since there is a huge number of test cases in the subject.exe sample.


No Recipes on 1.19.2

Mod Version: 1.19.02-1.0.8
there are no crafting or breading recipes for the following chickens
Sand, Nether Quartz, Log, Flint, and Soul Sand. I hope I did not miss any. if KubeJS is installed this is a work around. just place the file in yourmodpack/kubejs/server_scripts/ and then change the ext from .txt to .js
chickens.txt

It's normal that these chickens don't have a craft. They are spawning naturally, and they are the starting point to craft other chickens, you just have to find them

Yes, you don't really need them, the chickens are pretty common


Update README.md

Description
Please include a summary of the change and which issue is fixed. Also include relevant motivation and context. List any dependencies that are required for this change.
Contribution Guidelines
Please Read through the Contribution Guidelines.
Fixed Issues
Please mention any issues fixed in the PR by referencing it properly in the commit message.
As per the convention, use appropriate keywords such as fixes, closes, resolves to automatically refer the issue.
Please consult official github documentation for details.
Fixes #(issue)
"BEFORE" and "AFTER" output
To verify that the change works as desired, please include an output of terminal before and after the changes under headings "BEFORE" and "AFTER".
BEFORE
Any Terminal Output Before Changes.
AFTER
Any Terminal Output Before Changes.
Contribution checklist

[x] I have read the Contributing Guidelines.
[x] The commits refer to an active issue in the repository.
[x] I have added automated testing to cover this case.

Notes
Please mention if you have not checked any of the above boxes.

Hi @xmonish ,
Thanks the contribution. However adding the logo at the start adds no value to the README.
Loved your other PR #383 though!


Removed ability to manually pass in routes via config

Codecov Report

Merging #93 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #93   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          23     23
  Lines         539    539
  Branches       60     59    -1
=====================================
  Hits          539    539

Impacted Files
Coverage Δ

lib/sky-pages-route-generator.js
100% <100%> (ø)
:white_check_mark:

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 a41532b...66274f1. Read the comment docs.


Create Aliseducacional.txt

@eullercristian please excuse our delay with replying you. As you've mentioned the university website is http://www.aliseducacional.com.br

therefore the path should be br/com/aliseducacional.txt - please fix the path accordingly.

@eullercristian This pull request remains unanswered for a while. I am closing it because of wrong path provided, please reopen and change the path if the request remains relevant. Thank you!


unable to compile, please help

Hello, i try to use your code but unable to compile:
Error	CS0103	The name 'Azure' does not exist in the current context	AzureStorageExample	C:\Users\PeterCheung\desktop\AzureStorageExample\AzureStorageExample\Form1.cs	30	Active
This line "var azure = Azure.Configure()" not working
using Microsoft.WindowsAzure.Storage;
using Microsoft.WindowsAzure.Storage.Blob;
using Microsoft.Azure.Management.ResourceManager.Fluent;
using Microsoft.Azure.Management.ResourceManager.Fluent.Core;
using Microsoft.Azure.Management.Storage.Fluent;
        var credentials = SdkContext.AzureCredentialsFactory.FromFile(Environment.GetEnvironmentVariable("AZURE_AUTH_LOCATION"));

        var azure = Azure
            .Configure()
            .WithLogLevel(HttpLoggingDelegatingHandler.Level.Basic)
            .Authenticate(credentials)
            .WithDefaultSubscription();
        IStorageAccount storage = azure.StorageAccounts.Define("test_storage_account")
            .WithRegion()
            .WithNewResourceGroup("test_resource_group")
            .Create();

@peterremote1980 were you able to fix your issue?
Did you try by adding the following line?
using Microsoft.Azure.Management.Fluent;


newEnumConverter panic

What version of protobuf and what language are you using?
Version: (e.g., v1.1.0, 89a0c16f, etc)
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good with .proto and .go source code.
What did you expect to see?
What did you see instead?
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
Anything else we should know about your project / environment?

/internal/impl/convert.go
when repeated list is empty
fd.Enum().Values().Get(0) panic

Please provide a way to reproduce the problem, preferably as a complete runnable program with .proto and .go source code. Thanks.

Can you provide a reproducer for this? If I remember correctly, protoc refuses to compile a .proto with an empty enum set.

my mistake using gogofast.


Camera disappear when switching from background to foreground

hello.
I am currently using version 1.5.2.
There were no bugs when using version 1.4.0, but after version 1.5.2, a bug that the camera disappears intermittently when the app is returned from the background to the foreground during broadcasting.
After the camera disappears, When i change the camera to the front camera or the rear camera, it will be exposed normally.
It also often crashes in the background state.
Is there any solution?
thank you!

This is crash debug log

Hey @ppth0608, thanks for the report. That is a scenario we test and we haven't seen issues, but we will investigate it further.
Could you attach a full crash report here? If you've seen this in production you can export the .xccrashpoint file from the Organizer (right click on a crash -> Find in Finder), or if you have a crashing device locally you can export the .crash file from the Devices section of Xcode. Either would be very helpful for us resolving this issue for you.

@bclymer Thank you for reply.
It's not released, so it's hard to figure out the crash log.
But, After some tests, the above issue was reproduced in the following situations:
When using a mixer slot and layout the preview through IVSBroadcastSession.preview(aspectMode:)

In the MixerViewController example of the BasicBroadcast app, the same crash occurs when entering the foreground from the background.
At this time, the error log that occurs is as follows.

BasicBroadcast[2578:1449883] Execution of the command buffer was aborted due to an error during execution. Insufficient Permission (to submit GPU work from background) (IOAF code 6)

@ppth0608 I have a couple clarifying questions.

Are you able to get the BasicBroadcast app to actually crash, or just print the error you provided? I was able to get the error, but never a crash.
Are you using a custom image source in your own application?

What's happening in the MixerViewController is that image samples are still sent from the custom image source (the video) to the SDK after the app is in the background, and iOS devices restrict GPU access to foreground applications only. The SDK will manage the camera lifecycle and stop sending camera samples when entering the background, but custom image sources that are managed by the host application need to do that themselves (our sample app does not). Eventually Apple won't even process your GPU samples in the foreground without a long enough timeout.

Closing this ticket due to inactivity. Please open a new ticket if you are still experiencing issues.


Warm Coinbase (EIP-3651)

PR description
Add a flag to MainnetTransactionProcessor to add the miningBeneficiary to the list of pre-warmed addresses.
Have shandong fork set this flag to true by default.
Signed-off-by: Danno Ferrin<EMAIL_ADDRESS>Fixed Issue(s)

Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if
updates are required.

Changelog

[X] I thought about the changelog and included a changelog update if required.

No changelog.  Once shandong syncs or shanghai fork locks down it becomes changelog worthy.


How about using RCTEventEmitter for "getCurrentTime"?

I think better using "RCTEventEmitter" than 'setInterval' when we use "getCurrentTime".
What do you think about it?

That would be nice, and I think it would be possible for iOS, but not for Android. Android's MediaPlayer doesn't support a way to observe changes in current time.
So we have to decide if we want to support this for iOS and not for Android, what do you think @trepidity? And it would be something for a future release I think, if we decide to support this.

My first thought is no. Features that do not have parity across all systems is contrary React/React-Native.
Does adding this feature make implementation for the first-timer easier or harder?
I could see adding it as an "advanced" or undocumented API feature.

Perhaps in Android this can be implemented as a TimerTask that keeps checking on the player.getCurrentPosition.
This way the experience is consistent across OS.


Fix mailing list line

Fixes #19

BTW I'm not sure who merges PRs here. Feel free to merge your own given that there's at least one approval 😉


Can not add songs to playlist

Can not add songs to playlist. That bug both occur in the newest desktop and browser ver.

that's the reason why new version is not released. I think still need time to handle these bugs.
Could you add more details about the bug? Like console error log, network tab info.

that's the reason why new version is not released. I think still need time to handle these bugs.
Could you add more details about the bug? Like console error log, network tab info.

I tried to add the song during search, and while the green notice panel
showed up, the songs never end up getting into the playlist.
Listen 1<EMAIL_ADDRESS>于 2021年1月31日周日 上午7:50写道：

that's the reason why new version is not released. I think still need time
to handle these bugs.
Could you add more details about the bug? Like console error log, network
tab info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/listen1/listen1_chrome_extension/issues/456#issuecomment-770299441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHV2OFET6MKUM5WH2RXNTP3S4SLMTANCNFSM4W2TJCNQ
.

I tried to add the song during search, and while the green notice panel
showed up, the songs never end up getting into the playlist.
Listen 1<EMAIL_ADDRESS>于 2021年1月31日周日 上午7:50写道：

that's the reason why new version is not released. I think still need time
to handle these bugs.
Could you add more details about the bug? Like console error log, network
tab info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/listen1/listen1_chrome_extension/issues/456#issuecomment-770299441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHV2OFET6MKUM5WH2RXNTP3S4SLMTANCNFSM4W2TJCNQ
.

fix this in https://github.com/listen1/listen1_chrome_extension/commit/449859efe0005928fcebb8b7ba91417827d58df2
the reason is axios will automaticly parse json response, but old angularjs http lib leaves it to user.

fix this in https://github.com/listen1/listen1_chrome_extension/commit/449859efe0005928fcebb8b7ba91417827d58df2
the reason is axios will automaticly parse json response, but old angularjs http lib leaves it to user.

Thanks for your commit  👍

Thanks for your commit  👍


Bistatic delay

This PR adds a method to the BurstSLC class to compute bistatic delay as described by Gisinger et al 2021. I have tried to put documentation in the code for a caveat about unclear initial bulk bistatic delay correction applied by ESA. At the moment I don't have a clear evidence if ESA has used the middle of the busrt of interest to compute their approximate bulk bistatic delay or the middle of the subswath 2. The latter has been mentioned by Gisinegr et al , but was not found in any ESA document so far.

@hfattahi could you please do a git rebase for this PR so that I could also use the latest main branch for testing?

@hfattahi could you please do a git rebase for this PR so that I could also use the latest main branch for testing?
Update: I could git rebase locally myself, so it's fine to not rebase here.

I did rebase if that helps.

Here is a comparison of one burst in IW2, between S1-ETAD products from ESA and this PR: they are identical.

Here is a comparison of one burst in IW1, between S1-ETAD products from ESA and this PR: there is an offset of 0.00017.

Here is a comparison of one burst in IW2, between S1-ETAD products from ESA and this PR: they are identical.

Here is a comparison of one burst in IW1, between S1-ETAD products from ESA and this PR: there is an offset of 0.00017.

Update: after using the IW2 mid_range instead of the IW1 mid_range for the example above, the two results match, as below:

So, the solution could be calculating the iw2_mid_range in burst_from_xml() and save as metadata in the burst class. To implement this, we could:

guess the iw2_annotation_path from the input annotation_path inside burst_from_xml(), or
feed the iw2_annotation_path to burst_from_xml() as a new arguments.

Option 2) is simpler to implement and cleaner from the developer's view, while option 1) might look cleaner from the user's view. What do you think @LiangJYu @vbrancat @hfattahi ?
Note: I multiplied the s1-reader output by -1.

Thanks @yunjunz for doing this interesting test. At least now we are consistent with ETAD ATBD. However, we are still not sure if ETAD ATBD is correct or the ESA IPF document on using the middle of IW2 or near range of IW2. We can move forward with ETAD ATBD for now. We can revise when we hear back from ESA folks.
I think option-2 seems cleaner to me.

Thank you @yunjunz for addressing the comments from Liang and adding the mid range to IW2. Looks great.
Thank you @vbrancat. I agree. I opened this issue https://github.com/opera-adt/s1-reader/issues/35 to make sure we will get back to it when we hear from ESA on the inconsistency.
I was about to merge. Looks like someone pushed the button already. hahaha Thank you all.


smartdrive: upm implementation for SmartDrive motor driver

smartdrive: upm implementation for SmartDrive High Current Motor controller
from  mindsensors.com
Signed-off-by: Oussema Harbi<EMAIL_ADDRESS>

Merged thank you for the contribution!


Cleanup rs with timebound

Description
ReplicaSets serve as an intermediate controller for various Pod controllers like Deployments. When a new version of a Deployment is initiated, it generates a new ReplicaSet with the specified number of replicas and scales down the current one to zero. Consequently, numerous empty ReplicaSets may accumulate in the cluster, leading to clutter and potential false positives in policy reports if enabled. This cleanup policy is designed to remove empty ReplicaSets across the cluster within a specified timeframe, for instance, ReplicaSets created one day ago, ensuring the ability to rollback to previous ReplicaSets in case of deployment issues.
Checklist

[] I have read the policy contribution guidelines.
[] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

When you make and complete requested changes, please resolve conversations as you go along so reviewers know you have addressed the comments.


Add contextual_breadcrumbs tests and flag to show taxon breadcrumbs o…

…ver parent-based
Remember to update the CHANGELOG if your change needs to be listed in the next version of the gem.

Component guide for this PR:
https://govuk-publishing-compon-pr-[THIS PR NUMBER].herokuapp.com/component-guide/
Trello card
https://trello.com/c/yZfYuiJp/84-use-taxons-breadcrumb-in-b-sample

Can you please update the PR description so the link to the component guide works and points to the component you've changed?


[WFRP4e System] Add RAI houserule for Point Blank range

As explained by Andy Law, RAI meant to give 4 units (2 squares) as PBR (Point Blank Range) for all ranged weapons, instead of percentage based on their Range.
Could you add this as a house rule setting in the system? I can see computeRangeBands() function, but it has no hook so most likely I can't make it quick with a module.
Source: https://docs.google.com/document/d/1v37KH8Pa1BjTfnH4QT_N4_MGK3seKMvkvQKnWiJ4u3A/edit#heading=h.1qgs6qgua6b8 (Page 73, Chapter 5 -> Combat -> Ranged Combat -> Point Blank -> Q2 and Q3)
From: manufacturefoundry: 11.304wfrp4e: 6.5.9wfrp4e-core: 3.1.4wfrp4e-up-in-arms: 3.0.6wfrp4e-wom: 3.0.5Active Modules: wfrp4e-unofficial-grimoire

This is not RAI, simply a house rule, and I generally don't include UFAQ


Provider can confirm conditions met or not met for accepted offers

Context
We need a UI within the provider console to allow providers to confirm whether the conditions set out as part of an offer have been met or not. This is high priority because we now have a user on our system in the accepted state and the relevant provider can't progress their application.
The scope for this work is limited to confirming all conditions have been met, not individual conditions. This means our system cannot be used for tracking the progress of meeting conditions at this stage. Conditions are either met in their entirety or not at all.
A minimal set of views, including confirmation steps, has been designed for this process. As these provider actions represent a different feedback phase for an application, a new controller and set of views will be added to the system.
Changes proposed in this pull request
Add new controller (ConditionsController) and associated routes to provider interface.
get '/applications/:application_choice_id/conditions' => 'conditions#edit', as: :application_choice_edit_conditions

patch '/applications/:application_choice_id/conditions/confirm' => 'conditions#confirm_update', as: :application_choice_confirm_update_conditions

patch '/applications/:application_choice_id/conditions' => 'conditions#update', as: :application_choice_update_conditions

In the future, we are planning to allow marking individual conditions as met/not met, this setup will accommodate this.
Add ConfirmConditionsForm to handle the presence validation for :conditions_met.
Guidance to review
Run the specs.
To test manually, you'll need applications at the pending_conditions state (they show up as 'ACCEPTED' in the provider interface).
-- Starting point

-- Form

-- Confirmation page if conditions met

-- Confirmation page if conditions not met

-- Flash message when conditions met

-- Flash message when conditions not met

Note: The border to flash messages is blue instead of green, because that's the current definition of an :info level flash message in the app. Both are considered success cases.
Link to Trello card
1579 - Providers can confirm conditions met/not met (applies to all conditions)
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Nice work @mnacos just a few bits:

If there are no conditions, what do we do? Your screenshots so the user performing this task even when there are no conditions.

The flash message should be in a green border (‘success’)

We need to style the tags accordingly for these states

I think we'll need to make a few content tweaks (as you said on Slack)


CSS and JS aren't loaded on IE 9

When accessing via IE 9, CSS/JS gets 404 error and the site design comes to break.

Fixed in v1.9.0-M1.


Added apply, fixed IDE error on baseUrl is nullable

Added apply, fixed IDE error on baseUrl is nullable


Googlefit

There is a bug with stopping activity sessions however I am issuing this pull request as the v2 branch is getting very out of sync with this branch.  Will complete bugs tomorrow night.

Coverage increased (+3.32%) when pulling 701b987a451fb659d6d0c6816c9bd0c6790a6753 on googlefit into f758d09a223053241f86cada5f821026a71dcf39 on v2.

Tests are ok
I'm not sure 419807c70809ade46be03a349469e1549ab3f428 is still needed (may have been fixed in 3f845e3d9f2bba68794cd192013e1b177031a953)


Use LogitsProcessor for transformers integration

Fixes #806
Related: https://github.com/outlines-dev/outlines/issues/789
Requesting comment on whether we should unify the logits processor in this PR as described in the last section of this message. @brandonwillard @rlouf
Problem
transformers has two issues in its generate integration:

if input_ids is empty, it fails
it has an inconsistent implementation with other models, it is the only model which doesn't use logits processors

Solution
Implement logits processors for transformers.
Proposed Additional Work
We have four implementations of logits processors which do the same thing. Their implementations have drifted and now there are growing differences in their implementation details. We should simplify this and use a single logits processor for all models, ensuring a bug fix or enhancement for one logits processor applies to all models.

Additionally, this minimizes the changes necessary for https://github.com/outlines-dev/outlines/issues/918  minimal
Furthermore, this issue only would need to be fixed in one module with a singular implementation: https://github.com/outlines-dev/outlines/issues/855

This WIP PR shows generalized logits_processors.py which aims to be compatible with all inference engines, along with a modified generate/regex.py incorporating the new singular logits processor implementation.

closed in favor of https://github.com/outlines-dev/outlines/pull/966


Auto-play new releases

Since new releases come pretty often, I'd love to have an option for auto-playing new releases.
Prerequisite task here, I suppose, is to make loaded pages/clients ping the server for updates (or get them pushed).

Can you elaborate on the kind of behavior you're thinking of? If you're listening to a piece when a new one comes out, would you expect it to switch to the new one? Or if you visit the site after a new piece has been released, what would you expect to happen then?
I've been thinking of ways to at least notify users when a new piece is released but I'd love to hear your thoughts first!

If you're listening to a piece when a new one comes out, would you expect it to switch to the new one?

That's exactly what I was thinking of. (:


Setting-Privacy-Admin-App - Sprint 1

Description:

Build the Screen using the style guide.
Integrate Screen with the Backend

@sheyi103 assign this to me

@sheyi103 I see @evergreenx requested to work on this one, please assign it to him and move it to "In progress"


[Improve] Move FlinkStreamingInitializer from every version to the base module

What problem does this PR solve?
Issue Number: close #1758
Problem Summary:
Currently, there is FlinkStreamingInitializer in each version, and the code is the same, so it can be moved to the shims-base module.
What is changed and how it works?
Proposal: xxx
Feel free to ping committers for the review!
Contribution Checklist

Make sure that the pull request corresponds to a GITHUB issue.

Name the pull request in the form "[Feature] [component] Title of the pull request", where Feature can be replaced by Hotfix, Bug, etc.

Minor fixes should be named following this pattern: [hotfix] [docs] Fix typo in README.md doc.

Purpose of this pull request
Currently, there is FlinkStreamingInitializer in each version, and the code is the same, so it can be moved to the shims-base module.

Hi @wolfboys
The streampark-storage module failed during CI due to Unable to get dependency information. It might be network problem and the CI can success in my Mac . Please help rerun, thanks~
Error:  Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.22.2:test (default-test) on project streampark-storage: Unable to generate classpath: org.apache.maven.artifact.resolver.ArtifactResolutionException: Unable to get dependency information for org.apache.maven.surefire:surefire-junit4:jar:2.22.2: Failed to retrieve POM for org.apache.maven.surefire:surefire-junit4:jar:2.22.2: Could not transfer artifact org.apache.maven.surefire:surefire-junit4:pom:2.22.2 from/to central (https://repo.maven.apache.org/maven2): transfer failed for https://repo.maven.apache.org/maven2/org/apache/maven/surefire/surefire-junit4/2.22.2/surefire-junit4-2.22.2.pom
Error:    org.apache.maven.surefire:surefire-junit4:jar:2.22.2
Error:
Error:  from the specified remote repositories:
Error:    apache.snapshots (https://repository.apache.org/snapshots, releases=false, snapshots=true),
Error:    central (https://repo.maven.apache.org/maven2, releases=true, snapshots=false)
Error:  Path to dependency:
Error:  	1) dummy:dummy:jar:1.0: Connection timed out (Read failed)
Error:  -> [Help 1]
Error:
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
Error:
Error:  After correcting the problems, you can resume the build with the command
Error:    mvn <args> -rf :streampark-storage
Error: Process completed with exit code 1.

Hi @wolfboys
The streampark-storage module failed during CI due to Unable to get dependency information. It might be network problem and the CI can success in my Mac . Please help rerun, thanks~
Error:  Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.22.2:test (default-test) on project streampark-storage: Unable to generate classpath: org.apache.maven.artifact.resolver.ArtifactResolutionException: Unable to get dependency information for org.apache.maven.surefire:surefire-junit4:jar:2.22.2: Failed to retrieve POM for org.apache.maven.surefire:surefire-junit4:jar:2.22.2: Could not transfer artifact org.apache.maven.surefire:surefire-junit4:pom:2.22.2 from/to central (https://repo.maven.apache.org/maven2): transfer failed for https://repo.maven.apache.org/maven2/org/apache/maven/surefire/surefire-junit4/2.22.2/surefire-junit4-2.22.2.pom
Error:    org.apache.maven.surefire:surefire-junit4:jar:2.22.2
Error:
Error:  from the specified remote repositories:
Error:    apache.snapshots (https://repository.apache.org/snapshots, releases=false, snapshots=true),
Error:    central (https://repo.maven.apache.org/maven2, releases=true, snapshots=false)
Error:  Path to dependency:
Error:  	1) dummy:dummy:jar:1.0: Connection timed out (Read failed)
Error:  -> [Help 1]
Error:
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
Error:
Error:  After correcting the problems, you can resume the build with the command
Error:    mvn <args> -rf :streampark-storage
Error: Process completed with exit code 1.

done


Define PR template

Defines a template which mainly defines a checklist for the scenario of upgrading a repo version. This stems from the conversation in https://github.com/dotnet/source-build-externals/pull/352.

@mthalman - Can this be merged?


Not closed FileInputStream Object caused "To many open files" IOException

In our case, this part caused endless loop in the jenkins build process(tests execution) and was returning incorrect error message. The getBytes method was not closing FileInputStream at the end of reading files and caused IOException "To many open files".

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Your commit is correct, thanks a lot for it. Please just fix code identation before I merge it.

@mariofusco  When you're merging, can you please merge to 6.5.x as well?

We are using 6.3.0-Final. It would be nice if you can merge into 6.3.x branch as well.


Fix registry key, remove invisible unicode chars

One of the registry keys related to the Logon Script environment persistence mechanism incorrectly listed 'UserMprLogonScript' as the environment variable name when it should be 'UserInitMprLogonScript' (per http://www.hexacorn.com/blog/2014/11/14/beyond-good-ol-run-key-part-18/).
Also, two of the registry keys had the invisible soft hyphen character inserted.  I'm not sure if this affects things when these data files are used, but it's probably best to remove them. git diff from the console shows these changes better than the GitHub UI:
-      - '%%environ_systemroot%%\system32\Windows<U+00AD>PowerShell\v1.0\profile.ps1'
-      - '%%environ_systemroot%%\system32\Windows<U+00AD>PowerShell\v1.0\Microsoft.PowerShell_profile.ps1'
+      - '%%environ_systemroot%%\system32\WindowsPowerShell\v1.0\profile.ps1'
+      - '%%environ_systemroot%%\system32\WindowsPowerShell\v1.0\Microsoft.PowerShell_profile.ps1'

Thx for the PR, I'll have a look as soon as time permits

From doing some research, I found more artifacts that could be added in.  There are a few others I plan to add in next week, but I figured I'd push what I had thus far so you all can tell me if I'm way off base on these.


StringData fix for koki secrets

Fix for issue #228

Hi @wlan0,
Can you pls merge this change?

Thanks for the fix @msenmurugan! :+1:


Redshift - show permissions issue

System information:

Windows Server 2010
DBeaver version : <IP_ADDRESS>101311042

Connection specification:

Amazon Redshift
Driver name: com.amazon.redshift.jdbc42.Driver
Do you use tunnels or proxies (SSH, SOCKS, etc)?  No

Describe the problem you're observing:
On Table or View properties > DDL or Source
"Show Permissions" selected
Then the DDL script shows the table permission GRANT statement.
For Redshift, the GRANT statement shows wrong permission.
For example )
GRANT INSERT, SELECT, UPDATE, DELETE, UNKNOWN, REFERENCES, TRIGGER ON TABLE public.dummy TO myuser;
UNKNOWN, permission is a wrong permission. It supposed to be RULE such as:
GRANT INSERT, SELECT, UPDATE, DELETE, RULE, REFERENCES, TRIGGER ON TABLE public.dummy TO myuser;
Steps to reproduce, if exist:
Connect to AWS Redshift,
On Table or View properties > DDL  (or Source)
"Show Permissions" selected
Then the DDL script shows the table's permission GRANT statement which is wrong if RULE permission added.
Include any warning/errors/backtraces from the logs

Thank you for the report.

Fixed in 21.0.2
new ticket about Redshift permissions created https://github.com/dbeaver/dbeaver/issues/11912

verified


[Event Request] OnAfter Event on function IncrAmount in COD80 and COD90

In our localization scope we have withholding tax code executed on function IncrAmount in Cod80 and 90.
So we would like to request and event OnAfterIncrAmount in both codeunits 80 and 90.

These are already implemented. Please read: https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/


Extend 'spo page add' with support for promoting as template

Extend the 'spo page add' command with support for promoting as template.
spo page add --promoteAs Template
Equivalent of the PromoteAs Template option in the Add-PnPClientSidePage cmdlet.

I want to work on this. Will start as soon as my PR for #978 is merged.

Just started working on this one

Awesome! Looking forward to the PR! 👏


counter interrupts

There are 2 problems with counter interrupts.

The counter will continuously send interrupts after the count is finished. because of this condition  as long as irq_ena is 1 and stop_out is 1 (happened when the counter finish couting)

This can be fixed by trigger interrupts only at the positive edge of  stop_out

It's not interrupt bug but it appears while debugging interrupt. Register value_reset isn't updated with the write condition I believe it should depend on the reg_dat_we rather than reg_val_we like this part of the code

This can be fixed by replacing  reg_val_we  with reg_dat_we in lines 174 to 177

this file also needs the same changes

At a company where I used to work, they had a policy that all interrupts have a complicated defined behavior that includes a configuration that can allow the interrupt to either be one-shot or to remain high until reset by software;  seemed like a good policy.  However, without doing something like that, this solution should be fine.
I have been absent for this week due largely to Covid but I will review all changes today.

Fixed by merging https://github.com/RTimothyEdwards/caravel_openframe_project/pull/18


[Error: No driver for device]

[Error: No driver for device]
Receiving the above error while trying to open the serial port after giving all the permissions.

Hi, have resolved that issue? I am also encountering this issue however i am getting deviceName etc

Do you guy uncomment the console.warn(list) in fillDeviceListAndConnect() and have a look at the result? Maybe you need adjust the item.vendorId === 6790 if your device is notch34x


Fix ArrayProxy shape for signed numbers

fixes #476
This time for real hopefully. Any edge cases I forgot?

I had a hard time convincing myself about the new logic in this PR, so eventually I rewrote it in 00026c6e4a0d3f9c2eff054b1bd4d8d18342a2ed. I did use your tests though; thanks!

Clarity is in the eye of the beholder apparently haha
My brain seems to be wired in a more functional programming way than is usually considered Pythonic I guess.
I tend to favour complex list comprehensions over for loops that touch 4 variables.
Glad it's fixed either way :)

Clarity is in the eye of the beholder apparently haha
My brain seems to be wired in a more functional programming way than is usually considered Pythonic I guess.

It's mostly the use of arithmetic operations on bools (which most functional languages I know don't allow). The original code that I wrote did it, and that was pretty confusing (I think you actually got confused at first). I had to read it a few times to convince myself that it's correct, so I thought I might as well rewrite it.


Related Products Issue

It seems Easylife Switcher is causing the first image in the "Related Products/You May Also Like" Section to be replaced with the main image - do you have any insight on this?
Thanks

@dclinton  I didn't experience this before. Can you post the link where I can see this?
Most probably there is a problem with the dom selector for your main image. It may select multiple images in the page.

Thanks for the quick response im away from a computer but will get back to asap
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------From: Marius Strajeru  Date:04/17/2015  9:42 AM  (GMT-05:00) To: tzyganu/Switcher  Cc: dclinton  Subject: Re: [Switcher] Related Products Issue (#105)

@dclinton  I didn't experience this before. Can you post the link where I can see this?
Most probably there is a problem with the dom selector for your main image. It may select multiple images in the page.

Reply to this email directly or view it on GitHub:
https://github.com/tzyganu/Switcher/issues/105#issuecomment-93988454

Here is an example
http://www.miansai.com/shop/mens/mens-bracelets/beacon-leather-4dba70
then click the first related product below.
Date: Fri, 17 Apr 2015 10:10:13 -0400
Subject: RE: [Switcher] Related Products Issue (#105)
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS><EMAIL_ADDRESS>Thanks for the quick response im away from a computer but will get back to asap
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------
From: Marius Strajeru
Date:04/17/2015 9:42 AM (GMT-05:00)
To: tzyganu/Switcher
Cc: dclinton
Subject: Re: [Switcher] Related Products Issue (#105)
@dclinton I didn't experience this before. Can you post the link where I can see this?
Most probably there is a problem with the dom selector for your main image. It may select multiple images in the page.
—
Reply to this email directly or
view it on GitHub.

This is all set now - was a config issue.  thanks.
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: RE: [Switcher] Related Products Issue (#105)
Date: Fri, 17 Apr 2015 11:17:50 -0400
Here is an example
http://www.miansai.com/shop/mens/mens-bracelets/beacon-leather-4dba70
then click the first related product below.
Date: Fri, 17 Apr 2015 10:10:13 -0400
Subject: RE: [Switcher] Related Products Issue (#105)
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS><EMAIL_ADDRESS>Thanks for the quick response im away from a computer but will get back to asap
Sent from my Verizon Wireless 4G LTE smartphone
-------- Original message --------
From: Marius Strajeru
Date:04/17/2015 9:42 AM (GMT-05:00)
To: tzyganu/Switcher
Cc: dclinton
Subject: Re: [Switcher] Related Products Issue (#105)
@dclinton I didn't experience this before. Can you post the link where I can see this?
Most probably there is a problem with the dom selector for your main image. It may select multiple images in the page.
—
Reply to this email directly or
view it on GitHub.

@dclinton I'm glad it was fixed. I will close this issue now, but please explain in a few words what was the problem in case other people run into it. Thanks in advance.


Social-only login/registration

My app's workflow wants users to login only via social login; not by creating an app-specific password.
Therefore, I'd like the login form to appear with only the social login options visible. It does not look like you currently support this option (unless I've missed something?)
I've hacked this to work by creating a custom view based on your standard view. But, this requires copying and maintaining a lot of code for a trivial change.
It looks like a better solution would be to pass in an option, maybe web.login.socialOnly. Would this be generally useful? Something you can/will do? Or better for me to create a pull request? (if the latter... I'm happy to do so, but probably not immediately... I'll have some learning curve in your code and my current custom view is good enough to hold me for a little while).

Hmm, I think this is a cool feature request, although it will require a very different view to look decent.


pyam version

Thank you for the interesting package.
It might rather be a request than an issue. I expected to get the pyam version once I enter:
pyam.__version__
Thank you for consideration.

Hi @fallahnejad, thanks for making an issue! One quick question - how did you install pyam (i.e., from conda, pip, source)? On the current master branch, you indeed get the version:
$ python -c 'import pyam; print(pyam.__version__)'
0.1.2+59.geb3c327.dirty

Thanks for the fast reply.
well, I installed pyam via conda-forge channel: conda install -c conda-forge pyam
>>>pyam.__version__
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'pyam' has no attribute '__version__'

I just checked in the anaconda navigator. I'm using pyam version 0.1.2.

Ok, that makes sense. After the 0.1.2 release, we switched to using versioneer.py which provides the correct __version__ attribute. We are in the process of making a new release now, so you should get that attribute upon the next release.
I will close the issue for now, but let us know if you come across any other issues using the library!


DEVX-5926 and DEVX-5937 Add dvr and lowlatency options for HLS Broadcasts

Description

Adds dvr and lowlatency options for hls Broadcast objects
Updates code comments for usage of these options
Adds a check that both are not set to true simultaneously

Motivation and Context

It implements a feature in the SDK which has already been added to the API.
How Has This Been Tested?

Added a test for the conditional check.
Example Output or Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Please rev the version number (to 4.4.0) in the lib/opentok/version.rb file.

👍🏾 Done in this commit


Don't export Facebook and Twitter icons 🦅

Summary
We were exporting two icons related to social media platforms:

<Twitter />
<Facebook />

These icons were only used within our <ZopaFooter />.
This PR removes these exports and inlines the icons instead.
It's marked as a breaking change.
Issue

#30

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cannot handle  apk file size > 2G

Seems the centralDirOffset is wrong if apk file size > 2G

thanks for reporting!
it looks like I missed that those fields are unsigned, and ByteBuffer is treating them as signed ints, which isn't right. I also did a quick search and found that ByteBuffer doesn't support reading unsigned ints directly. So, we'll need to create our own way to handle this, maybe by extending the ByteBuffer class.
but sorry, I can't dive into this right now as I'm pretty swamped. I'll take a look next month. If anyone else wants to give it a shot in the meantime, feel free! :)

大佬，两月了，怎么样了？试了一下相关的改成long，结果out.write的时候只能int。


undefined method `supports' for Chef::Resource::User::LinuxUser

I think its because of the cookbook is using supports :manage_home => node[:rbenv][:manage_home] ?
https://github.com/chef/chef/issues/5318
> chef --version
Chef Development Kit Version: 0.17.17
chef-client version: 12.13.37
delivery version: master (f68e5c5804cd7d8a76c69b926fbb261e1070751b)
berks version: 4.3.5
kitchen version: 1.11.1

and the error message:
NoMethodError
       -------------
       undefined method `supports' for Chef::Resource::User::LinuxUser

       Cookbook Trace:
       ---------------
         /tmp/kitchen/cache/cookbooks/rbenv/recipes/default.rb:84:in `block in from_file'
         /tmp/kitchen/cache/cookbooks/rbenv/recipes/default.rb:81:in `from_file'
         /tmp/kitchen/cache/cookbooks/intake_rails/recipes/default.rb:13:in `from_file'

       Relevant File Content:
       ----------------------
       /tmp/kitchen/cache/cookbooks/rbenv/recipes/default.rb:

        77:  group node[:rbenv][:group] do
        78:    members node[:rbenv][:group_users] if node[:rbenv][:group_users]
        79:  end
        80:
        81:  user node[:rbenv][:user] do
        82:    shell "/bin/bash"
        83:    group node[:rbenv][:group]
        84>>   supports :manage_home => node[:rbenv][:manage_home]
        85:    home node[:rbenv][:user_home]
        86:  end

👍


Snackbar: Replace leftAlignFrom prop with a placement prop

leftAlignFrom should be replaced by a placement prop as this makes more sense

Got a question from a project about this weird behaviour where snackbar would float left when window was wide. He asked if we could have a placement prop instead, which is exactly what we are planning here so +1 :)


ColorBlindness palette

New Contributors:
Please review CONTRIBUTING.md
Hacktoberfest Participants:
Check-out the Hacktoberfest 2021 page
Purpose:

Add a new 8-color palette that has good overall variability and can be differentiated by individuals with red-green color blindness, as suggested by Bang Wong (Nature Methods, 2011).
This is one of the many subjects suggested in #1274.

Thanks @arthurits! As someone with red-green colorblindness, I very much appreciate this PR 😄
I'll merge this (and the adjacent ones) after @bclehmann approves 👍

Thanks @arthurits! As someone with red-green colorblindness, I very much appreciate this PR 😄

My pleasure!


Open link in room description in browser

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating System

[X] macOS
[ ] Windows
[ ] Linux

Operating System Version
macOS 13.1
It happens on the web browser too?
No, it just happens on the Desktop app
Rocket.Chat Desktop App Version
3.8.16
Rocket.Chat Server Version
5.4
Describe the bug
Links in room descriptions open in the client not in your browser.
How to Reproduce

Add a text as room description. The text includes a markdown link:

My room description. Follow this link [duckduckgo.com](https://duckduckgo.com/)

Save
Click the link.

Describe your Expected behavior
I expect, that external links are opened in my browser. Internal links, like a link to another room, should open within RocketChat.
Anything else
No response
Are you willing to submit a code contribution?

[ ] Yes, I am willing to submit a Pull Request!

Hello, this was fixed on the server update. Please update your server.
Even in browser it must be opening on the same tab and not in another one.

Hi @jeanfbrito ,
could you please post the corresponding release of the server version containing that fix? I couldn't find it in the release page

@jeanfbrito
I have the same issue with the current RocketChat version 5.4.2.

@stopal
I think I found the PR that fixes the issue on the server: https://github.com/RocketChat/Rocket.Chat/pull/27554
The PR was merged to the develop branch but was not included for the 5.x versions yet.
As a workaround for our RocketChat 5.4.2 instance I've used a custom script with the code from the PR that seems to work well enough until we update RocketChat to a version that contains the fix.
I've added the following script to Layout -> Custom Script for logged in Users:
// Fixes an issue with opening topic links in the same tab
// This custom script should be removed when the RocketChat version contains a fix for #27554.
// See https://github.com/RocketChat/Rocket.Chat/pull/27554
const dompurify = require('dompurify');

// Add a hook to make all links open a new window
dompurify.addHook('afterSanitizeAttributes', (node) => {
	// set all elements owning target to target=_blank
	if ('target' in node) {
		node.setAttribute('target', '_blank');
		node.setAttribute('rel', 'nofollow noopener noreferrer');
	}
});


Fake consumer subscribe

PR-Codex overview
This PR adds subscription functionality to the FkConsumer class in the io.github.eocqrs.kafka.fake package.
Detailed summary

Added subscribe method to FkConsumer that accepts a String or a Collection<String> of topics to subscribe to
Created SubscribeDirs class to generate subscription directives
Added tests for FkConsumer subscription functionality

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@h1alexbel thank you for your Pull Request. I'll assign someone to review it soon.
If this PR solves a todo from the code, please don't forget to remove it.

@h1alexbel please review this Pull Request. Deadline (when it should be merged or closed) is 2023-06-23T12:39:04.447547.
You should check if the requirements have been implemented (partially or in full), if there are unit tests covering the changes and if the CI build passes. Feel free to reject the PR or ask for changes if it's too big or not clear enough.
Estimation here is 30 minutes, that's how much you will be paid. You will be paid even if this PR gets rejected.

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 2min)

@h1alexbel thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


Optimize trace format of agent

Description
Trace format optimization

Calculate the nearest delta

startTime & endTime
depth
sequence
Reduce root(span) dependency for async #2584

More plugable architecture

The V2 version is implemented in 1.7.1.


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
775f6d2da94e3602a31dd0ee94c00dff0a49f156
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Allow to install latest release/revision for given cri-o version

Applies to debian/ubuntu.. centos/rhel already had this,.

@jnummelin @kke PTAL


Figure out why recording to rosbag throws errors

It looks like trying to save compressed image topics to ROSbag causes error and slows down system.


Adding and removing fields

Is there a way to remove and add fields in the header without refreshing the grid (calling loadData)?
Thanks.

By 'fields' you mean rows? If yes, then the answer is yes. You need to redefine loadStrategy (it's not documented yet, docs will be added soon).
See following tickets https://github.com/tabalinas/jsgrid/issues/43, https://github.com/tabalinas/jsgrid/issues/50, https://github.com/tabalinas/jsgrid/issues/54

No, I mean columns, to show and hide without making a post call. Sorry if I explained it wrong.

Oh, I see now. Then try the following: call render method and turn off autoload. In this case you should call loadData yourself.

This is what I'm doing and it seems to work:
grid.jsGrid('option', 'autoload', false);
grid.jsGrid('option', 'fields', fieldsToShow);
grid.jsGrid('refresh');

Thank you for the help.

 Glad that it helped.
Note: you can set autoload: false in the grid config or even set it as the default option for all the grids on the page.


GDScript 2.0: exporting enum requires default value or else it will be null

Godot version
4.0
System information
Windows 10
Issue description
When you export a var of type enum, it requires that you set the default value in code, or else it will only ever be null
Steps to reproduce
Create node/script
enum Type { ONE, TWO }
@export var test_type: Type

func _ready():
    print(type)

No matter what you select in the inspector, it will print null. You have to set a default value in the script first.
Minimal reproduction project
No response

Currently if you select Two in the inspector, it works. Afterwards, selecting One also works. But if you don't change it yourself, the inspector shows One but it will be null.


cloudflare_tunnel_config "no_tls_verify   = true" does not apply until manual save using Web UI

Confirmation

[X] My issue isn't already found on the issue tracker.
[x] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.4.5
on linux_amd64

provider registry.terraform.io/cloudflare/cloudflare v4.3.0
provider registry.terraform.io/hashicorp/random v3.4.3

Affected resource(s)

cloudflare_tunnel_config

Terraform configuration files
# Generate a random secret for the tunnel
resource "random_id" "tunnel_secret" {
  byte_length = 32
}

# Create a tunnel
resource "cloudflare_tunnel" "my_tunnel" {
  account_id = "acc_id"
  name       = "my-tunnel"
  secret     = random_id.tunnel_secret.hex
}

# Tunnel configuration
resource "cloudflare_tunnel_config" "tunnel_config" {
  account_id = "acc_id"
  tunnel_id  = cloudflare_tunnel.my_tunnel.id

  config {

    warp_routing {
      enabled = false
    }

    origin_request {
      connect_timeout = "2m0s"
      no_tls_verify   = true
    }

    ingress_rule {
      hostname = "test.example.com"
      service  = "https://<local_ip>:<port>"
    }

    # Catch-all rule to return 404
    ingress_rule {
      service = "http_status:404"
    }
  }
}

# Add the tunnel CNAME record to the domain
resource "cloudflare_record" "cf_tunnel_cname" {
  zone_id = "zone_id"
  name    = "test"
  value   = cloudflare_tunnel.my_tunnel.cname
  type    = "CNAME"
  comment = "Cloudflare Tunnel"
  proxied = true
}

Link to debug output
1st Apply: https://gist.github.com/algo7/7176c0ef7dfdf94ae299331835d480fc
2nd Apply: https://gist.github.com/algo7/f6c475f18662cb275bb9bcdb80ed7c64
Panic output
No response
Expected output
Terraform runs fine, and I should be able to go to the tunnel sub-domain without any problem.
Actual output
When I connect to my tunnel sub-domain (continuing with the example above), https://test.example.com I get 502 Bad Gateway.  However, if I go to the Cloudflare Zero Trust dashboard => Tunnels => Public Hostname => Additional Settings => TLS and manually toggle on No TLS Verify and save, the connection will be established successfully.
If I run terraform plan again, it will show that there is a configuration drift, but the changes shown are the same settings.
Terraform used the selected providers to generate the following execution plan. Resource
actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # cloudflare_record.cf_tunnel_cname will be updated in-place
  ~ resource "cloudflare_record" "cf_tunnel_cname" {
      + comment         = "Cloudflare Tunnel"
        id              = "<tunnel_id>"
        name            = "test"
        tags            = []
        # (11 unchanged attributes hidden)
    }

  # cloudflare_tunnel_config.tunnel_config_1 will be updated in-place
  ~ resource "cloudflare_tunnel_config" "tunnel_config_1" {
        id         = "<new_tunnel_id>"
        # (2 unchanged attributes hidden)

      ~ config {
          + origin_request {
              + connect_timeout          = "2m0s"
              + disable_chunked_encoding = false
              + keep_alive_connections   = 100
              + keep_alive_timeout       = "1m30s"
              + no_happy_eyeballs        = false
              + no_tls_verify            = true
              + proxy_address            = "<IP_ADDRESS>"
              + proxy_port               = 0
              + tcp_keep_alive           = "30s"
              + tls_timeout              = "10s"
            }

            # (2 unchanged blocks hidden)
        }
    }

Plan: 0 to add, 2 to change, 0 to destroy.

After running terraform apply again, I tried to connect to the tunnel domain and got 502 Bad Gateway . So apparently, the only way to get Public Hostname working is to manually toggle on No TLS Verify via Cloudflare Zero Trust web UI.
Steps to reproduce

Use the Terraform config provided
Connect to the tunnel sub-domain (you should get 502 Bad Gateway)
Go to the CF 0 Trust dashboard => Tunnels => Public Hostname => Additional Settings => TLS and toggle on the No TLS Verify.
Visit the tunnel sub-domain again; you should be able to visit it successfully
Run terraform plan again; you should see the config drift
Run terraform apply again
Visit the tunnel sub-domain; you should get the 502 Bad Gateway message again.

Additional factoids
Repo of my full terraform config
https://github.com/algo7/terraform_cloudflare
Domain was purchased on Cloudflare
Zone TLS is set to Full (strict)
Cloudflared
cloudflared was also running successfully on the server side. It even showed the updated config with \"noTLSVerify\":true, but still failed to establish the connection as can be seen below.
cloudflared log:
2023-04-14T00:57:57Z INF Updated to new configuration config="{\"ingress\":[{\"hostname\":\"test.example.com\", \"service\":\"https://<local_ip>:<port>\"}, {\"service\":\"http_status:404\"}], \"originRequest\":{\"bastionMode\":false, \"caPool\":\"\", \"connectTimeout\":120000000000, \"disableChunkedEncoding\":false, \"httpHostHeader\":\"\", \"keepAliveConnections\":100, \"keepAliveTimeout\":90000000000, \"noHappyEyeballs\":false, \"noTLSVerify\":true, \"originServerName\":\"\", \"proxyAddress\":\"<IP_ADDRESS>\", \"proxyPort\":0, \"proxyType\":\"\", \"tcpKeepAlive\":30000000000, \"tlsTimeout\":10000000000}, \"warp-routing\":{\"enabled\":false}}" version=3

2023-04-14T00:58:06Z ERR  error="Unable to reach the origin service. The service may be down or it may not be responding to traffic from cloudflared: dial tcp local_ip:port: i/o timeout" cfRay=7b77ffa11842026f-CDG ingressRule=0 originService=https://local_ip:port

References
https://github.com/cloudflare/terraform-provider-cloudflare/issues/2072#issuecomment-1338808113 => I also had to perform manual resave (go to the Tunnels => Public Hostname => Save hostname) on the UI for cloudflared on the server side to get the configuration created by Terraform. This INF Updated to new configuration config=<truncated> won't show until the manual resave is performed.
#2233 => I had to apply it twice for wrap_routing = false not to show config drift when using the terraform config supplied above.

thanks for the thorough report! i'm going to close this off as a duplicate of #2376 since that issue has the same underlying cause and resolution. this will allow us to focus on a single issue instead of spread across multiple.

Sorry if I'm mistaken but #2376 is this issue itself.

you're right; copy pasta. should have been #2233


k8s_status module: fix append issue, when condition was on 0 index

when old condition is on index 0, new condition with same type was always appended, instead of replaced.
Description of the change:
This PR fixes issue when old condition is on index 0 and new condition with the same type should replace old condition, but instead it is appended.
/cc @fabianvf @MarSik

/ok-to-test

@fabianvf these errors doesn't look, that they are cause by my change. Can you please confirm?

@ksimon1 yeah I think there's a separate CI issue that's getting worked out, I'll kick off a retest once it works again

/retest

/cherrypick v0.10.x

/cherrypick v0.10.x


Compute derived outputs from heads directly

In GitLab by @Huite on Sep 30, 2019, 17:21
Heads in combination with conductances are in principle sufficient to compute budgets. Add porosity, and velocity can be computed as well.
In that case budgets do not have to be written, merged, compressed, etc., but can simply can be computed as needed -- massively reducing size of output; especially since heads and concentrations can be efficiently reduced in precision, and compressed.
This requires duplicating modflow functionality, but all of the above is relatively simple to implement (density correction slight more complicated), and quite significantly increases flexibility.

In GitLab by @Huite on Dec 20, 2021, 21:27
See:
https://gitlab.com/deltares/imod/imod-python/-/snippets/2029565
Runs into significant issues when heads and concentrations are float32 -- should be okay for modflow6 since all floats are 64bit -- can reopen if needed.


Resolve directory dependencies with index.js

Currently when calculating the adjacency/visibility matrices, the require calls in the following form require('./something') are translated into require('./something.js').
That's fine if the script was really there, but we're missing the case where something is a directory and the intention was to require('./something/index.js'), which is very common.

Cool. Thanks for fixing this. 👍

I will mention, that require and the module resolution work in another way.
It is possible to add the checking in an sync way, but you have to go deeply into querying.
@see also https://nodejs.org/api/modules.html for more information.


Fix input names in README

Based on warnings that GitHub showed me, when using the action (trying to use enable-review-diffs as an option as shown in the README) in one of my workflows:

Here's the fix for the README :)

Will this be merged?


Allow GAE integration tests to run in parallel

Fixes GH-122

@averikitsch

Well that was certainly an adventure of edge cases. @averikitsch @bharathkkb please re-review when you get a chance.

Thanks 😄


benchmark: ValueError if invalid parameters are specified.

I implemented the proposal from here: https://github.com/marrlab/DomainLab/issues/141#issuecomment-1443658111. Maybe that's not the perfect solution, but I think it is way better than the current one.

Codecov Report

Merging #169 (951a4af) into master (bc01ee3) will increase coverage by 0.04%.
The diff coverage is 95.23%.

@@            Coverage Diff             @@
##           master     #169      +/-   ##
==========================================
+ Coverage   94.85%   94.89%   +0.04%
==========================================
  Files         113      113
  Lines        3788     3781       -7
==========================================
- Hits         3593     3588       -5
+ Misses        195      193       -2

Flag
Coverage Δ

unittests
94.89% <95.23%> (+0.04%)
:arrow_up:

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

Impacted Files
Coverage Δ

domainlab/arg_parser.py
98.75% <93.33%> (+0.10%)
:arrow_up:

domainlab/exp_protocol/run_experiment.py
84.21% <100.00%> (-0.08%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


1167/page-not-found-ui

Issue #1167

Closes #1167

Description
This quick PR addresses some style issues on the Page Not Found page - the text was unreadable on the green background and the button had a displeasing yellow border around it.
How Can This Be Tested/Reviewed?
Entry Point: any bad link i.e. http://localhost:3000/listing/abcd
Text should be white and readable over the green background. Unnecessary borders / background colors around button should not be there.
Checklist:

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[x] I have reviewed the changes in a desktop view
[ ] I have reviewed the changes in a mobile view
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have assigned reviewers
[ ] I have run yarn generate:client and/or created a migration if I made backend changes that require them
[ ] I have exported any new pieces added to ui-components
[x] My commit message(s) is/are polished, and any breaking changes are indicated in the message and are well-described
[ ] Commits made across packages purposefully have the same commit message/version change, else are separated into different commits

Reviewer Notes:
Steps to review a PR:

Read and understand the issue
Review the code itself from a style point of view
Pull the changes down locally and test that the acceptance criteria is met
Also review the acceptance criteria on the Netlify deploy preview (noting that these do not yet include any backend changes made in the PR)
Either explicitly ask a clarifying question, request changes, or approve the PR if there are small remaining changes but the PR is otherwise good to go

On Merge:
If you have one commit and message, squash. If you need each message to be applied, rebase and merge.

@emilyjablonski oops! Should be fixed now.


sync-error on join

One Player is in second shape, another joines, the first seems to be in first shape. ONLY if these players weren't in a world before -> clientside

see Command @a function


Building Gstreamer with -DBUILD_STATIC=ON not working

We are trying to build Gstreamer with the following cmake command -
cmake .. -DBUILD_GSTREAMER_PLUGIN=ON -DBUILD_DEPENDENCIES=OFF -DBUILD_STATIC=ON -DALIGNED_MEMORY_MODEL=ON -DCMAKE_VERBOSE_MAKEFILE:BOOL=ON -DBUILD_OPENSSL_PLATFORM=linux-armv4
and then when we run make - we are given an error.
We are building on an arm64 architecture but we are building inside a docker container with raspian image - FROM resin/rpi-raspbian:stretch .
** Logging **

** Any design considerations/constraints **
We are building this on a custom yocto platform on a modalai VOXL that will be streaming via kinesis and communicating with IOTCore, all inside a container - the end goal is to build with -DBUILD_STATIC=ON so we can just extract the gst-launch-1.0 binary file. We can then include just this in our dockerfile for a light and compact container. It would be great if you have the binary files already or were open to any other suggestions to achieve this as the whole SDK is too large to run on the IOT device.

@karshee ,
Can you ensure STATIC build works on a standard platform first?
I see it is unable to find any of the libraries in the paths.

Hey,
since we have got it working with -DBUILD_STATIC=ON - we first encountered errors of samples failing so we commented out the following lines in CMakeLists.txt

and the video stream worked with the following outcome after the build -

but when we try to run the gst-launch-1.0 binary file outside the container we are missing dependencies, we tried copying everything in the build folder over too but x264enc and kvssink are still seen as missing.

we used this CMAKE command -
cmake .. -DBUILD_GSTREAMER_PLUGIN=ON -DBUILD_DEPENDENCIES=OFF -DBUILD_STATIC=ON -DALIGNED_MEMORY_MODEL=ON -DCMAKE_VERBOSE_MAKEFILE:BOOL=ON -DBUILD_OPENSSL_PLATFORM=linux-armv4

No element "x264enc" indicates that the gstreamer plugins have not been installed successfully. To verify it is a linking issue, can you try to build the SDK without  -DBUILD_STATIC=ON and see if it works?

Yes it works without -DBUILD_STATIC=ON but our goal is to minimize the overall size of the docker container. We wanted to use -DBUILD_STATIC=ON to try and build the binary files then extract them and just use them in our own dockerfile. Will this work ? or do you have any other suggestions to minimizing the size of the overall kinesis sdk docker container ? as were running it on an IOT device so require minimal storage usage.

As long as your docker container has access to the library paths, it would work. I am closing this now. Please feel free to reach out if you have any more questions.


Develop

Codecov Report

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage        ?   82.72%
=======================================
  Files           ?       18
  Lines           ?      521
  Branches        ?        0
=======================================
  Hits            ?      431
  Misses          ?       90
  Partials        ?        0

Flag
Coverage Δ

unittests
82.72% <0.00%> (?)

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


[Feature] Allow subgraph developers to create on-chain event to indicate if and when a subgraph upgrade will occur

Description
User problem
Indexer unknowingly allocates to a soon-to-upgrade subgraph but has no way of knowing the upgrade is going to happen.
When the upgrade occurs, Indexers are essentially stuck earning no rewards until the current epoch is over or until they initiate a 0x0 transaction to "free" their allocation.
User stories
As a subgraph developers I want the ability to share with Indexers WHAT subgraph I plan to upgrade WHEN without needing to install any additional software
As an Indexer I want the ability to monitor WHAT will be upgraded WHEN on-chain using the existing monitoring tools deployed in my stack (i.e. Grafana)
As an Indexer I want to see a warning in the Graph Explorer not to allocate to soon-to-upgrade subgraphs.
Related tooling
The Subgraph Upgrade Pre-syncing functionality of Subgraph Radio works toward solving this issue: https://docs.graphops.xyz/graphcast/radios/subgraph-radio/subgraph-upgrade-presyncing
However, subgraph developers are continuing to update subgraphs but are not sending UpgradeIntentMessages with Graphcast CLI to notify Indexers. The utility of the Subgraph Radio Upgrade Pre-syncing functionality is largely dependent on subgraph developers and Indexers installing and using the Graphcase CLI (in addition to existing tooling requirements).
Hypothetical new UX
graph schedule <ACTION> <TIMESTAMP>
Examples
graph schedule upgrade <OLD_SUBGRAPH_SLUG> <NEW_SUBGRAPH_SLUG> <UPGRADE_TIMESTAMP>
graph schedule publish <SUBGRAPH_SLUG> <TIMESTAMP>

Are you aware of any blockers that must be resolved before implementing this feature? If so, which? Link to any relevant GitHub issues.
Implementing schedule functionality on The Graph CLI's other methods
Some information to help us out

[ ] Tick this box if you plan on implementing this feature yourself.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

An on-chain activity would enable various third-party tools (such as GraphScan, Web3Alerts, and indexer tools) to provide indexers with enhanced visibility regarding anticipated actions from the consumer side.


added twitter link

Fixes Issue
Closes #210
Changes proposed
Added Twitter link to the website and readme
Screenshots

Note to reviewers

I created a new fork, and then made changes. After that I deleted the package-lock.json file and push the commit

@yashnirmal You have closed and then reopened the same PR ;D.
let's connect on discord, let me know whenever you are available

@rupali-codes I have sent you a hi on discord

@rupali-codes I have sent you a hi on discord

yes got it, let's do it after the space

Hey @CBID2 , I have added the alt property in img tags
Is it good now?


chore(website): redirect from Netlify to Modulz domain

Right now you can access the Radix docs from https://modulz-radix.netlify.com and https://radix.modulz.app, is the same website served from two different URLs.
By adding the Netlify _redirects file with these two rules any url accessed from the Netlify domain will be redirected to the Modulz domain.
For example:
https://modulz-radix.netlify.com/docs/playground/ will be redirected to https://radix.modulz.app/docs/playground/

Ohh yeah that's quite cool 😄
@benoitgrelard @StephenHaney what do you guys think?

Yep, fantastic. Definitely want a single URL. 👍

Done 👌

Thanks @aarongarciah 👍

Opening another PR with a fix, the _redirects file should be in the Gatsby /static folder 😬


Some submission types ending script

Mostly noticed this with audio posts, but it seems to throw an "ftp" before the http in the url and then the entire process comes to a stop
also had trouble saving stuff from fav galleries, it just seems to go in a loop without downloading anything other than the first page
great script btw, it's been really helpful outside of these couple of issues!

Saving favs has been fixed in this fork: https://github.com/BlueLycan/furaffinity-dl
There are also behavioral changes in that commit, so I cannot integrate it as-is. But you can use it for now as a stop-gap measure.

Link to a specific post where the FTP issue happens would be helpful

One of the submissions it happened with is http://www.furaffinity.net/view/26904990/
I tried the fork and it runs for a little bit, but doesn't seem to download more than one image and gives grep errors with each submission (no such file or directory), and then ends saying Cannot write to [filename] and /home/[user]/[git repos]/furaffinity-dl/furaffinity-dl: line 105: failed_downloads.txt: Permission denied.
Thank you for your time, it's really appreciated.

Downloading favorites is now fixed. Sorry it took so long - thesis happened.

I did not run into the specific issue with ftp:// you've described, it malfunctioned in a different way for me on the specified post. That is now fixed.
Please test the latest version and see if it works for you.

It's all good, life is important.
Had to change to the classic skin or it gave the same issue with tacking "ftp" in front of image urls, but it seems to work now as long as I'm using that skin. Thank you!

I didn't know there are skins now. How do I switch them?

I've tweaked the script to work with FA beta template. Please check if this fixes the issue.

It works for me on the beta skin, so I assume the issue has been fixed. Please reopen if that's not the case.


Can't open images when inside callouts

Image inside callout:

Clicked (it squishes). When using a mouse I can get it to work some times but it's random

@itsonlyjames  Thanks for the feedback on this issue. I'm trying to reproduce it, and it seems that this plugin can still work but only display the source code of that Callout block in Preview Mode. I haven't encountered any squished images and so could you please provide more details about that if you still got that problem.

@sissilab Interestingly, I no longer have any issues with opening an image inside a callout, whether by an update or other means I'm unsure (maybe I caused the issue myself in my vault with custom CSS or something).
Anyway, when opening the image inside the callout, it does make active the callout, and as such can interupt the flow of the page. I wonder if it's possible to add a preventDefault or something to stop "editing" the callout, like it would if you click on any part of the callout in edit mode.
Behaviour is different depending on whether you click out of the image, or press escape. Both scenarios are encompassed in the below gif, respectively:

@sissilab, I will close this issue as it is fixed and will look into the other request I made in a seperate PR. Cheers