Unable to exclude nested fields with jsonable_encoder

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Example
Here's a self-contained, minimal, reproducible, example with my use case:
from typing import List

from fastapi.encoders import jsonable_encoder
from pydantic.main import BaseModel

class YourBaseModel(BaseModel):
    bar: int
    baz: int

class MyBaseModel(BaseModel):
    nest: List[YourBaseModel] = None
    foo: int

model = MyBaseModel(nest=[YourBaseModel(bar=2, baz=3), YourBaseModel(bar=4, baz=5)], foo=1)
exclude_pattern={'nest': {'__all__': {'baz'}}}
expected={"nest":[{"bar": 2}, {"bar": 4}], "foo": 1}

def test_pydantic_nested_exclude():
    assert model.dict(exclude=exclude_pattern) == expected # Passed

def test_jsonable_encoder_nested_exclude():
    assert jsonable_encoder(model, exclude=exclude_pattern) == expected # Failed
    # actual={'foo': 1} ## i.e. removes all nested elements!

Description

Define a nested model e.g. {"nest":[{"bar": 2, baz=3}, {"bar": 4, baz=5}], "foo": 1}
Try to exclude one of the fields in the nested model (e.g. baz) using jsonable_encoder
It excludes the whole nested field altogether i.e. nest and returns {'foo': 1}
But I expected it to return {"nest":[{"bar": 2}, {"bar": 4}], "foo": 1}.

Environment

OS: Windows
FastAPI Version: 0.65.1
Python version: 3.8.2

Additional context

I followed along with issue #2594 and it said the behaviour explained in https://pydantic-docs.helpmanual.io/usage/exporting_models/#advanced-include-and-exclude should work but this doesn't seem to be the case.
What am I doing wrong? :<
Thanks in advance.

Unfortunately, you seem to hit the bug #2007. There is a PR to fix it (#2016), but has not been merged yet.

Looks like it's been merged as of v0.66.0

DONT WORK: response_model_include={..., ('is_api_user', 'order_client_c')}
DUT WORK response_model_include={..., *('is_api_user', 'order_client_c')}

('is_api_user', 'order_client_c')

rersponse_model_include should be SetIntStr or DictIntStrAny or None, so I think
this

response_model_include={..., ('is_api_user', 'order_client_c')}

could not work. cause ('is_api_user', 'order_client_c') is an anonymous Set object which contains str.
And the latter one

response_model_include={..., *('is_api_user', 'order_client_c')} can works, this unpacks the str values in the Set object.

IMO, they are not the same thing.

And also, I would like to put a complete example here.

import typing as t

from fastapi import APIRouter, FastAPI
from pydantic import BaseModel, Field

class Base(BaseModel):
    a: int = Field(default=0)
    b: str = Field(default="None")

    class Config:
        orm_mode = True

class ListBase(BaseModel):
    c: t.List[Base]

app = FastAPI(title="test", docs_url="/api/docs", openapi_url="/api")
r = APIRouter()

@r.get("/test",
       response_model=ListBase,
       response_model_exclude={"c": {
           "__all__": {"b"}
       }})
async def test_exclude():
    results = {"c": [{"a": 1, "b": "aa"}, {"a": 2, "b": "ba"}]}
    return results

app.include_router(r, prefix="/api")

Output:

{
"c": [
{
"a": 1
},
{
"a": 2
}
]
}


