Unable to build with VS2019

First of all thnx for code sharing!
Tried to build this library on windows but faced some issues.
template
using selected_charconv = typename std::conditional_t<std::experimental::is_detected_v<from_chars_function, T>, std_charconv, my_charconv>;
this code from text.hpp is not valid, because VS2019 (even 2022) experimental has no is_detected_v<...> defined.
#include <experimental/type_traits> - there is no such a file in VS SDK, And is_detected_v should be there.
Also there is some problems to config project with cmake. Cmake cant find gxrio - even if im putting path to it in cmake-gui. I hacked cmake to bypass it (putting hardcoded path to cmake var) and generate sln.
Thnx in advance!

The trunk now contains code I managed to compile with MSVC 2019. I will make a release for it as soon as I've updated and tested all depending software as well.


Calling UseDefaultSerialization before UseNodaTime break nodatime serialization.

Hi,
I have found issue when using Marten.NodaTime while at the same time calling UseDefaultSerialization for parsing enum as string.
Steps to reproduce
Create configuration and call UseNodaTime before UseDefaultSerialization.
public class MartenConfiguration : StoreOptions
{
    private MartenConfiguration(string connectionString)
    {
        this.UseNodaTime();
        this.UseDefaultSerialization(enumStorage: EnumStorage.AsString);
    }
}

Effect
Parsing LocalDate (or any kind of NodaTime object) fails.
Problem
I have found in repo, that UseDefaultSerialization creates new serializator, so effect of calling UseNodaTime before makes configuration invalid.
Solution
I see two solutions.

A the very least - write it in documentation of Marten.NodaTime, that calling UseNodaTime should come before creating new serializator.
Make change to UseDefaultSerialization so it doesn't create new serializator, but update existing one like it's done in  UseNodaTime extension method.

Adding docs is a low hanging fruit, will add it.


Org files can easily be converted into Markdown

Hi Sir,
The README on the GitHub web site has some cosmetic issues (and copy-pasting issues) which are easy to fix; you can simply use (org-md-export-to-markdown) to generate README.md using README as source code.
Admittedly, you will run into the issue of GitHub selecting the first README in the git tree where "first" is determined by asciibetical order.  Renaming README to README.org will allow README.md to be displayed in the GitHub web page.
I have an example of a generated file here:
https://github.com/mettacrawler/geiser/blob/master/README.md
I do not know if it would make sense to do a pull request for this.
Thank you,
MC

Specifically, if you try to copy-paste this line from the GitHub web site README (the org README)
(setq geiser-active-implementations ‘(chicken guile))
The quote mark is a Unicode character.
The converted version is ASCII.
(setq geiser-active-implementations '(chicken guile))

hmm. i'd prefer not to include a generated file in the repo, specially
when github also knows how to display org files.  can't be just fix the
cosmetic issues in the org file without generating markdown?

I think that https://github.com/wallyqs/org-ruby isthe conversion software that GitHub uses.
The README.md in https://github.com/github/markup points to that Ruby code converter.
The README.org in the org-ruby git tree is full of markdown codes like
#+BEGIN_SRC sh

How do you feel about someone putting such markdown in the README.org of this project?

On Mon, Apr 02 2018, mettacrawler wrote:

#+BEGIN_SRC sh
How do you feel about someone putting such markdown in the README.org
of this project?

yes, that's definitely needed... i did some of it just a few minutes
ago.  when it comes to the keystrokes tables, either the org parser has
some glitches or i am missing something, because it inserts boldface in
a couple places for apparently no good reason.
do you see any other pending issues with the org markup after my
changes?

Some of the table entries are centered.  Not sure why yet but this is an interesting example:
https://github.com/jaor/geiser/compare/master...mettacrawler:patch-1

it seems the parser is taking those lines for headers... i've introduced
separators as you suggested and the centering/boldface is gone

Looks OK to me now.


Bundle breaks existing JQuery -

Which package(s) are affected?
Lit Core (lit / lit-html / lit-element / reactive-element)
Description
When I call the bundle of my custom element, the existing JQuery is not reachable anymore.
<script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/3.6.0/jquery.min.js"></script>
<script>console.log('it WORKS BEFORE the lit web component',$("html").attr('lang'));</script>
<!-- <script src="/build/itm-common-components.rollup.min.js"></script> -->
<script src="/dist/itm-common-components.vite.min.js"></script>
<script>console.log('This BREAKS AFTER the lit web component',$("html").attr('lang'));</script>

The bundle was done with Vite and with Rollup. Both failed
The bundle must include lit
If the bundle doesn't include lit (I add to vite/rollup config.js  rollupOptions: {external: /^lit/}, it works. But I must include it.
I get the same result with cjs or es

Reproduction
Look at the console (I pasted the bundled version to reproduce) Repro
Workaround
No workarounds
Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
<EMAIL_ADDRESS>Browser/OS/Node environment
All Browsers/Windows/Node 14

Thanks for the issue. In the given repro the script tag loading the Lit bundle isn't a JavaScript module. Changing <script src="./repro.js"></script> to <script src="./repro.js" type="module"></script> fixes the issue.
This happened in the repro because a standard script tag will pollute the global scope and there is a minified $ variable within the Lit bundle. My proposed solution changes that import to use JavaScript modules which automatically scope their contents unless explicitly exported or assigned to a global (such as window).
See this minimal example: https://lit.dev/playground/#gist=64046c92f4d3995b843f67b3e926b392
Another alternative is to explicitly reference jQuery from the window object. E.g. window.$("html").attr('lang'); fixes your issue.

Note that there are possibly two separate $ in play here. lit-html.js currently (this can change with each build) includes a const variable named $ that's currently set to our rawTextElement regex. But in the reproduction $ appears to be set to another variable, our marker text. I'm not sure if that's from a different build of lit-html.js, or from a subsequent minification pass by the user, but it looks like Terser at least will happily use $ and other symbols for minified names.
We could reserve $ so that it's not used, but other names could clash easily. Besides what @AndrewJakubowicz said about type=module, if you're bundling you should be able to output in IIFE format which will add a scope around the modules and prevent this kind of global pollution in general.

Thanks, @AndrewJakubowicz, and @justinfagnani. I have little control over the invoking script tag. Although I checked that type="module" works well, I think using an IIFE will provide more independence to the component.
Not using Terser, btw.
Cheers


Help DNS issues - and Non profit Billing issue!

Hi - Flywheel let us know 1) we have DNS being directed to you - can you let us know if we need to keep this dns on Godaddy?  2) Do we need to update the DNS? 3) We goto spam all the time - could this be an issue?  4) We are a 501c3 non profit - our billing statement does not reflect this? Can you check?  Thankyou see below for details from flywheel
FROM FLYWHEEL for you to answer

Sure, looking at the IP address associated with the new2016 A record, that is pointing to GoDaddy: https://who.is/whois-ip/ip-address/<IP_ADDRESS> I'd recommend checking with them before removing the record
Yep, the other strange A record that is associated with the IP address <IP_ADDRESS> is actually pointing to Sendgrid
https://who.is/whois-ip/ip-address/<IP_ADDRESS>

It is certainly a busy time! That's right, checking those strange A records (one with GoDaddy and one with Sendgrid),
Thank you!
Actual content
Link to page:

Hello @8241973  ,
We're sorry you're having this issue! Since this is not a docs-related issue, please contact our Support team directly and they can help you out. You can access support contact options by logging into https://support.sendgrid.com.
Twilio SendGrid Docs Team


X-Anylabeling

Hello, after fine-tuning the model, I used SameExporter to convert it to ONNX model, but it cannot be used in X-Anylabeling. Have you ever used it in the above tool?
https://github.com/vietanhdev/samexporter
https://github.com/CVHub520/X-AnyLabeling

What's the error message?


docs/windows: update instructions

I'm a newer of syzkaller and tried to build the executor of syzkaller  for windows.
The docs for Windows part seems too old to follow and build the executor for windows. Specifically, the executor\executor_windows.cc has not been existed in the source files.
make fuzzer execprog stress TARGETOS=windows
REV=git rev-parse HEAD
cl executor\executor_windows.cc /EHsc -o bin\windows_amd64\syz-executor.exe \
	-DGIT_REVISION=\"$REV\" \
	kernel32.lib user32.lib gdi32.lib winspool.lib comdlg32.lib advapi32.lib \
	shell32.lib ole32.lib oleaut32.lib uuid.lib odbc32.lib odbccp32.lib \
	winmm.lib rpcrt4.lib Crypt32.lib imm32.lib Urlmon.lib Oleaut32.lib \
	Winscard.lib Opengl32.lib Mpr.lib Ws2_32.lib Bcrypt.lib Ncrypt.lib \
	Synchronization.lib Shell32.lib Rpcns4.lib Mswsock.lib  Mincore.lib \
	Msimg32.lib RpcRT4.lib Rpcrt4.lib lz32.lib

I don't think anybody from core developers has Windows access now.
Re source files check out how other OSes are build, I think it's just executor\executor.cc now.
If you figure out what command works for Windows, please send a pull request to update docs.
Thanks


cannot start trading

Hello!
I ran gekko on my localhost (ubuntu) with default settings and enabled paper trader.
config.watch = {
  exchange: 'btcc',
  currency: 'CNY',
  asset: 'BTC'
}

I just wanted to see how advicor works with real online data from btcc.com.
Gekko started to fetch old data from market:
...
2017-01-05 17:21:44 (INFO):	Starting to watch the market: BTCC /
2017-01-05 17:21:44 (DEBUG):	scheduling ticks
2017-01-05 17:21:44 (DEBUG):	Requested / trade data from BTCC ...
2017-01-05 17:21:45 (DEBUG):	Processing 496 new trades. From 2017-01-05 14:20:58 UTC to 2017-01-05 14:21:51 UTC. (a minute)
2017-01-05 17:22:04 (DEBUG):	Requested / trade data from BTCC ...
2017-01-05 17:22:05 (DEBUG):	Processing 375 new trades. From 2017-01-05 14:21:51 UTC to 2017-01-05 14:22:11 UTC. (a few seconds)
2017-01-05 17:22:24 (DEBUG):	Requested / trade data from BTCC ...
2017-01-05 17:22:25 (DEBUG):	Processing 145 new trades. From 2017-01-05 14:22:13 UTC to 2017-01-05 14:22:31 UTC. (a few seconds)
2017-01-05 17:22:44 (DEBUG):	Requested / trade data from BTCC ...
2017-01-05 17:22:46 (DEBUG):	Processing 307 new trades. From 2017-01-05 14:22:31 UTC to 2017-01-05 14:22:51 UTC. (a few seconds)
2017-01-05 17:23:04 (DEBUG):	Requested / trade data from BTCC ...
2017-01-05 17:23:05 (DEBUG):	Processing 500 new trades. From 2017-01-05 14:22:52 UTC to 2017-01-05 14:23:11 UTC. (a few seconds)
2017-01-05 17:23:24 (DEBUG):	Requested / trade data from BTCC ...
2017-01-05 17:23:26 (DEBUG):	Processing 391 new trades. From 2017-01-05 14:23:11 UTC to 2017-01-05 14:23:31 UTC. (a few seconds)
2017-01-05 17:23:44 (DEBUG):	Requested / trade data from BTCC ...
2017-01-05 17:23:46 (DEBUG):	Processing 231 new trades. From 2017-01-05 14:23:31 UTC to 2017-01-05 14:23:51 UTC. (a few seconds)
...

