text:
Add NUL character at the end of copied error message

C++ error message returned by what must be NUL-terminated. However, the current copy function only copied the characters, but didn't add the NUL. Allocate one more byte and set it to NUL.

BTW, here is the issue fixed by this (from Valgrind):
==21911== Thread 2 test_c_call_r:
==21911== Invalid read of size 1
==21911==    at 0x4E0C934: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==21911==    by 0x18FE27: cxx_run_test (tests.cc:812)
==21911==    by 0x13781C: test::test_c_call_r::cxx_run_test (test.rs:240)
==21911==    by 0x1377E1: test::test_c_call_r (test.rs:246)
==21911==    by 0x13CD56: test::test_c_call_r::{{closure}} (test.rs:235)
==21911==    by 0x127364: core::ops::function::FnOnce::call_once (function.rs:507)
==21911==    by 0x171B9E: test::__rust_begin_short_backtrace (function.rs:507)
==21911==    by 0x14326B: core::ops::function::FnOnce::call_once{{vtable.shim}} (lib.rs:648)
==21911==    by 0x170B29: test::run_test::run_test_inner::{{closure}} (boxed.rs:2000)
==21911==    by 0x13DB60: std::sys_common::backtrace::__rust_begin_short_backtrace (lib.rs:601)
==21911==    by 0x14349A: core::ops::function::FnOnce::call_once{{vtable.shim}} (mod.rs:550)
==21911==    by 0x1D0AD2: std::sys::unix::thread::Thread::new::thread_start (boxed.rs:2000)
==21911==  Address 0x527240a is 0 bytes after a block of size 10 alloc'd
==21911==    at 0x4E0713F: operator new[](unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==21911==    by 0x1A38A0: rust::cxxbridge1::errorCopy(char const*, unsigned long) (cxx.cc:452)
==21911==    by 0x1A399B: rust::cxxbridge1::Error::Error(rust::cxxbridge1::Error const&) (cxx.cc:472)
==21911==    by 0x1A4216: std::__exception_ptr::exception_ptr std::make_exception_ptr<rust::cxxbridge1::Error>(rust::cxxbridge1::Error) (exception_ptr.h:250)
==21911==    by 0x1A3AAA: cxxbridge1$default_exception (cxx.cc:555)
==21911==    by 0x19FDDD: cxx::result::CxxException::new_default (result.rs:66)
==21911==    by 0x18308F: <&T as cxx::result::ToCxxExceptionDefault>::to_cxx_exception (result.rs:214)
==21911==    by 0x1769A6: cxx_test_suite::ffi::_::__r_fail_return_primitive::__r_fail_return_primitive::{{closure}} (lib.rs:304)
==21911==    by 0x183E4A: core::result::Result<T,E>::map_err (result.rs:861)
==21911==    by 0x17B000: cxx_test_suite::ffi::_::__r_fail_return_primitive::__r_fail_return_primitive (lib.rs:304)
==21911==    by 0x1769D7: cxx_test_suite::ffi::_::__r_fail_return_primitive::{{closure}} (lib.rs:23)
==21911==    by 0x180CA8: cxx::unwind::prevent_unwind (unwind.rs:23)

@dtolnay Maybe Valgrind tests should be added in the CI? No idea how hard that is.

@dtolnay

I think this is correct as currently implemented.

Hm, well, OK, let's assume you get the message "test" from Rust. That is, pointer to 4 characters "test" + len = 4. You allocate 4 bytes and copy the 4 characters "test" into those 4 bytes.
Now, the exception is thrown and caught in the C++ code (via catch(rust::Error &e). The C++ code calls e.what() and receives the pointer to the 4 characters "test" (but no len - how? It's a const char*!). Now it wants to print it to an ostream for diagnosis purposes. OK, the operator<<(const char*) now starts iterating characters starting at the pointer and it receives 't', 'e', 's', 't', ... and then? Missing NUL terminator! So it happily starts printing the garbage afterwards, potentially running into an unmapped page and crashing the process.
Care to reconsider? 🙂

For "test", the len passed into this function is 5, not 4, so the memcpy already includes the original \0 byte.

@dtolnay
Sorry for making a fool of myself. You are right, the NUL byte is pushed in to_c_error() (well, it is inefficient as hell, but hey, we are on the error path; and yes, it is correct).
The valgrind output was from the change required for handling C++ exceptions via std::exception_ptr, which I pushed in PR https://github.com/dtolnay/cxx/pull/1180 (and which is in turn prerequisite for a few other things - like transparent exception passing through Rust). There, the default error is constructed from a Rust &str, which in turn is constructed by using formatting functions to format the error into a buffer, which then for most use cases saves all but 2 allocations which are strictly necessary to construct the exception (only the allocation of the string and the exception proper remains).
The original code allocated at least once extra for the to_string() (and depending on the error's Display trait implementation, likely one or two times more).
Anyway, the changes in the other PR are self-contained and valgrind-verified, so this change is indeed not needed.

authors: dtolnay schreter

repo: dtolnay/cxx


text:
Do releases better

I've already started bumping versions after we cut a release, rather than before, but we should do better.

Changelog
Document the actual release process

Anything else?

I've taken to keeping publish = false in my Cargo.toml files to prevent accidental releases, and including a -dev suffix on versions to further limit damage. I don't have a good way to bump versions automatically though, so it's a bit fiddly.

Will keep this in mind. :+1:

source: gharchive/issue

authors: dtolnay kamalmarhubi steveklabnik

repo: dtolnay/semver

license_type: permissive

license_source: github-api


text:
Naming a field as "source" will cause the error E0599

Code as follows:
use thiserror::Error;

#[derive(Error, Debug)]
enum CustomError {
    #[error("Unable to convert value from {source} to {destination}")]
    UnableConvert { source: String, destination: String },
}

fn main() {
    println!("Hello, world!");
}

Running cargo check will cause the error E0599:
error[E0599]: the method `as_dyn_error` exists for reference `&String`, but its trait bounds were not satisfied
   --> src/main.rs:6:21
    |
6   |     UnableConvert { source: String, destination: String },
    |                     ^^^^^^ method cannot be called on `&String` due to unsatisfied trait bounds
    |
   ::: /path to/rustlib/src/rust/library/alloc/src/string.rs:279:1
    |
279 | pub struct String {
    | -----------------
    | |
    | doesn't satisfy `String: AsDynError`
    | doesn't satisfy `String: std::error::Error`
    |
    = note: the following trait bounds were not satisfied:
            `String: std::error::Error`
            which is required by `String: AsDynError`
            `&String: std::error::Error`
            which is required by `&String: AsDynError`
            `str: Sized`
            which is required by `str: AsDynError`
            `str: std::error::Error`
            which is required by `str: AsDynError`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0599`.
error: could not compile `thiserror-test`

To learn more, run the command again with --verbose.

But if I rename the source field to src, E0599 disappears this time. It looks so weird. Am I doing something wrong? Or something missing?
TFTHAOT
Jason

My bad.

The Error trait's source() method is implemented to return whichever field
has a #[source] attribute or is named source, if any. This is for
identifying the underlying lower level error that caused your error.
The #[from] attribute always implies that the same field is #[source], so
you don't ever need to specify both attributes.
Any error type that implements std::error::Error or dereferences to dyn std::error::Error will work as a source.
#[derive(Error, Debug)]
pub struct MyError {
    msg: String,
    #[source]  // optional if field name is `source`
    source: anyhow::Error,
}

repo: dtolnay/thiserror

license_source: bigquery


text:
Refactor x/dex to avoid recalculating price on each iteration.

…er than recalculating it whenever needed.

I am not a huge fan of moving functions between files. I find it very helpful to be able to trace how a function has changed over time -- this is much harder to do if it has been moved around from different files. There is a time for reorganization, but I think the bar needs to be pretty high.

I have a different philosophy, which is to not shy away from reorganization when necessary.

source: gharchive/pull-request

repo: duality-labs/duality


text:
Nothing happens

This is my Code, but when i Push the button, nothing happens.
{ ������������"accessory": "GPIODevice", ������������"type": "MotionSensor", ������������"name": "Hall Motion", ������������"pin": 27 ��������}
I use a contact button on gpio 27 Pin 13 with doorpi and homebridge-contactsensor and it works with an raspi3. But i need a Motion sensor.
Can you help me?

Hi,
For physical pin 13 (GPIO 2, BCM 27), you have to use the pin number "2" in your config file (see "Pin configuration" in the README).
Let me know if it solve.

thanks a lot.. now it works.. but only one time... i push the button and it hangs on triggered... i use the occupancy option and it does not go back to untriggered.

You have to push then release the button. Motion sensor will hang on triggered then hang off when button is released. Occupancy sensor will hang on triggered then hang off after specified "timeout" (60 sec if not specified).

thank you.. it works, great work!

authors: dubocr sohnkarlsruhe

repo: dubocr/homebridge-gpio-device


text:
Data Requirements

Hi, thanks for sharing.
Could you please, provide some info on the data requirements and its format. apparently, the ADEchallenge data is no longer available.
Thanks in advance.
Best

Uh, that sucks. I'll see if I can upload it somewhere (but it'll take sometime given my upload speed).
Anyway, the general image formats are simple:

images: RGB/BGR image (for grayscale you have to change the 3 in fcn.py to a 1)
annotations: grayscale image with values in [0, number_of_classes) (so you're limited to 255 classes currently, but it's possibile to surpass this limit with some tinkering)

Note: the annotations are not required.
Let's say you have a custom dataset with some external reference describing the polygons of the objects (for example a json or mat file). In that case while creating your own subclass of dataset_reader.py you could pass that structure to your class' constructor and generate the annotated images at runtime overriding _get_image accordingly.
Anyway with the ADE dataset the annotations were already available as images and after downloading the files were rearranged as
| ADEChallengeData2016/
    | train/
        | images/
        | annotations/
    | val/
        | images/
        | annotations/

Note: each image in X/images/ had a corresponding annotated image in X/annotations/ with the same name. Only the extension was different (jpg for the image, png for the annotations)
ade_dataset.py takes one of the directories containing images/ and annotations/ as the first argument (ade_dir).
If you want to specify a subset you can pass a list of filenames  as fnames (no file extension) that can be found in both images/ and annotations/
I hope this helps.

authors: dubvulture sevamoo

license: mit


text:
Describes the current fact that the read_csv function and the COPY statement support different parameters.

Describes the current fact that the read_csv function and the COPY statement support different parameters.

Thanks, this is a good start but the new phrasing is quite vague. Which options did you encounter that are not supported by COPY?

Oh, good question!  I have noticed the following differences:

COPY statement accepts "DELIMITER" and "DELIM", while the read_csv function only accepts "DELIM".
The read_csv function accepts "ALL_VARCHAR", while the COPY statement doesn't support it.

authors: michaelpopes szarnyasg

repo: duckdb/duckdb-web

license: MIT


text:
Core dumped when selecting all after loading csv

Core dumped when using this statement:
CREATE TABLE origin As SELECT * FROM read_csv('/script/1.csv', delim=',', header=true, columns={'id': 'Integer', 'geom': 'geometry'});
SELECT * FROM origin;
-- terminate called after throwing an instance of 'duckdb::SerializationException'
--  what():  Serialization Error: Trying to read past end of buffer
-- Aborted (core dumped)

csv file:
id,geom
0,"POLYGON EMPTY"

Version information:
v0.9.2-dev209 6545a55cfe

Hi! Thanks for filing this issue. This is reproducible on 0.9.1 by just doing SELECT 'POINT(1 2)'::GEOMETRY;.
The problem is that since GEOMETRY is technically a BLOB and VARCHAR can always cast to BLOB by just reinterpreting the bytes, you end up with a invalid geometry blob which then can't be deserialized when you try to print the result to the CLI.
What we should do instead is to implement an explicit VARCHAR->GEOMETRY that either errors or does the ST_GeomFromText conversion internally to avoid accidentally converting VARCHAR->BLOB->GEOMETRY
The crash is caused by exceptions that are thrown inside extension not being catchable by DuckDB due to the dynamically loaded extensions and duckdb core having different run-time type information, which some libc++ implementations use to identify exception types. This is a known long standing issue, see e.g. https://github.com/duckdb/duckdb/issues/6521

Thanks for clarifying the usage! I will update my script.^^

authors: Maxxen cuteDen-ECNU


text:
Update icons for apptp and netp notifs

Task/Issue URL: https://app.asana.com/0/488551667048375/1204905277193249/f
Description
Update notification icons used in AppTP and NP
Steps to test this PR
UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3414   👈

This comment was auto-generated by Graphite.

repo: duckduckgo/Android


text:
FFS: Fix old bookmarks migration

Task/Issue URL: https://app.asana.com/0/0/1204733967627383/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3704

PR #3705

PR #3721   👈
PR #3707

PR #3710

This comment was auto-generated by Graphite.

@malmstein can you add test cases to the description? Thanks!
Conflicts: You will have conflicts because FFS migration will only happen to internal users.

@cmonfortep there are test cases added to the migration tests. What do you not find clear?

Replaced witgh https://github.com/duckduckgo/Android/pull/3846

authors: cmonfortep malmstein


text:
Feature/david/remove sticky search deprecated jobs

Task/Issue URL: https://app.asana.com/0/1157893581871903/1171986548569356
Description:
As part of the remove of the Sticky Search Experiment we also want to make sure that if there was a job already scheduled it's properly removed from the WorkManager
This PR adds that functionality, making it easier in the future to remove old jobs.
Steps to test this PR:

Install current Production apk
Create a release version of this branch and update
Open the app
Use a db viewer to check that the old TAG CONTINUOUS_APP_USE_REQUEST_TAG is no longer present in the WorkManager

@malmstein, can you advise where in the DB to look for the tag name? I can't find it, even when inspecting the DB on develop so I can't verify it's gone on this branch. thanks

authors: CDRussell malmstein


text:
Advance content-scope-scripts forward

Task/Issue URL: https://app.asana.com/0/569473074191537/1201894298881763/f
Tech Design URL:
CC:
Description:
Advances content scope scripts forward which isolates globals from the page when we don't intend to modify them.

This repo now has a test to prevent this happening again.

Steps to test this PR:

Open https://good.third-party.site/security/js-leaks.html
Click Safari
Click check

Check for globals that look defined by us (the test gets outdated per Safari release: but previously we had 'init' and 'contentScopeFeatures' leaked)

OS Testing:

[ ] iOS 13
[ ] iOS 14
[ ] iOS 15
[ ] macOS 10.15
[ ] macOS 11

Internal references:
Software Engineering Expectations
Technical Design Template

Confirmed that this fixes the Doodle issue. The test is reporting two other globals being leaked:

window.FB - make a task for this: https://app.asana.com/0/1199178362774117/1201903875992988/f
navigator.duckduckgo.isDuckDuckGo - I'm not sure what this is exactly. Is this supposed to be accessible to webpages?

Confirmed that this fixes the Doodle issue. The test is reporting two other globals being leaked:
1. `window.FB` - make a task for this: https://app.asana.com/0/1199178362774117/1201903875992988/f

2. `navigator.duckduckgo.isDuckDuckGo` - I'm not sure what this is exactly. Is this supposed to be accessible to webpages?

Yeah the latter is expected.

I have attached the result of my test, please, could you confirm this is what we expect?

Thanks for asking, that's the expected set.

The window.FB is a known issue: https://app.asana.com/0/search/1201941626575164/1201903875992988
We purposefully add navigator.globalPrivacyControl and navigator.duckduckgo
The rest are differences since the last snapshot of these values were taken (they'll get out of date almost every release unfortunately, raised https://app.asana.com/0/892838074342800/1201941626575165/f to track fixing this and having an automated test)

authors: bstandaert-ddg jonathanKingston

repo: duckduckgo/BrowserServicesKit


text:
Issue when Removing Widgets

I have this JSFiddle Here:
http://jsfiddle.net/558wsrtb/1/
I have created a gridster to read data from my database (but in this JS Fiddle i have hardcoded the JSON).
I am trying to create functionality to remove a widget, but I occur a bug here and there when I try to remove certain widgets when I am not removing in order.
For example:
I can remove Store Chart3, CTC Chart2, Web Chart1, and Store Chart0 in this order successfully.
but then I try to remove Store Chart0, CTC Chart2.. it removes Store Chart 2.. and then if i continue clicking to remove CTC Chart2 nothing happens, but when I click Remove Web Chart 1, CTC Chart2 dissapears..
Do i have an issue with my index values or something or is this a bug?
The error from console.log: Uncaught TypeError: Cannot set property 'width' of undefined
Thanks!

Just change this lines
function rem(idrem)
{
	var rem = grid.remove_widget($(".layout_block").eq(idrem), true);
}

to this lines
function rem(idrem)
{
    var rem = grid.remove_widget($("#" + idrem), true);
}

and it will work!

Try this: http://jsfiddle.net/b0ndt/e1Lga24w/2/
Instead of working with ids, work with elements and paths. Note the changed lines:
linkToRemove = '<button onclick="rem(this);">REMOVE</button>';

and
var rem = grid.remove_widget($(idrem).parents(".gs-w"), true);

authors: ap0091 bondt dtamm


text:
🛑 epp server is down

In c8f1a3e, epp server (http://139.99.46.78) was down:

HTTP code: 0
Response time: 0 ms

Resolved: epp server is back up in 895c421.

repo: ducpw/status


text:
[WPPCONNECT] When no token and secret key is wrong, it fails

give it a better message.

fixed in https://github.com/dudanogueira/rocket.connect/commit/705075557e6d05d8778cdf6fc77f9d3ef436142a


text:
recylerview自动播放

运行recylerview自动播放demo，发现最后一个不会自动播放，只能手动点击

谢谢啊

repo: dueeeke/dkplayer


text:
Import stub template failed to serialize

Hi Duke of Haren,
I have created a couple of templates and the don't work any more.
Even the example in your Readme.MD doesn't work.
Reproduce steps:

Start latest docker container of dukeofharen/httplaceholder:latest
Create stub file with template

  conditions:
    method: GET
    url:
      path:
        equals: /users
      query:
        id:
          equals: 12
        filter:
          equals: first_name
  response:
    statusCode: 200
    text: |
      {
        "first_name": "John"
      }
    headers:
    Content-Type: application/json

Open httplaceholder application and import.
It will file at the line 'equals: /users'.

If I do the same template with dukeofharen/httplaceholder:2022.6.11.754 it works fine.
It looks like every sub element after path failed to parse.

@mvanleijenhorst, thanks for your bug report! I tried to reproduce your issue, and (sadly) succeeded in doing it. What I saw, however, is that, because I already had the "latest" version of the HttPlaceholder Docker container on my machine, it uses that version instead of checking if there is a newer "latest" version available.

The "latest" tag currently points to version "2022.8.7.27". If I put tag "2022.8.7.27" directly in the Docker Compose file, the importing works like a charm. Also, if I delete the image locally by executing sudo docker image rm dukeofharen/httplaceholder:latest, it will work again afterwards.
So, I'm curious if you use a version explicitly or delete the image from your machine, if it works again for you. Besides this, I guess it's not a good idea to use the latest tag if you're deploying any Docker, so perhaps I need to update the documentation to reflect this.

@dukeofharen, thanks for the quick response and sorry for my very late reply.
I use the latest version instead of a specific version. I removed the latest image and pulled it again. And it works like a charm.
The tag latest is indeed a little tricky.

@mvanleijenhorst yes, no problem. Good luck!

authors: dukeofharen mvanleijenhorst

repo: dukeofharen/httplaceholder


text:
Get top large files

I have a scenario where one of my Kubernetes nodes is experiencing disk pressure. Due to my use of the Bottlerocket OS, it's somewhat challenging to run utilities that are not containerized.
I tried to use the gdu docker image and get the files that cause the disk pressure, unfortunately without any success.
It would be amazing if I could run a k8s job like this and get the top X large files
apiVersion: batch/v1
kind: Job
metadata:
  name: disk-space-checker
  labels:
    app: disk-space-checker
    release: disk-space-checker
spec:
  backoffLimit: 0
  parallelism: 1
  ttlSecondsAfterFinished: 3600 # keep job for 1 hour
  activeDeadlineSeconds: 900 # timeout after 15 min
  template:
    metadata:
      labels:
        app: disk-space-checker
    spec:
      restartPolicy: Never
      containers:
      - name: disk-space-checker
        image: ghcr.io/dundee/gdu:v5.28.0
        imagePullPolicy: IfNotPresent
        securityContext:
          runAsUser: 0
        resources:
          requests:
            cpu: 250m
        args: ["--non-interactive","--show-apparent-size","--show-item-count","--max-cores","4","--no-delete","--ignore-dirs","/proc,/dev,/sys,/run,/local,/mnt,/etc,/boot","--no-cross","/host"]
        volumeMounts:
        - name: host
          mountPath: "/host"
          readOnly: true
      volumes:
        - name: host
          hostPath:
            path: "/"

With this k8s job I gets only root directories without getting sub directories and files.
Any chance to have additional parameter that can sort top X files by size?

This sounds like a valid use case. I will try to add it to the next release.

authors: devopsmash dundee

repo: dundee/gdu


text:
New filter API

Easier way to define multiple filters
Possibility to enable filter on all fields of an entity

Finally I've abandoned the idea of using LoopBack style URLs because they are too ugly.
As page and itemsPerPage parameters are now configurable, conflicts can be easily avoided (ex: using an URLs like ?_page=2&_itemsPerPage=3).
@theofidry @sroze What do you think about that? And what do you think about forwarding $request to DataProvider and Filter class? It's not very clean but it's powerful and useful when creating custom filters.

I don't mind, but we should be able to easily switch to a StrongLoop like system for API with lots of filters and options. I don't mean an option to do so, but maybe a snippet in the doc about how to do it in a clean way.
About the Request in the Filter class, I'm for it since it would let the user complete control of how to implement the filter and retrieve it. For the DataProvider, I don't see the use at the moment but I guess it can be quite helpful.

@theofidry it is possible thanks to the injection of Request by creating a custom filter. A cookbook entry about that would be nice to have.

That's definitively a better implementation than before, good job !

authors: dunglas sroze theofidry

repo: dunglas/DunglasApiBundle


text:
DUPLO-21150: TF Doc: Update doc to include samples for Redis logging configuration

User description
Overview
DUPLO-21150: TF Doc: Update doc to include samples for Redis logging configuration
Summary of changes
This PR does the following:

add sample to templates
...

Testing performed

[ ] Using unit tests
[X] Manually, on my local system
[X] Manually, on a remote test system

Describe any breaking changes

None

PR Type
Documentation, Enhancement

Description

Added new documentation examples for creating an Amazon ElastiCache of type Redis with log delivery configuration and automatic failover enabled.
Updated the Go schema description for destination_type by removing the reference link.
Enhanced the documentation by providing detailed examples for log delivery configuration in both Markdown and template files.

Changes walkthrough 📝
Relevant filesEnhancement

resource_duplo_ecache_instance.goUpdate schema description for destination type

duplocloud/resource_duplo_ecache_instance.go
Updated the description for destination_type in the schema.  Removed the reference link from the description.

+3/-4

Documentation

ecache_instance.mdAdd Redis log delivery configuration example

docs/resources/ecache_instance.md
Added a new example for Redis with log delivery configuration.  Removed reference link from the required section.

+35/-1

ecache_instance.md.tmplAdd Redis log delivery configuration example template

templates/resources/ecache_instance.md.tmpl

Added a new example for Redis with log delivery configuration.

+35/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

Task linked: DUPLO-21150 TF Doc: Update doc to include samples for Redis logging configuration

authors: pravin-duplocloud zafarabbas


text:
🛑 Mention Robot is down

In 4d7d34f, Mention Robot (https://backend.isbotdown.com/bots/mentionrobot) was down:

HTTP code: 200
Response time: 119 ms

Resolved: Mention Robot is back up in 1d7fbc3.

repo: durof/status


text:
🛑 Mention Robot is down

In eb82e5b, Mention Robot (https://backend.isbotdown.com/bots/mentionrobot) was down:

HTTP code: 200
Response time: 155 ms

Resolved: Mention Robot is back up in d6e40bf.


text:
🛑 Mention Bot is down

In 05fc975, Mention Bot (https://backend.isbotdown.com/bots/mentionbot) was down:

HTTP code: 200
Response time: 119 ms

Resolved: Mention Bot is back up in 1c161ce after 1 hour, 8 minutes.


text:
🛑 Mention Test Bot is down

In ed32a77, Mention Test Bot (https://backend.isbotdown.com/bots/mentiontestbot) was down:

HTTP code: 200
Response time: 153 ms

Resolved: Mention Test Bot is back up in 10f86c6 after 1 hour, 47 minutes.


text:
🛑 Mention Bot is down

In ef58033, Mention Bot (https://backend.isbotdown.com/bots/mentionbot) was down:

HTTP code: 200
Response time: 90 ms

Resolved: Mention Bot is back up in 01075db after 3 hours, 23 minutes.


text:
账户密码连接

需要账户密码连接的集群怎么设置

尴尬

authors: shaohongwu xxccll

repo: dushixiang/kafka-map


text:
New admin pages improvements

As discussed IRL with @Duncid

changing user role: avoid reloading page on success
confirmation modal (regular not side or full) for revoking access / revoking  invitation (screenshot below)
Email invite with bar header with custom save button "Invite" (see screenshot)
use empty avatars for pending invitations

@PopDaph please review post merge :bow:
I merge now to fix user revocation

repo: dust-tt/dust


text:
enh(notion conn): add support for forumula properties

Description
We're currently ignoring those properties. This commit adds support for them.
Risk

Deploy Plan

PMRW


text:
Change the Size (Height - width) of the AD Image

Hello Duyduong,
thank you very much for this package. it helps a lot.
if you could help with this request, it will be great.
is it possible to change the size of the Small image of the AD  ?

because, if you disable the showMediaContent the ad doesn't look intuitive.
one great feature will be, if we could have the possibility to change the position of the ad component ourself. as if the whole AD is a Stack and we can position (for example) "callToAction" to the bottom-left , MediaContent to the right ...etc

@korchix Hey, new version is released. New property NativeAdmobType that allows u to do that

@duyduong first thank you very much for the effort and your time.
the new NativeTextStyle property isVisible is very helpful, thank you for it
the other  Property NativeAdmobType is nice, but you still can't change the Size/Position of any Property.
here is the NativeAdmobType.banner of my app :

as you can see, the icon still small, and the space between Properties could be better used, nice will be if we can move them to get something like this :

or like this image, where the AD-Image and CallToAction Button take the whole height of the row

i don't know if what i'm saying is even Possible, hence the google team couldn't provide a better solution to Native Ads as you did. but if you can make it possible, that will be awesome.

@korchix cool, you can list down all the useful layouts, the I can support them in the plugin

@duyduong that sounds great. thank you a lot.
i will try to share some different layouts, that might be more engaging and intuitiv.

Hi @duyduong
here are two examples of NativeAdmobType.**full** in Landscape orientation :

and here some example of portrait orientation

and here are some examples of Banner Ads :

one more example of full ads:

@korchix cool, change to enhanced

Personally, I just want to be able to manipulate the image myself, so I can use different sizes and different BoxFits

This is very out of place right now. If I could have control over the ad icon, the info icon on the top right, the image height/width and its BoxFit, etc... that would be AMAZING

Also it seems the image doesn't always load?

Not sure if some ads just dont have images, but a placeholder that we can specify would be great.

@duyduong any update on this? Would be cool if we could position ad elements in the widget and change the size of the image/crop it/round corners. Thank you for the plugin!

@duyduong - any update on the feature will be greatly appreciated, thanks!

its been 9 months, is this going to be a feature or not?

I was done waiting so some days ago I started building my own library. https://github.com/bdlukaa/native_admob_flutter
It's android only for now (don't have an apple env).
It allows you to manipulate almost everything :)

Hello @bdlukaa,
Just for curiosity, why you didn't fork this repository, perform the changes you want, and make a PR?

@comlaterra because my package has a total different dynamic from this one.

@bdlukaa I'm new to flutter (i started learning it like 1 month ago) and I really liked how customizable your package is, I don't know much about flutter but I can help with apple env and test your package on it if you need help with it.

@bdlukaa I'm new to flutter (i started learning it like 1 month ago) and I really liked how customizable your package is, I don't know much about flutter but I can help with apple env and test your package on it if you need help with it.

@imcouri You can create a pull request with the implementation for it (guide), It'd be awesome :). I'll be implementing the web part next (using adsense). Maybe tomorrow

@imcouri You can create a pull request with the implementation for it (guide), It'd be awesome :). I'll be implementing the web part next (using adsense). Maybe tomorrow

authors: SwissCheese5 angel1st bdlukaa comlaterra duyduong imcouri korchix tvajjhala


text:
Température cible : Ne semble pas toujours ce mettre à jour au niveau des thermostats

A plusieurs reprise j'ai constaté que mes automatisations ne semble pas fonctionné quand il vient le temps de modifier la température des thermostats.
J'ai 2 automatisations de programmer dans la période d'appréciation pour augmenter la température mais très souvent aucune ne semble fonctionné.
Les thermostats commence a chauffer mais la température cible elle ne change pas.
Voici une de mes automatisation qui c'est bien déclencher à l'heure souhaité mais les thermostats n'ont pas changé de température cible.
alias: Hilo - Appréciation
description: ''
trigger:

platform: state
entity_id:

sensor.defi_hilo
to: appreciation
for:
hours: 0
minutes: 15
seconds: 0
condition: []
action:

service: climate.set_temperature
data:
temperature: 21
hvac_mode: heat
target:
area_id:
- chambre_emmie
- chambre_lohik
- bedroom
- kitchen
- entrainement
- passage
- salle_de_bain
- salle_a_manger
- salon
mode: single

On voit dans cette images que le chauffage ce déclenche mais la ligne de la température cible elle ne fait rien alors qu'au moment du préchauffage par Hilo la température augmente.

Version of the custom_component
2023.1.1
Debug log
Seul log qui semble correspondre environ au heure des automatisation
Refreshing websocket token https://apim.hiloenergie.com/GDService/v1/api/Locations/6980/Events/184
02:16:21 – (AVERTISSEMENT) Hilo (intégration personnalisée) - message survenu pour la première fois à 02:16:20 et apparu 2 fois.
Refreshing websocket token https://apim.hiloenergie.com/GDService/v1/api/Locations/6980/Events?active=true
01:16:19 – (AVERTISSEMENT) Hilo (intégration personnalisée)

Je vais essayer d'activé du debug supplémentaire. Sa me donne l'impression qu'il envoi trop de demande de changement d'un même coup étant donnée que tout les thermostat sont réajuster et que c'est de là que sa accroche.
Ce que je trouve étrange c'est qu'il semble activé le chauffage mais sans changé la consigne.

Les thermostats sont toujours en mode "heat", essaye de retirer
hvac_mode: heat
Ça m'a causé des problèmes par le passé sur d'autre type d'appareil. Passe juste ta commande de température.

Je crois que c'est un bug intemittent avec Hilo. J'ai réussi à reproduire récemment et pourtant, la bonne commande était envoyé à Hilo.

Je crois que ca devrait etre regled avec e3815885f2842a5471e334c57c36aa411d443e9e

repo: dvd-dev/hilo


text:
Ajustement tarifaire

Un ajustement tarifaire moyen de 2,6 %, établi en fonction de l'inflation conformément à la Loi visant à simplifier le processus d'établissement des tarifs de distribution d'électricité, est entré en vigueur le 1er avril dernier.   Ref: https://www.hydroquebec.com/data/documents-donnees/pdf/tarifs-electricite.pdf?v=20210517

pas moyen de refuser cette hausse right? :-)

pas moyen de refuser cette hausse right? :-)

hehe I wish..!  ;)

@valleedelisle  hey je ne comprends pas je n'arrive pas à avoir les nouveaux tarifs Hilo même en redémarrant, en reloadant l'intégration, même en la supprimant, et aussi en supprimant les entités 'rate' etc.. des idées.. !?
Ca revient toujours avec l'ancien tarif ...

authors: FrancoLoco valleedelisle


text:
Add end-to-end tests

Adds an end-to-end test script partially solving Issue #4. Rather than testing functionality on a unit level, it deploys several blockchain nodes in order to test if they function as expected in a real environment.
The script provides several tasks that can be run individually or in sequence. It is also easy to add more tasks if additional functionality is to be tested.
Instead of adding the script to this repository, I created a new repository with the idea that the script could also be applied to other blockchain implementations than this one.

@floscha Sorry but I don't see any tests here. Did you forget to commit them?

Don't see anything here...

Like explained in the last paragraph, this PR does not add the code for the tests but rather an explanation to the README.md on how to run the tests which exist in an external repository (https://github.com/floscha/blockchain-end2end-test). The motivation behind this is to not tightly couple the tests to precisely this Blockchain implementation but rather make the reusable for other implementations as well.

authors: dvf floscha

repo: dvf/blockchain


text:
Return of the Dirty Vagrant, Part III

It seems that using the vagrant box from drmyersii , now drm2, has certain repercussions. Either this is not the box belonging to the original author, or it is, and eclipse on said box is broken (which it is). Java needs to be updated to Java 8 and the eclipse release (broken - icon missing, won't start) needs to be removed and the C/C++ edition installed. After completing this, I can finally work on an app of my own. I'm overjoyed, even though a bit of debugging and witch-hunting (I kid) was involved.

This isn't a very well structured issue. The solutions I provide are patches (could temporarily be included in readme). I am unsure how to produce a vagrant box that would solve this issue, so at this time I haven't. I would do this if I knew how, but that is for another day. @dvhdr, you could either wait until then to update the link in your vagrantfile, or accept my pull request, update your readme, and complete that later. Thanks for the work you've done on this - It's beyond me.

Hello.... I'm about to setup the environment now. Your tips will help me a bit.
I'm downloading the drm2 box now.

If you have any issues, message me :3

I was going to post a tl;dr, but I forgot how I had it set up as I am away from my computer for the next few hours.

I think the installation of vagrant and other UBUNTU OS stuffs are redundant for the compiling of the codes. I am installing a Windows X64 eclipse. I will let you know of the test result.

@granadaxronos - you don't even need Eclipse, all you need is the gnu-arm-eabi tools.

I wouldn't agree with the latter comment of @granadaxronos; this repository, even in the anomalies of its approach, is very useful. I have had a very non-traditional introduction to development as a whole and am excited to possibly begin attending University next year, which will allow me to grow my theoretical knowledge. Besides that, for someone such as myself with a lack of knowledge in terminology, the current ease of setup for this project is wonderful (at least I can say that now that I've figured it out).

I dunno @oasis9 , he's right in the sense that I am indeed a complete nut :)

naww
no need for that 😋 this was definitely fun while it lasted, though I've become so tied up in other work that I don't have time to get it all set back up after reinstalling Windows on my shiny new SSD and not to mention that bastard 2 minute upload 😢

Yeah the two minute upload does make debugging pretty horrible. Did you try the simulator on macOS? I think the creator of Subsequencely made a really nice desktop one.

authors: dvhdr granadaxronos oasis9

repo: dvhdr/launchpad-pro


text:
"List participants cannot be empty."

On first start of the agent, I see this issue in the logs:
shutdown on port: 8000.
[2024-12-06 13:41:57,466] [INFO] [agent] HTTP Server has shutdown on port: 8833.
[2024-12-06 13:41:57,466][INFO] [agent] HTTP Server has shutdown on port: 8833.
Stopping AEA 'agent' ...
AEA 'agent' stopped.
Error: An error occurred while setting up item dvilela/memeooorr_chained_abci:0.1.0/SharedState:
Traceback (most recent call last):

  File "/root/agent/vendor/valory/skills/abstract_round_abci/base.py", line 862, in all_participants
    raise ValueError("List participants cannot be empty.")

ValueError: List participants cannot be empty.

This stopped the service, but I restarted and it's now running ok.

repo: dvilelaf/meme-ooorr


text:
docs: Add installation instructions for Fedora based systems

I created a copr repo (similiar to Ubuntu's PPA) for hstr. Available for Fedora 20-22, RHEL/CentOS/Scientific Linux 6-7.

Cool! Appreciate it - thanks!

BTW, is there a way to automate release process through this repo?

Yeah, I guess it would be possible. Both Copr and Github do have APIs, but I guess it would be too much work given that manual changes still need to me made :) For proper package maintainence, it's a good guideline to keep updated info & changelog in the package, which means manually editing the spec file anyway.
If you're worried about packages in the wild falling behind the releases, worry not. I watch this repo, so it should not happen. I guess the best way to prevent this would be tagging me in the "Release version X" github issue after the release has been done.

I guess the best way to prevent this would be tagging me in the "Release version X" github issue after the release has been done.

Great! This is exactly what I meant ;)

authors: dvorka tbabej

repo: dvorka/hstr


text:
🛑 Puffer is down

In f14ddcc, Puffer (https://api.pufferapp.com/healthz) was down:

HTTP code: 503
Response time: 1029 ms

Resolved: Puffer is back up in 4163360.

Resolved: Puffer is back up in 4163360.

repo: dwarvesf/upptime


text:
CliqueEmbeddingComposite

Add a clique embedding composite. Would incorporate https://github.com/dwavesystems/dwave-system/issues/300#issuecomment-638401351 and https://github.com/dwavesystems/dwave-system/issues/228

Would also need to be "topology aware"

Another feature this could support is automorphisms on the problem graph. Something like
sampler = CliqueEmbeddingComposite(DWaveSampler())
ss = sampler.sample(bqm, num_automorphisms=3, num_reads=12)

which would run 4 reads for each automorphism of the problem variables.

repo: dwavesystems/dwave-system


text:
Versi ARM

Ditambah rilis ke ARM dong om :v:

ARM OS apa dulu nih?

Raspbian juga boleh :v

Linux armv7 dan armv8 (arm64) bos. Please

authors: dwisiswant0 felangga sonyvaiopocket


text:
Proper modal cleanup with Bootstrap

I looked through the examples to see if there was already a documented way of properly cleaning up a modal that is dismissed when clicking outside the modal area. I didn't see anything and noticed there were many ng-click directives used to catch the modal closing.
Instead of attaching ng-click directives to all the elements and faking the animation time for the modal to disappear I used the events generated by bootstrap. I inserted this at the top of my controller to make sure the modal is properly cleaned up regardless of how it is closed and waits for the animation to finish. This handles clicking outside the modal to close, data-dismiss="modal", and $element.modal('hide').
$element.on('hidden.bs.modal', close)

I thought this might be useful documentation in the FAQ.

This looks awesome, much easier than having to message around with a timeout for animations (which would coincidentally fix #4 I think. I'll look into getting this into the samples and documentation ASAP. Thanks for sharing @cehoffman

Awesome solution, well done! :+1:

The easiest way (as of [0.12]9https://github.com/dwmkerr/angular-modal-service/releases/tag/v0.12.0) is to do this with a preClose hook:
ModalService.showModal({
  templateUrl: "some/bootstrap-template.html",
  controller: "SomeController",
  preClose: (modal) => { modal.element.modal('hide'); }
}).then(function(modal) {
  // etc
});

authors: cehoffman dwmkerr gaui

repo: dwmkerr/angular-modal-service


text:
nativebridge load changed to process type detection (instead of O/S)

fixed Issue #49

Makes perfect sense, the bitness of the OS doesn't matter of course, it's the bitness of the process.
One question, would a better solution be to use the property Environment.Is64BitProcess which makes it much more clear at a glance what we're checking for? If so, can you change the PR and I will merge it then?

Resolved with a40e66b.

authors: dwmkerr spustlik

repo: dwmkerr/sharpshell


text:
[727] Hiring staff can now copy expired vacancies

Trello card URL:
https://trello.com/c/m2eHnBMQ/727-hiring-staff-ui-user-can-make-a-copy-of-an-existing-job-on-the-expired-jobs-page-front-end
Changes in this PR:

Amended the expired vacancy factory to allow saving without validation
Added 'Copy' button to the expired vacancies view
Added new context for copying expired vacancies to feature spec

Screenshots of UI changes:
Before

After

 Hiring staff UI: user can make a copy of an existing job on the 'expired jobs' page (front end)

Yeah sure, I had the same concern yesterday and asked on the card: https://trello.com/c/m2eHnBMQ/727-hiring-staff-ui-user-can-make-a-copy-of-an-existing-job-on-the-expired-jobs-page-front-end#comment-5c644532036f2a878b54ea66
But I think you're right it will probably provide a better user experience not having to hit the validation error if we can

@tahb Added a commit that doesn't copy the expiry date on expired vacancies

repo: dxw/teacher-vacancy-service


text:
wallpaper doesn't change on kde plasma

without -n argument wallpaper doesn't change

diff --git a/pywal/wallpaper.py b/pywal/wallpaper.py
index 6a50627..2ba07ff 100644
--- a/pywal/wallpaper.py
+++ b/pywal/wallpaper.py
@@ -119,10 +119,13 @@ def set_desktop_wallpaper(desktop, img):

     elif "kde" in desktop:
         string = """
-            var allDesktops = desktops();for (i=0;i<allDesktops.length;i++){
-            d = allDesktops[i];d.wallpaperPlugin = "org.kde.image";
-            d.currentConfigGroup = Array("Wallpaper", "org.kde.image",
-            "General");d.writeConfig("Image", "%s")};
+            var Desktops = desktops();
+            for (i = 0; i < Desktops.length; i++) {
+                d = Desktops[i];
+                d.wallpaperPlugin = "org.kde.image";
+                d.currentConfigGroup = Array("Wallpaper", "org.kde.image", "General");
+                d.writeConfig("Image", "%s");
+            }
         """
         util.disown(["qdbus", "org.kde.plasmashell", "/PlasmaShell",
                      "org.kde.PlasmaShell.evaluateScript", string % img])

there is missing semicolon

repo: dylanaraps/pywal


text:
Termite support!

First time submitting a pull request so sorry if I broke any norms guys!
I added a template file for the termite terminal emulator. However, as its config does not have an include directive I added a snippet that merges the template into the config file if the user has pywal.conf in his ~/.config/termite. Happy to accept any criticism, and sorry again if this is not what is wanted here. Just wanted to help people who are attached to termite's vi bindings with customization when Termite is launched without running (cat ~/.cache/wal/sequences &) such as running htop from rofi.
Also, despite https://github.com/dylanaraps/pywal/issues/69, termite still prints junk characters when running (cat ~/.cache/wal/sequences &) on my pc, so this is the only way to get pywal to work for me.

Oh darn, didn't use snake case! Should modify this on master or check out a new branch and merge again? Or perhaps Squash the change?


text:
Sign out by barcode isn't working

Example string:
^03f45b4d-0c1c-4427-a448-f0a207b65de8;Andrew McAwesome;c0a7c62c-8c76-43f5-9357-e65394cca2cb;;6045555555;a@sfsar.ca;;;11000000;Joan;Mom;6045555555;2021-05-18 10:20:42 PM;0;^
Could be an issue with the barcode generated, or with the ICA app receiving it

Potential fix in version 1.1

Fixed in version 1.1

repo: dylancbaker/MySARAssist


text:
Bidi & complex text layout/shaping

From README.md:

Notable things that RustType does not support yet:
[…]

Right-to-left and vertical text layout.

With right-to-left you’ll probably want to also support bidirectional text. There’s a rust implementation of the bidi algorithm at https://github.com/servo/unicode-bidi
However, maybe this shouldn’t be part of RustType. Maybe RustType should ideally be one component in a larger "text rendering in rust" ecosystem. For example, complex text shaping is hard and I’m very skeptical of re-implementing HarfBuzz in Rust, I’d rather have some way to use HarfBuzz together with RustType.
(In case you’re not familiar with HarfBuzz, http://behdad.org/text/ explains where it sits in the stack overall and http://mces.blogspot.in/2009/11/pango-vs-harfbuzz.html how it compares to Pango, although these documents is are few years old already.)

With RustType I currently want to cover the most common use cases. Hence text with complicated layout is not a high priority. Having said that, I think it should be possible to cover a small-but-useful subset of this in pure Rust, without relying on HarfBuzz. This can hopefully exist as a separate crate. The Font::layout function as it is is just a convenience function that uses the public API to retrieve and position glyphs. An external layout crate can hopefully do the same.
I was kind-of familiar with HarfBuzz, but that http://behdad.org/text/ document really helps cement the larger picture in my mind. Thanks.
This issue can probably stay open until complex text layout is dealt with one way or another.

I've been working on an layout engine (in Elixir versus Rust) so here are the main points for "common use cases" that can later be extended to more complex layout. Hopefully this can help define a reasonably stable API that enables incremental development.
There are five basic steps:

Normalize the string (NFC for instance). This is recommended but optional and can be punted to the caller (ie, assume they've already normalized the string and document the assumption).
Use the font's CMAP to convert the codepoints into glyphs.
Select OpenType features. Simplest implementation is to have an API that lets the client select them with a reasonable default (usually kerning and standard ligatures are a good-enough default for a first implementation).
Do glyph substitution. (Fallback in the absence of a GSUB table is to do nothing). This enables ligatures, compound emoji, combining marks, and optional OpenType features like smallcaps, swashes, fractions, and so on.
Do glyph positioning. (Fallback in the absence of a GPOS table is to use a kern table; if no kern table just the horizontal metrics). This handles kerning, optical bounds alignment, cursive joining, positioning of diacritic marks, etc.

If you can do these four steps, you're easily at the 80/20 point for most uses.
I'll also point out that while both the GSUB and GPOS tables have lots of lookup types, you can get away with incrementally implementing them -- kerning is usually GPOS type 2 and ligatures are usually GSUB type 4. For those not yet implemented simply leave the glyphs and positions alone (and probably log something so you can prioritize the next one to implement). A wide number of interesting English-language OpenType features are commonly implemented as GSUB type 1.
Regardless of how complex your script is, your renderer is going to end up with a glyph, and a position that involves an offset (from the current cursor position) and an advance (to move the cursor). In the simplest cases the offset is 0,0 and the advance is glyph_width, 0.  In English an accent mark will typically have an offset to move it over the preceding character and an advance of 0, 0 since it doesn't actually move the cursor.
Extending for more complex layout:
Step 1 can be extended to determine the script and language; you can use a Unicode package to detect these or you can just make the client figure it out and pass the information in. Very reasonable defaults available if information not available.
Step 2 can be extended to handle missing glyphs by locating a fallback font.
Step 3 can be extended to select a shaping engine, which is responsible for reordering codepoints, identifying positional variants, direction of text, and so forth. Your default shaping engine would do nothing here, while a shaping engine for most RTL languages would look up join type information in the Unicode standard to do positional shaping (letters use different glyphs depending on the location within a word).
Steps 4 and 5 pretty much remain the same though you might extend them to handle other layout technologies beyond OpenType -- almost all of which involve less common font tables. Most likely handing GVAR (variable fonts) tops the list of such alternatives.

authors: SimonSapin dylanede jbowtie

repo: dylanede/rusttype


text:
[Snyk] Security upgrade @backstage/plugin-explore-common from 0.0.0-use.local to 0.0.1

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

plugins/explore-react/package.json

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

⚠ Warning
Failed to update the yarn.lock, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-JS-ROLLUP-8073097
  109

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

🎉 Snyk hasn't found any issues so far.
✅ code/snyk check is completed. No issues were found. (View Details)

repo: dylansnyk/backstage


text:
[Snyk] Security upgrade @backstage/core-plugin-api from 0.0.0-use.local to 0.1.0

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

plugins/kafka/package.json

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

⚠ Warning
Failed to update the yarn.lock, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Improper Input Validation SNYK-JS-NANOID-8492085
  44

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Improper Input Validation

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


text:
YQ error when calling ./deploy-helm.sh

Describe the bug
When running the ./deploy-helm.sh an error is thrown
To Reproduce
Steps to reproduce the behavior:

Go to 'Google Cloud shell'
Download git repo "dynatrace-oss/dynatrace-gcp-monitor"
Follow instructions and execute ./deploy-helm.sh
See error

$ ./deploy-helm.sh --create-autopilot-cluster

ERROR: yq (4.9.x+) is required to install Dynatrace function. Please refer to following links for installation instructions:
      YQ: https://github.com/mikefarah/yq
      Example command to install yq:
      sudo wget https://github.com/mikefarah/yq/releases/download/v4.9.8/yq_linux_amd64 -O /usr/bin/yq && sudo chmod +x /usr/bin/yq

user1@cloudshell:~/dynatrace-gcp-monitor/scripts (hl2-seme-mach-t1iylu)$ which yq
/usr/bin/yq
user1@cloudshell:~/dynatrace-gcp-monitor/scripts (hl2-seme-mach-t1iylu)$ yq --version
yq version 4.9.8

Expected behavior
./deploy-helm.sh --create-autopilot-cluster execute correctly and deploys the clustter
Screenshots
NB/A
Desktop (please complete the following information):

OS: Linux (GCP Cloud shell)

Additional context
Add any other context about the problem here.

Update. I commented out the test for YQ in the lib.sh and then hit the same error with JQ
$ ./deploy-helm.sh --create-autopilot-cluster

ERROR: jq is required to install Dynatrace function. Please refer to following links for installation instructions:
    JQ: https://stedolan.github.io/jq/download/"
    Example command to install jq:
    sudo wget https://github.com/stedolan/jq/releases/download/jq-1.6/jq-linux64 -O /usr/bin/jq && sudo chmod +x /usr/bin/jq

user1@cloudshell:~/dynatrace-gcp-monitor/scripts (hl2-seme-mach-t1iylu)$ which jq
/usr/bin/jq
user1@cloudshell:~/dynatrace-gcp-monitor/scripts (hl2-seme-mach-t1iylu)$ jq --version
jq-1.6

repo: dynatrace-oss/dynatrace-gcp-monitor


text:
구현체 설명

소스코드 내에서 주석을 적어서 sphinx문법으로 설명
그리고 모듈마다 주석을 적어서 설명할 것.

sphinx문법으로 안드로이드 스튜디오 내에 있는 코드를 설명할 수는 없다. period.

repo: dytlgozj/WebtoonAlarm


text:
Meta tweaks

Brings the repository closer to following Awesome guidelines. However, I would recommend going over each of the descriptions and polishing them up as well. Also, there are some guidelines that cover parts of the repository I can't touch like the keywords so be sure to keep those in check.
Related to: https://github.com/sindresorhus/awesome/pull/1931
Fixes: #97

@Richienb thank you for your effort. Based on this, I decided to make a change to contributing.md, which
explicitly disallows adding this list awesome-typescript to the awesome-awesome lists.
https://github.com/dzharii/awesome-typescript/commit/2b9a79c6b9b955b10ff7b18cfa8d4ed163dbee91
This new policy is effective immediately.
Future pull request regarding to this topic shall be closed as the ones violates the policy.
My apologies for this inconvenience, please don't take this personally. With respect to all awesome-awesome contributors and moderators, I am not going to do any changes to comfort with their standards.
I am not disallowing adding this repository to the awesome-awesome list, you are free to do this change on your side, but I am not going to accept any supporting chnage in this repo.
Best regards,
@dzharii

authors: Richienb dzharii

repo: dzharii/awesome-typescript

license: unlicense


text:
Push 0.2.2 to pypi

Could we push the latest version to Pypi please :)
Thanks!

Will do later today.

Done.

authors: SunDwarf kageurufu

license: WTFPL


text:
Update your install docs

This is for the depreciated project page.

Clone this project to your local environment, e.g. git clone https://github.com/ianmcburnie/mindpatterns.git

Done. Thanks for pointing this out!

authors: ianmcburnie mgifford

repo: eBay/mindpatterns


text:
[16078] Static EDP discovery core deployment tutorial

This PR depends on eProsima/ros2_documentation#13

I am not sure why the spell check is failing... I have run it locally and no error is shown. The CI job is not showing the file content so I do not know if it is something else.

repo: eProsima/vulcanexus


text:
Search box doesn't look good

Whoever set it up can fix it here: https://cse.google.com/cse/all
https://developers.google.com/custom-search/docs/element#customizing

Fixed by overriding the inheritance of box-sizing.

authors: alaninmcr fbacall


text:
Exception while opening any detailview from listview

XAF : 23.1.3
Xpand : 23.1.301
15.06.23 12:07:48.595	================================================================================
The error occurred:
Type:       ArgumentException
Message:    Object of type 'System.EventHandler`1[System.EventArgs]' cannot be converted to type 'System.EventHandler`1[DevExpress.ExpressApp.Win.Editors.LookupEditProviderEventArgs]'.
Data:       0 entries
Stack trace:

at System.RuntimeType.TryChangeType(Object value, Binder binder, CultureInfo culture, Boolean needsSpecialCast)
at System.RuntimeType.CheckValue(Object value, Binder binder, CultureInfo culture, BindingFlags invokeAttr)
at System.Reflection.MethodBase.CheckArguments(StackAllocedArguments& stackArgs, ReadOnlySpan1 parameters, Binder binder, BindingFlags invokeAttr, CultureInfo culture, Signature sig)    at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)    at Xpand.Extensions.Reactive.Transform.Transform.<>c__DisplayClass111_01.b__0(EventHandler1 handler)    at System.Reactive.Linq.ObservableImpl.ClassicEventProducer2.AddHandler(TDelegate handler) in //Rx.NET/Source/src/System.Reactive/Linq/Observable/FromEvent.cs:line 360
at System.Reactive.Linq.ObservableImpl.EventProducer`2.Session.AddHandler(TDelegate onNext) in //Rx.NET/Source/src/System.Reactive/Linq/Observable/FromEvent.cs:line 325
InnerException is null
================================================================================

the fix is on lab soon to be released, thnks

eXpand.lab release 23.1.301.1 includes commit that relate to this task:

Update rx packages #1002

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

is this fix ok?

yes , close #1002

eXpand.lab release 23.1.502.0 includes commit that relate to this task:

Update rx packages #1002

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

authors: apobekiaris vimarx

repo: eXpandFramework/eXpand

license: MS-PL


text:
duckdb, sequence function inside F.expr resolves to sequence function instead of generate_series

Hi,
I'm using the following code to create a sequence between start - end date, but within an expr() instead of F.sequence.
While the F.sequence correctly resolves to duckdb generate_series, F.expr("sequence") won't
import sqlframe.duckdb.functions as F
from sqlframe.duckdb import DuckDBSession

session = DuckDBSession()

df = session.createDataFrame(
    [("2024-01-01", "2024-05-05"), ("2024-05-21", "2024-12-05")],
    ["start_date", "end_date"],
).select(
    F.col("start_date").cast("date").alias("start_date"),
    F.col("end_date").cast("date").alias("end_date"),
)

df = df.withColumn(
    "date_sequence",
    F.expr(
        "sequence(start_date,end_date, interval 1 month)",
    ),
)

print(df.sql())
print(df.show())

will return
SELECT
  CAST(CAST("a1"."start_date" AS TEXT) AS DATE) AS "start_date",
  CAST(CAST("a1"."end_date" AS TEXT) AS DATE) AS "end_date",
  SEQUENCE(
    CAST(CAST("a1"."start_date" AS TEXT) AS DATE),
    CAST(CAST("a1"."end_date" AS TEXT) AS DATE),
    INTERVAL '1' MONTH
  ) AS "date_sequence"
FROM (VALUES
  ('2024-01-01', '2024-05-05'),
  ('2024-05-21', '2024-12-05')) AS "a1"("start_date", "end_date")
Traceback (most recent call last):
  File "/workspaces/playground.py", line 29, in <module>
    print(df.show())
  File "/workspaces/.venv/lib/python3.10/site-packages/sqlframe/base/dataframe.py", line 1576, in show
    result = self.session._fetch_rows(sql)
  File "/workspaces/.venv/lib/python3.10/site-packages/sqlframe/base/session.py", line 458, in _fetch_rows
    self._execute(sql, quote_identifiers=quote_identifiers)
  File "/workspaces/.venv/lib/python3.10/site-packages/sqlframe/base/session.py", line 430, in _execute
    self._cur.execute(self._to_sql(sql, quote_identifiers=quote_identifiers))
duckdb.duckdb.CatalogException: Catalog Error: Scalar Function with name sequence does not exist!
Did you mean "encode"?
LINE 1: ...37156 AS (SELECT start_date, end_date, SEQUENCE(start_date, end_date, INTERVAL...

The core of the issue is that SQLGlot doesn't understand that Spark's sequence maps to GenerateSeries:
>>> parse_one("SELECT sequence(start, end, interval 1 month)", dialect="spark").sql(dialect="duckdb")
"SELECT SEQUENCE(start, end, INTERVAL '1' MONTH)"

It should map to this: https://github.com/tobymao/sqlglot/blob/a8ce90ca8a8d78ffb4c87660a918fb6025de1800/sqlglot/expressions.py#L4813
If you would like to see that fixed then you would need to open an issue in SQLGlot.
In general, SQLFrame will have deeper understanding of converting PySpark's functions to other engines because I create overrides just within SQLFrame. Eventually I intend to get these applied SQLGlot, so SQL parsing would also have the conversion, but it would significantly slow down development so I'm waiting to do this later.

authors: cristian-marisescu eakmanrq

repo: eakmanrq/sqlframe


text:
Adds Feature for Copy Current Branch Name

Description
Summary of Changes

Registers a command to bind a command id under the namespace gitlens.views.branches
Adds a command contribution to make it available in the Command Palette
Registers an onCommand activationEvent
Uses GitService and env to copy the name to clipboard

Background and Motivation
I love using GitLens and it works great when managing multiple repositories and multiple branches. Given the large number of branches in my repos, I would find it extremely convenient to have a command from the Command Palette that copies the name of the currently checked out branch. I do this so many times in a day in order to paste it in trackers and pull requests.
While this can be done using the copy command in the branchesView, it will save having to open that view and scroll through the hundreds of branches to find the one with the checkmark next to it.
Related Issue: https://github.com/eamodio/vscode-gitlens/issues/1306
Checklist

[x] I have followed the guidelines in the Contributing document
[x] My changes follow the coding style of this project
[x] My changes build without any errors or warnings
[x] My changes have been formatted and linted
[x] My changes include any required corresponding changes to the documentation
[x] My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
[x] My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

I made some minor changes and merged it. Thank you so much for your contribution!

I made some minor changes and merged it. Thank you so much for your contribution!

Wow! Thank you for accepting the change and for doing it so quickly too! I've verified it on GitLens insiders--it's so cool to see it on there! 👍

Wow! Thank you for accepting the change and for doing it so quickly too! I've verified it on GitLens insiders--it's so cool to see it on there! 👍

repo: eamodio/vscode-gitlens


text:
GSE136719 - Single-cell analysis reveals dynamic changes of neural cells in developing human spinal cord

Project short name:
Zhang2021HumanSpinal
Primary Wrangler:
Secondary Wrangler:

Link to Ingest

https://contribute.data.humancellatlas.org/projects/detail?uuid=9288329e-0a0a-42f3-a7b7-61fa9cfb0600&tab=project

Associated files

Google Drive:

Published study links

Paper:
https://doi.org/10.15252/embr.202152728

Accessioned data: GSE136719

Key Events

[ ] Convert published metadata to HCA spreadsheet
[ ] Manually curate dataset to meet HCA metadata standard
[ ] Collect any matrix and cell-type annotation files
[ ] Upload sheet to validate metadata
[ ] Check linking using ingest graph validator
[ ] Transfer raw files to ingest to validate data files
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Paper needs to be requested to the library services

Emailed the authors to ask about getting the linkings between the sample IDs and the unique donors. This info is not available via the publication or GEO accession.

Closing for lack of response form contributor and no linking available

authors: ami-day ofanobilbao

repo: ebi-ait/hca-ebi-wrangler-central


text:
Possible duplicates: CHEBI:94433 + CHEBI:140628

CHEBI:94433 and CHEBI:140628 seem to be duplicates (the same InChI/InChIKey/structure).

Now merged. Thanks

repo: ebi-chebi/ChEBI


text:
PPU Suggestion: Aggregate months or provide month toggle

We have had the following from a user

I am having a good play with some of the data on your website, in particular PPU For my CCG you give November 2018 data and potential savings.  Is there any way of providing data over a time period e.g. April to November 2018? I don't know how much the data fluctuates much on a monthly basis, as we do see quite big swings on a month by month basis I guess what I'm asking is are the November savings data reflective of the true picture for the previous 6 months?

We have given instructions on how to add the month to the url as a work around however should we consider

Aggregating over a three month period
Adding a drop down to select the month of interest

In one CCG dashboard I did notice Ramipril Caps/Tabs appear. They run very close together in the drug tariff (before concession applied - so aggregation might help with removing that)

From discuss yesterday: aggregation doesn't feel like the right solution to the problem. A convenient fix which is possibly Good Enough is just to allow selecting the month of interest.
Note we don't have a full historic set of PPU data, so perhaps just "back a month" and "forward a month" buttons?

Slightly related to this when you go to the PPU page from a CCG dashboard (e.g.  https://openprescribing.net/ccg/15N/price_per_unit/) there's no date on the URL. But there a link under the CCG's name at the top of the page: "We have identified about £429,546 of possible savings in NHS Devon CCG in May 2019" - which goes to exactly the same results but the url has the date added on. The presence of the link could be a bit confusing and we should consider removing if we implement a date option as above

related to #1964

authors: HelenCEBM brianmackenna sebbacon

repo: ebmdatalab/openprescribing


text:
Do a tech stack review

Has anyone built a generic, web-based visualisation tool for large datasets, with affordable licensing for public use?
If so then we can throw away our bespoke code!
Last time I reviewed this I concluded that no-one had. Tableau is the closest, but they are licensed per-user, and most of their upcoming competitors seem to have adopted the same per-user model.
A couple seemed open to a discussion about making GP dashboards free, and charging for CCGs.
It would be worth spending some time looking into this.

We talked about this today, as a consequence of Seb's work on #101 and #73.
The options seem to be as follows:

Improve our Postgres performance by an order of magnitude
Move the back-end to a big data product that hosts data and offers an API out of the box (Anna feels like this must exist) - or move the whole thing to a visualisation product as above (probably doesn't exist)
Move the back-end to BigTable with Google App Engine  (or Amazon equivalent, though we are biased towards Google because we already use BQ and Django)
Move the back-end to Cassandra / CouchDB
Move the back-end to BQ

(1) is out because it will require dedicated hardware (we are running out of space on our current setup) and dedicated sysadmin skills. (4) is out because it would need too much resource.
We think the next steps are, in order:

hunt for a (2), which may well not exist
investigate BigTable with GAE
cost up the BQ solution.

Some research into (2) products this morning.
The Google (data ecoystem) has a variety of products - as well as BigTable there's Cloud Datastore and Cloud SQL.
Our requirements are:

suitable for 120GB of data (currently) and up to 250GB, on a 4-5 year horizon
out-of-the-box JSON API, or one that we can create easily, or a way to retain our existing JSON setup easily
support filtering and aggregation in queries
we can do without JOINs if we continue to use postgres for our small tables
acceptable pricing

Although it's not a strict requirement, any solution that supports on-the-fly percentile calculations would be extremely useful.
Quick notes:

Cloud SQL is interesting - it's basically full-featured MySQL, would cope with our size data, could simply drop in as a replacement for the prescribing tables
Datastore is designed for data of our size, has JSON API out of the box, but doesn't support aggregation which I think rules it out
BigTable is a columnar datastore, aimed at >1TB datasets (but not necessarily a problem?), looks like you might have to convert data to HBase before importing

I will investigate Cloud SQL and BigTable further. Cloud SQL is particularly interesting, as the friction/cost of moving might be quite low.

I've written up options here: https://docs.google.com/document/d/1cbO0KiagXPOLR32NAylqs7tkGMA2A5mPERrTxNMMZT4/edit
Additions, corrections, commiserations welcome.
I think the next steps are:

Set up an experimental ElasticSearch index as per #90 (but with more data if possible)
Set up an experimental Google DataStore dataset

I am also looking at using BigQuery as a back-end - will spend 1 more day on this.

I've started a test rig for various queries here and written up a brief for a postgres expert here.

Russ says he has some free time (yay) to look at Postgres once he's finished post-EMF work. Will nudge him middle of next week if necessary.

authors: annapowellsmith sebbacon


text:
Language code

Normalization rules and other resources will be organised by language codes, including country variants.
ISO 639-3 defines 3-letter codes: eng
AWS and GCP use 2-letter codes (ISO 639-1?) followed by country codes: en-US | es-US | en-AU | fr-CA | en-GB
Do we want to conform to existing vendor practice or use a combination of both, for example eng-GB?

Perhaps we don't have to be prescriptive about files being named with language codes? Continue discussion on Slack.

I think it would make sense to use the current vendor practices. Unless there strong arguments against.

I propose fulllocale to follow https://tools.ietf.org/html/bcp47 . This would support all languages and script variations, while offering decent fallback (we would follow standard L10N practises) E.g. these files could both exist: nl.csv, nl-BE.csv.
We could then select the most specifically appropriate locale automatically e.g., if text is in nl-NL nl.csv would be used. This would also allow us to support separation per script, eg zh-Hant-CN would be Chinese (Traditional, China) and available locale solutions can then go find the most appropriate replacement file for that locale (i.e. we support a lot of separation, while not introducing our own complexity and using a well established standard).
Or to use a European example: sr-Latn for Serbian (Latin) and sr-Cyrl for Serbian (Cyrillic).
This is forward compatible for "exceptional" languages, as well as following the AWS and GCP convention (which seems to be based on BCP 47 as well).
@EyalLavi FYI the first language code is ISO 639-1 ISO 639-2/T and 639-3 and 639-5 in that order of lookup (and with a preference to use 639-1 2-letter codes if available for that language).

BCP47 sounds like a good idea. My only concern is that if we don't use the same system that's used by vendors, we will have to build additional logic to resolve BCP47 codes to vendor codes. It might be trivial though.
BTW, only European languages are in scope for this tool: https://github.com/ebu/ai-benchmarking-stt/tree/master/docs/releases/1.0
Options:

Adopt BCP47 as @MikeSmithEU suggests. Pros: future proof, extensive, standards compliant. Cons: additional complexity, less user friendly, possibly out of scope.
Use vendors codes. Pros: easy option, user friendly. Cons: dependency on vendors code, not future proof.
Defer decision to future release and use vendor codes for now.

Some vendors, like google use BCP-47  might be useful to review what other vendors use?

A library already exists for translation from full BCP47 to the subset of two-letter language code followed by two-letter country case. The decision is to use BCP47 without further restriction on the format.

authors: EyalLavi MikeSmithEU pietrop

repo: ebu/ai-benchmarking-stt


text:
Does Supplementary Materials have a template?

Does Supplementary Materials have a template?

Please see #4

authors: liaopan-lp tsattler


text:
Move runtime-metamodel to its own repository

Feature Request
The runtime-metamodel is currently embedded as module in the GradlePlugins repository, from where it should be relocated to its new home https://github.com/eclipse-edc/runtime-metamodel
Which Areas Would Be Affected?
GradlePlugins, build, release
Why Is the Feature Desired?
break dependency cycle, decouple Gradle plugins from the runtime metamodel
Solution Proposal
move the code, adapt the build

can this be closed?

authors: ndr-brt paullatzelsperger

repo: eclipse-edc/Connector


text:
fix: add missing json fields for sample streaming http-to-http

What this PR changes/adds
Fixes issues #244 and #334 for sample transfer/streaming/streaming-01-http-to-http

Missing assigner field in contract request
Missing transferType in transfer request
Did not use JSON files in testcase

Why it does that
Sample can not be executed completely.
Further notes
Linked Issue(s)
Closes #244
Closes #334

@ndr-brt:
Since I can't find any doc about how to run the test-cases, I can't fix it that easily.
The official doc returns 404 for almost every link in there (i.e. https://eclipse-edc.github.io/testing.md found in https://eclipse-edc.github.io/documentation/for-contributors/#52-testing-best-practices).
I flew over the convention slightly, does the new name match?

@ndr-brt: Since I can't find any doc about how to run the test-cases, I can't fix it that easily. The official doc returns 404 for almost every link in there (i.e. https://eclipse-edc.github.io/testing.md found in https://eclipse-edc.github.io/documentation/for-contributors/#52-testing-best-practices).
I flew over the convention slightly, does the new name match?

link is broken, but you can find it under contributors manual/writing tests

link is broken, but you can find it under contributors manual/writing tests

Almost all links are broken, unfortunately ;-)
For me, it still does not work for half of the test cases, since Testcontainers does not recognize the docker-engine.
Using the official Docker engine (no Desktop) on Ubuntu 24.04. User is in correct group and can contact the daemon.
Path /var/run/docker.sock exists and has rw access for the group the user is in.
Tried it with setting the DOCKER_HOST variable as well. No change.

link is broken, but you can find it under contributors manual/writing tests

Almost all links are broken, unfortunately ;-)

contributions are the key ;)

For me, it still does not work for half of the test cases, since Testcontainers does not recognize the docker-engine. Using the official Docker engine (no Desktop) on Ubuntu 24.04. User is in correct group and can contact the daemon. Path /var/run/docker.sock exists and has rw access for the group the user is in. Tried it with setting the DOCKER_HOST variable as well. No change.

I'm also using docker engine on ubuntu without issues, it worked out-of-the-box, there's no need to do changes, as long as docker is executable from the command line by a non root user.
what's the error?

link is broken, but you can find it under contributors manual/writing tests

Almost all links are broken, unfortunately ;-)

contributions are the key ;)

For me, it still does not work for half of the test cases, since Testcontainers does not recognize the docker-engine. Using the official Docker engine (no Desktop) on Ubuntu 24.04. User is in correct group and can contact the daemon. Path /var/run/docker.sock exists and has rw access for the group the user is in. Tried it with setting the DOCKER_HOST variable as well. No change.

I'm also using docker engine on ubuntu without issues, it worked out-of-the-box, there's no need to do changes, as long as docker is executable from the command line by a non root user. what's the error?

Strange.
Basically, the long exception-message starts with:
Advanced01openTelemetryTest > runSampleSteps() FAILED
    java.lang.IllegalStateException: Could not find a valid Docker environment. Please see logs and check configuration
        at org.testcontainers.dockerclient.DockerClientProviderStrategy.lambda$getFirstValidStrategy$7(DockerClientProviderStrategy.java:274)

It seems to be one of those funny things where installing OS updates and rebooting fixes it.
Can't explain why, since OS was pretty up-to-date and settings seemed right, but it worked.

@ndr-brt: Took me a long time, but I fixed it. Not sure why ECA does not evaluate anymore, or why it evaluated before, since I am using my anonymized Github email. I filled out the form though.

I think it is becuse your first commit, that has no email reported: https://github.com/eclipse-edc/Samples/pull/335/commits/2f00b9d846db09c744125cbdaa25b99bc9dd31f4
maybe you could squash the three commits into one that has the correct info

@ndr-brt:
You mean the last one? Indeed this one was missing the author completely.
Shame on me.
Fixed that by a rebase / amend, should be better now.
Haven't seen the checkstyle errors during testing, fixed those as well.
Warnings were there before, I am not about to fix those as well.
So it should be good if all Github checks pass.
Btw: Is there a way to only run a certain test? It was quite annoying, since though I tried some things from the Gradle docs,
I could only get all tests running by firing:
.\gradlew test -DrunAllTests="true"

@ndr-brt: You mean the last one? Indeed this one was missing the author completely. Shame on me. Fixed that by a rebase / amend, should be better now.
Haven't seen the checkstyle errors during testing, fixed those as well. Warnings were there before, I am not about to fix those as well. So it should be good if all Github checks pass.
Btw: Is there a way to only run a certain test? It was quite annoying, since though I tried some things from the Gradle docs, I could only get all tests running by firing:
.\gradlew test -DrunAllTests="true"

if you are using Intellij you just need to add the -DrunAllTests="true" in the run configuration.
you could also create a configuration template and make it enabled for all the gradle commands (very handy for EDC projects)

@ndr-brt: You mean the last one? Indeed this one was missing the author completely. Shame on me. Fixed that by a rebase / amend, should be better now.
Haven't seen the checkstyle errors during testing, fixed those as well. Warnings were there before, I am not about to fix those as well. So it should be good if all Github checks pass.
Btw: Is there a way to only run a certain test? It was quite annoying, since though I tried some things from the Gradle docs, I could only get all tests running by firing:
.\gradlew test -DrunAllTests="true"

if you are using Intellij you just need to add the -DrunAllTests="true" in the run configuration. you could also create a configuration template and make it enabled for all the gradle commands (very handy for EDC projects)

But this would still run all tests, doesn't it?
Since it was pretty basic testing so far, the setup is quite simple (no sophisticated IDE).
May have to change that once I start real development.
Checks look good now, right?

if you are using Intellij you just need to add the -DrunAllTests="true" in the run configuration. you could also create a configuration template and make it enabled for all the gradle commands (very handy for EDC projects)

But this would still run all tests, doesn't it? Since it was pretty basic testing so far, the setup is quite simple (no sophisticated IDE). May have to change that once I start real development.

no, this will say to junit: "run the test you want without considering the tag attached to it.

Ok, might work with IDEA. You don't know a way to run a specific test through CLI?

sure, for this one just execute:
./gradlew :system-tests:test --tests "org.eclipse.edc.samples.transfer.streaming.Streaming01httpToHttpTest.streamData" -DrunAllTests=true

Ok, might work with IDEA. You don't know a way to run a specific test through CLI?

sure, for this one just execute:
./gradlew :system-tests:test --tests "org.eclipse.edc.samples.transfer.streaming.Streaming01httpToHttpTest.streamData" -DrunAllTests=true

Haven't tried it yet, but sounds as if it could work. Thank you.
I had tried it basically without the -DrunAllTests=true, since I thought it would do the opposite of what I wanted to achieve.
I read about the fact that otherwise system-tests would not be executed at all, but I thought it would be sort of a side-effect of running all existing test-cases.
Might be a case where we first enlarge the scope to then filter out.
Not too familiar with Gradle unfortunately.

Haven't tried it yet, but sounds as if it could work. Thank you. I had tried it basically without the -DrunAllTests=true, since I thought it would do the opposite of what I wanted to achieve. I read about the fact that otherwise system-tests would not be executed at all, but I thought it would be sort of a side-effect of running all existing test-cases. Might be a case where we first enlarge the scope to then filter out. Not too familiar with Gradle unfortunately.

the -DrunAllTests=true basically says "run the selected tests despite the junit tag", given the fact that this test has the "EndToEnd" label, that property is a sort of passepartout to not care about the labels and just run the tests.
For this test, this command does the same exact thing as the one before
./gradlew :system-tests:test --tests "org.eclipse.edc.samples.transfer.streaming.Streaming01httpToHttpTest.streamData" -DincludeTags="EndToEndTest"

Thank you for the explanation (and preventing me from dying uneducated ;-)).

anyway, thanks for the contribution, I'm gonna merge this 🚀

Perfect, thanks. You are welcome. Hope it helps somebody.

authors: klumbe ndr-brt

repo: eclipse-edc/Samples


text:
Delete SAS Token from vault after expiration

Feature Request
This Feature Request aims at delete the SAS Token, once is no longer viable to be used in transfers, to reduce the storage of not needed entries.
Why Is the Feature Desired?
Currently, the SAS Token is created with a fixed timeframe after which the token cannot be used for a transfer (even if it already started).
However, after secret is stored in the respective vault, it is never deleted, even after expiration, so unusable secrets are stored indefinitably.
Solution Proposal
Suggestion would go to delete the SAS Token after a Transfer has been processed (successfully or not). A possible approach would be to delete it during the deprovisioning phase (here), however this has not yet been tested so validity cannot be confirmed.

The delete of the secret token should be already done externally in the TransferProcess state machine by the DeprovisionResponseHandler
this because this behavior should be shared in all the provisioner implementations.
If there's an issue there then I think it could be considered a bug and opened in the Connector repo.

The delete of the secret token should be already done externally in the TransferProcess state machine by the DeprovisionResponseHandler this because this behavior should be shared in all the provisioner implementations.
If there's an issue there then I think it could be considered a bug and opened in the Connector repo.

What we are seeing is that the SAS Token is not being deleted from the vault, so a potential bug is happening. Will update this issue after investigating it. For now will delete and will reopen if needed. Thank you for the feedback, @ndr-brt !

repo: eclipse-edc/Technology-Azure


text:
Update the existing data migration tool

The existing tool should be pretty close. It should be updated to operate on the migration package format and it should also upload files. If possible, the tool should be able to resume on error. The tool will authenticate against pass-core as a backend user. The tool should be able to check that everything has been uploaded.

Note there is a pr for unit testing which was approved, but never merged: https://github.com/eclipse-pass/pass-data-migration/pull/5

repo: eclipse-pass/main


text:
Take over the responsibility of pass-auth and pass-docker reverse proxy

This pr updates to pass-core to take over the responsibility of the reverse proxy and pass-auth in pass-docker.
Major changes:

Support SAML login
Stateful because otherwise every request would result in hitting the IDP
Logout of session by using /logout. Does not log out of IDP.
Serve out /app/ from a resource specified by PASS_CORE_APP_LOCATION
Integration tests run against IDP in Docker container on port 8090 and pass-core on port 8080
Expected keys for SAML user properties are configurable
CSP header set here and not in pass-ui now
/ redirects to /swagger/index.html

Must redirect so swagger can load resources
/swagger/ now forward to /swagger/index.html
The /doc and /swagger endpoint locations are not actually configurable although the Elide docs say they are

By default pass-core is configured to run against an IDP on port 8090 base on SimpleSAMLphp. See https://github.com/kenchan0130/docker-simplesamlphp/ for info about the image. Resources for this are in in saml2/ on the classpath.
The /app/ requests are handled by the Spring Boot support for serving out static resources from a location. The location can be on the classpath, a file, or a http url.  Requests to /app/* that do not match a resource result in /app/index.html being served out. This is needed for the routes that that the ui handles.
In order to work around some SAML login issues /favicon.ico is forwarded to /app/favicon.ico and both are public.
For how to test see description on this pr: https://github.com/eclipse-pass/pass-acceptance-testing/pull/16
Related prs:

https://github.com/eclipse-pass/pass-docker/pull/365
https://github.com/eclipse-pass/pass-ui/pull/1260
https://github.com/eclipse-pass/pass-acceptance-testing/pull/16

Open questions:

Session timeout handling by pass-ui

Known problems:

Acceptance tests will not pass until related prs are merged
Looks like pass-support integration tests which use testcontainers waiting for /data/grant on pass-core are timing out. It may be the case testcontainers is setting Accept headers such that it looks like a browser and initiating SAML login. Fixed by: https://github.com/eclipse-pass/pass-support/pull/104

TODO (create issues for these):

Turn on CSRF
Update documentation in main:

https://github.com/eclipse-pass/main/blob/main/docs/dev/authentication-authorization.md

I did a round of testing using pass-docker and all the associated changes.  Here are the results:
Things that may need tuning:
test logout, switching to different user
- had to go and clear all cookies, then the login page was displayed
- I think it would be good to figure out how to clear all the cookies on logout and redirect to idp login page
after logged in, clear all cookies, browser redirects to idp and logs in.  I see two JSESSIONID cookies, one for / and one for /idp with an info that it was blocked.  not sure if this is an issue, but maybe something to investigate.
login, then restart pass-docker, then click on link, and basic auth prompt from browser popped up
If going to root path http://localhost:8080/, i think it should redirect to http://localhost:8080/app/ since most users are not concerned with swagger docs.  The swagger docs should be associated to another path, some like http://localhost:8080/api-docs/
type invalid path: http://localhost:8080/app/grants/52ggg resulted in blank page
logged in as nih-user, http://localhost:8080/app/grants/62 show grant page even though not a pi or co-pi, i guess this is how it is, but it seems odd from an authorization perspective
Tests completed that worked as expected:
test deposit into local jscholarship, passed
test deposit into local jscholarship and fake pmc sftp server, passed
test deposit with doi into local jscholarship and fake pmc sftp server, passed
test going to /,  /app/* and /data/* path without login, passed with redirect to idp login page
http://localhost:8080/swagger/index.html - passed, redirected to idp login before access
http://localhost:8080/app/assets/chunk.app.888ed776cb92702e5241.js without login, passed with redirect to login
Test with postman
In general, this worked fine.  However, I saw in some cases a JSESSIONID was set even though a 401 was returned.
http://localhost:8080/data/grant - no auth, return 401 html page (accept set to */*) No Set-Cookie of JSESSIONID=node0ue094ufgyokg1b77eud88mmr924.node0; Path=/
http://localhost:8080/data/grant - no auth, return 401 status, no body (accept set to application/vnd.api+json).  Has Set-Cookie of JSESSIONID=node0ue094ufgyokg1b77eud88mmr924.node0; Path=/
http://localhost:8080/data/grant - no auth, return 401 status with standard json body (accept set to application/json).  No Set-Cookie of JSESSIONID=node0ue094ufgyokg1b77eud88mmr924.node0; Path=/
http://localhost:8080/data/grant - with basic auth, returned grants.  JSESSIONID with node... set, cookie is expired session, httponly false, secure false
http://localhost:8080/user/whoami - with basic auth (backend/backend) - 500 error, json body (accept set to */*)
{
"message": "No user matching principal: backend"
}

@msp just confirming this is staying as is? https://github.com/eclipse-pass/pass-core/pull/82#discussion_r1548565243

@msp just confirming this is staying as is? #82 (comment)

Correct. Added a comment.

authors: markpatton rpoet-jh

repo: eclipse-pass/pass-core


text:
SD_FORBID_HELPERS not doing anything

Issue Description
Ran into some problem with DNN on an old AMD Turion processor so wanted to test without the native mkldnn helper. Tried various way to set SD_FORBID_HELPERS to disable the helper but couldn't get it to work.
Version Information
Running deeplearning4j-1.0.0-beta7
Additional Information
I found in libnd4j/include/legacy/impl/Environment.cpp what looks like a typo in the check. It pulls in the environment variable into a local variable and then performs a check against another local variable.
        /**
         * If this env var is defined - we'll disallow use of platform-specific helpers (mkldnn, cudnn, etc)
         */
        const char* forbid_helpers = std::getenv("SD_FORBID_HELPERS");
        if (max_master_threads != nullptr) {
            _allowHelpers = false;
        }

The forbid_helpers variable is not checked anywhere else in the code.

Try to set SD_MASTER_THREADS then and see if that makes a difference :)
Also try to set ND4J_FALLBACK, ND4J_MKL_FALLBACK=true, and SD_BLAS_FALLBACK.

Pro tip!
I already did as I found missing check in the code. And it worked. Now it's a lot slower on the DNN operations but it hasn't crashed yet.

Awesome! Please send a pull request with the fix if possible. Thanks!

You got it. I created PR #9126 but need to get the ECA signed.

Created PR #9127 instead. Looks like the signoff worked this time.

authors: Ulrhol saudet


text:
MP Talk by Cesar, Otavio and Rafael @ Code One 2019

Conference Name:
Oracle CodeOne 2019
Conference website and social media links:
https://www.oracle.com/code-one/
@OracleCodeOne
Talk name:
Keeping Brazil’s Medical Industry Safe with MicroProfile [DEV4113]
Talk Link:
https://events.rainfocus.com/widget/oracle/oow19/catalogcodeone19?search=DEV4113
Speaker Names:
Cesar Hernandez Mendoza, Senior Software Enginner, Tomitribe (@CesarHgt)
Otavio Santana, Senior Software Engineer, Platform.sh/Soujava (@otaviojava)
Rafael Guimaraes, Analyst, GBR Systems (@GuimaraesSv)
Tweet Message:
Come and learn how MicroProfile played a critical role in that enabled the cloud migration in the Brazilian medical industry. Wednesday, Sep 18, 17:00 | Moscone South - Room 310/311 @OracleCodeOne @MicroProfileIO @graalvm @Groundbreakers @Java_Champions #BreakNewGround
logos: groundbrakers, Java Champions & MP

@cesarhernandezgt all set up for two tweets.

authors: cesarhernandezgt rstjames

repo: eclipse/microprofile-marketing


text:
Barcode scanner when scanned second time (on the example) shows Cancelled

I am testing this barcode scanner for my own purpose. Even through Dev App, the barcode scanner once scanned, shows the scanned text. When  scan again another bar code, the result shows 'Scan cancelled'.
Some inherent issue?

If this is on Android only try using tabris nightly. They retooled the callbacks for a very specific workflow that this plugin may be using.

We are seeing a problem with the plugin in the latest nightly and will investigate. When using 1.9 the plugin should work.

Which versions of Tabris.js and Barcode Scanner?  Barcode scanner has been flakes in the last few versions: #841

@natanraj What version of Barcode Scanner, not Cordova (in fact you don't even need to install Cordova).  It's specified in cordova/config.xml.  Or if you aren't specifying a version you'll get the latest, which is currently 6.0.1.

Some update on the barcode scanner plugin:
With the update to cordova-android 5.2.2 they have fixed a bug that allowed us enforce the min sdk version to be 16. This means that the barcode scanner plugin that wants to enforce min sdk version 15 can now be overruled. So phonegap-plugin-barcodescanner (6.0.2) is the one that should be used (and will be used in the Tabris.js developer app).
In addition we have fixed a bug in the tabris integration with the cordova-android platform  where the request codes were not handled correctly. (illegal request code range used by plugins, http://stackoverflow.com/a/33332468/271877)

We have reports of successful scans so i am closing this issue.

authors: cookieguru dested mpost natanraj

repo: eclipsesource/tabris-js


text:
waffle.io Badge

Merge this to receive a badge indicating the number of issues in the ready column on your waffle.io board at https://waffle.io/eclipsesource/tabris-js
This was requested by a real person (user patrykmol) on waffle.io, we're not trying to spam you.

Thanks bot, but we've more or less stopped using waffle.

authors: ralfstx waffle-iron