Changing font should also change chat input font (#93)

Could do with a review first before merging.

make a new pr whenever you feel like it's ready for the review


SQLDatabase : Remove set search_path (or rewrite it)

as you have now created a specific dialect pr https://github.com/hwchase17/langchain/pull/2748
you better remove these lines or make them only applicable if the dialect is sqlite,
most of the dialects don't support this
https://github.com/hwchase17/langchain/blob/b634489b2e8951b880c2ec467cdcf00f11830705/langchain/sql_database.py#L218-L219
ps i have to set this value to None once it instantiate sqldatabase otherwise i run in trouble. (i'm using ibm_db_sa. dialect and it works as a charm with eg chatgpt
i also think there are some related tickets around this
ps i set my schema in the connection string
but i still have to set it in the model (otherwise it cannot find my included tables
but this might be another problem
thanks

I'm hitting this regularly too and applying the workaround, but would love to see it fixed. FWIW this syntax does not work with Spark SQL either.

Hi @srowen,
even I am facing issue due to this command while executing the query on Azure SQL database, what is the workaround that helped you?

Set ._schema=None

Same issue when querying AWS Athena.

Same issue when using Trino.  I get this when the following is executed:
                else:  # postgresql and compatible dialects
                    connection.exec_driver_sql(f"SET search_path TO {self._schema}")

This could be wrapped in a try-catch block and continue on failure instead of raising the error.

i set this value to None once it instantiate sqldatabase,


Migrate issue with no files table

This is  a **bug **.

Prerequisites

[X] Are you running the latest version?
[X] Are you reporting to the correct repository?
[X] Did you check the documentation?

Description
Migrating: 2017_01_01_100000_create_how_to_videos_table
Illuminate\Database\QueryException
SQLSTATE[HY000]: General error: 1824 Failed to open the referenced table 'files' (SQL: alter table how_to_videos add constraint how_to_videos_file_id_foreign foreign key (file_id) references files (id) on delete cascade on update restrict)
at vendor/laravel/framework/src/Illuminate/Database/Connection.php:712
708▕         // If an exception occurs when attempting to run a query, we'll format the error
709▕         // message to include the bindings with SQL, which will make this exception a
710▕         // lot more helpful to the developer instead of just the database's errors.
711▕         catch (Exception $e) {
➜ 712▕             throw new QueryException(
713▕                 $query, $this->prepareBindings($bindings), $e
714▕             );
715▕         }
716▕     }
  +43 vendor frames

44  artisan:37
Illuminate\Fo

Curtis, please wait for v5 release notes / changelog / upgrade steps before diving in core 8.x

Thank you.


[tricore] Fix usage of SLEIGH abs()

Received a report from @esaulenka and it looks like during my implementation I completely misused the abs() SLEIGH instruction.  I used abs() with integer, expecting integer results, which is incorrect.
I replaced the abs() calls with & 0x7fffffff / & 0x7fff / & 0x7f ... is that sufficient or should it be logic similar to x = x < 0 ? 0 - x : x

No-no-no, you misunderstood how works negative integers.
If we consider 8-bit values,  -2  is not  0x82, it is 0x100 - 2 = 0xfe.
I suggest to use macro
macro int_abs(value) {
  if (value s>= 0) goto <non_neg>;
    value = - value;
  <non_neg>
}

Yeah, this definitely needs to be fixed. I think the macros you defined look pretty good. I'll do a quick review of this and make sure everything is solid. Thanks!


Merge new features and modifications for next version.

This merges new features including ChaosGizmo etc. Plus, the organization of the entire package code.

Codecov Report
Attention: Patch coverage is 64.63816% with 215 lines in your changes are missing coverage. Please review.

Project coverage is 73.30%. Comparing base (fad5a16) to head (866b2f3).

Files
Patch %
Lines

src/ChaosGizmo/LyapunovExponent.jl
74.47%
49 Missing :warning:

src/operators/cubic.jl
58.40%
47 Missing :warning:

src/learn.jl
66.66%
29 Missing :warning:

src/utilities/unique_kronecker.jl
34.21%
25 Missing :warning:

src/model/KS.jl
50.00%
18 Missing :warning:

src/intrusiveROM.jl
22.22%
14 Missing :warning:

src/model/Burgers.jl
6.66%
14 Missing :warning:

src/operators/bilinear.jl
0.00%
8 Missing :warning:

src/analyze.jl
0.00%
5 Missing :warning:

src/operators/quadratic.jl
83.33%
4 Missing :warning:

... and 2 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
- Coverage   78.36%   73.30%   -5.07%
==========================================
  Files          13       22       +9
  Lines        1470     1993     +523
==========================================
+ Hits         1152     1461     +309
- Misses        318      532     +214

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


Fix ensure_tuple() to work with all Sequence types

I may have misunderstood the assignment, but attempting to address #640 and #510.
Essentially:

Added a check for range objects in ensure_tuple() since these were the last type of Sequence we did not support.
Changed T | Tuple[T] | List[T] in Many to T | Sequence[T]
Changed docstrings and instances of T | Tuple | List  to Many

If this actually looks right, I will add tests for ensure_tuple and every sequence type.
Side note: I'm a little confused when to use PytorchSchedulers vs ComposerSchedulers

There's some additional typing stuff will need to be adjust in iter_helpers.pyi -- specifically we'll need an additional overload methods to take a generic sequence and for str, bytes, and range objects.

Side note: I'm a little confused when to use PytorchSchedulers vs ComposerSchedulers

Good point; this will need to be clarified in the docs somewhere. At a high level, ComposerSchedulers are acceptable to use when constructing the trainer, but they are converted to PyTorchSchedulers which are used internally. (ComposerSchedulers simply return the learning rate factor and don't update the optimizer; instead, the trainer "compiles" them into a PyTorch LambdaLR scheduler)

I think I addressed most comments, but I'm not very familiar with interfaces so it's likely a set that up wrong 😅.
I will start making tests now
I have a couple of docstring consistency questions that maybe should be addressed somewhere else:

Should I remove the :attr:from types in the state docstring (specifically inState`)? They don't seem necessary, but not sure if that is the appropriate format. The dataloaders don't seem to have these.
Should we use the aliases Tensor and Optimizer in docstrings? To me, it adds an extra click to figure out the actual type with little benefit.
Should Algorithms and Callbacks have the same type format as optimizers and schedulers i.e. they accept a single object or a sequence of objects? Right now, they are Sequence[Algorithms]
To align with removing plurals, should Tensors -> Tensor | Sequence[Tensor] in docstrings?

Errr, there are a lot of Sequence object errors now... For example:
    first_optimizer = state.optimizers[0]
    expected_param_groups = deepcopy(first_optimizer.param_groups)

The second line gets error: Cannot access member "param_groups" for type "Sequence[Optimizer]". Do these need to be manually cast?

I think I addressed most comments, but I'm not very familiar with interfaces so it's likely a set that up wrong sweat_smile.
I will start making tests now
I have a couple of docstring consistency questions that maybe should be addressed somewhere else:

Should I remove the :attr:from types in the state docstring (specifically inState`)? They don't seem necessary, but not sure if that is the appropriate format. The dataloaders don't seem to have these.

It depends on what is being referenced; sometimes, :attr: is needed

Should we use the aliases Tensor and Optimizer in docstrings? To me, it adds an extra click to figure out the actual type with little benefit.

Good point; let's go with the torch types. I'd eventually like to remove types.py or make it significantly leaner.

Should Algorithms and Callbacks have the same type format as optimizers and schedulers i.e. they accept a single object or a sequence of objects? Right now, they are Sequence[Algorithms]

Yes, let's do the "singleton or sequence" approach.

To align with removing plurals, should Tensors -> Tensor | Sequence[Tensor] in docstrings?

Yup!

Errr, there are a lot of Sequence object errors now... For example:
    first_optimizer = state.optimizers[0]
    expected_param_groups = deepcopy(first_optimizer.param_groups)

The second line gets error: Cannot access member "param_groups" for type "Sequence[Optimizer]". Do these need to be manually cast?

It shouldn't need a manual cast; double check the type annotation for ensure_tuple.

@Landanjs I pushed a commit that (hopefully?) fixes the type errors with the ensure_tuple change.
If you could finish up the docs fixes in this PR, that'd be awesome.

The recent change tries to:

Use PyTorch's Tensor, Optimizer, Module classes where possible
Remove Tenors
Attempts to fix missing docstring links by prepending types.
Remove the illusion of multiple optimizer from docstrings

I did not convert Callbacks and Loggers to "singleton or sequence" format since these are converted into lists instead of tuples and I'm not confident the functionality is the same? But maybe should be changed in the future

There appear to be a few docstring formatting warnings that need to be fixed (Docstring warnings are treated as errors and cause the build to fail).
00:04:06.043  /home/mosaicml/.local/lib/python3.9/site-packages/composer/trainer/trainer.py:docstring of composer.trainer.trainer.Trainer:39: WARNING: Field list ends without a blank line; unexpected unindent.
00:04:06.043  /home/mosaicml/.local/lib/python3.9/site-packages/composer/trainer/trainer.py:docstring of composer.trainer.trainer.Trainer:42: WARNING: Inline substitution_reference start-string without end-string.
00:04:06.043  /home/mosaicml/.local/lib/python3.9/site-packages/composer/trainer/trainer.py:docstring of composer.trainer.trainer.Trainer:44: WARNING: Definition list ends without a blank line; unexpected unindent.


Source IP to upstream Requests changed from <IP_ADDRESS> to POD_IP

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
Hi,
I'm not sure if this is a Bug in Istio, a Bug in Envoy or just new undocumented behavior.
After updating from 1.20.5 to 1.21.0 we observed different behavior of the istio sidecar, the source IP of  the connection to the sidecar/upstream service changed:

I read the Changelog but I did not see any changes regarding network interfaces or source IP for Sidecars.
Regards
Timo
Version
# Many Versions since I wanted to know when the change started to occur.

istioctl version
client version: 1.21.1
istiod version: 1.19.3
istiod version: 1.19.3
istiod version: 1.19.4
istiod version: 1.19.4
istiod version: 1.19.5
istiod version: 1.19.5
istiod version: 1.19.6
istiod version: 1.19.6
istiod version: 1.19.7
istiod version: 1.19.7
istiod version: 1.19.8
istiod version: 1.19.8
istiod version: 1.19.9
istiod version: 1.19.9
istiod version: 1.20.0
istiod version: 1.20.0
istiod version: 1.20.1
istiod version: 1.20.1
istiod version: 1.20.2
istiod version: 1.20.2
istiod version: 1.20.3
istiod version: 1.20.3
istiod version: 1.20.4
istiod version: 1.20.4
istiod version: 1.20.5
istiod version: 1.20.5
istiod version: 1.21.0
istiod version: 1.21.0
istiod version: 1.21.1
istiod version: 1.21.1
data plane version: 1.19.3 (44 proxies), 1.20.5 (2 proxies), 1.21.0 (1 proxies), 1.21.1 (11 proxies)

kubectl  version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.4", GitCommit:"fa3d7990104d7c1f16943a67f11b154b71f6a132", GitTreeState:"clean", BuildDate:"2023-07-19T12:20:54Z", GoVersion:"go1.20.6", Compiler:"gc", Platform:"darwin/amd64"}
Kustomize Version: v5.0.1
Server Version: version.Info{Major:"1", Minor:"28", GitVersion:"v1.28.5", GitCommit:"05f2dca83369807327c976b24846de8687923cb6", GitTreeState:"clean", BuildDate:"2024-04-12T23:22:53Z", GoVersion:"go1.20.12", Compiler:"gc", Platform:"linux/amd64"}

Additional Information
No response

Check out: https://github.com/istio/istio/commit/59f2cf1911d2d82409fea27e98eb2a8d382285be#diff-d6b1447634d5ccd60af75e578dfe03da0e2eb1e6c14d1dccabf1e6ffd84e57b9

This could be it. We did not deviate from the default settings here. I will check it out tomorrow.
Thanks!

Thank you for the fast feedback. With a change ingunicorn settings, this is not an issue anymore.


ollama download时下载的server地址是开源的吗？在国内感觉不好拉，想弄个类似的

我速度拉满, 100MB/S

Hi there, I'm not quite sure what you are asking for specifically, but models are downloaded from ollama.com - see this existing issue for local model hosting https://github.com/ollama/ollama/issues/914 - hope this helps!


Add kavya2816 file

This pull request adds kavya2816 to _post field

This is pull request from feature to main


Refactor how final_state and final_propagator are saved

Supersedes https://github.com/dynamiqs/dynamiqs/pull/682

Not sure why ci is failing, passing locally...

Not sure why ci is failing, passing locally...

Seems like its related to latest version of mkdocstrings-python. By upgrading I can reproduce locally even on main. We need to fix in a separate PR, not sure what's it's due to right now.

https://github.com/dynamiqs/dynamiqs/pull/694 fixes the CI, we can merge right after that's done :)

Thanks @gautierronan !


@ForeignKey support for @ModelView missing

That ModelView doesn't work:
@ModelView(database = MyDatabase.class)
public class WeekModelView extends BaseModelView<MyEntry> {

    @ModelViewQuery
    public static final Query QUERY = new Select(MyEntry_Table.task_id, MyEntry_Table.name, MyEntry_Table.id)
            .from(MyEntry.class)
            .groupBy(MyEntry_Table.task_id);

    @Column
    public long id;

    @Column
    public String name;

    @ForeignKey
    public ForeignKeyContainer<Task> task; // not loaded. Is always null

}

Where MyEntry is pretty simple defined as
@ModelContainer
@Table(database = MyDatabase.class)
public class MyEntry extends BaseModel {
    @PrimaryKey(autoincrement = true)
    public long id;

    @Column
    public String name;

    @ForeignKey(onDelete = ForeignKeyAction.SET_NULL)
    public ForeignKeyContainer<Task> task;

}

As I'm new to this I'm not quite sure if it's actually missing or if I made a mistake.

Views in SQLite cannot have @ForeignKey, if you need a field that matches it, you will need to make it a primitive @Column

I am updating documentation, as well as some other major changes that should be merged into develop and released for next beta by End of Week


from lib import action   File "/opt/stackstorm/packs/aws/actions/lib/action.py", line 107     except Exception, e:                     ^ SyntaxError: invalid syntax

SUMMARY
st2 run command throwing syntax errors .
Provide a quick summary of your bug report.
STACKSTORM VERSION
Paste the output of st2 --version:
st2 --version
st2 3.4.1, on Python 3.6.13
OS, environment, install method
vagrant box
StackStorm v3.4.1 (Ubuntu 16.04 LTS GNU/Linux x86_64)
st2vagrant
Steps to reproduce the problem
Show how to reproduce the problem, using a minimal test-case. Make sure to include any content
(pack content - workflows, actions, etc.) which are needed to reproduce the problem.
st2 pack install aws
updated aws.yml file with keys from IAM role
excuted st2 run aws.ec2_get_all_tags
Expected Results
get list of ec2 tags
Actual Results
st2 run aws.ec2_get_all_tags
.
id: 60b4be16a161a571fbd73de6
action.ref: aws.ec2_get_all_tags
context.user: st2admin
parameters: None
status: failed
start_timestamp: Mon, 31 May 2021 10:44:38 UTC
end_timestamp: Mon, 31 May 2021 10:44:39 UTC
result:
exit_code: 1
result: None
stderr: "Traceback (most recent call last):
File "/opt/stackstorm/st2/lib/python3.6/site-packages/python_runner/python_action_wrapper.py", line 238, in _get_action_instance
actions_cls = action_loader.register_plugin(Action, self._file_path)
File "/opt/stackstorm/st2/lib/python3.6/site-packages/st2common/util/loader.py", line 166, in register_plugin
module = imp.load_source(module_name, plugin_abs_file_path)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/imp.py", line 172, in load_source
module = _load(spec)
File "", line 684, in _load
File "", line 665, in _load_unlocked
File "", line 678, in exec_module
File "", line 219, in _call_with_frames_removed
File "/opt/stackstorm/packs/aws/actions/run.py", line 1, in
from lib import action
File "/opt/stackstorm/packs/aws/actions/lib/action.py", line 107
except Exception, e:
^
SyntaxError: invalid syntax
Thanks!

It appears you are using an old version of the aws pack that was written for Python2, but you are running the pack using Python3.  Python2 is no longer supported and all packs were updated for Python3.  You should upgrade your pack to the latest version (v2.0.0 at the time of writing).
As a side note: You should consider running StackStorm on a new version of Ubuntu as v3.5 of StackStorm will not be available on Ubuntu 16.04, only Ubuntu 18.04 and 20.04.

`st2 pack install aws=2.0.0
    [ succeeded ] init_task
    [ succeeded ] download_pack
    [ succeeded ] make_a_prerun
    [ succeeded ] get_pack_dependencies
    [ succeeded ] check_dependency_and_conflict_list
    [ succeeded ] install_pack_requirements
    [ succeeded ] get_pack_warnings
    [ succeeded ] register_pack

+-------------+--------------------------------------------------------------+
| Property    | Value                                                        |
+-------------+--------------------------------------------------------------+
| ref         | aws                                                          |
| name        | aws                                                          |
| description | st2 content pack containing Amazon Web Services              |
|             | integrations.                                                |
| version     | 2.0.0                                                        |
| author      | StackStorm, Inc.                                             |
+-------------+--------------------------------------------------------------+`
able to clear that error
st2 run aws.ec2_get_all_tags
.
id: 60b4e185a161a571fbd74c18
action.ref: aws.ec2_get_all_tags
context.user: st2admin
parameters: None
status: failed
start_timestamp: Mon, 31 May 2021 13:15:49 UTC
end_timestamp: Mon, 31 May 2021 13:15:51 UTC
result:
exit_code: 1
result: None
stderr: "Traceback (most recent call last):
File "/opt/stackstorm/st2/lib/python3.6/site-packages/st2common/runners/utils.py", line 134, in get_action_class_instance
action_instance = action_cls(**kwargs)
TypeError: init() got an unexpected keyword argument 'action_service'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/opt/stackstorm/st2/lib/python3.6/site-packages/python_runner/python_action_wrapper.py", line 334, in
obj.run()
File "/opt/stackstorm/st2/lib/python3.6/site-packages/python_runner/python_action_wrapper.py", line 192, in run
action = self._get_action_instance()
File "/opt/stackstorm/st2/lib/python3.6/site-packages/python_runner/python_action_wrapper.py", line 261, in _get_action_instance
action_service=action_service)
File "/opt/stackstorm/st2/lib/python3.6/site-packages/st2common/runners/utils.py", line 143, in get_action_class_instance
action_instance = action_cls(**kwargs)
File "/opt/stackstorm/packs/aws/actions/lib/action.py", line 67, in init
self.account_id = self.session.client('sts').get_caller_identity().get('Account')
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/client.py", line 251, in _api_call
return self._make_api_call(operation_name, kwargs)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/client.py", line 526, in _make_api_call
operation_model, request_dict)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/endpoint.py", line 141, in make_request
return self._send_request(request_dict, operation_model)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/endpoint.py", line 166, in _send_request
request = self.create_request(request_dict, operation_model)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/endpoint.py", line 150, in create_request
operation_name=operation_model.name)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/hooks.py", line 227, in emit
return self._emit(event_name, kwargs)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/hooks.py", line 210, in _emit
response = handler(**kwargs)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/signers.py", line 90, in handler
return self.sign(operation_name, request)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/signers.py", line 147, in sign
auth.add_auth(request)
File "/opt/stackstorm/virtualenvs/aws/lib/python3.6/site-packages/botocore/auth.py", line 316, in add_auth
raise NoCredentialsError
botocore.exceptions.NoCredentialsError: Unable to locate credentials
"
stdout: ''
Unable to locate credentials is the location expected opt/stackstorm/aws/aws.yml not able to pick from there


