An error occurred in MODEL.BL1860 170510-manufacture

This version of MCU: F0513+L5203
An error occurred while reading MODEL. Dialog box: Battery is present but not supported.
I don’t know if the board on which you can read the model normally is 1860?
(pcb version number: LIPW012)

Have not heard of this PCB revision before. Is it BL1860 or BL1860B? Normally F0513 is not supported but i have been able to read model and voltages from mine. The reset is still unknown.
Could you also provide pictures?
If possible, front and back such as: https://github.com/mnh-jansson/open-battery-information/wiki/Makita#lipw017

之前沒聽過這個PCB修訂版。
也可以提供圖片嗎 ？

之前沒聽過這個 PCB 修訂版。是BL1860還是BL1860B？通常不支援 F0513，但我已經能夠從我的設備讀取型號和電壓。重置仍是未知數。
還可以提供圖片嗎？ 如果可能的話，正面和背面，例如：https://github.com/mnh-jansson/open-battery-information/wiki/Makita#lipw017

BL1860B (lipw012) and BL1850B (lipw011)

Have not heard of this PCB revision before. Is it BL1860 or BL1860B? Normally F0513 is not supported but i have been able to read model and voltages from mine. The reset is still unknown.
Could you also provide pictures? If possible, front and back such as: https://github.com/mnh-jansson/open-battery-information/wiki/Makita#lipw017

BL1860B (lipw012) and BL1850B (lipw011)
bl1860b.Opening the wrong screen

The Arduino OBI reports a strange version, could you try update ArduinoOBI with the latest commit?

The Arduino OBI reports a strange version, could you try update ArduinoOBI with the latest commit?

I am using the latest version you released to compile the exe program

Great, thanks for reporting! Sadly the F0513 is still missing unlock, but hopefully you see all the cell voltages.

Great, thanks for reporting! Sadly the F0513 is still missing unlock, but hopefully you see all the cell voltages.

The only unlocking method is to obtain a normal battery from F0513, remove the firmware, and re-program it?....The above is just speculation.

The way i've unlocked these is to rewrite a certain block with error counters to 0. The block is at address 0x6800 on the batteries i've extracted the firmware from, but it could differ on yours. If you are willing to risk bricking the batteries i can guide you through the process. In that case i'd prefer if you would contact me through the form on the blog https://martinjansson.netlify.app/posts/makita-battery-post-2
I pushed a commit that displays what i believe is the charge count. Please check it.

The number of charges can be displayed, but a new problem was discovered... After resetting the error and message, the indicator light on the battery returned to normal, and the battery damage alarm popped up immediately within 10 seconds of charging the DC18RC (it also popped up again within 10 seconds after unplugging and replugging it) Alarm) Pull out the battery and check if there is any abnormality.

Good point. The command to read the battery message triggers the evaluation of the battery, so if no cells are present or any active fault is present it will trigger the lock.
There must be an active fault, but everything looks good on the screenshot. Could the fuses be blown?

Good point. The command to read the battery message triggers the evaluation of the battery, so if no cells are present or any active fault is present it will trigger the lock.
There must be an active fault, but everything looks good on the screenshot. Could the fuses be blown?

The HT pin is open circuit, the fuse needs to be replaced first.... Your guess is probably correct.


Configurable "Header Filter" for tidy workflow

At the moment, we use a hard-coded value for tidy header filter: -header-filter=.*

A small addition that you might not have considered, would it be possible to create an option to only fix your own header. Currently when you have a large project you get spammed by warnings if you only try to fix one file. This of course assumes you have pairs of same name header and cpp but that is a very common pattern.

Hi,
Very good suggestion, @jcageman !
We like it so much, that I think you'll just surprise you with it in the next version to be released this week :)
We'll let you know.

This enhancement was implemented in version 2.4 released today.
Thanks for you feedback 👍


Fix tailwindcss duplicated class name

Next Generator pull request
Summary of changes made:
Remove duplciated class name
Summary of testing method for these changes:
Extra context or other helpful information to help reviewer:

corrects issue #

I'll check this out soon! Thanks for contributing!


test: S3DataPlaneIntegrationTest should be annotated with AwsS3IntegrationTest (fixes #1896)

What this PR changes/adds
Updated the annotation of org.eclipse.dataspaceconnector.aws.dataplane.s3.S3DataPlaneIntegrationTest.
Why it does that
In order to make the test executed by specifying -DincludeTags='AwsS3IntegrationTest'.
Linked Issue(s)
Closes #1896
Checklist

[n/a] added appropriate tests?
[x] performed checkstyle check locally?
[n/a] added/updated copyright headers?
[n/a] documented public classes/methods?
[n/a] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

tested that the test is kicked by -DincludeTags='AwsS3IntegrationTest' on my local environment.
$ ./gradlew clean test -p extensions/data-plane/data-plane-s3 -DincludeTags='AwsS3IntegrationTest' -PverboseTest

> Task :extensions:data-plane:data-plane-s3:test

S3DataPlaneIntegrationTest STANDARD_ERROR
    SLF4J: The requested version 1.8 by your slf4j binding is not compatible with [2.0]
    SLF4J: See http://www.slf4j.org/codes.html#version_mismatch for further details.

S3DataPlaneIntegrationTest > shouldCopyFromSourceToSink() STARTED

S3DataPlaneIntegrationTest > shouldCopyFromSourceToSink() PASSED

---------------------------------------------------------------
|  Results: SUCCESS (1 tests, 1 passed, 0 failed, 0 skipped)  |
---------------------------------------------------------------

BUILD SUCCESSFUL in 7s
54 actionable tasks: 5 executed, 49 up-to-date

@ndr-brt added you as a reviwer since you wrote the test. If I understand correctly, the annotation should be used on tests that require an AWS storage emulator to be running. Does this apply here?

@ndr-brt U approved but did not merge, can we merge this?


Failed to load plugin descriptor from file error

Tried to install plugin on 2017.2.1 version of pycharm, but get the above error.
Tried re-downloading it but still will not load

How did you install the plugin? Is is the master branch from GitHub or the published version from the plugin repository?

i tried to install from jetbrains but it said that it was not compatible with latest version of pycharm, so downloaded zip from github and tried to install from that but got the error message

Yes. The plugin 0.1 is incompatible with PyCharm 2017.2. Only with 2017.1. You can build the plugin from the master branch for 2017.2 as described in the readme. Don't hesitate to ask more questions if you encounter any issues.


Delete Characters and states: provide error message

When you delete a character that has any dependent Characters or states you must select cascade. If you do not, you get a message about blocking relationships. Please provide error message,  “This Character has dependent Characters and/or states. You must select Cascade to delete all dependent Characters and states.” or something shorter like it.

This is too difficult to have done in time for the hackathon, but there is a more general need for a way to handle error messages in the API ... loosely related to #135


add support for GCC 6.5.0

new patch: 0017-c++-abi-break.diff fixes a C++ ABI break regression.
0010-static-pie-support.diff was removed as it doesn't apply anymore,
waiting for somebody to forward port 0010-static-pie-support.diff

This needs forward port of 0010-static-pie-support.diff to move forward.

instead of forward-porting, i added the 2 patches reversing the changes, just like gcc-7.3.0 does.
should i squash the 2 commits together ?

Squashed and merged. Thanks!


rust-9p: init

Motivation for this change
A Rust-based implementation of 9p2000.L, seems more trustable than C-based diod
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify
cc @qyliss

Should the package be called unpfs?  rust-9p seems to be more of the
name of the library.

Should the package be called unpfs?  rust-9p seems to be more of the
name of the library.

Indeed; when I started the packaging I thought that packaing rust-9p in
a way that allows its use as a libary, too, would be straightforward.


Which GTK?

It'd be nice to specify which GTK version is required (2 or 3).

Whoops, nice catch. GTK 3 is the requirement I believe (comes from here). I'll update the README


Pyupgrade autofixes

Most typing imports aren't necessary for py39+, so we can eliminate a bit of boilerplate.
This was done automatically with
find . -name "*.py"|xargs pyupgrade --py39-plus
find . -name "*.py"|xargs pipx run autoflake --in-place

LGTM, thanks so much for the clean up!


Um...

If you're not happy about the scripts folder being taken out go complain to ShadowMario about it.

just realized shadowmario's x account got banned lol. violated the x rules, wow.

CLOSE THIS ISSUE, SOAB

why did shadow remove the scripts folder, great now my mods are useless :(

You could make a new folder named "scripts" I believe. I use 0.7.x and I usually just make a new folder when I add scripts

I did NOT remove the functionality of it, I just accidentally deleted the folder, you can just make a new one. Can you guys stop spreading misinformation?


about training time

Hello, I want to know how long it took you to train FClip-HR? It took me more than seven days. Is there any way to reduce the training time?

Hi,
If you are not pursuing the state of the art performance, just set the training epoch to 35 and make the learning rate decay at 25 and 30 respectively, that will be enough for you to obtain acceptabel results.

Hello, I have modified the base.yaml and fclip_HR.yaml file：
lr: 4.0e-3
max_epoch: 35
lr_decay_epoch: [25,30]
weight_decay: 8.0e-4
However, the results were not so good:
f1 measure is: 0.33950091807137656
AP-H is: 0.19708869533106066
I noticed in the paper that you used a single NVIDIA GTX 2080Ti GPU to train for 300 epochs. How long did this process take?

Hi, If you are not pursuing the state of the art performance, just set the training epoch to 35 and make the learning rate decay at 25 and 30 respectively, that will be enough for you to obtain acceptabel results.

if train HG1_D2 model.  How many epochs need to be trained?


detail: improved readability for 'point by point distances' exercise

numpy.atleast_2d() can take multiple arguments so both arrays can be passed in one function call

Thanks, did not know that. Did you check the notebook (is ok I mean) ?

Yes, the notebook is checked and works OK.

Thanks


[NPM-3074] Another attempt at fixing TestUDPPythonReusePort test

What does this PR do?
Adds retries and timeouts for read calls in the script used in the test. This should make the script more robust to blocking on these reads forever, which seems to be happening in the latest failures, e.g., https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/379488524
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[x] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Did you consider adding a socket timeout and adding retries to running testdata/reuseport.py in the go test? That might be simpler given we have all the utilities to do retries in Go, and we have to roll our own here

There is already a timeout on the process of 5s which never seems to work with this script hangs. This timeout is supposed to kill the process, but it just doesn't for some reason. We would need something like this to work if want to retry running the whole script.

There is already a timeout on the process of 5s which never seems to work with this script hangs

The reason the exec timeout doesn't work is because a forked process is stuck.

We would need something like this to work if want to retry running the whole script.

I'm saying if we set a socket timeout, I'd expect the exec timeout set in go to start working as expected.
Reposting the reproduction from slack:
# foo.sh
function run() {

while true
do
  sleep 5
  echo "hello"
done
}

# try removing the &
run &

// main.go
package main

import (
	"context"
	"fmt"
	"os/exec"
	"time"
)

func main() {
	ctx, _ := context.WithTimeout(context.Background(), time.Second*5)
	cmd := exec.CommandContext(ctx, "/bin/bash", "foo.sh")

	fmt.Println("here")
	our ,err := cmd.CombinedOutput()
	fmt.Println("there")
	if err != nil {
		panic(err)
	}

	fmt.Println(string(our))

}


kubernetes: 1.2.4 -> 1.4.0

Motivation for this change
While there's other pull request from me around which updates nixos module, i didn't had time to fully test the module, so i decided to do pull request with new kubernetes first, since some people(and me) want to have it.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

This compiles for me, but I'm not really able to test this.


fix(explorer): liquidity amend order showed incorrect fee

Description ℹ
The fee figure shown on amend LP TXs was not formatted as a percentage. Now it is.

I just realised I should be pulling this in to someone elses branch so I'm going to close this and open a very similar one


Static Documentation Site Using mkdocs

Description
Static site for documentation have been made with the help of mkdocs and can be deployed in the future.
Related Issues

Closes #495

Type of PR

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

Screenshots / videos (if applicable)

Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:
more changes like logo on the site and more screenshots of the project can also be updated for public refrence.

@Abhishek-Sood Can we also have a workflow for building the site for docs something like this?
name: Build site nightly

on:
  schedule:
    - cron: "0 0 * * *"

jobs:
  dot-release:
    name: dot-release
    runs-on: ubuntu-latest

    steps:
      - name: curl
        env:
          NETLIFY_BUILD_HOOK: ${{ secrets.NETLIFY_BUILD_HOOK }}
        uses: wei/curl@v1
        with:
          args: -X POST -d '{}' "$NETLIFY_BUILD_HOOK"

yes sir we can surely have a netlify nightly build workflow instead of the github pages one. the github one is more complex but is in detail and triggers only when there is pr change or if we have a documentation tag while the netlify is rather a simple one and will trigger every night ..
so if you want to i ll make changes in the workflows to work as you like :)

Implement it

please check sir updated the workflows to nightly build.

@Anishkagupta04 PTAL

@Anishkagupta04 mam please look into this asap i need to do other things too :)

