Fix: EC2 controller for vpcendpoint does not update status

Issue : https://github.com/aws-controllers-k8s/community/issues/2075
Description of changes:
The issue is that vpce CR status is not updated. This happens because when vpce is created, it takes around a minute for it to be created in aws. Due to this, it's status is kept pending. But there is no sync enforced on the reconciler. Due to this, reconciliation happens after 10h default time. At this time, status would be set correctly.
This fix enforces reconciliation to make sure CR is updated correctly when vpce creation in aws is successful.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/ok-to-test

@nnbu can you also update the e2e tests? i think we might have to wait more than 5x30 seconds
/retest

@nnbu can you also update the e2e tests? i think we might have to wait more than 5x30 seconds /retest

@a-hilaly this is done now

/unhold

@nnbu @a-hilaly
I found a potential bug with this implementation: https://github.com/aws-controllers-k8s/community/issues/1933#issuecomment-2149307339
It seems that we stop syncing the state once the VPCEndpoint becomes available. However an available VPCEndpoint can still later be rejected and the controller will not update the state accordingly.

The fix addresses the issue for the case when the vpc endpoint goes from
pending to available.
Your latest comment is about the case when the state changes from
available to rejected. As you mentioned, the trigger for this
transition is manual trigger which is not managed/controlled by ack
controller.
It is not possible to handle such cases without reconciling every single
resource at a sufficiently high frequency. E.g. consider that if we
delete/change any resource out of the scope of ack controllers, then it
won't be reflected in ack immediately.
Ack has a default reconciling frequency of 10 hours. So, only at that
frequency, ack detects the change of state of the resource.
On Wed, Jun 5, 2024, 2:55 PM Junfeng Wu @.***> wrote:

@nnbu https://github.com/nnbu @a-hilaly https://github.com/a-hilaly
I found a potential bug with this implementation: aws-controllers-k8s/community#1933
(comment)
https://github.com/aws-controllers-k8s/community/issues/1933#issuecomment-2149307339
It seems that we stop syncing the state once the VPCEndpoint becomes
available. However an available VPCEndpoint can still later be rejected
and the controller will not update the state accordingly.
—
Reply to this email directly, view it on GitHub
https://github.com/aws-controllers-k8s/ec2-controller/pull/191#issuecomment-2149317083,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASMB42WQGIMFZJF3N3TRNULZF3KSLAVCNFSM6AAAAABICO2JPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBZGMYTOMBYGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Does it make sense to only force the resource to be unsynced only when
the resource is in pending instead of when it's in a non available state?

This makes sense totally. What are some of the transitionary states for vpc
endpoint, similar to pending? We should basically treat all such
transitionary states as notSynced.
@amine will be able to check with the service teams and get such states.
Depending on that, we can make the change. Your PR looks good to me if
pending is the only transitionary state.
On Wed, Jun 5, 2024, 4:42 PM Junfeng Wu @.***> wrote:

@.**** commented on this pull request.
In templates/hooks/vpc_endpoint/sdk_read_many_post_set_output.go.tpl
https://github.com/aws-controllers-k8s/ec2-controller/pull/191#discussion_r1627546438
:

if !vpcEndpointAvailable(&resource{ko}) {

// Setting resource synced condition to false will trigger a requeue of

// the resource. No need to return a requeue error here.

ackcondition.SetSynced(&resource{ko}, corev1.ConditionFalse, nil, nil)

} else {

ackcondition.SetSynced(&resource{ko}, corev1.ConditionTrue, nil, nil)

}

@nnbu https://github.com/nnbu @a-hilaly https://github.com/a-hilaly ,
if it makes sense to you, here's the fix I proposed on top of this change:
#198 https://github.com/aws-controllers-k8s/ec2-controller/pull/198
—
Reply to this email directly, view it on GitHub
https://github.com/aws-controllers-k8s/ec2-controller/pull/191#discussion_r1627546438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASMB42RVPZASY5FXIMXYM53ZF3XC7AVCNFSM6AAAAABICO2JPWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOJYHAZDMNJYGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Your PR looks good to me if pending is the only transitionary state.

pending and deleting are the only two transitionary states AFAIK and deleting should already be handled correctly assuming the resource is fully managed by the controller. It would be awesome if you can confirm this internally:)


[REST] Document /code-scanning/analysis/status

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/rest/authentication/permissions-required-for-github-apps?apiVersion=2022-11-28
What part(s) of the article would you like to see updated?
https://docs.github.com/en/rest/authentication/permissions-required-for-github-apps?apiVersion=2022-11-28#repository-permissions-for-code-scanning-alerts
Additional information
This is the only thing nearby, but it isn't technically the same api:

GET /repos/{owner}/{repo}/code-scanning/analyses
read
UAT IAT
✖

All I know is that it's possible to not have enough permission to access the endpoint: https://github.com/github/codeql/issues/15462

I believe /code-scanning/analysis/status is an internal API for telemetry purposes, it's not part of the public API. I wonder why it fails for you though. Perhaps GitHub Advanced Security / CodeScanning is not enabled for the repository for some reason.

Note: asking about this internally 💛

I'm on the owning team 👋 can confirm that this is not meant to be externally documented. We'll follow up on why it's returning a 403/404 sometimes though...

This issue can be closed as we're not planning to document this internal-only API. I'll write more detail in https://github.com/github/codeql/issues/15462 but we're planning on making sure failures from this endpoint don't fail the entire code scanning workflow, given that they are telemetry errors and unrelated to the actual analysis.


Public database import staralliance_prod (into mysql 5.5) hanging?

It was started yesterday afternoon and the UI still shows Importing...
Dani, could you please have a look what is going on in the background?
Thanks