Alter plot output before display

I want to combine a holoviews plot with an existing svg image, such that in my layout A is a drawing, and B and C are plots. In order to obtain this I propose another hook: before_display_hook, where I can alter the raw data.

As described in #433 we've now added post_render_hooks allowing you to specify post-processing functions per output format. I'll close this issue now.


ci: deploy app in actions

fix in https://github.com/kokic/UniTeX/commit/0c274830951a0c53eb4bf9fbb3cc45b2c0bfc50e


Improvements

If power-connection to LiveU is not 'perfect', and LiveU sometimes gets disconnection from power-source and back on, it will spam the chat like:
LiveU: Now charging
LiveU: Fully charged
LiveU: Now charging
LiveU: Fully charged
etc.
Option: Possibility the disable 'charge status', to disable the notifications above, but still keep percentage-status notifications. So it will not notify anything before eg. 50% if that' s the first warning-level in config.

Hi @DigibitDK ,
The battery status gets checked every 10 seconds, I could change that into a configurable setting.

Hi @715209 :)
Not sure that will do the trick. Of course it will get rid of spamming, but it will still (under the right circumstances) post 'Now charging' and 'Fully charged' in chat from time to time i guess. But it might be enough to be acceptable.
If its the easiest fix, lets try it out :)


Slides displayed are semi-transparent

I tried implementing both Simple Slide and even Custom Slide. I both cases, the displayed Slide appears semi-transparent. Here is the image for Reference.

There is nothing different from your Code
public class TourActivity extends MaterialIntroActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        enableLastSlideAlphaExitTransition(true);

        addSlide(new TourOneFragment());
    }

    @Override
    public void onFinish() {
        super.onFinish();
        startActivity(new Intent(this, MainActivity.class));
    }
}

Am I missing something?
Regards

It will be fixed in future releases. Right now please add another fragment and the issue will be gone.

I have a question, do you use 0.0.5 version of the library?

@bezmian sorry for the late reply. My City was blacked out for the past 10 days (no power, no internet) due to recent Hurricane. Hence no way to reply you. To answer your question, yes I am using the 0.0.5 version of the library.

Also got this problem. For now the workaround is to add a stub Fragment?

You tried changing the background color?

I'm seeing this too, even with a background color explicitly set using SlideFragmentBuilder().backgroundColor().

Fixed in https://github.com/TangoAgency/material-intro-screen/pull/72/files


Move metric definitions to ContainerMetrics.java

... or @gjoranv

hm - PR build failed but not able to open the PR run

Unrelated failure, a known issue in clustercontroller-core. Job has been restarted.


Release 0.4

Coverage remained the same at 81.103% when pulling a78ac931a85c2d5ab26a2733d91984fb9ff7f06a on master into b186daf0fbbe8f4f2daf5ec03895dfea89224810 on develop.


Revert TokenNetworkUtils library

Fixes https://github.com/raiden-network/raiden-contracts/issues/1503
Fixes #1434
Fixes #1435
Revert the addition of the TokenNetworkUtils library
This eases

deployment and the code in this package
usage and verification of the contracts in other packages (raiden itself)

Contract size changes
Those seem mostly related to the disabled yul optimizer. But we add it still fits we can add those back later.
Before:
running compile_contracts
TokenNetworkUtils 896 3.6458333333333335%
TokenNetwork 17217 70.05615234375%
TokenNetworkRegistry 20515 83.47574869791666%

After:
running compile_contracts
TokenNetwork 19055 77.53499348958334%
TokenNetworkRegistry 23336 94.95442708333334%

What this PR does
Why I'm making this PR
What's tricky about this PR (if any)

Any reviewer can check these:

[ ] If the PR is fixing a bug or adding a feature, add an entry to CHANGELOG.md.
[ ] If the PR changed a Solidity source, run make compile_contracts and add the resulting raiden_contracts/data/contracts.json in the PR.
[ ] If the PR is changing documentation only, add [skip ci] in the commit message so Travis does not waste time.

[ ] But, if the PR changes comments in a Solidity source, do not add [skip ci] and let Travis check the hash of the source.

[ ] In Python, use keyword arguments
[ ] Squash unnecessary commits
[ ] Comment commits
[ ] Follow naming conventions

solidityFunction
_solidity_argument
solidity_variable
python_variable
PYTHON_CONSTANT

[ ] Follow the Signature Convention in CONTRIBUTING.md
For each new contract

[ ] The deployment script deploys the new contract.
[ ] etherscan_verify.py runs on the new contract.

Bookkeep

[ ] The gas cost of new functions are stored in gas.json.

Solidity specific conventions

[ ] Document arguments of functions in natspec
[ ] Care reentrancy problems
if the PR adds or removes require() or assert()

[ ] add an entry in Changelog
[ ] open an issue in the client, light client, service repos so the change is reflected there
Just adding a message in require() doesn't require these steps.

[ ] When you catch a require() failure in Solidity, look for a specific error message like pytest.raises(TransactionFailed, match="error message"):

And before "merge" all checkboxes have to be checked.  If you find redundant points, remove them.

This will also fix raiden-network/raiden#7203, at least in the sense that no chages are needed then on the raiden side. However, we'd still like to use a single mechanism to deploy contracts, i.e. the ContractDeploy from raiden-contracts.


Release web microsoft.web 2022 09 01

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

about 44 of ModelValidation is from PR https://github.com/Azure/azure-rest-api-specs/pull/21015#issuecomment-1269122734
other appears to be api-version in example is 2021-03-01

pending PR to this branch

https://github.com/Azure/azure-rest-api-specs/pull/21984
https://github.com/Azure/azure-rest-api-specs/pull/22159

PR is not generated properly for a new API version
The first commit needs to be the previous api version and the new changes should only be added in the subsequent commits. This is explained in detail here (If using Open API which is recommended)  https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/208/OpenAPI-Hub-Adding-new-API-version or here(For manual process) :  https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/83/Manual-Process-Adding-new-API-version . Please remove the "ARMChangesRequested" label once you have fixed this issue.

@raosuhas web/specifications have multiple RPs in the same client. We would love to move them apart but that is a big breaking change. Unfortunately, openAPIHub does not support creating a branch with multiple RPs as base. This has been our feature request to the openAPI team (@akning-ms , @josefree )

@raosuhas
All PR merging into this feature branch as been approved by ARM reviewer

https://github.com/Azure/azure-rest-api-specs/pull/21015 (Tim)
https://github.com/Azure/azure-rest-api-specs/pull/22159
https://github.com/Azure/azure-rest-api-specs/pull/22211
https://github.com/Azure/azure-rest-api-specs/pull/22491

@MaryGao Please check JS SDK breaking changes
https://github.com/Azure/azure-rest-api-specs/pull/22084/checks?check_run_id=11705120988

@msyyc on python failure.

We will fix the failure soon and please ignore it temporarily.

We will fix the failure soon and please ignore it temporarily.

Sure. I will probably ask Arthur to force merge this tomorrow.


Error 0xc000007b when trying to run BCUninstaller.exe

I tried installing Microsoft Visual C++ Redistributable 2008, 2010, 2012, 2013, 2015-2019 but none of them fixed this issue. I'm pretty sure I'm missing some dependencies, and I don't understand why I should be installing random packages and restarting my laptop when this can probably be taken care of during installation. I think you should at the very least include whatever dependency it is I'm missing in the System Requirements list
And by the way, I do have .NET SDK installed.
Edition	        Windows 11 Pro
Update	        21H2
Installed on	7/6/2021
OS build	22000.51
Experience	Windows Feature Experience Pack 421.16300.0.3

Try the test build from #291
The fix is commited but wasn't released yet https://github.com/Klocman/Bulk-Crap-Uninstaller/commit/7fb88904e3bbef97dbbcbd7d6ae9f6ee3383c6f5


log_connection_exception should be a class method

log_connection_exception is only called from within one class method. In addition, it fails when it is called because no self.log_connection_exception exists.

