How to create custom json log message format?

Is the below log message:-
{"level":"info", "message" : "hello world", "tId":"T12345", "reqId" : "R3232", "appId": "Discovery", "sessionId": "12325", "userId":"AB3435", "stackTraces" : ["Call Trace 1", "Call Trace 2"],  "timestamp" : "2015-07-09T18:00:00.875Z"}

where along with message, tId, reqId appId,  sessionId and userId will be passed by caller.
Is it possible to achieve this using custom formatter?

Hi!
If those tId, reqId and others are dynamic, then currently this is not possible to do with custom formatters, because you can only use this context info in custom formatter handlers:
// Represents runtime caller context.
type LogContextInterface interface {
	// Caller's function name.
	Func() string
	// Caller's line number.
	Line() int
	// Caller's file short path (in slashed form).
	ShortPath() string
	// Caller's file full path (in slashed form).
	FullPath() string
	// Caller's file name (without path).
	FileName() string
	// True if the context is correct and may be used.
	// If false, then an error in context evaluation occurred and
	// all its other data may be corrupted.
	IsValid() bool
	// Time when log function was called.
	CallTime() time.Time
}

But, we can think of solving this, probably it is possible to extend the LogContextInterface with something like CustomContext interface{} so that you could be able to do something like this:
    logger, err := log.LoggerFromSomething(...);
    if err != nil {
        return nil, err
    }
...
    logger.SetContext(&SomeContext{tId, reqId, appId, sessionId, userId});
...
   logger.Debug(...);

If such a feature could be implemented, then yes, such a custom formatter could be possible, because you set the context and it is available through the LogContextInterface:
func createTIdFormatter(params string) log.FormatterFunc {
    return func(message string, level log.LogLevel, context log.LogContextInterface) interface{} {
       myContext, _ := entity.(SomeContext)
       return myContext.tId
    }
}

func init() {
    err := log.RegisterCustomFormatter("TID", createTIdFormatter)
    if err != nil {
         ...
    }
}

and then you could use it in your configs:
<seelog type="sync">
    <outputs formatid="main">
        <console/>
    </outputs>
    <formats>
        <format id="main" format="%TID %Msg%n"/>
    </formats>
</seelog>

If that feature is needed, I'll be able to look at it in a couple of weeks.
If you need it faster, then you could help with a PR. Basically, you just need to add some customContext field to commonLogger, add SetContext to LoggerInterface and implement it for commonLogger, so that it just writes to the customContext field. Then you just have to add CustomContext to LogContextInterface and customContext field to logContext and errorContext. Last step is to change specifyContext and add customContext as a parameter and then here change the  specifyContext(stackCallDepth + cLogger.addStackDepth) call to specifyContext(stackCallDepth + cLogger.addStackDepth, cLogger.customContext).
But if you could wait, I'll do it myself in August.

Thanks @goodsign . I will give it a try.

I will give it a try. Thanks @goodsign  for pointing me to the relevant code sections.

I'm using seelog in development of a new product.  We just had a new requirement to be able to forward logs to an aggregator (e.g., graylog, logstash), which is typically done via JSON structured logs.  It would be really great to have improved support for this.
What's been discussed so far on this issue is the ability to set a custom context on a logger, which will add those fields to every log message.  I think this does not quite go far enough, as there would also be a need to add custom fields per log message.  I'd love to see something similar to the log15 library:
This is the Ctx type declaration:
type Ctx map[string]interface{}

Which can then be used to set key/value pairs on the logger (global to all log messages created by the logger), and/or the log message (unique to the log message):
srvlog := log.New(log.Ctx{"module": "app/server"})
srvlog.Warn("abnormal conn rate", log.Ctx{"rate": curRate, "low": lowRate, "high": highRate})

Hey!
Basically, this is possible, however we should think about not getting too complex and trying to fit all possible features into one package and ending up with a cluttered interface.
I haven't had a chance to look at the issues in August, because I had to leave for another couple of months, unfortunately (until October). So, PRs are still accepted.

Probably we could start with a logger-level context and then think about message-level context.

Hey!
I returned to my normal schedule, so I'm going to look at this issue today, stay tuned.

I've added SetContext to LoggerInterface. You can now use it to set custom context to any interface and then call CustomContext on LogContextInterface to get it inside your formatter. Please check it out and tell me if everything is ok.

Hi @goodsign  I am using seelog in our new project and it is really a powerful log library. However, I got stuck on this problem too. I am afraid the custom context way maybe not as useful as you think. In our situation, there are many concurrent goroutines using a shared logger. So I cannot just call logger.SetContext in one goroutine because this may overwrite the context of another. Do you have suggestions about how to solving this ?

Hi @shafreeck!
Usually the seelog way of handling such cases is one logger per goroutine/per handler/etc. In our examples we show web servers creating a logger per request. For performance issues I thought about implementing the fast copy for loggers (to avoid deserializing from config every time).
We don't have other means to do this at the moment.

If that's too much overhead, let's think about possible solutions. Maybe we could take the idea described above and allow &Context as last parameter to any of the existing funcs. For example, now you do:
seelog.Debugf("Something %s: %s", string1, string2)

, but you can possibly do it this way:
seelog.Debugf("Something %s: %s", string1, string2, seelog.CustomContext{something: "smth1", something2: "smth2"})

Let me know what you think.

@goodsign I did not see the web server example you mentioned above. Could you give me a link address? I think the optional context argument is a good idea. It is compatible with the original apis and supplies a powerful way to custom the log format. However , there is also something painful, users have to pass context as argument every call of logging.

This is the example I was talking about.
Yeah, that's why currently the main path is setting the context, but it may not work for complex asynchronous cases with multiple goroutines with one logger.

@goodsign
Thanks your great seelog library!
It seems that the struct named cfgParseParams can't be exported in current version.
Please refer to your examples: "parserParams := &log.CfgParseParams{", here is CfgParseParams.
Do you plan exported the CfgParseParams struct? Thanks.

Hey @skypacer210,
Made it exported, check it out!

Hey @debraj-manna,
This issue was left here for a while, but I believe I've implemented what was needed, so I'm closing the issue now, please reopen if needed.

Got it, thanks @goodsign !


golang-docker: update Dockerfile to latest Managed VM base image.

Change-Id: I10602aad6bf370ce83bccf68f5d8e4750792366d

Let's hold off on this PR until we can figure out how the current contents can be handled.


fix(deps): update dependency graphql-tag to v2.11.0

This PR contains the following updates:

Package
Type
Update
Change

graphql-tag
dependencies
minor
2.10.4 -> 2.11.0

Release Notes

apollographql/graphql-tag
v2.11.0
Compare Source

package.json sideEffects changes to clearly identify that graphql-tag doesn't have side effects.
@hwillson in #313

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v5.1.10
GitHub release

Your semantic-release bot :package::rocket:


Update stepper in storefront

Merge Stepper usage page in storefront

Update Figma link in guideline page

Related to #845

Is this still relevant @equinor/eds-ux ?

If you publish the updates from our last comment then this one is closed @mimarz

If you publish the updates from our last comment then this one is closed @mimarz

This will not be published as Storefront is in a broken state now, and will not be fixed due to being replaced by Zeroheight. Your are better off just copying the text to Zeroheight now @lucasveil


[Port Request] 1.17.1

Thanks for the great mod.
Would it be possible to port this mod to 1.17.1. I tried to do it myself for 3 - 4 hours but I am not experienced in Minecraft modding. I was rewrite the mod do use the older version of forge-config-api-port-fabric. But I didn't get Minecraft running.
Thank you in advance.

I might consider it at some point but it is not a priority right now.


[Base] Add dispose last flag

This is so that we can, for instance, dispose the API last.
The UI client will then be able to call the API right up until the last feature is disposed.

What happens if component is disposed but UI client calls API that uses that component? I think that might lead to crashes in some cases.

Closing as another solution has been implemented in the UI.
https://github.com/stratisproject/StraxUI/pull/69


adding related concept model

Implemented RelatedConcept model.

closes #33

Thanks @HassanAkbar !

@HassanAkbar can you please incorporate #25 into this PR so that we don't miss functionality? Thanks.


Can snapshots dircetory be configured?

Hello, and thanks for a nice looking script.
I wonder, am I able to configure the snapshopshot directory?
I'd like to save snapshots in a mounted subvol '.snapshots'.
I assume that the snapshots are saved in the root folder / by default.
cheers, john

This is not currently provided, but you can change the path to any in the mount section of the script yourself.
I will think about how to add this to the config file in the future.


How do I choose a style to use?

This will probably sound very dumb. I have been staring at the readme and the help of nightfox to try and figure out how to use nordfox style.
Am I missing something obvious here?

It looks like they are all different color schemes. Answering my own question :colorscheme nordfox


Support a curated list of directives: const [ ... ]

This is a meta-tracking issue.
I know angular_analyzer_plugin (/cc @MichaelRFairhurst) supports a bit of this.

[ ] Emit warnings (later errors?) for unused directives and components:

@Component(
  selector: 'not-used',
  template: '',
)
class NotUsed {}

@Component(
  selector: 'example',
  directives: const [
    NotUsed,
  ],
  template: '',
)
class Example {}

[ ] Emit warnings (later errors?) for tag names that aren't matched.

This is a little harder. We do this "for free" for structural directives today, but we don't for normal components or directives. There are a couple options here:

Add a tag whitelist, and allow the user to supplement it in compiler options.
Support an alternative way to render an angular component. My pick:

<div>
  <HelloWorld></HelloWorld>
</div>