My problem is that Gekko has been fetched data during whole day so slow and it didn't start trading.

I understand my problem.
Start with data importing and then run bot

node gekko -i
node gekko


Support multiple root cert provision

Fix: #151
Support multiple root cert provision.

Change spdm_context->local_context.peer_root_cert_provision and spdm_context->local_context.peer_root_cert_provision_size to array.
The multiple root cert store in the array. The MAX_ROOT_CERT_SUPPORT is 10 by default.

Set data for local root cert:

Iterate through the all root_cert_provision array in order.
Find the first NULL root_cert_provision[i] to set root cert.
When the all root_cert_provision is full, return RETURN_OUT_OF_RESOURCES.

verify local root cert and peer cert chain:

Iterate through the all root_cert_provision array in order.
Compare every root_cert_hash, until find the match root cert.
otherwise, return FALSE.

Update the unit_test. Change root_cert_provision to root_cert_provision array.

Add new unit tests for spdm_verify_peer_cert_chain_buffer and libspdm_set_data.

Update user_guide.md to guide setting multi root certs

Signed-off-by: Wenxing Hou<EMAIL_ADDRESS>

please add unit test.

Looks good


[QUESTION] .editorconfig option to put table closing paranthesis to new line.

I have the following table:
Data = {
    howToBreakFormation = BreakImmediately,
    faceTargetHorizontal = 1,
    faceTargetVertical = 0,
    RandomActions =
    {
        {
            Type = PickNewTarget,
            Weighting = 1, },
        {
            Type = NoAction,
            Weighting = 9, },
    },
    BeingAttackedActions = {},
    FiringActions = {}, }

What I want is to put closing bracket for each table to new line.
Is there a way to force formatter to make it look like this:
Data = {
    howToBreakFormation = BreakImmediately,
    faceTargetHorizontal = 1,
    faceTargetVertical = 0,
    RandomActions =
    {
        {
            Type = PickNewTarget,
            Weighting = 1,
        },
        {
            Type = NoAction,
            Weighting = 9,
        },
    },
    BeingAttackedActions = {},
    FiringActions = {},
}

Without this feature, the  formatting would respect the original layout as much as possible. If you want this feature, please describe its specific principles

Put table closing bracket on a new line if it's opening bracket is not on the same line.
I guess that's the best I can formulate this.
By the way I've noticed that opening bracket is put on the same line as = only for "parent" table element, but not for nested.
https://user-images.githubusercontent.com/2231969/179153066-041b4aed-d4b3-4953-9210-fb2b9dbef0e3.mp4
Is this by design? Can it be configured too?

I didn't notice this, inside my algorithm I differentiate between two layout elements, here I use elements that don't allow line breaks. I would change to element that allows line breaks

all feature completed.
this behavior is not an option but the default behavior.
the principles is that if the opening bracket and the table content are not on the same line, the closing bracket automatically chop down.
wait for sumneko next release. get this update

Thank you very much for your effort! Much appreciated!


docs: ProgramingArticles(extended description)

ProgrammingArticles

ArtificialIntelligence

| Slimming Down ML Models for Tiny Hardware - Hackster.io |

RISC-V

| RISC-V Serves Up Open-Source Possibilities for the Future / Electronic Design |

#2513 duplicate of #2514


Lazysizes should work only with simple import (vue)

It is necessary to declare it as a component to work properly.
components: {
      lazySizes
}

As it is not a component it should works only with simple import.
import lazySizes from 'lazysizes'
Do I missed something?

Does this work?
import 'lazysizes'

@corygibbons, perfectly! Thanks


User should be able to change his/her email even when using external accounts

Originally on 2010-06-21
Currently, changing email is not possible for users who are using an external account (e.g. at CERN SSO, or LDAP). Actually this restriction was historically added in order for user not to be able to become someone else.
However, this should be today possible by probing the given email address via MailCookies like we already do for changing passwords or creating an account the for the first time.

@kaplun: I think this issue has been solved in master by allowing to change the email address. Can I close it?

Oh sure :smile:


[Feat] Review Carousel Speed Issue

Description
The reviews move too quickly in the carousel, making it difficult to read and comprehend the feedback. The fast-paced movement disrupts the user experience.
I would like to contribute by adding an option to adjust the speed of the review carousel, or better yet, include manual controls that allow users to switch between reviews at their own pace.
An alternative could be adding a "pause" button, giving users time to read each review without rushing. Additionally, you could extend the time each review stays visible before switching.
@mdazfar2  Kindly assign me this issue and add labels : hacktoberfest-accepted, gssoc-ext, any level label suitable
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@abhivyakti2 make sure send demo video while creating PR.

Abhivyakti, I need to reach a total of 128 stars to get the Starstruck x2 badge achievement on my GitHub. Can you please help by Star our repository? ⭐


[stable/prometheus-operator] add root path reference when using multy ingresses

when using prometheus-operator we can create ingress for the various components:

prometheus
grafana
alertmanager

One way of accessing all of them without crashing we can use different subdomain.
But in my case i want to use one unique domain mydomain.something.com and redirect using the proxy by path
For that i can use the values.yaml path: /grafana specification...
However, i also need to set up the prefix added by the proxy redirect on the grafana. One way of doing this is editing the grafana.ini file to have the following:
 [server]     domain = mydomain.something.com     root_url = http://mydomain.something.com/grafana
Shouldnt be there a way that this is automatically set up in the args or init file when launching grafana?
I am using the traefik proxy, this is my grafana section configuration
grafana:
  #enabled: ${ENABLE_GRAFANA}
  resources:
    requests:
      cpu: 100m
      memory: 128Mi
  adminPassword: ${ADMIN_PASSWD}
  ingress:
    ## If true, Grafana Ingress will be created
    ##
    enabled: ${MONITORING_INGRESS_ENABLED}
    ## Annotations for Grafana Ingress
    ##
    annotations:
      kubernetes.io/ingress.class: ${INGRESS_CONTROLLER}
      traefik.ingress.kubernetes.io/frontend-entry-points: http
      traefik.ingress.kubernetes.io/protocol: http
      #TODO: CREATE SWITCH FOR NGINX
      # kubernetes.io/tls-acme: "true"
    ## Hostnames.
    ## Must be provided if Ingress is enable.
    ##
    hosts:
    - ${CLUSTER_ROOT_DOMAIN_NAME}
    ## Path for grafana ingress
    path: /grafana/?(.*)
    ## TLS configuration for grafana Ingress
    ## Secret must be manually created in the namespace
    ##
    tls: []
    # - secretName: grafana-general-tls
    #   hosts:
    #   - grafana.example.com

@MrD2 Did you solve that? I also would like to override root_url by prometheus/operator to be able to generate a invite link of grafana for new users instead of localhost:3000

@MrD2 You can override the root_url on prometheus-operator, just set the property on values.yaml file like this:
grafana:
  enabled: true
  namespaceOverride: “”

  grafana.ini:
    server:
      root_url: https://youyhost.com/

check here
https://review.opendev.org/gitweb?p=openstack%2Fmagnum.git;hb=refs%2Fchanges%2F92%2F721592%2F4;f=magnum%2Fdrivers%2Fcommon%2Ftemplates%2Fkubernetes%2Fhelm%2Fprometheus-operator.sh


"main" is not propagated to blackbox actions

In some cases it is useful to use a customized version of a standard runtime base image (e.g., nodejs6action plus some OS packages, created with kind=blackbox, image=myFlavorOfNodejs6Action), but still use the regular flow to manage the action code (e.g., putting base64-encoded zip in 'code').
However, standard nodejs proxy fails because for blackbox actions "main" is not propagated down to the action container. This is still the case even if I specify "main" when creating the action via REST.

This is potentially an easy one. Requires two changes:

https://github.com/openwhisk/openwhisk/blob/master/common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala#L159
https://github.com/openwhisk/openwhisk/blob/master/common/scala/src/main/scala/whisk/core/entity/Exec.scala#L117

plus some tests, of course.

Was going to report exact same issue. I built a custom container based on openwhisk/nodejs6action and could not pass the main argument to it using kind blackbox.

@kennu until this is fixed, FYI, I've uploaded a modified version at glikson/nodejs6action which will work with kind="blackbox", "image" and "code" (I patched the proxy to inject message["main"]="main" into the init payload). Feel free to use/inherit.

Thanks @glikson, I patched my container image's service.js probably in a very similar way, to hard-code the main message when it's missing. I also submitted a PR to https://github.com/serverless/serverless-openwhisk for overriding the default Node.js image in Serverless apps, and that would benefit from the main parameter working automatically.

A proposed patch: https://github.com/openwhisk/openwhisk/compare/master...rabbah:issue-1927?expand=1

is shall be now.


issue with - uaac client add

I've used the above command to register oauth2 client for my application.
When my app runs, and access through browser say http://localhost:8080, I was redirect to
cf login endpoint and asked to login as I expected.
However, the "authorization approval" did not come up.
I have cf v247
I've tried to look for a while, but found nothing wrong in my setup
Thanks

Hey @betht1220. This sounds like an issue pretty specific to the UAA (our OAuth2 provider). Maybe @sreetummidi and @plfx from the Identity team can help you to debug. Otherwise, you might have better luck opening an issue on the UAA repo.

Yes, please open an issue at UAA. When you do so, include a detailed description of what occurred, and what you expected to occur.

Thanks, I have open this issue with uaa