I have manually merged this.
I did not have it eat the exception as the patch suggested, but I did correctly make it a class method.

oh ok, I will update my local.  Thx


Add a configuration warning when using Ignore mouse filter with tooltip

This closes #30144.

@akien-mga I updated the pull request, please check it again.

Thanks!


Change link to Zulip stream

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Make Data logging optional

This PR is adding a listener which makes datalogging optional
fixes https://github.com/AlbinEriksson/Java-Steam-Controller/issues/1

Wow, I can't believe I left that in by mistake. I approve of the LogListener class idea, but since it accepts the raw data expressed as a String, wouldn't it be better to create a listener which accepts the raw data as a byte[]?

Yeah your right. Let the user decide how to handle the Byte array. Will commit the change tomorrow. Good night

Approved, but I will make some stylistic changes before I merge to the master branch. Thanks for your contribution!

Yes that’s a solution I thought of, too. This cleans it up.


Avatar animate is delay 3s

In SeedAvatarStuido with SeedArkitUnity 0.2.0  avatar animate delay 3s , the fps is about 20

I have some ideas what may affect performance in 0.2.0. Let me try to make some fixes and please help me to verify.

Get it

Discussed that the attempt fix doesn't solve the issue. This issue happened before 0.2.0-alpha drop (which explains why the fix is not sufficient), and the reason is can be anything from computer bottleneck, memory leak, upstream bugs, etc.

Added some logs in PR #36 to try to narrow down the problem to a certain component.

0.3.0-alpha the delay is solved in my mac

Hmm, I didn't expect the animation delay can be improved by this version.
Maybe the delay was introduced in https://github.com/SeedV/SeedUnityVRKit/pull/24. https://github.com/SeedV/SeedUnityVRKit/pull/32 reverts most of that change and switches to a new implementation.
Anyway, this is great. I'll close the issue now.


Fix incorrect check if $options['input'] was ArrayAccess.

Previously, $input was always null, and that should have checked
$options['input'] instead. (for  && ! $input instanceof ArrayAccess)
Move the assignment to $input above the conditions.
Provide a narrative description of what you are trying to accomplish:

[x] Are you fixing a bug?

[x] Detail how the bug is invoked currently.
[x] Detail the original, incorrect behavior. ArrayAccess was not allowed for SessionArrayStorage and an exception would be thrown
[x] Detail the new, expected behavior. ArrayAccess is now allowed for SessionArrayStorage
[x] Base your feature on the master branch, and submit against that branch.
[x] Add a regression test that demonstrates the bug, and proves the fix.
[ ] Add a CHANGELOG.md entry for the fix.

[ ] Are you creating a new feature?

[ ] Why is the new feature needed? What purpose does it serve?
[ ] How will users use the new feature?
[ ] Base your feature on the develop branch, and submit against that branch.
[ ] Add only one feature per pull request; split multiple features over multiple pull requests
[ ] Add tests for the new feature.
[ ] Add documentation for the new feature.
[ ] Add a CHANGELOG.md entry for the new feature.

[ ] Is this related to quality assurance?

[ ] Is this related to documentation?

Thanks, @TysonAndre!


Adding policy issue

Is this a request for help?: Yes

Is this a BUG REPORT or a FEATURE REQUEST? (choose one): BUG REPORT

Version of Anchore Engine and Anchore CLI if applicable:
anchore-cli, version 0.2.0
Name: anchore-engine, Version: 0.2.1
What happened: I am getting following message while trying to add a policy generated using anchor cloud
Detail: 'policies' is a required property
What did you expect to happen: It should successfully add the policy

Any relevant log output from /var/log/anchore:
400 error:
In engine logs:
[service:api] 2018-06-08 08:17:45+0000 [-] "<IP_ADDRESS>" - - [08/Jun/2018:08:17:44 +0000] "POST /v1/policies?detail=True HTTP/1.1" 400 123 "-" "python-requests/2.17.3"
What docker images are you using:
docker.io/anchore/anchore-engine:latest
image id: 6eafdab1ce30
How to reproduce the issue:
Execute: anchore-cli policy add /PATH/policy1.json
Anything else we need to know:

hi @listenerme -
I believe this is likely due to an attempt to load a single policy item, rather than the entire policy bundle (which is what the engine expects).  We do have some terminology overlap that we need to clean up for sure, but for now note that the engine is expecting the document you get when clicking 'Download All Saved Policy Items' (which generates a 'bundle-XZY...json' document).
Let us know how that goes - our action I think is to clarify in the CLI (minimally) that the engine is working with 'bundles' and not individual 'policies' which is an overloaded term, currently.

@nurmi ...
Yes, thanks. I had to use the bundle to add the policy. I was using just the policy that I created in anchore cloud instead of using the bundle with other items like whitelists

thanks @listenerme - i’ll close the ticket since you’re able to get a bundle uploaded from anchore.io!


Write Part -1 Step - 10 & unit test

Implement Step10Controller and write all unit tests.  Verify the display name of the test conforms to the pattern of "Part X Step Y".  Follow the standardize naming, comments, etc. to be consistent.

#131 Merged updates

Fix tests


Removed sentence with reference to callbacks

Coverage remained the same at 67.64% when pulling 1583a275471a3978061bf1937a4fad04e55ca980 on rippler:patch-5 into 61c3e8280f5df06b1752dd8a93ad3f585f85229c on CounterpartyXCP:9.49.4.


📖  Track recent API change, in example of combining status

Summary
This PR makes an overlooked doc update, due to the recent API change.
This PR fixes the release-0.26.0-alpha.2 branch.
Related issue(s)
Fixes #

/cc @clubanderson
/cc @waltforme
/cc @ezrasilvera
/cc @vMaroon

/retest


WorldEdit - Pocket Edition

Hey! I am a PocketMine (the main server software on Minecraft Pocket Edition written in PHP) plugin developer and I am working in the most powerful WorldEdit for this Minecraft's version. I am looking to be recognized as an official WorldEdit developer, can we talk by twitter or another social network?
Also... I apologize for asking here, I didn't find you anywhere else.

Email http://sk89q.com/contact

Did you read my email or just ignored it lol


SAA-411 dependency updates.  Linting has changed hence the large number formatting changes in this commit.

As a result of moving to a more update plugin version we have inherited a linting formatting change hence the large number of files affected by it in this PR.  There are no actual functional changes.

Codecov Report
Patch coverage: 78.36% and project coverage change: +0.01 :tada:

Comparison is base (6d28109) 73.66% compared to head (5cfa4bc) 73.67%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #202      +/-   ##
============================================
+ Coverage     73.66%   73.67%   +0.01%
  Complexity      941      941
============================================
  Files           176      176
  Lines          4314     4316       +2
  Branches        141      141
============================================
+ Hits           3178     3180       +2
  Misses         1047     1047
  Partials         89       89

Impacted Files
Coverage Δ

...agementapi/client/prisonapi/api/PrisonApiClient.kt
100.00% <ø> (ø)

...iesmanagementapi/config/AuthAwareTokenConverter.kt
0.00% <ø> (ø)

...th2ClientCredentialsGrantRequestEntityConverter.kt
0.00% <ø> (ø)

...vitiesmanagementapi/config/OpenApiConfiguration.kt
0.00% <0.00%> (ø)

...anagementapi/config/ResourceServerConfiguration.kt
0.00% <0.00%> (ø)

...hmppsactivitiesmanagementapi/config/UserContext.kt
0.00% <ø> (ø)

...tiesmanagementapi/config/WebClientConfiguration.kt
0.00% <0.00%> (ø)

...ivitiesmanagementapi/entity/ActivityEligibility.kt
88.88% <0.00%> (ø)

...gementapi/entity/ActivityScheduleEntityListener.kt
0.00% <0.00%> (ø)

...esmanagementapi/entity/AllocationEntityListener.kt
0.00% <0.00%> (ø)

... and 88 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Shortcut for single line comment

Hi,
I tried to press command + / to comment/uncomment single line. But it was not working.
Is there any shortcut for commenting?
Thanks!

Hi @yangzichao, if there is a keyboard shortcut it will be completely handled by the front-end you are using (rather than the kernel). That being said, on windows, ctrl + / will line comment in jupyter lab and jupyter notebook with the default bindings.
Try clicking Help > Keyboard Shortcuts in the jupyter notebook front-end and look for comment.

Thank you so much!


Configure to run on pi

Use trs80gp program by George Phillips to emulate the scripts current that have been developed.


invalid URLs are reported as 505 HTTP Version Not Supported

I had a few malformed URLs containing markup left from a crawling an HTML page. This is definitely user mistake but vegeta was reporting those as 505 (505 HTTP Version Not Supported
The server does not support the HTTP protocol version used in the request.)
What do you think about flag and ignore invalid URLs?
Example URL causing problem:
GET valid_url" class="accordion-toggle-sub"

@agenteo: Did you manage to reproduce the bug?


Incorrectly displaying error messages for DatePicker/DateTimePicker

Describe the bug
Error messages for DateTimePicker (this is the one I use) and DatePicker (see on documentation website) are inconsistent with respect to other form fields (icon in a different place) and overall don't appear correctly - part of the message is overlapping with the icon.
Only on the "Compact" form layout message and icon don't overlap BUT on this layout icon is placed on top of the next field.
To Reproduce
Just look at documentation website

Environment

Blazority Version [1.1.1]

Fixed in v1.2.0, to be released

Fixed in v2.0.0, released today


Is it possible to add a feature like filtering?

Great work and huge thanks!!!
Looking forward to a feature that enables us to filter our notes on graph!
Or are there some tricks that could work around for now?

Can you give an example

I find it helpful to filter the graph via the titles of the notes and for now it fits my need. If i got  any new ideas, i would add additional comment here!


swagger UI return files instead of http pages

Describe the bug
when i point to my project  http://localhost:8080/swagger/index.html, it return a download index.html files instead of parse html page
To Reproduce

create a basic gin app
follow instruction on github.
create a simple route with comments
run swag init
check that docs contain the files with right code

Expected behavior
it must serve http swagger ui
Your swag version
last version, gin last stabe version 1.2
Desktop (please complete the following information):

OS: OSX 10.13
Browser chrome
Version last stable

ok i found the problem. Adding gzip middleware break:
router.Use(gzip.Gzip(gzip.DefaultCompression))

I have the same problem. But I would like to use gzip middleware. Is it possible to use it with working swagger?

How does work about middleware?