(Capitalized tags are assumed to be components, and don't require the selector).

?????

the plugin doesn't actually expose this yet!
Its on my wishlist, but it complicates the dependency order (analyze dart,
in order to be able to analyze html, in order to finish analyzing dart...).
Its not that complicated but its been enough to keep it off my next-in-line
queue...though this would usefully also allow me to validate
@ViewChildren().
On Mon, Feb 26, 2018 at 3:22 PM, Matan Lurey<EMAIL_ADDRESS>wrote:

This is a meta-tracking issue.
I know angular_analyzer_plugin (/cc @MichaelRFairhurst
https://github.com/michaelrfairhurst) supports a bit of this.

Emit warnings (later errors?) for unused directives and components:

@Component(
selector: 'not-used',
template: '',
)class NotUsed {}
@Component(
selector: 'example',
directives: const [
NotUsed,
],
template: '',
)class Example {}

Emit warnings (later errors?) for tag names that aren't matched.

This is a little harder. We do this "for free" for structural
directives today, but we don't for normal components or directives. There
are a couple options here:

Add a tag whitelist, and allow the user to supplement it in compiler
options.
Support an alternative way to render an angular component. My pick:

(Capitalized tags are assumed to be components, and don't require the
selector).

?????

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/angular/issues/936, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABjWe0VdRcjkTXa-nbnI0SPFNK9b8r1Eks5tYzyegaJpZM4SUGKT
.

We are tracking this elsewhere.


Error code

Every time i put my Instagram information in i wait about a minute and the it gives me a error code like this
Error: connect ETIMEDOUT <IP_ADDRESS>:443
I don't know what i'm doing wrong can you help

@DMD003 since this is a timeout issue, can you confirm this has not to do with just the internet connection of the device from which you run the app?

it shouldn't be the connection because I have disconnected and then reconnected to my device to the internet.

but now its giving me a different error code
Error: getaddrinfo ENOTFOUND i.instagram.com i.instagram.com:443


Failed to build with sccache wrapper

Hello, I'm using sccache as rustc-wrapper:
# contents of .cargo/config

[build]
rustc-wrapper = "sccache"

This causes an error while using cargo-auditable:
error: failed to run `rustc` to learn about target-specific information

Caused by:
  process didn't exit successfully: `sccache /usr/bin/cargo-auditable rustc - --crate-name ___ --print=file-names --crate-type bin --crate-type rlib --crate-type dylib --crate-type cdylib --crate-type staticlib --crate-type proc-macro --print=sysroot --print=cfg` (exit status: 2)
  --- stderr
  sccache: error: failed to execute compile
  sccache: caused by: Compiler not supported: "Unrecognized command: \"-E\"\n"

When I disable sccache, everything works as expected. Is there a way to work around this?
Additional Info
Rust version: rustc 1.66.0-nightly (6e95b6da8 2022-10-22)
cargo-auditable version: 0.5.2

This is known (cf. #55) and has been fixed in sccache (https://github.com/mozilla/sccache/pull/1280) but the fix was not released yet. The current workaround is to use build sccache from its main branch.

Thanks!


多边形周长面积问题

您好,我看到您库中有length和area,我想他们分别代表周长和面积,但是当我传进入一个多边形经纬度顶点进去以后,他们两个算出来的结果都是很小的数字,类似0.02323....我想知道这两个分别如何使用,他们返回的单位又是什么  这是我构建多边形的方式
let polygon = try! Geometry(wkt: "POLYGON((39.984118 116.297110,39.984145 116.297936,39.983892 116.297947,39.983876 116.297110,39.984118 116.297110))")
return try! polygon.area()
得出的结论都是很小的小数

Hi @fenghedemo1, I'm going off of the following Google Translate output, so please let me know if I'm answering the wrong question:

Hello, I see that there are length and area in your library. I think they represent the perimeter and area respectively, but when I pass in the latitude and longitude vertices of a polygon, the results of the two calculations are very small numbers. Similar to 0.02323.... I want to know how these two are used, and what is the unit they return. This is how I construct the polygon
let polygon = try! Geometry(wkt: "POLYGON((39.984118 116.297110,39.984145 116.297936,39.983892 116.297947,39.983876 116.297110,39.984118 116.297110))")
return try! polygon.area()
The conclusions reached are all very small decimals

GEOSwift geometry is always done on the Cartesian plane. So the output that you get is just arbitrary squared units. For example, if you took a 2x2 square POLYGON((0 0,0 2,2 2,2 0,0 0)) you should get an area of 4.
You'll need a different library (or your own implementation) to convert this to something like square meters.

Hi @fenghedemo1, I'm going off of the following Google Translate output, so please let me know if I'm answering the wrong question:

Hello, I see that there are length and area in your library. I think they represent the perimeter and area respectively, but when I pass in the latitude and longitude vertices of a polygon, the results of the two calculations are very small numbers. Similar to 0.02323.... I want to know how these two are used, and what is the unit they return. This is how I construct the polygon
let polygon = try! Geometry(wkt: "POLYGON((39.984118 116.297110,39.984145 116.297936,39.983892 116.297947,39.983876 116.297110,39.984118 116.297110))")
return try! polygon.area()
The conclusions reached are all very small decimals

GEOSwift geometry is always done on the Cartesian plane. So the output that you get is just arbitrary squared units. For example, if you took a 2x2 square POLYGON((0 0,0 2,2 2,2 0,0 0)) you should get an area of 4.
You'll need a different library (or your own implementation) to convert this to something like square meters.


Errors when run evaluation scripts with checkpoints loaded from huggingface

Describe the bug
Hello, When I want to run https://github.com/SalesforceAIResearch/uni2ts/blob/main/project/moirai-1/eval/lsf_small.sh to evaluate moirai with checkpoints from huggingface, a problem occured: "type object 'MoiraiModule' has no attribute 'from_pretrained'".
The problem seemingly exists in https://github.com/SalesforceAIResearch/uni2ts/blob/main/cli/conf/eval/model/moirai_1.0_R_small.yaml, where "from_pretrained" is not defined in MoiraiModule.
To Reproduce
Error report.
[2024-07-17 08:23:03,525][datasets][INFO] - PyTorch version 2.2.2 available.
[2024-07-17 08:23:03,527][datasets][INFO] - JAX version 0.4.26 available.
[2024-07-17 08:23:04,055][root][INFO] - using dataset already processed in path /home/lefeishen/.gluonts/datasets/us_births.
Error executing job with overrides: ['run_name=monash_eval', 'model=moirai_1.0_R_small', 'model.patch_size=32', 'model.context_length=1000', 'data=monash', 'data.dataset_name=us_births']
Traceback (most recent call last):
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/utils.py", line 644, in _locate
obj = getattr(obj, part)
AttributeError: type object 'MoiraiModule' has no attribute 'from_pretrained'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/instantiate/_instantiate2.py", line 134, in _resolve_target
target = _locate(target)
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/utils.py", line 661, in _locate
raise ImportError(
ImportError: Error loading 'uni2ts.model.moirai.MoiraiModule.from_pretrained':
AttributeError("type object 'MoiraiModule' has no attribute 'from_pretrained'")
Are you sure that 'from_pretrained' is an attribute of 'uni2ts.model.moirai.MoiraiModule'?
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/home/lefeishen/uni2ts_reproduce/cli/eval.py", line 258, in
main()
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/main.py", line 90, in decorated_main
_run_hydra(
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/utils.py", line 394, in _run_hydra
_run_app(
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/utils.py", line 457, in _run_app
run_and_report(
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/utils.py", line 222, in run_and_report
raise ex
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/utils.py", line 219, in run_and_report
return func()
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/utils.py", line 458, in
lambda: hydra.run(
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/hydra.py", line 132, in run
_ = ret.return_value
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/core/utils.py", line 260, in return_value
raise self._return_value
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/core/utils.py", line 186, in run_job
ret.return_value = task_function(task_cfg)
File "/home/lefeishen/uni2ts_reproduce/cli/eval.py", line 73, in main
model = call(cfg.model, partial=True, convert="all")(
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/instantiate/_instantiate2.py", line 226, in instantiate
return instantiate_node(
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/instantiate/_instantiate2.py", line 342, in instantiate_node
value = instantiate_node(
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/instantiate/_instantiate2.py", line 333, in instantiate_node
target = _resolve_target(node.get(_Keys.TARGET), full_key)
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/hydra/_internal/instantiate/_instantiate2.py", line 139, in _resolve_target
raise InstantiationException(msg) from e
hydra.errors.InstantiationException: Error locating target 'uni2ts.model.moirai.MoiraiModule.from_pretrained', set env var HYDRA_FULL_ERROR=1 to see chained exception.
full_key: model.module
^C^CTraceback (most recent call last):
File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/home/lefeishen/uni2ts_reproduce/cli/eval.py", line 18, in
import torch
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/torch/init.py", line 1427, in
_C._initExtension(manager_path())
File "/home/lefeishen/uni2ts_reproduce/venv/lib/python3.10/site-packages/torch/cuda/init.py", line 1138, in
class HalfStorage(_CudaLegacyStorage):
KeyboardInterrupt
Environment

Python version: 3.10.12
PyTorch version: 2.2.2
Lightning version: 2.2.2
uni2ts version (have you pulled the latest version?): yes

please pull the latest version of the repo, and also reinstall the dependencies


Connection closed by remote peer Code: 1006 Reason:

Connection closed by remote peer Code: 1006 Reason:

I have the same problem in version 1.1.5


epub validated file

plz resolve this error, replay soon

Regarding your first screenshot: You cannot upload OPF files. The IDPF online EpubCheck only supports checking .epub files. If you want to validate .opf files you need to use the EpubCheck commandline tool.
Regarding your second screenshot: That's most likely an error in your ZIP mechanism. The path in the column "file" leads to the assumption, that you ziped your epub file twice...

sir .opf  file not send, so will send .zip extension file

what is zip mechanism

what is epub check command line tool
how to do this  epub check command line tool

what is epub check command line tool
how to do this epub check command line tool

Please have a look in the EpubCheck Wiki for further instructions on how to run EpubCheck in your Terminal: https://github.com/IDPF/epubcheck/wiki

sir .opf file not send, so will send .zip extension file

You're right, you uploaded Ganesh.epub file.
However, you cannot just rename .zip to .epub. You have to zip the EPUB contents in a specific order. Let me google that for you: http://lmgtfy.com/?q=how+to+zip+epub
Best explanation: https://ebookflightdeck.com/handbook/zipping

Anyways, this issue tracker is for issues with EpubCheck, it's not a support forum for EPUB questions.
If you're having trouble to create an EPUB file, please visit the community forum at http://idpf.org/forums/epub-3 and ask your question there...

sir
What is zip mechanism ?
does not solved this error

What is zip mechanism ?

One last time: You have to zip the EPUB contents in a specific order. Let me google that for you: http://lmgtfy.com/?q=how+to+zip+epub
Best explanation: https://ebookflightdeck.com/handbook/zipping
Again: This is NOT a support forum for questions on how to create an EPUB file!!
If you're having trouble to create an EPUB file, please visit the community forum at http://idpf.org/forums/epub-3 and ask your question there...


Request null default value in Controller overrides the object in routeDependencyResolverTrait

Laravel Version: 5.6.39 ( happens since ver 5.5)
PHP Version: 7.2.2
Database Driver & Version: MYSQL,  Ver 14.14 Distrib 5.7.21

Description:
Hi guys, thank you for working on the framework, it's awesome :)
During upgrading an older project from 5.3 I got a problem:
I have a couple of functions in controllers that I'd call through ajax and sometimes manually from a controller.
For this, I'd need to add a default value for the Request parameter of the functions.
Unfortunately the routeDependencyResolverTrait respects the default value against the passed value, so I'm getting $request == null even If I send the parameters.
The 'issue' is below, it seems this function override the passed value with the default value that I don't understand why. Shouldn't be respected the passed value against the default and get the default value if it has no value?
    protected function transformDependency(ReflectionParameter $parameter, $parameters)
    {
        $class = $parameter->getClass();

        // If the parameter has a type-hinted class, we will check to see if it is already in
        // the list of parameters. If it is we will just skip it as it is probably a model
        // binding and we do not want to mess with those; otherwise, we resolve it here.
        if ($class && ! $this->alreadyInParameters($class->name, $parameters)) {
            return $parameter->isDefaultValueAvailable()
                ? $parameter->getDefaultValue()
                : $this->container->make($class->name);
        }
    }

I changed this function, deleted the condition, after this I got the expected result:
return $this->container->make($class->name);
I know adding default parameter for the Request parameter is not the best solution but I think it should be possible though.
My concrete example (I need the Request for the JWAuth if user id hasn't passed):
Route:
Route::get('/users/{id}/photos', 'UserController@getPhotos')->where('id', '^([0-9]+)|(me)$');
URL:
/api/users/1/photos?limit=5&orderBy=&page=1
Function:
    public function getPhotos($id=false,Request $request = null)
    {
            if(empty($id) || $id === 'me'){
                JWTAuth::parseToken()->authenticate();
                $user = Auth::user();
            }else{
                $user = User::find($id);
            }
         //......
     }

Steps To Reproduce:
See my example above

I have a couple of functions in controllers that I'd call through ajax and sometimes manually from a controller

I'm trying to understand this.
Are you saying you are trying to call a function inside Controller A from Controller B?

@laurencei
Yes, exactly.
Sometimes I'd call a function inside Controller A from Controller A, sometimes from Controller B, sometimes through ajax request.
Thank you for the clarifying.
The real question is now, why transformDependency() respects the default value against the passed value in the Treat?
The PHP reference suggests I should be able to use the default value for the Request parameter:
http://php.net/manual/en/functions.arguments.php

To be honest - this is well outside the scope of support. Laravel controllers are designed to be called once per request cycle, via a HTTP request (including AJAX).
They are not designed to be called as functions from other controllers (or other classes). While this might have worked in the past, it was never a supported function.
Without trying to sound rude, this is a bad design layout, and you should refactor your code, extracting the "function" into a class that can then be called independently of either Controller.
For example:
class ControllerA extends Controller
{
    public function index() {
           resolve(ExtractedClass::class)->someFunction();
    }
}

class ControllerB extends Controller
{
    public function index() {
           resolve(ExtractedClass::class)->someFunction();
    }
}

class ExtractedClass
{
     public function someFunction()
     {
         // Your logic here
     }
}

Dear @laurencei,
thank you for your answer, I really appreciate it, but this issue is not about the best practice of using a controller or structuring the app, maybe you misunderstood a bit (sorry for my English):
I think there is a logical mistake (or I don't get the point) with this part:
            return $parameter->isDefaultValueAvailable()
                ? $parameter->getDefaultValue()
                : $this->container->make($class->name);

There is no sense to overwrite the sent ( and excepted) value with the default value of the parameter that's clearly should be handled in the opposite way, as the PHP reference suggests.

Try swapping the arguments in the controller method. It's best to keep the parameters last and other dependencies first. The Request class is always resolved. You're also using a fallback for the id but it's always required in your route so it can't have a fallback.
   public function getPhotos(Request $request, $id)
    {
            //...
     }

@driesvints
I don't understand, how I can call this function from another Controller without passing a Request in the first parameter? This is not okay, but this isn't even my question. I'm telling this third time now:
This ticket is about the logic of the routeDependencyResolverTrait's transformDependency() method is wrong, because it overwrite the passed value of the function with a default value. This is clearly a mistake in the logic since the default value should be used only if the variable(parameter) is empty/null/undefined.
I've even linked the regarding part of the PHP documentation about the function's default value how should work, and some reason Laravel goes against this logic and basically don't let the developer to pass default value because it will ALWAYS overwrite the passed value.
PHP reference says:
Example #3 Use of default parameters in functions

<?php
function makecoffee($type = "cappuccino")
{
    return "Making a cup of $type.\n";
}
echo makecoffee();
echo makecoffee(null);
echo makecoffee("espresso");
?>
The above example will output:

Making a cup of cappuccino.
Making a cup of .
Making a cup of espresso.

In our case ALL of the outputs would be:
Making a cup of cappuccino.
Making a cup of cappuccino.
Making a cup of cappuccino.

See my previos comment where I offered a solution how the regarding code should be changed to respect the added/passed/assigned value against the default value if it has one.


Updated wording

Updated wording

cc: @MicrosoftGuyJFlo
GitHub Issue: https://github.com/MicrosoftDocs/azure-docs/issues/86526

@JamesTran-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MicrosoftGuyJFlo Is my edit correct or does it change the meaning?
Administrators can exclude the entire Office 365 suite or specific Office 365 client apps from the Conditional Access policy.
Thanks.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


motors getting locked with using with intel joule and openCR (turtlebot3)

Hello all,
I recently bought turtleobt3, it came with dynamixel motors Xm430-w210-t, intel joule and openCR board. I followed the tutorial and done all the steps.
when i launched turtlebot3 bringup and lauched teleoperation it didn't show any errors in the terminal, but the turtlebot didn't move at all when i pressed the keys. I can see the output on /odom topic.
It looks like motors got locked. I tired rotating them manually but they didn't rotate. I didn't force, i thought i might break something inside.
Can anyone help me what's happening?
Thanks

Hi :)
Firmware version 1.1.0 has bug to control dynamixel.
We hotfixed it and now you can download v.1.1.1.
We apologize for any inconvenience.
Thanks
Darby

where and how should i download it?

Hello, @allanis79
You can download it using arduino or shell.
Please refer to e-manual for details.

thanks, I will try it tomorrow. @OpusK

This issue will be closed since there were no actions for a while.
You can reopen this issue to show this issue to the users whenever.
Thanks. :)


encoding of events in XTA is wrong

🐛 The encoding of 4-kind events into 2-kind events in XTA is wrong.
It generates wrong traces eg for

define

	basic component type A is
		signature
			messages m1, m2;
			provides x(m1):m2, y(m1):m2;
		behavior
			start in s0;
			stop in eos;
			s0: receive x then doA,
				receive y then doB;
			doA: reply x then eos;
			doB: reply y then eos;
			eos:;
	end

	basic component type B is
		signature
			messages m1, m2;
			provides x(m1):m2, y(m1):m2;
		behavior
			start in s0;
			stop in eos;
			s0: receive x then doA,
				receive y then doB;
			doA: reply x then eos;
			doB: reply y then eos;
			eos:;
	end

	basic component type Env is
		signature
			messages m1, m2;
			requires x(m1):m2, y(m1):m2;
		behavior
			start in s0;
			stop in s4;
			s0: invoke x then s1;
			s1: result x then s2;
			s2: invoke y then s3;
			s3: result y then s4;
			s4:;
	end

	composite component type ClosedSystem is
		signature
		subcomponents
			e: Env, a:A, b: B;
		bindings
			@1: e.x --> a.x,
			@2: e.y --> b.y;
	end

	in c: ClosedSystem

we get a trace: e--x-->a ; a--x-->b ; b --x-->e.
💡 the error is in the TA to XTA model to text transformation. It should respect what is given in the paper.

closed by https://github.com/pascalpoizat/veca-haskell/commit/28f5af588777bee1ce3533c6e3e4d7fba70ce9b1


Convention for W is not consistent between orbital_weights and vertex_pk ?

https://github.com/DavisThuillier/Ludwig.jl/blob/e12cc78de30933ce5456febaf2c79c125132d167/scripts/materials/Sr2RuO4.jl#L94
The calculation of W in this function orbital_weights is not super transparent because it uses some unrelated elements of W as scratch space. I imagine this is to save time, although I am not sure if it is needed since mesh generation is probably pretty fast compared to integration?
Nonetheless, it is not obvious to me that the convention for eigenvectors used in this function is the same as the you use in vertex_pk for the momentum which is not on a patch, for which the second element of each eigenvector is chosen to always be positive.

This use of already allocated but non-referenced memory was important earlier when orbital_weights() was directly referenced in each integration loop, but this requirement is now deprecated since Fpk and Fpp are referenced instead without calls to orbital_weights(). Gauge disagreement fixed in iss6 branch and merged into develop.


Add Chinese-Simplified translation

DE BG7QWH and thank you for your awsome SSTV app

Okay and I will do some changes according to your comments :)

Change submitted, please see if there's another problem

Thank you for the update.
Just one more thing:
translatable="false" for "app_name" is missing

@olgamiller  OK, see if there's another thing to do :)

Thank you.
Unfortunately, I've noticed a few issues (that existed before the changes, but are now “visible” with the changes):

The "size" string will be used for both, Font and Outline (and therefore "字体大小" will also be used for Outline Size)
The "color" string will be used instead of "outline_color" for the Outline Color
The following values are not in the strings.xml (and therefore will not be translated):

EditTextActivity:

{"Small", "Normal", "Large", "Huge"}
{"Thin", "Normal", "Thick"}

FontFamilySet: "Default"

Would you like to fix these issues as well, all of them or a few? Otherwise, I'll merge this PR as it is now and fix the issues afterwards. Please let me know what you would prefer.
Additionally, I could not test the Font selection with System Language 简体中文. My emulator gets the "Default" item only, in the Font selection list. Would it be possible for you to check if it works as expected?

Thank you.
Unfortunately, I've noticed a few issues (that existed before the changes, but are now “visible” with the changes):
* The "size" string will be used for both, Font and Outline (and therefore "字体大小" will also be used for Outline Size)

* The "color" string will be used instead of "outline_color" for the Outline Color

* The following values are not in the strings.xml (and therefore will not be translated):

  * EditTextActivity:

    * {"Small", "Normal", "Large", "Huge"}
    * {"Thin", "Normal", "Thick"}
  * FontFamilySet: "Default"

Would you like to fix these issues as well, all of them or a few? Otherwise, I'll merge this PR as it is now and fix the issues afterwards. Please let me know what you would prefer.
Additionally, I could not test the Font selection with System Language 简体中文. My emulator gets the "Default" item only, in the Font selection list. Would it be possible for you to check if it works as expected?

Oh I got what happend: string "size" and "colour" have been used twice and I failed to notice it😂, I will now update the translation and test it, screenshot and new commit will be done soon

@olgamiller My IDE notices me that 10 warnings generated about activity_edit_text.xml, maybe some fixes are needed but at least now that two strings work on my phone

Check if there's any issues and I will try my best to assist

Yeah, this project is old, Android has changed a lot, therefore the warnings everywhere.
Your changes are merged. Thanks for the fixes, and for the translations.
One question: Would the following translation make sense?

"Small" -> “小的”
"Normal" -> “普通的”
"Large" -> “大的”
"Huge" -> “巨大的”
"Thin" -> “薄的”
"Normal" -> “普通的”
"Thick" -> “厚的”
"Default" -> “默认”

I've added missing strings to the string resources (see the latest updates). May I ask you to check whether the translation makes sense:
    ...
    <string name="progressbar_message_sending">发送…</string>
    <string name="progressbar_message_saving_to_file">%1$s 保存…</string>
    ...
    <string name="font_default">默认</string>
    ...
    <string name="font_size_small">小的</string>
    <string name="font_size_normal">普通的</string>
    <string name="font_size_large">大的</string>
    <string name="font_size_huge">巨大的</string>
    ...
    <string name="outline_size_thin">薄的</string>
    <string name="outline_size_normal">普通的</string>
    <string name="outline_size_thick">厚的</string>

(Please feel free to update/fix via another PR if necessary. Thanks.)

Yeah, this project is old, Android has changed a lot, therefore the warnings everywhere. Your changes are merged. Thanks for the fixes, and for the translations.
One question: Would the following translation make sense?
* "Small" -> “小的”

* "Normal" -> “普通的”

* "Large" -> “大的”

* "Huge" -> “巨大的”

* "Thin" -> “薄的”

* "Normal" -> “普通的”

* "Thick" -> “厚的”

* "Default" -> “默认”

Generally speaking, that translation makes sense. But in many cases it depends on which kind of objects you're describing. For example, about the description of text or font, if you translate "huge" into "巨大的"—— although it might still make sense, but some how makes people feel strange to see expression like that and we call it "translation tone（翻译腔）" so in this example I prefer to translate "huge" into "特大号" like "XXL" size of clothes

@WeiguangTWK: Could you please provide an improved translation of the following strings (so the Chinese-Simplified translation would be really complete):
    <string name="progressbar_message_sending">发送…</string>
    <string name="progressbar_message_saving_to_file">%1$s 保存…</string>
    ...
    <string name="font_default">默认</string>
    ...
    <string name="font_size_small">小的</string>
    <string name="font_size_normal">普通的</string>
    <string name="font_size_large">大的</string>
    <string name="font_size_huge">巨大的</string>
    ...
    <string name="outline_size_thin">薄的</string>
    <string name="outline_size_normal">普通的</string>
    <string name="outline_size_thick">厚的</string>

@olgamiller Ok, here we go
    <string name="progressbar_message_sending">发送…</string>     <string name="progressbar_message_saving_to_file">%1$s 另存为文件</string>     ...     <string name="font_default">默认字体</string>     ...     <string name="font_size_small">小</string>     <string name="font_size_normal">常规</string>     <string name="font_size_large">大</string>     <string name="font_size_huge">很大</string>     ...     <string name="outline_size_thin">细</string>     <string name="outline_size_normal">常规</string>     <string name="outline_size_thick">粗</string>

@olgamiller Ok, here we go
‘
发送…
%1$s 保存…
...
默认
...
小的
普通的
大的
巨大的
...
薄的
普通的
厚的
’

@olgamiller Ok, here we go
(Since some problem occured I deleted and reedit my reply, never mind)
    <string name="progressbar_message_sending">发送…</string>     <string name="progressbar_message_saving_to_file">%1$s 另存为文件</string>     ...     <string name="font_default">默认字体</string>     ...     <string name="font_size_small">小</string>     <string name="font_size_normal">常规</string>     <string name="font_size_large">大</string>     <string name="font_size_huge">很大</string>     ...     <string name="outline_size_thin">细</string>     <string name="outline_size_normal">常规</string>     <string name="outline_size_thick">粗</string>

@WeiguangTWK: Thank you, I've implemented your suggestions.


New rule: 'no-literals'

This is new style rule that enforces wrapping literal strings in a JSXExpressionContainer element.

{'text'}
 and not allowing
text
.
Besides being sure of the actual rendered text, this also helps removed problems with some highlighters and JSX.

{"this can't possibly be a problem"}

this can't possibly be a problem

As you see the ' causes problems with the rest of the code, despite being valid.
I'm unsure if no-literals is an appropriate name, but since that seems to be the name of the node type I went with it.

Seems good, but since it's a JSX related rule I think the rule name should be prefixed by jsx-: jsx-no-literals

@yannickcr Renamed. Any other suggestions?

LGTM, Thanks!


Can't connect to mirror with "--insecure-registry http://example.com:5000".

I have try to use registry mirror. Docker client is 1.7.0,and registry is 2.0.
I have try to
--insecure-registry http://example.com:5000
--insecure-registry example.com:5000
--insecure-registry=example.com:5000
It didn't work at all.just throw error, and you can see it in this link http://stackoverflow.com/questions/31517388/how-to-create-docker-registry-mirror-on-centos:

INFO[0025] POST /v1.19/images/create?fromImage=hello-world%3Alatest
ERRO[0029] Unable to create endpoint for http://<IP_ADDRESS>:5555/: invalid registry endpoint https://<IP_ADDRESS>:5555/v0/: unable to ping registry endpoint https://<IP_ADDRESS>:5555/v0/
v2 ping attempt failed with error: Get https://<IP_ADDRESS>:5555/v2/: EOF
v1 ping attempt failed with error: Get https://<IP_ADDRESS>:5555/v1/_ping: EOF. If this private registry supports only HTTP or HTTPS with an unknown CA certificate, please add --insecure-registry <IP_ADDRESS>:5555 to the daemon's arguments. In the case of HTTPS, if you have access to the registry's CA certificate, no need for the flag; simply place the CA certificate at /etc/docker/certs.d/<IP_ADDRESS>:5555/ca.crt

https://github.com/docker/distribution/blob/master/docs/deploying.md#2-instruct-docker-to-trust-your-registry-as-insecure
Use exactly that syntax. This is a daemon flag, so you need to restart your daemon. Finally, if you use an ip, specify an ip, if you use a domain name, same thing, that needs to be consistent.
Hope that helps.

And this github repository is clearly marked as deprecated.
Registry 2 repository is linked on the front page.


[BugFix] Fix bug dynamic partition date type does not support hour (backport #40256)

Why I'm doing:
Dynamic partitioning supports hour, but it does not consider that date type partitioning cannot be supported.
What I'm doing:

When creating a table, directly prevent such prompts from being created.
Provide friendly error prompts for stock creation.
Add a little more coverage for other issues that may go wrong.

Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[x] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[x] 3.1
[x] 3.0
[x] 2.5

https://github.com/Mergifyio backport branch-3.1


Raise ArgumentError when default has invalid type

This is achieved through a dump on the default value. We ignore nil
values and empty lists. Parameterized types use their own dump
implementation.
I added a failing test so we can discuss something: I noticed some
parameterized types will raise on dump if the input is not of the
expected type. What shall we do in this scenario? Catch those
errors and build the default argument error?

Oh, and this closes #3483

I have added some comments. Can you please look at the tests too? Thank you!

Hi, José!
About the failing test, I added it because I wanted an opinion. Some
parameterized types will raise instead of returning an error in the dump/2
function when receiving invalid input. The failing test reproduces this
scenario. Should I catch those errors?
About the clauses, I'll try to improve them
Em sex, 20 de nov de 2020 05:29, José Valim<EMAIL_ADDRESS>escreveu:

I have added some comments. Can you please look at the tests too? Thank
you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/elixir-ecto/ecto/pull/3487#issuecomment-731022764,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSBOVW6M6GWFLCC6OO2EXDSQYSGRANCNFSM4T4DSY6Q
.

I think we can just let them raise for now, yes.

I believe it is ready now. If there is any other issue, let me know :)

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Cannot download download onnx because input tensor data's shape [N, 3, 224, 224] is not valid

Hi,
I'm facing an error while trying to download a modified model.
How to get the error

download resnet18-v1-7.onnx
with python, modify its opset

resnet18 = onnx.load('path/to/resnet18-v1-7.onnx')
resnet18 = version_converter.convert_version(resnet18, 14)
onnx.save(resnet18, "path/to/resnet18-v1-14.onnx")

upload resnet18-v1-14.onnx to onnx-modifier
delete every thing below the last relu (at this point, i can still download the modified model)

add an output at the end of modified model

Then I cannot download

Error message
In the terminal, the error message is as follows:
[2023-08-03 10:21:45,202] ERROR in app: Exception on /download [POST]
Traceback (most recent call last):
  File "/home/sd/ml/onnx-modifier/lib/python3.10/site-packages/flask/app.py", line 2525, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/sd/ml/onnx-modifier/lib/python3.10/site-packages/flask/app.py", line 1822, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/sd/ml/onnx-modifier/lib/python3.10/site-packages/flask/app.py", line 1820, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/sd/ml/onnx-modifier/lib/python3.10/site-packages/flask/app.py", line 1796, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
  File "/home/sd/ml/onnx-modifier/app.py", line 27, in modify_and_download_model
    onnx_modifier.modify(modify_info)
  File "/home/sd/ml/onnx-modifier/onnx_modifier.py", line 337, in modify
    self.add_outputs(modify_info['added_outputs'])
  File "/home/sd/ml/onnx-modifier/onnx_modifier.py", line 199, in add_outputs
    inferred_value_info = shape_inference_using_onnx_tool(copy.deepcopy(self.model_proto))
  File "/home/sd/ml/onnx-modifier/utils/thrid_party_tools.py", line 11, in shape_inference_using_onnx_tool
    g.shape_infer()
  File "/home/sd/ml/onnx-modifier/lib/python3.10/site-packages/onnx_tool/graph.py", line 735, in shape_infer
    raise ValueError(
ValueError: The input tensor data's shape [N,3,224,224] is not valid, Please set it to a valid shape.
<IP_ADDRESS> - - [03/Aug/2023 10:21:45] "POST /download HTTP/1.1" 500 -

Thanks in advance for any help you can provide

@SimonDemarty Thanks for the feedback. It is a runtime error raised by onnx-tool, as it does not support dynamic shape currently.
I have added a fallback strategy. If shape inference using onnx-tool fails, primitive ONNX Python API will be invoked. The code has been updated and it should work now.

It does work! Thank you for the quick reply


intellisense can not find 'wchar.h' on osx big sur

intellisense can not find 'wchar.h' on osx big sur
Brief Issue Summary
Hi!
I am using this extension to configure my CXX project. Recently I upgrade to OSX 11.2. And, it seemed the Intellisense was broken. For exmaple, it failed to find some system headers like  'wchar.h'.  (However, the project can be built using the extension)
I use the following helloworld example
//main.cpp
#include <iostream>

int main()
{
    std::cout << "hello" << std::endl;
    return 0;
}

#CMakeLists.txt
cmake_minimum_required(VERSION 3.3)

project(hello)

add_executable(hello main.cpp)

//.vscode/setting.json
{
    "C_Cpp.default.configurationProvider": "ms-vscode.cmake-tools",
    "files.associations": {
        "iostream": "cpp"
    }
}

Expected:
system headers can be found as expected.
CMake Tools Log
CMake configure
[main] Configuring folder: hello
[proc] Executing command: /usr/local/bin/cmake --no-warn-unused-cli -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -DCMAKE_BUILD_TYPE:STRING=Debug -DCMAKE_C_COMPILER:FILEPATH=/usr/bin/clang -DCMAKE_CXX_COMPILER:FILEPATH=/usr/bin/clang++ -H/Users/zdyang/lab/hello -B/Users/zdyang/lab/hello/build -G Ninja
[cmake] Not searching for unused variables given on the command line.
[cmake] -- The C compiler identification is AppleClang <IP_ADDRESS>00032
[cmake] -- The CXX compiler identification is AppleClang <IP_ADDRESS>00032
[cmake] -- Detecting C compiler ABI info
[cmake] -- Detecting C compiler ABI info - done
[cmake] -- Check for working C compiler: /usr/bin/clang - skipped
[cmake] -- Detecting C compile features
[cmake] -- Detecting C compile features - done
[cmake] -- Detecting CXX compiler ABI info
[cmake] -- Detecting CXX compiler ABI info - done
[cmake] -- Check for working CXX compiler: /usr/bin/clang++ - skipped
[cmake] -- Detecting CXX compile features
[cmake] -- Detecting CXX compile features - done
[cmake] -- Configuring done
[cmake] -- Generating done
[cmake] -- Build files have been written to: /Users/zdyang/lab/hello/buil

Log Diagnostics
-------- Diagnostics - 2/18/2021, 7:13:51 PM
Version: 1.2.1
Current Configuration:
{
    "name": "Mac",
    "includePath": [
        "/usr/local/include"
    ],
    "defines": [],
    "macFrameworkPath": [],
    "compilerPath": "/usr/local/bin/gcc-10",
    "cStandard": "gnu17",
    "cppStandard": "gnu++14",
    "intelliSenseMode": "macos-gcc-x64",
    "compilerArgs": [],
    "intelliSenseModeIsExplicit": true,
    "cStandardIsExplicit": true,
    "cppStandardIsExplicit": true,
    "compilerPathIsExplicit": true,
    "configurationProvider": "ms-vscode.cmake-tools",
    "browse": {
        "path": [
            "/usr/local/include",
            "${workspaceFolder}"
        ],
        "limitSymbolsToIncludedHeaders": true
    }
}
Custom configurations:
[ /Users/zdyang/lab/hello/main.cpp ]
{
    "defines": [],
    "standard": "c++17",
    "includePath": [],
    "intelliSenseMode": "clang-x64",
    "compilerPath": "/usr/bin/clang++",
    "compilerArgs": [
        "-g",
        "-isysroot",
        "/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.1.sdk"
    ]
}
Translation Unit Mappings:
[ /Users/zdyang/lab/hello/main.cpp ]:
    /Users/zdyang/lab/hello/main.cpp
Translation Unit Configurations:
[ /Users/zdyang/lab/hello/main.cpp ]:
    Process ID: 4382
    Memory Usage: 93 MB
    Compiler Path: /usr/bin/clang++
    Includes:
        /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1
        /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/12.0.0/include
        /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include
    Standard Version: c++17
    IntelliSense Mode: macos-clang-x64
    Other Flags:
        --clang
        --clang_version=100000
Total Memory Usage: 93 MB

------- Workspace parsing diagnostics -------
Number of folders and files enumerated: 32748
Number of files discovered (not excluded): 30453

Platform and Versions

Operating System: OSX 11.2
CMake Version:  cmake version 3.18.4
VSCode Version: VS Code 1.53.2
CMake Tools Extension Version: 1.6.0
Compiler/Toolchain: clang++ 12

Other Notes/Information

It looks like cpptools might be having trouble probing your compiler.  I'm not sure why the isysroot wasn't used.  Do you see anything in the C/C++ Configuration Warnings channel of the OUTPUT window?

cc: @Colengms


Negative extra_cull_margin values break depth ordering

Tested versions

Reproducible on 4.2.1 stable

System information
macOS
Issue description
The documentation for GeometryInstance3D doesn't define extra_cull_margin behaviour when set to negative values, and setting it to a negative value doesn't result in any errors. However, it DOES result in an unexplainably broken z-ordering for your quad meshes on the Z axis (and probably a bunch of other things, but I was only using quads for a 2.5D game), presumably because the AABB of the quads was already at size 0 along that axis (meaning that the negative cull margin.. inverted the AABB...?? or made it negative?).
I wasted multiple days trying to figure out what in the world was going on. Things that would have mitigated the issue:

Explicitly documenting behaviour of negative extra_cull_margin values in the docs linked above
Logging an error, or at least a warning when setting a negative extra_cull_margin value
Ignoring extra_cull_margin values that result in negative AABB dimensions(?)

I'm not sure if it's appropriate to throw out negative values or if I'm unaware of a legitimate use case, but at a minimum updating the documentation should be a gimme here.
Steps to reproduce

Set up a scene with a few overlapping quad meshes, at positions -2, -1, 0, 1, 2 along the Z axis
Set extra_cull_margin to -100 on all meshes
Be thoroughly confused when your meshes' draw order gets inverted

Seems I had it backwards 😢
POSITIVE cull margin values bork the depth sorting, and I'm back to square 1 trying to figure out why

Kept digging, better identified issue and made repro project here: https://github.com/godotengine/godot/issues/90351


setup() is required

It probably shouldn't be, and instead display a warning recommending you put loading behavior in setup().

Fixed by #369


Readme fixes

R: @jeffposnick @addyosmani
Fixes #289

Changes LGTM. Thanks, Matt!
Only comment was that the links to demos often take you to a directory just containing an index.js. If we could generate or eventually work on a README/some "next step" to get started with those that would be awesome.


Course material

It would be amazing if the lecture slides and the exercises would be available in addition to the lecture itself. (e.g. MIT OpenCourseWare)

FYI: If the lecturer uploads them, they are avaliable.
Here is a screenshot from the lecturer UI:

From a lecturer point of view, this is quite hidden beind the edit lecture on the top right. In general, I think that the lecturer view is a bit counterintive. (f.ex why are statistics a button next on the settings tab next to "save settings"?)
Also from a lecturers point of view, why would I upload stuff to moodle and tum.live..


Assemblies missing after build.

I currently have a project that has a mixture of PCL (.NetStandard) dependencies and normal .net 4.6 Class library dependencies.
When I build the ASP.net MVC project 3 of the required system dll's are not outputted (namely System.Runtime, System.ObjectModel & System.Collection.Concurrent) to the bin directory of the project.
As seen in the image.

When I build one of the normal .net projects, which itself references a .NetStandard PCL the missing dll's are outputted to the ASP.net bin directory even though the class has no reference to this directory and is the only project built as see below:

How would I be able to get the same result from just building the ASP.net project, which intern causes the class library to build due to being a dependency?

Could you file this issue in https://github.com/dotnet/cli?


Feature: Provide mapping between all-paragraph and visible-paragraph index systems

Given an index in area.getParagraphs(), one should be able to map it to its respective area.getVisibleParagraphs() index. Likewise, the index of the latter should be mappable to the index of the former. Thus, we should add an API like
public final Optional<Integer> allParToVisibleParIndex(int allParIndex);
public final int visibleParToAllParIndex(int visiblePar);

@JordanMartinez Great! Thank you. How can I use it?

@grenche see the Readme's instructions on the snapshot release.


Sony channel not working

Sony channel not working in Kodi TV......no Sony  channel are working......

Hello guys can u help me

Is there any one who can help us

In the latest 7.1.3 (Mobile version), all the Sony channels are available.

Wat abt kodi

No.  In Kodi Sony channels are still not working.

In the latest 7.1.3 (Mobile version), all the Sony channels are available.

You are talking about which Mobile app (7.1.3)?

Yes.

any updates?

As there is no update for kodi addon. I have found temporary solution. U can use this app on android tv and also for mobile. https://github.com/jtvpremium/jio-jtv-premium
i have checked all sony channels are working.

idk if krreet can gain sony channel links from the above app and integrate into jio tv plugin.

@krreet i found this live link for sab hd https://dai.google.com/linear/hls/event/CrTivkDESWqwvUj3zFEYEA/master.m3u8
this is playing in vlc.

How to use this link in Kodi TV......@kreet

https://github.com/tobalan/tobalan.github.io/issues/42#issuecomment-2151411198


Data explorer and psy chart - dropdown - add all variables available in the data frame

@giobetti I have added more variables to the dropdown but I have left these out, is it okay?
'year', 'month', 'day', 'hour', 'Cheight', 'PWobs', 'PWcodes', 'Pwater', 'AsolOptD', 'SnowD', 'DaySSnow', 'fake_year', 'month_names', 'DOY', 'UTC_time', 'times', 'apparent_zenith', 'zenith', 'apparent_elevation', 'elevation', 'azimuth', 'equation_of_time', 'erf', 'delta_mrt', 'MRT', 'Wspeed_utci', 'Wspeed_utci_0', 'p_sat', 't_dp'
If not please reply with the list of variables you would like to include and what label you would like to assign to them in the dropdown.

In data explorer tab shall I add all the same options, as we added to the psy tab, in both the customizable heatmap and the more charts dropdowns?

Thanks Federico!
Please add the followings:
'elevation': solar elevation
'azimuth': solar azimuth
'p_sat': saturation pressure
't_dp': dew point temperature (is this duplicated?)
Please add this “extended list” to all the graphs in the tab “data explorer”

On 22. Jul 2021, at 10:58, Federico Tartarini @.***> wrote:
'elevation', 'azimuth', 'equation_of_time', 'erf', 'delta_mrt', 'MRT', 'Wspeed_utci', 'Wspeed_utci_0', 'p_sat', 't_dp'

shall I add the extended list also to the main dropdown? the one at the top of the page?

At the top of which page?
Data explorer= Yes
Sent from my iPhone
On 22. Jul 2021, at 11:32, Federico Tartarini @.***> wrote:

shall I add the extended list also to the main dropdown? the one at the top of the page?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCenterForTheBuiltEnvironment%2Fclima%2Fissues%2F81%23issuecomment-884778215&data=04|01||299fcbfa76bf452d7eb208d94cf3ad12|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637625431723411446|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=WmoQMt5tD1Je96GkYIhDJvKUyF8AIKB9YfrZtYYkvCQ%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIWZJAE2KC7Z2C5M2NYLRX3TY7QUHANCNFSM5ACI5KOQ&data=04|01||299fcbfa76bf452d7eb208d94cf3ad12|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637625431723421440|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=vDo2P3iDTsHKLOiUQBQVGsteeNJBKCx8Je%2F1tjdVNAg%3D&reserved=0.


Add a public API for subchart interactions

Would it be appropriate to add a public API that allows for more programmatic subchart interactions? e.g. clearing the subchart selection or setting the extent?
Something along the lines of:
var chart = c3.generate({
  ...
});

chart.subchart.extent([5,10]).update();
chart.subchart.clear();

I'm currently providing a similar functionality in my wrapper API using the internal API's.
function clear(chart) {
    if (chart.internal.config.subchart_show) {
      chart.internal.brush.clear().update();
      chart.internal.redrawForBrush();
    }
  };

I don't really want to be depending on any internal APIs, and would be willing to contribute something back if this is a desired functionality.

:+1:


Enable mutable-content (IOS)

Currently for iOS 10+ devices only. On iOS, use this field to represent mutable-content in the APNS payload. When a notification is sent and this is set to true, the content of the notification can be modified before it is displayed, using a Notification Service app extension. This parameter will be ignored for Android and web.
This is taken from docs: https://firebase.google.com/docs/cloud-messaging/http-server-ref#mutable_content

Coverage decreased (-31.03%) to 65.99% when pulling 36fd218418812232dcb962c2735dcd9c780d6fce on Kilgaloon:master into 897979470a596a30ce6782ee229db53f673e54b6 on Paragraph1:master.

Coverage decreased (-31.03%) to 65.99% when pulling 36fd218418812232dcb962c2735dcd9c780d6fce on Kilgaloon:master into 897979470a596a30ce6782ee229db53f673e54b6 on Paragraph1:master.

Coverage decreased (-31.03%) to 65.99% when pulling 36fd218418812232dcb962c2735dcd9c780d6fce on Kilgaloon:master into 897979470a596a30ce6782ee229db53f673e54b6 on Paragraph1:master.

Coverage decreased (-97.01%) to 0.0% when pulling 0e805ab267a27534d5f8f8bbca17ace45bddb8e9 on Kilgaloon:master into 897979470a596a30ce6782ee229db53f673e54b6 on Paragraph1:master.

Coverage decreased (-29.0%) to 68.02% when pulling 045e834216515b0a304478b040bbd46c44d67d6c on Kilgaloon:master into 897979470a596a30ce6782ee229db53f673e54b6 on Paragraph1:master.

Coverage decreased (-29.0%) to 68.02% when pulling 045e834216515b0a304478b040bbd46c44d67d6c on Kilgaloon:master into 897979470a596a30ce6782ee229db53f673e54b6 on Paragraph1:master.

Coverage decreased (-29.0%) to 68.02% when pulling 045e834216515b0a304478b040bbd46c44d67d6c on Kilgaloon:master into 897979470a596a30ce6782ee229db53f673e54b6 on Paragraph1:master.


Support flipping mouse scrolling axes through modifier keys

Fixes https://github.com/flutter/flutter/issues/75180
Fixes https://github.com/flutter/flutter/issues/109433
Dependent on https://github.com/flutter/engine/pull/38338
This adds the ability to flip scroll axes by default with a mouse wheel when the shift key is pressed, or any other key developers would like to configure through the added ScrollBehavior.pointerAxisModifiers.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

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

sorry about all the comments. 😅

sorry about all the comments. 😅

Why? Feedback is life coach. ⚽
Thanks for it! Apologies for some silly mistakes.

Following up on conversation from https://discord.com/channels/608014603317936148/608021234516754444/1047638459742957630

if you're scrolling and tap the shift key, does the delta change briefly?
I just checked, it does!
I found a list of flags on the NSEvent that is giving us the input from the mouse wheel - so we can know when shift is applied. Mac flips the axes without providing us a raw-versus-altered data, so those flags currently appear to be the only sign that the axes might have been flipped.
I have tried multiple key combos and mice, and found that no matter what, if shift is pressed, the input flips from Y to X.

like, if macOS just flips regardless, then flipping back regardless in the embedder seems like the way to go?

This sounds like the way forward? Given https://github.com/flutter/flutter/pull/115610#discussion_r1036545331 and the issue of not knowing on web what physical device we are on, normalizing the input from Mac and implementing the native feature in the framework might be the cleanest way to go.

but if macOS reports "mouse wheel" and "scroll" separately, maybe we should be forwarding both and listening for "scroll" events in the framework and ignoring "wheel" events when we have "scroll" events?

We do filter the mouse events we receive for scroll wheels etc. - that is how trackpads end up taking another path - but we also need to alter the scroll wheel input further so it takes into account the scale factor and pixels per line.
https://github.com/flutter/engine/blob/3c1de68824a3da330c459c406bc7374bf03e5a57/shell/platform/darwin/macos/framework/Source/FlutterViewController.mm#L540
and
https://github.com/flutter/engine/blob/3c1de68824a3da330c459c406bc7374bf03e5a57/shell/platform/darwin/macos/framework/Source/FlutterViewController.mm#L633

I have tried multiple key combos and mice, and found that no matter what, if shift is pressed, the input flips from Y to X.

Does it flip Y to X in that situation too?
e.g. if you have a horizontal wheel and you hold shift, do you get deltaX != 0 or deltaY != 0?
My worry is that we may not be able to distinguish "horizontal wheel + shift with a natural deltaX" from "vertical wheel + shift + macOS flipped it to deltaX".

but if macOS reports "mouse wheel" and "scroll" separately, maybe we should be forwarding both and listening for "scroll" events in the framework and ignoring "wheel" events when we have "scroll" events?

We do filter the mouse events we receive for scroll wheels etc. - that is how trackpads end up taking another path - but we also need to alter the scroll wheel input further so it takes into account the scale factor and pixels per line.

Sorry yeah I didn't meant to distinguish track pad from rolling wheel here, what I was asking is whether macOS distinguished between "rolling wheel" (a "wheel" event) and "rolling wheel with the flip applied" (what I confusingly called a "scroll" event). Sounds like no?

Does it flip Y to X in that situation too?

Yes! It does, sorry. They both flip.

My worry is that we may not be able to distinguish "horizontal wheel + shift with a natural deltaX" from "vertical wheel + shift + macOS flipped it to deltaX".

This edge case is nagging at me too. Perhaps this should not be only applied to Axis.horizontal, and we should follow suit and just flip both when the modifier is active.  I want to test it out so I don't lose sleep over it. 😂
If we neutralize what Mac is doing (I can detect shift in NSEvent.modifierKeys), we can provide a consistent result by flipping both axes in the framework across all platforms via the framework modifier key as defined in the ScrollBehavior.

distinguished between "rolling wheel" (a "wheel" event) and "rolling wheel with the flip applied" (what I confusingly called a "scroll" event). Sounds like no?

Correct!

@Piinks What's the status of this PR? (Desktop Triage)

The engine change has landed and rolled in, this is ready for another review. Thanks!

Oh wait apologies, I just realized one more test case, let me add it right quick.

Alright, for real this time, this is ready. :)

@Piinks It would be great if flipAxes can be passed in by user. The use case is for a horizontally scrollable list, it would be nice if I can use mouse wheel to scroll it without press any key on the keyboard since it's the only directly that is possible.


Re-organise wiki content in a single parent repo

Github doesn't support a wiki on the project level, and I want to have most Time tracking demo documentation centralised in one wiki, not per repository. For simplicity I would like to stick with Github Projects, so we need a "parent" repo which will contain the main wiki. Any README from codebases can point to that.

Content reorganised from
https://github.com/ericsmal/time-tracker-react/wiki
to
https://github.com/ericsmal/time-tracker/wiki


Request for access to BH cloud services

Title
Identifying Neuropsychiatric Diagnoses using fMRI Connectivity Data.
Is this a new project?
yes
People involved and contact details (including email address!)
Peter Brotherwood<EMAIL_ADDRESS>Project Description

What are you doing, for whom, and why?
Requesting access to HPC services for utilisation during my brainhack school project.

Link to project repository/sources
https://github.com/brainhack-school2022/brotherwood_project
What resources do you require?
HPC
Anything else?
No response

Done - @PeteBro you should have received two emails giving you access to the cloud. Let us know if you have any questions.

This project is now closed and all resources on the Oracle Cloud will be deleted within the next few days.


android Swipe issue

Which OS ?
Version
Which versions are you using:
"react-native-swiper": "v1.6.0-nightly.2",

Expected behavior
I have 3 component to swipe  right and left ios is working perfectly
Actual behaviour
when I want to swipe from the first component to second its working ok
but when second component to 3 it's just coming back to the first page
and when second page to the first page  its some time going back to the second page instead of the first page
How to reproduce it>
To help us, please fork this component, modify one example in examples folder to reproduce your issue and include link here.
Steps to reproduce

Can you reproduce it on v1.6.0-nightly.5 and give me the minimal case like https://github.com/leecade/react-native-swiper/blob/develop/examples/index.js?

I can confirm I see the same behavior on Android with master branch (latest commit)

@ArrayZoneYour to summarize:
-> the problem is Android only
-> happening on swiper with more than 2 slides
-> swiping right is always OK
-> reproducing steps:

swipe twice right to third slide, OK
swipe once left to second slide, OK
swipe once left to first slide, NOK => swiper will come back to second slide. You have to swipe left one more time to come back to first slide.

Hi, any update on this?


SLIM-245 SLIM-435_Refactoring object deleting converters

 You should sign our Contributor License Agreement in order to get your pull request merged.

All is well.

All is well.

SonarQube analysis reported 10 issues:

 9 major
 1 minor

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Define a constant instead of duplicating this literal "reserved" 3 times.

All is well.

SonarQube analysis reported 15 issues:

 9 major
 6 minor

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Define a constant instead of duplicating this literal "reserved" 3 times.

All is well.


Optimizing the Binary comparison method in CompareBinaryColumnTransformer

Related doc: https://apache-iotdb.feishu.cn/docx/VjxMd1BdHotEeIxHzsxc382Jnec
Simply put, this PR avoids unnecessary creation of String objects when comparing Binary types, and ceases computation when encountering an entirely empty column.
Before this PR, doTransform occupied 18.6% of the execution time：

After this PR, doTransform occupied 13.1% of the execution time：

Codecov Report
Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (cac7a52) 49.03% compared to head (75c179b) 49.05%.
Report is 3 commits behind head on master.

Files
Patch %
Lines

...engine/transformation/dag/util/TransformUtils.java
0.00%
6 Missing :warning:

.../column/binary/CompareBinaryColumnTransformer.java
58.33%
5 Missing :warning:

...ne/execution/fragment/FragmentInstanceContext.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11630      +/-   ##
============================================
+ Coverage     49.03%   49.05%   +0.02%
- Complexity    24902    24908       +6
============================================
  Files          2810     2810
  Lines        175996   176013      +17
  Branches      21114    21119       +5
============================================
+ Hits          86301    86349      +48
+ Misses        89695    89664      -31

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


Update polybench.rs

Fixed 2 errors causing build to fail

@dylandylandylandylandylan there is still a problem see https://github.com/dcompiler/dace/actions/runs/5290797902/jobs/9575458683?pr=22


[BUG]: [UAT|Unable to submit a case failing for advocate search

Describe the bug
While registering a case getting error message for advocate API like you are not authorized

Issue is fixed in UAT environment


pull request for e2e test at 12/31/2016 12:15:07

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


copy workflows from blueapi, [still need to configure env values -not a code change]

@

thanks for the comments @coretl . I deleted the 'if swift language' branches, kept the comments though to keep this more similar to the template workflow for codeql. now sure fully about the syntax in the periodic file though


Health check status  for node ok,although node status is halted

Issue
node status is halted, but when you check node health check , you will find it ok as shown :

Software Version
0d238efa01e6f8ee291f2d3cec4d1e918aa39212

software version
master : 5c7668358


Sh/dependency separation and clean up

Separating the USD dependency libraries into separate cmake\X.cmake files.
Added options to build all dependencies or using the ones installed on the system.
Added support to build ColorIO, OpenVDB and PRman plugins.

@sherholz looks good and works on ubuntu 10.04 with python2, but added some comments.  Overall there is a lot of commented out code.  Can that be removed before merging?


[Enhancement] update jobs parameter logic to be more flexible

if i were to deploy this program to a server dedicated for image processing, i would want to be able to utilize all cores of the server.
if hogging all cores on a multipurpose server is a concern, could update logic to something like 'all but 2 cores'.

now i understand that the jobs/cores parameter takes into the account of ImageMagick's parallelism.


Touch up release CI job

In preparation for today's 0.69.0 release (the first to be cut out of this repository!), run the release.yml CI through its paces, publishing both kolena and kolena-client to test PyPI and trunk CodeArtifact. Verified that both work by installing kolena/kolena-client from test PyPI in a fresh venv.

i think we should setup the correct version at here as well

i think we should setup the correct version at here as well

This is set automatically by the release CI job:
      - name: Update kolena package version to PEP 440-compliant production release tag
        run: poetry version $(git describe --tags --abbrev=0)

i think we should setup the correct version at here as well

This is set automatically by the release CI job:
      - name: Update kolena package version to PEP 440-compliant production release tag
        run: poetry version $(git describe --tags --abbrev=0)

cool, i thought pyproject.toml is part of the pypi artifacts, but seems like not, i think it's fine to leave as is.


add skip_send_invite field to opslevel_user resource

Resolves #463
⚠ depends on this opslevel-go PR
Downstream terraform-opslevel-modules PR updated for eventual use
Problem

Solution

Checklist

[ ] I have run this code, and it appears to resolve the stated issue.
[ ] This PR does not reduce total test coverage
[ ] This PR has no user interface changes or has already received approval from product management to change the interface.
[ ] Make a changie entry that explains the customer facing outcome of this change

Replaced by PR #495


Edit and Search entry

Please support editing and search (if possible using regex) entry.

Can you please provide more details on how this feature should work

Create
a +  icon when clicked can create an empty object for a json file with object array, or it lets user define a new JSON pair appended at the end for non-object array JSON file.
Edit
user can edit only values of the given existing field
Search

For example in above one could search for the regex linux.*jupiter/i where i indicates case-insensitive search, and it could highlight all matches, and point to the first result and let user navigate to next/previous results


Add Command that emits events on nodes

I think it could work like this
$ emit $path/of/node event_name "additional args for the event emitter function"
For convinience sake the user could copy paths into the input field at cursor position by clicking on it on a recent output to the ls command (#6 ).
( Awesome Plugin by the way! )

This command would have to be disabled on export builds (debug builds should still be able to use this command imo) because I intend that GDShell would be safe-to-deploy by default, meaning that all the built-in commands should not be able to modify the game's logic flow in any way as someone might one day release their game with GDShell and it could be abused to pirate or hack the game if it's multiplayer. If the user makes his own command for cheats, or enables this command on export builds it's up to them.
Anyway I like the idea, usage and how it integrates with the proposed ls command. The copy on click should be easy to implement as GDShell expects a RichTextLabel for output.
There could also be more commands like this one for overriding (set) and printing (see/display/print?) variables, calling functions (call) and so on. They would all use the same syntax for ease of use.
This however would be really clunky when you try to modify multiple values, call a function and emit a signal. For this frequent use the user should make their own command that can be specialized for the usecase. For general one-time editing however this could be a great tool.

Do you want to tackle this command? I currently don't have much time, so I would really appreciate it.
The other ones, that I suggested would be very similar I believe, so the it should be quite easy.

( Awesome Plugin by the way! )

❤

Hey Thank you for the answer
I would like to collaborate I'm just new to this whole open source thing so I hope that this works out :)

I'm pretty new to this too, so don't worry about it and consider it a learning opportunity.
Feel free to ask when you encounter any problem!

Hey Friend,
It's possible that I wont have time to look at this proposal as I am working on my own things.

Hey,
that's totally understandable. You know where your priorities are and I can't force you to contribute to an open source repository.
Thanks for the proposal and for telling me! ❤

I think emitting signals like that is not possible and would be very problematic.
You can't emit a node's  signal from another script without instancing it in that script.
And besides once you start emitting signals that should only be emitted when particular  things should happen, without them happening could cause a lot of trouble.
A better solution would be to add a command that would call a given function from a node, the user may then decide to create a function in that node that would emit the signal.
This is also a very helpful command either way!

Did some testing on this, I have a complete command that emits a signal on a given node, with error handling and everything, BUT
I tried it on a few nodes and most of the times the emit signal would just return ERR_UNAVAILABLE...
after more testing i believe the emit function can only emit custom signals and already connected signals.

Sadly there is no way of know what type a signal expects it's arguments to be, therefore it's not possible to support signal arguments.
Currently my command can emit any connected signal from any node on the tree, but doesn't allow arguments
Is this worth adding to the codebase?

I think emitting signals like that  would be problematic, once you start emitting signals that should only be emitted when particular  things should happen, without them happening could cause a lot of trouble.
A better solution would be to add a command that would call a given function from a node, the user may then decide to create a function in that node that would emit the signal.

For this I thought about adding a call command that would handle method calling and it would be functionally similar to the emitting especially with the handling of arguments.
Both of them would be unsafe without any workaround I can think of, but they would be intended only for debugging and not for production (they should be disabled on export builds).
Also the user can work around this by making their own functions or commands that would handle the arguments the way they need for their project.

I read over the discussion and I think having both a useful shell for debugging and a "safe to deploy by default" shell are incompatible. This is especially true since godot games are inherntly decompileable, making it easy to hack anyway.
The multiplayer concern I feel is worth thinking about, but if multiplayer actions happen without getting a signed message from the host its a poorly designed multiplayer system, and asking to be hacked. Having a shell do that isn't much harder than injecting a PCK, which is easy to do, even for games which don't allow that by default.
If your really worried about it I would suggest being able to mark commands as either "safe" or "unsafe". Safe commands can ship with the game and dont alter logic or anything. Unsafe commands can do whatever, and add something in project settings (if possible, I am not sure how the GDExtension API works) to toggle between allowing or not allowing safe commands in a final build. If they are not allowed, they don't even ship with the game--so it can't be bypassed.

Why didn't I think of this? I opened a proposal that would solve this


When Dusk node starts, and there is no Rusk server, it will silently do nothing.

I would expect some log or something to tell me what is going on

This has been fixed yesterday, as the deployment system was examined by me and @pmoranga. #803 builds on this issue.


Update SetupMPI.cmake

This modification allows for smooth MPI+CUDA RAJA configuration and compilation using nvcc with ENABLE_FIND_MPI. It needs RAJA_EXISTS variable to be set ON in RAJA (part of a RAJA patch).

It would be good to know which mpi compiler flags need the -XCompiler treatment.
Typically this is only needed for cases where the compiler provides features that don't look like standard libs. Like pthreads, or openmp flags -- where the features you are accessing a built into the compiler and nvcc doesn't know the special cases.
Given that mpi is a library nvcc should be less picky about how it is presented.
Maybe this is a case where MPI compiler wrappers are echoing magic flags?

Canceling the PR because of version mismatch between RAJA's BLT and BLT/master.


how to do photo image blob in typegoose

Hi
does anyone have an example of how to persist a image or clip in typegoose
thank you
awesome framework its holding up beautifully

you are searching for gridfs, for more on support in typegoose look in #216
@Ben305 this can be closed


[connect modal] Torus/ walletconnect issues found

Torus threw a 'browser not supported' error
WalletConnect got stuck and kept popping up, even when id click torus

Still need to figure out the reproduction steps
This might just be stale sessionstorage/cookies

@Atmosfearful I'm able to connect with wallet connect and torus. Were you able to reproduce this?


Deployments failing due to rendering bootstrap manifests

Something recently introduced is causing deployments to fail.
Apr 03 18:16:08 localhost.localdomain systemd[1]: Started Bootstrap a Kubernetes cluster.
Apr 03 18:16:27 localhost.localdomain bootkube.sh[11222]: Rendering MCO manifests...
Apr 03 18:16:28 localhost.localdomain bootkube.sh[11222]: I0403 18:16:28.454325       1 bootstrap.go:84] Version: 4.0.0-alpha.0-154-gc1ba218c-dirty
Apr 03 18:16:28 localhost.localdomain bootkube.sh[11222]: F0403 18:16:28.456056       1 bootstrap.go:104] error rendering bootstrap manifests: open /assets/tls/etcd-metric-ca-bundle.crt: no such file or directory
Apr 03 18:16:28 localhost.localdomain systemd[1]: bootkube.service: main process exited, code=exited, status=255/n/a
Apr 03 18:16:28 localhost.localdomain systemd[1]: Unit bootkube.service entered failed state.
Apr 03 18:16:28 localhost.localdomain systemd[1]: bootkube.service failed.

We're looking into it, looks like the kni-installer is going to need to be rebased

see https://github.com/openshift-metalkube/kni-installer/pull/35

the installer rebase (and a bunch of other fixes) landed, @jtaleric can you pls close this if things are OK now, or provide an update with more details if you still have issues?  Thanks!


232 quales and z sgov split

see [#232]
minor cleanup
refactoring of Z-SGoV and modeling full/compact separately.

@blcham FYI


How can I understand better why some time allocation fail

This is my cache implementation:
int32_t PutKey(CacheKey key, const std::string& value)
{
	size_t valueSize = value.size();
	size_t numChunks = GetChunkNUmber(valueSize);

	// parenthesize will hold all the pointers to the chained items
	size_t parentItemSize = numChunks * sizeof(void*);
	if (parentItemSize > gMaxChainedSize)
	{
		// For now - this is too big, later we will implement a chain of chains (currently we support > 2GB)
		if (gDbg)
		{
			cout << "[-][PutKey] Error parentItemSize:" << parentItemSize << " is bigger then gMaxChainedSize:" << gMaxChainedSize << endl;
		}

		return 2;
	}

	// Get parrent
	auto parentItemHandle = gCache_->allocate(defaultPool_, key, parentItemSize);
	if (!parentItemHandle)
	{
		if (gDbg)
		{
			cout << "[-][PutKey] Error we didn't succeed to get the parentItemHandle" << endl;
		}

		return 3;
	}

	//Fill in the chunk number in the parent
	// CustomParentItem* parentItem = reinterpret_cast<CustomParentItem*>(parentItemHandle->getMemory());
	// parent item->numChunks = numChunks;

	// Get char * representation of the data
	char* ourValue = const_cast<char*>(value.c_str());

	// Create the chunk - Now split user data into chunks and cache them
	for (size_t i = 0; i < numChunks; ++i) {
		size_t chunkSize = std::min((size_t)(gMaxChainedSize), valueSize);
		auto chainedItemHandle = gCache_->allocateChainedItem(parentItemHandle, chunkSize);
		if (!chainedItemHandle)
		{
			//We failed to allocate the chunk
			if (gDbg)
			{
				cout << "[-][PutKey] Error we didn't succeed to allocateChainedItem for size:" << chunkSize << endl;
			}
			return 4;
		}

		// Compute user data offset and copy data over
		char * dataOffset = ourValue + (gMaxChainedSize * i);
		std::memcpy(chainedItemHandle->getMemory(), dataOffset, chunkSize);

		// Add this chained item to the parent item
		gCache_->addChainedItem(parentItemHandle, std::move(chainedItemHandle));
		valueSize -= chunkSize;
	}

	// Now, make the parent item visible to others
	gCache_->insertOrReplace(parentItemHandle);

	return 1; // 1 == Success
}

When I run this, I get from time to time 4 or 3 in response (i.e., we failed in allocate/allocateChainedItem).
A. How should I handle that? should I retry allocate/allocateChainedItem before returning the error code?
B. How can I debug the root cause of this?
p.s.
My cache is an hybrid cache (with NVME), in the tests I assign to the RAM part 100mb.
This is my full settings:

// Set Nvme Cache
	nvmConfig.navyConfig.setBlockSize(4096); // Default is 4096 - Device block size in bytes (minimum IO granularity)
	nvmConfig.navyConfig.setSimpleFile(gCacheFilePath,
<PHONE_NUMBER>00, 		/*fileSize*/
								false 			/*truncateFile*/);

	// Set the block cache - Large Object Cache - caches objects that are larger than KBs in size
	nvmConfig.navyConfig.blockCache().setRegionSize(33554432);

	// Set admission policy
	nvmConfig.navyConfig.enableRandomAdmPolicy().setAdmProbability(0.9); // a float number that decides Acceptance probability. The value has to be in the range of [0, 1].
	nvmConfig.enableFastNegativeLookups = true;

	// Set the Small Object Cache - caches objects that are 100s of bytes
	nvmConfig.navyConfig.bigHash()
      .setSizePctAndMaxItemSize(5, 4052) // bigHashSizePct (0-100), the % of the file that bigHash will use , bigHashSmallItemMaxSize must be smaller than bucket size which is 4096 by default
      .setBucketSize(4096)
      .setBucketBfSize(8); // bloom filter size per bucket, default is 8

	// Ram config

	config
		.setCacheSize(1073741824) // 1GB = 1 * 1024 * 1024 * 1024
		.setCacheName("My Case")
		.setAccessConfig(40000000) // assuming caching 20 = {25 /* bucket power */, 10 /* lock power */})
		.enableNvmCache(nvmConfig)
			.validate(); // will throw if bad config

Hi @Chokoabigail,
allocate can fail if CacheAllocator cannot find items to be evicted at the end of eviction queue. The maximum number of retries (say N) is determined by evictionSearchTries and the eviction will be failed if none of the N items at the end of eviction queue for given allocation class are eligible to be evicted; i.e.,
To debug this, you can check how many handles are outstanding by checking CacheAllocator::getNumActiveHandles.
If the number is much higher than expected, you need to take a look at the client code for any leaking handles.
Otherwise if the number is expected, I think you can retry after some delay.
Thanks.

The client code is actually very simple, the most complex function is the PutKey and in it, there is a concern for memory leaks.
What should I do with this code:
auto chainedItemHandle = gCache_->allocateChainedItem(parentItemHandle, chunkSize);
if (!chainedItemHandle)
{
...
}

It potentially can be after several chainedItemHandle allocations succeed (or none), should I call gCache_->remove(key); on the same key I used in auto parentItemHandle = gCache_->allocate(defaultPool_, key, parentItemSize); ?
Should I call free on parentItemHandle ? Should I call free on every successful chainedItemHandle allocation before the loop?
This is the rest of the client code:

std::string GetCacheStatus()
{
	std::string res;
	std::string seperator = "[::]";

	res += "isNvmCacheEnabled: " + std::to_string(gCache_->isNvmCacheEnabled()) + seperator;

	// Get the statistics map
	auto statsMap = gCache_->getNvmCacheStatsMap();
	for (const auto& kv : statsMap.getCounts()) {
		std::string k = "statsMap.getCounts()" + kv.first + ":";
		res += k + std::to_string(gCache_->isNvmCacheEnabled()) + seperator;
	}

	// Get the rate map
	for (const auto& kv : statsMap.getRates()) {
		std::string k = "statsMap.getRates()" + kv.first + ":";
		res += k + std::to_string(gCache_->isNvmCacheEnabled()) + seperator;
	}

	return res;
}

bool RemoveKey(CacheKey key)
{
	auto rr = gCache_->remove(key);
	if (rr == Cache::RemoveRes::kSuccess) {
		return true;
	}
	return false;
}

std::string GetKey(CacheKey key)
{
	std::string result;
	uint32_t fregmentNumber = 0;

	auto parent = gCache_->find(key);
	if (!parent)
	{
		return result;
	}

	auto iobuf_chainedItems = gCache_->convertToIOBuf(std::move(parent));

	for (const auto& item : iobuf_chainedItems)
	{
		// First item is the parent item so we can continue over it
		if (fregmentNumber == 0)
		{
			fregmentNumber++;
			continue;
		}

		// Reconstruct the value
		folly::StringPiece sp2{item};
		result += sp2;
	}

	return result;
}

@Chokoabigail As long as the handle is disposed, you are good. Those have zero references, so should be eligible to be evicted.
I can see you allocated 100MB for RAM cache, meaning you would have only 24 slabs or so. Some allocation class might not have enough slabs allocated or even 0. I would suggest check the distribution of slabs to alloc classes and what was the failing allocation class.

@Chokoabigail As long as the handle is disposed, you are good. Those have zero references, so should be eligible to be evicted.
I can see you allocated 100MB for RAM cache, meaning you would have only 24 slabs or so. Some allocation class might not have enough slabs allocated or even 0. I would suggest check the distribution of slabs to alloc classes and what was the failing allocation class.

How can I do that? (p.s. In production, we have the same client with a 16GB of RAM, and it also fails there from time to time (once a month or so))

@Chokoabigail You can refer to the ACStats  which is part of the PoolStats returned by CacheAllocator::getPoolStats(...)
I noticed that there is no public support to print those stats. I think you can refer to the cachebench implementation here  (-report_ac_memory_usage_stat option)

Let me close this for now. Feel free to reopen or open a new one if needed


fix: rustler update to v0.35.0

changes needed to compile against rustler v0.35.0

@fabriziosestito one tiny recap before I go to sleep.
I tried to drop _build before running the tests and it works. My theory is that restoring the prebuilt dependencies and compiled code prevents the actual Rust code to be built, especially the dylib .so library that is needed by some of the tests. Dropping _build is a caveman way of triggering the rebuild of just everything.
This is a very naive approach and I absolutely don't like it. :smile:

@dottorblaster I removed the commit you added and now somehow the tests are passing. There was something wrong with the GH cache.


Docker image manifest

Bonjour Benoit,
It seems there is a problem with the docker manifest when trying to retrieve images from ghcr.io (on aarch64 plateform / pi4)
$ docker -v
Docker version 20.10.5+dfsg1, build 55c4c88
$ docker pull  ghcr.io/benoit3/diematic_to_mqtt:latest
latest: Pulling from benoit3/diematic_to_mqtt
manifest unknown
$ uname -a
Linux xxxx 6.1.47-v8+ #1674 SMP PREEMPT Fri Aug 25 13:17:22 BST 2023 aarch64 GNU/Linux

Hello,
I thank you for your feedback.
I didn't understand what's happen because it was testes OK 6 months ago with an PC/Ubuntu 20.04 x86_64 GNU/Linux PC ( platform info given by docker version is " linux/amd64" ) and with a Rpi/ Bullseye aarch64 GNU/Linux ( platform info given by docker version is " linux/arm64" ). Same tests are now KO.
As I didn't modify the repository, solution is not obvious. It may take some days to find it (an access pb to manifests? An expired certificate somewhere?, docker pull  command is not verbose :-( ). If you need a result quickly I can suggest you manual installation
Regards,
Benoit

Id did a near dummy update to force image rebuild.
Seems to work....
I didn't understand really what's happen...
Can you confirm ??

Everything is ok now after your rebuild.
I confirm that I can download the docker image without any troubles. I've no idea about the reason why it was broken, but this issue is now solved.
Thanks Benoit.


Position of splitter between Console and Viewer is lost after hiding and showing Console

When I hide Console and then show it again, position of splitter is lost and returned to default.
Now I'm thinking, maybe it's by design? Anyway, really minor one...

I feel this was recently closed, but perhaps not yet released. Look for closed issues.

If you mean #56 than it concerns remembering position after closing to tray and opening again.

Yeah, the console being hidden is counted as having a height of 0. I did it that way because it simplified things slightly and I honestly didn't think anyone would care :smiley:
If it bothers you, it shouldn't be too hard to put in.

It doesn't bother me too much, only that I really like this app and trying to contribute to it the way I can, which sadly seems to be only issues raising. But since this is really something that nobody will notice/care about, don't waste your time on it. I will think more about importance of things I discover before I click that green button here next time :wink:

Fair enough! I appreciate the helpful attitude :smile:

I think this is wontfix