Smartshunt not working over ESPhome bluetooth proxy

Describe the bug
From a working setup (used the device for 3 months now) , now i have a fault when loading the integration.
Failed to setup.
Removed it, restarted HA en added it again, now it fails to setup, no entities are loaded.
To Reproduce
Steps to reproduce the behavior:
Add the integration the normal way, fill in name, mac and token.
Expected behavior
That it connects over the ESPhome bluetooth proxy, as this has before.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
logs:
Logger: homeassistant.loader
Source: loader.py:1249
First occurred: 18:59:22 (6 occurrences)
Last logged: 21:10:23
Unexpected exception importing platform custom_components.victron_ble.sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in _load_platform
cache[full_name] = self._import_platform(platform_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in protected_loop_func
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/importlib/init.py", line 90, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1387, in _gcd_import
File "", line 1360, in _find_and_load
File "", line 1331, in _find_and_load_unlocked
File "", line 935, in _load_unlocked
File "", line 995, in exec_module
File "", line 488, in _call_with_frames_removed
File "/config/custom_components/victron_ble/sensor.py", line 198, in
PassiveBluetoothDataProcessor[Optional[Union[float, int]]]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/typing.py", line 384, in inner
return func(*args, **kwds)
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/typing.py", line 1066, in _generic_class_getitem
_check_generic(cls, params, len(cls.parameters))
File "/usr/local/lib/python3.12/site-packages/typing_extensions.py", line 2947, in _check_generic
raise TypeError(f"Too {'many' if alen > elen else 'few'} arguments"
TypeError: Too few arguments for <class 'homeassistant.components.bluetooth.passive_update_processor.PassiveBluetoothDataProcessor'>; actual 1, expected at least 2
AND:
Logger: homeassistant.config_entries
Source: config_entries.py:594
First occurred: 18:59:23 (3 occurrences)
Last logged: 21:10:23
Error setting up entry SmartShunt for victron_ble
Error setting up entry Smartshunt for victron_ble
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in _load_platform
cache[full_name] = self._import_platform(platform_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in protected_loop_func
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/importlib/init.py", line 90, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1387, in _gcd_import
File "", line 1360, in _find_and_load
File "", line 1331, in _find_and_load_unlocked
File "", line 935, in _load_unlocked
File "", line 995, in exec_module
File "", line 488, in _call_with_frames_removed
File "/config/custom_components/victron_ble/sensor.py", line 198, in
PassiveBluetoothDataProcessor[Optional[Union[float, int]]]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/typing.py", line 384, in inner
return func(*args, **kwds)
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/typing.py", line 1066, in _generic_class_getitem
_check_generic(cls, params, len(cls.parameters))
File "/usr/local/lib/python3.12/site-packages/typing_extensions.py", line 2947, in _check_generic
raise TypeError(f"Too {'many' if alen > elen else 'few'} arguments"
TypeError: Too few arguments for <class 'homeassistant.components.bluetooth.passive_update_processor.PassiveBluetoothDataProcessor'>; actual 1, expected at least 2
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/victron_ble/init.py", line 36, in async_setup_entry
await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, in async_forward_entry_setups
await integration.async_get_platforms(platforms)
File "/usr/src/homeassistant/homeassistant/loader.py", line 1162, in async_get_platforms
import_future.result()
File "/usr/src/homeassistant/homeassistant/loader.py", line 1150, in async_get_platforms
platforms.update(self._load_platforms(platform_names))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1075, in _load_platforms
platform_name: self._load_platform(platform_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1269, in _load_platform
raise ImportError(
ImportError: Exception importing custom_components.victron_ble.sensor
At the moment i have no idea why this is not working, The smart shunt still workings fine with the victron app, also tried updating the firmware. no differance.
Can somebody check this?

What i also see is that the auto discover still works.
So if i remove the integration, restart HA, and wait a bit. the Auto discover finds the smart shunt.
But when i press configure the integration fails the setup (i'm still presented with the 3 input fields).


tBTC dApp IA

An IA map reference that reflects new UX patterns and features.
@mhluongo FYI working on this to review when you return from PTO.

@Shadowfiend would love to walk you through this at your convenience.

Interesting. I could probably use a walkthrough of this as well


List of available key to use in pinpoint_add_clue/pinpoint_add_clues

Hi,
Is there a list of available key to use in the functions pinpoint_add_clue and pinpoint_add_clues?
Can we add custome ones?
Thank you.

Hello~
Please ref to https://github.com/naver/pinpoint-c-agent/blob/master/collector-agent/CollectorAgent/Readme.md, hope it help.
Custome ones not support now.

Hi,
Yes that was what I wanted it's just that it says that is for trift and I'm using grpc. Do they use the same keys?
By the way in a lot of plugins in the "PHP Excample" you are using the 'EXP' key for exceptions not the 'ER' + 'EMSG' so that make me believe that this list in not updated as the 'EXP' key it's not in that list. Are there any more keys beside EXP that are missing from the documentation?
What type of data can you send in each key, ex: int|float|string?
What limitation are applied over the data that you can send in each key, or there is none, ex: length|no negative|etc...?
For 'stp' you have pointed me to the valid list to use in the #196 does a list like that exists for any other key?
Best regards,
Thank you.

"Yes that was what I wanted it's just that it says that is for trift and I'm using grpc. Do they use the same keys?"

the same

What type of data can you send in each key, ex: int|float|string?

Only support string.

What limitation  ...

Try to make agent simple and easy.

By the way in a lot of plugins in the "PHP Excample" you are using the 'EXP' key for exceptions not the 'ER' + 'EMSG' so that make me believe that this list in not updated as the 'EXP' key it's not in that list. Are there any more keys beside EXP that are missing from the documentation?

About this ?

I'm closing this issue as the initial scope is reached.


RDF lists created using RDFCollections.asRDF() are not properly serialized using TurtleWriter

Current Behavior
When serializing an RDF list created using RDFCollections.asRDF() with TurtleWriter the first segment of the list is not properly inlined as it is not recognized as a well-formed list.
The result looks as this:
@prefix ex: <http://example.com/ns#> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .

ex:Cities ex:list [ a rdf:List;
      rdf:first ex:NewYork;
      rdf:rest (ex:Rio ex:Tokyo)
    ] .

Please notice the first part with explicit rdf:first and rdf:rest statements instead of the inlining of the list, which is done for the last two segments of the list.
This is caused by org.eclipse.rdf4j.rio.turtle.TurtleWriter.isWellFormedCollection() not recognizing the rdf:type rdf:List type statement for the first part of the list segment which is generated in RDFCollections.asRDF() (or more specifically in org.eclipse.rdf4j.model.util.RDFCollections.consumeCollection(Iterable<?>, Resource, Consumer<Statement>, ValueFactory, Resource...)).
A simple solution would be to adjust org.eclipse.rdf4j.rio.turtle.TurtleWriter.isWellFormedCollection() to recognize and accept that statement when checking for additional statements within the list.
Expected Behavior
The proper visualization of the list would instead look like this:
@prefix ex: <http://example.com/ns#> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .

ex:Cities ex:list (ex:NewYork ex:Rio ex:Tokyo) .

Steps To Reproduce
This test case reproduces the behavior:
package com.metaphacts.services.ontologies;

import static org.junit.Assert.assertEquals;

import java.io.StringWriter;
import java.util.List;

import org.eclipse.rdf4j.model.BNode;
import org.eclipse.rdf4j.model.IRI;
import org.eclipse.rdf4j.model.Model;
import org.eclipse.rdf4j.model.impl.TreeModel;
import org.eclipse.rdf4j.model.util.RDFCollections;
import org.eclipse.rdf4j.model.util.Values;
import org.eclipse.rdf4j.model.vocabulary.RDF;
import org.eclipse.rdf4j.rio.RDFFormat;
import org.eclipse.rdf4j.rio.Rio;
import org.eclipse.rdf4j.rio.WriterConfig;
import org.eclipse.rdf4j.rio.helpers.BasicWriterSettings;
import org.junit.Test;

public class RDFCollectionsTest {
    @Test
    public void testBNodeValuesInList() throws Exception {
        final String ns = "http://example.com/ns#";
        IRI newyork = Values.iri(ns, "NewYork");
        IRI rio = Values.iri(ns, "Rio");
        IRI tokyo = Values.iri(ns, "Tokyo");
        IRI cities = Values.iri(ns, "Cities");
        IRI exList = Values.iri(ns, "list");
        BNode listHead = Values.bnode("n1");

        Model data = new TreeModel();
        data.setNamespace("ex", ns);
        data.setNamespace("rdf", RDF.NAMESPACE);
        RDFCollections.asRDF(List.of(newyork, rio, tokyo), listHead, data);
        data.add(cities, exList, listHead);

        String expected = "" +
                "@prefix ex: <http://example.com/ns#> .\n" +
                "@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .\n" +
                "\n" +
                "ex:Cities ex:list (ex:NewYork ex:Rio ex:Tokyo) .\n";

        StringWriter stringWriter = new StringWriter();
        WriterConfig config = new WriterConfig();
        config.set(BasicWriterSettings.INLINE_BLANK_NODES, true);
        config.set(BasicWriterSettings.PRETTY_PRINT, true);
        Rio.write(data, stringWriter, RDFFormat.TURTLE, config);
        String actual = stringWriter.toString();

//        System.out.println("### ACTUAL ###");
//        System.out.println(actual);
//        System.out.println("#################\n");

//        System.out.println("### EXPECTED ###");
//        System.out.println(expected);
//        System.out.println("#################\n");

        assertEquals("The visual representation should be a proper list for all elements", expected, actual);
    }
}

Adding this line before serializing the model as string makes the test work, but the fix should not remove that statement but rather recognize and accept it.
        // deleting the type statement in the list would make it work
        // [ a rdf:List ]
        // data.remove(listHead, RDF.TYPE, RDF.LIST);

Version
3.7.4 (and likely earlier versions as well)
Are you interested in contributing a solution yourself?
Perhaps?
Anything else?
No response

In the ShaclSail I ended up writing my own rdf list implementation because of this issue. My fix was to not generate the type statements.
https://github.com/eclipse/rdf4j/blob/main/core/sail/shacl/src/main/java/org/eclipse/rdf4j/sail/shacl/ast/ShaclAstLists.java

This should be a simple fix in the TurtleWriter without further side effects, so I would rather go for a real solution here instead of working around it (although we do currently have a workaround in our application until this is fixed, because the resulting RDF text looks rather weird). This seems just like an oversight when checking list whether they are well-formed.
I guess it doesn't hurt to have simplified solutions for special cases as you created for the ShaclSail, but in general I'd rather reuse a single, well-tested and compliant implementation.

If we do a minor change in consumeCollection Method in RDFCollections Class, we will be able to produce the desired output.
consumeCollection is called in asRDF method by RDFCollections.asRDF(..).
public static void consumeCollection(Iterable<?> values, Resource head, Consumer<Statement> consumer,
			ValueFactory vf,
			Resource... contexts) {
		Objects.requireNonNull(values, "input collection may not be null");
		Objects.requireNonNull(consumer, "consumer may not be null");
		Objects.requireNonNull(vf, "injected value factory may not be null");

		Resource current = head != null ? head : vf.createBNode();

		//Statements.consume(vf, current, RDF.TYPE, RDF.LIST, consumer, contexts); --just comment out this line in order to

		Iterator<?> iter = values.iterator();
		while (iter.hasNext()) {
			Object o = iter.next();
			Value v = o instanceof Value ? (Value) o : Literals.createLiteralOrFail(vf, o);
			Statements.consume(vf, current, RDF.FIRST, v, consumer, contexts);
			if (iter.hasNext()) {
				Resource next = vf.createBNode();
				Statements.consume(vf, current, RDF.REST, next, consumer, contexts);
				current = next;
			} else {
				Statements.consume(vf, current, RDF.REST, RDF.NIL, consumer, contexts);
			}
		}
	}
	public static void consumeCollection(Iterable<?> values, Resource head, Consumer<Statement> consumer,
			Resource... contexts) {
		consumeCollection(values, head, consumer, SimpleValueFactory.getInstance(), contexts);
	}

	public static <C extends Collection<Statement>> C asRDF(Iterable<?> values, Resource head, C sink,
			Resource... contexts) {
		Objects.requireNonNull(sink);
		consumeCollection(values, head, sink::add, contexts);
		return sink;
	}

Output:
@prefix ex: http://example.com/ns# .
@prefix rdf: http://www.w3.org/1999/02/22-rdf-syntax-ns# .
ex:Cities ex:list (ex:NewYork ex:Rio ex:Tokyo) .
It would be a great help if someone could help me solve this problem from inside turtlewriter which will be the ideal fix for this issue. Ready to collaborate.

Thanks for picking this up @The-Nightwing ! I've left a few comments on your first PR, including a suggestion for a better fix in the TurtleWriter itself. Happy to talk further if anything is unclear.


Android Compose Change the App Icon

Name of the Codelab or Codelab URL
Describe the problem
A clear and concise description of what the problem is.
In which lesson and step of the codelab can this issue be found?
Lesson number + step number.
How to reproduce?
What are the exact steps to reproduce the problem?
Versions

What version of Android Studio are you using?
What API level are you targeting?

Additional information
Add any other context about the problem here.
codelab: basic-android-compose-training-add-scrollable-list

The issue is closed as no further clarification provided.


Replace simplejson with json and fix missed print statement

Address from this PR https://github.com/tobegit3hub/harbor-py/pull/2

Thanks for merging 😃


feat/places-to-destinations

Please review before merging.

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

GitHub release
v3.8.0

Your semantic-release bot :package::rocket:


Loss remains stable around 2.0 on 12h long training

Hi Moises. Not really an issue, more a question: I've been observing that some (most) of my RAVE LD trainings tend to oscillate around a train and validation loss of 2.0 even though they're running for hours. Is this an expected behaviour?
In a few earlier trainings, the loss went down straight close to 0 but I cannot reconstruct why that was.
Any ideas or suggestions would be much appreciated 😸
Best, Martin

Hey Martin,
Would you mind posting some more details? Aproximately how long is your dataset? How many dimensions is your RAVE model and which version are you using? Need to so further trainings, but generally speaking, on a normal training, the loss should definitely go bellow 2.0 and slowly drop. Did you change the LR?
Best,
M

Hi Moises, thanks for the fast reply. Certainly, here are the details:
The model is a RAVE V1 model with 32 dimensions (i've been using the rave_dims==32 for training and generation accordingly). I've been training the model with a higher than default num_signal value (5x) and capacity (1.5x).
The training set is roughly 1.5-2h. Going back to the logs, it seems, however, that only two 1MB .npy files have been created in the latent folder, considering the file names, it seems like the majority from the files have been skipped for some reason. I missed that on my earlier investigation. I've used a latent_length of 8192 in dataset preparation.
The training has been per default configuration, i've only changed save_interval to 200.
Does that help narrow it down for an intuition of what's happening?
Thanks and all best,
Martin

Yes, it seems your latent dataset is too small. By default the preprocessing script skips audios which are smaller than the context window size. So considering you're using a context window of 8192, the audios should be at least 349.52 seconds long (or 5.84 minutes) at 48Khz.
Hope this helps.

That did the trick. Reducing latent_length to 2048 -> 13 epochs in and we're at a loss of 1.4 already. Thank you for the clarification.


Document and add tests for static cephfs PV definitions

With the change in https://github.com/ceph/ceph-csi/issues/382 we got in static PV support for CephFS, and removed the older scheme that made provisioning a no-OP.
The means to use static CephFS PVs with ceph-csi needs to be documented and also test cases added to ensure we do not regress the functionality.
This issue is raised to track the same.
Initial sample yaml that helps create a static-PV and use it in a pod is provided here.

we need to add support for rbd also.

Could you guys add documentation for static pv in external cluster pls?

@agarwal-mudit whould you like to take this up?

@agarwal-mudit would you like to take this up?

Sure, assign please.

you can refer https://github.com/ceph/ceph-csi/blob/master/docs/static-pvc.md. we already have doc for RBD

closing as this is fixed in #1059


Just want to test if GitHub Actions if they are in the "Workflows" folder instead of "workflows"

Checklist
General

[x] I locally pass all tests or calculated manually the correct solutions on paper.
[x] I chose a title conforming to the naming conventions for pull requests.
[x] The newly added tests do not reveal the solutions dynamically calculating the solutions as expected in the exercise.

Description

Review Progress

Review

[ ] Review 1
[ ] Optional Review 2

We will see it in #2 :)


Gaborish filter regression

Describe the bug
As noted by eddie.sato on this message https://discord.com/channels/794206087879852103/803645746661425173/1225342521014485103
The generation loss of the jxl exibit an issue which might be induced by a regression on the gaborish filter (encoding part, as stated by _wb_)
To Reproduce
Encode and decode the same image twice in a row
Expected behavior
The second encoding should insert a minimal distortion
Screenshots
genloss.webm
Environment
Additional context
cf. the dicussion following the post on discord:
_wb_ — Looks like jpegli zeroes AC quite quickly. And inv_gaborish(gaborish) should probably be made closer to the identity function.
_wb_ — [...] the encoder side gaborish has been changed since then, I suspect causing a regression in terms of generation loss.

Realistically, who is going to save the same image in the same format 10+ times? If you are a photo editor, you use an intermediate format, such as PSD, KRA, XCF, TIFF.

Indeed, the use case is useless but:

This regression may hide a quality issue, gaborish and gaborish inverse filters should match as much as possible.
low generation loss was an argument put forward since the first releases of the format. It will be benchmarked by supporters and detractors.

if the origin of the issue is a regression, everybody could benefit from correcting it

Repeated recompression does occur in practice. Memes typically get shared in lossy formats, and can get recompressed dozens of times because:

the image goes from one social media platform to the other, getting recompressed every time
the image gets copy/pasted, which often has the effect of copying the decoded pixels, not the original compressed format
the image gets edited (e.g. the photo remains the same but the caption changes)

When I look at some of the memes I encounter in the wild, they do exhibit the kind of artifacts corresponding to dozens or even hundreds of generations.
Also if we want to use very high quality lossy as a plausible alternative to lossless in capturing/authoring workflows (e.g. using lossy JXL payloads in DNG), I would argue that avoiding generation loss is one of the key requirements.
So I think this issue should be investigated and I think we should re-tune the encoder-side Gaborish to optimize not the first generation but the Nth one.


Not able to run my own yolov7 inference model

Dear team,
thank you for your time. I downloaded the hefs by get_hefs_and_video.sh, it works fine.
Then I tried with my own inference model, here is the error message:
[HailoRT] [warning] HEF was compiled assuming clock rate of 400 MHz, while the device clock rate is 200 MHz. FPS calculations might not be accurate.
[HailoRT] [warning] HEF was compiled assuming clock rate of 400 MHz, while the device clock rate is 200 MHz. FPS calculations might not be accurate.
-I-----------------------------------------------
-I-  Network  Name
-I-----------------------------------------------
-I-  IN:  yolov7_tiny/input_layer1
-I-----------------------------------------------
-I-  OUT: yolov7_tiny/conv43
-I-  OUT: yolov7_tiny/conv51
-I-  OUT: yolov7_tiny/conv58
-I-----------------------------------------------

-I- Started write thread: yolov7_tiny/input_layer1 (640, 640, 3)
-I- Started read thread: yolov7_tiny/conv43 (80, 80, 21)
-I- Started read thread: yolov7_tiny/conv58 (20, 20, 21)
-I- Started read thread: yolov7_tiny/conv51 (40, 40, 21)

-I- Starting postprocessing

Config file doesn't exist, using default parameters
[HailoRT] [error] CHECK failed - yolov7_tiny/input_layer1 (H2D) failed with status=HAILO_TIMEOUT(4), timeout=10000ms
[HailoRT] [error] CHECK_SUCCESS_AS_EXPECTED failed with status=HAILO_TIMEOUT(4)
[HailoRT] [error] CHECK_EXPECTED failed with status=HAILO_TIMEOUT(4)
[HailoRT] [error] CHECK_SUCCESS failed with status=HAILO_TIMEOUT(4) - HwWriteElement15_yolov7_tiny/input_layer1 (H2D) failed with status=HAILO_TIMEOUT(4)
[HailoRT] [error] CHECK failed - yolov7_tiny/input_layer1 (H2D) failed with status=HAILO_TIMEOUT(4), timeout=10000ms
[HailoRT] [error] CHECK_SUCCESS_AS_EXPECTED failed with status=HAILO_TIMEOUT(4)
[HailoRT] [error] CHECK_EXPECTED failed with status=HAILO_TIMEOUT(4)
[HailoRT] [error] CHECK_SUCCESS failed with status=HAILO_TIMEOUT(4) - HwWriteElement15_yolov7_tiny/input_layer1 (H2D) failed with status=HAILO_TIMEOUT(4)

To be more specific, there is no detection while running.
Do you have any idea how to address this issue?
Best,
Hui

Hi @HY1998CN,
You are using a model that is trained on 21 classes, while by default the example is considering a 80 classes model.
To change that, run the inference example with the flag --classes 21.
Regards,

Hi @HY1998CN ,
The example is using the default yolov7.json in the folder that have 80 classes, and is loaded when you run the run command with the flag 'arch=yolov7'. Please try running the example with the arch flag.
In addition, the yolov7.json to that is used have 80 classes. To have the correct detection labels, please change the json file to the correct classes your model was trained on.
Regards,

Hi @omerwer ,
Thank you for your reply. I modified the anchors and labels in yolov7.json according to our model and loaded the config file while running. However, we still have this error message.
Best,
Hui

Hi @HY1998CN
Can you paste the HEF file of your model?
Plus, can you check if your model is successfully inferenced with out Python yolo inference code here:
https://github.com/hailo-ai/Hailo-Application-Code-Examples/tree/main/python/yolo_general_inference
Regards,

Hi @omerwer ,
Thank you for your reply. We have successfully compiled the hef model with previous version of Hailo Software Suite. https://github.com/hailo-ai/hailo_model_zoo/issues/60#issue-1807547366
Best,
Hui


Fix Policy Recommendation E2E test in Kind Cluster

Kind cluster introduced a new Pod local-path-provisioner recently which
causes Policy Recommendation E2E test because of recomming extra Network
Policies.
To fix test, we add the new Pod's namespace local-path-storage
into the argument when starting the recommendation job in Kind cluster.

/theia-test-e2e

/theia-test-e2e

/theia-test-e2e

/theia-test-e2e

/theia-test-e2e

/theia-test-e2e


Avoid dead links in Component status table

The automated generation of the Component status table from the yaml file is not sophisticated enough for components that does not have a separate page in components folder.
See Icon and Typography at the bottom, and Select that does not yet exist.
Find a more flexible approach to avoid links that redirects to the frontpage.

Outdated


client runs into a deadlock when Redis goes down

Hi,
After adding your DisableRetry suggestion in https://github.com/redis/rueidis/issues/680, my demo program started to work. But unfortunately, there's still some problem with the real application.
I have two relevant stack traces for you:
1:
2 @ 0x4788ae 0x45415d 0x454134 0x47a0c5 0x48e748 0xf95e4b 0xf97334 0xf9732f 0xf7c503 0xfb8667 0x18b7a62 0x18b08ac 0x222bf6d 0x2713eec 0x2705999 0x2705f19 0x2640fb6 0x244fa7f 0x7851a9 0xc3d8c2 0x23bfde4 0x7851a9 0x7a54ae 0x78b751 0x757095 0x47fe81
#	0x47a0c4	sync.runtime_Semacquire+0x24														/home/agniva/gosource/go/src/runtime/sema.go:71
#	0x48e747	sync.(*WaitGroup).Wait+0x47														/home/agniva/gosource/go/src/sync/waitgroup.go:118
#	0xf95e4a	github.com/redis/rueidis.(*mux)._pipe+0x34a												/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:147
#	0xf97333	github.com/redis/rueidis.(*mux).pipe+0x93												/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:175
#	0xf9732e	github.com/redis/rueidis.(*mux).DoCache+0x8e												/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:285
#	0xf7c502	github.com/redis/rueidis.(*singleClient).DoCache+0xe2											/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/client.go:141
#	0xfb8666	github.com/mattermost/mattermost/server/v8/platform/services/cache.(*Redis).Get+0x186							/home/agniva/mattermost/mattermost-server/server/platform/services/cache/redis.go:138
#	0x18b7a61	github.com/mattermost/mattermost/server/v8/channels/store/localcachelayer.(*LocalCacheStore).doStandardReadCache+0x61			/home/agniva/mattermost/mattermost-server/server/channels/store/localcachelayer/layer.go:492
#	0x18b08ab	github.com/mattermost/mattermost/server/v8/channels/store/localcachelayer.LocalCacheChannelStore.GetAllChannelMembersForUser+0x12b	/home/agniva/mattermost/mattermost-server/server/channels/store/localcachelayer/channel_layer.go:291
#	0x222bf6c	github.com/mattermost/mattermost/server/v8/channels/app.(*App).SessionHasPermissionToChannel+0xcc					/home/agniva/mattermost/mattermost-server/server/channels/app/authorization.go:85
#	0x2713eeb

1 @ 0x4788ae 0x439337 0x477ba5 0x4c0cc7 0x6371ab 0x63718f 0x6371ac 0x64ce5c 0x64c99b 0x6416f8 0x650aa7 0x650949 0x65094a 0x62c311 0x62bbce 0x62b2a7 0x62ad2f 0xfaab0c 0xfaaae4 0xf95063 0xf98679 0xf985b7 0xf94faf 0xf95f45 0xf97334 0xf9732f 0xf7c503 0xfb8667 0x18b7a62 0x18b08ac 0x222bf6d 0x2713eec 0x2705999 0x2705f19 0x2640fb6 0x244fa7f 0x7851a9 0xc3d8c2 0x23bfde4 0x7851a9 0x7a54ae 0x78b751 0x757095 0x47fe81
#	0x477ba4	internal/poll.runtime_pollWait+0x84													/home/agniva/gosource/go/src/runtime/netpoll.go:351
#	0x4c0cc6	internal/poll.(*pollDesc).wait+0x26													/home/agniva/gosource/go/src/internal/poll/fd_poll_runtime.go:84
#	0x6371aa	internal/poll.(*pollDesc).waitWrite+0x6ea												/home/agniva/gosource/go/src/internal/poll/fd_poll_runtime.go:93
#	0x63718e	internal/poll.(*FD).WaitWrite+0x6ce													/home/agniva/gosource/go/src/internal/poll/fd_unix.go:692
#	0x6371ab	net.(*netFD).connect+0x6eb														/home/agniva/gosource/go/src/net/fd_unix.go:141
#	0x64ce5b	net.(*netFD).dial+0x3bb															/home/agniva/gosource/go/src/net/sock_posix.go:124
#	0x64c99a	net.socket+0x29a															/home/agniva/gosource/go/src/net/sock_posix.go:70
#	0x6416f7	net.internetSocket+0xf7															/home/agniva/gosource/go/src/net/ipsock_posix.go:167
#	0x650aa6	net.(*sysDialer).doDialTCPProto+0xe6													/home/agniva/gosource/go/src/net/tcpsock_posix.go:85
#	0x650948	net.(*sysDialer).doDialTCP+0x68														/home/agniva/gosource/go/src/net/tcpsock_posix.go:75
#	0x650949	net.(*sysDialer).dialTCP+0x69														/home/agniva/gosource/go/src/net/tcpsock_posix.go:71
#	0x62c310	net.(*sysDialer).dialSingle+0x2f0													/home/agniva/gosource/go/src/net/dial.go:717
#	0x62bbcd	net.(*sysDialer).dialSerial+0x24d													/home/agniva/gosource/go/src/net/dial.go:682
#	0x62b2a6	net.(*sysDialer).dialParallel+0x3a6													/home/agniva/gosource/go/src/net/dial.go:583
#	0x62ad2e	net.(*Dialer).DialContext+0x6ae														/home/agniva/gosource/go/src/net/dial.go:574
#	0xfaab0b	net.(*Dialer).Dial+0x8b															/home/agniva/gosource/go/src/net/dial.go:500
#	0xfaaae3	github.com/redis/rueidis.dial+0x63													/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/rueidis.go:430
#	0xf95062	github.com/redis/rueidis.makeMux.func1+0x22												/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:67
#	0xf98678	github.com/redis/rueidis._newPipe+0x58													/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/pipe.go:91
#	0xf985b6	github.com/redis/rueidis.newPipe+0x16													/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/pipe.go:83
#	0xf94fae	github.com/redis/rueidis.makeMux.makeMux.func2.func3+0x2e										/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:71
#	0xf95f44	github.com/redis/rueidis.(*mux)._pipe+0x444												/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:152
#	0xf97333	github.com/redis/rueidis.(*mux).pipe+0x93												/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:175
#	0xf9732e	github.com/redis/rueidis.(*mux).DoCache+0x8e												/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/mux.go:285
#	0xf7c502	github.com/redis/rueidis.(*singleClient).DoCache+0xe2											/home/agniva/go/pkg/mod/github.com/redis/rueidis@v1.0.50/client.go:141
#	0xfb8666	github.com/mattermost/mattermost/server/v8/platform/services/cache.(*Redis).Get+0x186							/home/agniva/mattermost/mattermost-server/server/platform/services/cache/redis.go:138
#	0x18b7a61	github.com/mattermost/mattermost/server/v8/channels/store/localcachelayer.(*LocalCacheStore).doStandardReadCache+0x61			/home/agniva/mattermost/mattermost-server/server/channels/store/localcachelayer/layer.go:492

The second one is just about to time out. So that doesn't get stuck, but the first goroutine remains stuck forever.
The steps to reproduce it is very straightforward from the application, but unfortunately I am unable to create a self-contained reproducer for this.
I am using 1.0.50 version.

Hi @agnivade, are these the only two goroutines that get stuck? Is there any other goroutine that also gets stuck?

Those are the only goroutine stacks from rueidis. But hang on, now I am getting the timeout exactly after 34-35s always. It almost feels like there is a timeout of 30s being forcibly applied from somewhere.
I am attaching the full goroutine trace.
This is my client config:
client, err := rueidis.NewClient(rueidis.ClientOption{
	InitAddress:       []string{"<IP_ADDRESS>:6379"},
	Password:          "",
	SelectDB:          0,
	ForceSingleClient: true,
	CacheSizeEachConn: 32 * (1 << 20), // 32MiB local cache size
	DisableCache:      false,
	MaxFlushDelay: 250 * time.Microsecond,
	Dialer: net.Dialer{
		Timeout:   time.Second,
		KeepAlive: time.Second,
	},
	DialFn: func(dst string, d *net.Dialer, _ *tls.Config) (conn net.Conn, err error) {
		ctx, cancel := context.WithTimeout(context.Background(), time.Second)
		defer cancel()
		return d.DialContext(ctx, "tcp", dst)
	},
	DisableRetry: true,
	ConnWriteTimeout: time.Second,
})

And note that all my client.Do/DoCache calls use context.Background(). There's no timeout being set from there. And it's not that I don't see any errors at all in the logs. I do see i/o timeout errors. But some HTTP requests still take 30-34s to return.
all_stack_trace.txt
Let me know what you think.

Hi @agnivade,
Actually, I don't have an idea how can a goroutine stuck on the waitGroup for the pipe. It seems impossible to me.
I still try to come up with a fix v1.0.51-alpha.1, please let me know if it works.

Thanks @rueian. Please hold on for a moment. It looks like my application itself was making 30-40 calls from various Redis caches. That's why the whole request was taking so long inspite of each Redis request timing out after a second.
Let me double-check this and get back to you.

Yes, that was it. Apologies for the false alarm and wasting your time @rueian.


mac tests failing with "go build command-line-arguments: module requires Go 1.13"

https://source.cloud.google.com/results/invocations/1bebb69a-c64f-4d22-abb1-f9b21b8d49fd/targets/grpc%2Fcore%2Fpull_request%2Fmacos%2Fgrpc_build_artifacts/log
go build command-line-arguments: module requires Go 1.13
make[3]: *** [third_party/boringssl/crypto/err_data.c] Error 1
make[3]: *** Deleting file `third_party/boringssl/crypto/err_data.c'
make[2]: *** [third_party/boringssl/crypto/CMakeFiles/crypto.dir/all] Error 2
make[2]: *** Waiting for unfinished jobs....
[ 12%] Building CXX object third_party/abseil-cpp/absl/strings/CMakeFiles/absl_strings.dir/string_view.cc.o
[ 12%] Building CXX object third_party/abseil-cpp/absl/strings/CMakeFiles/absl_strings.dir/substitute.cc.o
[ 15%] Linking CXX static library libabsl_absl_strings.a
[ 15%] Built target absl_strings
make[1]: *** [CMakeFiles/grpc_csharp_ext.dir/rule] Error 2
make: *** [grpc_csharp_ext] Error 2

2020-01-09 12:13:25,484 FAILED: build_artifact.csharp_ext_macos_x86 [ret=2, pid=95593, time=188.0sec]

I've investigated and this is due to b/138784095
I'm working on a workaround / way to fix this.


Feature/as 3485 save statuses and dates

This PR covers both AS-3042 and AS-3485.
Update schema with new fields and ensure that the correct values are updated in tables, stored procedures, triggers and views.
Also updated the web-app to set the correct date and status fields when the Validation Officers clicks the submit button on the Check and Confirm page.
Also fixed some unrelated unit tests that were displaying an error in the test run.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add System.Numerics.Tensors library

Add a library to represent tensors with some basic mathematical and resize operations.  Tensors are stored as a single-dimensional array with C-style ordering of elements to aide in easy interop with native libraries without copying the array.  Currently a prototype and primarily a means for experimentation.
/cc @mellinoe @migueldeicaza

Tests failed with
The test source file "D:\j\workspace\windows_nt_re---4abe806d\tests\System.Numerics.Tensors.Tests\bin\Release\netcoreapp1.1\System.Numerics.Tensors.Tests.dll" provided was not found.

Something about my project configuration appears busted.  I'll see what I need to fix.

High-level commentary:

Have you tested the performance impact of this IArithmetic abstraction? Going through a couple levels of indirection, and through interface dispatch, seems like a lot of overhead just to start with for every single arithmetic operation. Given that most of the code is generated, and all internal details, it could be inlined.
Having some of the IArithmetic implementations throw NotSupportedException seems very bad. Is there a different way to express the concepts without these problems leaking out?
You mention interop with native libraries. Have you seen how well this works with CUDA/OpenCL/whatever?

Also, a high-level commentary. If the tensor support gets more developed and aims to target machine learning needs, please take a glance over PyTorch / TensorFlow / NumPy designs. These communities have been exploring most efficient platform-dependent ways to accelerate tensor operations (CUDA / OpenCL / BLAS / OpenMP / SSE) and tensor type polymorphism (including float16 types).

@mellinoe right now this is really just an experiment focused on the functional behavior, once that's in place we can play with different ways to represent the arithmetic ops.  I agree I want this stuff inlined where possible.  I haven't spent a ton of time looking into perf yet.  If you have a suggestion file an issue and tag it for the tensors area path.
As far as native libraries, we've experimented with directly passing these to our own native libraries using signatures like double* and int* and it works as expected without copying.  It'll create a memory layout that's consistent with a C-style multi-dimensional array so it should be usable by most native libraries.  We aren't intending for it to work directly with existing libraries, but be easy to use with folks who are developing libraries and want them to interop with C# with minimal work.  Taking a brief look at CUDA and OpenCL I see they use C-style multi-dimensional arrays so I believe this should work.

It may help to describe the storage layout as "row-major", or use some other appropriate term for higher-dimensional arrays. That would be more explicit than describing them as "C-style multidimensional arrays".
It looks like other similar libraries actually let you control the storage layout explicitly, noting that some algorithms may favor one storage order or the other. We could consider adding that flexibility here, too.

We aren't intending for it to work directly with existing libraries

Hm, I thought part of the motivation for this was to interface with ML frameworks like TensorFlow, etc. I guess what you mean is that we don't necessarily want to constrain ourselves to work with any particular existing framework at the expense of the API itself?

@mellinoe right now this is prototyping we'll do what is necessary once we discover what our real requirements are.  We don't want to go engineering a bunch of knobs yet until we know what folks want to use.  Plumbing different layout/indexing schemes is a significant amount of work, and it isn't clear to me if there is a real need for differentiation.  I'm refactoring this stuff daily as I understand more about requirements and we may bring in the different layout schemes as needed.  Right now I'd like to provide something and see if its useful for folks and get feedback.  WRT to naming I use the C-style term as that is what python uses as well, which differentiates C-Style (first-to-last-dimension-order, aka row-major) vs Fortran-style (last-to-first-dimension - aka column major).  I have some existing folks willing to use this that are fine with C-style ordering which is why I started there.
My meaning behind not making this work with existing libraries is we aren't trying to make this look like an existing technology-specific exchange type.  We're trying to create a single .NET exchange type for tensors that is compatible with as many libraries as possible.  As such we'll start with a feature set that we think is a good starting point and get feedback from the adopting libraries around what we need to add.

Folks I'm not expecting an in-depth code review of this stuff.  Most of it is all experiment/prototype quality.  The things I do want folks to review is the project setup and infrastructure.

Ubuntu failing with Reason: Unhandled Exception: System.IO.Pipelines.Networking.Libuv.Interop.UvException: Error -16 EBUSY resource busy or locked
@dotnet-bot test Innerloop Ubuntu16.04 Debug Build and Test
@dotnet-bot test Innerloop Ubuntu16.04 Release Build and Test

Ubuntu failing

This failure has been recurring (and intermittent). I have filed an issue: https://github.com/dotnet/corefxlab/issues/1687
Thanks.

Unless there is further feedback I plan to merge this so that I can do further work: https://github.com/dotnet/corefxlab/issues?q=is%3Aopen+is%3Aissue+label%3Aarea-System.Numerics.Tensors


feat: custom conv_template

In case we don't want to provide the conv_template name, we can use custom and pass every the entire ConvTemplateConfig data instead.

That's useful!
@tqchen, do you plan to update https://www.npmjs.com/package/@mlc-ai/web-llm with this change soon?


send HTTP requests to the API

This tool will have to translate the selections a user makes on the UI into a (single) HTTP request that is sent to the bagelAPI, so that the bagelAPI can in turn make the corresponding SPARQL query to the graph and return the results to this tool. This tool then needs to be able to display the results it has received from bagelAPI in some way that is interesting to the user.

I think it makes more sense to address this as part of each sub-issue that is tracked in #2 because they will all have to be hooked up to the API request individually. I will close this issue.


Asset-level source files

Description
eLife allows source code and source data files to be directly associated with figures and figure supplements, tables and videos.
User Stories
Author

As an author, I want to be able to clearly see any source data or code associated with assets (figures, figure supplements, videos or tables) so that I can be sure that they are associated correctly and that none are missing.
As an author, I want to be able to edit the title and legend for any source data or code associated with assets so that I can ensure that my work is accurately represented for publishing.
As an author, I want to be able to download any source data or code file so that I can check that the correct file is being used.
As an author, I want to be able to replace source data or code files so that my manuscript is up to date and ready for publication.

Production staff

As production staff, I want to be able to remove, replace, add or re-allocate source data or code on behalf of the author during the proofing process, so that I can correct association errors before publication.
As production staff, I want to know why an author has changed a source data or code file so that I know reasons for the change.
As production staff, I want to provide an explanation for why I am replacing a source data or code file so that I can record the reason for the change.

But what if . . . ?

Considerations

Labelling example, in context of figure and figure supplements:

Figure 1

Figure 1—source data 1
Figure 1—source code 1
Figure 1—figure supplement 1

Figure 1—figure supplement 1—source data 1

Figure 1—figure supplement 2

Figure 2

Figure 2—source data 1

As with main figures, we don't want authors to be able to sweepingly decide to delete or add source data files. These kinds of changes may need editorial input, so we need to be able to exercise a degree of arbitration.
We would expect the hierarchy/order of files in the additional files section to follow this logic:

Asset level source files follow the hierarchy of the asset in question (i.e. figure supplement source data is placed after figure supplement source data)
Asset level source files appear before article level source files

The title for the section containing the source files would have to be editable.

XML requirements
Asset level source data and code should be captured in the additional files section in the back:
<back>
     ...
     <sec id="s6" sec-type="supplementary-material">
          <title>Additional files</title>
          ...
          <supplementary-material id="fig3scode1">
               Figure 3—source code 1.

                    <title>Custom python scripts used to generate Figure 3A-C</title>

               <media mime-subtype="zip" mimetype="application" xlink:href="elife-44527-code1-v3.zip"/>
          </supplementary-material>
          <supplementary-material id="fig3s3sdata1">
               Figure 3—figure supplement 3—source data 1.

                    <title>Detailed counts of cells quantified in <xref ref-type="fig" rid="fig2s3">Figure 2—figure supplement 3</xref> in the different experimental conditions.</title>

               <media mime-subtype="xlsx" mimetype="application" xlink:href="elife-44527-fig3-figsupp3-data1-v2.xlsx"/>
          </supplementary-material>
          <supplementary-material id="video1sdata1">
               Video 1—source data 1.

                    <title>Rho mutations from all replicate evolution experiments.</title>

               <media mime-subtype="xlsx" mimetype="application" xlink:href="elife-44527-data1-v1.xlsx"/>
          </supplementary-material>
          <supplementary-material id="table4sdata1">
               Table 4—source data 1.

                    <title>Thermograms and normalized plotted data from ITC titration of TRAP220 peptide into PPARγ LBD.</title>

...

               <media mime-subtype="pdf" mimetype="application" xlink:href="elife-44527-table4-data1-v2.pdf"/>
          </supplementary-material>
          ...
           <supplementary-material id="sdata1">
               Source data 1.

                    <title>Source data for the whole article.</title>

...

               <media mime-subtype="pdf" mimetype="application" xlink:href="elife-44527-data1-v2.pdf"/>
          </supplementary-material>
     </sec>
     ...
</back>

This should then be referenced with an xref element, which is a direct child of the respective element (fig, table-wrap, or media). The xref should contain the content of the label for the supplementary-material it points to:

     ...
     <fig-group>
          <fig id="fig3" position="float">
               Figure 3.

                    <title>...</title>

...

               ...
               <xref ref-type="supplementary-material" rid="fig3scode1">Figure 3—source code 1.</xref>
          </fig>
          <fig id="fig3s3" position="float">
               Figure 3—figure supplement 3.

                    <title><italic>Hmx3</italic>; tdTOM+ cells negative for <italic>Htr3a</italic>-GFP often express glial markers.</title>

...

               <xref ref-type="supplementary-material" rid="fig3s3sdata1">Figure 3—figure supplement 3—source data 1.</xref>
               ...
     </fig-group>
     ...
     <media mimetype="video" mime-subtype="mp4" id="video1" xlink:href="elife-00666-video1.mp4">
           Video 1.

                 <title>A description of the eLife editorial process.</title>

...

          <xref ref-type="supplementary-material" rid="video1sdata1">Video 1—source data 1.</xref>
     </media>
     ...

          Table 4.

               <title>ITC analysis of TRAP220 peptide titrated into PPARγ LBD.</title>

...

          <xref ref-type="supplementary-material" rid="table4sdata1">Table 4—source data 1.</xref>

...

...

Mock ups
This is not required, but if you have mock ups of what you would like to see please provide them here.
Proposal
This will be added by the Texture team after the feature request is discussed and agreed.

Labelling example, in context of figure and figure supplements:
Figure 1
Figure 1—source data 1
Figure 2
Figure 2—source data 1

Please also take into consideration, that a data file might be the source for two or more figures. This is particularly common if e.g. a larger table of data is processed in different ways, such as using different columns.

@oliver---- This is why I'd like the tickets to be combined - that's not an eLife use case, but is a SourceData use case

https://groups.niso.org/apps/group_public/view_comment.php?comment_id=830&add_comment=1

@Melissa37, @oliver----  - On reviewing the requirements/user stories that @source-data have in #1245, I think it is important that we distinguish two different kinds of data that can be associated with figures: files that are going to be included in the DAR file, and external data on an FTP site, a data repository etc. This is not something that I expect to be reflected in the user experience, but it does mean we have the case where a file is being directly included (which is what eLife's requirements are centred around) and the case where the data is represented by a link to some external site. I think these will likely need different underlying XML solutions.
Given that the other user stories on this ticket are centred around the case where the files will be included in the DAR, I've summarised what I think are @source-data's user stories for this case here. The requirement to also be able to associate source data on external sites with figures and panels should, I think, be captured in a separate ticket, even if the UI ends up similar.

@Melissa37, @oliver---- - On reviewing the requirements/user stories that @source-data have in #1245, I think it is important that we distinguish two different kinds of data that can be associated with figures: files that are going to be included in the DAR file, and external data on an FTP site, a data repository etc. This is not something that I expect to be reflected in the user experience, but it does mean we have the case where a file is being directly included (which is what eLife's requirements are centred around) and the case where the data is represented by a link to some external site.

We already have support for external supplementary files. Instead of a file (included in the DAR), those take a URL. Both are represented as supplementary-material elements in JATS.

We already have support for external supplementary files. Instead of a file (included in the DAR), those take a URL. Both are represented as supplementary-material elements in JATS.

@michael that's great!
For eLife we don't list them in supplementary material but have http links or put them in references, but I assume that's not a problem either.

'as a Collaborator (I am not sure how this role is defined; we had the role "Recipient" defined in our doc which is similar to a 'reader') I want a separate View where all the Files are listed with their associations.'
This may or may not have implications on the tagging but it could put some constraints on how the tagging at the figure or panel level can be used to render an overview list.

Hi @source-data - thanks for commenting. Sorry, I misunderstood what you meant by 'recipient' previously. Would it be better to use 'reader' in these user stories? At a glance, it looks like all recipients of the data (either collaborators with the authors or post-publication readers) would count as readers for the purposes of the Texture interface?
The user story in your comment sounds like a solution to "As a [collaborator/recipient/reader], I want to be able to clearly see which data files are associated with a figure or panel so that I can see the source data for the images" rather than a separate requirement. Is that right? Or should we have another user story that is something like, "I want to be able to see all the files at once so that I can study the source files for the entire article"?

Here is a updates list of user stories. The description is also updated below:
Linked source files
EMBO SourceData allows files to be associated with the whole document, individual or multiple figures,  individual or multiple figure panels.
User stories

As an author, I want to link or unlink one or several files with one or several panels of a given figure so that datasets, protocols, computer scripts related to the results depicted in the linked panels are available to those who will use the panels.
As an author, I want to link or unlink one or several files to one or several figures so that datasets, protocols, computer scripts related to the results depicted in the linked figures are available to those who will use the figures.
As an author, I want to associate one or several data files to the whole document so that datasets, protocols, computer scripts related to the results depicted in the linked figures are available to those who will use the document.
As an author, I want to add or edit a title to a file associated to panels, figures or document so that I can provide a short description of the content and nature of the file (data, protocol, scripts, …).
As an author, I want to link or unlink a local file (to a panel or figure or document) so that I can easily share in a lossless manner the document and its associated files.
As an author, I want to link a remotely hosted file (to a panel or figure or document) so that I can share a document associated with files that would be not practicable to move around in particular because of size, legal or controlled access issues.
As an author, I want to link files hosted on a public database so that I can easily use database accession numbers.
As an author, I want to provide an overview of all the linked files so that I and readers can have an overview of the number and nature of the files and easily visualize which panel or figure depicts the corresponding results.
As a reader, I want to easily access the files associated with a specific panel or figure so that I can use them locally.
As a reader, I want an overview of all the linked files so that I see how many files are linked, what is their nature and size and easily identify and access the panels or figures that depict the corresponding results.

Considerations:

The author user stories take into account both the authoring and editing context.
Linking to local or remote files will require different implementations. However from the point of view of an author and reader user experience, it may not need to be fundamentally different. To keep in mind a general solution, I kept this in the user stories above even if it is understood that this may require separate planning.
The co-packaging of files in the .dar archive is a particular implementation of how the document and associated files can be easily shared. 'Easy sharing' may then include sending the archive as an attachment by email or posting on sharing tools similar to DropBox etc... For very large files, 'easy sharing' may not involve co-packaging of data files but rather linking to downloadable resources.

Hey @NickDuf do you have any comments on these user stories and considerations before I integrate them into the ticket?
@FAtherden-eLife do you have any amendments to make to the XML?
Thanks!
M

@FAtherden-eLife do you have any amendments to make to the XML?

@Melissa37, presumably I do, but I'm not sure exactly what's required here.
@source-data, for this user story:

As an author, I want to link a remotely hosted file (to a panel or figure or document) so that I can share a document associated with files that would be not practicable to move around in particular because of size, legal or controlled access issues.

Is the requirement here just to be able to add an ext-link in the caption or something more akin to the xref for local files? If the latter, please can you let us know if any markup has been proposed/decided upon so that this can be included in the ticket above. (I can't see any XML samples for this in #1245)

@oliver---- and @michael need to decide on the suitable xml implementation. In terms of the underlying tagging, it seems to me more akin to ext-link since it refers to external resources. I don't see how xref could be used in this context, but my knowledge is imited.
In terms of UI implementation for the authoring scenario, linking to remote or local files will most likely be somewhat related since the reasons indicated in both user stories are indeed similar: "so that I can easily share in a lossless manner the document (the figure) and its associated files". It should be made clear to the user whether s/he is linking to a remote or local file, but the purpose from the user's point of view is simialr. So, it needs some more specialized implementation than just instering some link somewhere in the caption. It needs to make it clear that this is THE data for THIS particular panel.


Document usage of SRes::into_inner on the RenderCommand trait

Objective

Fixes: #7187

Since avoiding the SRes::into_inner call does not seem to be possible, this PR tries to at least document its usage.
I am not sure if I explained the lifetime issue correctly, please let me know if something is incorrect.
Solution

Add information about the SRes::into_inner usage on both RenderCommand and Res

Me and @ErickMVdO reviewed this, and for us who don't understand much of this code, it looks great and explanatory.


Import data from "Where Is It?" software

Greetings! For the users of "Where Is It?" software, it would be essential to have an import feature in librer where it can read the data contained in its .CTF catalog files and make it available to search.
For example, WinCatalog has this import feature: https://www.wincatalog.com/whereisit-export.html

Hi. It definitely looks doable (if we mean importing from an .xml report file and not a .ctf one). I used "Where is it" myself many years ago, in the era of CDs and DVDs. I'll think about that.

librer v1.0092.584
stay tuned.

https://github.com/PJDude/librer/releases/tag/v1.0096.566


feat: support for case-insensitive matches

Implements WICG/urlpattern#168
Seems there are no WPTs yet

generate_matcher is going to significantly suffer here. It will be unable to generate matchers for case insensitive parts

thats a problem, but i dont think there is much we can do (I'd be happily proven wrong)

@crowlKats @lucacasonato - Any chance that this might land in Deno? According to https://developer.mozilla.org/en-US/docs/Web/API/URLPattern#browser_compatibility Deno has everything else implemented according to the spec except for the ignoreCase feature.


Cleaned up package.json deps

Updated dep versions (starting with https://github.com/Netflix/falcor/pull/468), and adding to it.
Removed unused deps, and made wildcards consistent (^ for devDeps, fixed versions for product deps)
Tested all npm run tasks, and main gulp tasks.

This PR replaces https://github.com/Netflix/falcor/pull/468 so will close that one out.

+1


Control Node jumps to right by screen width when building the game

Tested versions
Produced in both 4.2.1 and 4.3d4 Mono versions
System information
Godot v4.2.1.stable.mono - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2060 (NVIDIA; <IP_ADDRESS>52) - AMD Ryzen 5 3500X 6-Core Processor (6 Threads)
Issue description
I have structure like this.
MainScene (2D Node) -> Canvas Layer -> [ Control -> Panel ]
Brackets part above is a scene of its own, and has other unrelated nodes both under the Control and Panel nodes.
When I'm in the control scene and change Panel position and anchor it to top right and start the game, no problems occur. When I change the scene to the main one and then I start the game, position for the panel changes. (position in the control .tscn file changes as soon as I hit F5)
This change is always in the X axis, and is the width of the default viewport width I set in the project settings, in my case 628px.
I have my AnimationTree that controls only the size of the panel, but I disabled it just in case. Still the same.
Here is a video of the problem
Update: I found out that the problem was actually caused by the animations, even if my AnimationTree is disabled. Problem still persists of course, for some reason. I just tried recreating the same panel from beginning, everything was working until I made the collapse animation, which only changes the size property of the panel. When I created this animation, panel started jumping again.
I added position, anchor and other properties I tought my be the problem to the RESET track, nothing changed. Still the same problem.
Steps to reproduce
I could not produced the problem when I created a new project with the same version. So I'll just leave the repository for the project, which is really small as I just began. Just clone the repo, go to prefabs/WorldGenUI.tscn, move the "PlayerInfoBox" to top right if jumped again, then go to scenes/MainScene.tscn, hit F5 and you'll see.
Minimal reproduction project (MRP)
Direct link to the project files

This may be the same issue present in this reddit thread. Two users there report being able to fix it by removing the RESET animation from the AnimationPlayer.


Upgraded youtube_player_iframe plugin + Cropped out Youtube Thumbnail letter-boxing

Also swapped flutter_inappwebview with webview_flutter
as youtube_player_iframe depends on webview_flutter already.

Closing as this has already been implemented. Thx!


CM access the Moderator Mail and send custom reply if it's triggered.

I will explain better this as there are multiple situations where this can be handy.
1 FIRST

user JaneDoe  sends a moderator mail, without a link on it.

Cm search thru the posting history of janedoe
-- If  JaneDoe doesn't have a post made in the subreddit,  CM to send a custom reply: I couldn't find any post made in our community, if you have any questions regarding a post or a comment. Send us a replay that contains a proper link to that comment or post.
-- If JaneDoe - DOES HAVE posts made in our subreddit. CM to send A MODERATOR PRIVATE MESSAGE where to list the last posts JaneDoe made on the subreddit.

user JaneDoe sends a moderator mail With a link on it.
-- If  JaneDoe doesn't have a post made in the subreddit,  CM does nothing, Janedoe might be a subscriber tho can send us a report or something we need to know.
-- If JaneDoe - DOES HAVE posts made in our subreddit. CM to send A MODERATOR PRIVATE MESSAGE where to list the last posts JaneDoe made on the subreddit.

2 SECOND.
Some subreddits does request verification for their posters.
If there is a moderator mail with the title: Verification. ( or a trigger word )
CM should check the author history and in case of:

low account age: >=20 days
low link karma: >=300
low comment karma: >=30
cross posts: >=10
links in comments more than 95%
To send a custom Message: Unfortunately your verification was denied due to your account it's to new, it has less posting history, it looks spammy due to multiple crossposts ... Come back in a 30 days with a new verification.

Other criteria can be:

posting history less than 200 posts
7 yo account who started posting 2 months ago - signs of purchased account - ( I know this might be complicated )

CM also to be able to add a Note - Verification denied due to: - low comment karma ( I might be to much already )
If all the criteria are fulfilled - CM can send a message as the subreddit like - Hey {{author}} we received your Verification - please  have patience until a human moderator checks the verification album and reaches back to you. Thank you.
This sounds like its a work for a different bot. For me sounds MORE than complicated but it would be amazing to have this implemented  ... as it would save us from a lot of time spend on browsing thru profiles and rejecting tons of verification from shady accounts.
To be honest I'm more interested in the second part of this suggestions as this would help me and fellow moderators who use verification on their subreddits a lot. As we all have criteria that makes an account eligible or not for verification.
Thank you

CM interaction with modmail has been on my todo list! Reddit's API for modmail is different than everything else though so I need to familiarize myself with it first and figure out how best to translate it to easy to do things with CM.
All of these things are things I'd like CM to be able to do eventually!


ungzip on the fly if client doesn't accept gzip encoding

Rather than have to keep a unzipped copy of every static file on the server, and ensure it is kept in sync, could the server dynamically uncompress the gziped file if the 'Accept-Encoding' header doesn't accept gzip? This may even be faster in cases where the disk is the bottleneck - ungzipping doesn't require that much cpu or memory.

So basically an evil twin of compress midleware ;-)
Quite frankly I don't think it has a place here. connnect-gzip-static is meant to be super simple (and thus fast), at the price of using extra disk storage. It's also meant to be used if your build process already takes care of compressing files.
That of course that should not discourage you from forking and publishing your own version.


[ERROR]: milvus[client] doesn't exist (for Milvus Lite)

Is there an existing issue for this?

[X] I have searched the existing issues.

Issue
https://github.com/milvus-io/milvus-docs/blob/v2.2.x/site/en/getstarted/milvus_lite.md?plain=1#L64
PyPi returns an error when looking for milvus[client]
Suggestion
maybe just get rid of that bit of code
Anything else?
No response

cc @XuanYang-cn


user ssl files need to be present

user ssl files need to be present or they won't be updated on cert changes, ran into this testing in docker.

@koaps a bit late to ask, but would you mind adding tests for this?

Can you rebase?

Closed in favor of #307


refactor(rpc): allocate PingPongLatencyTimer on start

Followup to #2792 which closed #2771.
#2792 does not handle the case where Start is never called. If Start is not called, Stop returns an error, thus with #2792's implementation the only way to ensure that PingPongLatencyTimer is cleaned up is to call Start and Stop, even when not using any of the features provided by Start (i.e. events).
This PR moves initialization of PingPongLatencyTimer into OnStart so that it is only initialized if it is going to be used. This PR also moves cleanup of PingPongLatencyTimer into readRoutine's defer statement to align it with other cleanup (i.e. closing the connection).
PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

Another example for https://github.com/cometbft/cometbft/issues/2041.


C89 compatibility (the old ANSI standard)

Happy to up the warning and permissive level and make this even more strictly compliant, but thought this was a trivial start that should make it C89 compatible enough.
Advantages:

Support MSVC (no version of C newer than C89 is fully supported in any version of MSVC)
Support older compilers targeting esoteric hardware (relevant to your IoT selling-point)
Support FFI layers (almost no popular FFI layers in other language supports a standard newer than C89)

Disadvantages:

None really, as you're not [really] using any newer features anyway.

PS: Just say the word and I'll go strict-mode C89 support.

EDIT: Strict-mode C89 support is mostly done.

@danewalton ?

Is there interest in this support? - It'll increase cross-platform cross-toolchain support, and FFI.

Hi @SamuelMarks
I will let @RickWinter give the ultimate decision here but from the beginning, being C99 compliant was a core tenet of this repository.

While we appreciate the work done here, even if we take these updates, there is no guarantee that in the future, all other parts would be built to be C89 compliant; some parts would work and other parts broken. The user experience for that would not be great and we would break one of the core tenets. Again, I will let others weigh in but that is what I believe the answer will be.

Hi @SamuelMarks, thank you for taking the time to propose this PR and change.  As Dane mentioned we target C99 and had lengthy discussions about that target.  Although your change is relatively straight forward it does bring complexity to the code base and future maintenance/support costs which we do not want to take on.
That said there was one part of the change which we would accept,
typedef void (*az_precondition_failed_fn)(void);
If you'd like to break that out into a PR we would gladly accept that contribution.


Purge agent connections when node is unhealthy

Coverage increased (+0.03%) to 92.283% when pulling 2ba514ae59e76ba4c2818a4f59195ab4a89ab86f on mprimi:GENIE-485-purge-agent-connections into 42537f2f0c404ae6dc4c940435b16f8713e1665d on Netflix:master.


[consul/stable] Added support for multi-cluster consul deployments.

What this PR does / why we need it: This does three things because you need both for this to work right.
1: Adds support to create the secret for a pre-defined gossip key. (gossip.create: true)
2: Adds support to add arbitrary hostnames to join as either peers or as WAN peers. This allows the deployment of multi-cluster consul deployments.
3: Adds support to define an arbitrary top-level domain to the cluster (instead of cluster.local).
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

/assign @foxish

@linki @lachie83 @foxish : Is there something we're waiting on to get this merged?

Hi! What about merging this pr?

I've tested this pull request in my cluster - it works. Consul joined to another consul cluster.

Ping! It's still needed! :)

Me need this feature too, it's great! 👍


test: major examples refactoring for docs, unit tests and e2e tests

Description
This PR implements the possibility to create examples that support the following:

generate  documentation pages
can be used in the playground
generate unit test spec files
generate end-to-end tests for Google Apps Script

Type of change

[x] Tests and documentation

How has this been tested?
Several new test examples have been added:

[x] actions/actionAttachmentExtractText.ts
[x] actions/actionExport.ts
[x] actions/actionThreadRemoveLabel.ts
[x] basics/simple.ts
[x] features/convertToGoogle.ts
[x] migrations/migrationAdvanced.ts
[x] migrations/migrationMin.ts
[x] regressions/issue301.ts

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing tests pass locally with my changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

573 of 601 (95.34%)  changed or added relevant lines in 11 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.08%) to 91.287%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/lib/Context.ts
1
10
10.0%

src/lib/e2e/E2E.ts
29
48
60.42%

Files with Coverage Reduction
New Missed Lines
%

src/lib/Context.ts
3
94.34%

src/lib/processors/BaseProcessor.ts
3
77.57%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
8579

Relevant Lines:
9266

💛  - Coveralls


App cannot be installed by certain 16 year olds

As discussed in #632, due to the age requirement in the app store being set to 17+, certain 16y olds, who have age restrictions on their device, cannot download the app.
This is a bug. It's unwanted behaviour.
The reasons given by @mynchau for closing are weak and were not properly discussed.
Since the Play store age requirement is 0+, legal requirements, as claimed, cannot be the reason for the need to set the age requirement to 17+.
Should be of interest to at least @an3k @Christian-Hemker @jdslab @thomasaugsten @jlnprssnr @fpientka

The way I understand it is: iOS age restrictions are set on device, and installation of 17+ apps can be explicitly allowed in iOS Settings in Screen Time -> Content & Privacy Restrictions -> Content Restrictions -> Apps. This can also be done temporarily, just for downloading this single app. Could be at least a workaround.

Thanks for your comment @jdslab. A workaround doesn't fix the underlying root cause though, which is the setting 17+. The reasoning from maintainers for closing #632 was very weak in my opinion.

I think it is fair to use 12+ (as WhatsApp does) because there are no GPDR data used. Even if iOS age restrictions is deactivated for installation of 17+ apps, it disappears and is unusable if activated again

@jdslab as mentioned by me before, this age restiction prohibilts people that are 16 years old from downloading the app. https://apps.apple.com/sa/story/id1440847896 It's not just a preference on the phone.

@jdslab as mentioned by me before, this age restiction prohibilts people that are 16 years old from downloading the app. https://apps.apple.com/de/story/id1440847896 It's not just a preference on the phone.

This is what the description from Apple looks like. Sounds like Corona-Warn-App is a Sex or Drug app... And it says in bold you MUST be 17+
So this clearly underlines how this choice of age is highly misleading and harmful.

@jdslab as mentioned by me before, this age restiction prohibilts people that are 16 years old from downloading the app. https://apps.apple.com/de/story/id1440847896 It's not just a preference on the phone.

I see, never mind then

As I wrote in the other issue: Just set it to "0 years" because

The Apple App Store age restriction settings has no legal meaning in Germany as they are ESRB ratings from the USA, thus the SAP employees fear of breaking the law because of the DSGVO is non-existent.
The "17+" setting was made because of the DSGVO which requires you to be 16 years or older but the App itself has an integrated age check especially because of the DSGVO, thus the Apple App Store age setting is not necessary.
The age setting in the Google Play Store was set to "0" because it uses the USK ratings. That however doesn't overwrites the DSGVO age restriction in any way so if 0 years is possible there, it is possible in the Apple App Store too.
People younger than 18 years cannot use the app if their parents enabled youth protection settings even if their parents give their permission because iOS prevents you from using this app (because it's rated "17+"). Only choice for parents is to allow "17+" content or to completely disable youth protection settings. In both solutions would allow eg. 12 year olds to access content for 18 year olds just because of this app.

If you understand German you can read it here https://github.com/corona-warn-app/cwa-app-ios/issues/632#issuecomment-645605146

Dear corneliusroemer, thanks for bringing up that issue. However, please understand that the representation in the app stores is the exclusive responsibility of the Robert Koch Institute. Please reach out to them at<EMAIL_ADDRESS>or via the other contact options listed on https://www.rki.de/EN/Service/Contact/Contact_node.html
Since we are the wrong addressee we're closing this issue here.
Best regards
MC
Corona Warn-App Open Source Team

Disclaimer: This is not a personal attack. I am stating my honest opinion as a member of the community that is advertised as a big part of this project.
I disagree with the way maintainers deal with this issue. To me it looks like deflecting responsibility.
You may be right that it's up to the RKI to change this. But this is not how you phrased it in the past.
Also, because this is a semi-technical issue, it is correct for it to be discussed here. You can say it is nothing that SAP/Telekom/maintainers can fix, but please do more about it by notifying RKI of this issue yourself.
I consider it a responsibility of contractors to inform the client of potential mistakes they are making.
Can you confirm that you have escalated this issue with RKI? There's a big difference between me writing as a lone citizen and a community-raised issue that has a community consensus.
I would like to invite community members @an3k @fpientka @jlnprssnr @jdslab to comment with their thoughts.

@mynchau
Dear Mynyna Chau,
thanks for your detailed answer. I understand that this setting in the App Store is not under your control, as you are not responsible for that.
However, the classification of the age limit seems to be based on the classification of medical treatment information. Because under the age specification 17+ the description: "Frequently/strongly pronounced: medical/treatment information" is listed.

You have advised us to contact the RKI directly. But there is no direct communication possibility there, as here in the Open Source Project. You have a direct line to the RKI and can communicate this suggestion much more easily so that it arrives at the right place.
Thanks for your time.

Further to @Christian-Hemker's point about us not having a direct link to the RKI:
I called the technical hotline to report the playstore country issue as well as this issue here. I was told they are aware but cannot say anything about it - they haven't received information from the RKI. I asked them to escalate, but they weren't very convinced that would change anything. I was told over 50% of calls in the past few days had been related to the app store issue. Yet we're still waiting for any updates there. Calling doesn't help, writing doesn't help. So it's natural the community feels let down a bit by maintainers.
So, just speaking to someone on a hotline doesn't work. It personally feels a bit like an insult for the app maintainers to tell us in the community to go down weird routes to raise awareness of an obvious issue for which there is community consensus.

Hi everybody,
Thanks for your concerns & for submitting the issue. As I do not have family sharing enabled, and I haven't worked with it before, I would like to ask you a question - does the workaround mentioned by @jdslab work? Does it enable 'minors' to be able to use the CWA as well?
Thanks for checking! Here the comment again:

The way I understand it is: iOS age restrictions are set on device, and installation of 17+ apps can be explicitly allowed in iOS Settings in Screen Time -> Content & Privacy Restrictions -> Content Restrictions -> Apps. This can also be done temporarily, just for downloading this single app. Could be at least a workaround.


iPhone: Cursor of input textbox moving to end of text after single character update

Subject of the issue
I have added pascalprecht.translate  as dependency to my app. I have an input textbox for username. Strangely, whenever I go back and edit mid-text in the username textbox, the cursor automatically moves to the end of the username text after a single character edit. This is annoying to the user as if they want to type multiple characters, they have to move the cursor back to the position after typing ever character.
I could narrow down that the issue is due to pascalprecht.translate since when I removed this dependency the issue is no more present.
This issue is only present in iPhone Safari and Chrome. It works fine in Android and desktop (mac and windows browsers).
Your environment

version of angular-translate: angular-translate - v2.17.0
version of angular -  AngularJS v1.6.4
which browser and its version - iPhone Safari and Chrome.

@aloysiustany Please provide a demo always. As we do not add explicit "cursor management", I would assume there taking something "intelligent" into account. Try switching filter/directive, or try using the service itself so you can apply the translation on your own for that field.

I am not able to understand when you say

Try switching filter/directive, or try using the service itself so you can apply the translation on your own for that field

Please can you give me more details on what should be tried ?

I have no clue what you have done. Maybe <input translate-attr-value="TRANSLATION_ID"></span>, maybe <input value="{{'TRANSLATION_ID' | translate}}"/>.

Do have checked out the alternative with translate-attr? Do you have checked with a poc that's an AngularJS expression (without filter) behaving different?
My point is: I would guess this behavior is for all AngularJS bindings (curious: first time I've heard about this). Beside the alternative ways (translate-attr), there aren't other options and we are the wrong address.

Hello,
I'm closing this issue due inactivity. We have asked for additional details, but you could not helped us. Therefore we close this issue as not reproducible.
If you still have an issue, please create a new issue because we cannot be sure to track closed issues.


SAP Cloud Platform Workflow - Ex.03 - can't get container to run

I'm at step 3. Instantiate a container and I'm trying to run the following command: docker run -p 8443:8443 --name myscc -d scc. When I check docker ps it's not listed as a running container, when I use docker ps -a I see that' it's exited.
When I want to see the log of my container with the following command: docker logs <container_id> I see the following:

Error: Unable to access jarfile plugins/org.eclipse.equinox.launcher_1.1.0.v20100507.jar

I have used the latest versions when it comes to the downloads:

sapcc-2.13.1-linux-x64.zip
sapjvm-8.1.072-linux-x64.rpm

How can I fix this?

@xtreemrage  A little OT. Perhaps you can help.
Going a step back; how did you manage to complete Ex 2 with the updated booster that is no longer compatible with these exercises.  I'm unable to deploy BPMFLP https://github.com/SAP-samples/sap-devtoberfest-2020/issues/91
Do you have any inputs that might be useful for my case.

@mk-in
I used my trial account and I could deploy it just fine, but with the booster, you don't need BPMFLP. You will get the following:

That will lead to the workflow "launchpad":

All the workflow definitions/instances/approvals will be in there.

@xtreemrage I skipped Ex.02 for now
At Ex 03 - I face this error too. I used same source files as mentioned by OP.

I fixed this error and encountered this

@xtreemrage Just a shot in the dark .. what is your host os ???

FIXED!
@xtreemrage - Can you check at your end
Modified the docker file as followed
FROM centos:7

RUN yum -y install initscripts which unzip wget net-tools less

COPY sapdownloads /tmp/sapdownloads/
WORKDIR /tmp/sapdownloads/

COPY java /tmp/java/

RUN unzip sapcc*.zip && \
        rpm -i sapjvm*.rpm && \
        rpm -i com.sap.scc*.rpm

ENV JAVA_HOME=/opt/sapjvm_8/

RUN chsh -s /bin/bash sccadmin

EXPOSE 8443
USER sccadmin
WORKDIR /opt/sap/scc/

CMD = ./go.sh

Thanks @mk-in ! I'll look into this and see about modifying / updating the sources. Appreciated!

Thanks again folks, I've updated the README instructions for Exercise 03, to point directly to the nzamani/sap-cloud-connector-docker repository for all the SAP Cloud Connector setup instructions. This is up to date, and I have sent a couple of pull requests to it recently too to help keep it up to date.
Following the instructions in the nzamani/sap-cloud-connector-docker repository means that the setup should work fine.
@mk-in please let me know if there's still a question about the booster (and perhaps open up a new issue if there is, so we can handle it together) - thanks!


Error Deploying with dockerizePip true

Getting this error on sls deploy
Error: docker run --rm -v /Library/Caches/serverless-python-requirements/479953baaa6395d6d18bb6db8935af2fa13b0c841bbeb72ae8ddca94f69e38bf_x86_64_slspyc:/var/task:z -v /Library/Caches/serverless-python-requirements/downloadCacheslspyc:/var/useDownloadCache:z -u 0 lambci/lambda:build-python3.8 /bin/sh -c python3.8 -m pip install -t /var/task/ -r /var/task/requirements.txt --cache-dir /var/useDownloadCache && find /var/task -name \*.so -exec strip \{\} \; Exited with code 2
serverless.yml
service: image-service

frameworkVersion: '3'

provider:
  name: aws
  runtime: python3.8

package:
  # individually: true
  exclude:
    - api/__tests__/**
    - api-schemas/**
    - node_modules/cache/**
    - venv/**
    - .pytest_cache/**

plugins:
  # - serverless-layers
  - serverless-python-requirements
custom:
  # serverless-layers:
  #   layersDeploymentBucket: image-service-layers
  #   dependencies: './requirements.txt'
  pythonRequirements:
    dockerizePip: non-linux
    noDeploy:
        - pytest
    zip: true
    slim: true

functions:
  imageSimilarity:
    # TODO: this doesn't work offline: https://github.com/dherault/serverless-offline/issues/993
    handler: api/image_similarity.handler
    events:
      - http:
          private: true
          path: /v1/image-similarity
          method: post
          cors: true
          request:
            schemas:
              application/json: ${file(api-schemas/image-similarity.json)}

requirements.txt
numpy==1.22.1
opencv-python==<IP_ADDRESS>
attrs==21.4.0
boto3==1.21.36
botocore==1.24.36
certifi==2021.10.8
cffi==1.15.0
charset-normalizer==2.0.12
cryptography==36.0.2
idna==3.3
iniconfig==1.1.1
Jinja2==3.1.1
jmespath==1.0.0
MarkupSafe==2.1.1
moto==3.1.4
packaging==21.3
pluggy==1.0.0
py==1.11.0
pycparser==2.21
pyparsing==3.0.7
pytest==7.1.1

+1

Same here!

This behavior can - amongst other things - be caused by a bad encoding of your requirements.txt file. Check if it's UTF-8 or ASCII. If it's not, convert it, clear your cache, and try again. See https://github.com/serverless/serverless-python-requirements/issues/695#issuecomment-1203745734 for avoiding the bad encoding (in PowerShell)

Can confirm I'm seeing the same issue, but my file encoding is fine
$ poetry export -o requirements.txt --format=requirements.txt
$ file -bi text/plain; charset=us-ascii

Ignore the above, my problem was totally unrelated! 😵💫

I've been hitting the same issue myself when trying to run a serverless deploy from an AWS codebuild project.  Anyone figure out a solution?


Laravel 11.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:

Checkout the l11-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: The latest version of Laravel has a dependency for phpunit/phpunit of ^10.5. If you would like to also upgrade your tests, you may run the PHPUnit 10 Shift for free.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 11. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-prerender.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "codebar-ag/laravel-prerender": "dev-l11-compatibility",
    }
}

Finally, run: composer update


Missing dict file if "Pretrained model" checked

If UI "Pretrained model" checked, operation fails with empty descriptive message (just Error training...).
Traceback:
AssertionError
	assert os.path.exists(dict_source)
File "/usr/lib/wunjo/app/speech/tps/tps/utils/__init__.py", line 44, in load_dict
	self.entries = load_dict(dict_source, fmt)
File "/usr/lib/wunjo/app/speech/tps/tps/modules/custom/replacer.py", line 30, in __init__
	super().__init__(dict_source, "Omograph")
File "/usr/lib/wunjo/app/speech/tps/tps/modules/omographs/rule_based/independent.py", line 14, in __init__
	super().__init__(dict_source, prefer_user)
File "/usr/lib/wunjo/app/speech/tps/tps/modules/omographs/rule_based/russian.py", line 10, in __init__
	md.RuOmograph([omographs_dict, "plane"], True),
File "/usr/lib/wunjo/app/speech/tps/tps/handler.py", line 487, in _get_default_modules
	modules = _get_default_modules(charset, data_dir, verify_checksum, silent, use_cleaner)
File "/usr/lib/wunjo/app/speech/tps/tps/handler.py", line 384, in from_charset
	text_handler = Handler.from_charset(hparams.charset, data_dir=os.path.join(os.path.dirname(os.path.abspath(__file__)), "data"), silent=True)
File "/usr/lib/wunjo/app/tacotron2/train.py", line 104, in prepare_dataloaders
	train_loader, valset, collate_fn = prepare_dataloaders(hparams, distributed_run)
File "/usr/lib/wunjo/app/tacotron2/train.py", line 278, in train
	train(hparams, distributed_run=distributed_run, rank=rank, n_gpus=n_gpus)
File "/usr/lib/wunjo/app/tacotron2/train.py", line 412, in main
	tacotron2(hparams_path=config_path)

dict_source in that case contains path /usr/lib/wunjo/app/tacotron2/data/omographs.dict (missing in distribution).
Tech info:

OS: ubuntu 22.04.3 x64
Python version: 3.10.12
wunjo version: 1.5.3
Installation source: https://wladradchenko.ru/static/wunjo.wladradchenko.ru/build/linux/wunjo_1.5.3.deb

Upd: manual copy of files omographs.dict, stress.[a,b,c].dict from /usr/lib/wunjo/app/speech/tps/rules/ into /usr/lib/wunjo/app/tacotron2/data/ fails with next error:
KeyError: 'state_dict'
	pretrained_dict = checkpoint_dict["state_dict"]
File "/usr/lib/wunjo/app/tacotron2/train.py", line 144, in warm_start_model
	model = warm_start_model(
File "/usr/lib/wunjo/app/tacotron2/train.py", line 285, in train
	train(hparams, distributed_run=distributed_run, rank=rank, n_gpus=n_gpus)
File "/usr/lib/wunjo/app/tacotron2/train.py", line 412, in main
	tacotron2(hparams_path=config_path)
File "/usr/lib/wunjo/app/train/tacotron2train.py", line 37, in run_train
	Tacotron2Train.run_train(config_path)
File "/usr/lib/wunjo/app/train/tacotron2train.py", line 27, in train
	Tacotron2Train.train(param)
File "/usr/lib/wunjo/app/train/utils.py", line 19, in training_route
	training_route(param)
File "/usr/lib/wunjo/app/wunjo/app.py", line 714, in common_training_route
	Traceback (most recent call last):

Interest. I will solve this path bug in version 1.5.4. Now, you can copy those files as you did. Your error checkpoint_dict["state_dict"] means what in checkpoint witch you use was trained on another version of Tacotron and doesn't have key "state_dict". It means what you need to train the tacotron2 model by Wunjo AI and after you can continue train the tacotron2 model as "Pretrained model".

Solved in version 1.5.4