@pei0804 Middleware in general do some useful work common for all server endpoints. Typically logging middleware is set to log config standard unified logging for all endpoints. Prometheus middleware can be used to expose metrics from all endpoints. Gzip middleware ensures response compression if client requested it using Accept-Encoding.

@pei0804 I created pull-request which solves this problem. I see you are the contributor of this project. Could you accept it or review, please?

ok.


feat: 5673 cell guide tissue specific cell type route

Reason for Change

#5673

~⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠~
~DO NOT MERGE UNTIL @ainfeld is back to help sort out analytics impact~
UPDATE(12/1): @ainfeld has confirmed that no engineering changes for analytics changes
~⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠~
Changes
Behavior changes:

On a cell type page, the tissue dropdown now navigates to tissue specific cell type page
On a cell type page, the right cell type info panel's link now navigates to tissue specific cell type page
On a tissue page, clicking on a cell type node now navigates to its tissue specific cell type page. HOWEVER, if the cell type page ends up not having the specific tissue in its tissue dropdown, we will redirect the page to its tissue agnostic equivalent

Code changes:

Move tissueId from hook state to the route state
Refactor: Extract hooks into connect.ts
Add SEO snapshots for new tissue specific cell type pages
Add isSuccess to hooks useAllOrgansLookupTables() and useOrganAndOrganismFilterListForCellType(), so we can do behavior change 3 above
Add a new useEffect in connect.ts to redirect user to tissue agnostic cell type page, using isSuccess from the change right above
Extract getCellTypeLink() utility function for all three places (dropdown, ontology viewer, and right side cell type info panel) that push user to cell type page to use
Add e2e tests

Testing steps
Three behavior changes:
BEHAVIOR 1:

Navigate to https://pr-6232-frontend.rdev.single-cell.czi.technology/cellguide/tissues/UBERON_0000955/cell-types/CL_0000540
You should see Neuron Cell Page with Brain selected in the tissue dropdown

Video:

Changing tissue changes the URL, which is a shareable URL by default
Route change doesn't trigger a full page reload, which is more performant
App state also changes accordingly to show tissue specific data when a tissue is selected

https://github.com/chanzuckerberg/single-cell-data-portal/assets/6309723/cb40658b-2f19-4383-a4e6-3815a18c5dab
BEHAVIOR 2:

On a tissue page, clicking on a cell type node in ontology viewer should navigate you to its tissue specific cell type page

BEHAVIOR 3:

On a tissue page, clicking on a cell type node in ontology viewer should navigate you to its tissue AGNOSTIC cell type page, IF the cell type page turns out not having such tissue in its tissue dropdown.

For example, on Brain tissue page, clicking on "cell" node should take you to the tissue agnostic "cell" page
Checklist 🛎

[x] Add product, design, and eng as reviewers for rdev review
[x] For UI changes, add screenshots/videos, so the reviewers know what you expect them to see
[x] For UI changes, add e2e tests to prevent regressions

Notes for Reviewer

@tihuan The page should maintain scroll position when changing the tissue. Is that still possible? Otherwise looks good!

Oh yes! I wasn't sure if it was a bug or a feature to keep the scroll position, so went with the default navigation behavior 😆
Just updated the code to keep the scroll position. Rdev should be updated soon 🙆♂
Thanks for the great feedback!

@atarashansky PTAL this PR for feature related implementation 🙏
I'm keeping this PR in draft until @ainfeld is back to collaborate on analytics changes
Thank you!

@kaloster Thanks for reviewing this!
@ainfeld sorry that this didn't go out this week, but it will get merged today and go out to prod next week for sure now. Thank you!
(I'm still on bereavement, just thought I'd get this merged 😆 Will be back 12/18. Thanks all!)

thanks @tihuan!! no worries about when it actually goes to prod, doesn't really matter to me haha just useful to know when it's expected so I can plan the work I need to do after it. This gives me lots of time to play around with it in staging to understand what I need to change for analytics so really really appreciate that  🙏

Ahaha awesome! Glad some extra time in staging is useful this time 🎉 Thanks again, @ainfeld 😄 🙏


folding: provider event to signal that folding ranges have changed

In the C/C++ Extension, we have a scenario in which folding ranges may not be accurate if partially overlapping with inactive regions, until a semantic parse has been completed.  The semantic parse allows us to determine if a #if/#ifdef region is active or inactive.
A contrived example:
#define TEST 0

void foo(bool b)
{
    if (b)
    {
#if TEST
        // ...
    }
#else
        // ...
    }
#endif
}