@Anishkagupta04 PTAL

@Anishkagupta04 is PA. So, we need request her feedback to add any new features to project. Wait will ping Anishka in discord linking the this PR

sir very very thank you  :) u d best


Support multiple cron schedules within one cron scheduler or make the end field optional

Proposal
Hi team,
Does it make sense to make the end field in the cron scaler optional? https://github.com/kedacore/keda/blob/main/pkg/scalers/cron_scaler.go#L99
When the end field is empty, the controller will just set the replicas to desiredReplicas, and leave it there.
Or even better, does it make sense to support multiple schedules within one cron scheduler? I figure it could be hard to implement the isActive function if we make the end field optional.
Use-Case
This makes configs cleaner if users want to use multiple crons.
Before (note how we want to make sure the previous scaler's end match the current scaler's start):
  triggers:
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 39 * * * *
      end: 41 * * * *
      desiredReplicas: "2"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 41 * * * *
      end: 43 * * * *
      desiredReplicas: "4"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 43 * * * *
      end: 45 * * * *
      desiredReplicas: "6"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 45 * * * *
      end: 47 * * * *
      desiredReplicas: "8"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 47 * * * *
      end: 49 * * * *
      desiredReplicas: "10"

After (if we make end field optional):
  triggers:
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 39 * * * *
      desiredReplicas: "2"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 41 * * * *
      desiredReplicas: "4"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 43 * * * *
      desiredReplicas: "6"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 45 * * * *
      desiredReplicas: "8"
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      start: 47 * * * *
      desiredReplicas: "10"

After (if we support multiple schedules within one cron scheduler):
  triggers:
  - type: cron
    metadata:
      timezone: America/Los_Angeles
      schedules:
        - start: 39 * * * *
          desiredReplicas: "2"
        - start: 41 * * * *
          desiredReplicas: "4"
        - start: 43 * * * *
          end: 45 * * * * (optional)
          desiredReplicas: "4"
           ...

Questions:
Happy to submit a PR if the team think either proposal makes sense.
Anything else?
No response

Hi @hochuenw-dd
Currently the first option (having multiple triggers with end optional) isn't possible because each trigger is treated a single metric, so you can't overlap them during the time, and the HPA Controller will operate them using MAX between them, so you will scale to the max desiredReplicas since the beginning.
WRT the second approach (all the schedules in the same trigger), it's doable (basically we have to check if the schedule has an ending and if not, look for the next start as current ending) and I see the improvement here, but I'm not sure if makes sense to do it here or directly as part of the window-trigger
WDYT @kedacore/keda-core-contributors ?


modify unserializable struct in plugin runtime without global state

examples/example-rust-wasmer2-runtime/src/spec/mod.rs  gives a way for plugin to modify a host struct using global state. can this be done without global state?
BTW, the example-rust-runtime/spec/mod.rs in readme does't exist.

Thanks for the heads-up!
What do you mean by modifying a host struct exactly? The global state it is modifying in that example lives within the plugin, so wouldn't affect the host at all.
Could you explain what you would like to achieve?

this import function can change value in GLOBAL_STATE. GLOBAL_STATE is init in the runtime code, so i think it is a host value.
https://github.com/fiberplane/fp-bindgen/blob/4e95816ded538ff7bde4ddf1d460326d95dff046/examples/example-rust-wasmer2-runtime/src/spec/mod.rs#L183-L190
the GLOBAL_STATE can also hold an unserializable struct, like a egui context. thus i can let plugin control ui via such import function.
but i don't want to avoid GLOBAL_STATE style.
here is an example code from mlua. the rust_val is a host value, it's can be modified in plugin(lua side). it can be a complex unserializable struct and don't need to be global.
https://github.com/khvzak/mlua/blob/c0c6a33f94127188a528801656effa89387880eb/examples/guided_tour.rs#L195C1-L215C6
    {
        let mut rust_val = 0;

        lua.scope(|scope| {
            // We create a 'sketchy' Lua callback that holds a mutable reference to the variable
            // `rust_val`. Outside of a `Lua::scope` call, this would not be allowed
            // because it could be unsafe.

            lua.globals().set(
                "sketchy",
                scope.create_function_mut(|_, ()| {
                    rust_val = 42;
                    Ok(())
                })?,
            )?;

            lua.load("sketchy()").exec()
        })?;

        assert_eq!(rust_val, 42);
    }

Oh, you're right, that's my bad. Yes, that one is indeed kept in host memory. Frankly, it's not something we support very well, since the use cases we focus on are more situations where the plugin does things for the host without assuming persistence. Still, I think it's a useful request to consider when we eventually migrate to Wasmer 4.


Partially missing docs for call, cps, and fork

Description
My team is using the syntax below to explicitly pass context and fn to fork, call and cps:
call({context, fn}, ...args)
cps({context, fn}, ...args)
fork({context, fn}, ...args)
It seems that the API reference is missing documentation for this style of invocation -- do we know if these are being omitted intentionally?
If it's simply missing from the documentation -- let me know if you need help with the PR for adding these.
As a followup, it'd be great to have these added to the flow-typed definitions. I can help with that as well if we want to move forward with this change.

The code that handles this style of invocation is here. I think the documentation is missing this style of invocation. Welcome to make a PR.
The typescript declaration file already contains type definitions for this style of invocation.

Thanks shinima -- I opened a PR here for doc additions. I'll pursue the type additions in the flow-typed repo.

closed by #1484


Consistant alignment, colors, and font sizes for ngx-select and ngx-inputs

Rebase and go ahead and merge @Hypercubed


Option to ignore untracked files

Having a flag / config item to ignore untracked files would be lovely. Looks like it's just a matter of tacking on -u no to infra/git/status.go if the option is set. I don't code in go, but I could probably wing a PR if you want.

I should add: I have a TON of projects (~200). Knowing which projects actually need to commit vs have junk laying around would help a ton.
I should also add: I'd really rather this be the default and you opt into an extra layer of reporting to know when you should clean up your repo or might need to add files.

@zenspider https://github.com/javijuol/git-remind/releases check out this fork


🚧 Dev ➾ Docusaurus Fixes from Michael K

🔗 Epic|Feature Link
No response
🚥 Definition of Done

[x] 🎯 Planning and Design
[ ] 🏎 Dev Implementation
[ ] 🏁 Test Plan Executed
[ ] 🛩 Documentation Updated
[ ] 🏟 Peer / User Reviewed
[ ] 🏆 Validated and Accepted

✅ To Do
No response
🔭 Scope
🏎 In Scope:

Implementing fixes mentioned
Fixing any other issues noticed while implementing this ticket

🙅♂ Out of Scope:

...

