Unit-test: add necessary ownerRef to the pod.

There are cases where Pods have not had ownerReferences added, leading to certain test scenarios not yielding the expected outcomes. For instance, in the scenario "case: without priorities, but excluding namespaces," the reason pod1- pod5 were not evicted was due to "the Pod does not have any ownerReferences," as opposed to a namespace mismatch, because these Pods were not configured with ownerReferences.
I0621 19:51:50.729704   15876 defaultevictor.go:238] "Pod fails the following checks" pod="kube-system/p8" checks="[pod does not have any ownerRefs, pod has system critical priority]"
I0621 19:51:50.729715   15876 defaultevictor.go:238] "Pod fails the following checks" pod="namespace1/p1" checks="pod does not have any ownerRefs"
I0621 19:51:50.729722   15876 defaultevictor.go:238] "Pod fails the following checks" pod="namespace1/p2" checks="pod does not have any ownerRefs"
I0621 19:51:50.729729   15876 defaultevictor.go:238] "Pod fails the following checks" pod="namespace1/p3" checks="pod does not have any ownerRefs"
I0621 19:51:50.729736   15876 defaultevictor.go:238] "Pod fails the following checks" pod="namespace1/p4" checks="pod does not have any ownerRefs"
I0621 19:51:50.729757   15876 defaultevictor.go:238] "Pod fails the following checks" pod="namespace1/p5" checks="pod does not have any ownerRefs"

And I haven't fixed "case:  without priorities, but include only default namespace"， because LowNodeUtilization does not support inclusion.   I will proceed with this work when inclusion is supported.
https://github.com/kubernetes-sigs/descheduler/blob/cdbd101eaecb97c56b5a520078528e86d736e9ea/pkg/framework/plugins/nodeutilization/lownodeutilization_test.go#L227-L251
BTW,  why LowNodeUtilization does not support  namespace inclusion？

/ok-to-test


Dunst not showing notifications

i have it in my exec-once. I dont have any other notif daemon running WARNING: Setting always_run_scripts in section global doesn't exist CRITICAL: [dbus_cb_name_lost:1044] Cannot acquire 'org.freedesktop.Notifications': Name is acquired by 'dunst' with PID '34974'.. It works on bspwm, but for some reason its not working here

that's obviously an issue with your system...? dunst works for everyone else. Seems like you have dunst already running before your start hyprland, which you should not.

at any rate, this is a dbus error, and hyprland has nothing to do with that. It clearly states it's acquired by dunst with pid 34974

Even if you don't start dunst dbus will start it on first notification. You can check it - remove it from hypr config and do notify-send 123.


Switch search-admin to ARM64 in all envs

This has worked a treat in integration 🎉

I don't think we have the ARM node pool in staging/prod yet but I think there's no reason we can't turn it up at this point? cc @dj-maisy

I don't think we have the ARM node pool in staging/prod yet but I think there's no reason we can't turn it up at this point? cc @dj-maisy

Indeed - so far, I don't think we've encountered any issues in Integration so I'm happy to start adding ARM capability to our other environments.
...but that's something we're going to have to do before we approve/merge this!

I'm going to close this for now until we have ARM node's available in other environments.


[Technical Blog] - Launching Apache DevLake (Incubating) on an AWS ec2-instance

Quick Pointers

Launching an AWS instance
Setting up an SSH connection for both Linux and Windows User
Installing DevLake on the ec2- Instance
Configure the Inbound Rule of AWS
Final Changes on IP and Launch the containers.

Reference Issue #1

Hey @shubham-cmyk, could you rename this to "12-Shubham-DevLake-on-AWS-ec2.docs/md"?

https://shubham-2001.medium.com/launching-apache-devlake-incubating-on-an-aws-instance-1a9ada1ecc86
@shubham-cmyk add hyperlinks to wherever Apache Devlake is mentioned

@JoshuaPoddoku  Hyperlink directing to devlake github repo or our apache devlake website ?
I could edit in Medium and google doc. which I have submitted.

DevLake website should be good!


Search: Exclude setting is not being respected in my workspace.

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version:
Version: 1.82.2 (user setup)
Commit: abd2f3db4bdb28f9e95536dfa84d8479f1eb312d
Date: 2023-09-14T05:55:25.390Z
Electron: 25.8.1
ElectronBuildId: 23779380
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

OS Version:
Windows 10 Pro x64 Version 22H2 (Build 19045.3448)

Steps to Reproduce:

Add workspace folders to the Search: Exclude setting:

Perform a search and notice that search results are still returning results from the excluded folders:

Just to add some additional information, this is my workspace settings file:

Am I doing something wrong? Why isn't this working?