We could wait until the semantic pass is complete, but for very complex source it can take several seconds, possibly minutes.
We could provide ranges (as we are now) without considering inactive regions.  If we receive a subsequent request for folding range (such as the user editing the file), we could take the inactive regions into consideration.  Though, this would leave inaccurate folding ranges for files that are viewed and not edited.
If there were a way for us to programmatically trigger a new folding ranges request as inactive regions become available (or change), that would seem to provide the best experience.
(Related: https://github.com/microsoft/vscode-cpptools/issues/5429 )
Version: 1.46.0 (user setup)
Commit: a5d1cc28bb5da32ec67e86cc50f84c67cc690321
Date: 2020-06-10T09:03:20.462Z
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19041

I think a 'hasChanged' event makes sense, but returning ranges in patches is problematic.
The folding range infrastructure currently relies on the fact that it gets all folding ranges in one go.
In a document where the user has folded some ranges, we need to decide on every update whether the region needs to stay folded, or unfolded as it is no longer reported as a folded region.
If we were to keep it folded (because more ranges might come in), how long should we wait? It might never appear again as it really got removed.
Also, from a user experience it will be confusing of some ranges do not show up for several seconds, or if hidden line suddenly reappear or some get folded.
So could C++ have a good guess on the ifdef evaluations in order to return a complete set of ranges?
Or come up with a mix of ranges that are practical to the user regardless of the actual ifdef evaluations?

We cannot know the active/inactive status of all if/ifdef regions until a semantic parse has been completed.  The boundary of such regions may be anywhere in code, except for within a string or comment.  Either we wait for a semantic parse, proceed without it and possibly fold incorrectly, and/or return a fresh set once that information is available.

Is it is possible to already do some syntax based analysis before knowing which regions are active and inactive?
Would it make sense to always create folding ranges for the if/ifdef regions themselves (one range from #if to #else and one from #else to #end), and for blocks inside and around the regions, and discard all intersecting ranges?

Is it is possible to already do some syntax based analysis before knowing which regions are active and inactive?

Yes.  Currently, our folding ranges are based on a (fast) syntactic parse.  The only reason we would need a semantic parse would be to handle ranges split by preprocessor blocks.

Would it make sense to always ... discard all intersecting ranges?

That seems reasonable.  Our folding algorithm is shared with VS.  I assume VS has to address a similar scenario.  Trying this in VS, it initially applies saved collapsed ranges against the syntactic results, then corrects them after a semantic parse completes.  Perhaps it might be worthwhile to have a chat between VS Code and VS devs to share a common approach?  (I'm not personally convinced that folding the other side of a split range to within the active preprocessor block is necessarily ideal behavior).

I personally would like to stick with our current model that folding providers have to return all ranges on every request. It's simpler from the implementation perspective but also beneficial to a the user experience. My hope is that C++ can construct useful folding ranges not depending on the active/inactive information.
The change event still makes sense, e.g. if a provider has folding settings that that impact the folding ranges visible in a document. I'm happy to add it if it's still useful.
If this is not good enough for your use case then I suggest we discuss this more in the next C++ sync call.

The change event still makes sense

@aeschli  An event to indicate that folding ranges have changed, would work.  :)  Thanks.

I added the following proposed API:
	export interface FoldingRangeProvider {

		/**
		 * An optional event to signal that the folding ranges from this provider have changed.
		 */
		onDidChangeFoldingRanges?: Event<void>;

		/**
		 * Returns a list of folding ranges or null and undefined if the provider
		 * does not want to participate or was cancelled.
		 * @param document The document in which the command was invoked.
		 * @param context Additional context information (for future use)
		 * @param token A cancellation token.
		 */
		provideFoldingRanges(document: TextDocument, context: FoldingContext, token: CancellationToken): ProviderResult<FoldingRange[]>;
	}

@aeschli Did this go into 1.51?  I'm not seeing the event in  @types/vscode 1.51.0

It's in vscode.proposed.d.ts and beeing finalized fin 1.52: https://github.com/microsoft/vscode/issues/108929


Fleet Bundles is not available

On 2.6head commit id: c91dcaf

Deploy RKE1 and RKE2 clusters
Navigate to Continuous Delivery --> Advanced --> Bundles page.
Nothing is loaded
Error in console:

vue.runtime.esm.js:1897 TypeError: Cannot read properties of undefined (reading 'summary')
    at fn (PlusMinus.vue:3:21)
    at r (vue.runtime.esm.js:2605:37)
    at f.Re [as _t] (vue.runtime.esm.js:2701:7)
    at fn (settings.js:3:184)
    at r (vue.runtime.esm.js:2605:37)
    at f.Re [as _t] (vue.runtime.esm.js:2701:7)
    at grouping.js:12:344
    at f.Re [as _t] (vue.runtime.esm.js:2706:47)
    at grouping.js:10:221
    at f.Ce [as _l] (vue.runtime.esm.js:2646:16)

Note:

FLeet controller and App are in bad state.
Error on fleet controller: 17 mins ago	Failed to pull image "rancher/fleet:v0.3.10-rc1": rpc error: code = Unknown desc = Error response from daemon: manifest for rancher/fleet:v0.3.10-rc1 not found: manifest unknown: manifest unknown

On 2.6head commit id: 2c21373

Deploy RKE1 and RKE2 clusters
Navigate to Continuous Delivery --> Advanced --> Bundles page.
Page is loaded. No bundles displayed. No error in console.

After an offline chat with @aalves08  Closing this issue since we aren't able to reproduce this.


Is there a method to address a single sprite other than the name given in the spritemap?

I've got a spritesheet with playing cards, row = suit, col = value.
Is there a built-in function/method to call up a single sprite from the sheet by coordinate? so that say map[3,1] would be the 3 of Clubs at the sprite location 4th row, 2nd col?
The example at Sprite.sprite seems to be used to rename the sprite at that coordinate position, so I was looking for a related function to get/show a sprite.

Calling e.sprite(3, 1) should work.  The sprite method is overloaded -- you can pass in either coordinates or a name.
The example is just showing two different ways of calling the method.  (The first passing the exact position/dimensions in tile coordinates, the second passing a predefined sprite name.)

Thank you.

How should I refer to this when defining the element?
I've used
Crafty.e('2D, Canvas, Mouse, SpriteAnimation, Tween, Sprite, FullDeck')

But FullDeck is only a name mapped to 0,0 coordinates map{FullDeck: [0, 0]}, which seems to work-ish but doesn't seem quite right.
Not sure if this is the source, but there winds up being a downline problem:
function go() {
  let front = true;
    let cardsuit = Math.floor(Math.random()*4);
    let cardcard = Math.floor(Math.random()*13);
    let actual = (cardcard, cardsuit);
    Crafty.e('2D, Canvas, Mouse, SpriteAnimation, Tween, Sprite, FullDeck')
    .sprite(cardcard, cardsuit)
    .reel("Flip2Back", 10, [actual ,'Back'])
    .reel("Flip2Front", 10, ['Back', actual])
    .bind('Click', function(e){
      if (front) {
        front = !front;
        this.animate('Flip2Back');
      } else {
        front = !front;
        this.animate('Flip2Front');
      }
    }
    );
}

If I go .sprite(actual), then it fails silently, showing nothing. The display is blank, but the flip reel animates, showing the sprite defined at Back when clicked for toggle.
With the code as shown, the card at the coordinate shows, and then a click shows the back. However, a 2nd click shows a blank front. It then does as first, showing the sprite at Back but no front when toggled.

Please advise?

let actual = (cardcard, cardsuit);
I think this is the root of your confusion here -- it looks like you're thinking of actual as a tuple, but that's not how javascript works.  Really you're accidentally using javascript's comma operator -- (1, 2) is an expression that returns 2.  (Try logging actual, or checking its type).

If you're testing in a browser, you might want to invest some time in learning how the debugging tools work -- by stepping through the code you could have seen that actual wasn't of the type you thought it was.  And remember that javascript isn't an untyped language -- you can't declare the type of a variable up front, but values do have types, and you need to understand what types you're passing around.
So although conceptually you can think of a sprite's location as being identified by either a name or a tuple of coordinates, since a tuple isn't a construct in javascript, you need to pass in either a string or two separate coordinates when calling sprite:
// ok
e.sprite('name');
// ok
e.sprite(1, 4);
// doesn't do what you want -- you're passing one argument,
// and javascript doesn't support tuples like this
e.sprite( (1, 4) );

But FullDeck is only a name mapped to 0,0 coordinates map{FullDeck: [0, 0]}, which seems to work-ish but doesn't seem quite right.

When you create a sprite map, under the covers Crafty generates a new component for each named sprite in the map.  (You can see that each such component has the same initialization method, since they all use the same asset -- the only difference is the initial coordinate.)


contractcourt: only halt resolution for TapscriptRoot channels

As suggested here.

@carlakc, remember to re-request review from reviewers when ready

This PR is no longer relevant after recent merges.


Add missing google-guest-agent package

What happened:
As of now, the gcp and gcp-dev image can not be built properly. This is because debootstrap isn't able to initalize the rootfs. It tries to install the google-guest-agent package, that isn't part of the Garden Linux Repo anymore.
What you expected to happen:
The Garden Linux build should be able to build the gcp and gcp-dev variant. Therefore, it must offer the google-compute-engine package. Since, it isn't built specifically for Garden Linux anymore, it must at least mirror the package from debian/testing
How to reproduce it (as minimally and precisely as possible):
make gcp-dev BUILD_OPTS="--lessram"
Anything else we need to know:
This issue is fixed once this merge request has been completed.

The missing google-guest-agent package has successfully been added to the Garden Linux Repo as one can see here:

https://github.com/gardenlinux/gardenlinux/runs/5291490400?check_suite_focus=true

Therefore, this issue can be closed. However, there still seems to be a problem with the new google-guest-agent package that will be fixed in #672


Move Add-Type into inside Process-JAR ?

It's not needed anywhere else. Would depend on #13.

Actually, nevermind, this is a bad idea because we call Process-JAR multiple times.


Bump WebTools and remove legacy CLaSP package

Fixes https://github.com/dotnet/razor/issues/10160

I'm a bit late to this PR, but that doesn't stop me from being super happy about it! ❤


Base grenade prototype

About the PR
Grenades got their parent prototype
No C#

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

You know, it is done 🐸

Hm


fixes scrollbar on tooltip in firefox

Overflow hidden to remove scrollbars on firefox

Tested on

[x] Desktop Safari
[x] Desktop Firefox
[x] Desktop Chrome
[x] Desktop IE11
[x] Desktop Edge
[x] Mobile iOS Chrome
[x] Mobile iOS Safari
[x] Mobile Android Chrome


Login screen doesn't work, can't do anything (Linux)

I think it's an issue with runelite? I'm not sure. OSBuddy works just fine.

See https://github.com/runelite/runelite/wiki/Disable-Hardware-Acceleration and how to disable OpenGL hw acceleration which is causing your issue.

wtf that was the fastest response ive ever had thanks a lot

I haven't experienced this using Intel driver with Arch Linux or Nvidia driver with Ubuntu 16.04.

I'm using Arch 4.16.2 with Nvidia driver 390.48, haven't seen this problem before.

It mostly because of misconfigured graphics cards, what is apparently very common with Linux users :P

I fixed this by running java -jar RuneLite.jar --mode=OFF (I found this on Reddit) and was hoping someone could explain what the --mode=OFF is doing and should I be doing it?
Also I experienced this problem on Debian (buster) with what I think is a properly configured Intel HD 620.

@CrashyBang That's the same as is documented here: https://github.com/runelite/runelite/wiki/Disable-Hardware-Acceleration

@Nightfirecat sorry I completely missed that part my apologies.


Update observer_reports.md

2024-1-24 Obs. Sch. Bd. Rpt. added

2024-1-24 Obs. Sch. Bd. Rpt. added


Decorators import scoped to @ember

I am trying to figure out how to use @ember-decorators in a project consuming this addon.  Because the import is still import { tagName } from 'ember-decorators/components, I am getting a build error because in my project, I am importing it like so: import { tagName } from '@ember-decorators/components (2.0.0-beta.1).  Is there any way to resolve this other than forking and updating this addon to use the beta version of ember-decorators?

I'm attempted to use the ember-decorators beta in #80, but ran into a bug with ember 3.1. @pzuraq how would you feel about me removing ember-decorators for now so people can still use ember-popper until we can get around to a fix in ember-decorators?

It would be a pretty big overhaul to remove decorators, if you'd rather drop them and classes for stability no problem, but it's not likely to be as simple as pop them out then back in. I can try to take a look at #80 tomorrow

https://github.com/kybishop/ember-attacher/issues/114#issuecomment-370095854

I believe this issue is actually from ember-decorators, which don't have full ember 3.0 support yet, and ember-decorators is used in the positioning library, ember-popper.

Is that correct? I haven't yet discovered any problems with the v1 line of ember-decorators in my 3.0 app, but it might just be me.
If so, we either need to

rip out ember-decorators
upgrade to ember-decorators v2 beta

The latter would require users to upgrade their apps to v2 as well, which they probably should do anyway, because of the incompatibility with Ember 3. So basically users would have to install 0.8.3 and below for Ember 2 and 0.9.0 and above for Ember 3.
I would be happy to pitch a PR upgrading ember-popper to ember-decorators v2. Because right now, my app is broken :/

#80 does this, we just haven't had time to dig into why tests are failing

Oh yes, silly me. I'll try to look through it later. Maybe I find out something.

This will be fixed once https://github.com/ember-decorators/argument/pull/62 lands

This should be fixed by #80


Add optional standard and packaged fields to available field set options

Critical Changes
Changes

Add additional option standard and packaged fields to packaged field sets

Issues Closed
New Metadata
Deleted Metadata
Definition of Done
Refer to Asteroids DoD document to see any additional details for the items below
Any net new LWC work has Sa11y tests & 50% or above lines JEST test coverage
CRUD/FLS is enforced in Apex
Permission sets are updated to account for CRUD|FLS|Tab|Classes

[x] Field sets are updated to account for new fields
UX approval or UX not necessary
[x] Link the pull request and work item by PR comment and Chatter post respectively, e.g. GUS: W-0000000: Work Name (https://github.com/SalesforceFoundation/PMM/pull/303)
[ ] All acceptance criteria have been met

[x] Developer
[ ] Code Reviewer
[ ] QA

[x] PR contains draft release notes
[ ] QE story level testing completed

W-8833056

Closing need to rename branch as 230 child.


Created Google Logo

I have created the Google Logo
Fixes #130
Insert the image of the art below

Checklist:
To fill the checklists, put x inside the brackets. [x]

[x] My code follows the guidelines of this project
[x] I have tested my code and it works properly
[x] I have added just one folder that includes my Art files (jsx and css)
[x] I have not edited App.js file
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] This is my original code and I have NOT copied the code from anywhere else.

Hi @sg7801, there is already a PR open for Google Logo. Requesting you to make some other art.


[Bug]: Cantact and about page is same

Contact Details
<EMAIL_ADDRESS>What happened?
just look both images URL👀
about page

Contact page

What browsers are you seeing the problem on?
Chrome, Microsoft Edge
Relevant log output
No response
Are you interested in contributing?

[X] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

Which program are you contributing in?
gssoc

Hi @pranshugupta54 the pages are same or what?

We're still working on the contact page - it's currently on contact branch


Make vertex_data and edge_data private

In cage topologies, there is no reason for vertex_data and edge_data to be public attributes. Update the introduction docs to account for this change.

Actually, the user needs to be aware of how many vertices there are in the topology graph to use vertex alignment s


Add windows engine-session binaries to engine image.

Signed-off-by: Erik Sipsma<EMAIL_ADDRESS>NOTE: we have no automated testing of Windows and we never have even really manually tested in the past to my knowledge.
We can currently build these client binaries successfully though. When I run the engine integration tests w/ these in Windows, a lot of tests pass but a lot fail (seems to primarily be ones involving local dirs).
We need to decide if it's worth including these binaries in the engine image.
Pros:

Windows users can use at least some of the features of Dagger, same as before the engine image changes

Cons:

It doesn't actually fully work yet

Counterpoint: that has been true before

It increases the size of our engine image by ~60MB uncompressed

I'm leaning on the side of including them anyways for now since it will essentially retain the same state of half-functionality that existed previously. However, we shouldn't advertise dagger as working from Windows clients yet of course. That will require fixes and adding automated testing.
cc @shykes @aluzzardi

Agree with including it. If we have a lot of Windows users who want to use us and who run into the limitations, it will be good signal to work on them sooner.


SequelizeDatabaseError: column Groups.Permissions.get does not exist

I am getting this error when I run the code locally. I haven't changed anything except setting the database to be postgres. I have also run the migrations. In my database I have the GET column, but it is in uppercase letters.

In my database I have the GET column, but it is in uppercase letters.

Can you try to drop the database and restart the app without running the migrations? Sequelize will automatically provision the database when empty.
FYI, GET is in uppercase due to an issue with Sequelize that doesn't allow columns to be named get.

Nopes. Just to be sure I created a brand new database. Got the same error. I am running node 4.2, postgres 9.4 and mac osx (not with docker)

You're using the latest code, aren't you? I remember fixing this bug a while ago.

Yes that must have been the issue. I used the yeoman generator a couple of days ago. But I see now that the code that is generated is older then the newest on github. Cloned the repo instead, and now it works. Thanks!

Created this issue instead: https://github.com/FredericHeem/starhackit/issues/32


Improve MRTK project settings text

Overview
After https://github.com/microsoft/MixedRealityToolkit-Unity/pull/6315, all our MRTK settings are now on a per-project basis (some were previously global), serialized into a .asset file. Some of our settings descriptions still mentioned the previously global nature of things.
I also:

updated some of the settings text, for example to explicitly call out the one that handles the project configurator
updated some summary tags that were incorrect (copied and pasted from the first setting and not changed)
added a note to the settings page about where these settings are serialized and how they can be shared
Fixed some instances of incorrect Unity null propagation

Instance?. in ProjectPreferences, where Instance is a UnityEngine.ScriptableObject.

Removed an unused key

/azp run

/azp run


Proprosal of redesigning csrf protection

We use Laravel 5.1 and Redis as our session drive on our production environment,  and have randomly came up withTokenMismatchException over last two months, which made us very annoyed.
I know this is a known issue which hasn't been fixed yet, because of session persistence bug. So I refer Django's solution to rewrite the csrf middleware, just compare the token in cookie and request header, avoiding dependency with laravel session. It works fine now.
This bypasses the session bug but could be a temporal solution to help people who need csrf protection.
If needed, I am glad to send a pull request.

Thanks, but as per our contribution guidelines, we don't process proposals here.

Alright, issues are strictly for bug reports. If I issue it as a session bug, I think you will close it because of duplication.

I read your title:

Proprosal of redesigning csrf protection

All proposals need to go to our dedicate repo please.

Thanks, I will move this issue.


[Bug] Knowledge space not found

Search before asking

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

Operating system information
MacOS(M1, M2...)
Python version information

=3.11

DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
CPU
Models information
LLM:tongyi_proxyllm
embedding model：text2vec-large-chinese
What happened
create knowledge ->upload a  pdf file->document persist into index store success
begin chat ,it shows [Knowledge space not found]
What you expected to happen
normal to chat
How to reproduce
create knowledge ->upload pdf file,select vector type
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

please make sure knowledge space name is correct

the name

please make sure knowledge space name is correct

now it is ok，restart the service


Why is K3s still syncing back to a lock file?

I've installed k3s and it works fine ... but when I try to change contexts, I'm getting a weird error:
error: open /etc/rancher/k3s/k3s.yaml.lock: permission denied

The command I'm using to get this error:
KUBECTL_BIN=kubectl; $KUBECTL_BIN config set 'contexts.'`$KUBECTL_BIN config current-context`'.namespace' kubeflow

I'm just doing a vanilla install - followed up by a merging of the config file into my core config file.
curl -sfL https://get.k3s.io | sh -s - --write-kubeconfig-mode 644
KUBECONFIG=$HOME/.kube/config:/etc/rancher/k3s/k3s.yaml kubectl config view --flatten > $HOME/.kube/config

Is there anything I can do to get this to work?

The command you provided looks to be attempting to modify the /etc/rancher/k3s/k3s.yaml kubeconfig, which shouldn't really be done. It looks like the user you are calling kubectl as may not have permissions to write to that directory, hence the error creating the lock file.

Hmm... does it? I thought the command reads in the /etc/rancher/k3s/k3s.yaml file and then allows you to flatten it into your file (which you have perms for} - or are you talking about the middle command? Wouldn't that just act on my local .kube/config?

Hmm... does it? I thought the command reads in the /etc/rancher/k3s/k3s.yaml file and then allows you to flatten it into your file (which you have perms for} - or are you talking about the middle command? Wouldn't that just act on my local .kube/config?

The third command snippet you provided will perform a flattening of both $HOME/.kube/config and /etc/rancher/k3s/k3s.yaml and place them into $HOME/.kube/config; however, the second command snippet is what I was referring to in regards to writing to the /etc/rancher/k3s/k3s.yaml kubeconfig.
K3s performs some interesting wrapping around kubectl -- if you inspect the kubectl located in /usr/local/bin, it is actually a symlink to /usr/local/bin/k3s, which functionally is kubectl but has a little bit of logic wrapped into it to default to using /etc/rancher/k3s/k3s.yaml (if it exists) as the default kubeconfig if the environment variable KUBECONFIG (or flag) isn't set.
This can be traced through:
https://github.com/k3s-io/k3s/blob/master/pkg/cli/kubectl/kubectl.go
which calls the exported main here: https://github.com/k3s-io/k3s/blob/master/pkg/kubectl/main.go
which calls https://github.com/k3s-io/k3s/blob/master/pkg/server/server.go#L257 which checks for the GlobalConfig.
Unfortunately, if you want to modify the kubeconfig and still use the k3s-provided kubectl, you'll need to explicitly set the KUBECONFIG environment variable.
I think there is room for enhancement here though to fix this such that kubectl conforms more to expected behavior where it utilizes the $HOME/.kube/config if it exists, and otherwise falls back to the /etc/rancher/k3s/k3s.yaml (rather than the inverse, which is what it is doing now).

Ahhhhhh I was SO confused why I was getting this message when I already had installed a kubectl. This makes complete sense.
To be clear, I had run the third command already to flatten and change perms, so this was the confusion. Yes, this behavior was unexpected. The flow that I would like:

Install k3s
Have an option to write it into my standard .kube/config with my perms and flatten/merge if file already there
Kubectl has same behavior whether installed with k3s or other service

I've tried messing around with --write-kubeconfig-mode but that didn't seem to help.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Chris Kim @.>
Sent: Tuesday, March 16, 2021 11:26:34 PM
To: k3s-io/k3s @.>
Cc: David Aronchick @.>; Author @.>
Subject: Re: [k3s-io/k3s] Why is K3s still syncing back to a lock file? (#3087)
Hmm... does it? I thought the command reads in the /etc/rancher/k3s/k3s.yaml file and then allows you to flatten it into your file (which you have perms for} - or are you talking about the middle command? Wouldn't that just act on my local .kube/config?
The third command snippet you provided will perform a flattening of both $HOME/.kube/config and /etc/rancher/k3s/k3s.yaml and place them into $HOME/.kube/config; however, the second command snippet is what I was referring to in regards to writing to the /etc/rancher/k3s/k3s.yaml kubeconfig.
K3s performs some interesting wrapping around kubectl -- if you inspect the kubectl located in /usr/local/bin, it is actually a symlink to /usr/local/bin/k3s, which functionally is kubectl but has a little bit of logic wrapped into it to default to using /etc/rancher/k3s/k3s.yaml (if it exists) as the default kubeconfig if the environment variable KUBECONFIG (or flag) isn't set.
This can be traced through:
https://github.com/k3s-io/k3s/blob/master/pkg/cli/kubectl/kubectl.gohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fk3s-io%2Fk3s%2Fblob%2Fmaster%2Fpkg%2Fcli%2Fkubectl%2Fkubectl.go&data=04|01|david.aronchick%40microsoft.com|4d6d458af98c4370c6da08d8e90d9d13|72f988bf86f141af91ab2d7cd011db47|1|0|637515591977242856|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PyP2CDLpvcs3ZyFkmve6Nziu2JJEpy99ro8BCIe3B8I%3D&reserved=0
which calls the exported main here: https://github.com/k3s-io/k3s/blob/master/pkg/kubectl/main.gohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fk3s-io%2Fk3s%2Fblob%2Fmaster%2Fpkg%2Fkubectl%2Fmain.go&data=04|01|david.aronchick%40microsoft.com|4d6d458af98c4370c6da08d8e90d9d13|72f988bf86f141af91ab2d7cd011db47|1|0|637515591977252853|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9fiScl58JbxlEUDEr4Y5HQoFIjVhL2fkA069pHS2wHI%3D&reserved=0
which calls https://github.com/k3s-io/k3s/blob/master/pkg/server/server.go#L257https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fk3s-io%2Fk3s%2Fblob%2Fmaster%2Fpkg%2Fserver%2Fserver.go%23L257&data=04|01|david.aronchick%40microsoft.com|4d6d458af98c4370c6da08d8e90d9d13|72f988bf86f141af91ab2d7cd011db47|1|0|637515591977252853|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=SMNCZre%2FoAP%2BV9uH4I5xA9k9CDykNtd%2BeFAlyugo5vk%3D&reserved=0 which checks for the GlobalConfig.
Unfortunately, if you want to modify the kubeconfig and still use the k3s-provided kubectl, you'll need to explicitly set the KUBECONFIG environment variable.
I think there is room for enhancement here though to fix this such that kubectl conforms more to expected behavior where it utilizes the $HOME/.kube/config if it exists, and otherwise falls back to the /etc/rancher/k3s/k3s.yaml (rather than the inverse, which is what it is doing now).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fk3s-io%2Fk3s%2Fissues%2F3087%23issuecomment-800830801&data=04|01|david.aronchick%40microsoft.com|4d6d458af98c4370c6da08d8e90d9d13|72f988bf86f141af91ab2d7cd011db47|1|0|637515591977262844|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=OVl4nYMf78OuHFRtZpni1zLXzkrewS7QHlfdz92XQ1I%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAAMQ5IW7EOZKNNMEQR7UZDTEBDRVANCNFSM4ZJ3E5NQ&data=04|01|david.aronchick%40microsoft.com|4d6d458af98c4370c6da08d8e90d9d13|72f988bf86f141af91ab2d7cd011db47|1|0|637515591977262844|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=nClj1IdTXbg0MffAMunepAY%2FaLZedmYAyQtPDPM6sSk%3D&reserved=0.

You can always use your own version of kubectl that doesn't have our special config loading behavior. Just make sure your version is first in the $PATH.

totally understood - it's just super hidden that this is what k3s does (put a custom kubectl in there that is incompatible with standard kubectl behavior)


add Object back for ad-hoc serialization/deserialization - fixes #183

Doing this for now because I need a solution. If a better solution arises I'm all happy to change it.


Missing Recipes 1.17.1

Mod version for 1.17.1 : GIRC-Redstone-1.0.141.jar  Installed using Curseforge launcher.
I seem to be missing recipes for your mod.  JEI is displaying the items for your mod but no recipes. I cannot use the recipes that are shown in Curseforge website images either.
I added your mod to an existing world (85 mods total).  I've reloaded minecraft twice but it still doesn't give me the ability to craft the items.  I've checked for any required dependencies too.  Am I missing something?  Are there any known mods that could cause this to happen?
As a test I'm downgrading to GIRC-Redstone-1.0.95.jar

I've downgraded to GIRC-Redston-1.0.95.jar - but still no recipes and I've tested in creative on this version.  The items that can be placed in the world do not have a texture but the linking torch works.  If it's a conflict with another mod then I'm stumped as to which one without deleting everything and starting again.
I'm afraid I'll have to uninstall.  But I loved the idea of your mod and sad that it's not working for me.

Hello @dittymargoo it is unlikely to be an issue with an other mod we are investigating the issue thanks for report

The new version should be out any minute

Thank you so much for doing this.  I thought it might have been something I was missing.

Your welcome,
please notify us if there should be further issues
Cheers
~MrTroble


Downloads only last 24 photos for each album

For some reason it seems to only download the last 24 images.

Thank you, it is fixed.

Thanks for the fast fix! Is it possible to download public images from any given profile with this?

iambibhas Would you please more specific about your question?
For FB privilege FB Graph API only work with following condition:

public page
Your photo
If your given profile is personal account it might need more privilege while you request the token.


Add Slackdown parser util

Would it be acceptable to merge if I were to add a utility function for converting slackdown (the modified markdown that slack uses) into HTML?

hmm, not terribly interested in maintaining that. can I ask why you'd want to have it in this library?


FIRMessagingModule tried to override FIRMessagingModule for module name RNFIRMessaging

I got this error on startup of my Android app, someone know how to solve it ?

Anyone got this issue on Android devices ?

Ok it's my fault, I just include module two times in MainActivity --"


CU12 additional parameter needed on event OnBeforeInsertGlobalGLEntry

Hi
we need additional parameter on event OnBeforeInsertGlobalGLEntry
GLReg. so:
OnBeforeInsertGlobalGLEntry(GlobalGLEntry,GenJnlLine,GLReg);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


terminal doesn't keep cursor position if "delete" key is pressed

Expected behavior
Deleting a character using delete key should not move the cursor to the end of the line.
Actual behavior
If you delete a character using delete key, and start typing again, the cursor will jump to the end of the line
I believe it's related to #293

Fixed in 1.0.11, it was similar to #293, the input event code was executed for non-character keys.


Add additional queue and channel metrics

What does this PR do?

Add new metrics requested by user. See updated metatada.csv.

Include fix issue for the excessive warnings logs due to queue being part of a cluster (See detailed description here https://github.com/DataDog/integrations-core/pull/3997)

Filter by PREDEFINED DEFTYPE (may be better to add a DEFTYPE config, ongoing discussed, CC @dsahni )

This issue causes a lot of warnings when trying to get stats from dynamic queues.

Add config queue_definition_types (TBC)

More unit and integration test coverage

Motivation
User request for more channel and queue metrics support.
Additional Notes
Review checklist (to be filled by reviewers)

[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] Feature or bugfix must have tests
[ ] Git history must be clean
[ ] If PR adds a configuration option, it must be added to the configuration file.

PR need rebase.

Superseded by https://github.com/DataDog/integrations-core/pull/5116


Carpentry BOD Script

UOSteam Scripts has yet to add a script for Carpentry BOD. This would be a great first issue.
Here's a Blacksmith BOD script for reference to maybe get you started. If you want to take this on and need some help, please feel free to post here with questions.
Thanks!
https://github.com/her/uosteam/blob/master/resource/BlacksmithBOD.uos

I am the one requesting this specific script / macro. The server I play on has carpentry bods, and no one does them because they cant find a script. well, this will be a script that I can use on this server. Please contact me on discord Atrevion#8830. I am paying for a good working script for this task.
Thank you

Interesting. Can I look at this? I'm not really a our pro but I modified lots of uos scripts I could potentially do this.

On Jul 10, 2019, at 15:01, Atrevion<EMAIL_ADDRESS>wrote:
I am the one requesting this specific script / macro. The server I play on has carpentry bods, and no one does them because they cant find a script. well, this will be a script that I can use on this server. Please contact me on discord Atrevion#8830. I am paying for a good working script for this task.
Thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Please contact me on discord at Atrevion#8830. I can get you the information for the server i play on, and maybe you can take a look at the program I need it to fit i to.
Sent from Yahoo Mail on Android
On Thu, Jul 11, 2019 at 10:47 AM, Matthew<EMAIL_ADDRESS>wrote:   Interesting. Can I look at this? I'm not really a our pro but I modified lots of uos scripts I could potentially do this.

On Jul 10, 2019, at 15:01, Atrevion<EMAIL_ADDRESS>wrote:
I am the one requesting this specific script / macro. The server I play on has carpentry bods, and no one does them because they cant find a script. well, this will be a script that I can use on this server. Please contact me on discord Atrevion#8830. I am paying for a good working script for this task.
Thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@fleekix totally! I don’t actually play on a server that has BODs (shout out to UOSA 🥰) but maybe I should consider it.
@Atrevion there’s benefit in keeping programming related questions on the issue tracker here so others can use them as a reference. Please do not hesitate to add any information in this issue that you think would be helpful to solving this! (screenshots, server specific rules, etc.)

Hey guys, I don't have a carpenter but I'm a fairly good coder and this looks like a fun project. Only issue I don't have a carpenter to use Object Inspect on the items I need serials for, and I don't have Discord to comm with.
Can one of you guys message me on Skype? I'll try to work this out but I need the Object IDs and as I don't have a carpenter character yet, I can't inspect the items lol
live:joshua.alayon_1

Oh man... I dropped the ball on this project.. lol sorry
On Jan 12 2020, at 5:06 pm, joshuaalayon<EMAIL_ADDRESS>wrote:

Hey guys, I don't have a carpenter but I'm a fairly good coder and this looks like a fun project. Only issue I don't have a carpenter to use Object Inspect on the items I need serials for, and I don't have Discord to comm with.
Can one of you guys message me on Skype? I'll try to work this out but I need the Object IDs and as I don't have a carpenter character yet, I can't inspect the items lol
live:joshua.alayon_1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://link.getmailspring.com/link/BAFB6E06-0B10-4450-9D28-1FC0C69C84CB@getmailspring.com/0?redirect=https%3A%2F%2Fgithub.com%2Fher%2Fuosteam%2Fissues%2F11%3Femail_source%3Dnotifications%26email_token%3DADZYUXVXGWG65XZU6VP6BALQ5O5CRA5CNFSM4H73HMB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIXJUVI%23issuecomment-573479509&recipient=cmVwbHkrQURaWVVYV0VTT0pSTlROQzZSQlNaWk40RkQyU1JFVkJOSEhCWFRXSEpZQHJlcGx5LmdpdGh1Yi5jb20%3D), or unsubscribe (https://link.getmailspring.com/link/BAFB6E06-0B10-4450-9D28-1FC0C69C84CB@getmailspring.com/1?redirect=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADZYUXT347OKL5N3W2D2AR3Q5O5CRANCNFSM4H73HMBQ&recipient=cmVwbHkrQURaWVVYV0VTT0pSTlROQzZSQlNaWk40RkQyU1JFVkJOSEhCWFRXSEpZQHJlcGx5LmdpdGh1Yi5jb20%3D).


Prevent UAT and Dev sites from being indexed by google

https://developers.google.com/search/docs/crawling-indexing/block-indexing
Add response header:
X-Robots-Tag: noindex

Will likely require an update to the web.config, but how can this be applied to dev/UAT only?
https://github.com/laingsimon/courage_scores/blob/main/CourageScores/ClientApp/public/web.config

	<httpProtocol>
         <customHeaders>
            <add name="X-Robots-Tag" value="noindex" />
         </customHeaders>
      </httpProtocol>

How can this be varied between UAT and PROD?


03-16 3.0 Fixes

Starting with joinWithSeparator deprecation.

Awesome. Once Vapor is updated to 03-16 I will merge this.


Add html_to! macro

Hi, this PR targets the feature requested in #90
It's based on your current work in the better-lints branch.
It's now possible to write:
struct Css(&'static str);

impl Render for Css {
    fn render_to(&self, buffer: &mut String) {
        // With a mutable reference
        html_to! { buffer,
            link rel="stylesheet" type="text/css" href=(self.0);
        }
    }
}

fn main() {
    let mut s = String::with_capacity(100);

    // With the owned string in scope
    html_to!{ &mut s,
        (DOCTYPE)
        html {
            head (Css("style.css"))
            //…
        }
    };
    println!("{}", s);
}

All tests were rewritten to use html_test! instead of html! (a wrapper to test against html! and html_to! with its two forms)
Please tell me if you see anything I should change, like moving the new functions in their own file etc, I wasn't sure if it was the right place to put them.

Heads up @abdullahibnnadjo -- I've merged the better-lints branch now, so please base on top of master instead :)

Closing due to inactivity. Feel free to reopen if you're still interested (I think a rewrite would be cleaner at this point).


Sync-y stuff (probably)

Trying to get GitHub to stop telling me my fork is behind vs VideahGams' master.

Weiiiiiiiird.

 I'm not behind! XD


Feature: Windows MSYS2 CI

This PR introduces a CI configuration for building against the 3 main MSYS2 compilers on windows to help cover all our bases.

Ah, ig we could - true. Main reason we didn't was to keep the flow following the form of the other flows, but that's easily tweaked.
Think for now, we'll merge and see how it performs and if that not using caching and being separate starts causing problems like failures or excessively long build times, we'll combine.