🧰 Work Description
@michaelkernaghan  had gone through our Docusaurus docs and identified some issues that need fixing
The thread can be found here: https://ecadlabs.slack.com/archives/C021J32T05B/p1657914143071919
Please get his help to review any docs if anything get's merged while Houston is away
⚖ Acceptance Criteria
📚 Documentation:

[ ] Update all relevant documentation with fixes
[ ]

🎢 Test Plan
No response
🎆 Release Notes
No response
🦄 Sizing
No response
Code of Conduct

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

Looked through thread and it looks like all the things Michael has pointed out have been fixed/are no longer valid


raspbery-pi-4: add i2c clock-frequency option

Recently I had an application where I wanted to tweak the default 100khz clock frequency of the i2c0 bus to 1mhz, so I decided to expand the i2c dts'es here to include that option.
I'm still very new to nix expressions, so I am sure this code can be simplified in many different ways - feedback is appreciated.
Test
Applying the following:
{
  options.hardware.raspberry-pi."4" = {
    i2c0 = {
      enabled = true;
      frequency = 1000000;
    };
  };
}

Results in:
xxd -g4  /proc/device-tree/soc/i2c@7e804000/clock-frequency
00000000: 000f4240 # 1_000_000

cc @06kellyjac for review.

my only thought is the frequency could default to null and then if null we don't add that line to the dtb but that's probably more effort than it's worth

Just undraft it, when you made your change.

@06kellyjac @Mic92 alright I made this change, please have another look.


Inconsistent sentry-trace and baggage values on incoming trace

Version
incoming trace version - 2.11.0, outgoing trace version - unknown
Steps to Reproduce
https://sentry.sentry.io/api/0/projects/sentry/sentry/events/6122ec07d3694b7ba1aa88db962afde5/json/

trace_ids on the two headers are different
sentry-trace is sampled but the baggage is not

sentry-trace:  ac9fb4acad4a4f3d90ea2660a9d060b9-aa7d2d9bb68db6d5-1
baggage:       sentry-trace_id=85f29184b20d47c28dde78483288c3d7,sentry-sample_rate=0.0,sentry-sampled=false,sentry-environment=stage,sentry-public_key=[REDACTED]

the request comes from <IP_ADDRESS> which are our internal proxies. they usually relay the client's IP unless the request actually originates internally. Combined with the fact that we're dealing with a silo RPC, the request that triggered this transactions may come from an internal service. However, this may in turn have been triggered by an external request. That I/we don't know.

@sl0thentr0py I am seeing this as well in my project. Im curious, are you seeing these inconsistencies and still able to effectively use distributed tracing across systems?
I am running into an issue where each new request to my BE from my FE / browser is causing these two methods below to return a very similar output as you see above:
sentry_trace = sentry_sdk.get_traceparent()
sentry_baggage = sentry_sdk.get_baggage()
// then print in a dict i get something like this:

{
'sentry_trace': '3311a76eb75844509d1e4406e3efc075-b9d29f60220bc5f0-1',
'baggage': 'sentry-trace_id=c867096f09a44c67892077b37d7cfd15,sentry-environment=development,sentry-release=777d725ed6de3660deaf1996ea88ceb847d21cfe,sentry-public_key=18a4b04ee1c04110abcb6302a134e3fa,sentry-transaction=/,sentry-sample_rate=1.0,sentry-sampled=true'
}

In my situation I have a my FE setup with sentry/browser sdk, and my backend is with the django sdk. My app is SSR, so this is the flow I have:

I add the meta tags to my html template with the sentry-trace and baggage values (This works as intended, when i throw a FE error, the trace starts with my BE / initial server request, and then shows the FE trace before crash).
I have allowed all targets with tracePropagationTargets as I am just testing as of now, which adds the sentry-trace and baggage headers (this also works as expected).
So, the page has loaded and when I finally when i make a request to my api / BE to an endpoint that throws and error sentry starts a new trace even though the sentry-trace and baggage headers are in the request header.

The interesting thing, is when I log the get_traceparent and get_baggage values I see something very similar to you.
// headers on request:
sentry-trace: c867096f09a44c67892077b37d7cfd15-...-1
baggage: c867096f09a44c67892077b37d7cfd15-...-1, sentry-trace_id=c867096f09a44c67892077b37d7cfd15,sentry-environment=development ... (the rest)

//
sentry_sdk.get_traceparent(): 3311a76eb75844509d1e4406e3efc075-...-1
sentry_sdk.get_baggage(): sentry-trace_id=c867096f09a44c67892077b37d7cfd15, ... (the rest)

So you can see that the baggage sentry-trace_id is actually the correct one from the header that is being passed from the FE, but my django sentry setup is not using that trace and is rather starting a new one.
I may open my own issue, but I saw a similarity from your post to what I am seeing and wanted to see if this give you an idea of what could be wrong, and also is the distributed tracing is still working for you even though you have this inconsistency?

thx for the report @scott-schibli, I haven't gotten time to debug this at the moment but I'll keep you posted when I pick this up. There's definitely some sort of case we're not handling properly.

@sl0thentr0py Do you have any inclination to why distributed tracing isn't continuing the trace with the django sdk even when the sentry-trace and baggage headers are present? Documentation from the sentry/browser and django sdk says that distributed tracing for django system should work out of the box when these headers present.
In the meantime I will try to manually continue the trace in our middleware when the headers are present.


Update to charrua.1.6.0

I am going off CI in #24

can we in config.ml as well put a lower bound to charrua >= 1.6.0?


OpenNebula: Snapshots per/user enable or disable feature

Author Name: kvaps kvaps (kvaps kvaps)
Original Redmine Issue: 5396, https://dev.opennebula.org/issues/5396
Original Date: 2017-09-25

This is really needed feature for allow or disallow selected user or group for use snapshots functionality.
I think it may be reached alowing override VM_ADMIN_OPERATIONS, VM_MANAGE_OPERATIONS and VM_USE_OPERATIONS variables per user group or per single user.

Original Redmine Comment
Author Name: kvaps kvaps (kvaps kvaps)
Original Date: 2017-10-13T22:52:19Z

Yes, it is very necessary to have possibility for override this parameters via VM template.

This should be implemented with the user/group level user/group wise API call-right permission mapping


[feature] Add a GeminiPro bot for creating ast-grep rules

⭐ Suggestion
Google's gemini pro is free for individual usage. We can build a chain of thought bot to create users ast-grep rules!
💻 Use Cases
Help people understand ast-grep and write people rules :)

fixed by ast-grep-bot!


Alternate Pem Keys

I needed the ability to use pems other than one sitting locally with the app.  Added the optional alternate_pem argument to public methods in APNS to give users the option of sending a different pem with each notification.
Also did some housekeeping and moved duplicate code into it's own methods.

This is now over 4 years old. No need to keep it open. Closing


Suggestion

I'd add "It's important that you push an item, and not an array that contains only that item, because those are two different things".
Challenge Stand in Line has an issue.
User Agent is: Mozilla/5.0 (iPad; CPU OS 10_0_2 like Mac OS X) AppleWebKit/602.1.50 (KHTML, like Gecko) Version/10.0 Mobile/14A456 Safari/602.1.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

function nextInLine(arr, item) {
  // Your code here
arr.push(item);
  var miss = arr.shift();

  return miss;  // Change this line
}

// Test Setup
var testArr = [1,2,3,4,5];

// Display Code
console.log("Before: " + JSON.stringify(testArr));
console.log(nextInLine(testArr, 10)); // Modify this line to test
console.log("After: " + JSON.stringify(testArr));

This assignment is under "Basic Javascript" and I think that assignments should be the same way "basic" like they actually are. It is true that that item !== array_item but that's not the point here. The point here is to teach people a new method from JS and how to implement it. Everything in depth would probably confuse people even more than it would actually teach them.

I think this adds a jump in thought not necessarily experienced by all campers.

The instructions say

Write a function nextInLine which takes an array (arr) and a number (item) as arguments.

where it says you've given a number, not an array. The challenge tests also indicate this as well. Campers can take a look at the MDN or .push() reference to find more. As others have pointed out, this may give campers unnecessary information at this point. Closing as the challenge instructions as sufficient. Happy coding!


editor.tokenColorCustomizations comments does not modify comment punctuation

VSCode Version: 1.38.1
OS Version: Windows_NT x64 6.3.9600

Steps to Reproduce:

Modify "comments" in "editor.tokenColorCustomizations"
Note how comment punctionation (i.e. // or /* */) are not highlighted

For instance,
"editor.tokenColorCustomizations": {
     "[Material Theme Darker High Contrast]": {
         "comments": "#f00"
     }
}

produces

I was able to target comment punctuation via the TextMate rule punctuation.definition.comment

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

To verify:

create a shell script with a comment

echo "asdasd"  # comment

set the comment color

"editor.tokenColorCustomizations": {
    "comments": "#FF00FF"
}

observe that also the # gets the new color


Where to enter refresh token in HAOS HACS ?

I installed this plugin today using HAOS 2022.8.1 and restarted but I cannot find anywhere to enter the refresh token and was not asked to enter a refresh token at any time during the process.  The only thing I can see in HAOS entities is "Google WiFi update" for HACS.
Am I doing it wrong ?

The refresh token could be entered for me after searching for and selecting the Google Wifi integration under the "+ Add Integration" menu under Devices & Services.
If you get the "[%key:common::config_flow::error::cannot_connect%]"  error like I did, reboot HA and make sure you're entering the refresh token generated by the "Your refresh_token" box that appears on the "_https://www.angelod.com/onhubauthtool/_" website after clicking the "Get refresh_token" (not the raw oauth_code cookie contents)


[BUG] Resourceful Routes Conflicting

I have two resource controllers, publicController and adminController within a package. publicController contains only index() and show() methods whereas adminController includes them all. Currently each method just returns the name of the method and, if relevant any parameters.
My routes.php file:
Route::resource('blog', 'Wfuk\Blog\publicController');

Route::group(array('prefix' => 'admin'), function()
{
    Route::resource('blog', 'Wfuk\Blog\adminController');
});

With this setup all of the admin routes seem to work ok. But none of the public ones work. If I place the public routes after the admin ones then example.com/blog works (calling the index() method) but example.com/blog/x returns 404.
At first I thought maybe it was an issue with using 'blog' for both resources so I tried changing the public one to:
Route::resource('test-blog', 'Wfuk\Blog\publicController');
This resulted in the admin routes working, the public list working, but again the show() route resulted in a 404.
I have tried dumpautoload etc.
Unsure if this is package related, I will try later today and see if I get the same results.

A similar issue occurs in L6 when a route resource and a route prefix have the same name. Moving the prefix'd route(s) before the resource definition causes it to behave as expected.
Doesn't work:
Route::resource('example', 'ExampleController');

Route::prefix('example')->group(function () {
    Route::resource('test', 'TestController');
});

Works:
Route::prefix('example')->group(function () {
    Route::resource('test', 'TestController');
});

Route::resource('example', 'ExampleController');

@hastinbe What you're describing is expected behavior. Your prefixed route would evaluate to example/test, but the resource routes define example/{example_id} which will catch the request to example/test and pass test as example_id.
This is something that happens with all routes that have parameters. You must define foo/bar before foo/{bar} in order for the first one to work as you expect it to.

@36864 exactly. left the comment to help others


Examlple DICOM study

I was able to get the OHIFViewer running on my machine with meteor, and successfully started the Orthanc DICOMWeb docker image on my machine as well (i.e http://localhost:3000 and http://localhost:8042 are both running).
I want to be able to start the actual viewer and learn how it work but I don't know where or how to get a valid study.
I uploaded some example DICOM file to Orthanc and it shows up in that list now, but the viewer doesn't show it.
Is there something I'm missing here?
Is there some way to get an example study to work with the viewer?

What do you mean, exactly, when you say "doesn't show it"?
Do you get an error in the viewport?

I mean that the study list is empty. I will try what you sent

Try changing the date filter. The default is to only show studies acquired in the past day.


Hide Actions tab from module "character-actions-list-5e"

Do not display actions tab in the loot sheet:

Should this be handled in this module or on the other one?
https://github.com/ElfFriend-DnD/foundryvtt-dnd5eCharacterActions

Probably the other one… it could have a list of exemption sheet types or something.

On Jun 28, 2021, at 4:11 AM, jocolamarco @.***> wrote:

Do not display actions tab in the loot sheet:
Should this be handled in this module or on the other one?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Do you think this is easy to add:

Ah this is something the other module can (hopefully) opt out of:
If the blockActionsTab option in the Application is true, the actions tab will not auto-inject.
app.options.blockActionsTab = true

Not an issue after I updated both and removed actions injection from NPC sheets


Sample should demonstrate the process to upgrade to a new identity

This may be a necessary behavior in case of identity compromise, or a transition from system-assigned to user-assigned

not necessary as sample


Add OrgReader

Description
This is a draft PR to add an Org mode reader that can create documents at different heading depths. (It's not uncommon for Org users to have more than 5 levels of headings). I still need to add unit tests and go through the checklist.
Type of Change

[x] New Loader/Tool for Org files

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have added a library.json file if a new loader/tool was added
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

I am totally mind blown! I started working yesterday on a OrgmodeReader, created the base structure, init... then came back to check the status of a PR... then discovered your PR. Tears of joy are about to happen!
Thanks so much for putting this up!


MM-T925 Users - Profile image on User Configuration page is round

Summary

Added test which verifies user profile image in user configuration page of system console is round
Ticket Link

MM-T925 Users - Profile image on User Configuration page is round
Related Pull Requests

N/A
Screenshots

N/A
Release Note

Hello @M-ZubairAhmed,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


Tutorial fiori-ios-hcpms-secure-keystore/fiori-ios-hcpms-secure-keystore: Several issues

In tutorial
https://github.com/SAPDocuments/Tutorials/blob/master/tutorials/fiori-ios-hcpms-secure-keystore/fiori-ios-hcpms-secure-keystore.md
there are several issues.
Step 3:

SAPCommon has to be imported to be able to use the Logger in step 4.

Step 4:

Name parameter for Logger.shared wrong (Logger.shared(withName ... -> Logger.shared(named ...)

Step 7:

SAPURLSession has not init function with a parameter urlSessionDelegate. The init function with parameter "delegate" needs to be used. That all is in sync, the class KeystoreAuthenticator has to implement protocol SAPURLSessionDelegate instead of URLSessionTaskDelegate (like described in step 4).

Step 12:

The task delegation logic for the SAPURLSession has to be changed according to the requirements for protocol SAPURLSessionDelegate.

@qualiture - please review


Facilitation of DCOP

Description
The Digital Code of Practice (DCOP) is a guide for everyone - public service employees and contractors - working with digital service delivery. By running a 1-hour workshop using the kit provided, it is hoped that everyone in the team/business area get the same practical guidance on modern digital services to citizens.
Acceptance Criteria
[Note: Use 'Given/When/Then' format if it makes sense to. Otherwise, a simple checklist that can be tested.]
Development Checklist

[x] Getting access to the facilitation kit from the DCOP team
[ ] Note the steps to be taken to before, during, and after the workshop and expected outcomes - discuss with @Dianadec
[ ] Start preparation by reading the principle and preparing the presentation
[ ] Check-in with @Dianadec and @JLWade about any changes to make before finalizing the workshop plan
[ ] Do a demo workshop (optional)
[ ] Prepare the email (with purpose, date & time, weblinks, tools, expectations) that Jess can send to the attendees

Dependencies
Blocked by:

[x] Requesting the kit by emailing
[ ] Get in touch with @Dianadec to facilitate this workshop (and estimating a date)
[ ] Preparing the steps by @kgorana

Blocking:

[ ] Facilitation of the workshop
[ ] Post-event form to be sent to the DCOP team
[ ] People's awareness on the DCOP's 10 principles

Relevant documentation as reference

Miro board for preparation notes: https://miro.com/app/board/uXjVK1CcmOM=/
Miro board to be used for the workshop: https://miro.com/app/board/uXjVK3RRwoY=/

Definition of Ready

[ ] Acceptance criteria are included
[ ] Wireframes are included (if applicable)
[ ] Design / Solution is accepted by Product Owner (if applicable)
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done
In progress:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] UI meets accessibility requirements
[ ] Unit tests are written
[ ] Work is traceable in GitHub
[ ] PR linked to ticket number
[ ] If needed/required - Dev adds flag/label to highlight any migration steps necessary prior to PROD deployment

PO Review:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Reviewed and approved by Product Owner

Notes

https://digital.gov.bc.ca/policies-standards/dcop/
https://compass.gww.gov.bc.ca/introducing-digital-code-of-practice/

using this file temporarily: https://app.mural.co/t/kavispesonalmuralworkspace2376/m/kavispesonalmuralworkspace2376/1712775878770/b81286142f69e2c9ae6fb3a76c473422be8e8357

I need to work on this: https://miro.com/app/board/uXjVK1CcmOM=/?moveToWidget=3458764593973054155&cot=14

post-facilitation survey is not submitted and all the tasks in this ticket are done - so moving to PO review :)

@Dianadec - are there next steps associated with this ticket that need to chat about?

Confirmed with Diana and Jess that this ticket is no longer needed. Closing.


style: Remove unneccesary method in FastAPI example

I was looking at this example and noticed the get_token method wasn't really necessary since it was just returning what it was given. If you want to keep the method though it can be simplified to just return posit_connect_user_session_token. The if statement is not needed.

Thanks @hekhuisk!
You can ignore the error in Python 3.12 testing. There is a configuration error when running on forked PRs.

I made https://github.com/posit-dev/posit-sdk-py/issues/87 for fixing this.
Thanks @hekhuisk!


Video Files Not Supported

I uploaded four video files and neither file was pushed to GCS.
.webm, .ogv, .mp4, .mov
It also rewrote the file paths and they are broken.

Replacing this issue with another issue that covers ALL file types. https://github.com/wpCloud/wp-stateless/issues/7


Mux Controller, Mux/Demux blueprints? Missing from book.

Hi,
Your google doc mentions a very interesting multiplexing technique for the main powerlines, and claims that there are mux/demux blueprints available, but I can't find them. Let me know if I'm missing something. Thanks.

They're in one of the sub-books.  Don't remember exactly which one.
On Mon, Jun 24, 2024 at 12:54 AM Mutable Devotions @.***>
wrote:

Hi,
Your google doc mentions a very interesting multiplexing technique for the
main powerlines, and claims that there are mux/demux blueprints available,
but I can't find them. Let me know if I'm missing something. Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/bcwhite-code/brians-blueprints/issues/16, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLEM3FAMJNGREUDSYOX54DZI6RABAVCNFSM6AAAAABJZB7TPSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM3DSMRQGIZTEMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Brian
@.***
Treat someone as they are and they will remain that way.Treat someone as
they can be and they will become that way.

I probably should have thought to search the codebase.
It's in the self-building factory.
Consider adding that to the trains aux book for good measure but I'll close the issue, thanks.

I've updated the doc to indicate such.


Add SQL Server codespaces code examples

Why:
Closes #21581

What's being changed (if available, include any code snippets, screenshots, or gifs):

Add a new SQL sample in the Code examples section of Codespaces docs landing page
Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

I don't have the staging address, however I've reviewed https://docs-21580-ec9634.preview.ghdocs.com/en

Hi @NostalgicGareth,
I think there is an issue with your comment :)

@cmaneu Thank you for opening a PR and linking it to your issue! ✨
I'll get this triaged for review ⚡


New version: TheQucsTeam.Qucs-S version 24.2.0

Created by 🥟 Dumplings in Run #282.

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！


dockerfile: update for Node 10

Ah maybe update the references to node:8 in the README.md as well?


As promised, changed the bot to use databases lib making the calls async.

Type:

[ ] Bugfix
[x] Enhancement
[ ] New feature

Description of the changes:
Mostly just replacing the use of sqlite3 to databases lib. Also, running the bot from the old database won't work cause I changed the queries. And, as you said, the database is empty just delete the old database, then try to boot the bot.

lgtm

Thanks bro. :D


Can I use this from C#?

In C#, Console is provided by System, so I can't actually seem to find a way to add a command, from C#.

I haven't tried it but I think it should be possible.
I suppose based on the docs registering a command would look something like this:
Console.Call("add_command", "sayHello", this, "print_hello")
	.Call("set_description", "Prints \"Hello %name%!\"")
	.Call("add_argument", "name", Variant.Type.STRING)
	.Call("register");

Docs for reference:
https://docs.godotengine.org/en/stable/getting_started/scripting/c_sharp/c_sharp_differences.html#singletons
https://docs.godotengine.org/en/stable/getting_started/scripting/cross_language_scripting.html#calling-methods
If you'll be able to set it up please get back to me I'd like to know the results)

I haven't tried it but I think it should be possible.
I suppose based on the docs registering a command would look something like this:
Console.Call("add_command", "sayHello", this, "print_hello")
	.Call("set_description", "Prints \"Hello %name%!\"")
	.Call("add_argument", "name", Variant.Type.STRING)
	.Call("register");

Docs for reference:
https://docs.godotengine.org/en/stable/getting_started/scripting/c_sharp/c_sharp_differences.html#singletons
https://docs.godotengine.org/en/stable/getting_started/scripting/cross_language_scripting.html#calling-methods
If you'll be able to set it up please get back to me I'd like to know the results)