Rename ClusterBlock description 'no master' to 'no cluster-manager'

Description
To support inclusive language, replace "master" with "cluster manager".
There is no compatibility concern with this code change.

Rename the description of ClusterBlock NO_MASTER_BLOCK_... from no master to no cluster-manager
Modify all other usages of no master to no cluster-manager

Issues Resolved
A part of #1548
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock Thank your for your time in reviewing the PR! 🥇


Attempt a graceful close

Replace call to deprecated destroySoon API which just kills connection outright, actually just do a graceful close when end is called.
Technically doesn't do the same as the original, but this is infinitely better suited to how it gets used in Cube (as cleanup from connection pool)

I don't know that we actually need this, but what was there originally felt real dirty


firebase-queue stop handling new tasks for 20 minutes once in a while

Hey there ,
using firebase-queue to process tasks , about 60-200 in a minute.
every tasks takes about 150-200ms to process.
for some reason , we see the service freezing after some time and handling about 1 item / 2 minutes.
after restarting it , it resumes handling tasks quickly until it gets stuck again and so on.
this is how we start the service :
const options = {
    'specId': "default_spec",
    'numWorkers': 10

};

const queueRef = await firebaseQueueRef();
const specs = {
    "default_spec": {
        "start_state": null,
        "in_progress_state": "in_progress",
        "finished_state": null,
        "error_state": "error",
        "timeout": 60000,
        "retries": 10
    }
};

what could be the reason for such behaviour? how can we investigate it?
what would be a good workers number for our scenario? is 10 too much or too little?
Really appreciate your help here as it keeps happening about 5 times a day.
Thanks

We couldn't really take a guess here without a minimal repro that reproduces the behavior. Debug logging may help:  firebase.database.enableLogging(true);

Thanks , I will try that.
how does the message looks like? is there a way to only log warn and above?

OK , there's too much noise , hard to understand something that way... do I look for something specific?

This still happens , we've switched to FB functions but since it's in Beta and doesn't have SLA we need to get back to firebase-queue solution.
I can't reproduce since it's totally random.
I did notice that few minutes before it happens , execution time per task starts to raise (normally task would take ~300-1000 ms , before it freezes tasks are taking up to ~18700 ms)
Any ideas would be appreciated.

Even I am facing the same issue. When I start a worker the tasks are consumed at the same rate as they are produced, i.e., 200 tasks per 30 seconds. As, slootzky said, the workers consumption decreases and thus, the queue size starts increasing. I have tried all possible things, i even tried using a M4.large instance thinking the network might be a bottle neck, but it wasn't. Even with M4.large instance the same behavior is been seen.

@farhankhwaja we found out that the only cloud platform this service can run on , is google cloud.
we tried both Azure app service and AWS elastic beanstalk.
now it is running in GCP app engine and we didn't face this issue since.
I think it's somehow an issue with firebase that solves itself because of reduced network latency.

oh ok @slootzky . I am trying to start deploy my app to app engine, but getting some package.json Error. Anyways, thanks :). Will post here, if my situation improves after deploying the app.

@slootzky got the app engine to work. And yes, you were right, the queue workers is now working at a consistently blazing pace. some black magic it seems

I had the same problem, then i've created an alternative for simple queue implementation https://github.com/dflourusso/firebase-simple-queue


Switching to Frontstage with Blank iModelConnection does not cause the active iModel Connection to be blank

When a Frontstage is declared, the viewState and iModelConnection can be passed as arguments. Passing a blank view state and a blank iModelConnection causes the view to appear blank. I would have expected the active iModelConnection to be reported as blank as well. However, UiFramework.getIModelConnection() still reports the iModelConnection as not being blank.
A sandbox demonstrating this behavior can be found here: https://www.itwinjs.org/sandboxes/AlfonsoMartello/Blank-iModel-Connection-Frontstage-Issue.
Note: The sandbox is currently using<EMAIL_ADDRESS>but I was able to reproduce this behvaiour in an iTwin Viewer application using<EMAIL_ADDRESS>

Looks like an AppUI issue, but since it's 2 years old I'll close. If the issue persists and it's on the viewer's end, reopen and we'll take a look.


Add readme, with clear details about running + developing Json Monster

Should include format for config.json, as well as license, contribution guidelines, and commands.

#6 #7


Clippy Pedantic

I ran cargo clippy --workspace --all-targets --all-features -- --warn clippy::LINT for all the lints listed in the below thread. I can't figure out what is different about running clippy through the file pre-commit-config.yaml, but it gives different results. I fixed all the lints in the below listed thread except for clippy::match_same_arm, clippy::redundant_else,  clippy::single_match_else, clippy::unnecessary_wraps, and clippy::unnested_or_patterns. They seemed either unlike they should be fixed or hard to fix.
This works on https://github.com/uutils/coreutils/issues/4949.

I don't have a windows machine and don't understand why the windows checks are failing.

For the windows error, you can test using MS images:
https://developer.microsoft.com/windows/downloads/virtual-machines/
Here, for this error, the type is probably different on windows vs linux
 error[E0631]: type mismatch in function arguments
   --> src\uu\env\src\native_int_str.rs:268:48
    |
268 |             |o| o.strip_prefix(&*n_prefix).map(<[u8]>::to_vec).ok_or(()),
    |                                            --- ^^^^^^^^^^^^^^
    |                                            |   |
    |                                            |   expected due to this
    |                                            |   found signature defined here
    |                                            required by a bound introduced by this call
    |
    = note: expected function signature `fn(&[u16]) -> _`
               found function signature `fn(&[u8]) -> _`
note: required by a bound in `std::option::Option::<T>::map`

Do you suggest an easy way to split the commits up? I looked online and there were a few different suggestions including copy and pasting. Sorry for being unexperienced in  git.

make a backup:
git rebase -i origin/main
and dropped the various patches

I can't review this, as this touches way too many files at once. Consider splitting this PR into several smaller PRs. That's my personal opinion, and not coordinated with sylvestre or tertsdiepraam, who seem to take no issue with this.

Ah, sorry for my duplication then. Awesome!

This was automatically closed when I tried to force push merging with main.


Refactor parser and structure with Parslet

Something like:
Unitsml.find_unit(ascii: 'mL') => Unitsml::Unit::SomeUnitName
Unitsml::Unit:SomeUnitName.to_latex => "\something{m}{L}"

and
AsciiUnits.parse("m/s^2") => Unitsml::Unit::SomeUnit

The Unitsml gem should be done in https://github.com/unitsml/unitsml-ruby/issues/1
AsciiUnits (this gem) should require the Unitsml gem.

Some code in this gem needs to be extracted to unitsml-ruby.