There currently isn't any way of excluding a whole workspace root in a multi-root workspace from those settings. You'd need to use the files to exclude field in the search widget and exclude it like ./00 External/*. Adding the ability to exclude in search.exclude is tracked here https://github.com/microsoft/vscode/issues/50638
/duplicate https://github.com/microsoft/vscode/issues/50638


Ability to enable HTTP Strict Transport Security

In the SSL module you can configure the whole site to be strictly under SSL and redirect all non-HTTPS requests to their HTTPS counterparts (which is mostly what you'd like to do). This leaves a possible man-in-the-middle attack open for the initial unencrypted HTTP requests (as an attacker can serve a forged site to the user) so users won't even get to the HTTPS URL if they didn't already typed that into the browser bar.
While we can't do much (AFAIK) if this is the first time the client accesses the site if they already visited the HTTPS site once we can basically tell them to only use HTTPS in the future. This is what HTTP Strict Transport Security is about: by setting a header value browsers can be instructed never to even try HTTP (see: https://developer.mozilla.org/en-US/docs/Web/Security/HTTP_strict_transport_security).
We could add the feature to the SSL module to send this header (with a configurable expiration time and a checkbox to send the preload parameter).

Another article I like about it: http://www.troyhunt.com/2015/06/understanding-http-strict-transport.html
It doesn't hurt to have that. I don't think it should be enabled by default though.

I think we could create these settings in the SSL module as @Piedone said. After we tick the "Force SSL on all pages" the HSTS settings should appear. The "Expiration time" should be a textbox (with validation), and the "Preload" should be a checkbox.
I'm not sure about this, but the NWebsec.Owin package(https://docs.nwebsec.com/en/4.2/nwebsec/Configuring-hsts.html) seems to be good for the implementation.
Is this a good idea?
@sebastienros

I'd be hesitant to add a NuGet package (that also has another 3rd party dependency) just to add a few HTTP headers. Also I'm not sure whether the below style of configuration works from an IOwinMiddlewareProvider.
public void Configuration(IAppBuilder app)
{
    app.UseHsts(options => options.MaxAge(days:30).IncludeSubdomains());
}


Bug report

Microsoft PowerToys version
"0.56.1"
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
PowerToysReport_2022-03-06-01-04-11.zip
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

Is there any error prompted? Please attach a screenshot if available. /needinfo


Use async with pusher connects

Summary by CodeRabbit

New Features

Enhanced WebSocket management for improved audio and transcript data handling.
Improved error handling for WebSocket disconnections and data processing.

Bug Fixes

Refined logic for sending data over WebSocket to ensure connections are active.

Chores

Added user ID (uid) logging for better debugging during connection attempts.

Issue: #1261


⚠ GitLab Server has degraded performance

In d5b8c3b, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 14205 ms

Resolved: GitLab Server performance has improved in 834d43c.


Verify user arguments

In #167, we added options to allow users to pass arguments to Meson, but that implementation allows them to pass options that may conflict with ours, which may mess things up (eg. overriding --prefix).
While I think we should try to detect conflicting options and either error out or issue a warning, which is tricky to implement, most issues I think can be mitigated by disallowing arguments that don't start with -, I checked and there's no positional argument we'd want users to be able to specify, and putting our options last in the call, other than just defaults that we want the user to be able to override. I think this was a big overlook from my part when we merged that PR, probably because I was meaning to do the argument validation.

If we move away from having to run meson install (see #279 and #280) we don't need to pass the --prefix option (or the --python.purelibdirand--python.platlibdiroptions) tomeson setupbecause we don't care anymore about the real installation paths, only about the installation paths relative to the placeholder directories inintro-install_plan.json`.
The remaining command line arguments are: $source_dir, $build_dir, --native-file=$native_file, -Ddebug=false, -Doptimization=2, $user_args. I think this ordering is the right one because it allows the user to override the default -Ddebugand-Doptimization(and in the near future-Db_ndebug) options. I would need to check what happens when more than one --native-fileargument are present. It would be cool if Meson could merge the content of the two. If he does not, things are most likely not going to work very well if the user adds their own--native-file` argument.
meson setup errors out if more than two positional command line arguments (arguments not starting with a -) are specified:
$ meson setup . _build foo
usage: meson [-h] {setup,configure,dist,install,introspect,init,test,wrap,subprojects,rewrite,compile,devenv,env2mfile,help} ...
meson: error: unrecognized arguments: foo

Therefore, the command line validation is already performed and I don't think meson-python needs to adds its own validation on top.
The only open question is about the --native-file argument. If Meson does not try to merge the two native files, the easy fix would be disallow passing this option via meson-python. The more elaborate fix would be to detect it and merge the user provided native file with the meson-python generated one.

I've verified that Meson merges multiple native files. The only thing to decide is whether we want the user --native-file option to take precedence over the one generated by meson-python or vice versa.

If we move away from having to run meson install (see #279 and #280)

We run meson install because binaries in the build directory have a RPATH set, so that you can use them from there.
$ ls builddir/
build.ninja  compile_commands.json  example.cpython-310-x86_64-linux-gnu.so*  example.cpython-310-x86_64-linux-gnu.so.p/  libexample.so*  libexample.so.p/  meson-info/  meson-logs/  meson-private/  mesonpy-wheel-libs/
$ patchelf --print-rpath builddir/example.cpython-310-x86_64-linux-gnu.so
$ORIGIN/

While this is okay for editable wheels, we must keep meson install when building wheels.

we don't need to pass the --prefix option (or the --python.purelibdir and --python.platlibdir options) to meson setup because we don't care anymore about the real installation paths, only about the installation paths relative to the placeholder directories in intro-install_plan.json.

That is needed by the heuristics, not because we run meson install. Once we drop the heuristics, we can drop those.

I've verified that Meson merges multiple native files. The only thing to decide is whether we want the user --native-file option to take precedence over the one generated by meson-python or vice versa.

We don't, ours should take precedence, as it is used to set the Python interpreter, and the PEP 517 hooks are required to build packages for the current interpreter.

We run meson install because binaries in the build directory have a RPATH set, so that you can use them from there.

Don't we adjust the RPATH anyway when building the wheel because the included libraries are relocated?

We don't, ours should take precedence, as it is only used to set the Python interpreter, and the PEP 517 hooks are required to build packages for the current interpreter.

Ok, we just need to order the command line arguments in the right way.

With #381 merged, there aren't arguments that the user may pass to Meson that conflict with the ones passed by meson-python in a way that may prevent meson-python to work properly. Even passing --prefix does not have any visible effect. The only way I know to interfere with the meson-python assumptions is to pass a --native-file specifying a different path for the python interpreter. This is taken care of in #384.


cffi: fix linger in Socket.close() and Context.destroy()

This changeset fixes bugs regarding how 'linger' was handled in the cffi
backend. Prior to this fix, running the following example would exit
immediately (as expected) when run under the cython backend, but would hang
under the cffi backend (PYZMQ_BACKEND=cffi).
import zmq

ctx = zmq.Context()
s = ctx.socket(zmq.REQ)
s.connect('tcp://localhost:34567')
s.send('test')

s.close(linger=0)      # <- cffi backend hangs here...
ctx.destroy(linger=0)  # <- ... and here

Context.destroy() did not correctly handle the case where linger=0, in which
case linger evaluates to False. The fix is to check specifically for None, not
for any non-True value.
Socket.close() was completely ignoring the 'linger' argument.

There are a couple of open issues which may be related to this -- #837, possibly also #845? But there are many variables (what backend is being used? where exactly is the hang occurring? etc.), and it's hard to tell whether they are the same issue or not...
The truth is that I was having some issues with linger, and when starting to dig into the code I came across the above bugs, which are quite obviously bugs; however, I don't think they're really related to the problems I was having, 'cause I doubt that I was using the cffi backend! However, the minimal test-case I used to reproduce this (the one included in the commit message) seems to be working for me with the cython backend, so at the moment I can't point to a specific problem with the cython backend...

@dovf thanks! I didn't think to check which implementation people were using.


Where to enlarge batch size?(each GPU has more than 32G)

Hi, thank for your nice work!
I know the batch size is now only one for each GPU. How to make full use of these GPUs:

Thx!

https://github.com/tensorpack/tensorpack/blob/2edcf101c173bd9b36a834d373d19e195cf87276/examples/FasterRCNN/NOTES.md

It seems tensorpack is not quite mature now.


AttributeError with streamline_features

Hi, as shown below, the streamline_features-related code from the data_loaders tutorial works well when I cloned the main branch.
ds.streamline_features(match_to_release="104", subset_genes_to_type="protein_coding")  # Choose a reference genome and subset to only protein-coding genes
ds.streamline_metadata(schema="sfaira")  # make sure the metadata annotation of all datasets are in line with the sfaira schema, so they can be cleanly concatenated in the next step
print(ds.adata) # Use the adata object for your analysis or modelling

While following the store_cart_generator tutorial, I came across KeyError: 'ethnicity' with the following code:
cache_path = os.path.join(".", "data")
dsg = sfaira.data.dataloaders.databases.DatasetSuperGroupDatabases(data_path=cache_path, cache_metadata=True)

To avoid the ethnicity issue I switched to the dev branch.
However, another error AttributeError: 'Dataset' object has no attribute 'streamline_features' appeared when I tried to create store with the code below following the store_cart_generator tutorial:
for k, ds in dsg.datasets.items():
    if ds.adata is None:
        ds.load(load_raw=False, allow_caching=True)
    ds.streamline_features(
        remove_gene_version=True,
        match_to_release={"Homo sapiens": "104", "Mus musculus": "104"},
        subset_genes_to_type="protein_coding"
    )
    ds.streamline_metadata(
        schema="sfaira", clean_obs=True, clean_var=True, clean_uns=True, clean_obs_names=True
    )
    ds.write_distributed_store(
        dir_cache=store_path,
        store_format='dao',
        dense=True,
        chunks=128,
        compression_kwargs={"compressor": "default", "overwrite": True, "order": "C"}
    )

Similarly, I got the error AttributeError: 'Universe' object has no attribute 'streamline_features' with the aforementioned code from the data_loaders tutorial, which I did not have before with the main branch.
I'd appreciate if you could kindly help with this issue.
Thanks,
Yiqi

Hi!
On the dev branch, please use:
ds.streamline_var(match_to_release={"Homo sapiens": "104", "Mus musculus": "104"}, subset_genes_to_type="protein_coding")
ds.streamline_obs_uns()

Thank you!


Status field for most/all spec fields

Is your feature request related to a problem?
Currently we do not have status fields for all the spec fields. This can be problematic as one might not be able to get the required server side information using k8 APIs.
For example aws elasticache describe-replication-groups provides many information which would be beneficial to be in status.
Encryption
"TransitEncryptionEnabled": true,
 "AtRestEncryptionEnabled": true,

Log delivery configuration
"LogDeliveryConfigurations": [
    {
        "LogType": "slow-log",
        "DestinationType": "cloudwatch-logs",
        "DestinationDetails": {
            "CloudWatchLogsDetails": {
                "LogGroup": "lognew"
            }
        },
        "LogFormat": "json",
        "Status": "active"
    }
]

Describe the solution you'd like
Status fields for most/all spec fields
Describe alternatives you've considered
Only add very critical server side information in status

+1
We also had to add a workaround for Endpoint resource in SageMaker  -
https://github.com/aws-controllers-k8s/sagemaker-controller/blob/main/pkg/resource/endpoint/custom_set_output.go#L64
EndpointConfigName is already part of Spec but the customer might want to look at service side value

@nmvk all you should need to do is add the following to the generator.yaml file for elasticache-controller:
resources:
  ReplicationGroup:
    fields:
      TransitEncryptionEnabled:
        from:
          operation: DescribeReplicationGroups
          path: TransitEncryptionEnabled
        is_read_only: true
      AtRestEncryptionEnabled:
        from:
          operation: DescribeReplicationGroups
          path: AtRestEncryptionEnabled
        is_read_only: true
      LogDeliveryConfigurations:
        from:
          operation: DescribeReplicationGroups
          path: LogDeliveryConfigurations
        is_read_only: true

@jaypipes it doesn't generate code to populate the field if it's already part of spec

@jaypipes it doesn't generate code to populate the field if it's already part of spec

I'm not following you... are you saying that a same-named field of a different Go type is present in the Spec?

same name, same go type. Not checked if different go type makes any difference
https://github.com/aws-controllers-k8s/sagemaker-controller/blob/main/generator.yaml#L108-L112 does not populate any code in sdkFind and we added this workaround

same name, same go type. Not checked if different go type makes any difference
https://github.com/aws-controllers-k8s/sagemaker-controller/blob/main/generator.yaml#L108-L112 does not populate any code in sdkFind and we added this workaround

Why are two different APIs being used in the generator.yaml file for the same Endpoint resource? One is reading DescribeEndpoint.EndpointConfigName and the other is reading DescribeEndpointConfig.EndpointConfigName. Makes no sense to me.
Also, you should not have a field in the spec and status that refer to the same exact thing... :(

https://github.com/aws-controllers-k8s/sagemaker-controller/blob/main/generator.yaml#L108-L112 does not populate any code in sdkFind

The Spec.EndpointConfigName is the field that is populated. You cannot and should not have same named fields in the Spec and Status.
https://github.com/aws-controllers-k8s/sagemaker-controller/blob/32777f0e34d6feb17032564fa0ac2dfa5860df2e/pkg/resource/endpoint/sdk.go#L86-L90

I am against this proposal. You should not have copies of spec fields in the status.

/lifecycle frozen

Refer to https://github.com/aws-controllers-k8s/community/issues/1110#issuecomment-1005844199

/close


Pagination harcode pageLimit

What steps will reproduce the problem?
I extend yii2/framework/data/Pagination.php
public function setPageSize($value, $validatePageSize = false)
{
    if ($value === null) {
        parent::setPageSize($value, $validatePageSize);
    } else {
        $value = (int)$value;

        $page = (int) $this->getQueryParam($this->pageParam, 1);
        if ($this->totalCount && $this->pageSize * $page > (int) $this->totalCount) {
            $page -= 1;
            $value = $this->totalCount - $this->pageSize * $page;
        }
        parent::setPageSize($value, false); //in this code I redefined number of model
    }
}

What is the expected result?
"perPage": 3
I wonna more control for default parameter  "perPage" and separate from $value from setPageSize
What do you get instead?
3 - default
2 - custom
"perPage": 2 not 3 as default
but for perPage should remain 3
Additional info

Q
A

Yii version
2.0.13

PHP version
7.1

Operating system
ubuntu

if (!in_array($value, $this->getPageSizeLimit())) {

what is getPageSizeLimit(), what does it do?

$this->pageSize = $value;

this looks like a recursive call of the setter, or have you defined the property?
what exactly is the bug/problem in the framework. I do not get it, please explain in more detail.

when I need to limit the sample, I set the parameter pageSize, but the parameter perPage also changed

strong connectivity of parameters

$pageSize is the name of the poperty for the number of items per page. per-page is the name of the URL parameter that controls $pageSize. They are the same.


Add the partnership channel in the white-list

add the partnership channel in the white-list, i am a partnership and i cant send links there due to automod of coding bot.
Description
Please include a summary of the change and which issue (if any) is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
If this fixes an issue, please list it below.
Fixes # (issue)
How to fill this out
Please mark options that are not relevant with the * character (like this*).
Replace the space in the box with an x to check the box (it should look like [x]) OR you may mark your changes immediately after submitting the PR.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other (Please specify):

Has this been tested:

[ ] Yes
[ ] No
[ ] Not needed

Checklist:
If any are false, you may still submit, but may be asked to make changes.

[ ] My code follows the style guidelines of this project

[ ] I have changed/added the help doc at the beginning of any commands changed/added.

[ ] I have performed a self-review of my own code

[ ] I have commented my code, particularly in hard-to-understand areas

[ ] My changed files do not include any files regularly changed in saves (something where my bot would interfere with this).

[ ] My changes generate no new warnings

[ ] I have tested and concluded that my changes are effective or that my feature works - If you have not tested, dont check this so I know to review it more. You may still make a PR for untested code.

[ ] My code does not cause issues with existing code

[ ] Any dependent changes have been merged and published in downstream modules

[ ] Some of these options did not apply to my change (Please specify which):

Does this problem affect others ppl?

Only admins can send the invite links and things.


Short define alias when defining a new binding

Usage:
Ioc.alias('Helpers').singleton('Adonis/Src/Helpers', ...)

Still supports the old way of defining an alias.
Also bindings are being stored in a nested object by using lodash and converting the namespaces to dot notation, so you can basically get a better overview of what class belongs to what namespace as they're nested in an object containing the identifier (class) as key and the rest as value
E.g
Ioc.alias('Config').singleton('Adonis/Src/Config', ...)
Ioc.alias('Env').singleton('Adonis/Src/Env', ...)

// Get bindings
Ioc.getBindings()

// Old way it would return
{
  'Adonis/Src/Config': {},
  'Adonis/Src/Env': {}
}

// Now it simply returns a nested object with an array of identifiers instead
{
  Adonis: {
    Src: {
      Config: {},
      Env: {}
    }
  }
}

Sorry for getting late on this.

Making alias chain-able looks fine.
I am not sure what purpose does splitting the namespace into nested objects serve?


Multi word search does not work in PDFJS viewer.

Attach (recommended) or Link to PDF file here:
HebrewText.pdf
Configuration:

Web browser and its version: Mozilla Firefox 115.0.2 (64-bit), Google Chrome 114.0.5735.198
Operating system and its version: Windows, Linux (Redhat and Ubuntu)
PDF.js version: 3.8.162
Is a browser extension: No

When I search multi word search, search results in 0 result. When I select multi words using text selection and paste that in my notepad, copied text does not include space with in that.
When I inspect texts in dev-tool in Firefox browser I see each word is separated with <span> tag.
Steps to reproduce the problem:

Search text ההתרחבות הבינלאומית התרחשה גם היא in pdfjs viewer and you will not be able to search it even though text is present.
Search text ההתרחבות הבינלאומית התרחשה גם היא in Firefox browser directly (without pdfjs viewer) with provided test document and you will not be able to search it even though text is present.
Search text ההתרחבות הבינלאומית התרחשה גם היא in chrome browser directly (without pdfjs viewer) with provided test document and search will result in positive result with highlight text.
Search text ההתרחבות הבינלאומית התרחשה גם היא in Acrobat reader with provided test document and search will result in positive result with highlight text.
Sect multi word text when document is opened in Firefox and paste it in note pad, you will see "no space" in pasted content.

Conclusion : pdfjs viewer or Firefox fails to represent text in PDF as compared to Acrobat reader.
What is the expected behavior?
Positive search result

What went wrong? (add screenshot)
Search does not result in text to find.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Our heuristics to detect white spaces with RTL languages is likely adapted to this case.
The first word (from the right) ההתרחבות is drawn from left to right and before starting the second word we've a big negative move.

Any updates

@calixteman Thanks for considering this issue. Do you have any updates on this defect or ETA ?

Regards Dhananjay

@calixteman Do you have any update on this ? and can this be prioritized in next release ?

@calixteman Currently this feature "Text selection and Search" is impacting languages like Hebrew in Mozilla. By when can we expect fix for this problem in pdfjs ?


looks like some bugs about moving the virtual camera, include TF

TF and joint states
After start the bebop_nodelet.launch, use rqt_tf_tree tool, seems work fine.
use rostopic echo /bebop/joint_states seems get the correct camera Pan&Tilt states.
and I try to move the virtual camera, for example,

rostopic pub /bebop/camera_control geometry_msgs/Twist "linear:
  x: 0.0
  y: 0.0
  z: 0.0
angular:
  x: 0.0
  y: -10.0
  z: 0.0"

The camera view is moving, but use rostopic echo /bebop/joint_states can't get any messages any more, and rqt_tf_tree tools shows bellow,

It seems that the connection is interrupted.
How to fix it?
2. how can I set absolute tilt and pan of the camera? After read the docs, I tried to let the camera watch directly below use set the angular.y to a small negative number such as -90, but it seems doesn't move 90 degrees. How can I do that?
Thanks!

I have tried indigo-devel and sdk-3-12 two branches of this package, it seems that this problem still exist.

I have the same problem. Would you fix it? Is any other topic where I can get the camera angles?
Thank you.

I forked the repository and fixed the bug for the beobp2: https://github.com/itssme/bebop_autonomy

@itssme, great!! Why you didn't open a pull request? It would be nice to share your results with others.

@gsilano Because I am not sure if the package would still work with the bebop 1. I only have a bebop2 to test :/

@itssme, don't worry. I think the maintainer or contributors have a Bebop 1 to test the code.

@gsilano Ok, I made the pull request: https://github.com/AutonomyLab/bebop_autonomy/pull/178

@itssme @gsilano  why it is called virtual since there is a camera there!!! I don't get it


Editor page not using full width

Example. There should be no line wrapping when the browser window is wide enough.

This is already fixed in https://github.com/dlang-tour/core/pull/545

That doesn't seem like the right issue # ?

That doesn't seem like the right issue # ?

Yep sorry.
-> https://github.com/dlang-tour/core/pull/544

Actually I noticed that this issue is reproducible on Firefox but not on Chromium, like #552. So, just to clarify, a screenshot:

#544 has been merged, so this should be fixed.


Allow the use of a global installation of Node

Hi, first thanks for that great package!
I am developing a collaborative blog using JuDoc.
I may need to deploy JuDoc in a docker container with a Node.js application which has a list of dependencies in its package.json file.
I just need to run npm install in the application folder to install all the js dependencies.
This package.json is the perfect place to put the JuDoc dependency highlight.js.
However, since issue #262 and PR #263, JuDoc uses exclusively NodeJS.jl and its "julia specific" node installation, which is not convenient as I install highlight.js using my docker image "global" node install.
Are you against something like the following in the build.jl of JuDoc for people who want to choose the node install?
const NODE = begin
    if "NODE" ∈ keys(ENV)
        ENV["NODE"]
    else
        NodeJS.nodejs_cmd()
    end
end

I tested it works.
It would be consistent with the doc troubleshooting section https://tlienart.github.io/JuDoc.jl/dev/#Troubleshooting-1.
Moreover it is related to NodeJS but the function, NodeJS.nodejs_cmd, to launch node,
errors in a docker with alpine 3.10.2 and julia1.2.0.
Building the Dockerfile:
FROM fredrikekre/julia-alpine:1.2.0
RUN julia -e 'import Pkg; Pkg.add("NodeJS")'
RUN julia -e 'using NodeJS; run(NodeJS.nodejs_cmd())'

returns
Step 3/3 : RUN julia -e 'using NodeJS; run(NodeJS.nodejs_cmd())'
 ---> Running in 07e33df46554
ERROR: IOError: could not spawn `/root/.julia/packages/NodeJS/rx0mH/deps/bin/node-v10.15.0-linux-x64/bin/node`: no such file or directory (ENOENT)

Hey thanks a lot, I'll review your proposal asap, thanks for taking the time! (I'm keen to have an easy way for most people and a specific way  for folks like you who know their way around NPM 😋)
The project sounds great btw!

Ok so now with a bit more time to look at what you're suggesting, it seems perfectly reasonable, thanks a lot, would you mind opening a PR? otherwise I'll do it tomorrow.
As a side note, I think you should report the error on Docker to the NodeJS repo!
PS:  I'm keen to learn more about your collaborative blog idea; if I can help in any way or just watch as something nice happens, I'll be happy to, cheers.

Thanks for the answer I just did a PR !
I will investigate a bit about the NodeJS.jl error and if I don't find the error I'll report it yes!
PS: Of course I'll be happy to discuss with you about the collaborative blog we build with a colleague! I propose on the Julia slack

closed by  #295


Nil assertion in the Compiler when defining constants

This code:
A = B
B = A = 1

p B

Crashes the compiler:
Nil assertion failed (NilAssertionError)
  from /crystal/src/nil.cr:108:5 in 'not_nil!'
  from /crystal/src/compiler/crystal/semantic/cleanup_transformer.cr:300:10 in 'transform'
  from /crystal/src/compiler/crystal/syntax/transformer.cr:7:14 in 'transform'
  from /crystal/src/compiler/crystal/semantic/cleanup_transformer.cr:288:20 in 'transform'
  from /crystal/src/compiler/crystal/syntax/transformer.cr:7:14 in 'transform'
  from /crystal/src/compiler/crystal/semantic/cleanup_transformer.cr:161:18 in 'transform'
  from /crystal/src/compiler/crystal/syntax/transformer.cr:7:14 in 'transform'
  from /crystal/src/compiler/crystal/semantic/cleanup_transformer.cr:14:14 in 'cleanup'
  from /crystal/src/compiler/crystal/semantic/cleanup_transformer.cr:11:5 in 'semantic:cleanup'
  from /crystal/src/compiler/crystal/compiler.cr:172:14 in 'compile'
  from /crystal/src/compiler/crystal/command.cr:215:5 in 'run_command'
  from /crystal/src/compiler/crystal/command.cr:122:7 in '__crystal_main'
  from /crystal/src/crystal/main.cr:115:5 in 'main'
  from src/env/__libc_start_main.c:94:2 in 'libc_start_main_stage2'
Error: you've found a bug in the Crystal compiler. Please open an issue, including source code that will allow us to reproduce the bug: https://github.com/crystal-lang/crystal/issues

Most likely a duplicate of https://github.com/crystal-lang/crystal/issues/2874 ?

Yup, closing it.

BTW, @lbguilherme did you find out by chance, or did you happen to actually want to do that?

No, it was by chance.


when the native page is mixed with Flutter, hide the native tabbar on the push new page , the flutter page will be stretched during the transition to hide the tabbar animation。

when the native page is mixed with Flutter, hide the native tabbar on the push new page , the flutter page will be stretched during the transition to hide the tabbar animation。

 native
channel!.setMethodCallHandler { (call: FlutterMethodCall, result: @escaping FlutterResult) in
  if call.method == "countChange" {
      if let value = call.arguments as? Int{
          print("countChange ===> \(value)")
          self.tabBarController?.setTabBarHidden(value != 0, animated: true)
      }
      result(nil)
  } else {
      result(FlutterMethodNotImplemented)
  }
}

func setTabBarHidden(_ hidden: Bool, animated: Bool) {
    guard let vc = selectedViewController else { return }
    guard tabBarHidden != hidden else { return }

    let frame = self.tabBar.frame
    let height = frame.size.height
    let offsetY = hidden ? height : -height

    UIViewPropertyAnimator(duration: animated ? 0.3 : 0, curve: .easeOut) {
        self.tabBar.frame = self.tabBar.frame.offsetBy(dx: 0, dy: offsetY)
        self.selectedViewController?.view.frame = CGRect(
            x: 0,
            y: 0,
            width: vc.view.frame.width,
            height: vc.view.frame.height + offsetY
        )

        self.view.setNeedsDisplay()
        self.view.layoutIfNeeded()
    }
    .startAnimation()
}

 flutter
 _channel.invokeMethod<void>("countChange", ShareModel.shared.count);

demo.zip

@zeqinjie Thanks for the report with the complete sample code. But as far as I can see, the image doesn't seem to be stretched. Perhaps it is an optical effect when there is animation transition.
Verifying on the latest Flutter stable 3.7.3, iPhone 7, iOS 15.7.2.
Can you take screenshots and confirm again?

Tabbar
No Tabbar

@huycozy ok thx


Generate Engine.ini based on ENV vars

Context
Fixes https://github.com/thijsvanloef/palworld-server-docker/issues/257 https://github.com/thijsvanloef/palworld-server-docker/issues/176
Enable users to generate the Engine.ini using evironment variabels
Choices
Using the same method as compile-settings.sh, compile the engine settings.
Test instructions

Build the container
set DISABLE_GENERATE_ENGINE to false
check that ./palworld/Pal/Saved/Config/LinuxServer/engine.ini has been updated

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I've added documentation about this change to the docusaurus.
[x] I've not introduced breaking changes.

Looks good to me. Tested it locally and it works without issues. It think it would be good to document in the docs how to enable it tho.
Also is there a specific reason why it is disabled by default?

Mostly because of backwards compatibility, I do not want to overwrite the changes someone has made already.
Documenting the need to enable is a good one, need to do that


[#3186] feat(spark-connector): Support Iceberg Spark Procedure

What changes were proposed in this pull request?
Support Iceberg Spark Procedure.
Why are the changes needed?
Support manage Iceberg metadata using Spark SQL.
Fix: https://github.com/datastrato/gravitino/issues/3186
Does this PR introduce any user-facing change?
No.
How was this patch tested?
New ITs.

Hi @FANNG1 could you help review this pr when you are free?

Hi @FANNG1 could you help review this pr when you are free?

ok

comments have been addressed. cc @FANNG1

@FANNG1 could you help review this again?

@caican00  could you rebase with lattest code?

building failed as NameIdentifier cannot be converted to String in CatalogKafkaIT.
/home/runner/work/gravitino/gravitino/catalogs/catalog-kafka/src/test/java/com/datastrato/gravitino/catalog/kafka/integration/test/CatalogKafkaIT.java:181: error: no suitable method found for createCatalog(NameIdentifier,Type,String,String,ImmutableMap<String,String>)
        metalake.createCatalog(
                ^
    method SupportsCatalogs.createCatalog(String,Type,String,String,Map<String,String>) is not applicable
      (argument mismatch; NameIdentifier cannot be converted to String)
    method GravitinoMetalake.createCatalog(String,Type,String,String,Map<String,String>) is not applicable

> Task :catalogs:catalog-kafka:compileTestJava
      (argument mismatch; NameIdentifier cannot be converted to String)

@caican00 could you rebase with lattest code?

It is already the latest code

building failed as NameIdentifier cannot be converted to String in CatalogKafkaIT.
/home/runner/work/gravitino/gravitino/catalogs/catalog-kafka/src/test/java/com/datastrato/gravitino/catalog/kafka/integration/test/CatalogKafkaIT.java:181: error: no suitable method found for createCatalog(NameIdentifier,Type,String,String,ImmutableMap<String,String>)
        metalake.createCatalog(
                ^
    method SupportsCatalogs.createCatalog(String,Type,String,String,Map<String,String>) is not applicable
      (argument mismatch; NameIdentifier cannot be converted to String)
    method GravitinoMetalake.createCatalog(String,Type,String,String,Map<String,String>) is not applicable

> Task :catalogs:catalog-kafka:compileTestJava
      (argument mismatch; NameIdentifier cannot be converted to String)

https://github.com/datastrato/gravitino/pull/3424

@FANNG1 could you please help re-trigger the pipeline? thank you.

@caican00 ,merged in main, thanks for your work


Fix dead link

Fixes #477
Replaced old link with https://github.com/mattermost/mattermost-plugin-jira/blob/master/readme.md

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

@aaronrothschild - ping for review


Adding search node to the cluster breaks snapshot repository

Describe the bug
We have a cluster of ten nodes running for a long time.
We use SMB share to store our snapshot repository. Many years it receives daily snapshots and sometimes some of them got restored successfully.
After testing the searchable snapshot feature at the test installation, we decided to implement it to our production system.
However, it turned out that adding the node.roles: [search] node makes storage snapshot inoperable. Here is the output:
% curl logs:9200/_snapshot/searchable/_verify -XPOST | jq . {   "error": {     "root_cause": [       {         "type": "repository_verification_exception",         "reason": "[searchable] cannot delete test data at "       }     ],     "type": "repository_verification_exception",     "reason": "[searchable] cannot delete test data at ",     "caused_by": {       "type": "directory_not_empty_exception",       "reason": "/usr/share/opensearch/searchable/tests-WuWXrwFrTd-BiVX9VfEkTw"     }   },   "status": 500 }
At the same time, using any node, including new search node:
% ssh <any_node> sudo docker exec <container_name> 'ls -l /usr/share/opensearch/searchable/tests-WuWXrwFrTd-BiVX9VfEkTw' total 0
Just by switching new search node off:
% curl logs:9200/_snapshot/searchable/_verify -XPOST | jq .                                      {   "nodes": {     <all 8 data/master nodes here> }}
I.e. it is definitely an empty directory.
Related component
Other
To Reproduce
Just add node.roles: [ search ] node to existing '[data]', '[master]', and '[ ]' nodes and see weird verification error.
Expected behavior
Adding search node should not affect existing cluster.
Additional Details
Plugins
Security with Keycloak SAML
Host/Environment (please complete the following information):

OS: Oracle Enteriprise Linux 8.9

Additional context
Master node log at the next message.

Here is the related master node log excerpt:
[2024-04-02T17:12:40,341][WARN ][o.o.r.RepositoriesService] [v4-master.company.com] [searchable] failed to finish repository verification org.opensearch.repositories.RepositoryVerificationException: [searchable] cannot delete test data at  	at org.opensearch.repositories.blobstore.BlobStoreRepository.endVerification(BlobStoreRepository.java:1986) ~[opensearch-2.12.0.jar:2.12.0] 	at org.opensearch.repositories.RepositoriesService$3.lambda$doRun$1(RepositoriesService.java:383) [opensearch-2.12.0.jar:2.12.0] 	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingRunnable.run(ThreadContext.java:854) [opensearch-2.12.0.jar:2.12.0] 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) [?:?] 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) [?:?] 	at java.base/java.lang.Thread.run(Thread.java:840) [?:?] Caused by: java.nio.file.DirectoryNotEmptyException: /usr/share/opensearch/searchable/tests-evLsfSTJQAOl5JGzamyFwQ 	at java.base/sun.nio.fs.UnixFileSystemProvider.implDelete(UnixFileSystemProvider.java:246) ~[?:?] 	at java.base/sun.nio.fs.AbstractFileSystemProvider.delete(AbstractFileSystemProvider.java:105) ~[?:?] 	at java.base/java.nio.file.Files.delete(Files.java:1152) ~[?:?] 	at org.opensearch.common.blobstore.fs.FsBlobContainer$1.postVisitDirectory(FsBlobContainer.java:141) ~[opensearch-2.12.0.jar:2.12.0] 	at org.opensearch.common.blobstore.fs.FsBlobContainer$1.postVisitDirectory(FsBlobContainer.java:137) ~[opensearch-2.12.0.jar:2.12.0] 	at java.base/java.nio.file.Files.walkFileTree(Files.java:2828) ~[?:?] 	at java.base/java.nio.file.Files.walkFileTree(Files.java:2882) ~[?:?] 	at org.opensearch.common.blobstore.fs.FsBlobContainer.delete(FsBlobContainer.java:137) ~[opensearch-2.12.0.jar:2.12.0] 	at org.opensearch.repositories.blobstore.BlobStoreRepository.endVerification(BlobStoreRepository.java:1984) ~[opensearch-2.12.0.jar:2.12.0] 	... 5 more

To collect more information, I've set logger.level: DEBUG at all four master-eligible nodes and restated them.
And now my repository got successfully verified...

[Triage - attendees 1 2 3 4 5 6 7 8]
@rlevytskyi Thanks for filing this issue

I wrote a simple Python program to check the contents of the snapshot directory every 0.01 seconds.
Here is it's output when search node stopped and repo verified successfully:
dir-ok.txt
Here is it's output when search node connected and this error returned

"type": "directory_not_empty_exception",
"reason": "/usr/share/opensearch/searchable/tests-s4d0kZgWQL6dXHjTnIcMiw"
dir-fail.txt

I.e. two file are sitting here extra couple of seconds.

In case somebody interested, I managed to conquer this issue.
Our cluster is running Oracle Linux Server 8.9 with ol8_UEKR6 kernel (5.4.17).
New node is running Oracle Linux Server 8.9 with ol8_UEKR7 kernel (5.15.0).
We've upgraded all our nodes to 5.15 and got backup broken (repository verify failed).
Then I've restarted all nodes at 5.4 kernel and found repo verification is working properly again.
Then downgraded new node kernel to 5.4, add it to cluster and it works properly now.

Thanks for narrowing it down! Still, what is the root cause of the verification failure?

I was unable to identify the exact cause.
What I've noticed that with 5.4 kernel, we see some time discrepancy (sometimes up to 2 seconds) between system clock and hardware clock of VMs (as per 'timedatectl' ouptput).
With 5.15 kernel, time is the same at system clock and hardware clock.
However, with 5.4 kernel snapshot just works as it supposed to works and we are now successfully implementing the searchable snapshot feature.

Hello team,
We are working together with @rlevytskyi and we have tried to identify the root cause of the failure, unfortunately, without any meaningful results.
Our lab:

snapshot repository backed with SMB storage at TrueNAS
searchable snapshot node working with kernel 5.15
other cluster nodes working with kernel 5.4

We have identified that the search node can successfully create the test files in the repository. Afterwards it seems that the files should be deleted and the master node. Afterwards, it's verified that the files could be deleted by the master. However with kernel 5.14 it looks like the master node cannot delete the files created by the search node and verification fails.
Probably it's some synchronization problem between the two nodes when the master node tries to delete the files too early or the file is still "locked" by the snapshot node.
Yet it is still unclear how it can be related to the kernel version. We tried to play with SMB options - disabled cache and played around with versions - same issue. In addition, the files can be successfully created and deleted on the share by any node. The problem seems to be only about the repository verification procedure.
We have also tried to boot all the nodes to kernel 5.15 but it has also resulted with the error.
Since we are not Java developers, it is not so easy for us to analyze the code and completely understand the algorithm of the repository verification to find out the root cause of the issue. So we would be grateful for any tips about where we can dig deeper.
Thanks!


[Feature request] Add configuration for default scheme for tests

On my project we have multiple build schemes but the tests run on only one of them and most of the time, if not all, I develop and run the app on one scheme and run the tests on another one.
we could have the default test scheme, similar to the existing default run scheme, to not have to switch schemes to run the tests.
And it might be possible to use
xcodebuild test-without-building \
    -workspace <workspace> \
    -scheme <scheme for testing> \
    -destination <dest> \
    -enumerate-tests \
    -test-enumeration-format json \
    -test-enumeration-output-path <path to result>

to get the correct test plan and show the tests on test explorer also

Oh, interesting insight! I think it would be great to use the 'ask' scheme by default and store it in a separate context value than the one we have for the build. Additionally, for users who prefer configuration files, it would be good to provide the option to define the scheme in .vscode/settings.json

I've added few additional commands in v0.1.40:
      {
        "command": "sweetpad.testing.setDefaultScheme",
        "title": "SweetPad: Set scheme for testing",
        "icon": "$(file-code)"
      },

      {
        "command": "sweetpad.destinations.selectForTesting",
        "title": "SweetPad: Select destination for testing",
        "icon": "$(file-code)"
      }

Also "configuration" (Debug, Release) option is also separated for "launch" and "build" contexts

Amazing! I already updated the extension and it's soooo helpful!


Get Glimpse button interaction

@chetsangnil commented on Thu Sep 22 2016
Once click on "Get Glimpse", design the pop up that allow users to choose to start with either NodeJS of .net.

Issue moved to Glimpse/Website #26 via ZenHub


support spotify

Is your feature request related to a problem? Please describe.
support spotify
What is your ideal solution to the problem?
support spotify
How would this feature be used?
support spotify
Additional Info
support spotify
Checklist

[X] I have checked the documentation to make sure this feature doesn't already exist
[X] I have searched for similar feature requests
[X] I have confirmed that my requests is not on the list of things that will not be added
[X] I am running the latest version of the bot:

Duplicate of #66


Shopify Events are not accessible

All event codeunits in Shopify has "Access = Internal;" and is defined as "[InternalEvent(false)]"
Please give us access to the events in codeunit 30115, 30154, 30162, 30177, 30192,30196 and 30200.
All tables also have "Access = Internal;". Please remove this for at least tables used as parameters in the events in above codeunits.

@CorpitalFDP  Please check the following yammer post of @AndreiPanko .
https://www.yammer.com/dynamicsnavdev/threads/1762140027789312

As source code of Shopify Connector is on GitHub we would like to reuse pattern we have for System App:
Prioritize submitting improvement over extensibility.

For example, if your customer needs to export multiple translations of a product to Shopify, instead of asking for making some objects extensible – submit PR that implements this feature.
If extensibility is really necessary – for example to orchestrate multiple extensions dealing with various sales channels – please create PR directly. Remember to write the proper justification of scenario.

We will not process extensibility requests for connector registered as Issues on GitHub.

Duplicate of similiar issues: https://github.com/microsoft/ALAppExtensions/issues/18681, https://github.com/microsoft/ALAppExtensions/issues/18680

See above, closing


pull request for e2e test at 01/12/2017 19:56:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix error handling on front and backends

Searching for a keyword without results causes an error in api.py. On
the frontend, an error disables the dropdown.
This change sets results to an empty array on error and doesn't disable
the dropdown so users can search again.

Thanks for fixing! Looks good -- merging


export: change first day of semester

@danyang685 Would you please help deploy this change to server after CI builds succeed on master branch?


column profiles, ydir

same as before, just pulled into my master

Is there anybody that can review this pull request?


TestPipelineKeepSendingWhenPostError fails; need to split performance testing from liveness testing

--- FAIL: TestPipelineKeepSendingWhenPostError (0.01s)
        pipeline_test.go:74: request count = 19, want 50

https://github.com/coreos/etcd/blob/336d177c82a062d6cd02a5f9fb0926e693f01071/rafthttp/pipeline_test.go#L73-L75
The test requires 50 requests performed in just 10 msecs, which cannot be guaranteed in certain environments.
So I suggest to split the performance aspect from "liveness" test cases (including this TestPipelineKeepSendingWhenPostError, but there are also many similar things) by introducing some retries.
The performance evaluation should be done in another tool.
Example
package testutil

// f should be idempotent
func TryWithRetries(t time.Duration, retries int, f func() error) error {
	var err error
	for i := 0; i < retries; i++ {
		err = f()
		if err == nil {
			return nil
		}
		time.Sleep(t)
	}
	return err
}

package rafthttp
func TestPipelineKeepSendingWhenPostError(t *testing.T) {
      ..
	err := testutil.TryWithRetries(10 * time.Milliseconds, 100, func() error {
		if tr.reqCount != 50 {
			return fmt.Errorf("request count = %d, want 50", tr.reqCount)
		}
	})
	if err != nil {
		t.Error(err)
	}
}

/cc @heyitsanthony

I've seen other tests do something similar but so far it's all ad-hoc. Ideally we wouldn't want to poll at all (e.g., count events via testutil.RecorderStream) but implementing interfaces to support Recorder is a chore. Overall, this suggestion is certainly better than the WaitSchedule stuff that's there now.

After some work, we made our tests pretty stable. We always can switch to a more stable env to do the CI work. We will see how it goes. If it does not go well, we might want to separate the tests.


Prefetch not working with Nuxt

I want to fetch a 'customer' based on customerId in his cookie. I'm using Nuxt for SSR. Below is the code that supposed to work:
<template>

    {{customer}}

</template>

However it doesn't work. But below one works. But I'm not sure where the customerId is taken in query. It isn't clear. Is the below code correct? Anyway to reduce it?
<template>

    {{customer}}

</template>

Yes your code below is correct.

Ok. I've to use the same query in many components with cache-only or cache-first , by using the same cookie as query variables. Is it possible to reduce the code my moving this logic to middlewares/plugins or something? So that customerId from cookie will be injected to data in every component!
Also if that's possible, is it possible to get customer query result in every component without writing cache-only in every component.


fix: Make countries human readable (#647)

Summary
Make countries human readable (#647)
Before this change, countries other than United States were shown in the data reports using their alpha-3 code. Now they use the full, human readable name.

Changes
Changed the transformIds function to transform alpha-3 country codes. The function was previously only relying on countryLevels.transformLocationIds, which does not recognize alpha-3 country codes.
Additional notes
Unfortunately this does not fix the cross check report - the locations there are still not human readable. This would require more work, likely moving the transformIds function to run before the dedupeLocations function, and modifying the findFeatures and findPopulations functions to work with the transformed ids (perhaps by using the id fields like countryId instead of fields like country).

Thanks for this fix. Right now we are workign very hard in making sure that all countries have ids and nice human readable names!

Ok so this is close, but we need to get countryId right. I had a branch I was working on this already in, so I may close this out.

We got this fixed up tonight, thanks for the PR though!


[Android] Is it possible to use system-provided fonts?

I found that Electrum has an unfixed  CJK issue for several years. As long as the language is switched to Chinese/Japanese etc, the characters just turn blank. It's quite annoying. I've done some googling, but I found no answer other than "pack your own font into your app" - but why should every app pack their own font? It's so weird. What if the owner of smartphone customize the font? Should kivy apps "resist"?
I wonder if it's possible to let kivy simply use the system-provided font, so that the long-lasting CJK issue could be resolved?

In contrary, are you 100% sure that the font you asked will be there on all system?
On desktop you can reference system font, we could do the same on Android. But if the font you asked is not there, then that's the same, you'll end up with app that are broken.
We don't have any kind of font management/fallback, and per glyph fallback.
Currently there is the Pango text backend that starts to support it, but that's it :/

@tito
I've came across a lot of Android apps which seemed to be using machine translation. The translated text might not be accurate, but I've never came across "tofu" issues like Kivy.
Also, although I have little knowledge about Android system, I once tried to build my own toy app - I didn't remember that I must specify a font explicitly.
I think the solution to the localization problem is quite simple: use the default font provided by the system - is this impossible for Kivy?

Why should the app specify a font explicitly? Why should an app use complicated "per glyph fallback" at all?
For an American user, if he installed a Japanese app, he might came across the "tofu" issue - but in my opinion it's actually a non-issue. What actually matters is that the real Japanese user only sees "tofus" on the screen.

On desktop you can reference system font, we could do the same on Android. But if the font you asked is not there, then that's the same, you'll end up with app that are broken.

Sorry, it's already broken, now, for any CJK user.

@andronoob yes and just switching mindlessly over to a system provided font may break it for everyone else, so calm down a little.
See, it's not currently how it is because we hate CJK users, but because SDL_TTF and other font renderers have quite simple font handling and as @tito already said, there is no fallback handling (so basically right now you can pick one font and if it's not there it won't work, if I got right how kivy works).

I think the solution to the localization problem is quite simple: use the default font provided by the system - is this impossible for Kivy?

No, but it would most likely require new fallback handling so it doesn't all just break if there is somehow a problem with fetching a default TTF. Someone needs to implement that and make sure it works properly, it's most likely more than a one-line change.

A more constructive summary: your suggestion to use system fonts for international support where needed is reasonable, kivy just needs to go about this in a way that doesn't break it for others. And that may be more work than just switching to a specific Android system font if fallback handling is needed (and certainly will need to be tested well)

Sorry for my rudeness. But I'm still confused - did the above comments imply that:
(1) Generally, Android system is responsible to handle all the complicated fallback stuff of text rendering job;
(2) No matter how well Android OS could handle it, perfect or broken, it's impossible for Kivy apps to make use of those features provided by Android system directly, so that Kivy has no choice but to re-implement all of those complicated "per-glyph-fallback" itself?

It's impossible for Kivy apps to make use of those features provided by Android system directly

Yes that is exactly the problem, because kivy uses SDL2 and/or OpenGL which completely bypasses the entire Android UI toolkit, and it doesn't use any of the Android functions for text widgets a normal app would use (because they aren't cross-platform)

@andronoob You miss one point that will enlight you: the default font don't have all the glyph available in the world. If you talk about CJK, sometimes it's additionnal font that provide the set of characters.
Let's you want to render something "Hello 你好".

in the system, you may have 2 differents font used for this. The way the renderer works is checking the availability of the Glyph for the first prefered font, if not available, fallback on the second one, etc.
in Kivy using SDL2 and freetype, you point on a single font filename/font alias. There is no way to point on multiples font filenames, as freetype itself doesn't support Glyph fallback. So yes if you use a font that doesn't have both charset, you're stuck.
Pango backend leverage this as they support Glyph fallback. So you may have a list of available font, and always have the possibility to use the Glyph from another font even if that wasn't the one used in the first place.

Yes, so that, either we need to implement the per-glyph rendering, and/or push the pango backend on all platform more that will solve this (bonus, it also supports emoji rendering)

@tito

the default font don't have all the glyph available in the world.

I've already pointed out that:

Generally, Android system is responsible to handle all the complicated fallback stuff of text rendering job

Even if Andoid can't handle this correctly, it's still not really a problem - for an American user, if he installed a Japanese app, he might came across the "tofu" issue - but in my opinion it's actually a non-issue. What actually matters is that the real Japanese user only sees "tofus" on the screen.

Sorry for copy&paste, but, I'm really, really confused why Kivy cannot just let Android to do its own job.

As a user, I assume that:

Basic level of localization support: the user could just install the app & start to use it in his native language.

Kivy is already broken at this level.

Advanced level of localization support: even if the OS cannot handle font/text rendering correctly, the App could still work flawlessly - but in some scenarios like copy&paste it still inevitably has "tofu" issue.

It seems that Kivy is currently pursuing this.

Additionally, packing TTFs into the APK would obviously make it bloat - although packing several TTFs still won't "have all the glyph available in the world."

You have a misconception about what the system can or cannot do.

Generally, Android system is responsible to handle all the complicated fallback stuff of text rendering job

-> I would say yes, for native application. All the OpenGL application must do their own stuff, and mostly any project that implement their own rendering engine (as us.)
Just have a look, take Chromium, walk through the source code and see by yourself: https://github.com/chromium/chromium/tree/master/ui/gfx
And if you're not native, then you want to implement your own "text renderer". Check https://stackoverflow.com/questions/7165097/render-text-in-purely-native-android-app for example, there is no NDK api for that, we cannot just "use Android".
Saying that "Kivy is currently pursuing this" is misleading, and just throw away the work toward it.
We know what's wrong, and if it would be easy to fix it would already be done.

@andronoob maybe just let us conclude that we know what needs to be done, but it's not trivial. What do we gain from discussing this further?

Sorry for copy&paste, but, I'm really, really confused why Kivy cannot just let Android to do its own job.

Then maybe you should stop here :smile: as @tito (and me) already explained this is simply due to how NDK/OpenGL/SDL2 applications inherently work. Like it or not that's how things are :woman_shrugging: if you're dragging us in a discussion circle here, that won't magically change it or get it implemented.
If you have a pull request, we'd be very interested. Otherwise, this will be done once someone of us gets around to doing it

Sorry for disturbing (again), but, I'm still curious if this Canvas.drawText method is possible to solve text rendering issue: https://stackoverflow.com/a/4336679/5713110
I know I'm very ignorant, but, it looks very likely that "using Android" is possible.
I noticed that, as some comments had pointed out, this method might suffer from low efficiency - however, it doesn't look like a issue for an app like Electrum to me.

I also found that most Chinese Kivy app developers don't see this as a problem - they are happy to pack font files into the APK.
However, this doens't seem to be the same case of Electrum, because Electrum has to support much more languages.

However, in China, fonts vary with ROM packages. That's to say, fonts used in Huawei devices are different from those in Xiaomi. If I package my Xiaomi font into the apk and release, and then a Huawei user installs on his device and sees my Xiaomi fonts, won't he feel weird?

Kivy does have support for using system provided fonts, just not for using font fallback or reshaping.
https://github.com/kivy/kivy/blob/master/kivy/core/text/__init__.py#L381
You should be able to specify any font present in the dir /system/fonts on android
On electrum thus you should be able to specify a font specific to CJK at the time of changing the language, and things should work!

for ref:
https://github.com/spesmilo/electrum/issues/2032
https://github.com/spesmilo/electrum/issues/5703

@akshayaurora To my understanding, Android knows which font to use, furthermore, it can combine multiple fonts to render text in complicated scenarios like mixed language (aka "per-glyph fallback"), while Kivy (for Android, which doesn't have/use pango library) can't do this at all (Kivy application has to hard-code font related things) - that's the problem. You mentioned DroidSansFallback.ttf, however it may or may not exist.
On Electrum it's not only about CJK, it's also about many other languages like Arabian.

It is possibke to compile pango for android. I am not sure how it happens but i give some sources..

https://github.com/termux/termux-packages/tree/master/packages/pango
I think by waking some changes in the source code it can be compiled.


Enable travis builds for this project

Waiting on authorization

https://blog.travis-ci.com/2018-05-02-open-source-projects-on-travis-ci-com-with-github-apps
looks like travis-ci.com is the goto location ( not travis-ci.org )


Disable parts of markdown syntax

I know this question has been asked before in #179, but it was 6 years ago, so I think it's worth a try asking again.
I'd like to disable some parts of the markdown syntax in my app. In particular, but not only, I'd like to disable the embedded HTML. Is it possible somehow?

The answer is mostly the same as it was for #179. The one difference is that safe_mode was fully deprecated in version 3.0. In fact, the release notes provide a few suggested alternatives for dealing with raw HTML. I expect you'll find what you are looking for there.
Finally, I will note that we recommend using an HTML sanitizer (like Bleach) rather than disabling raw HTML, as disabling raw HTML does not make Markdown secure. You still need a sanitizer if you are parsing Markdown from untrusted sources. See Markdown and XSS for an analysis of the security vulnerabilities of Markdown syntax itself. While that article is written by the developer of PHP Markdown and concentrates on PHP solutions, the principles apply equally to Python-Markdown.
If you are looking to disable other parts of the Markdown syntax (for reasons other than security), then the Extension API is available to alter anything you would like. For the foreseeable future, the Extension API will be the only way to alter the behavior of the parser.

Hi, I have a related question. Is there a way to selectively enable core constructs. For instance I would like to convert links and headings but not blockquotes or emphasis. Is that possible?

@geoffroy-noel-ddh yes, the Extension API can alter any part of the parser.

Do you have more specific examples of that? I can see how to add extensions but I'm not sure how to remove all the core syntax and add them one at a time. Say if I want to start with just links (and perhaps add other core constructs later), what's the approach? Is it as simple as passing a list of names/constants or would I have to override a lot of functions and reuse the built-in patterns?

You might find this tutorial in the wiki to be helpful. However, be aware that it has not yet been updated for version 3.x. Regardless, the basic concepts still stand.
The primary difference between 2.x and 3.x is that 3.x uses the new Registry. So, rather than using del md.inlinePatterns['strong_em'] to remove a processor as demonstrated in the tutorial, you would do md.inlinePatterns.deregister('strong_em').
Of course, if you wanted to rewrite most of the parser, you might find it more efficient to subclass the markdown.Markdown class and override the build_parser method. In this way, you can build up the parts you want, rather than having the default parser built up, only to tear it back down.

@waylan thanks for taking the time to reply, that's very useful.


Change README from fork

README changed from the original content of the fork to the current modified content.

commits from other branches was also included and will be corrected.


Step 6 in Tutorial marks step as correct when incorrect

Was running through the tutorial and accidentally did:
docker run learn/ping www.google.com
instead of
docker run learn/ping ping www.google.com (as the answer states)
My mistake validated as a incorrect answer in the emulator with a popup claiming it was correct.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Oh! Thanks for having the courage to report this! You realise we'll now have to invalidate your exam, right? :smile:
Seriously, good find, and thanks for reporting this!

this is not the right place for this, please email<EMAIL_ADDRESS>


huggingface demo (No GPU)

https://huggingface.co/spaces/yizhangliu/Grounded-Segment-Anything

https://huggingface.co/spaces/yizhangliu/Grounded-Segment-Anything

Thanks so much for building this! We will highlight it in this project!


Feature Request - Scheduler Improvements

[x] First minor - make hours headers customizable instead of strict "h tt" format.
[ ] Second major - resize and drag & drop appointments ability.

I would love to see the 'second major' implemented as well.

@recomrad Are you still working on this? Can we help with anything?

Resize for duration is important.

So if the resize(start, end) with visual cursor feedback and the drag-drop functionality works now, then I'm the happiest developer in the whole world. Probably XD

Closing this one in favor of #1833. Drag and drop to move is already supported.


Added docs for ResponseFlag::DurationTimeout

Signed-off-by: Manish Kumar<EMAIL_ADDRESS>Added docs for ResponseFlag::DurationTimeout which was missing in #12495 .
part of https://github.com/envoyproxy/envoy/pull/12495

CC @moderation

/retest

/retest


Multi-language window title

Currently, Application support's window title only support alphabets. I'm not a programmer, so I think it's incomplete, but I thought of a multi-language version. in addition regular expressions to make the match more flexible.
line 53
- a = line[1:-1].split('|')
+ a = line[1:-1].split('|', 1)

line 58
- title = '*'
+ title = '.*'

line 342
- if fnmatch(cls, cexp) and fnmatch(title, texp):
+ if fnmatch(cls, cexp) and re.match(texp, title):

line 351
- return re.sub('[^A-Za-z0-9@:/?.]+', '-', s).strip('-').lower()
+ return re.sub('[^[:ascii:]A-Za-z0-9@:/?.]+', '-', s).strip('-').lower()

keyd 2.2.6-beta

Thanks. I believe this should be fixed in the latest commit.

Why use fnmatch instead of re.match for window titles?
re.match is more flexibility.
example: Layer that matches filechooser
[*|(save-as|open|all-files|import|export)]

Mainly because it makes parsing more difficult ([, | and ] are all valid regex characters) and also because I didn't think it would be useful in most cases. The end result also looks a bit cleaner (though this is subjective).
I can see the utility of it in the example you have listed, though if the number of bindings is small enough I would just create a separate section for each title. If it bothers you enough or you frequently run into this problem I might consider adding some kind of hybrid to allow for alternation.

Ok. Create a section for each title.

To clarify, I am not proposing a modification. I simply meant that you should define a dedicated section for each set of bindings. If you have a lot of bindings like this then I am willing to consider adding an alternation character.


Fix for issue #8539

#8539 Abide: Clear all errors for radio button group.
Can be tested with the existing test: /test/visual/abide/abide-radio.html.
Steps

Load test
Press 'Submit' on required form
Select an option

The errors on the radio button group will clear as expected.

Fix done separately https://github.com/zurb/foundation-sites/commit/a2745ae950da02bffa66b6b68c4ce768355a5d35


#1261 Update go-getter dependency version

Description
Updated go-getter dependency to avoid issues from malware scanners.
Fixes #1261.

TODOs
Read the Gruntwork contribution guidelines.

[ ] Update the docs.
[ ] Run the relevant tests successfully, including pre-commit checks.
[ ] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[ ] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Updated go-getter dependency to 1.7.1
Migration Guide

+1 same issue here. Hard blocker for us


Add floating "Not Production" banner in North Star

Description
Add floating "Not Production" banner in North Star.
This component does not have mocks. See https://github.com/civiform/civiform/issues/8490
Screenshots

Checklist
General
Read the full guidelines for PRs here

[x] Added the correct label: < feature | enhancement | bug | under-development | dependencies | infrastructure | ignore-for-release | database >
[x] Assigned to a specific person, civiform/developers, or a more specific round-robin list
[x] Added an additional reviewer from outside your organization as FYI (if the primary reviewer is in the same organization as you)
[x] Removed the release notes section if the title is sufficient for the release notes description, or put more details in that section.
[x] Created unit and/or browser tests which fail without the change (if possible)
[x] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[x] Extended the README / documentation, if necessary. For user-facing features, consider updating the user docs. For "under-the-hood" changes or things more relevant to developers, consider updating the dev wiki.

User visible changes

[x] Followed steps to internationalize any new strings

[x] Added context strings to new messages
[x] Didn't use a message in applicant facing code that isn't translated yet (unless behind a flag)

[x] Wrote browser tests using the validateAccessibility method
[x] Made equivalent changes in Thymeleaf for applicant-facing features and changes (or created an issue in the Northstar epic to track that it's missing from Thymeleaf)
[x] Tested on mobile view. See mobile device mode
[x] Manually tested at 200% size
[x] Manually evaluated tab order
[x] Manually tested with a screen reader if the feature is applicant-facing. See screen reader testing

Issue(s) this completes
https://github.com/civiform/civiform/issues/8490

Hi @ssandbekkhaug - Can you please add a screenshot of the banner in the PR description?

Hi @ssandbekkhaug - Can you please add a screenshot of the banner in the PR description?

Done. PTAL


Channel names with spaces not properly resubscribed

If you use channel names with spaces, client.subscribe('foo bar') works fine, but after a reconnection, only foo is resubscribed on redis.

In order to better support this project and its new group of collaborators under a new org we are trying to clean up and close issues older than 6 months. Your issue may have been fixed since the issue was open, however, if you are still experiencing a problem please re-open this issue and we will label it accordingly.


question

Hello, I am a beginner interested in embedded, can you add your contact information?

OK, I will add information in Project introduction.


Deprecate and plan to remove RowMetadata

After #9224 , RowMetadata interface in pinot-spi was identified to be redundant. This can be deprecated and removed. See https://github.com/apache/pinot/pull/9224#issuecomment-1255908191 for details.

Please assign this to me. Tks!


Disables Razor Intellisense in Visual Studio 2015

I installed this extension and it removed all Intellisense formatting for Razor CSHTML files. I uninstalled it and I still can't get the formatting back.

Unfortunately I think this may be due to something else.  I was unable to reproduce in a number of different configurations.


Updated Package Versions

updated package versions as there were compatibility issues when doing npm install
added new test case for fail condition with chai-as-promised
updated solidity version in the test token contract
minor prettier automatic changes (converted single quotes to double quotes)

Thank you for the contribution @kautukkundan
I have made some breaking changes, taking some ideas from your PR.
I've added a test for a failed transfer, but I used waffle built-in functionality instead of chai-as-promised.
Also update the dependencies, which required making some changes for typechain to work
Apologies for not taking your PR as-is, there were some additional changes I wanted to make and it was easy to only do them once.
Please feel fee to make additional PRs and I'll do my best to merge :)


Request new module: wazi-dev-spaces

Related user story
https://github.com/cloud-native-toolkit/software-everywhere/issues/275
Type of module
gitops
If requesting a terraform module, what is the target cloud platform?
n/a
What is the purpose of the module?
Deploy Wazi Dev Spaces into an OpenShift cluster
What is the category of the module?
middleware

released


Disable TLS 1.3 in the C-core.

This PR is a minimal change needed to disable TLS 1.3 in the gRPC C-core. This PR leaves all of the plumbing and fixes that were necessary for TLS 1.3, and disables it by ignoring any min/max TLS version that was selected by the SSL options. Note that setting the min/max TLS version is not currently user visible (it was to be made visible in PR #23527).
TLS 1.3 was enabled in PR #23165 (and an additional test case was added in #23413).

@jtattermusch This will need to be back-ported to the 1.31 branch before the release.

Thanks for the very quick review @markdroth! I'll merge as soon as all tests are passing.
@cesarghali @jiangtaoli2016 FYI.


Decrease the number of processors when compiling ArborX for the CI

That fixed the CI issue. Since we are never waiting on the host-only tests, reducing the number of processors used for compilation doesn't increase the time it takes to get back the results from the CI.

HIP build did not start, SYCL failed to build the container. Ignoring.


Track2 Management SDK for StackHCI

All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

Please add a ci.yml(example) file to trigger track2 validation.

Please generate an api file by running ./eng/scripts/Export-API.ps1 azurestackhci. We will mainly review the api from there for this PR.

/azp run prepare-pipelines

/azp run net - azurestackhci - ci

/azp run prepare-pipelines

/azp run net - azurestackhci - ci


Feature request: Allow inheriting files from parent templates

At the moment, children templates can inherit values in the parent Values.yml file if that file includes the value, prefixed by the chart name. For example,
mysql:
    mysqlRootPassword: herapderpa # This overrides the mysqlRootPassword on the top level of its Values.yml

There is no such inheritance allowed for files. It appears that values are being explicitly passed down the stack in the following snippet:
https://github.com/kubernetes/helm/blob/c8af660b181daf4000aca87e19e583fd1abec0b3/pkg/engine/engine.go#L217-L222
It would be good for files to be passed down the stack also, though there is no convention for doing as such. One suggestion might be to create a folder with the name of the child chart, and inherit the files supplied in that folder. For example,
mysql/
    mysql.conf

In the case above, the mysql.conf file would override the mysql.conf file in the root of the mysql chart, stored in charts/ (note: don't go looking, I made that up)
Thaanks!

I think we could do this, provided we have a good convention. Maybe along side charts/ we have something like an overrides/ directory that could contain information like this. It might get tricky. For example, how do I override mychart/charts/wordpress/charts/mysql/my.cnf but not mychart/charts/drupal/charts/mysql/my.cnf?

If I was to choose an API, I would try and model it after the inheritance behaviour of .Values.yml. So, instead of
overrides/my.conf

You would have
overrides/drupal/mysql/${my.conf
overrides/wordpress/mysql/my.conf

So, the format would be something like
overrides/${CHART_A}/${CHART...N}/${PATH_TO_FILE}

If I understand correctly, .Values.yml inherits along a simliar syntax
---
# child
foo: bar

# parent
child:
  foo: baz

@andrewhowdencom Closing this issue due lack of activity for 6 months. Please feel free to re-open  or create a new proposal.

Hola @jascott1; I think this is still valid -- A pattern of embedding files in the yaml has started to emerge, which is not so elegant:
https://github.com/kubernetes/charts/blob/master/stable/prometheus/values.yaml#L545
I just don't have the skills to implement it (:
Cannot reopen.

Re-opening based on OP request.

I would like to see this as well. A pattern would be to override config files, but not override the Kube templates. I'm now adding support for splunk in the fluent-bit chart, and I need to copy the whole chart to do it.
/reopen
/remove-lifecycle rotten

/reopen /remove-lifecycle-rotten
Mostly I've been editing files as part of Values.yml but it's kind of nasty

cc @technosophos  directly, as I think the bot is misbehaving

we're no longer part of the kubernetes org, so slash comments and the stale issue bot have been disabled for the time being.

FYI, this is being addressed as part of the Helm 3 proposals through the extension model: https://github.com/helm/community/blob/master/helm-v3/001-charts.md#the-ext-directory

@bacongobbler I really need it on Helm v2 do you any work-around to let my child chart inherit files from the parent templates ?

@HamzaK8s how about using tpl functions? See example in stable/keycloak

I'm having this issue also, and can't find a good solution to it rather doing something hacky such providing a ConfigMap from the parent chart and let the subchart use that.
Anyone has a solution ?

FYI, this may be addressed as part of the Helm 3 proposals through the extension model: https://github.com/helm/community/blob/master/helm-v3/001-charts.md#the-ext-directory
If you have feedback on that particular model we'd love to hear it!

Now that Helm 3 is released, I'm trying to do this with a library chart and it doesn't seem to be working.  I still get the error:
at <.Files.Glob>: nil pointer evaluating interface {}.Glob

Was the 'hoisting' thing mentioned here (https://github.com/helm/community/blob/master/helm-v3/001-charts.md#the-ext-directory) implemented?

Library charts were implemented, yes. @hickeyma gave a talk about this feature at Helm Summit EU. You can find the recording on YouTube if you want to learn more about it.

Is there a document somewhere showing how to do this with Helm 3? It still seems to not be possible.

FYI this issue is documented in HELM with no resolution. If grafana is a sub-chart it can't get files from the parent chart. There are some ugly work-arounds but I personally think the easiest one is to make them config maps and then link in the values file.
https://github.com/helm/helm/issues/1892

I think this is still of interest.

I would like to see this as well. The only way to add files to a subchart seems to be embedding them into the values.yaml file, which becomes very unwieldy for big files, e.g. grafana dashboards: helm/charts#1079
The parent chart would define the dashboards in overrides/grafana/dashboards and the grafana child chart could pull them into a configmap like this:
data:
{{- (.Files.Glob "dashboards/*.json").AsConfig | indent 2 -}}

/remove-lifecycle stale

Something like this would be very helpful to define a configuration model for applications based on Helm charts that wouldn't require actual knowledge about Helm for developers. Application configuration would be modelled as a simple chart with values and the actual configuration files, with a dependency to a chart packaging the templates that define all required Kubernetes objects for deployment, including ConfigMaps and Secrets. Not having this inheritance for files implemented requires packaging the ConfigMap and Secret templates on each application configuration chart for this to work, which leads to code repetition and is a bit messy.
And I agree that embedding all configuration files in the values.yaml is not a suitable solution for this, as developers would be forced to mess with a different format instead of working with the type and format for files they are used to.
I'm really interested in this feature being implemented.

I am pretty sure this still applies. I am unable to find a solution for this and I believe this is a common pattern.
In my case I want to have a template in a subchart that renders configmaps with content of a directory on the parent chart. Any suggestions without embedding the files on the values of the parent or adding code to the parent are more than welcome.

I found a way around this limitation for anyone interested. We were trying to render files from a parent and child into the same config map.
In the parent (we will call  it parent in the example) in a non-underscored file, so for example "templates/blah.yaml"
{{- $_ := set .Values "renderedfiles" (($.Files.Glob "files/**").AsConfig ) -}}

this renders the files to a string and stores it in .Values.renderedfiles
Then in the child chart, you can do something like this
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: myconfigmap
  namespace: default
data:
# from base
{{ .Values.parent.renderedfiles | indent 2 }}
# from local (child) context
{{ ($.Files.Glob ""files/**" ).AsConfig | indent 2 }}

Hello Brian, Could you please give the helm version you use to do the trick ? Did you get it work using a binary file ? ( using b64enc i guess)
thank you Bruno

version.BuildInfo{Version:"v3.5.3", GitCommit:"041ce5a2c17a58be0fcd5f5e16fb3e7e95fea622", GitTreeState:"dirty", GoVersion:"go1.16"}
We were were merging source code files into a single configmap from both the parent and child. Not 100% how this would work with binaries. I am guessing as long as the render into variable didn't try to mangle the encoding it should work? I mean if .AsConfig  works for you today it should work when rendering into a variable unless there is some encoding/decoding that Helm would do implicitlly.

This is still a problem and the work around of @brian-groux-hs does not work for us.
We for now fallback to the workaround the the parent chart renders the needed files into config-maps.

Unfortunately @brian-groux-hs 's example does not work when a library chart is involved either, because library charts are not permitted to execute templating code and $.Files.Get runs in the context of the calling chart (not the subchart / library chart where I want it to). It would be very nice to be able to embed files into the "defining chart"'s context.
As it is, my workaround will likely be to generate an out-of-helm helper that embeds files in helper functions (appending define, the file and end together, for instance)

Wanted to share the solution we came up with in case it is useful to anyone else. Objective is to embed files inside of a library chart so that they can be easily used and referenced in parent charts. Technically the inverse of what is requested in this issue, but related in some fashion.
Specifics:

use a script / helper / etc. (we used just, which took a bit of escaping). This creates a file that looks like a usual .tpl

# example of a single line... weird escaping is a Justfile convention
echo {{ '{{- define \"my-chart.b64.job\" -}}' }} >> $FILEPATH

Inline and Base64 encode the input files (to protect against escaping / etc.). These blobs can then be referenced by parent charts

echo {{ '{{- define \"my-chart.b64.job\" -}}' }} >> $FILEPATH
cat $INCLUDE_DIR/job.tpl | base64 >> $FILEPATH
echo {{ '{{- end }}' }} >> $FILEPATH

Test and ensure that the desired output is being generated

{{- define "my-chart.b64.job" -}}
IyBWZXJzaW9uOiAxCmFwaVZ...
{{- end }}

Define the user-facing helper that base64 decodes the data:

{{- define "my-chart.job" -}}
  {{- include "my-chart.b64.job" nil | b64dec }}
{{- end }}

Call this template helper if/when needed in parent charts to inject the desired files (i.e. {{ include "my-chart.job" nil }})

I'm still looking for this as well. The config map pattern of including files works best when using {{ Files.get "" }}, and so this only works if it's in a top level chart.

I found a way around this limitation for anyone interested. We were trying to render files from a parent and child into the same config map.
In the parent (we will call it parent in the example) in a non-underscored file, so for example "templates/blah.yaml"
{{- $_ := set .Values "renderedfiles" (($.Files.Glob "files/**").AsConfig ) -}}

this renders the files to a string and stores it in .Values.renderedfiles
Then in the child chart, you can do something like this
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: myconfigmap
  namespace: default
data:
# from base
{{ .Values.parent.renderedfiles | indent 2 }}
# from local (child) context
{{ ($.Files.Glob ""files/**" ).AsConfig | indent 2 }}

@brian-groux-hs I'm trying to use your approach above. I can't get the .Values.parent to work in the child - I assume 'parent' is the 'name:' supplied in the parent's Chart.yaml. I don't think this works - the only values from parent that I can see in child are 'global:' ones. But I can't see how to get your syntax for setting values to work for global.
Please could you share the details of the parent that you got this working with?
Thanks!

@brian-groux-hs I would also love to see a reproduction of that. As you can see a few out here would love to adopt this. Passing files to subcharts is discussed in other issues as well.
According the docs, and my experience, this is true:

For that reason, a subchart cannot access the values of its parent.

Source: https://helm.sh/docs/chart_template_guide/subcharts_and_globals/
I did some digging and found this statement:

Helm works by pulling in a chart and all the dependencies into a global namespace and then renders all of the templates there. The first time a template is defined it is the one used everywhere. This is one of the things that enables library charts. This is what is causing you a problem.

Source https://github.com/helm/helm/issues/8804#issuecomment-700733687
Which made me belive, if the order of the files would be correct, this might work.
But no matter how hard I drop a and 0 before stuff, when looking at the output of helm template... it seems like subcharts are rendered before the parent, so this wouldn't work.

We have kinda similar thing and I currently solve it by passing the files via --set-file and some wild bash one liner
$(for i in $(ls config/); do echo -n \ --set-file mySubChart.files.${i//./\\.}=./config/$i;done)"


Process Data Missing on Log Page

The log page is currently missing the process data which should populate for the instance that was clicked.

app data populating on the logs page
commit: a48e96d8f066b6c13b895e95119a288f10ac7e75


hail and heavy rain

Just a quick point we live in south Florida system says hail when it is heavy rain ,,,

The rain / hail sensor is direct from the tempest and not a calculated point.
Out of curiosity do you have the rain rate when you see this?
What does the WeatherFlow app say about the precipitation type?

the app  says heavy and yes it is raining lol......hard

Can you post screen shots of the app and what you get in HA?
Might have to post in either the HA or WF forums for the pics (easier)

I will ......

I grew up in central Florida, everyday at 1500 (3pm) it would rain for about 15 mins.
I the app says heavy that means it is getting a '3' from the Tempest which is either Rain+Hail or Heavy Rain.  But a '2' is hail.
WF did just announce they pushed some changes out yesterday.
I will look to see if they updated the UDP API guide and see there are changes.

i am on 170 on the hub and 156 on the unit

They did not post any updates to the API.  What is weird is that you get Hail in HA but heavy in the app.  Only mention of a change regarding rain is the image in the app.
Post the shots when you get them.
Is this a common thing, recent, or just noticed?

No it's been since I've been trying the program it's never been correct

I was able to grab these it just happened as you can see tempus says heavy

The data we get from the station is a Bit value. From the UDP Documentation:
0 = none, 1 = rain, 2 = hail, 3 = rain + hail (experimental)
I renamed the bitvalue 3 to Heavy Rain, based on pure observation from my Window - every time I got a bit value 3, it was not hailing, but Heavy Rain. Happy to modify the text?

I believe they use the word heavy to cover all basis not heavy rain thats all I was trying to show thank you for the change since we live in south Florida hail is not very often but it does happen lol......

Thanks. I will rename it to Heavy (without the Rain) Makes sense to me, and covers all bases.


Not pulling Correct Data

When we run this script, a good amount of our accounts, about 2/3, are returning the last sign in value as N/A. However, these are definitely folks who have signed in very recently. As we spot check the N/A entries, every one we've checked has signed in within the last 24 hours.

Same thing is hapening for me. I have been using this script for a few months now just fine! I will use this script to generate a CSV file and will narrow it down to Sign-in greater than or equal to 60 days and send that out to stakeholders to get some buy-in to delete accounts for inactive users.
Worked fine last month. This month, I am now getting a LOT of N/A for brand new users, users who for sure have signed in. I see in Azure AD a date for Last Sign-In. The weird part about it is that these N/As are happening on new users who have started within the last week or two. All data prior to that appears to be 100% legit and good.
Did MS change anything?
Also, FWIW, I went into the .ps1 and changed "beta" to "v1.0" just to see if that would fix it, but no dice.
Help us Mohammad Zmaili!!!


idk how to do this

Describe your problem here.

remove the health text
change the Screwed Engine text on the bottom

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x86/x32
Did you edit anything in this build? If so, mention or summarize your changes.
nope

That’s possible
That’s not possible, unless you credit us in the credits state

if you follow the rules, I’ll give you the code

ok not 2 then

In the create() (in the principal lines), put this to deactivate the health text:
ClientPrefs.showHealth = false;

And that’s it