I don't think that works, because in C# system provides console and so far as I can tell, you can't override that from GD.

I don't think that works, because in C# system provides console and so far as I can tell, you can't override that from GD.

Could you please share the code you are using?

Could you please share the code you are using?

using System;
using Godot;

public class ConsoleScript : Node
{
    // Declare member variables here. Examples:
    // private int a = 2;
    // private string b = "text";

    // Called when the node enters the scene tree for the first time.
    public override void _Ready()
    {
        Console.Call("add_command", "sayHello", this, "print_hello")
            .Call("set_description", "Prints \"Hello %name%!\"")
            .Call("add_argument", "name", Variant.Type.STRING)
            .Call("register");
    }

//  // Called every frame. 'delta' is the elapsed time since the previous frame.
//  public override void _Process(float delta)
//  {
//
//  }
}

using System;
using Godot;

public class ConsoleScript : Node
{
    // Declare member variables here. Examples:
    // private int a = 2;
    // private string b = "text";

    // Called when the node enters the scene tree for the first time.
    public override void _Ready()
    {
        Console.Call("add_command", "sayHello", this, "print_hello")
            .Call("set_description", "Prints \"Hello %name%!\"")
            .Call("add_argument", "name", Variant.Type.STRING)
            .Call("register");
    }

//  // Called every frame. 'delta' is the elapsed time since the previous frame.
//  public override void _Process(float delta)
//  {
//
//  }
}

I guess a quick fix would be to go project settings and rename Console to something else.

I guess a quick fix would be to go project settings and rename Console to something else.

Could you point me at where I would do that?

Could you point me at where I would do that?

Yeah sure: Open Project > Project Settings > Plugins, search for quentincaffeino-console

Yeah sure: Open Project > Project Settings > Plugins, search for quentincaffeino-console

Sorry, instead of Plugins open Autoload

Sorry, instead of Plugins open Autoload

Changing the name seems to not work, specifically it gives me this:

Changing the name seems to not work, specifically it gives me this:

Oh yeah, I'm in progress of fixing that. You could go and find&replace all occurrences of Console. in res://addons/quentincaffeino-console. Sorry for inconvenience, the project is still in active development and there are plenty of space to improve.

Oh yeah, I'm in progress of fixing that. You could go and find&replace all occurrences of Console. in res://addons/quentincaffeino-console. Sorry for inconvenience, the project is still in active development and there are plenty of space to improve.

No worries, and I totally get it ^>^
For sure, C# is still fairly new to Godot, so compatibility I've found to be hit or miss; Thank you for taking the time to help me out 👍

No worries, and I totally get it ^>^
For sure, C# is still fairly new to Godot, so compatibility I've found to be hit or miss; Thank you for taking the time to help me out 👍

That's not really a problem with C# but with how badly I wrote this project since I don't have much time to spend on it)

That's not really a problem with C# but with how badly I wrote this project since I don't have much time to spend on it)

I did a full search and replace with find . -type f | xargs sed -i 's/Console\./InGameConsole\./g' which worked (after I fixed references to some files haha), but I still can't figure out how to use it in C#, InGameConsole doesn't seem to exist 🤔 .

I did a full search and replace with find . -type f | xargs sed -i 's/Console\./InGameConsole\./g' which worked (after I fixed references to some files haha), but I still can't figure out how to use it in C#, InGameConsole doesn't seem to exist 🤔 .

And you have it renamed in project autoload settings? That's odd, could you please share your project.godot or whole project?)

And you have it renamed in project autoload settings? That's odd, could you please share your project.godot or whole project?)

project.zip

project.zip

https://docs.godotengine.org/en/stable/getting_started/scripting/cross_language_scripting.html
This seems to suggest that I would need your console to be a node which doesn't seem to be how your project operates at this point :/

https://docs.godotengine.org/en/stable/getting_started/scripting/cross_language_scripting.html
This seems to suggest that I would need your console to be a node which doesn't seem to be how your project operates at this point :/

On line 24 in your project godot I see you also renamed the actual file you load, this file doesn't exist actually so I suppose you also renamed actual file?
InGameConsole="*res://addons/quentincaffeino-console/src/InGameConsole.tscn"

On line 24 in your project godot I see you also renamed the actual file you load, this file doesn't exist actually so I suppose you also renamed actual file?
InGameConsole="*res://addons/quentincaffeino-console/src/InGameConsole.tscn"

This seems to suggest that I would need your console to be a node which doesn't seem to be how your project operates at this point :/

What do you mean to be a node? When console is autoloaded it is added to the root of the scene tree so in gdscript you could retrieve it by doing something like get_node('/root/Console'), for C# there is similar method GetNode("/root/InGameConsole").

This seems to suggest that I would need your console to be a node which doesn't seem to be how your project operates at this point :/

What do you mean to be a node? When console is autoloaded it is added to the root of the scene tree so in gdscript you could retrieve it by doing something like get_node('/root/Console'), for C# there is similar method GetNode("/root/InGameConsole").

Ok I thought that best way to test this is to try myself, I got almost everything working with no changes to the addon code. Only thing that doesn't work for me is Call, but if I do c is Object it returns True, and Call is defined on Object. So I would suggest the problem is with C# support since docs clearly states that Call is defined on object:
https://docs.godotengine.org/en/stable/getting_started/scripting/cross_language_scripting.html#calling-gdscript-methods-from-c
My code:
using Godot;
using System;

public class new_script : Node
{
	// Declare member variables here. Examples:
	// private int a = 2;
	// private string b = "text";

	// Called when the node enters the scene tree for the first time.
	public override void _Ready()
	{
		var c = GetNode("/root/Console");
		GD.Print(c);
		GD.Print(c is Godot.Object);

//		c.Call("add_command", "sayhello", this, "printhello")
//			.call("set_description", "prints \"hello %name%!\"")
//			.call("add_argument", "name", Variant.Type.String)
//			.call("register");
	}

	public void PrintHello(String name) {
		GD.Print("Hello " + name + "!");
	}

//  // Called every frame. 'delta' is the elapsed time since the previous frame.
//  public override void _Process(float delta)
//  {
//
//  }
}

Ok I thought that best way to test this is to try myself, I got almost everything working with no changes to the addon code. Only thing that doesn't work for me is Call, but if I do c is Object it returns True, and Call is defined on Object. So I would suggest the problem is with C# support since docs clearly states that Call is defined on object:
https://docs.godotengine.org/en/stable/getting_started/scripting/cross_language_scripting.html#calling-gdscript-methods-from-c
My code:
using Godot;
using System;

public class new_script : Node
{
	// Declare member variables here. Examples:
	// private int a = 2;
	// private string b = "text";

	// Called when the node enters the scene tree for the first time.
	public override void _Ready()
	{
		var c = GetNode("/root/Console");
		GD.Print(c);
		GD.Print(c is Godot.Object);

//		c.Call("add_command", "sayhello", this, "printhello")
//			.call("set_description", "prints \"hello %name%!\"")
//			.call("add_argument", "name", Variant.Type.String)
//			.call("register");
	}

	public void PrintHello(String name) {
		GD.Print("Hello " + name + "!");
	}

//  // Called every frame. 'delta' is the elapsed time since the previous frame.
//  public override void _Process(float delta)
//  {
//
//  }
}

Well shit, Thanks!

Well shit, Thanks!

No problem)
Going to leave this issue open for now as this needs to be fixed.
Best

No problem)
Going to leave this issue open for now as this needs to be fixed.
Best

Thanks to folks in discord and telegram I was able to solve this pretty quickly.
C# is fine, it just required a little time to play around to get along with how it works. This is a working version:
using Godot;
using System;

public class new_script : Node
{
	public override void _Ready()
	{
		(((GetNode("/root/Console").Call("add_command", "sayhello", this, "PrintHello") as Godot.Object)
			.Call("set_description", "prints \"hello %name%!\"") as Godot.Object)
			.Call("add_argument", "name", Variant.Type.String) as Godot.Object)
			.Call("register");
	}

	public String PrintHello(String name = null) {
		GD.Print("Hello " + name + "!");
		return "test";
	}
}