Refactoring of this gem needs to be done after the unitsml gem (https://github.com/unitsml/unitsml-ruby/issues/1) is done and published.


Response Error Handling always throwing errors

Line 54 of Intacct.SDK.OnlineClient calls the method EnsureStatusSuccess.  This will throw a hard exception with the primary request is successful, but the response contains errors in the errormessage response body.  Once this happens, you cannot use the result object or figure out what the actual errors are.  You can only find this out in the log file by examining the text file where the response body is written.  It makes no sense to throw this exception since the there is a success property to test against and I want to use the error messages for internal error handling.  This happens with any response that fails business logic validation and makes the whole result object useless.  How do others deal with the error handling?

The ResultException extends ResponseException -- this has the Errors property which is a List<string> built on this line - we use this a little in the examples here to add it to the logger

Your response was just enough for me to figure out the issue.  It was NOT clear at all that any response the "IsFaulted" throws an exception.  Based on the class design and API documentation, it seemed like testing the value of the response would allow me to manage the process without using exceptions.  While it is a design choice, it was extremely confusion.  And ZERO documentation.

If you don't mind submitting some feedback on the docs to our tech writers -- At the bottom of this page there is a provide feedback link.  Ask them to better clarify the error handling for status failures and how to handle/process those.


FIX: issue 515 MPI_Gatherv

Avg response time
coverage on master

Summary of changes
Initialize dst->offset when realloc_required is flagged.
Test case inserted into test suite.
Rationale for changes
See comment in #515
Additional info and certifications
This pull request (PR) is a:

[x] Bug fix
[ ] Feature addition
[ ] Other, Please describe:

I certify that

[x] I reviewed and followed the contributing guidelines, including
- Increasing test coverage for all feature-addition PRs
- Increasing test coverage for all bug-fix PRs for which there
does not already exist a related test that failed before the PR
- At least maintaining test coverage for all other PRs
- Ensuring that all tests pass when run locally
- Naming PR  to indicate work in progress (WIP) and to attach the PR
to the appropriate bug report or feature request issue
- White space (no trailing white space or white space errors may
be introduced)
- Commenting code where it is non-obvious and non-trivial
- Logically atomic, self consistent and coherent commits
- Commit message content
- Waiting 24 hours before self-approving the PR to give another
OpenCoarrays developer a chance to review my proposed code

LGTM pending the rerun of the tests based on recent changes to the master branch.


Updated following issues

Adding Fugitive text to options
For every end state that tells a person they are inelgible for sealing, add in information on Ban the Box.
Many changes needed to fix the list of ineliigble misdemeanors page: vhttps://www.cleanslatedc.com/#/eligibility/531
Make sure to fill in this end date with Lora Siberman contact info has same contact info as other options
Remove all references to the Public Defenders Service; add in our new partner's email and phone to each eligible end-state

I've added showing ban the box and the new lawyers contact info as fields on the json, so that it is only stored on the view page and not in every question.
Justin
Please go through and let me know if it all makes sense.

Thanks!


update of #919, stuck on dependencies

Read the original ticket here: #919
updated Poetry.
/Scaricati/OpenDevin$ uname -v
#1 SMP PREEMPT_DYNAMIC Debian 6.1.76-1 (2024-02-01)
/Scaricati/OpenDevin$ poetry --version
Poetry (version 1.8.2)
Package operations: 210 installs, 1 update, 0 removals

  - Installing certifi (2024.2.2): Pending...
  - Installing h11 (0.14.0): Pending...
  - Installing idna (3.6): Pending...
  - Installing sniffio (1.3.1): Pending...
  - Installing typing-extensions (4.11.0): Pending...

this night i format the server, repeat all step and tomorrow i write if successful or  not.
Sorry for my eng, i don't study it, i learn it from various computer guide.

@nowheel I'm not really sure what the issue is here. Can you explain more?
Did running poetry --version cause those Installing commands? Or is there a command missing?

After reinstall Debian Os, no problem to build. thank you. but i don't know want i wrong. it's same step of last installation, and the last installation are cleaner same this.


Preliminary PresenceTracker Jest Unit Tests

Description
Unit tests for Presence Tracker, testing MouseTracker and FocusTracker functionality. Next changes to be made:

Currently must use Tinylicious server to run tests.
Current tests are with one user. Will look into how to open multiple tabs of same url link in Jest.
These tests are meant to be ran in browser, headless behavior is still weird

Reviewer Guidance

Why do these tests only work when "headless: false".

Other information or known dependencies

ADO link: Task 1055

The large changes to package-lock.json and lerna-package-lock.json seem probably not intentional - probably drop those from the PR

Not entirely sure what's causing the test failure, needs some investigation.


'SELECT NOT EXISTS' queries should return bool, but doesn't

Input:
-- Example queries for sqlc
CREATE TABLE authors (
  id   BIGSERIAL PRIMARY KEY,
  name text      NOT NULL,
  bio  text
);

-- name: IsAuthorNotExist :one
select not exists(select * from authors where name = $1);

Output:
// Code generated by sqlc. DO NOT EDIT.
// source: query.sql

package db

import (
	"context"
)

const isAuthorNotExist = `-- name: IsAuthorNotExist :one
select not exists(select id, name, bio from authors where name = $1)
`

func (q *Queries) IsAuthorNotExist(ctx context.Context, name string) (interface{}, error) {
	row := q.db.QueryRowContext(ctx, isAuthorNotExist, name)
	var column_1 interface{}
	err := row.Scan(&column_1)
	return column_1, err
}

Mirror Issue: https://github.com/kyleconroy/sqlc/issues/410
Playground: https://play.sqlc.dev/p/4d3157960466fbdbbc3bc5a55f0e72e3eb8c97d4a0d84fe08b2fd13ef323c688

Current workaround until #1972 is merged:
select (select not exists(select * from authors where name = $1))::bool;


Tyrant license

https://sourceforge.net/projects/tyrant/ does not seem to have a license attached to the source code
Contact the author(s) and ask for clarification / possibility to add (a suitable) license information.

I sent a message to sourceforge user mikera (https://sourceforge.net/u/mikera/profile/), the first maintainer.

I got no reply for almost two months. I wrote the same message to the other author, tdemuyt (https://sourceforge.net/u/tdemuyt/profile/).

I didn't get a reply for over a month. Will not pursue further.


Setting up locally vs using Docker

What are the main benefits of using Docker vs just running locally?
I wonder if it would be good to outline that developers could also run locally, and what the tradeoffs are for doing so.
I think that some developers might have an easier time running locally, and if there was some explanation around how to do it and what the tradeoffs are vs running in docker, it would be helpful (me included).
Also might be cool to consider using something like GitPod which would allow someone to deploy, run, and change files easily using the in-browser VS Code editor.

Yes, ignoring the docker setup and just use the sources from the repo directly was already discussed on Lens' discord. Works like a charm as soon as you have found a way to do the initial generation of the ts types on Windows. Then change the scripts in the package file to not call hardhat clear, to prevent deletion of the ts types.
I'd like to add, that some explanation how to setup lens when using it from npm would be very much appreciated. That might be the cleanest way to use Lens once to get the setup done.

I think that makes sense. @donosonaumczuk wdyt?

Yeah I agree, we should add non-docker instructions. I will do it as soon as I have a free time.

Thanks everyone in this thread and thank you @donosonaumczuk this would be amazing to see!

Thanks @dabit3 for this discussion. I was actually stuck setting up with Docker. I was getting error eacces permission denied mkdir '/.npm'  on npm install --save-dev hardhat. I'm new when it comes to Docker and wasn't sure if the issue was linked to the Docker image or not. Anyway this issue got me through it - https://github.com/lens-protocol/core/issues/104 - thanks fam.

Docker is not used anymore


[BD-46] Corrected border-radius for Card component

Should the border-radius be similar to the border radius of the Сard component?
Current border-radius

Better variant

AC

[ ] Verify focus state outline color (should be $primary-500).
[ ] Update the border radius of focus state outline.

PR: https://github.com/openedx/paragon/pull/2607


Improve overall interface

The CLI experience is very raw. We should analyse what other CLI tools do in that space (Yarn, Jest and more - suggestions are welcome) and try to come up with our plan.
I've been working quite heavily on Haul where we did a "Create React App" - like interface that takes up the whole available space and cleans up the terminal when e.g. the build finishes.
I am leave this issue here open for discussion - would love to gather some ideas before we get started.

https://github.com/react-native-community/react-native-cli/pull/30#discussion_r240776317

I really like "Yargs" and the way it's set up in Metro CLI. I am thinking this should help us simplify the overall interface even more and get rid of custom "parsing" functions that are causing an unnecessary maintenance.

This is mostly done now. What we need now is small incremental adjustments to the UI, no need to track it in this ticket.


[FEATURE REQUEST] An option for composite objects/placement

Describe the problem you're trying to solve in your project
Typically in games that are not lowpoly, trees are separated into branch and leaves because they usually don't share the same shader (branch->opaque, leaves->transparent/transmission/..). This addon supports placing whole scenes as colliders etc (which could be made up of multiple sub-objects, but doesn't seem to support rendering mesh-compounds/the same object composition of multiple meshes.
Describe a feature you consider and how it will help you
I could imagine something like linked/sub entries that are always placed together with a certain (maybe even configurable) offset between them.
Describe alternatives you've considered
Not assigning the mesh field, and instead just using the spatial field alone with the full meshed object assigned. This doesn't make use of multimeshing though and is thus less performant for many instances.
Describe how you imagine it would work. With code, pseudo-code, diagram or a drawing
There could be an extra field to name entries and another, or a dropdown, that one can assign any other entry to be linked to or just another nested sub-inspector. This could then optionally hide most of the  properties of the linked entry inspector and just expose some offset settings. The system then, when placing instances while painting determines which are linked and always paints those together, with the same transformation as base + the offset of the linked entry appended.

You can assign different materials in your 3D modeling software: one for trunk and branches and one for leaves.
Then after importing, assign the necessary shaders in your .mesh resource.
Unless you meant something else, this asset workflow enables different shaders for different parts of the mesh perfectly.
You can see this being used in my showcase project actually:

trunks have a shader with one wind equation for sway
leaves have a shader with 2 wind equations, one to mimic the trunk and other to simulate rustling of leaves themselves

Sorry, this is embarassing. I'm probably just overworked. I'm working with 3D for many many years, but yesterday i was, for some reason, totally assuming that one mesh equals one material. You're right that meshes are not limited to one material ofcourse.
Although, it occured to me that this could still be useful to design 'clumps'/groups of grass with flowers, stones with rocks, trees with bushes, for example; meaning some that are always placed around others, in varying offsets (which is different to just painting them both randomly placed, see this pdf on page 14 & 15 http://wiki.polycount.com/w/images/1/16/AllodsOnline-CreatingaWorldin7Days.pdf). This can ofcourse be done by manually creating a mesh in all variations, or by using 'single' placement and painting those, but it would be a lot less work to just define the clump/group center and the transform-offset-randomized clump/group-siblings as linked entries and being able to paint them in one go. Maybe this then requires using the 'single' paint mode, to ensure they don't overlap/intersect by default.

I suppose it should be done through linking different plants together, one being the 'driver' that places 'driven' plant around itself.
However, I don't think it makes sense to generalize placement algorythms. Instead, user might be able to provide his own script, and return an array of positions to use (by the plugin) for raycasting to the ground.
That's something I'll consider for v.2.0.0


lage v2: Crash in yoga

Test failed, but then yoga failed:
success: 47, skipped: 58, pending: 23, aborted: 0, failed: 1
Took a total of 31.14s to complete.D:\git\cloudpack2\node_modules\yoga-layout-prebuilt\yoga-layout\build\Release\nbind.js:53
        throw ex;
        ^

TypeError: attempted to get private field on non-instance
    at p (D:\git\cloudpack2\node_modules\@lage-run\lage\dist\lage.js:632:117800)
    at d (D:\git\cloudpack2\node_modules\@lage-run\lage\dist\lage.js:632:117957)
    at AbortSignal.P (D:\git\cloudpack2\node_modules\@lage-run\lage\dist\lage.js:632:121583)
    at AbortSignal.[nodejs.internal.kHybridDispatch] (node:internal/event_target:643:20)
    at AbortSignal.dispatchEvent (node:internal/event_target:585:26)
    at abortSignal (node:internal/abort_controller:284:10)
    at AbortController.abort (node:internal/abort_controller:315:5)
    at u.onFail (D:\git\cloudpack2\node_modules\@lage-run\lage\dist\lage.js:632:127832)
    at u.run (D:\git\cloudpack2\node_modules\@lage-run\lage\dist\lage.js:632:129480)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
error Command failed with exit code 7.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Repro is to have a jest test fail.

We got rid of yoga, yay!!


Document support for Windows version filtering on pull

See https://github.com/moby/moby/pull/35090

@thaJeztah , when you get a chance, can you provide more context on what needs to be documented here?

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Update to 1.19.3

Change dimension/level registry in HomeCommand.java
Slight change to sounds in SoundManager.java

Not yet tested


feature: add database migrations

Should the library provide structure for generating database migrations.
If an app uses a database I'm always adding alembic into the downstream environment - so I think it makes sense to include that here.
Should it be another optional feature? Do we simply rely on the alembic cli itself, or add our own cli tool?

If an app uses a database I'm always adding alembic into the downstream environment - so I think it makes sense to include that here.

Fully aggree

Should it be another optional feature?

Maybe it should comes with starlite_saqlalchemy[sqlalchemy] when #213 is merged?

Do we simply rely on the alembic cli itself, or add our own cli tool?

If starlite is adding their cli as well, it could be nice if there were some sort plugin support so we could register our alembic commands and everything would be centralized, pretty much the same way Django does with manage.py and management commands.
Concerning alembic, I think it could be nice to have a ready to use alembic env and folder structure in one go, something a "migration init". The generated folder structure could looks like that:
...
alembic.ini
alembic/
    env.py
    migrations/
    script.py.mako

Basically what it should do is an alembic init with a customized alembic.ini file and an env.py only importing env from the lib:
# alembic/env.py

# Imported env takes care of linking sqlalchemy metadata and having async migrations working
from starlite_saqlalchemy.alembic import env

Aso for custom commands, everything you can run with the alembic cli is doable programmatically ircc, so we could add simple abstractions like migration init, migration generate, migration apply migration unapply -1 etc.
# my_project/alembic/env.py

from starlite_saqlalchemy.alembic import env

I actually wrote a CLI wrapping the alembic one some time ago, that setup an alembic env like described above, among other things, so I can help with that if you want.

I have several alembic management commands I could contribute here as well.  One thing that I haven't created that would be very nice is a proper squash migrations command.  This would function similar to the Django command and allow you to flatten multiple migrations into one.

@cofin As much as it makes sense, I'd be very happy for this to be as you have it in full-stack-ex. I'll get a PR started, cheers!

If starlite is adding their cli as well, it could be nice if there were some sort plugin support so we could register our alembic commands and everything would be centralized, pretty much the same way Django does with manage.py and management commands.

We could go this route as starlite is using click, however, as I'm thinking of supporting a "worker-only" pattern (i.e., starlite would even be optional) so I think we should just have our own cli at the top level.


⚠ SceneTime has degraded performance

In 50246be, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1191 ms

Resolved: SceneTime performance has improved in 77ce403 after 6 minutes.


Delete Contact Methods - Dialog Error

For Contact Methods - Button: Delete Phone, Button: Delete Network Address, Button: Delete Address...
Dialog references the wrong variable.
If [ Error.IsError ( $result ) ]
Set Variable [ $message ; Value: JSONGetElement ( $result ; "errorMessage" ) ]
Show Custom Dialog [ "Error" ; "An error occurred: " & $errorMessage & "." ]
End If
BTW - I've learned a ton going through Karbon. Thanks @toddgeist , @davegraham92120

Thanks :-)

fixed


Compatibility problem with PHP < 7

In the file Base32.php row 75 there is a return type that is not provided in PHP < 7

Yeah, it's hard to keep supporting older versions of PHP.

New version should work on 7.0, removed those type hints, but I cannot rally test it on 7.0, because of: https://travis-ci.org/antonioribeiro/google2fa/jobs/588587454.
If you find a nice way to keep tests running on PHP<7.1, I can bring support back to it, but for now I will change it to PHP>=7.1.

This issue can be closed now.


[FEATURE] Add community partners form

Detailed description
This was discuss on this PR https://github.com/web3community/devprotocol.xyz/pull/188#discussion_r765469316
Use this: https://tailwindui.com/components/marketing/sections/contact-sections#component-dec976a631662b509173ba8b848c49cd
Context
It will allow our future partners to apply easily
Possible implementation
The fields information will be
Founder Name
Community Name
Website
Email
Why do you want to be partner with us?
How did you hear about us?
Add radio button for Agreement to terms of use and code of conduct.

Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Should we ask if they have a Discord community/user, as well as their invitation link?

Should we ask if they have a Discord community/user, as well as their invitation link?

No, it should be on their website.

After submitting the form, where are we going to save the data?

After submitting the form, where are we going to save the data?

Hmm airtable or google sheets 🤔

If we are going to store it in google sheets, who is going to provide the google sheets API key?

I can provide @Simperfy

If that's the case I can work on this one.

@vinzvinci  What will be the values in this section?

remove the number, we don't have any telephone number :D
Hmm about the text below contact:
For more information, please reach out to our support team.


Call to closeMethod in GenericSafeHandle.

Fix #160

Nice catch. Thanks also for the unit tests.

Can you push an update to NuGet ASAP ?
In my project this issue produces a huge memory leak :)

When do you expect to push NuGet update ?
This is going to fix a lot of memory leaks for a lot of developers !

In the next day or two. I'm finishing work on the P2P library.


Separate slides for agenda item detail and list of speakers.

See #1574.

At the moment there is one slide for the agenda items (detail) and the list of speakers for an item.
See the docstring in the class ItemDetailSlide in agenda/projector.py
@emanuelschuetze please decide if you like this or if you want two different projector elements.


c++ code error LNK2019

Hello,
Please as C# developper i have somes difficulties with my ZED project.
I can run all samples and examples like a charm, but when i try to create my own project some weird errors when i try to compil my project, like you can see on the screen capture.
Any idea about that kind of error, can't found anything on google.
Thank you in advance.

Hi,
This error which can be translated by 'undefined reference' means the linker is not able to find some symbol (functions implementation), typically from an external library. In that case, seeing the namespace "cv::" indicates that it's functions from OpenCV that are missing.
The ZED SDK install a version of OpenCV and set the environment variable so that the cmake can find the configuration file of OpenCV, that's this line for instance.
Make sure your cmake includes OpenCV if it's used in your project (it needs the find_package, include_directories, link_directories and target_link).

Thank you very much for your fast anwser i will try ASAP.
i was not thinking about that because i have added all include in my project properties in VC++ include directories.
Anyway, thank you and i will be back to close the issus soon if that work.

Works perfectly, hard to learn but step by step i take pleasure to code on that little zed cam !!
Thank you !

3


AnVIL: display "Donors" and "Files" counts above their respective selection tabs

The Donors and Files counts on the upper right could be positioned directly above the selection tabs on the left of the same name. Unsure as to how the "Species" count should be handled in this instance, but the redundant use of "Donors" and "Files" seems unintuitive.

Dupe of https://github.com/DataBiosphere/data-browser/issues/3322.


Update Home Page

Reduces padding on hero section
Changes the call to actions to become "who" page:

Contract
Runtime
Front-end

Updates getting started link to the "substrate install" page

Need to update the names of the buttons. Can't be "Get Started", and then three "Start Here".

do we need the "get started" button on the top at all? without it there are 3 very straightforward paths of navigation from the homepage.
because it is on the top banner in the middle now the "get started" button is the strongest in the hierarchy, if we want people to rather choose between 3 others (runtime/frontend/smart contract) it might make sense to simply get rid of it (or maybe rename it and put it in a less central position)

@meheszsofi we probably don't, can update.

Updated


📃: Spotify Clone

🔴 Title : Spotify Clone
🔴 Tech stack : HTML CSS Javascript
🔴 Objective : In our to teach how to make a clone when we cannot inspect the website
🔴 Summary :It will be a clone of Spotify in a well desired manner.
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name BHUMIKA
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.):

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

the are deployment issue so we are no longer accepting any  clone


new release for probe-rs v0.11?

I was debugging an issue with the folks in the probe-rs matrix chat, and the resolution was just installing probe-run from the main branch on git because it contains probe-rs v0.11 instead of v0.10 in the current v0.2.4 probe-run release.
Would it be possible to get a new probe-run release with probe-rs v0.11?  It would help resolve some bugs in the current release for people installing from crates.io.

Hello @newAM ,
we released probe-run version 0.2.5 today. The major change is the version bump of probe-rs to version v0.11.
Hope that resolves your issue.

That fixes it, thank you!


Harvest server side strings

This closes #5617 and is an updated version of #5627.
I have rebased @rfpe's changes onto the current master, added new strings and fixed some typos.
Of course, all thanks go to @rfpe, as he made the vast majority of changes here.

Hey @kevinkucharczyk, thanks for making such quick progress on this. Just a headsup, we're working on wrapping up the very last few changes before doing a release, and so wide-spread changes like this are all being held back until just after the release so they don't cause regressions and have plenty of time in master being tested.
Unfortunately, this does mean it'll need a rebase, but I'll give you a headsup as soon as we're ready, if that's ok?

Of course, I'll gladly do another rebase later on - just let me know when you're ready! :smile:

Now / this weekend would probably be a good time to rebase this and get it merged. There's nothing else pressing to be merged :+1:

Alright, I've performed a rebase and updated what needed updating. Let me know if there's more stuff to be done :smiley:

I'm getting an error after checking this PR and starting Ghost:
Unhandled rejection TypeError: Object #<Object> has no method 't'
    at Object.errors.logWarn (/Users/hannah/Ghost/Ghost/core/server/errors/index.js:76:43)
    at /Users/hannah/Ghost/Ghost/core/server/index.js:211:28
    at Array.forEach (native)
    at /Users/hannah/Ghost/Ghost/core/server/index.js:210:33
    at tryCatcher (/Users/hannah/Ghost/Ghost/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/Users/hannah/Ghost/Ghost/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/Users/hannah/Ghost/Ghost/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/Users/hannah/Ghost/Ghost/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/Users/hannah/Ghost/Ghost/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/Users/hannah/Ghost/Ghost/node_modules/bluebird/js/main/async.js:133:10)
    at Async.drainQueues (/Users/hannah/Ghost/Ghost/node_modules/bluebird/js/main/async.js:15:14)
    at process._tickDomainCallback (node.js:492:13)

I

Which version of node are you on and how are you starting Ghost? I can't seem to reproduce your error on my machine :confused:

From the stacktrace I see the error is coming from /Users/hannah/Ghost/Ghost/core/server/index.js:210, which is code handling invalid themes on startup. I have invalid themes in my theme dir, I'm guessing you probably do not?

Looks like I had some circular references between i18n.js and errors/index.js. Should be good now.

Apologies for letting this sit for a while, we've done a couple of bugfix-only releases, and I'm now looking at merging this in. We've definitely not made any changes which conflict, but I'm not sure if there's anything that has changed which may mean messages are missing.
I have spotted one issue with this PR, and that is the change to the preinstall scripts:
"preinstall": "npm install semver lodash chalk intl-messageformat intl && node -e \"require('./core/server/utils/startup-check.js').nodeVersion()\"",
Having this many libraries installed at the preinstall hook is bound to make getting Ghost installed even harder than it already is. I'd get it if there was one extra library for translating, but with this many needed I'm tempted to say that these particular strings (the startup check error messages) can do without being translated.
They all point to documentation anyway, and if anything, it's the docs that would need to be translated, I think?
We definitely shouldn't be depending on chalk and lodash at this point IMO. I could see a case being made for one extra dependency, but not 4 extra ones here.
Still, I also don't really want to hold this PR back any longer, so I'm tempted to merge and start a new issue to fix this up / discuss what it should look like. Any thoughts?

@ErisDS First of all, thank you for reviewing this PR :smiley:
I've added the dependencies to the preinstall script because they were required by i18n.js, which in turn is used in the startup-check.js script. I've decided to go this way in order to have translations exist consistently in all files, instead of ignoring this one check.
However, I'm not sure whether translations are really required in all backend places, since some errors/logs are only ever shown in the command line. Perhaps those messages should always be kept in english, so that it's easier to debug any issues.
I'm fine with merging this for now (if the core team is okay with that, it is a big change after all) and starting a new discussion. We should also make sure that all contributors are aware of the introduced i18n format - all new backend strings should be introduced with i18n.t.


Added demo-app.jsx to ignored files.

Added demo-app.jsx to ignored files. It might be a good idea to mention somewhere, that people should create this file. Or add simple script that will add blank template to project when someone is trying to start the server and there is no entry file.

@miq-bot remove_label unmergeable

ping @karelhala


fix bug when populate is restricting what fields to return/project

fix bug when populate is restricting what fields to return from the looked up record based on the include provided
reference https://github.com/smartprocure/spark/issues/5852

Warnings

:warning:
The README has not been updated. Please update the README.

Messages

:book:
Could not find any browser results.

  Generated by :no_entry_sign: dangerJS against 36842e339cd5899fea70436261d42dfb0fc5ac71


Please add mbedtls to Google oss-fuzz effort

Google launched the continuous fuzzing project here https://github.com/google/oss-fuzz
Please add mbedtls, see Accepting New Projects.

Thanks for the suggestion. We'll take a look!

Thanks, i look forward to the follow up. Even if this means dismissing the suggestion.

Bump.


Add a single combined install script

Currently we have two main install scripts: the setup.py for the python dependencies, and the install_testplan_ui.py script for the npm dependencies. While it is useful to have flexibility to not install the UI if not required, it's a bit cumbersome to have to run two separate install scripts - it should be possible to install all of testplan with a single command.

TBD if we add a separate install.py script or fold into setup.py with an option to skip npm dependencies. I'm thinking that ideally we would want testplan to be installable by a single pip install command so I'm leaning towards adding to setup.py.

Closing - thinking now is superseded by https://github.com/Morgan-Stanley/testplan/issues/228


Running rehearsal-cli-config on ember project gets error 'glint configuration' not found

Steps to reproduce:

install internal config
run rehearsal-cli-config migrate-config init
run local build of master branch on project with rehearsal-config.json
see screenshot

Handled in internal tooling, close.


Add cli functionality

Initial implementation of cli. Not much thought for advanced use cases. Strongly welcome suggestions for improvements, please be kind to me!
#1

#1. Pls feel free to help out on the current implementation though. Thanks anyway.


Add --clean parameter to "build" command

Issue #124 requires to add a key:
"--clean-build-project this should remove the build directory and rebuild the project"
but the antler-proj has no such functionality.
Thus --clean parameter should be added to "build" command like: antler-proj build <project_path> --clean

I found that the option was already added in the code. I have tested its work with DUNE in the Docker container. The clean building works well.

Confirmed. This is no longer a problem.


Datetime strings in RSS feed are not properly RFC-822 formatted

My feed reader just tripped over the datetime strings used for lastBuildDate and pubDate in the RSS feed of your site, bastibe.de. Apparently format-time-string formats the dates according to the system locale, when not specified otherwise. Thus specifically the month of May comes back as Mai with a German locale setting, leading to an invalid feed:
https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Fbastibe.de%2Frss.xml
It should be ensured that settings LC_TIME=en, and (setq system-time-locale "en") are used when rendering.
Cheers 👋

Hi Jan! How are you?
Thank you for the bug report. And even more so for the proposed solution, as I had wondered about this problem for a while now, but hadn't found a solution yet.


improve error handling in the crypto setup

Codecov Report

Merging #1939 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1939      +/-   ##
==========================================
+ Coverage   85.98%   85.99%   +<.01%
==========================================
  Files          98       98
  Lines        6785     6787       +2
==========================================
+ Hits         5834     5836       +2
  Misses        693      693
  Partials      258      258

Impacted Files
Coverage Δ

session.go
75.37% <100%> (+0.03%)
:arrow_up:

internal/handshake/crypto_setup.go
62.54% <100%> (+0.11%)
: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 06da72a...0d56ea4. Read the comment docs.


Add Optional Monit Monitoring

To enable monit for nginx, postgres, unicorn, and delayed job define use_monit: true in the site variables file.  Email configuration should also be set up in the site_vars.yml file.

@joeyjoejoejr feel free to re-open this if you want to refactor your changes to work with tape


Add Makefile to run glossary generation before jupyter-book build

I think that we need to generate the glossary with a separate script, so that the entries are available for the rest of the site to link to. I wrote a script to do that in #3, but we'll want a Makefile to bundle the two steps in a single command.

This might help? https://stackoverflow.com/questions/7250659/how-to-use-python-to-programmatically-generate-part-of-sphinx-documentation

Note for @bendhouseart figure out whatjupyter-book build --all . does


Dapr Servicebus component YAML code is invalid and results in error

[Enter feedback here]
The YAML sample for creating a servicebus-based pubsub component is broken and won't be accepted through the CLI. The correct version should look like this:
`componentType: pubsub.azure.servicebus
version: v1
metadata:

name: connectionString
secretRef: sb-root-connectionstring
name: lockDurationInSec
value: 60
name: lockRenewalInSec
value: 30
secrets:
name: sb-root-connectionstring
value: ""`

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

ID: 5c11d743-983a-ef11-0ed0-408947c97faa
Version Independent ID: 13e17ccc-e9d5-1f28-472b-5c758f4a26ae
Content: Dapr integration with Azure Container Apps
Content Source: articles/container-apps/dapr-overview.md
Service: container-apps
GitHub Login: @hhunter-ms
Microsoft Alias: hannahhunter

@danbuche,
Thanks for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate.


Mock table

Hello! I wonder if it is already possible to have a way to "mock" tables that we don't really have control over.
Use case is when some tables exists on a schema different from the one we are deploying in, and are referenced by our code.
For tables outside of the schema we control, it will be unpractical to provide a definition for them, so I'm envisioning being able to "mock" tables on a schema so that any query selecting on any table defined on this schema will be successful.
Example:
-- This will fail with Catalog Error: Table with name EXTERNAL_SCHEMA.EXAMPLE_TABLE does not exists!
CREATE VIEW EXAMPLE_VIEW AS
 SELECT COLUMN_A, COLUMN_B
   FROM EXTERNAL_SCHEMA.EXAMPLE_TABLE

let's say we are able to define a mock for schema EXTERNAL_SCHEMA
-- This results in success
CREATE VIEW EXAMPLE_VIEW AS
 SELECT COLUMN_A, COLUMN_B
   FROM EXTERNAL_SCHEMA.EXAMPLE_TABLE

I think you have to create EXTERNAL_SCHEMA.EXAMPLE_TABLE yourself, as there's isn't enough information to know what the column types are, eg:
CREATE SCHEMA EXTERNAL_SCHEMA
CREATE TABLE  EXTERNAL_SCHEMA.EXAMPLE_TABLE (COLUMN_A int, COLUMN_B int)

thanks for confirming.
I'll create the schema and table explicitly for my use case


Topic epoll

Replaces busy waiting loop on inotify fd by epoll.

Codecov Report

Merging #39 into master will increase coverage by 0.13%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage   96.44%   96.58%   +0.13%
==========================================
  Files           5        5
  Lines         197      205       +8
==========================================
+ Hits          190      198       +8
  Misses          7        7

Impacted Files
Coverage Δ

test/unit/NotifierBuilderTests.cpp
96.35% <100%> (+0.15%)
: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 ccbeb92...888bb8e. Read the comment docs.


Propagate DECIDE messages from the previous instance

fixes #579

The fuzzer now appears to be hanging pretty consistently. Not sure what's up with that but I'll try to debug it a bit.

The fuzzer now appears to be hanging pretty consistently.

I suspect it's the runners. I'll switch to IPDX runners with extended fuzz time to investigate further.

These changes make no difference to the operation of the core GPBFT protocol, because on RecieveMessage all messages from previous instances are dropped at GPPBFT layer

Yes.

Fuzzing is failing on 'main' as well.

At least on my machine...


Refactor of BitcoinTestFramework main function.

In order to wrap the BitcoinTestFramework class in a user-defined class, which can be started and shutdown, the code in BitcoinTestFramework main has been encapsulated in respective functions. Additionally, test-run specific state in temp-directory and NetworkThread is now reset after shutdown.
Behavioural changes:
Potential exceptions raised from any code in setup section of the code are now handled. Previously, only exceptions raised by code beginning here was handled.
TODO's:

[ ] Refactor NetworkThread class in mininode.py, so each BitcoinTestFramework object initialises a different network thread.
[x] Documentation (markdown file) describing how to run an interactive test with TestWrapper.

You should also reword the second commit log to remove reference to TestWrapper if you want to get this merged to Bitcoin Core.

You should also reword the second commit log to remove reference to TestWrapper if you want to get this merged to Bitcoin Core.

Thinking about this some more, perhaps a good way to get this merged to Bitcoin Core would be to include the TestWrapper and add some documentation about how to run an interactive test. That's quite a compelling rationale for making these changes.
I'd hold off on that while we're still iterating on TestWrapper, but after the workshops let's open a PR against https://github.com/bitcoin/bitcoin.

I am adding another TODO for this PR.

[ ] Refactor NetworkThread class in mininode.py, so each BitcoinTestFramework object initialises a different network thread name.

Refactor NetworkThread class in mininode.py, so each BitcoinTestFramework object initialises a different network thread.

I don't think we want more than one instance of BitcoinTestFramework to exist at a time. @jachiang - do you have a reason for wanting this?

@jnewbery Sorry this took a while, but I've addressed all feedback, reworked commits for easier review and added a documentation file for the TestWrapper class.
I will review again tomorrow and then open a PR against master.
A very big thank you for your time spent reviewing and improving this!

PR opened against bitcoin/bitcoin here: https://github.com/bitcoin/bitcoin/pull/17288


Nanoflann v1.5.5

Nanoflann v1.5.5

Hello @d-vo, modules you maintain (nanoflann) have been updated in this PR. Please review the changes.


Kubernetes API server unreachable from some containers

Rancher Versions:
Server: 1.5.3
healthcheck: 0.2.3
ipsec: 0.9.4
network-services: 0.9.1
scheduler: k8s:v1.5.4-rancher1-3
kubernetes (if applicable): 1.5.4-rancher1
Docker Version:
1.12.6
OS and where are the hosts located? (cloud, bare metal, etc):
CentOS 7.2 on KVM
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:

Run Rancher server
Create Kubernetes environment
Add host (run Rancher agent on the same node as Rancher server)

All the steps are done via Ansible scripts using Rancher's V2 API.
Results:
Kubernetes Dashboard does not launch for some time, container repeatedly starts and fails with error:
4/19/2017 9:08:04 AMUsing HTTP port: 9090
4/19/2017 9:08:04 AMCreating API server client for https://<IP_ADDRESS>:443
4/19/2017 9:08:34 AMError while initializing connection to Kubernetes apiserver. This most likely means that the cluster is misconfigured (e.g., it has invalid apiserver certificates or service accounts configuration) or the --apiserver-host param points to a server that does not exist. Reason: Get https://<IP_ADDRESS>:443/version: dial tcp <IP_ADDRESS>:443: i/o timeout
4/19/2017 9:08:34 AMRefer to the troubleshooting guide for more information: https://github.com/kubernetes/dashboard/blob/master/docs/user-guide/troubleshooting.md

I have checked all the things suggested on Kubernetes debug service and K8s API server seems to be setup properly, i.e. service is deployed, endpoint for the service exists, iptables rules seem to be correct. API server responds when I query it from host using either its cluster/service IP or endpoint IP. However, from within containers (which don't have network mode set to host) API server is reachable only via endpoint IP, querying it using its cluster IP results in timeout (like in case of Kubernetes Dashboard above).
What's even more puzzling is that after some time, like 10-15 minutes, API server starts responding on the cluster IP and Dashboard kicks in.
I've checked logs of lb-controller, kubernetes agent, kube proxy, etc but I haven't found anything suspicious, but since I started using Rancher quite recently I might be missing something.

Same happened to me a couple of times, but I could not precise what happened. Do you have any update on this? I have created new environments from scratch and in some of them I did not observer this behavior and some yes.

No, no update from my side. The only thing that comes to my mind is that since API server starts responding on cluster IP after some time maybe there is some cache in one of the Rancher's component involved.

I'm having the same issue with 1.5.4-rancher-1-2

The issue seems to be fixed in rancher 1.5.4. Though, I am not sure which bugfix/change resolved the issue.


Add transform for ExternalFunctionBodyNode

Codecov Report

Merging #23393 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #23393   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 9c2f479...0a9e461. Read the comment docs.


tv-casting-app: Fixing issues passing uniqueIDs and Video player caching

Fixes https://github.com/project-chip/connectedhomeip/issues/24744
Change summary

Sometimes a corrupted uniqueId gets set on the SDK from the android tv-casting-app. To fix this, we are setting/passing copies of the uniqueId in the get/setRotatingDeviceIdUniqueId functions to avoid potential tampering with the data from elsewhere in the app and allocating the jniRotatingDeviceIdUniqueIdByteArray buffer in ConversionUtils.cpp on the heap instead of the stack.
In this PR, we also fix an issue with the caching of the video players where only one video player was being written to the cache.

Testing
Tested with the android tv-casting-app running with the linux tv-app

@sharadb-amazon could you shed some light on Sometimes a corrupted uniqueId gets set on the SDK ? It seems somewhat vague ... what creates a corrupted unique ID? Is this a software bug, malicious/testing operations, something else?

@sharadb-amazon could you shed some light on Sometimes a corrupted uniqueId gets set on the SDK ? It seems somewhat vague ... what creates a corrupted unique ID? Is this a software bug, malicious/testing operations, something else?

This is a software bug, it occurred because we created jniRotatingDeviceIdUniqueIdByteArray on the stack in ConversionUtils.cpp, set it in the outParams and then the stack unwound. This will sometimes lead to the case where the memory allocated gets overwritten. In the fix, we allocate the jniRotatingDeviceIdUniqueIdByteArray on the heap instead.


Vineyard integration with data processing engines

Describe your problem
The Apache ecosystem has developed many matural big data processing engine, and vineyard may could bring value via tight integration:
Workflow Engine

airflow integration: #374.

PyData

ML frameworks: #280

mars

dask: #412

SQL & DataFrame

apache/data-fusion: vineyard works as a kind of data source https://github.com/apache/arrow-datafusion/tree/master/datafusion/src/datasource

requires a rust client (and sdk)

apache/presto: a vineyard connector https://prestodb.io/docs/current/connector/localfile.html

requires java client

shared memory layer for spark:

http://events17.linuxfoundation.org/sites/events/files/slides/shared-memory-layer-for-spark.pdf
https://ignite.apache.org/docs/latest/extensions-and-integrations/ignite-for-spark/overview

apache hive:

https://medium.datadriveninvestor.com/hadoop-integration-with-apache-ignite-and-using-hive-with-ignitemr-db8b8c09110e

Additional context
Add any other context about the problem here.

The integration with bigdata framework depends on Java SDK (#227).

Close as fixed.
Will open new tickets when we meet new integration requirements.


[Bug]: ATCH ERROR: NOK: Transaction not found with hash:

Version

[X] Yes

General Question

[X] No

What happened?
A bug happened! I receive the following error when I run npm run get_crontract:
CATCH ERROR: NOK: Transaction not found with hash: 0x02369b3da8e493278ca4efe59dca3fa98c1b1f10818d141f7efc5dc33395e380
Please help!
What have you tried?
I Googled and asked the community in the discord server but could not find a solution.
Relevant log output
No response
Protect your information

[X] I Agree

I have the same issue. Can someone pls help?


Refactor Python Table and TableLike

This PR refactors the Table object and TableLike in the Python UDF.
TableLike
TableLike is now defined as a TypeVar, which can be viewed as a collection of Tuple instances. The following types are considered as valid TableLike:

Table
pandas.DataFrame
List[TupleLike]

List[Tuple]
List[Dict[str, Field]]

Table
To support easier usage, Table still inherits pandas.DataFrame.
We provide multiple conversion methods to help convert other TableLike instances into a Table. We also add converters between TableLike and Table to ease the UDF code writing. With those converters, this PR also fixes an issue #2068 that was caused by previous improper conversions from a Table to a collection of Tuples.

Looks okay. It would be better if you mention in the description that you also refactored Tuple and TupleLike.

There are some cleanings needed (remove some unused code and comments). The Tuple and TupleLike has no actual difference, only syntax and type hint changes.


Keybindings Gone

After using it for a day and restarting the program and my macbook, the keybindings have gone away. Even when adding it into the keymap.cson file it still doesn't work.

Fix
Fixed it myself, in the Settings, select Core and remove Script from Packages With Keymaps Disabled.


Add configurable colors for Forestry events.

Customizable Colors for Forestry events
Current Behavior

The Woodcutting plugin primarily uses a bright green color for most events.

Proposed Changes

Introduce an option for users to configure their own colors for various Forestry events
This feature enhances user experience by allowing personalization and supports better accessibility

Example
Below is an example of how the color customization interface looks within the plugin settings

Yes please, good for us colorblind users.

I think this is a really smart idea. The colours enhance the playability and promote a much more fulfilling user-end experience.


not paired

When I test the following HTML
1/<wbr>2

HTMLHint shows this message:
Tag must be paired, missing: [ </wbr> ], start tag match failed [ <wbr> ] on line 8.

According to MDN usage context:
Tag omission - It is an empty element; it must have a start tag, but must not have an end tag.

This is pretty much the same issue as #110.

Can you show me the browser name and version whol support this tag?
I test failed with Chrome 49 and Firefox 44:
http://www.w3schools.com/tags/tryit.asp?filename=tryhtml5_wbr


release pretrained model

Hi,
We have released the pretrained model at:

News: https://huggingface.co/yyu/news_contrastive_pretrain
Wiki: https://huggingface.co/yyu/wiki_contrastive_pretrain
Review: https://huggingface.co/yyu/review_contrastive_pretrain

Best,
Yue


allow child onTap to work

If a child item has onTap events they wouldn't trigger even if the CarouselView has on onTap: null
If I do not supply onTap for the CarouselView widget it should at least not block onTap events for my child widgets...
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Running into the issues this PR would address myself too. Disappointing that this isn't the default behavior for CarouselView

We have not heard back for a while, and it looks like https://github.com/flutter/flutter/pull/154745 will resolve this issue, so I am going to close it for now. If that PR lands and you would like to revisit this, please open a new PR. Thanks!


Fix for non existing Daikin zones

Description:
This fixes non existing Daikin zones so they will not show up as switches.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly (example).
[x] New dependencies have been added to requirements in the manifest (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Bug fix for an bug introduced in #23792

#23792 is this PR. Is there an issue we should close?

#23792 is this PR. Is there an issue we should close?

I ment #23113, that PR didn't really fix the problem, this does and have been verified by the users with AirBase units.


Support KMS Alias for EBS worker node encryption

Tell us about your request
At the moment the block device mapping supports only kmsKeyID which is of format, e.g.
kmsKeyID:: arn:aws:kms:eu-west-1:***********:key/2bcf2845-9c4a-4fh3-a2c7-gf0bc912d204

it would be nice to support the alias name for KMS alternatively, e.g.
kmsKeyAlis: alias/eks/ebs

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
We do re-provision often the EKS clusters in environments and would like to avoid changing the Key ID every time in IaC repo. It's inconvenient. While KMX key ID changes the alias is persistent.
Are you currently working around this issue?
Just updating the provisioner config in repo every time key ID is changing.
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

im working on it

would like to avoid changing the Key ID every time in IaC repo

Does the default launch template configuration support this? I'm assuming not since none of the docs indicate that it does. If not, we could just check if the passed-in value looks like an ARN. If it doesn't, then we can build an ARN based on the known cluster information.

This would be a good feature to have - specifying the arn in IaC is troublesome and cause issues :)

Team, this could be a real time saver in terms of running automation, where users have generic KMS key alias across all regions and allow us to have specific key Alias instead of KeyID per region.

We do re-provision often the EKS clusters in environments and would like to avoid changing the Key ID every time in IaC repo. It's inconvenient

Does having an alias here still require you to get permission on the KMS key for the controller role to be able to provision the EBS volume? If it does, I'm assuming the policy would reference the KMS key by ARN anyways, or are you just relatively wide-open when it comes to allowing access to KMS keys to encrypt volumes in your policy?

would like to avoid changing the Key ID every time in IaC repo

Does the default launch template configuration support this? I'm assuming not since none of the docs indicate that it does. If not, we could just check if the passed-in value looks like an ARN. If it doesn't, then we can build an ARN based on the known cluster information.

I've spent a few days working on this Issue, based on the AWS doco from LaunchTemplate that clearly specifies receiving a "KMS Key ARN", nothing about an alias.
I've changed the karpenter code to allow an alias, created some tests, tested the whole thing locally and on EKS, until I test again using Karpenter's main branch (without my changes), only to find out that AWS does support passing an alias instead..
has anybody else tested this to confirm the original issue is real? if not we could close this issue I think. But I'd like to double-check my testing is accurate now haha.
I'll keep my patch handy just in case.

@fedecz : I have tried the full ARN of the alias, the string alias/${key_alias} - none of that worked. Only the key ID seems to be supported. I think the issue remains relevant.
My use case is the following: I have a CI cluster. In the cluster I have N worker pools, each pool gets its own node pool with specific setup. I use the KMS ack to create a KMS key and set an alias to it. In the node class used for my worker pool, I want to be able to reference the alias of my key as I can't predict the KMS key ID nor do I want to deploy my worker pools in N manual steps.


Can't differentiate Tree Gnome Stronghold/Village

Noticed a dragon imp spawn and its location was cut off in the side panel showing "Tree Gnome...". But it could be either the stronghold or the village according to this: https://github.com/Hablapatabla/ImplingFinder/blob/master/src/main/java/com/hablapatabla/implingfinder/model/ImplingFinderRegion.java#L69-L70
Ideally the location shouldn't be cut off or have a way to hover over to see the full name of the location, but a quick fix could be to just change "Tree Gnome Stronghold" to just "Gnome Stronghold" like how it's listed in the spirit tree teleportation menu in-game.

thanks. made the change. for anyone who may read this, clicking the specific impling should bring up a marker in your world map so there is no ambiguity.

Ah I didn't know that was a thing! I think I tried double clicking to see if anything would happen but I probably didn't notice the flicker when I was doing that with the world map open. In any case though, thanks for the quick fix!


How do we handle the mailer config

This is something I have to set up every time, but restricting usage to SendGrid seems silly. Need to think this one through.

Perhaps we could add the creds to the initializer and then set the mailer in the gem, with an option to use mailcatcher unless in production?


add mx.edu.conalepmex.txt

Colegio Nacional de Educación Profesional Técnica

@JosepAntonie I am sorry, the domain conalepmex.edu.mx is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Implement function to target schema by name

What do you want to achieve?
Now, every schema is fetched or addressed by id. The data service recently implemented a function to target them by name.
This should also be possible in the SDK

Going to be fixed by: https://github.com/ExtraHorizon/javascript-sdk/issues/713


Add SDK example for Service Invocation that shows DaprClient.CreateInvokeHttpClient();

Currently there are HTTP examples for service invocation, that show httpclient here https://github.com/dapr/quickstarts/blob/master/service_invocation/csharp/http/checkout/Program.cs
However the preferred way of using this via the SDK using DaprClient.CreateInvokeHttpClient(); is not covered and this should be where we direct users. It is described here https://learn.microsoft.com/en-us/dotnet/architecture/dapr-for-net-developers/service-invocation#invoke-http-services-using-httpclient
The docs in .NET also need updating here https://docs.dapr.io/developing-applications/sdks/dotnet/dotnet-client/ to show this method for both HTTP and SDK for Service Invocation.

/assign

Close this?
I think both links have been addressed.

Agree


Generic oracle: errors in log files

During the testing session, TESTAR could poll new lines from log files and take into account standard log file format (for example: web apps, Win apps, Android apps) or regexp and identify if errors were raised.
FYI @tatu-aalto, @sorin-patrasoiu

There is a related pull request from Sietse #302

The new LogOracle implementation #302  allows users to specify suspicious regular expressions that should be searched for in specific log files. Currently, the user needs to indicate the static log file name in the TESTAR settings.
To improve this feature, we will track into #271 the extension of this implementation to additionally track the generation of dynamic log files in specific folders.


Unhandled Exception: Concurrent modification during iteration

I am new to Flutter and I am trying to use this dependency to record the screen.
The following exception is being thrown after calling the export function:
[VERBOSE-2:ui_dart_state.cc(198)] Unhandled Exception: Concurrent modification during iteration: Instance(length:7) of '_GrowableList'.
#0      ListIterator.moveNext (dart:_internal/iterable.dart:336:7)
#1      ScreenRecorderController.export (package:screen_recorder/src/screen_recorder.dart:111:25)
<asynchronous suspension>

The code to reproduce it:
  @override
  Widget build(BuildContext context) {
    ...
    return IconButton(
      icon: IconBuilder.get(...) :
      onPressed: () async {
        if (widget.recordController == null) {
          gameState.stop();
          return;
        }

        if (gameState.isPlaying) {
          gameState.stop();
          var gif = await widget.recordController!.export();

          showDialog(
            context: context,
            builder: (context) {
              return AlertDialog(
                content: Image.memory(Uint8List.fromList(gif!)),
              );
            },
          );

          return;
        }

        gameState.play();
        widget.recordController!.start();
      },
    );
  }

Any suggestion?
Many thanks

Nevermind. Missed the call to: widget.recordController!.stop();


(enh) CodeActAgent: improve logging; sensible retry defaults in config

Short description of the problem this fixes or functionality that this introduces. This may be used for the CHANGELOG
Improve logging in case of error and more sensible error retry values.

Give a summary of what the PR does, explaining any non-trivial design decisions

in case of an LLM error, actually log the error to the UI as well
LLMConfig class: changed default retry values to more sensible, lower values after testing to prevent ridiculously long wait times
improved descriptions in the config template and docs

I've tweaked the values to these now:
num_retries = 8
retry_min_wait = 15
retry_max_wait = 120
retry_multiplier = 2.0

Also changed docs and config template accordingly.

@xingyaoww @enyst

I had to apply an urgent fix to format_messages in Message class that caused an empty message (and thus errors with LLM's).
refactored a bit the tenacity @retry decorator to reduce duplicate code
custom wait method to especially raise immediately the wait time for RateLimitError exceptions
2 messages the wording was rather unhelpful

Please have another look, thanks. 🤗


onboarding no longer working on ipad

after entering email details, modal window for web view opens, spins for a moment and then dismisses itself

They might have changed the routes that onboarding uses without us knowing.

new onboarding :) no longer relevant


Issue with k8s.io/docs/setup/production-environment/container-runtimes/

https://github.com/kubernetes/website/issues/22591
/language zh
/assign

done.
/close


Ticket/2.6.x/8312 zypper latest

Previously the regular expression to match the correct column from
'zypper list-updates' was wrong, it seems to have been based on the command
'zypper packages' instead. This was caused ensure => 'latest' to fail as the
provider couldn't adequately figure out what newer versions were actually
availabe.
So I've fixed the regular expression (based on Felix Frank's patch) and updated
the spec test so that it uses the real output from zypper list-updates and now
references an external spec file, as apposed to referencing the content inline.

@kbarber This patch looks good, but I thought 2.6.x is closed except for security fixes?  Have you talked to Matthaus or Mike about this change?

Okay - I figured that was the case @jeffmccune let me re-open against 2.7.x.


[DO NOT MERGE] Proof of concept for including organisation pages in an organisation search

When filtering by organisation, we expect to see the organisation page itself in the results.
Right now, we achieve this by automatically adding the organisation page to its
own organisations field - i.e. tagging the page to itself.
Publishing api doesn't know about this tagging, and we need to make the two
consistent. This PR is one possible way of doing this without changing the behaviour
in rummager. The alternative is to have whitehall send organisation self-tags into
publishing api.
This isn't intended to be mergeable, I just want to see if this is an approach is worth continuing with. Thoughts?
Further discussion on https://trello.com/c/aOTMr3WJ/640-whitehall-organisations-tagged-to-itself
cc @alphagov/team-finding-things

Yeah, this would work. I wonder if we could make this more explicit and keep the filter API clean (= only return things that are physically tagged to the organisation).
I think there are three options:

Explicitly enable this behaviour with a flag (include_organisation_itself=true or something). Search query would be:

https://www.gov.uk/api/search.json?filter_organisations[]=hmrc&include_organisation_itself=true

Add a special search case for organisation-scoped search and add behaviour based on that:

https://www.gov.uk/api/search.json?mode=organisation_scope&filter_organisations[]=hmrc
In time we could add more special-behaviour for this kind of search.

Add a parameter that force-includes a document into the search results:

https://www.gov.uk/api/search.json?filter_organisations[]=hmrc&force_include_results[]=https://www.gov.uk/government/organisations/hm-revenue-customs

@MatMoore this looks sensible. Certainly no worse than our other option where we mess with the data in publishing-api.
My order of preference for @tijmenb 's options is essentially in order of level of detail of abstraction.
2, then 1. I think we should avoid 3.

Why not 3?

I think option 3 exposes too much control over the details of rummager to the client, without actually adding much benefit.
With 3, the functionality is not explicitly tied to any particular use case or abstraction, which will make it harder to maintain in future.
If there is a shortish list of urls that the client believes belong in the results, it could just add them in itself.

Discussed with @tijmenb offline.
We're going to keep this logic in rummager, but do it at indexing time instead, i.e. add to the organisations array when indexing organisations.
This solves the immediate problem: publishing api and rummager are sent different data, which blocks us from moving indexing responsibility to the publishing api.
This means we will defer dealing with the underlying inconsistency until we have a clearer idea of what we want we want to do with organisation tags. For the migration work we will need to add a whitelist entry for this.


merge header values in AbstractMessage case insensitively

This code can be reached when an HTTP response is received containing appearance of same header with differing cases. (Believe it or not, I encountered this in a third party production system from Microsoft; presumably headers being added to their responses by multiple layers of disparate code.) Here is a sample of an HTTP response that surfaces the bug:
Content-Type: text/plain
Set-Cookie: foo=bar
SET-COOKIE: baz=1

Expected result should be header collection like:
[
  'Content-Type' => 'text/plain',
  'Set-Cookie' => ['foo=bar', 'baz=1'],
]

Checking out the latest Guzzle 5.3, I'm not seeing the same issue:
<?php

require 'vendor/autoload.php';

use GuzzleHttp\Message\Response;

$r = new Response(200, [
    'Set-Cookie' => ['bar', 'baz'],
    'SET-COOKIE' => ['bam', 'qux']
]);

echo $r;

Outputs:
HTTP/1.1 200 OK
SET-COOKIE: bar, baz, bam, qux

Closing. Feel free to reopen if you can provide more details.