According to the output of "show processlist" command, the import is still running:
| 4780 | sebike86 | localhost       | --obfuscated-- | Query   |    1 | update | INSERT INTO --obfuscated-- VALUES ('--obfuscated--',--obfuscated--,--obfuscated--,--obfuscated--,' |

Thanks, we'll wait then.


Cannot compile tensorflow lite minimal.cc with libtensorflowlite.so

System information
Ubuntu 16, armeabi-v7a, bazel 0.18, tried out with NDK 16b and 15c
Describe the current behavior
When I run this command to compile tensorflow/lite/examples/minimal.cc using a libtensorflowlite.so I built:
<path-to>/android-toolchain-15c/bin/clang++  -std=c++11 -I<path-to>/git/tensorflow-android -I<path-to>/git/flatbuffers/include -L<path-to>/git/tensorflow-android -L<path-to>/git/flatbuffers/build minimal.cc -ltensorflowlite -lflatbuffers
I get an error
undefined reference to 'tflite::InterpreterBuilder::operator()(std::__ndk1::unique_ptr<tflite::Interpreter, std::__ndk1::default_delete<tflite::Interpreter> >*)'
Describe the expected behavior
To compile.
Code to reproduce the issue

Download the android ndk. Call build/tools/make_standalone_toolchain.py --arch arm --api 21 --stl=libc++ --install-dir android-toolchain

Build libtensorflowlite.so, I added this to the BUILD file:

cc_binary(
    name = "libtensorflowlite.so",
    linkopts=[
        "-shared",
        "-Wl,-soname=libtensorflowlite.so",
    ],
    linkshared = 1,
    copts = tflite_copts(),
    deps = [
        ":framework",
        "//tensorflow/lite/kernels:builtin_ops",
    ],
)

And then called
bazel build //tensorflow/lite:libtensorflowlite.so --crosstool_top=//external:android/crosstool --cpu=armeabi-v7a --host_crosstool_top=@bazel_tools//tools/cpp:toolchain --cxxopt="-std=c++11"

Clone and compile the flatbuffer repo.

Then call the problematic command

android-toolchain/bin/clang++  -std=c++11 -Igit/tensorflow-android -Igit/flatbuffers/include -Lgit/tensorflow-android -Lgit/flatbuffers/build minimal.cc -ltensorflowlite -lflatbuffers

And you should get the error:
undefined reference to 'tflite::InterpreterBuilder::operator()(std::__ndk1::unique_ptr<tflite::Interpreter, std::__ndk1::default_delete<tflite::Interpreter> >*)'

I needed to be using NDK 18. I didn't think of using it previously because during ./configure a warning was output saying I should only <=16 was supported.


Connecting to MSSQL using UNC path

Hi.. I have an MSSQL server which can be located with \ABC\A where ABC is some string and A is DB name.  I didn't find any example to connect to MSSQL using such unc address. I could connect using IP and port i.e server=localhost;user id=xxx;password=xx";database=A. Can anyone help?
Thanks.
Aditya

Hi,
I need to connect to a database with addressing \ABCSQLServer\SampleDB ( username XX and password XX). I am not sure how to connect to this address. I find an example-
server=localhost\SQLExpress;user id=sa;database=master;app name=MyAppName
I feel I can replace it like -
server=localhost\ABCSQLServer;user id=XX;database=SampleDB;password=XX. Will this work? I am sorry I dont have direct access to db and can't test it. Thank you very much for your time.
Regards,
Aditya

Please readme for connection string format. If you need help please post to golang-nuts.


dind-rootless-latest need git as dependency

Hello,
I'm using vegardit/gitea-act-runner:dind-rootless-latest and trying to build a docker image.
Runner config
version: "3.8"
services:
  runner:
    image: vegardit/gitea-act-runner:dind-rootless-latest
    privileged: true
    environment:
      GITEA_INSTANCE_URL: https://git.**REDACTED**
      GITEA_RUNNER_REGISTRATION_TOKEN: **REDACTED**
      GITEA_RUNNER_NAME: **REDACTED**
      GITEA_RUNNER_MAX_PARALLEL_JOBS: 2
      ACT_CACHE_SERVER_HOST: **REDACTED**
      ACT_CACHE_SERVER_PORT: 22380
    ports:
      - 22380:22380
    volumes:
      - ./data:/data

Gitea action config
name: Build Server

on:
  push:
    branches:
      - main

jobs:
  docker:
    runs-on: ubuntu-latest
    container:
      image: catthehacker/ubuntu:act-22.04
    steps:
      - name: Checkout
        uses: actions/checkout@v4
      - name: Fix Docker Permission
        uses: https://github.com/imlonghao/gitea-runner-docker-permission-fix@main
      - name: Login to Docker Hub
        uses: docker/login-action@v3
        with:
          registry: git.**REDACTED**
          username: imlonghao
          password: **REDACTED**
      - name: Docker meta
        id: meta
        uses: docker/metadata-action@v5
        with:
          images: git.**REDACTED**-server
          tags: |
            type=sha
            type=raw,value=latest,enable=${{ github.ref == format('refs/heads/{0}', 'main') }}
      - name: Build and push
        uses: docker/build-push-action@v5
        with:
          push: true
          tags: ${{ steps.meta.outputs.tags }}
          file: Dockerfile.server

I got this error on the Build and push phase
[command]/usr/bin/docker buildx version
github.com/docker/buildx 0.12.0-1 542e5d810e4a1a155684f5f3c5bd7e797632a12f
::endgroup::
[command]/usr/bin/docker buildx build --file Dockerfile.server --iidfile /tmp/docker-actions-toolkit-E6zD2Y/iidfile --secret id=GIT_AUTH_TOKEN,src=/tmp/docker-actions-toolkit-E6zD2Y/tmp-149-vIMXPDX5m5Bl --tag git.**REDACTED**-server:latest --tag git.**REDACTED**-server:sha-5c1d154 --metadata-file /tmp/docker-actions-toolkit-E6zD2Y/metadata-file --push https://git.**REDACTED**/imlonghao/**REDACTED**.git#**REDACTED**35d02e1754
#0 building with "default" instance using docker driver
#1 [internal] load git source https://git.**REDACTED**.git#**REDACTED**135d02e1754
#1 ERROR: failed to init repo at /data/.local/share/docker/fuse-overlayfs/5xrld60bvz69ds96yah8z4d6z/diff: exec: "git": executable file not found in $PATH
------
 > [internal] load git source https://git.**REDACTED**.git#**REDACTED**135d02e1754:
------
ERROR: failed to solve: failed to read dockerfile: failed to init repo at /data/.local/share/docker/fuse-overlayfs/5xrld60bvz69ds96yah8z4d6z/diff: exec: "git": executable file not found in $PATH
::error::buildx failed with: ERROR: failed to solve: failed to read dockerfile: failed to init repo at /data/.local/share/docker/fuse-overlayfs/5xrld60bvz69ds96yah8z4d6z/diff: exec: "git": executable file not found in $PATH

Patching the image with this Dockerfile works for me
FROM vegardit/gitea-act-runner:dind-rootless-latest
RUN sudo apt update && sudo apt install -y --no-install-recommends git && sudo rm -rf /var/lib/apt/lists/*

I suggest we can add git to https://github.com/vegardit/docker-gitea-act-runner/blob/55af759535bfb6b05a683816358db58c50da0d99/image/Dockerfile#L64

Interesting, I don't really understand why this is necessary. To my understanding all commands incl. git should be executed within the catthehacker/ubuntu:act-22.04 container and not on the host.

Yeah, it's a little weird behavior.
I have another action worker running in vegardit/gitea-act-runner:latest image, which works fine without the git installed.
So I sometimes just keep restarting the action job until it got assign to this runner.

still an issue

Having something similar happen on the dind image:
name: Build and push container
on: [push]

jobs:
  release-image:
      runs-on: ubuntu-latest
      steps:
        - name: Checkout
          uses: actions/checkout@v4
          with:
            fetch-depth: 0 # all history for all branches and tags

        - name: Log in to the Container registry
          uses: docker/login-action@v3
          with:
            registry: http://xxx
            username: root
            password: ${{ secrets.GITEAROOT }}

        - name: Build and push
          uses: docker/build-push-action@v6
          with:
            file: ./a/Dockerfile
            allow: network.host,security.insecure
            push: true
            tags: |
              xxx/root/b:1.0.0.${{ github.run_id }}
              xxx/root/b:latest

worked recently but now has a
ERROR: failed to solve: failed to read dockerfile: failed to init repo at /var/lib/docker/overlay2/kcd6252t1mjm4xik53ozs8d30/diff: git stderr:
: exec: "git": executable file not found in $PATH
::group::Reference
default/default/b7bazlswz6loj7fmds4iinjbj
::endgroup::
::group::Check build summary support
Build summary is not yet supported on GHES
::endgroup::
::error::buildx failed with: : exec: "git": executable file not found in $PATH

error


Added collapsible text

Made the lists of items excluded from the table collapsible

Looks good, thank you for contributing to the project!


build: enable safeicf for lld

sync shinchiro's changes.

I'll merge this if no one reports bugs after shinchiro releases an update in sourceforge that includes this option. Let's wait a few days.


Added some text for Ckpt 48 pull request example

@ryanbalfanz An example of a pull request as required per Checkpoint #48.

:+1:


Update EventPoller.cpp

可能err一直不等于UV_EAGAIN,然后一直在这循环

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


added attr parsing support

added attr parsing support
added _id row support

#49, #50
For now in simple rule we can set:

attr: 'attr-name' - if specified will be parsed from attribute, otherwise from textContent
id: true | Function, will be set with name _id: {parsing result or result of function call}

@jifeon for review

cool stuff!


Roar::Hypermedia uninitialised constant Representable::Inheritable

Roar::Hypermedia 1.0.3 (and I think 1.0.1) refer to Representable::Inheritable which appears to have disappeared after Representable 2.3 (I think).  Means Roar::Hypermedia isn't compatible with later versions of Representable.  Error detected using Representable 2.4 and Representable 3.0 but its Roar that is making the reference.
NameError: uninitialized constant Representable::Inheritable
from /Users/kip/.rvm/gems/ruby-2.2.2/gems/roar-1.0.3/lib/roar/hypermedia.rb:50:in `link_configs'
from /Users/kip/.rvm/gems/ruby-2.2.2/gems/roar-1.0.3/lib/roar/hypermedia.rb:128:in `link'

Please upgrade to Roar 1.0.4.

Nick, thanks for the response.  I think 1.0.4 has a dependency challenge:
Bundler could not find compatible versions for gem "representable":
In Gemfile:
roar (= 1.0.4) ruby depends on
representable (< 2.4.0, >= 2.0.1) ruby
Meaning it won't run with either Representable 2.4 or 3.0.  An unintended regression?

Sorry, no! :grimacing: Roar 1.0 doesn't work with Representable > 2.4. You have to pull roar/master until I release 1.1, which should happen "this year" haha.

Thanks Nick, no problem, I'm running on edge rails so whats one more flexible moving part!

Rails 5? lol

I enjoy the pain :-), but actually its working out pretty well for me.


Fix GLTF scene dependencies and make full scene renders predictable

Objective
Fixes #10688
There were a number of issues at play:

The GLTF loader was not registering Scene dependencies properly. They were being registered at the root instead of on the scene assets. This made LoadedWithDependencies fire immediately on load.
Recursive labeled assets inside of labeled assets were not being loaded. This only became relevant for scenes after fixing (1) because we now add labeled assets to the nested scene LoadContext instead of the root load context. I'm surprised nobody has hit this yet. I'm glad I caught it before somebody hit it.
Accessing "loaded with dependencies" state on the Asset Server is boilerplatey + error prone (because you need to manually query two states).

Solution

In GltfLoader, use a nested LoadContext for scenes and load dependencies through that context.
In the AssetServer, load labeled assets recursively.
Added a simple asset_server.is_loaded_with_dependencies(id)

I also added some docs to LoadContext to help prevent this problem in the future.

Changelog

Added AssetServer::is_loaded_with_dependencies
Fixed GLTF Scene dependencies
Fixed nested labeled assets not being loaded

Looks good, but any chance we add tests for some of these things so they don't regress in the future?

I'm on board for more tests, but that will require engineering effort that would otherwise delay 0.12.1 so I'm not going to block on that.


use github repo.

No. That's wrong. Use gopkg.in/olivere/elastic.v3 if you want to use the stable version of Elastic for ES 2.x.


Feature: http_sd monitor

What's changed?
#1526
支持Http sd监控。
因使用Spring Cloud提供的统一接口DiscoveryClient对代码改动较大，目前采用了各个注册中心原生的SDK进行监控。

目前只添加了最常见的两种注册中心：Consul和Nacos，如果后期需要接入更多注册中心，我开放了一个接口org.dromara.hertzbeat.collector.collect.httpsd.discovery.DiscoveryClient，直接实现该接口即可。
代码位置：

本地调试结果：

Checklist

[x]  I hereby agree to the terms of the HertzBeat CLA
[x]  I have read the Contributing Guide
[x]  I have written the necessary doc or comment.
[x]  I have added the necessary unit tests and all cases have passed.

Add or update API

[x] I have added the necessary e2e tests and all cases have passed.

NB Plus!

@all-contributors please add @Calvin979 for doc, code, design

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

@all-contributors please add @Calvin979 for doc, code, design

Great! This feature we support monitoring nacos consul throught http_sd protocol. How about this other feature, we auto discovery monitoring instance throught http_sd, like prometheus discovery. Like this:

http request http_sd api to get service instance ip, port and more.
auto create monitor to monitoring service instance throght prometheus protocol

This is more complicated and needs to be designed carefully.

I'd like to try! Should we create another issue to trace it?

I'd like to try! Should we create another issue to trace it?

welcome, yes we can create a issue to trace and discuss it.

I'd like to try! Should we create another issue to trace it?

welcome, yes we can create a issue to trace and discuss it.

I've created an issue.
#1625


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/82130

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/85878

TagBot fix

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/88724

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/91227

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/98740


added support for a yum proxy and setting the baseurl for yum repositories

we are using an internal mirror of the puppetlabs repo which is only accessible via a proxy. so this pull requests adds support for specifing

yum proxy url
yum proxy username
yum proxy password
baseurl for the dependencies repo
baseurl for the products repo

and the corresponding rspec tests.
the defaults did not change, so existing installations are not affected by this change.

any news on this?
thanks
toni

Lgtm, sorry for the delay.

I'll merge tomorrow.

thanks!
toni


No confirmation before deleting storage service

We should probably ask for conformation before deleting a storage service.

Should also add confirmation before deleting a fetch job.

PR for CR: https://github.com/artefactual-labs/AIPscan/pull/353/files


H5 Minute Reader Bug in Python 3

If an equities or futures update file was written in python 2 but read in python 3, pandas.read_hdf would error trying to read the tz attr in the file.

Coverage decreased (-0.06%) to 88.149% when pulling 922c1c525d53477552117417460cc02ba2d25c45 on read-hdf-fix-py3 into 837b1e44c7a0090fa1b8ff261a771778cd19f5ce on master.

Right, thanks, I'll fix and then merge.


Unable to launch amplify command with npm scripts

Describe the bug
When I try to run amplify through npm scripts i have errors whereas launching it in the command line cause me no errors
To Reproduce
Steps to reproduce the behavior:

Add amplify cli to your dev dependencie project
Add some command in a npm script (eg. publishThings)
npm run publishThings
feel the despair

Expected behavior
Be able to run amplify cli through npm scripts
Error
home/user/Documents/project/node_modules/gluegun/build/index.js:13
    throw up;
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "id" argument must be of type string. Received type undefined
    at Module.require (internal/modules/cjs/loader.js:631:11)
    at require (internal/modules/cjs/helpers.js:22:18)
    at context.exeInfo.projectConfig.providers.forEach (/home/user/Documents/project/node_modules/@aws-amplify/cli/src/commands/env/checkout.js:39:30)
    at Array.forEach (<anonymous>)
    at Command.run (//home/user/Documents/project/node_modules/@aws-amplify/cli/src/commands/env/checkout.js:38:45)
    at Runtime.<anonymous> (/home/user/Documents/project/node_modules/gluegun/build/runtime/run.js:90:58)
    at step (/home/user/Documents/project/node_modules/gluegun/build/runtime/run.js:32:23)
    at Object.next (//home/user/Documents/project/node_modules/gluegun/build/runtime/run.js:13:53)
    at/home/user/Documents/project/node_modules/gluegun/build/runtime/run.js:7:71
    at new Promise (<anonymous>)

Desktop (please complete the following information):

OS: Linux
Amlify: 1.1.6

@clementAC Amplify CLI should run when launched thru an NPM script -- I use them daily without any problems. Can you post your NPM script for reference. From your error message re The "id" argument must be of type string I think it might be something else. Some possible suggestions:

Have you tried a clean install of the latest Amplify CLI npm install -g @aws-amplify/cli@latest
Initiating a new amplify project amplify init
In your package.json file, you don't have to do anything fancy except reference the Amplify CLI command, an example "scripts": { "gql-compile": "amplify api gql-compile" }

@clementAC what command are you trying to run from npm script. Could you share the scripts section from your package.json

Ok my bad. I had amplify cli into my project dependency and npm script was using this one.
After removing it, the global install was taken and everything was fine.  Thanks for your time @lennybr @yuth


Fix for issue 88 that fragmenter settings in pipeline reflect back to fragmentation settings in main view

Now made sure that fragmenter instances are always copied when used in the pipeline settings view controller.
Also some renaming to make things more clear and also added a note about pipeline fragmenter settings to the tutorial.

@FelixBaensch, very sorry to bother you, but could you maybe test and review this. Please forgive me for the refactoring that makes the "actual" changes harder to see.

@FelixBaensch, very sorry to bother you, but could you maybe test and review this? Please forgive me for the refactoring that makes the "actual" changes harder to see.

I will leave these extraordinarily long variable names uncommented....
But I will review it in the next few days

@FelixBaensch I have restructured the code and removed everything that I regarded as legacy code. Could you maybe test whether I broke sth and whether the original issue is fixed?

@FelixBaensch I have made adjustments according to your requested changes. Do you approve?

Merge at will


Consistent New Hover Effect on "Book Now" Buttons

Title and Issue number

Title : Consistent New Hover Effect on "Book Now" Buttons in topdeals.html file
Issue No. : #1611
Code Stack :  html, css
Close #1611

Description

The "Book Now" button has a smooth 3D flip hover effect:
Initial State: The button is styled with a bold red background (#ff5a5f) and white text, with padding and rounded corners.
Hover Effect: On hover, a secondary color layer (#ff787d or #ffa3a6) flips up from below, replacing the original text with a new label.
Smooth Transition: The flip effect is achieved with a rotation animation, giving a dynamic yet subtle 3D appearance that draws attention without being distracting.
Video/Screenshots (mandatory)

https://github.com/user-attachments/assets/bf5f1fbe-ef36-4e8b-82f8-4797d3940c45
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[X] I have performed a self-review of my code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have gone through the  contributing.md file before contributing
[X] I have Starred the Repository.

Additional context:

##Are you contributing under any Open-source programme?

[X] I am contributing under GSSOC'24 Extended
[X] I am contributing under Hacktoberfest'24

@apu52  can you review this so i can open new pr for packages.html styling

@apu52 can you review this so i can open new pr for packages.html styling issue #1610

Make a new branch in your forked repo & try to push all of the changes there for the next issue also & make a separate PR , Meanwhile I am reviewing this PR

got it thank you


[test] Update simd checks for Alps

Tested with nsimd/3b6e339-cpeGNU-20.10 and nsimd/3b6e339-CrayGNU-20.10

Codecov Report

Merging #1667 (3c13503) into master (01d7efc) will decrease coverage by 0.29%.
The diff coverage is 45.28%.

@@            Coverage Diff             @@
##           master    #1667      +/-   ##
==========================================
- Coverage   87.54%   87.24%   -0.30%
==========================================
  Files          45       45
  Lines        7492     7545      +53
==========================================
+ Hits         6559     6583      +24
- Misses        933      962      +29

Impacted Files
Coverage Δ

reframe/core/modules.py
56.62% <45.28%> (-1.21%)
:arrow_down:

Continue to review full report at Codecov.

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

retest this

@jgphpc The modules are not present on Eiger, are they?

@jgphpc The modules are not present on Eiger, are they?

@vkarak not installed but the recipes are available:
https://github.com/eth-cscs/production/blob/master/easybuild/easyconfigs/n/nsimd/nsimd-3b6e339-cpeGNU-20.10.eb

@ekouts Can you approve this one?


Brewing Stand Quests Bypasses

What is your Quests version / build number (do not say "latest")?
4.8.1-b404
Describe the problem and how to replicate it.
Any quest created that requires brewing a potion can be easily bypassed by simply putting the correct potion into the brewing stand, taking it out of the brewing stand, and then putting it back. So all a player needs to do is own exactly 1 potion that the quest is calling for, accept the quest, and just spam putting in and taking it out of the stand to complete the quest. Having it like this makes all brewing quests entirely pointless. An option similar to "ignore-block-replace: true" for break/place quests should exist for other types of quests. Brewing, smelting, crafting, etc should all behave like that to prevent easy abuse.
Any console errors to share?
n/a

If applicable, can you provide an example quest from quests.yml?
custom2:
    name: Witching 101
    ask-message: The trick is getting the nether wart before the lava gets you. Get
      $30,000 for completion!
    finish-message: Rewarded for completing Witching 101!
    stages:
      ordered:
        '1':
          items-to-brew:
          - ==: org.bukkit.inventory.ItemStack
            v: 3337
            type: POTION
            amount: 64
            meta:
              ==: ItemMeta
              meta-type: POTION
              potion-type: minecraft:fire_resistance
    rewards:
      money: 30000
      quest-points: 1
    planner:
      cooldown: 86400
      override: false
    options:
      allow-commands: true
      allow-quitting: true
      ignore-silk-touch: true
      use-parties-plugin: false
      share-progress-level: 1
      same-quest-only: true
      share-distance: 50.0
      handle-offline-players: false
      ignore-block-replace: true

@PikaMug As far as I can see, this just blocks brewing quests completely. It won't even allow you to place your water bottles into the brewing stand anymore using build #406

@HorusApl Thanks for following up. This build should allow placement of empty bottles:
https://ci.codemc.io/job/PikaMug/job/Quests/408/

Shows that build as failing, is it safe to test?

Shows that build as failing, is it safe to test?

Yes, it's safe.

Just tested, still very glitchy and still doesn't appear to function. I can sort of place empty glass bottles in the brewing stand with, but you have to spam click it a bit to get it there. I cannot get water bottles in still.

https://ci.codemc.io/job/PikaMug/job/Quests/409/

Just tested this and I can't see a difference between this and the last update. Issue still occurs.

Not having used brewing in a while, I thought empty Glass Bottles were supposed to go in the base slots, when it's actually Water Bottles. The last build corrects that. I'm not having any issues placing Water Bottles in the bottom slots.

I've tried this build and I don't see any difference. If you see my other ticket, I am definitely downloading build 409 but /version says it's build 406.


'festival' is not recognized as an internal or external command, operable program or batch file.

Hi,
I am using Windows 10 and Python 3.7.
When I run the example file I am getting following error:-
'festival' is not recognized as an internal or external command, operable program or batch file.

I have followed the below link to install festival in windows-
http://homepages.inf.ed.ac.uk/jyamagis/misc/Practice_of_Festival_speech_synthesizer.html
festival is working fine, please see below screen shot-

Let me know if I am missing anything here.

link not working


Json as return format from script

Let's start with simple json object containing:

status
output
errorOutput


Clean SkipPasswordPatternValidationThreadLocal properly

Purpose
Resolves https://github.com/wso2/product-is/issues/21143 and https://github.com/wso2/product-is/issues/20261

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/11365272374

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/11365272374
Status: success


Strip optional falsy/nullish values from checked nodes

Currently in badgecheck/tasks/validation.py validate_property():
    if not required and (is_empty_list(values_to_test) or is_null_list(values_to_test)):
        return task_result(True, "Optional property {} is null in {} {}".format(
            prop_name, node_class, node_id
        ))
        # TODO Return STRIP_PROPERTY action

It would be nice in many cases to get consistent behavior that optional properties that are null, empty list, empty string etc. just get stripped out of the resultant graph.
We should get consensus on whether that is the right thing to do.

I agree that this kind of flexibility should be permitted for non-required properties.   We should allow for empty and null values including:

null
empty lists
"" or even " " empty strings


Fixed the defaultFeatureId problem that could potentially cause ArrayIndexOutOfBound problem

When we recompute featureToIndex, we need to reset all defaultFeatureId to zero (which belong to static Feature members in the Feature classes and are only set to zero once during initialization). This is handled by a resetDefaultIndex() method in Feature, which is called in makeKeyToIndexMap() in Training.java.

Other minor fixes: in PennTreebank.java, changed the file matching regex to also consider .MRG files for future convenience ...; in Training.java, changed names/added comments of several boolean variables.

This is great, thanks a lot! :-)


fix trasparency and mouse interaction

This commits fix trasparency mode and move all the listeners from canvas to window to enable fixed position for the canvas as background of the entire page. See the updated playground for better understanding.

I have a large upcoming rewrite of the package that solves this so I wont merge it. Regarding the changing from canvas to window, I want to be able to position the simulation anywhere and not limit it to be just a background on the webpage. I think all of this is fixed in the new version already


Get rid of CRDS directory.

Possibly get rid of crds directory structure with this?
CreateReplace seems new, or at least new to me.
https://github.com/onedr0p/home-ops/blob/6e4739e25ea053324e919ab4443741a7b154e204/kubernetes/apps/monitoring/kube-prometheus-stack/app/helmrelease.yaml#L22
https://github.com/fluxcd/helm-controller/blob/main/docs/api/helmrelease.md

Turned off the kube-stack crds. Need to look at metallb and cert-manager.


hydrator dupe 1.12.2

Minecraft version & Mod Version:
forge - <IP_ADDRESS>07
minecraft - 1.12.2
cyclic - 1.12.2-1.11.1
Single player or Server:
server
Describe problem (what you were doing / what happened):
placed item into the bottom left slot (red concrete powder) and it continued to make concrete but the powder did not get used.
Crash log (if any):
null

Yeah good find! It was trying to drain the recipe cost from top left only!

Resolved in https://minecraft.curseforge.com/projects/cyclic/files/2530166

Seems this is still a valid issue in 1.14.9 @PrinceOfAmber
https://i.artdude.uk/jo5Jhdir.mp4

Sorry i didnt notice the update to to this for some reason, must have got lost in the github emails . on my radar now

@artdude543  how did you even get a recipe of size 2 in the hydrator?
I dont know why i coded it this way, but when i use script
Hydrator.addRecipe(<minecraft:gravel>, [<minecraft:sand>,<minecraft:sand>], 50);
i get error in log
 Error executing {[0:crafttweaker]: hyd.zs}: Input array must be length 1 or length 4
Assuming that grey block in video is hardened_clay, it looks like its it is using that shapeless recipe of 1 grey sand to 1 clay.  BUT theres still a bug here, its assuming the top left slot has ingredient, so it should be reducing one from your lower left stack in that video

If its not too much work ill see about allowing size 2/3 recipes, no idea why i had it that way


Library for Arduino?

Have you thought about making this as a library for the Arduino IDE? I think it get a lot more usage since the current web server possibilities are quite unnstable.

Thanks, that's a good idea. It will require some core changes though (e.g. add option to choose static allocation) so this may take a while. And I could use some help since I don't even have arduino yet.

Cool, looking forward. Let me know if you need help with arduino IDE or
testing.
Am 07.11.2015 4:46 nachm. schrieb "solusipse"<EMAIL_ADDRESS>
Thanks, that's a good idea. It will require some core changes though (e.g.
add option to choose static allocation) so this may take a while. And I
could use some help since I don't even have arduino yet.
—
Reply to this email directly or view it on GitHub
https://github.com/solusipse/ureq/issues/6#issuecomment-154717127.

basically you can find all information here in case you did not know yet:
https://github.com/esp8266/Arduino


unable to close the image upload modal and field title is missing with starpi v4

node version : 14.16.0
npm version: 6.14.11
os: windows 10
Database: mySql 8.10
strapi-plugin-react-editorjs: 2.0.0-beta.1

Same problem here.

@ganeshmohanty @nourseofcourse. Sorry, I didn't see your comments here, as I'm not a maintainer of this project. However, see the below comment for how to get the updated changes until those changes are merged.
https://github.com/melishev/strapi-plugin-react-editorjs/issues/11#issuecomment-1063523065

@jaskipper thanks for your reply. can you add one more step which is pull your pull request and merge it to beta branch else the image  modal and link will not work.

@ganeshmohanty - Actually, that step is not necessary if you download just the beta-v4 branch. I removed the first part of the instructions that referenced the main branch. I think that is what you must have done. If you clone with the below command, you only get the beta-v4 branch.
git clone --single-branch --branch beta-v4 https://github.com/jaskipper/strapi-plugin-react-editorjs.git

@jaskipper - Thanks for helping the community
@ganeshmohanty , @nourseofcourse - today or tomorrow, we will release a new update for the beta

@ganeshmohanty , @nourseofcourse
Well, the bot didn't send out an automatic beta update message, so I do it. Please check if your issue is fixed in the new beta version⚠.

💥 Well guys, I've released the beta to the official release.


Support print build date and version

If there is a docker mirror cache, printing out the build date and version is good for debugging
https://github.com/kubesphere/tower/blob/786fdd16ea6f4001acaa1e15c56a73ecd6545478/.github/workflows/go.yml#L56-L57

Can I do it ?


optimize generated tables ; clean up unicode.py

There was an easy opportunity to better optimize the tables generated
by unicode.py. Not sure why I didn't catch this long ago, but in any
case now the tables are substantially smaller and should maybe improve
performance slightly.
There was also some dead code sitting in unicode.py that I pulled out.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

Could you give a high-level summary of how the tables changed? (github isn't showing the diff and I'm finding it hard to visualise from just the python changes.)

My suspicion is that this script was once used to generate other Unicode data that wasn't used by regex (e.g., east asian character widths?). So this is probably trimming it back down?
@kwantam I'm still interested in a high level summary though. I think I wrote the original version of this over a year ago for regex stuff only, but I no longer understand the code.

Quick background: the property tables that unicode.py generates are sorted arrays of non-overlapping ordered pairs of chars, (x, y), x<=y. When matching against a certain property (say, with \p{Alphabetic}), the regex code does a binary search in the corresponding table to identify whether there is a pair that contains a given character.
Now, in some cases, the tables that were previously generated contained ordered pairs that should have been merged together. For example, Alphabetic_table previously contained a sequence like this:
...
    ('\u{d8}', '\u{f6}'), ('\u{f8}', '\u{1ba}'), ('\u{1bb}', '\u{1bb}'),
    ('\u{1bc}', '\u{1bf}'), ('\u{1c0}', '\u{1c3}'), ('\u{1c4}', '\u{293}'),
    ('\u{294}', '\u{294}'), ('\u{295}', '\u{2af}'), ('\u{2b0}', '\u{2c1}'),
...

Note that, for example, ('\u{f8}', '\u{1ba}'), ('\u{1bb}', '\u{1bb}') can be collapsed into a single range, because 0x1bb = 0x1ba + 1. In fact, the above sequence, in the new version of the table, is just
...
    ('\u{d8}', '\u{f6}'), ('\u{f8}', '\u{2c1}')
...

In case you're wondering, the reason this was happening is because of the way that the Unicode-provided tables were formatted. The function that imports these tables, load_properties(), was written with the assumption that all sequential ranges in the tables were collapsed, but this isn't the case. The very small addition, among all the cuts in unicode.py, that fixes this, is at the end of the load_properties(), function:
for prop in props:
    props[prop] = group_cat(ungroup_cat(props[prop]))

Nice! Thanks @kwantam!


asset_policy_info + asset_address_list

Is your feature request related to a problem? Please describe.
I would like to limit the amount of requests needed to grab address information for all policy tokens. Right now you can grab all tokens to a policy but in order to see who owns them you have to query one by one.
Describe the solution you'd like
One request that would query a policy [asset_policy_info] and include [asset_address_list] for each token.
Describe alternatives you've considered
Hosting my own cardano-dbsync (again, its annoying though) or cardano-graphql to do own queries...
Additional context
Working on a d3js visualization for a few key areas of cardano ecosystem and for this specific feature, it would be for a policy(ies) overview. This shows all related accounts to specific policy(ies) to see the distribution of tokens visually.

Unfortunately, it's not something doable for a free/public instance as it's an easy way to DDoS an instance provider. IMO - it's a requirement that conflicts with the design of how ledger was created (within public API realm)
Do acknowledge that it might definitely handicap your requirement, but given that policy to asset map as well as asset to tx (which cotains address) is infinite, it's not a sustainable endpoint to workaround against.
Instead this would likely be better served via Koios Lite (part of catalyst proposal 9) which could have much smaller subset of queries to host and return within public instance realm. In the meanwhile, one can run Koios instance locally and create a custom cache table OR create filtered asset dumps that they're  interested in.

Makes sense.
Dbsync can be annoying (it is annoying) and the Koios Lite would be ideal to circumvent that requirement.

Will just keep this open so it's not lost from radar (as it's increasingly common requirement in DeFi world)

Instead of increasing amount of data being queried/returned on infinite map, a better outcome might be reducing info on asset_policy_info by splitting it into asset_policy_list and allowing bulk queries to asset endpoints


[BUG] Docs: ECJ page links to missing pom.xml

This page https://projectlombok.org/setup/ecj links to https://projectlombok.org/files/pom.xml which gives 404

It should link to https://projectlombok.org/setup/pom.xml I think


volcano plots legend has duplicate entries

config.yaml
volcano:
    pCutoff: 0.05
    FCcutoff: 0

probably fix by adding something like drop=FALSE during plotting


Fix encoding for problem characters

This is a slightly simpler fix for #19 than @non's #28. Instead of checking the entire JSON document for bad characters and setting the default style accordingly, we check each individual string (and object key) and change the style to be double-quoted for that node specifically.
After this change Erik's tests pass, my new object key tests pass, and in a dozen or so runs I've not seen the occasional errors we'd been getting from the round-trip tests, so I'm pretty sure this works fine.

Also it's confirmed by MiMa to be binary compatible with 0.6.0, so if this looks okay to everyone I'll publish it as 0.6.1 tonight.

Codecov Report

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

@@          Coverage Diff          @@
##           master    #29   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines          99    102    +3
  Branches        4      6    +2
=====================================
+ Hits           99    102    +3

Impacted Files
Coverage Δ

src/main/scala/io/circe/yaml/Printer.scala
100% <100%> (ø)
:arrow_up:

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 b6a7c5b...a71ed68. Read the comment docs.


Starting multiple services

When i'm trying to run more than one service, i'm getting the error:
unable to bind domain socket with id=0 (see --base-id option)
Error: exit status 1

I think this is related to Envoy proxy!!!

@junr03 thanks for the response. I'm using service mesh to communicate between microservices, for example we have two microservices, and when we start each microservice we start with it an Envoy data plane. the problem is when i want to start the second service, Envoy can't run

From your error it looks like you are running both services (and their corresponding envoy process) on the same machine, is that correct?
Did you take a look at the linked issue, is your issue like that one?

Yes you are right i'm trying to run two microservices in the new kuma service mesh witch use Envoy like a dataplane


Class table block

Make a replacement class table block using a div class. Replacement the current snippet

Added in v2.2.1


Modbus coil support

Everything looks good to me! Thanks for making those extra updates I mentioned as well! Let's wait for Paulus to take a final look at this one when he gets back, but it looks good to me!

Looks good to me! Awesome!


doesn't work properly with react-css-modules

demo
Only inner styleName compiles to 'class', but inners not. You can see that div-2 value has stylename attribute, but should class as div-1.

Component code:
import React, { Component } from 'react';
import CSSModules from 'react-css-modules';
import styles from './styles.css';

@CSSModules(styles)
export default class App extends Component {
  render() {
    return (

          Big square should be red;
 Small square should be black.

    );
  }
}

styles.css:
.div-1 {
  width: 500px;
  height: 500px;
  background: red;
}

.div-2 {
  width: 200px;
  height: 200px;
  background: black;
  color: white;
}

How it looks:

How it should:

react-css-modules is not supported in preact core.
If you are willing to change the way you use CSS modules you can make it work today by changing your App.js file to be like this:
import React, { Component } from 'react';
import styles from './styles.css';

export default class App extends Component {
  render() {
    return (

          Big square should be red;
 Small square should be black.

    );
  }
}

This way you don't need the decorator nor react-css-modules.

@mkxml seems like he is using preact-compat - I was looking through the react-css-modules source to see if I could find where they're doing the VNode manipulation. Maybe just an issue with vnode.props being non-enumerable?

@developit Hmm, with preact-compat it should work. Updated my comment.

indeed - your comment is how I use CSS modules too 👌

It's very verbose to write styles every time, css-modules allows to write it only once, that's why I use it. Will be very great for me if this will be fixed, I want to try preact in production. Thanks.

@mqklin understandable! This will likely be solved over in preact-compat, but I'm fine tracking it here to avoid lots of issue juggling.

Any progress on this issue? Maybe I should create issue in react-css-modules repo?

@mqklin Might be useful - link me in there so maybe we can get more context. I'm not super familiar with react-css-modules, so any help is very much appreciated!

I've migrated to styled-component. I don't think someone needs this issue to be fixed, so I'm closing it.

so sad, hoped it to be fixed... =/

@developit maybe it could get a bit of attention for upcoming release of Preact X? I know this issue was a kind of dead-end, but I'm ready to help if had some clues where to start (have a little to no experience in Preact/Babel plugins development).
Currently react-css-modules moved to babel-plugin-css-modules, and here is corresponding issue in its repo: https://github.com/gajus/babel-plugin-react-css-modules/issues/250.
As for me, this is real helper in React project as you can switch from writing className={styles.foo} to just styleName="foo", and would be awesome to use it in Preact projects as well.


Add extension points for build scripts

Signed-off-by: Steve Groeger<EMAIL_ADDRESS>Checklist:

[x] Read the Code of Conduct and Contributing Guidelines.

[x] Followed the commit message guidelines.

[ ] Stack adheres to Appsody stack structure.

This PR adds extension points to CI build scripts
Fixes: https://github.com/appsody/stacks/issues/183

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

Does there need to be an extension point for lint.sh and package.sh?

@groeges We should look to add extension mechanisms for pre-

Looks OK. Though I'd add extension for each of the scripts..

Have added extension points to all CI scripts


Update cats-core to 2.6.1

Updates org.typelevel:cats-core from 2.3.1 to 2.6.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-core" } ]

labels: library-update, semver-minor

Codecov Report

Merging #20 (f77bee0) into main (a211e9f) will increase coverage by 2.76%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
+ Coverage   76.39%   79.15%   +2.76%
==========================================
  Files          17       17
  Lines         572      547      -25
  Branches       29       29
==========================================
- Hits          437      433       -4
+ Misses        135      114      -21

Impacted Files
Coverage Δ

...in/scala/io/circe/schema/validation/Compiler.scala
97.59% <0.00%> (-0.05%)
:arrow_down:

...a/io/circe/schema/validation/ValidationError.scala
12.50% <0.00%> (+5.83%)
:arrow_up:

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 a211e9f...f77bee0. Read the comment docs.


Enable no-errors-in-logs check by default, and extend it to all Cilium components

Currently, we check for errors in Cilium agent logs only when unsafe tests are enabled. Let's promote this check to be always performed for Cilium v1.14 and subsequent (earlier versions are not compatible with the Cilium CLI v0.15 series), to ensure that we catch unexpected errors early (as unsafe tests are only enabled in a limited set of CI workflows) given that it does not perform any unsafe operation. Additionally, let's extend it to also cover the init containers in the cilium agent pods, as well as other cilium components, such as the operator and the clustermesh-apiserver.

Additionally tested in https://github.com/cilium/cilium/pull/30604 and https://github.com/cilium/cilium/pull/29890


Added -a and --auth flags for basic auth

I've been using crul and so far it's great :-)
But when I need basic auth I need to switch back to curl :-(

Wow, thanks a lot, I'm also very glad that you found crul useful :smile:

And I'll make a release tonight with this included!


As a manager/reviewer, a "Reading view" for applications, creating a button to hide/expose the reviews block and other elements, creating wider margins for app text.

I did some experimenting and by replace div with p in opentech/apply/stream_forms/templates/stream_forms/render_field.html I get Safari to recognise the submission as an article and the reader view becomes available.
https://mathiasbynens.be/notes/safari-reader


Undefined symbol xla::HloComputation::CollectUnreachableRoots() const with tf-nightly pip package

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf-nightly-cpu==2.12.0.dev20221019 and newer
Custom Code
Yes
OS Platform and Distribution
Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Starting from version 2.12.0.dev20221019 Horovod cannot be built correctly with tf-nightly. tf-nightly-cpu==2.12.0.dev20221018 and earlier are fine.
Upon import of horovod.tensorflow an undefined symbol error is encountered:
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.8/dist-packages/horovod/tensorflow/mpi_lib.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZNK3xla14HloComputation23CollectUnreachableRootsEv

In demangled form that is xla::HloComputation::CollectUnreachableRoots() const.
Has the implementation been moved to a different dynamic library recently? Horovod currently links to libtensorflow_framework.so.2 and to _pywrap_tensorflow_internal.so.
Standalone code to reproduce the issue
$ pip install tf-nightly-cpu
Collecting tf-nightly-cpu
  Downloading tf_nightly_cpu-2.12.0.dev20221024-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (225.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 225.0/225.0 MB 1.2 MB/s eta 0:00:00
# ...
$ pip install -v horovod
# ...
  Tensorflow_LIBRARIES := -L.../horovod-tf-nightly-venv/lib/python3.9/site-packages/tensorflow -l:libtensorflow_framework.so.2 .../horovod-tf-nightly-venv/lib/python3.9/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so
  -- Found Tensorflow: -L.../horovod-tf-nightly-venv/lib/python3.9/site-packages/tensorflow -l:libtensorflow_framework.so.2 .../horovod-tf-nightly-venv/lib/python3.9/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so (found suitable version "2.12.0-dev20221024", minimum required is "1.15.0")
# ...
$ python -c 'import horovod.tensorflow'

Error message quoted below under "relevant log output".
Relevant log output
2022-10-24 16:12:49.274846: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File ".../horovod-tf-nightly-venv/lib/python3.9/site-packages/horovod/tensorflow/__init__.py", line 27, in <module>
    from horovod.tensorflow import elastic
  File ".../horovod-tf-nightly-venv/lib/python3.9/site-packages/horovod/tensorflow/elastic.py", line 24, in <module>
    from horovod.tensorflow.functions import broadcast_object, broadcast_object_fn, broadcast_variables
  File ".../horovod-tf-nightly-venv/lib/python3.9/site-packages/horovod/tensorflow/functions.py", line 24, in <module>
    from horovod.tensorflow.mpi_ops import allgather, broadcast, broadcast_
  File ".../horovod-tf-nightly-venv/lib/python3.9/site-packages/horovod/tensorflow/mpi_ops.py", line 53, in <module>
    raise e
  File ".../horovod-tf-nightly-venv/lib/python3.9/site-packages/horovod/tensorflow/mpi_ops.py", line 50, in <module>
    MPI_LIB = _load_library('mpi_lib' + get_ext_suffix())
  File ".../horovod-tf-nightly-venv/lib/python3.9/site-packages/horovod/tensorflow/mpi_ops.py", line 45, in _load_library
    library = load_library.load_op_library(filename)
  File ".../horovod-tf-nightly-venv/lib/python3.9/site-packages/tensorflow/python/framework/load_library.py", line 54, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: .../horovod-tf-nightly-venv/lib/python3.9/site-packages/horovod/tensorflow/mpi_lib.cpython-39-x86_64-linux-gnu.so: undefined symbol: _ZNK3xla14HloComputation23CollectUnreachableRootsEv

Can you check if it was related to https://github.com/tensorflow/tensorflow/issues/58287 ?

Thank you for the pointer, @bhack!
So https://github.com/tensorflow/tensorflow/pull/55941 probably was merged shortly before tf-nightly-cpu==2.12.0.dev2022101. That one moves implementations from _pywrap_tensorflow_internal.so to libtensorflow_cc.so and that's quite likely to have caused issue. So presumably we can fix the error in Horovod by linking to that newer library.
For now the PR has been reverted in https://github.com/tensorflow/tensorflow/commit/970c3b44ea8b0db78d92ada624f03aeacf2e4518, though. So the acute problem might just go away for a while once wheels for tf-nightly-cpu==2.12.0.dev20221025 are available.
Anyway, I will look into linking to libtensorflow_cc.so as well.

Linking to libtensorflow_cc.so.2 works for me, https://github.com/horovod/horovod/pull/3755


get from image source failed

curl -sSLo argocd https://files.m.daocloud.io/github.com/argoproj/argo-cd/releases/download/v2.9.3/argocd-linux-amd64
cat argocd
# output is error
# {"error":"get from image source failed: E502"}%

it's okay tody
/close


Get es_opts faster in docker-entrypoint.sh

Hi
I use ES on k8s 1.11, and there are many Services in the same namespace.
When a container is started, environment variables related with Services are injected.
(https://kubernetes.io/docs/concepts/services-networking/service/#discovering-services)
Problem is, if there are many env vars, iteration for es_opts takes several minutes. It seems that read is especially slow. So, in this PR it filters environment variables before iteration.

Maintenance of the docker files for elasticsearch has moved to the elasticsearch repo. As we will be archiving this repository, I am going to close this PR. If you are still interested in making this contribution, please open a new pull request there.


parbox inside coffin produces persistent Parent/Child warnings

\DocumentMetadata{%
	pdfstandard=A-3b,
	pdfversion=1.7,
	lang=en-US,
	testphase={phase-III},
}
\documentclass{article}

\usepackage{xcoffins}
\NewCoffin{\AuthorBlock}

\begin{document}

\SetHorizontalCoffin\AuthorBlock{%
	\parbox{\textwidth}{John Steinbeck}% <=== parbox breaks tagging.
%	\mbox{John Steinbeck}%               <=== mbox does not.
}

To the red country and part of the gray country of Oklahoma, the last rains came gently and they did not cut the scarred earth.

The last rains lifted the corn quickly and scattered weed colonies and grass along the sides of the roads so that the gray country and the dark red country began to disappear under a green cover.
\end{document}

This code produces multiple warnings, which repeat with each paragraph added to the document.  (These warnings occur without typesetting the coffin.)
Package tagpdf Warning: Parent-Child 'StructTreeRoot/pdf' --> 'Div/pdf2'.
(tagpdf)                Relation is not allowed (struct 1, /StructTreeRoot -->
(tagpdf)                struct 5) on line 17

Package tagpdf Warning: Parent-Child 'StructTreeRoot/pdf' -->
(tagpdf)                'text-unit/user'.
(tagpdf)                Relation is not allowed (struct 5, /Div --> struct 6)
(tagpdf)                on line 17

Package tagpdf Warning: Parent-Child 'StructTreeRoot/pdf' --> 'text/user'.
(tagpdf)                Relation is not allowed (struct 6, /text-unit -->
(tagpdf)                struct 7) on line 17

Package tagpdf Warning: Parent-Child 'StructTreeRoot/pdf' --> 'text/user'.
(tagpdf)                Relation is not allowed (struct 1, /StructTreeRoot -->
(tagpdf)                struct 8) on line 17

Package tagpdf Warning: Parent-Child 'P/' --> 'text-unit/user'.
(tagpdf)                Relation is not allowed (struct 8, /text --> struct 9)
(tagpdf)                on line 19

Package tagpdf Warning: Parent-Child 'P/' --> 'text/user'.
(tagpdf)                Relation is not allowed (struct 9, /text-unit -->
(tagpdf)                struct 10) on line 19

Package tagpdf Warning: Parent-Child 'P/' --> 'text-unit/user'.
(tagpdf)                Relation is not allowed (struct 8, /text --> struct
(tagpdf)                11) on line 21

Package tagpdf Warning: Parent-Child 'P/' --> 'text/user'.
(tagpdf)                Relation is not allowed (struct 11, /text-unit -->
(tagpdf)                struct 12) on line 21

A coffin in vertical mode is a better idea... no issues when done properly.
\SetVerticalCoffin\AuthorBlock{\textwidth}{%
    \parbox{\textwidth}{%
    John Steinbeck: To the red country and part of the gray country of Oklahoma, the last rains came gently and they did not cut the scarred earth. The last rains lifted the corn quickly and scattered weed colonies and grass along the sides of the roads so that the gray country and the dark red country began to disappear under a green cover.
	}
}
\TypesetCoffin\AuthorBlock
}

It still seems to be a bug, because one should be allowed to put a parbox into a horizontal coffin/box

Many thanks for the Steinbeck quote, John!
Reminding me of when, more than 50 years ago, I read a lot of his books.

The parbox and minipage tagging code assume that they are inside a paragraph (as they issue a \leavevmode) and so they try to close the P-structure. This fails if they are in a \hbox (and so horizontal coffin) which itself is in vmode as that doesn't trigger the para hooks.
It is easy to avoid the wrong tagging but I have no idea if one could detect that automatically.
\DocumentMetadata{%
	pdfstandard=A-3b,
	pdfversion=1.7,
	lang=en-US,
	testphase={phase-III},
}
\documentclass{article}

\begin{document}

\hbox{\tagpdfparaOff\parbox{\textwidth}{John Steinbeck}}

xxx\hbox{\parbox{\textwidth}{John Steinbeck}}

% wrong:
%\hbox{\parbox{\textwidth}{John Steinbeck}}

% wrong:
%xxx\hbox{\tagpdfparaOff\parbox{\textwidth}{John Steinbeck}}
\end{document}

Side remark: Even if is easy to avoid the warning, moving of boxes with structure is non trivial (and there are difference between lualatex and pdflatex) and must be done with care.

In the code I'm actually working with, it's easy to replace the horizontal coffin + parbox construction with a vertical coffin (and I have done so). That avoids the paragraph tagging error.
I gather than a general fix for "parbox in hbox" is going to be tricky.

This looks like an unfortunate bug! But maybe it is necessary? 😀
"The parbox and minipage tagging code assume that they are inside a paragraph"
Also some questions:
How are these two (and similar boxes) then tagged?
How is \mbox (and its primitives) tagged?

This fails if they are in a \hbox (and so horizontal coffin) which itself is in vmode as that doesn't trigger the para hooks.

I would think it isn't hard to distinguish the two cases, it is basically checking for restrictied or unrestrict hmode and then the tagging code code could alter its behavior. Or am I missing something?

@FrankMittelbach if it were for a direct use \hbox alone one could probably \currentgrouptype it seems to give different values (but the documentation is a bit scarce about when you get 2 and 3).  But the main problem is the storing of the content in boxes.  I haven't found anything yet that would allow me to handle the tagging of the parbox correctly in all cases if it is stored first in a hbox, even if the box is used directly and only once: \setbox0\hbox{\parbox{\textwidth}{John Steinbeck}}\box0.

3 is adjusted_hbox so more or less this, I think
@d adjusted_hbox_group=3 {code for `\.{\\hbox}\grp' in vertical mode}

testing currentgrouptype has dangers though tabu made great use of it to test what kind of table cell was current but then when \begingroup added to ensure color scopes all that information was lost (and somewhat hard to recover)

@davidcarlisle well yes. But the "in vertical mode" doesn't work for setbox:
\documentclass{article}
\begin{document}

\setbox0\hbox{\the\currentgrouptype}\box0

in para \setbox0\hbox{\the\currentgrouptype}\box0

\hbox{\the\currentgrouptype}

in para \hbox{\the\currentgrouptype}

{\hbox{\the\currentgrouptype}}

in para {\hbox{\the\currentgrouptype}}
\end{document}

well it works it's just not useful which was really all I meant to say, and if you use \sbox0  instead of \setbox0\hbox of course you get 14 not 2 or 3 which means I think that currentgroup is not really a useful test except in very closely controlled code (where probably you know what kind of group you are in without testing it....)


TASK: delete obsolete code in sapcc/juno

Identify and remove outdated or unused code in the sapcc/juno repository to streamline the codebase. This will help improve maintainability and reduce technical debt.

[x] delete deprectated_* folders
[x] all other packages should be placed in packages, apss in apps

done with #664


Unify receiver and pipeline IDs in 3rd party app sample configs.

Background
It's preferred for the receiver and pipeline IDs to stay the same as the receiver type in the sample configuration, enable_ files and the main confgenerator/testdata/valid/{LINUX_OR_WINDOWS}/{logging_or_metrics}-receiver_{APP}/input.yaml test case, because some users look at the valid cases for sample configurations as well. Additional more complex test cases could use more complex receiver or pipeline ID.
The reason behind that is that Log names are derived from logging receiver IDs, and the dropdown in UI + sample query / dashboard relies on the log name. So the sample dashboards and queries can use that. If users customize it, they can then adjust dashboards and queries accordingly. For consistency, the same convention applies to the sample configurations on the metrics side.
Included changes

s/{APP}_metrics/{APP}, s/{APP}_pipeline/{APP}, s/{APP}_default/{APP} in the receiver / pipeline IDs in the following files:

Sample configuration in markdowns
enable_<APP> files
The main confgenerator/testdata/valid/{LINUX_OR_WINDOWS}/{logging_or_metrics}-receiver_{APP}/input.yaml test case

In the confgenerator/testdata/valid/{LINUX_OR_WINDOWS} folder, make sure that the {logging_or_metrics}-receiver_{APP}/input.yaml test case is the minimum configuration of the application. Split the test case into a separate {logging_or_metrics}-receiver_{APP}_custom/input.yaml when necessary to test customized configuration
Make sure the enable_<APP> files are testing the minimum configuration as well. e.g. Remove unnecessary collection_interval and endpoint etc. optional config parameter settings.

To review locally
The golden file changes are not important for purpose of this PR, thus can be excluded to reduce the noise.
git clone --recurse-submodules git@github.com:GoogleCloudPlatform/ops-agent.git
cd ops-agent
git checkout lingshi-id
git diff origin/master..HEAD -- . ':(exclude)*golden*'

Tests look okay:

Debian seems to be failing for an unrelated reason: tomcat install on debian 10 fails. It is being addressed separately by https://github.com/GoogleCloudPlatform/ops-agent/pull/344.
Gonna rebase then merge.


initial commit, will add tests in next commit

Hi @SatishThareja, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Satish Thareja). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


gen-pkcs12-keystore init container fails if the tls secret contains no ca.crt

Configure the solrTLS with a TLS secret that contains only a tls.crt and a tls.key
It seems to be dependent on the issuer if there is a separate ca.crt or not e.g. see https://github.com/cert-manager/cert-manager/issues/1571
    solrTLS:
      keyStorePasswordSecret:
        name: security
        key: password-key
      pkcs12Secret:
        name: solrcloud-tls
        key: keystore.p12

The generated init container command
openssl pkcs12 -export -in /var/solr/tls/tls.crt -in /var/solr/tls/ca.crt -inkey /var/solr/tls/tls.key -out /var/solr/tls/pkcs12/keystore.p12 -passout pass:${SOLR_SSL_KEY_STORE_PASSWORD}
fails with
Can't open /var/solr/tls/ca.crt for reading, No such file or directory<PHONE_NUMBER>72192:error:02001002:system library:fopen:No such file or directory:../crypto/bio/bss_file.c:69:fopen('/var/solr/tls/ca.crt','r')<PHONE_NUMBER>72192:error:2006D080:BIO routines:BIO_new_file:no such file:../crypto/bio/bss_file.c:76:
However if the keystore is generated with the tls.crt and tls.key as input it works. (The tls.crt contains the full chain of certificates)

I will provide a pull request to include the -in ca.crt only if there is a ca.crt file present:
Tested:
    Command:
      sh
      -c
      OPTIONAL_CACRT=$(test -e /var/solr/tls/ca.crt && echo ' -in /var/solr/tls/ca.crt'); openssl pkcs12 -export -in /var/solr/tls/tls.crt $OPTIONAL_CACRT -inkey /var/solr/tls/tls.key -out /var/solr/tls/pkcs12/keystore.p12 -passout pass:${SOLR_SSL_KEY_STORE_PASSWORD}


Getting the submission results on the extension itself

We have tried our best but need some help with implementing this functionality.
So this will go like when you submit, the redirect from codeforces stops and the submission result will show live on the extension itself, just like it does in leetcode.
If someone is interested in taking this part, do let me know before hand right here.

The only thing that is problematic here is the redirection from codeforces, rest can be done easily according to me.

Hey, we can handle that actually, if you think u can get this done, please let me know, we will handle the stopping of redirection part

Hey, we can handle that actually, if you think u can get this done, please let me know, we will handle the stopping of redirection part

From the submission result? you mean the AC, WRONG_ANSWER, TLE, Compile Error right?
Well you know you can use, the following API end point, and set from=1&count=1 to get the recent submission
https://codeforces.com/api/user.status?handle=iamrlm&from=1&count=10
I have implemented a similar logic
https://github.com/Dev-Code24/My-Codeforces-Journal-Development/blob/6d12996644f6390d9034077824bcb4da2a6d05a4/src/services/add-problem.ts#L79

Omg dude that's amazing, I was searching for this for so long. Wanna collaborate here with us to make this happen? We also are working on another feature for running and checking pretests so yeah we can definitely get this done

ok, we can collaborate : )

ok, we should collaborate : )

Hey

ok, we should collaborate : )

Hey, we have started working upon the code running part and can pick the submit after that's done, do u want to pick the other 2 issues?

Hey, we have started working upon the code running part and can pick the submit after that's done, do u want to pick the other 2 issues?

have you completed the issue ? I just did not see the notification of your comment

Hey, we have started working upon the code running part and can pick the submit after that's done, do u want to pick the other 2 issues?

have you completed the issue ? I just did not see the notification of your comment

Hey, the version 1.1.2 will be out soon having the code running feature implemented, although it's there on the GitHub as of now in the main branch. If you want to get the feature done, please start with and and let me know when it's done.
If it integrates well, we can get it merged here


Tiles in high DPI are way too big

We continued our usage of MapFish Print with high DPI and I think there is a bug in how the size of the image is computed: in A3 300 DPI, the tiles are over 16000x16000 pixels. However, if we rely on image sizes from http://www.din-formate.de/reihe-a-din-groessen-mm-pixel-dpi.html or on GIMP, the images should be less than 5000x5000 pixels wide for A3 and 300 DPI. Furthermore, if I use image sizes provided on the previous link the representation of the layers is correct.
I think that part of the problem comes from the fact that height and width are too big even in 72DPI (the default). In a test, a A4, I get WIDTH=1093 and HEIGHT=1510 whereas I should get at most WIDTH=595 and HEIGHT=842. Which means that in high DPI since we multiply the sizes, the error gets amplified beyond what we can handle.
I tried to look into it today to find out where imageSize used in the tile size comes from but I wasn't able to find the piece of code that can cause this error.

This looks  solved on master. Closing.


Fix CI's podman apt-get repo url

This PR fixes the CI issue related to the podman package installation step.
Context
The CI test job has consistently failed with a 404 error in the Podman setup step.
[...]
Ign:35 https://download.opensuse.org/repositories/devel:kubic:libcontainers:unstable/xUbuntu_22.04  InRelease
Err:36 https://download.opensuse.org/repositories/devel:kubic:libcontainers:unstable/xUbuntu_22.04  Release
  Redirection from https to 'http://download.opensuse.org/repositories/devel:kubic:libcontainers:/unstable/xUbuntu_22.04/Release' is forbidden [IP: <IP_ADDRESS> 443]
Reading package lists...
E: The repository 'https://download.opensuse.org/repositories/devel:kubic:libcontainers:unstable/xUbuntu_22.04  Release' does not have a Release file.
Error: Process completed with exit code 100.

Issuing a GET request to that endpoint reveals that the path has changed (and the podman documentation has not been updated accordingly).
curl -fsSL -kv https://download.opensuse.org/repositories/devel:kubic:libcontainers:unstable/xUbuntu_22.04/Release

Multiple redirections (301 Moved permanently) happen before the final URL is queried:
[...]
* Ignoring the response-body
* Connection #1 to host download.opensuse.org left intact
* Issue another request to this URL: 'http://download.opensuse.org/repositories/devel:/kubic:/libcontainers:/unstable/xUbuntu_22.04/Release'
* Found bundle for host: 0x5589f0eef300 [serially]
* Can not multiplex, even if we wanted to
* Re-using existing connection with host download.opensuse.org
> GET /repositories/devel:/kubic:/libcontainers:/unstable/xUbuntu_22.04/Release HTTP/1.1
> Host: download.opensuse.org
> User-Agent: curl/8.4.0
> Accept: */*
>
< HTTP/1.1 200 OK
< date: Wed, 15 Nov 2023 15:49:46 GMT
< server: Mojolicious (Perl)
< cache-control: public, max-age=101
< content-disposition: inline;filename="Release"
< content-length: 732
< content-type: application/x-download;name="Release"
<
[...]

So, the new URL should contain / after the ::
http://download.opensuse.org/repositories/devel:/kubic:/libcontainers:/unstable/xUbuntu_22.04/Release

I can confirm the issue has been solved with this fix: https://github.com/LNSD/testcontainers-node/pull/3

Merged, since the failing tests were unrelated to this change, thanks again for looking into this @LNSD 👍

Merged, since the failing tests were unrelated to this change, thanks again for looking into this @LNSD 👍

Nice! Happy to help 💪🏼 😁


Add TypeScript support

Hey 👋,
Thank you for building this fantastic project! I saw you implemented it using flow which would be, at least for me, one of the major selling points to switch from ink. Do you think you could also add support for typescript?
Let me know if I can help in any way, I would love to rewrite emma-cli in react-cli 🎉

Hi @maticzav, thanks for the interest and opening this issue!
Would you mind clarifying what you mean by supporting typescript? Even though this library is written using flow, its still compiled to native js (code link), so I think it should work just fine with any projects using typescript? I haven't actually tried a workflow like that myself so let me know if you're running into any specific issues - definitely want to make sure this library works no matter what platforms people are using!

Definitely, I was thinking more in terms of getting type safety which flow features. That's why I was wondering about TypeScript support. I have no doubt that react-cli would work if I used the compiled code but I can already do that in ink and get a dozen premade components on top of it. Did that clarify the idea? 🙂

Ahh yes got it, thanks! 😄 I've never published typescript definitions for a module before but I'm assuming that should definitely be doable. Now that you mention it I believe I need to do some work for flow as well to make sure those types are available when using this with flow-typed projects. I'll leave this issue open for typescript and open a new one for flow. If you have any suggestions or want to help with a PR for either, feel free!

Typescript instructions: https://github.com/DefinitelyTyped/DefinitelyTyped#create-a-new-package


See Through Leaf

When you enable "remove caves" it also removes modded leaf's grounds which is you can see the WHOLE world through, i remember you said something like without MaterialX you cannot tell either its a leaf or not so that algorithm also deletes under the modded leaves cuz it doesnt know its leaf

MiEx by default doesn't know how to handle blocks occluding other blocks. The block state and block model files don't really say anything about this. That's all hardcoded in Minecraft. In order for MiEx to know how to handle various kinds of blocks, you need to tell it to. This is done in the miex_config.json file. It has a few lists for different kinds of occluding, like transparent blocks and leaves blocks.
The default miex_config.json file has the vanilla leaves blocks added to the leavesOcclusion list, but not any of the modded leave blocks. So, these modded blocks need to be added to the right lists so that MiEx knows how to properly handle them. You can read more about the miex_config.json file and the various lists here: https://github.com/BramStoutProductions/MiEx/wiki/3.-Pipeline-Integration-and-Customisations#configurations

The example miex_config.json file that I put in this post, shows some of the leaves blocks added to the leavesOcclusion list: https://github.com/BramStoutProductions/MiEx/issues/35#issuecomment-2081631048


Migrate framework devicelab tests to iOS 14 SDK and Xcode 12

[ ] https://github.com/flutter/flutter/issues/70414 - integration_ui_ios_screenshot
[x] https://github.com/flutter/flutter/issues/70416 - external_ui_integration_test_ios
[x] https://github.com/flutter/flutter/issues/70422 - flutter_view_ios__start_up
[x] https://github.com/flutter/flutter/issues/70869 - tiles_scroll_perf_ios__timeline_summary
[x] https://github.com/flutter/flutter/issues/71232 - Pods-Runner does not support provisioning profiles

3 remaining migration blockers:

error: unable to resolve product type 'com.apple.product-type.application.watchapp2' for platform 'macosx' (in target 'watch' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to find sdk 'watchos' (in target 'watch' from project 'Runner')

https://logs.chromium.org/logs/flutter/led/godofredoc_google.com/5aaab33f10eae0d7e0be177c15763c1f22a0f80a6fd382ee3f8e1ff4e9431ee3/+/steps/run_ios_app_with_extensions_test/0/stdout

[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: Installing and launching...                                        39.4s
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stderr: Application failed to start on attempt: 1
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: Installing and launching...                                        36.6s
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stderr: Application failed to start on attempt: 2
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stdout: Installing and launching...                                        36.9s
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stderr: Application failed to start on attempt: 3
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] stderr: Application failed to start. Will not run test. Quitting.
[flutter_gallery__transition_perf_e2e_ios] [STDOUT] "/opt/s/w/ir/k/flutter/bin/flutter" exit code: 1
[flutter_gallery__transition_perf_e2e_ios] [STDERR] Task failed: Executable "/opt/s/w/ir/k/flutter/bin/flutter" failed with exit code 1.

https://logs.chromium.org/logs/flutter/led/godofredoc_google.com/3168dfdb0b4ef7b09880118b9c8eafe92a1861dc291386a9fc2724ebbbd2ab78/+/steps/run_flutter_gallery__transition_perf_e2e_ios/0/stdout

[smoke_catalina_start_up_ios] [STDOUT] stderr: [        ] Error running application on iPhone.
[smoke_catalina_start_up_ios] [STDOUT] stdout: [   +4 ms] "flutter run" took 53,520ms.
[smoke_catalina_start_up_ios] [STDOUT] stderr: [   +4 ms]
[smoke_catalina_start_up_ios] [STDOUT] stderr:            #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
[smoke_catalina_start_up_ios] [STDOUT] stderr:            #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:617:7)
[smoke_catalina_start_up_ios] [STDOUT] stderr:            <asynchronous suspension>
[smoke_catalina_start_up_ios] [STDOUT] stderr:            #2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1080:18)
[smoke_catalina_start_up_ios] [STDOUT] stderr:            <asynchronous suspension>
[smoke_catalina_start_up_ios] [STDOUT] stderr:            #3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:931:33)
[smoke_catalina_start_up_ios] [STDOUT] stderr:            <asynchronous suspension>

https://logs.chromium.org/logs/flutter/led/godofredoc_google.com/131f3dde81ce0d85410c00987bdcacd9c83a3aadb2e2e893f8b8076abc02e6b5/+/steps/run_smoke_catalina_start_up_ios/0/stdout

Note for reference:
ios-deploy[1929:21390] [ !! ] Unable to locate DeviceSupport directory with suffix 'DeveloperDiskImage.dmg'. This probably means you don't have Xcode installed, you will need to launch the app manually and logging output will not be shown!

Means forgetting to xcode-select a newer xcode to be able to deploy to iOS 14.

Cleared up a little bit. More failure types are being revealed.

The devicelab is on Xcode 12.3.


please share some docs to versioning of schema

Description
its particularly not an issue, but i want to work for the schema versioning int he schema.
i am using 1.0beta version of  dotnet library. most of the doc and examples are not from that.
doing the POCs for my organization get more challenging.
so cna you please point me to some docs and examples for Schema registry versioning etc.
which will be the latest? AvroSerdeProvider or AvroSerializer<> ??
AvroSerializer is for primitive types?
How to reproduce
Checklist
Please provide the following information:

[ ] Confluent.Kafka nuget version:
[ ] Apache Kafka version:
[ ] Client configuration:
[ ] Operating system:
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

the API related to this is changing again in -beta3 (or the next release may even be -RC) unfortunately. stay tuned for that. but i hear you, we need better documentation.


NSM init monitoring container

Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[X] Added integration testing to cover
[X] Tested locally
[ ] Have not tested

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 not work as expected)

Checklist:

[X] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

TODO: add documentation

Do we intend to add this to the admission controller?

@edwarnicke

Do we intend to add this to the admission controller?
Yes, we do. I'll use this with admission webhook in DNS PR.
In this PR it is not needed


Get data on all housing units in DC

We want to contrast the subsidized affordable housing we have with what is available in market rate for the rest of a neighborhood/ward/area. We then can see what proportion of housing is subsidized - and in some areas this may be significant, and for example can influence things like American Community Survey stats that include these residents.
Ideally we will have total counts of residential units with their total bedroom count.
Potential city-maintained data sources that we can cross reference.

Master Address Repository
PIVS
CAMA
Property tax database

This should be done in conjuction with #149 , which is a proof of concept to link up the Master Address Repository.

Recommend using http://opendata.dc.gov/datasets/integrated-tax-system-public-extract as the main property tax database file.

@ptatian is there a reason you recommend the public-extract over the public-extract-facts table I first linked to? It looked like adding the description fields in the facts version was useful, though I also see there's a slight difference in the number of records which I don't understand.

@NealHumphrey No, no reason. My understanding is that they should be identical (so odd about the record count) other than the replacement of codes with descriptions. So, fine to use either version.


docs: update flagging logic for IFE

:clipboard: https://github.com/omisego/plasma-contracts/issues/470
Overview
There was a bug found for cross-competin IFE with one end up canonical and one not. As a result, the logic of flagging output is changed while fixing that.
Also, trying to update for this comment: https://github.com/omisego/elixir-omg/pull/1118#issuecomment-557132890

Coverage remained the same at ?% when pulling d08c4e4146b3f3f541b24f7cb20fd9811c61762a on boolafish/1108_update_se_ife_interaction into d9103e164d308aa5ebff568647830995a08cc767 on master.

ah...didn't notice now it needs 2 approval. add @paulperegud


Cannot launch browser chrome: Could not calibrate. (MacOs)

Following the basic example in the readme does not work for me because of chrom calibration failing.  If I disable calibration it seems to work as expected.
Output of gemini --version:
4.18.1
Contents of .gemini.js file:
module.exports = {
    rootUrl: 'http://yandex.ru',
    gridUrl: 'http://<IP_ADDRESS>:4444/wd/hub',

    browsers: {
        chrome: {
            desiredCapabilities: {
                browserName: 'chrome'
            }
        }
    }
};

Test source code:
gemini.suite('yandex-search', (suite) => {
    suite.setUrl('/')
        .setCaptureElements('.home-logo')
        .capture('plain');
});

Command used to run the test:
gemini test --reporter flat --reporter html

Expected behaviour:
A chrome window should open, load the yandex.ru page, and generate a test report
Actual behavior:
A chrome window opens with a solid green background, waits for some time, and then closes.  The command line shows the message:
annot launch browser chrome:
Could not calibrate. This could be due to calibration page has failed to open properly.
Total: 1 Passed: 0 Failed: 1 Skipped: 0 Retries: 0
Your HTML report is here: file:///Users/bjorn/test/gemini-report/index.html

simply adding calibrate: false, to the gemini.js file makes it work.  Here is the updated contents:
module.exports = {
    rootUrl: 'http://yandex.ru',
    gridUrl: 'http://<IP_ADDRESS>:4444/wd/hub',
    calibrate: false,

    browsers: {
        chrome: {
            desiredCapabilities: {
                browserName: 'chrome'
            }
        }
    }
};

Hello!
In my case everything works fine without calibrate: false.
Which version of chrome and selenium-standalone do you use?

chrome is Version 56.0.2924.87 (64-bit)
selenium is from https://www.npmjs.com/package/selenium-standalone, which
looks to have installed Selenium build info: version: '3.0.1', revision:
'1969d75'
On Fri, Mar 24, 2017 at 8:38 AM, Rostislav Shtanko<EMAIL_ADDRESS>
wrote:

Hello!
In my case everything works fine without calibrate: false.
Which version of chrome and selenium-standalone do you use?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gemini-testing/gemini/issues/753#issuecomment-289058295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABpiGAtCY_wxjqsq2T99ZMJ2dDbLcOJLks5ro-OEgaJpZM4Ml8n5
.

I tried to reproduce it on latest gemini and selenium-standalone versions (macOS) - all works fine for me

I have the same issue. Can it be OS thing? I switched to Sierra and it's not working. Last time I've been on my old Mac, it used to launch normally.

Ah, my issue it a little bit different, but I suppose related
✘ 2.1 plain [chrome-latest]
Cannot launch browser chrome-latest:
undefined while parsing file: /Users/varya/WebDev/SC5/sc5-styleguide-visualtest/node_modules/gemini/lib/browser/client-scripts/gemini.js.

I have the same issue. Can it be OS thing?

I will try it on latest version. But it works on Sierra 10.12.3

Ah, my issue it a little bit different, but I suppose related

it can be because of switching node version. You can try to reinstall gemini

I'm getting this error intermittently on build machines... @sipayRT any update? @varya @bjornicus any workaround you came up with?

Same issue here... I'm making it as written in https://github.com/gemini-testing/gemini, selenium-standalone: Selenium build info: version: '3.6.0', revision: '6fbf3ec767'. Chrome launches but there is a green screen than it fails with: Cannot launch browser chrome:
Could not calibrate. This could be due to calibration page has failed to open properly.

@campside try to check your page on client errors - while calibration gemini should execute js-scripts on the page. So if there will be some client errors calibration will fail

The same issue. Even adding calibrate: false to the config doesn't help:

with option chrome is open with solid green, closes after a few seconds and reports an error
w/o option – just reports an error

gemini: 5.0.0-alpha.10
Mac OS 10.12.6
node v9.2.0

can you provide your error log without calibration?
Also it will be helpful to get your page which you tried to open.

👀

In terminal, it is like that
  𝒟ℴ gulp test:visual
[13:23:44] Failed to load external module @babel/register
[13:23:44] Requiring external module babel-register
[13:23:49] Using gulpfile ~/WebDev/SC5/Elisa/pattern-library/gulpfile.babel.js
(node:43088) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
[13:23:49] Starting 'dev:options'...
[13:23:49] Finished 'dev:options' after 245 μs
[13:23:49] Starting 'test:visual'...
Test server:  selenium
Test resolution:  1250x730
Test server: SELENIUM
Selenium started
✘ <IP_ADDRESS> plain [chrome-latest]
Cannot launch browser chrome-latest:
Could not calibrate. This could be due to calibration page has failed to open properly.

can you share your test page?
so I can check your problem locally

I will try to develop the test page later today.

I had the issue on example from the readme, nothing extra.
Try to install gemini and its deps on a clean macOS and run a basic test from readme. I suppose the problem is about using https, but not sure.
I switched off calibration and it doesn't help. Later I opened yandex.ru in Chrome (since this host is used in example test), and some time later I tried to run the test again and it worked as expected. However calibration still doesn't work. I suppose something additional should be done to make genimi works.

I am facing this issue as well.
MacOS: 10.13.4
package.json
    "chromedriver": "^2.36.0",
    "gemini": "^5.6.2",
    "selenium-standalone": "^6.14.0"

.gemini.js
const puppeteer = require("puppeteer");

module.exports = {
  rootUrl: "http://localhost:9000",
  gridUrl: "http://<IP_ADDRESS>:5678/wd/hub",
  calibrate: true,
  windowSize: "1920x1080",
  screenshotsDir: "./tests/screenshots",
  browsers: {
    chrome: {
      desiredCapabilities: {
        browserName: "chrome",
        chromeOptions: {
          args: ["--no-sandbox", "--headless"],
          binary: puppeteer.executablePath()
        }
      }
    }
  },
  system: {
    debug: true,
    plugins: {
      "html-reporter/gemini": {
        enabled: true,
        path: "gemini-reports/",
        defaultView: "all"
      }
    }
  }
};

I found more info that might help.
Gemini is not working fine in following cases:

Calibration fails when using chrome on --headless mode.
In mac. I guess this is due to the high-resolution display. I just connected an external monitor(Full HD) to my mac and tested. All working fine.

Investigation details:
I compared the resolution of the image taken in my mac(without calibration and calibration doesn't work in mac display in my case).

Mac Screenshot without calibration: 3808 × 96
Screenshot using external monitor: 1904 × 48

Right now solved my issue by just using an external monitor and enabling calibrate: true so that everything works fine in travis

I'm seeing this happen in Windows 10 Pro x64 as well, using today's gemini, and selenium-standalone 6.15.4 with the example files as indicated over on https://github.com/gemini-testing/gemini
Chrome gets launched in "used for testing" mode, but it ends up showing a green screen, with the URL bar showing that it's trying to load data:, followed by about:blank, instead of loading the url indicated in the .gemini.js configuration.


Resolve syntax error from Localizable.strings file

@aleffert  @rahul-clarice  Please review.

Sorry about that.