One thing to take a note of: Even tho name is provided by default here, it would be still a required argument for command (right: sayhello hello, wrong: sayhello would throw no such method exception because of signature mismatch).
Also the code style is very bad because call returns generic object since Call could return any value.
I am planning to write some kind of C# wrapper to hide all that. Ideally it would look something like this:
...

	public override void _Ready()
	{
		QSConsole.Default()
			.AddCommand("sayhello", this, "PrintHello")
			.SetDescription("prints \"hello %name%!\"")
			.AddArgument("name", Variant.Type.String)
			.Register();
	}

...

I'm closing as I consider this solved. If you'd have any problems you can reopen this or open new issue.
Best.

Thanks to folks in discord and telegram I was able to solve this pretty quickly.
C# is fine, it just required a little time to play around to get along with how it works. This is a working version:
using Godot;
using System;

public class new_script : Node
{
	public override void _Ready()
	{
		(((GetNode("/root/Console").Call("add_command", "sayhello", this, "PrintHello") as Godot.Object)
			.Call("set_description", "prints \"hello %name%!\"") as Godot.Object)
			.Call("add_argument", "name", Variant.Type.String) as Godot.Object)
			.Call("register");
	}

	public String PrintHello(String name = null) {
		GD.Print("Hello " + name + "!");
		return "test";
	}
}

One thing to take a note of: Even tho name is provided by default here, it would be still a required argument for command (right: sayhello hello, wrong: sayhello would throw no such method exception because of signature mismatch).
Also the code style is very bad because call returns generic object since Call could return any value.
I am planning to write some kind of C# wrapper to hide all that. Ideally it would look something like this:
...

	public override void _Ready()
	{
		QSConsole.Default()
			.AddCommand("sayhello", this, "PrintHello")
			.SetDescription("prints \"hello %name%!\"")
			.AddArgument("name", Variant.Type.String)
			.Register();
	}

...

I'm closing as I consider this solved. If you'd have any problems you can reopen this or open new issue.
Best.


Visual instance layers are regarded during shadow culling

This PR essentially is a cherry-pick from 16517ec (merged for 3.x in #61841) adapted to some rendering server and culling modifications. It addresses #43827, #54613 and #58322 and seems to have the same effect as #61841. Which also means that shadows are not yet updated after changing the Camera cull_mask itself.
Bugsquad edit:

Fixes #58322

Thanks! And congrats for your first merged Godot contribution :tada:


Xcode 7.0 and Swift 2.0: error in subscript function 'result type JSON does not match expected type'

I can't resolve the syntax error I get for the 4 lines in sbscript function. Have anyone resolved this error somehow?
  private subscript(sub: SubscriptType) -> JSON {
        get {
            if sub is String {
                return self[key:sub as! String]    // Error result type JSON does not match expected type
            } else {
                return self[index:sub as! Int]   // Error result type JSON does not match expected type
            }
        }
        set {
            if sub is String {
                self[key:sub as! String] = newValue    // Error result type JSON does not match expected type
            } else {
                self[index:sub as! Int] = newValue    // Error result type JSON does not match expected type
            }
        }
    }

Just found the latest source https://github.com/SwiftyJSON/SwiftyJSON/blob/master/Source/SwiftyJSON.swift has the update for Xcode 7.0. No syntax errors with the file.

Cool, please use version 2.3.0 or up for Xcode7 support


Graph feedback

(Desktop) ParaTime should directly navigate when zoomed out.
When zoomed in the user should be able to see that status of the ParaTime.
When zoomed into a specific ParaTime any unrelated ParaTimes should have 50% opacity.
The outer stroke should be display on top of the graph content.
Reduce opacity on mobile tooltip.
Fix ‘Coming soon’ label displayed on mobile.
Shadows cut off on testnet graph.
Add selected network label.

Graph position if wrong. It's moved to right and bottom and it overlaps Selected Network
Emerald is broken on mobile (it should behave like Cipher)


Minimal update for objection > 0.8. Needs first 'find' to be updated.

Coverage increased (+7.3%) to 88.55% when pulling 652a6aa8673c4f970edede259f06c909786bc7c1 on devel-pa:master into a609cb1da32353cd17ca6a66d1f1530975d96f2b on Vincit:master.

Hi,
it's fixed with passing tests in master branch.


Return unsigned delegation certificate

Context
When joining a stake pool, the process is as follows:

Check if the account has already a registered stake key.
Check if the requested pool can be joined (it exists and it's not retiring).
Select coins to cover for either the delegation, or the delegation AND the key registration.
Sign the delegation (provide a witness for the stake key).
Submit the transaction.

The user wants to be able to:

Stop after step 3
Get the unsigned delegation certificate made by the wallet
Do step 4 and 5 on the client side

This ticket is about 1. and 2.
Decision
Acceptance Criteria

The endpoint POST /wallets/{walletId}/coin-selections/random should be extended to accept a new request body, either { "action": "join", "pool": <pool id> } or { "action": "quit" }.
Request bodies providing both "payments" and "action" should fail to parse with an appropriate error.
Request bodies providing "pool" without "action" should fail to parse with an appropriate error.
When provided with a poolId and an action the handler must verify that the action is valid and be consistent with the existing POST /stake-pools/{poolId}/wallets/{walletId} and DELETE /stake-pools/*/wallets/{walletId}` handlers (i.e. re-use the same logic).
If valid, the request must be processed and return a list of resolved inputs, outputs, and a list of "certificates" depending on the scenario. Inputs and outputs should be already balanced to accommodate for the certificates.
Certificates can be either: (1) a key deregistration certificate, (2) a key registration certificate AND a stake delegation certificate, (3) a stake delegation certificate.
Certificate must be represented as JSON objects:

Key registration certificates should include a tag identifying the certificate, a bech32 serialized public stake key (according to CIP-5) and the derivation path corresponding to that key. (e.g. { "type": "key_registration", "stake_public_key": { "key": "stake_xvk1____", "derivation_path": "1852H/1815H/0H/2/0" } })
Key de-registration certificates should include a tag identifying the certificate, a bech32 serialized public stake key and the derivation path corresponding to that key.
Stake Pool delegation certificates should include a tag identifying the certificate and a bech32 serialized pool id.

QA

I have two questions/remarks:

Check if the requested pool can be joined (it exists and it's not retiring).

Currently one is able to join pool that is retiring... so does this means in coin-selection there will be a check to prevent that? I guess that wouldn't be quite right...

The endpoint POST /wallets/{walletId}/coin-selections/random should be extended to accept a new request body, either { "action": "join", "pool":  } or { "action": "quit" }.

Just a thought, but maybe a bit more clear would be to have something like:

POST /wallets/{walletId}/coin-selections/stake-pools/join/{pool id}
POST /wallets/{walletId}/coin-selections/stake-pools/quit

@piotr-iohk QA section updated

lgtm.


Lock/limit the angle at which body joints are detected (pose)

How can the (camera) angle be limited in the body-joints detection of the MediaPipe pose? I am experiencing random horizontal flips in the body joints (between the front angle and the back angle). Because the faces in my video dataset are blurred, MediaPipe has difficulty detecting the angle of the body joints and alternates between front and back angles. Can I restrict MediaPipe to only detect body joints from the front angle?

Hi @abedicodes ,
We have similare issue https://github.com/google/mediapipe/issues/2999 , please check this issue for angle calculations.


add rbenv and pg9.5 bash

add pg 9.5
add rbenv
add nginx config alias when the nginx only support these for commands: stop, quit, reopen, reload

@lexuszhi1990 I just saw your PR 👯

@therealechan  close it plz...................................


Win32 COFF: fix std.process unittests

use CRuntime_Microsoft instead of Win64.

Does nobody care about Windows support?! It's been 4 days and not a single review...
Anyway, I'm not exactly qualified to review this either since I don't have a Windows dev environment, but the autotester has been green for a while now and I'm tempted to merge this if nobody else looks at it in the next few days.

I did look over this minutes after it was filed, but I have no experience with DMD/Win64. That said, I understand what it does: that line of code is specific to Microsoft's C runtime, both 32-bit and 64-bit, but DMD only supported the Microsoft runtime for 64-bit before COFF was implemented for 32-bit as well very recently by @rainers himself.
So, in theory, LGTM :)

Should we just merge then? I'm not seeing the point of just letting this sit here and rot. It is just a 1-line change after all.

Auto-merge toggled on

@quickfur, agreed, thanks for paying attention :)

Does nobody care about Windows support?! It's been 4 days and not a single review...

@quickfur Most people don't care about win32 coff.  We really need an autotester for it.


GBM_WSSM.h5 link broken

Hi, the link to GBM_WSSM.h5 is broken. Please, can you check it?

Hi, the link to GBM_WSSM.h5 is broken. Please, can you check it?

Hello,
I just checked the link and it works properly.
Change your browser, please and check again.
Thanks,

Hello, it looks like the link to GBM_WSSM.h5 is broken again. Can you please update it?


security - fix dependabot alert - ecstatic

problem:

dependabot found a http-server's dependency ecstatic in package-lock.json, which has a security vulnerability.

http-server has an issue opened with lots of developers naming the problem, but based on the current status, in the recent future http-server may not offer a version without the vulnerability.
forcing ecstatic to be the latest version cannot work with existing http-server.

solution:

based on the discussion with Nick (theoutlander), and based on the existing repo setup with simulator service and connector examples, we would migrate http-server to serve-static, which also supports recursively hosting all resources among different resource folders.
developers can further add more settings as needed for service hosting in index.js with more flexibility.
tested on mac and windows with bundle and tests.

Coverage remained the same at 79.474% when pulling 28fd7a11d9eeffdb3085f876dce8a11e3bca10aa on security/dependabotAlert into 807e3830c166e43331c70df06f75ac7971e80fed on master.


Are there any differences which have to be considered in the code?

Hi,
It's not that I forbade it specifically, it's just that inline support wasn't available when I released the bot, and I simply haven't updated it with inline support when it came out. To be honest, it's not in the plans right now for the immediate future but it's been on my mind to upgrade.


General enhancements

[ ] We should probably start working on a minimal documentation, and clean the package from older/barely used scripts.

I'm now working on documentation, however since ribo has been moved to rp-bp, I don't want to put too much effort on pbiotools.


Correction to markdown list coloring

This is a follow-up to https://github.com/ginfuru/vscode-better-solarized-dark/issues/1.
Thanks for the update!
I think there's a small typo:
punctuation.definition.list.begin.markdown should actually be beginning.punctuation.definition.list.markdown. Right now there's no effect.

I'll take a peek at it... If you think you've got a fix PRs welcome and encouraged

Unfortunately I have no experience. I was just guessing after noticing that it doesn't work for me, and looking at the scope name. But it seems to show up fine in the screenshot in #2 (the blue - for unordered list) so I'm not sure. For me there's still no coloring for either numbered or unordered lists.

@linhtto No worries
This is what I have for lists inside of Markdown:


Initial list of third party dependencies for #1610

Signed-off-by: Christopher Grote<EMAIL_ADDRESS>

Thanks Chris - nice work.
Were you able to generate this with a script?
Also the UI dependencies need adding
(more later)

Were you able to generate this with a script?

See comments on #1610

Also the UI dependencies need adding

I've simply taken everything that was in the root-level pom.xml, given our push to use dependency management...  Where / how are the UI dependencies defined?

I'm still waiting for feedback.
My suggestion is we merge this as a first-pass then make incremental improvements. I'll defer copying to 1.1 until more complete


Add @since annotation to Kotlin DSL reference

Expected Behavior
Our current Javadocs distribution includes @since annotation that shows the version of Gradle in which a method was introduced.

Current Behavior (optional)
Kotlin DSL shows no information about @since

Context
It seems this is currently blocked on Dokka feature:
https://github.com/Kotlin/dokka/issues/1618

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team and is prioritised by them.

This was fixed already by

https://github.com/Kotlin/dokka/pull/2967

This is part of Dokka 1.9.0 released last week

We need a new version of Dokkatoo to be published before we can fix this

Thanks for taking care of this!

Had to revert the PR fixing this... See

https://github.com/gradle/gradle/issues/26479

Dokatoo v2.0.0 is out now and should do the job.

Fixed in

https://github.com/gradle/gradle/pull/27697


Prevent multiple verification emails

This issue is imported from pivotal
I\'ts possible to repeat the web-requests leading to a validation  email when registering a new eduID. This could be used for mass-mailing.

We should check if a validation mail was recently sent, before sending another email.

https://github.com/OpenConext/OpenConext-myconext/commit/f3e1bc7f08474e470c5ad4846f971c557ff70330 (Okke Harsta - Jun 14, 2022)


Change Java to compiled language to facilitate multi-file

Pull Request: Enhance Java Support for Multi-File Submissions
Overview
This pull request aims to extend the functionality of our Java execution environment to support multi-file Java programs. Historically, our system has only supported single-file submissions, limiting the complexity and scope of projects that users can run. This update is a significant step towards accommodating more advanced Java applications and improving the versatility of our execution platform.
Goals

Primary Objective: Introduce multi-file support for Java submissions without impacting the existing support for single-file Java programs.
Maintain Compatibility: Ensure that single-file submissions continue to function as they have, even when the submitted .java file's name does not match the public class name.

Changes
1. Multi-File Support Implementation

New Compile File: Added a compile file to the Java package. This script leverages the existing Piston infrastructure to compile and execute multi-file Java programs. It automatically detects and compiles all .java files present in the request

2. Single File Submissions

No Changes for Single File: The process for single file submissions remains unchanged. Users can submit a .java file where the file name does not necessarily match the public class name, and it will continue to be supported seamlessly.

3. Multi-File Java Program Support

Main File Requirement: For multi-file submissions to be executed correctly, the "main" file containing the public static void main(String[] args) method must be the first file passed into the API. This ensures the correct entry point is identified and executed.

Hi!
I talked about this pull request awhile ago with Hex who asked me to push it upstream. I'm only about a year late in actually getting to this, so it may not be something that is wanted/needed anymore.

Hi!
I talked about this pull request awhile ago with Hex who asked me to push it upstream. I'm only about a year late in actually getting to this, so it may not be something that is wanted/needed anymore.

I think this feature is required. I am having a similar request in these lines, where i can be able to have a multi class single java file to execute. Currently it is not supported with piston.

Just a note!
I identified a minor bug with file passing, aka if you want to pass in a .txt file that a .java file should read from. I have resolved this issue and will commit changes soon.

Any updates?


Quick question about time factor

Hi Alex,
Thanks for your help with my last issue. I am currently running the timewarping code and I wonder what the time factor represent. The neuron factor is easy to understand since every neuron signal contributes to the template. But the time factor is more abstract to me. Does it mean how to align the template time bins and clock time bins?
Thanks,
Dongming

Yes they are the points that define the piecewise linear warping functions mapping clock time to template time.

Thank you，Alex！

Hi Alex,
Thanks for your last reply. I have two more questions and I hope you could help me out.
Firstly, I still could not identify the differences between time factor, neuron factor and tau_factors clearly. I find out the time factor has a shape of (time_bins, compoents) and tau has a shape of (trials, time_bins). If time factor maps clock time to template time, which clock time it refers to, the PC clock time? If so, the tau factor would map all trials' clock time to template time. I just want to make sure I get those factors right.
Moreover,  I see the loss function defined as RMS between data_predicted and input data. Since the input data remains invariant, the minimal loss and data_predicted should be one-time-determined. I do not understand why iteration of optimization needed.
Thank you very much!
Best regards,
Dongming

What class are you fitting? PiecewiseWarping?

Nonlinear warping. I am sorry that I made a confusion that my last question was about the TWPCA repository (https://github.com/ganguli-lab/twpca). I should try your new code, but this paper (High-performance brain-to-text communication via handwriting) used your former code.

I got all the meanings of factors through reading your paper and debugging the code many times. Thanks for your time discussing with me, Alex. Since I have no more question about the time-warp code. I would close this issue. Thank you again for your help!
Dongming


prognostic run: total_precipitation refactor in runfile loop

The handling of the total_precipitation variable in the sklearn runfile loop is hard to follow if total_precipitation is one of the variables set/predicted in _step_prephysics. If computed in prephysics it's retained in the loop's _state_updates dict through the various steps until the _apply_postphysics_to_dycore_state, when it is applied as a state update. However, that step also computes a total_precipitation as max(PRATE - , 0) in a separate dict of state updates. How these two are applied and which one is actually used (answer: the one from prephysics) is not obvious.
Suggestions include refactoring the postphysics step's total precip computation to the ML and nudging steppers and making it optional in the steppers, and adding better logging of how total_precipitation is being computed.

@brianhenn @AnnaKwa Is this still relevant now that 06934dc6494cd282cb404b3624c084292a1329f4 is in?

Yes, that commit only affected the ML stepper, which gets used in the postphysics update. The first issue described here, where a state update can be computed in the prephysics step but applied in the postphysics step, is still relevant. The second issue (unclear which total precip is used) has a comment line that explains what's going on (but the implementation is the same).


View all customer

Create html file to view all customer informations

Great there is no problem in the code


DynamoDB UpdateExpression Syntax with Special Characters Rejected with ValidationException

Looks like this might relate to #4580
Problem Statement
I am working with some internal tables where we use attributes prefixed by the underscore character to indicate metadata fields on DynamoDB records. For example, the _lastModified attribute in the record below.
{
    "name": "Zelda",
    "released": "1985",
    "_lastModified": "2020-01-01T19:17:07.000000"
}

When testing out the update_item operation on a mocked DynamoDB table, I received a ValidationError (traceback below) indicating that the UpdateExpression formed to modify the _lastModified field was invalid syntax.
Expected Behavior
I expect the same syntax validation that runs in boto3 to apply to the statements that I run against mocked resources. In this case, if the syntax is valid, I would expect that moto processes the update_item call and updates the mocked table accordingly.
Observed Behavior
When testing with moto, I get a ValidationError indicating that the UpdateExpression syntax is invalid. However, when testing against real DynamoDB resources, processing update_item requests containing the same syntax was allowed and processed successfully.
Tested versions:
I performed a clean install of boto3 and moto with the versions below using python 3.7, 3.8, 3.10 and saw the same failure on each
boto3 1.26.91 / botocore: 1.29.91
moto: 4.1.4
The version info below is from my regular working environment. I didn't do any sort of clean install, just observed the issue.
boto3 1.20.112 / botocore 1.17.112
moto: 2.2.15
Traceback
Traceback (most recent call last):
  File "./test_dynamo_ops.py", line 23, in <module>
    ExpressionAttributeValues={":_lastModified": "YESTERDAY"}
  File "/Users/andrewrohm/Projects/moto/testenv/lib/python3.7/site-packages/boto3/resources/factory.py", line 580, in do_action
    response = action(self, *args, **kwargs)
  File "/Users/andrewrohm/Projects/moto/testenv/lib/python3.7/site-packages/boto3/resources/action.py", line 88, in __call__
    response = getattr(parent.meta.client, operation_name)(*args, **params)
  File "/Users/andrewrohm/Projects/moto/testenv/lib/python3.7/site-packages/botocore/client.py", line 530, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/Users/andrewrohm/Projects/moto/testenv/lib/python3.7/site-packages/botocore/client.py", line 960, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (ValidationException) when calling the UpdateItem operation: ExpressionAttributeNames contains invalid key: Syntax error; key: "#_lastModified"

Steps to reproduce
The script included below will trigger the exception. When I rewrite it to target real AWS DynamoDB resources, no exception is generated and the update operation succeeds as expected.
import boto3
from moto import mock_dynamodb2
TABLE_NAME = "Games"
with mock_dynamodb2():
    mocked_dynamo_resource = boto3.resource("dynamodb", region_name="af-south-1")
    mocked_dynamo_resource.create_table(
        TableName=TABLE_NAME,
        KeySchema=[
            {"AttributeName": "id", "KeyType": "HASH"},
        ],
        AttributeDefinitions=[
            {"AttributeName": "id", "AttributeType": "S"},
        ],
        BillingMode="PAY_PER_REQUEST",
    )
    game_table = mocked_dynamo_resource.Table(TABLE_NAME)

    game_id = "ZELDA"
    game_response = game_table.update_item(
        Key={"id": game_id},
        UpdateExpression="set #_lastModified=:_lastModified",
        ExpressionAttributeNames={"#_lastModified": "_lastModified",},
        ExpressionAttributeValues={":_lastModified": "YESTERDAY"}
    )

    print(game_response)

Thanks for raising this @arohm-ff - marking it as a bug!


Update reactive-resume.yaml

Add new env variable
https://github.com/AmruthPillai/Reactive-Resume/pull/1922

Thank you for the PR!


Creating a queue in a specific region doesn't reflect in the queue's messages

When I create a SQS queue like so:
aws sqs --endpoint-url http://<host-ip/name>:9324 create-queue \
    --queue-name TestQueue --region eu-central-1

The specified --region doesn't seem to reflect the region represented by the messages received from the queue.
If I send a message to the either through the JavaScript aws-sdk or command line:
aws sqs --endpoint-url http://<host-ip/name>:9324 send-message \
    --queue-url http://<host-ip/name>:9324/queue/TestQueue  \
    --message-body "Some message"

The message received from the queue would look something like:
{
    "Records": [
        {
            "messageId": "6e9e6c35-df6a-4362-ae98-6d5f7f0e8a6a",
            "receiptHandle": "6e9e6c35-df6a-4362-ae98-6d5f7f0e8a6a#fa8da3e0-447a-4163...",
            "body": "Some message",
            "attributes": {
                "SentTimestamp": "1549118769614",
                "ApproximateReceiveCount": "1",
                "ApproximateFirstReceiveTimestamp": "1549118769618",
                "SenderId": "<IP_ADDRESS>",
                "MessageDeduplicationId": "",
                "MessageGroupId": ""
            },
            "md5OfBody": "2ae04ba2776203c2479661da8aa4417c",
            "eventSource": "aws:sqs",
            "eventSourceARN": {
                "Fn::GetAtt": [
                    "TestQueue",
                    "Arn"
                ]
            },
            "awsRegion": "us-west-2"
        }
    ]
}

But I would have expected the awsRegion attribute to say eu-central-1 instead of us-west-2.
I'm not sure if this as any practical implications or if it's just cosmetics, because I seem to be able to send messages even if the region specified in the aws-sdk when sending messages is not the one specified for the queue, at creation, nor the one specified for the queue in serverless.yml.
AWS.config.update({region: 'eu-north-1'}); // not the region specified for queue
var sqs = new AWS.SQS({
  apiVersion: '2012-11-05',
  endpoint: 'http://sqs:9324'
});
const params = {
      MessageBody: "Some Message"
      QueueUrl: "http://<host-ip/name>:9324/queue/TestQueue"
    };
await sqs.sendMessage(params).promise(); // this still works

I've created a small example where the queue is created in eu-central-1, but the messages are still reported as being from us-west-2.

@SirSkorpan, I'm trying to reproduce the problem but I'm unable to do so. I've tried to create a queue, send a message and finally receive it on both aws sqs instance and using elasticmq. Here are the "logs":

aws sqs

 ~/ aws --version
aws-cli/1.16.108 Python/3.7.2 Darwin/18.2.0 botocore/1.12.98
 ~/ source ~/.aws/keys/me/key.env
 ~/ aws sqs create-queue --queue-name remoteq --region eu-central-1
{
    "QueueUrl": "https://eu-central-1.queue.amazonaws.com/906175111765/remoteq"
}
 ~/ aws sqs send-message --queue-url https://eu-central-1.queue.amazonaws.com/906175111765/remoteq --message-body "whatever" --region eu-central-1
{
    "MD5OfMessageBody": "008c5926ca861023c1d2a36653fd88e2",
    "MessageId": "fb8b6277-1d56-4890-a1df-331f89230c7e"
}
 ~/ aws sqs receive-message --queue-url https://eu-central-1.queue.amazonaws.com/906175111765/remoteq --region eu-central-1 --attribute-names All --message-attribute-names All
{
    "Messages": [
        {
            "MessageId": "fb8b6277-1d56-4890-a1df-331f89230c7e",
            "ReceiptHandle": "AQEBH4e+CTmSeg3He6SnZsFU/aY+nMBxpAJZXaU389iX8IRJg9kFGsxehs84SLiqoapwBSIUKlFeagW9bvTUiXNMk6ooTXMsWPR6A97a5wIjWMdC+yUtCmrJTODOi/N7nhXT6VkY5xOWO7mGB/Iwg8Nhuo0H0MxdgJvvOC6aLpUoT+C9VhgHF3uDRFYkuyRTPvXvKDPoX9ygExpRqaEAye5ziy48aDrDXttptbg2sR8TH/o8vvJyNc+OWcCJtuVllPwqIXcC2QEJweYAiYnppLM/aM+VOC5dBWr/OQgee25NPHBfrjKwcRmNmicjUzf50PnpCf68gJLHMIpCQ86HbIiCquxkieuB/W7l83Pp5b76PHZVnC94zWCowRQmQs//+N0bA7p9BTUCUfjJGGaqbF6FQw==",
            "MD5OfBody": "008c5926ca861023c1d2a36653fd88e2",
            "Body": "whatever",
            "Attributes": {
                "SenderId": "906175111765",
                "ApproximateFirstReceiveTimestamp": "1550651720649",
                "ApproximateReceiveCount": "1",
                "SentTimestamp": "1550651717923"
            }
        }
    ]
}

elasticmq

 ~/ aws sqs create-queue --endpoint-url http://localhost:9324 --queue-name localq --region eu-central-1
{
    "QueueUrl": "http://localhost:9324/queue/localq"
}
 ~/ aws sqs send-message --endpoint-url http://localhost:9324 --queue-url http://localhost:9324/queue/localq --message-body "whatever" --region eu-central-1
{
    "MD5OfMessageBody": "008c5926ca861023c1d2a36653fd88e2",
    "MD5OfMessageAttributes": "d41d8cd98f00b204e9800998ecf8427e",
    "MessageId": "bab4de73-08f8-464f-816e-7dc6605179a5"
}
 ~/ aws sqs receive-message --endpoint-url http://localhost:9324 --queue-url http://localhost:9324/queue/localq --region eu-central-1 --attribute-names All --message-attribute-names All
{
    "Messages": [
        {
            "MessageId": "bab4de73-08f8-464f-816e-7dc6605179a5",
            "ReceiptHandle": "bab4de73-08f8-464f-816e-7dc6605179a5#82bb65ab-2a28-4975-af50-e5b315e79222",
            "MD5OfBody": "008c5926ca861023c1d2a36653fd88e2",
            "Body": "whatever",
            "Attributes": {
                "SentTimestamp": "1550652631309",
                "ApproximateReceiveCount": "1",
                "ApproximateFirstReceiveTimestamp": "1550652895282",
                "SenderId": "<IP_ADDRESS>",
                "MessageDeduplicationId": "",
                "MessageGroupId": ""
            }
        }
    ]
}

As you can see elasticmq returns some additional attributes but both servers don't return anything even related to awsRegion. Also the format of the message you provided as an example more resembles cloudtrail format than a message returned via invoking receive-message action on sqs instance.  But, what's interesting cloudtrail doesn't support logging events related to message creation.
Could you please clarify how it can be reproduced?

I'm closing this issue since I've received no reply.


CAMEL-12575:NPE on GET request with Content-Type header

CAMEL-12575:NPE on GET request with Content-Type header

And maybe it would be good to use ObjectHelper too to check for null


Registration keys and Version info

Add registration key option for invite-only and version info in the API

[x] Registration keys (API Only and Signup)
[ ] Version Info

@JSH32 👍


Rling -2 cuts off after exactly 25MiB without errors, destroying the file

I have two separate lists that contain sorted data. When sorting them the program completes without errors, but at the end of the process exited without error after 25600KiB exactly, or 25MiB
.\rling.exe -2 .\412.left.sorted .\412.left.sorted
Estimated memory required: 52,429,024 (50.00Mbytes)
Allocated in 0.0197 seconds
Start processing input ".\412.left.sorted"
Input file ".\412.left.sorted" complete. 327680 unique (0 dup lines) read. 0.0402 seconds elapsed
327,680 total lines written, 0 lines removed
Total runtime 0.0692 seconds

.\rling.exe -2 .\412.left.sorted .\412.left.sorted -M 10M
Memory for cache set to 10485760 bytes (was 52428800)
Estimated memory required: 10,485,984 (10.00Mbytes)
Allocated in 0.0053 seconds
Start processing input ".\412.left.sorted"
Input file ".\412.left.sorted" complete. 65536 unique (0 dup lines) read. 0.0061 seconds elapsed
65,536 total lines written, 0 lines removed
Total runtime 0.0218 seconds

Further proving the cause - memory allocation
The output appears normal, except that the file does not contain 327,680 lines but 1.8 Million sorted and deduplicated.

I don't think you meant to do that.
rling input output [remfil1 remfile2 ...]
In your case, you are reading 412.left.sorted, and putting the output to 412.left.sorted.

In-place file overwriting works for the other modes, just not -2 hence the issue

Ah, so trying to combine -2 and in-place should produce a "you can't do that" message?


Page Header docs still use Kibana logo

The page header docs still use the Kibana logo for the first example. This should be as simple as changing it to logoOpenSearch

@BSFishy - there shouldnt be a logo there at all - https://github.com/opensearch-project/oui/issues/178
We also deprecated the use of logo / icon in the page header here  :https://github.com/opensearch-project/oui/issues/644

Thanks for the context. Closed as duplicate of #178