API ?

Sorry in advance, but I can't manage to found anker API?
I need it for the following info
S2M_USER (required): A Solix API client id
S2M_PASSWORD (required): The client secret
I created a debian LXC , I installed nodeJS and did  update but then I don't know how to continue since I can't the API...
Thinking about it I always used API published by companies's dev website, but maybe I got that wrong here?

Ah sorry, I never noticed that the description of these values is wrong. It’s actually your username & password you use for logging into the the app.

Thank you!


Grammar not correctly recognizing doc comment types in some cases

When using the new grammar in VS Code, highlighting of doc comments fails if either the Class.<type> syntax is used (this is valid in JSDoc), or if types are nested more than one level deep. It also currently does not recognize explicitly documenting the properties of an object.

Are there currently plans to support any of these notations?
Thanks!

@abgivant I have added support for the {Array.<string>} but support for {Promise<Array<string>>} and {{prop: string}} is still not added. We use the jsdoc grammar from atom/language-javascript repo and would be waiting on their implementation of it.

Thanks!
And that's fine. I can open a new issue in that repo with the remaining two issues so you can close this if you'd like.

Opened https://github.com/atom/language-javascript/issues/451 in the Atom language repo.


Reported version should be match the official release

This is related to the change at https://github.com/messense/openwrt-netbird/issues/28.
netbird is now showing the correct version, but it seems its not quite as the official release, as it has an extra v letter prefix, and I think it should be removed. Here's a screenshot from https://app.netbird.io/peers:

And from the command line:
root@OpenWrt:~# netbird version
v0.10.7

Versus what is displayed by the official release:
vagrant@dm1:~$ netbird version
0.10.8

opened a PR with the new version, see #37

Please note that this issue is not about upgrading to 0.10.8. It's about changing the way the version is reported, that is, v0.10.7 vs 0.10.7

That's true; I totally missed that. Looking here, it seems to be all set correctly; even the workflow outputs the v0.10.7 in the ldflag, see here
Can you share the arch type you are running there?

It's amd64.
I think we need to remove the v letter at https://github.com/messense/openwrt-netbird/blob/main/netbird/Makefile#L20

Done in https://github.com/messense/openwrt-netbird/commit/fc3b11e6ea24dbc9996d385e4bf83e5f87077c97


version 5.0 add problem

PocketClient.Add method in version 5.0 takes a very long time to complete and it was not the case in 4.2

The Add implementation has not changed. Where do you experience performance problems? Could you debug your code and see if it's a network bottleneck?

Unfortunately I have almost spent one whole work day in figuring out where the problem is as it made no sense but it was very reproducible.
What happens is that items get double during the user initiated sync. First they are added to Pocket with the mentioned method when they are added to the app. However the result is not returned for a long time (could be very well that the method is stuck until the next sync wakes it up by calling the PocketClient) and the item has no Pocket ID, and then the app figures out that it is the new item it doesn't have.
Reverting to 4.2 and it works as before.
It doesn't depend on the network it works like that in every situation from 3G to the optical internet.

By using a profiler you can see which method in your code or the library's code takes too long. In case it really was the Add method there was eventually a problem with the Pocket API.


Fix idle wires False with barrier

Summary
Fixes #8313
Details and comments
This PR fixes a problem where setting the idle_wires option to False in the 'mpl' or 'latex' circuit drawers when there was a barrier in the circuit would cause the drawer to fail.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 10 (30.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.003%) to 83.99%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/visualization/matplotlib.py
0
7
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.003%

Covered Lines:
55820

Relevant Lines:
66460

💛  - Coveralls


[VirtualBox] (VMSVGA) second display not detected

First boot of 0.5.0 (0E43) from hello-0.5.0_0E43-FreeBSD-12.2-amd64.iso

% freebsd-version -kru
14.0-CURRENT
14.0-CURRENT
14.0-CURRENT
% uname -v
FreeBSD 14.0-CURRENT #90 main-n245648-66f138563be: Thu Mar 25 08:00:54 GMT 2021     root@mowa219-gjp4-8570p:/usr/obj/usr/src/amd64.amd64/sys/GENERIC-NODEBUG
% pkg info -x virtualbox
virtualbox-ose-6.1.18
virtualbox-ose-kmod-6.1.18_1
%

Please try on real hardware, too.

Works with VBoxSVGA.

Thanks for testing @grahamperrin - so should we add to the documentation that VirtualBox users should select VBoxSVGA in the VirtualBox Monitor Settings? (Or does this have significant downsides?)

▶ https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=254266#c5

Given that

VBoxSVGA is the official virtualbox device, the one the additions provide a specific driver for. It supports 3D acceleration, is based on the VMWare driver but extended

I am tempted to write in the documentation that people should be using VBoxSVGA, unless I am overlooking something.

Does running /usr/local/bin/vmware-user-suid-wrapper change the situation?
It looks like this is supposed to be started by /usr/local/etc/xdg/autostart/vmware-user.desktop but nothing in helloSystem currently does anything with /usr/local/etc/xdg/autostart/.

@grahamperrin

Works with VBoxSVGA

How did you get that window on the right-hand side? And I seem to only getting Mirroring when I set 2 screens...


remove file_name parameter because it's not used in function

Leaving "file_name" argument in also causes the AML pipeline to fail because elsewhere in the CI pipeline, the function is called without a filename argument, and throws an error.

I was running into the same issue, see #253 .
Another solution would be to switch lines for the file_name assignment and then call create_sample_data_csv(file_name) as described in #253 . And replace the constant file_name in df.to_csv(...):
import pandas as pd
from sklearn.datasets import load_diabetes

# Loads the diabetes sample data from sklearn and produces a csv file that can
# be used by the build/train pipeline script.
def create_sample_data_csv(file_name):
    sample_data = load_diabetes()
    df = pd.DataFrame(
        data=sample_data.data,
        columns=sample_data.feature_names)
    df['Y'] = sample_data.target
    # Hard code to diabetes so we fail fast if the project has been
    # bootstrapped.
    df.to_csv(file_name, index=False)

@martin-weber I looked into those changes but chose this solution purely because it was fewer code edits and didn't seem to bork anything :)

@cindyweng Fine for me. Both solutions work.


git clone routine not able to find bbhn packages

When installing  through the install.sh routine an error occurs when it attempts to run "git clone git://ubnt.hsmm-mesh.org/bbhn_packages". The error is a failure to connect to the url... I checked, it no longer exists. It doesn't appear that they use any of the same directory structures at broadband-hamnet.org anymore. Any suggestions?
John, KY8T

Problem solved. BEFORE!!! running install. sh you MUST edit the line calling the bbhm pckages from:
git clone git://ubnt.hsmm-mesh.org/bbhn_packages
To:
git clone git://ubnt.broadband-hamnet.org/bbhn_packages

Recently removed the dependency on bbhn_packages completely, as the patch I was pulling from that repo is no longer needed. So this shouldn't be an issue anymore.


"Application entity" link broken

In this paragraph:
Application object
An Azure AD application is defined by its one and only application object, which resides in the Azure AD tenant where the application was registered, known as the application's "home" tenant. The Azure AD Graph Application entity defines the schema for an application object's properties.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6c12e139-8cb6-dd6d-2e08-89d4f22c7eef
Version Independent ID: 675b5ee9-bfe6-344f-7b39-ca0e2bc18b9b
Content: Application and service principal objects in Azure Active Directory
Content Source: articles/active-directory/develop/app-objects-and-service-principals.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

Thank you very much for the feedback, @dlyall-maine
We have a PR that's waiting to go live for today's ~3:30pm PST docs update. It will address this issue, which was also reported in a related GitHub issue (https://github.com/MicrosoftDocs/azure-docs/issues/25761).

Change has been merged to master. Just waiting to go live ~3:30pm PST today.
#please-close


⚠ Cioos National CKAN has degraded performance

In bc14a58, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 10698 ms

Resolved: Cioos National CKAN performance has improved in b5c1fe6 after 5 minutes.


Schema support

Hello there!
Would it be possible to add a schema support in the Column as well as in the EnumMigration?
As it is, trying to "bypass" this issue by using the schema_name.table results in an (expected) error.
EnumMigration(
    op=op,
    enum_name="schema_name.enum",
    old_options=EXISTING_OPTIONS,
    new_options=EXISTING_OPTIONS + ["MY_VALUE"],
    columns=[
        Column(
            "schema_name.table",
            "column",
            old_server_default=None,
            new_server_default=None,
        )
    ],
)

sqlalchemy.exc.ProgrammingError: (psycopg2.errors.InvalidSchemaName) schema "_tmp_schema_name" does not exist
LINE 1: ..._tmp_schema_name.enum_3224  USING category::text::_tmp_schema...

Of course, it would make more sense to add a schema: Optional[str] = None in both classes for support rather than add the raw string, but I had to test it out :)

@theo-ardouin, thank you so much for your contribution! Your PR #7 has been merged and I have released a new version, 0.4.1.


Add food recommendation

In the trip planner, add a food recommendation system which will give you dishes you must try in a destination selected

Hey @sumana2001 . Can you please assign me and @arpitBhalla this issue?


Installation issue on Fedora 36

Fedora 36 is being recognised as redhat in main task of install_ansible_rulebook therefore it try to include task install_redhat.yml which doesn't exist.
{{ ansible_os_family }} returns redhat which should be fedora
https://github.com/ansible/event-driven-ansible/blob/445579834ceb418ba5f34befe3ef566ad7475458/roles/install_ansible_rulebook/tasks/main.yml
ERROR
TASK [ansible.eda.install_ansible_rulebook : Install for RedHat] ******************************************************************************************************************** fatal: [localhost]: FAILED! => {"reason": "Could not find or access '/home/gunjankumar/.ansible/collections/ansible_collections/ansible/eda/playbooks/install_redhat.yml' on the Ansible Controller."}

{{ ansible_os_family }} returns redhat which should be fedora

This is actually correct, Fedora is a distribution in the RedHat family.
Same as Ubuntu is a distribution in the Debian family.
The file should be renamed to install_redhat.yml.


Feature: Add background colors for signal and waitlist

Why?
The design system has been updated with new background colors for signals and waitlist.
What?

Add new colors to assets catalog
Make new colors available to SwiftUI and UIKit
Update color tests
Update demo view

Version Change
This is a minor change.

https://app.circleci.com/pipelines/github/sats-group/SATSCore-iOS/215/workflows/6521e77d-b998-4aba-9d08-c27c5aac1297/jobs/174/tests


Generated beta models and request builders

This pull request was automatically created by Azure Pipelines. Important Check for unexpected deletions or changes in this PR.

Closing as superseded by #26


add missing release notes from other branches

Various branches had release notes specific to micro-versions that were not present in the master branch. This PR collects all of them in master.

great, thanks!


nitcorn application for nitiwiki

nitiwiki is a simple wiki manager based on markdown files and git repositories. Nitiwiki generates a static site from the markdown source and does not yet offer any dynamic features. For more information, see http://nitlanguage.org/catalog/p/nitiwiki.html
Goal: Write a nitcorn application to add and edit content of a nitwiki server from the browser.

[ ] This application should display a simple form, accepts POST submissions, write the content to markdown files, and trigger nitiwiki to regenerate the static content.
[ ] Add an option to display a preview using the markdown module.
[ ] Integrate this interface with the default template of Nitiwiki.
[ ] Document how to install this application alongside an existing nitiwiki structure.
[ ] Implement a minimal access control using a simple server side structure, such as a Map[String, String] filled from a text file.

I've started something which can be found in commit a06994a27c20fba3b378a33a373f170952acb866. I don't know when I'll have time to continue. Don't hesitate to start from it if you're interested...

@ablondin please PR the current code, reviewer will help to improve it so it could be merged


MOLFiniteDifference periodic boundaries

https://discourse.julialang.org/t/periodic-boundary-conditions-for-pde-system-using-modellingtoolkit-and-diffeqbase/51827/3

Closed by #34


Add a debugging info Popover

Add a Popover to the UI that shows some debugging information about the game. It should contain IDs for key game entities, such as the game, factions, step, phase, turn and actions.

Was closed by #343, but forgot to link it


FE: Add retry logic to router for get blob

This PR adds retry logic for GetBlobOperation in router.
Why are we doing this:
Router might fail client's get requests due to temporary network error, or server being overloaded. If frontend returns 503 right away, client would would probably retry the entire blob. However, if this is a composite blob made of hundreds of chunks, then client resending get request would end up wasting lots of resources to refetch those already fetched blob in router.
This PR adds retry logic to prevent resource from wasting. It will retry on GetBlobOperation. But there is a limit to which an GetBlobOperation can retry. This limit is based on both count and duration. We can configure the maximum number of retries and the longest duration for retries. A retry can't exceed either limit.

can u pls address my comments before merging ?

can u pls address my comments before merging ?


[Enhancement] Add sum() JMESPath filter

Description
https://github.com/kyverno/kyverno/pull/6552
Slack discussion
No response

Turns out this was already done and I created a duplicate. Merged in the PR.


build Dockerfile causes segmentation fault

Hi, while trying to run the Dockerfile using docker build -t ratsql . the process keeps failing due to a segmentation fault while trying to download Bert (step 7).
Has anyone else encountered this issue?
Thanks!
Step 7/14 : RUN python -c "from transformers import BertModel; BertModel.from_pretrained('bert-large-uncased-whole-word-masking')"
 ---> Running in 43837f926306
To use data.metrics please install scikit-learn. See https://scikit-learn.org/stable/index.html
Segmentation fault (core dumped)
The command '/bin/sh -c python -c "from transformers import BertModel; BertModel.from_pretrained('bert-large-uncased-whole-word-masking')"' returned a non-zero code: 139

@tomerwolgithub I am facing the same error  but haven't come across a solution yet.

And me too.

+1

Hi, i modified the docker file as follows and it's still working.
just Annotate one line
#RUN python -c "from transformers import BertModel; BertModel.from_pretrained('bert-large-uncased-whole-word-masking')

it is a simple matter of updating to a later version of transformers than specified in the requirement.txt file.

Thanks for the tip. It worked with transformer 3.5.0, BUT NIOT  with  3.4:

you need to specify in the setup.py :
'transformers == 3.5.0',
1.1) BUT
the above line should be THE FIRST in the setup.opy in order to download the newest library, otherwise I got an incompatibility (and error) message:
ERROR: transformers 3.5.0 has requirement sentencepiece==0.1.91, but you’ll have sentencepiece 0.1.94 which is incompatible.

Thanks again

it also works to remove the version number from the transformer pip package, your solution is more complete, thanks for fleshing out the details
Sam


Page Treebeard fixes

Fix for #4102

Coverage decreased (-0.2%) to 90.858% when pulling a06f6f937ad301ff1d62fc05450c96720517d384 on yakky:feature/treebeard_fix into b86a461c5714cb51f67228063210bd7d6ee76d93 on divio:support/3.1.x.

Coverage decreased (-0.2%) to 90.858% when pulling a06f6f937ad301ff1d62fc05450c96720517d384 on yakky:feature/treebeard_fix into b86a461c5714cb51f67228063210bd7d6ee76d93 on divio:support/3.1.x.

Coverage decreased (-0.2%) to 90.858% when pulling a06f6f937ad301ff1d62fc05450c96720517d384 on yakky:feature/treebeard_fix into b86a461c5714cb51f67228063210bd7d6ee76d93 on divio:support/3.1.x.


Improved packaging mark 6

Description
Unfortunately #267 did not fix the issue with secrets due to the fact that these are not sharable to PRs. We can either remove this check (not wise, since the build step is very useful to check if anything has caused the build to fail before merging and incrementing the version number), or just allow the final step to fail silently when it is unable to upload (which is what this PR proposes).
I also reordered some of the build so that the automatic git tag reflects the version number in the corner case that the build fails for some reason in between PR and merge.
Checklist:

[X] I have performed a self-review of my own code
[X] My code is PEP8 compliant (flake8 anesthetic tests)
[X] My code contains compliant docstrings (pydocstyle --convention=numpy anesthetic)
[X] New and existing unit tests pass locally with my changes (python -m pytest)
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have appropriately incremented the semantic version number in both README.rst and anesthetic/_version.py

The issue lies with untrusted forks not having access to the GitHub secrets, right?

Yeah, and I don't necessarily disagree with them here

Is there not an option of labelling certain forks as trusted?

Not that I've found.

Otherwise, in this CircleCI post they introduce an option for a reviewer with write privileges to manually launch an action (after a check of the changes) that would otherwise not even launch for a PR from an untrusted fork. Is there maybe something similar for GitHub actions?

Not obviously -- I think github is quite security conscious (see our unfilled 'Security' box in 'community standards' tab in Insights).


Please add Tmorph to runelite, transmog for runelite.

Please add Tmorph as it would allow players to Tmog client side. I don't know if this causes issues with the bot detection system but I know runelite is capable of it. Could you please add it? This would be great feature for players who really like to fashions cape but can't while they're doing certain skilling/bossing/slayer. I would greatly appreciate adding support for this if you can.

duplicate #12482


Add GitHub issue and Pull request templates

What type of PR is this?

New feature

What this PR does / why we need it:

To standardize Github issues and pull requests

Which issue(s) this PR fixes:

https://github.com/kubernetes-sigs/windows-operational-readiness/issues/62

/approve
/lgtm


⚠ API - Lens has degraded performance

In acada46, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 502
Response time: 337 ms

Resolved: API - Lens performance has improved in 65c755a after 23 minutes.


Safari High Sierra 11.1 syntax error

Expected behaviour
That the player will initialize and work as expected
Actual behaviour
fails to initialize syntax error ?
Steps to reproduce
Use High Sierra Safari v11.1  and load the min or regular plyr.js
Environment

Browser:

Safari

Version:

11.1

Operating System:

Mac OS

Version:

High Sierra

Console errors (if any)
synatx error line 2 unexpected ?
Link to where the bug is happening
No link but I was able to change your ?? to || and older versions of Safari are now working .  I did this in two places
this.scrollPosition = {
x: window.scrollX ?? 0,
y: window.scrollY ?? 0
}
to this
this.scrollPosition = {
x: window.scrollX || 0,
y: window.scrollY || 0
}
I did that with the other ?? that was basically an or

Safari 11? I'd really suggest upgrading, not just for this but for security and your experience in general. Safari 11 registers as 0% usage:
https://caniuse.com/usage-table


PathFinder: Can't project end-point to navmesh, skipping:  Vector(0, 0, 0) & Early abort, can't project any points to navmesh

Habitat-Lab and Habitat-Sim versions
Habitat-Lab: v0.2.4
Habitat-Sim: v0.2.4
Habitat is under active development, and we advise users to restrict themselves to stable releases. Are you using the latest release versions of Habitat-Lab and Habitat-Sim? Your question may already be addressed in the latest versions. We may also not be able to help with problems in earlier versions because they sometimes lack the more verbose logging needed for debugging.
Master branch contains 'bleeding edge' code and should be used at your own risk.
Docs and Tutorials
Did you read the docs? https://aihabitat.org/docs/habitat-lab/
yes
Did you check out the tutorials? https://aihabitat.org/tutorial/2020/
yes
Perhaps your question is answered there. If not, carry on!
❓ Questions and Help
Hi, when I use habitat-lab to train on the mp3d dataset, it continually report the following info:
[11:25:13:513281]:[Nav] PathFinder.cpp(1365)::findPathSetup : Can't project end-point to navmesh, skipping:  Vector(0, 0, 0)
[11:25:13:513354]:[Nav] PathFinder.cpp(1375)::findPathSetup : Early abort, can't project any points to navmesh.
[11:25:13:590906]:[Nav] PathFinder.cpp(1365)::findPathSetup : Can't project end-point to navmesh, skipping:  Vector(0, 0, 0)
[11:25:13:590976]:[Nav] PathFinder.cpp(1375)::findPathSetup : Early abort, can't project any points to navmesh.
[11:25:13:683487]:[Nav] PathFinder.cpp(1365)::findPathSetup : Can't project end-point to navmesh, skipping:  Vector(0, 0, 0)
[11:25:13:683559]:[Nav] PathFinder.cpp(1375)::findPathSetup : Early abort, can't project any points to navmesh.
[11:25:13:797314]:[Nav] PathFinder.cpp(1365)::findPathSetup : Can't project end-point to navmesh, skipping:  Vector(0, 0, 0)
[11:25:13:797383]:[Nav] PathFinder.cpp(1375)::findPathSetup : Early abort, can't project any points to navmesh.
[11:25:14:051908]:[Nav] PathFinder.cpp(1365)::findPathSetup : Can't project end-point to navmesh, skipping:  Vector(0, 0, 0)
[11:25:14:051980]:[Nav] PathFinder.cpp(1375)::findPathSetup : Early abort, can't project any points to navmesh.
[11:25:14:380497]:[Nav] PathFinder.cpp(1365)::findPathSetup : Can't project end-point to navmesh, skipping:  Vector(0, 0, 0)

Can anyone tell me why this information is being reported? What results this situation will lead to?

Hey @pioneer-innovation looks like there are calls to the PathFinder with MultiGoalShortestPath which contain point(s) that are not on the NavMesh. For example (0,0,0) the origin may not be a navigable point for some scenes. Not sure where this is being called from, but unless it is causing training issues it is likely not a problem. It could be culling path end points or goals that are infeasible.


Bug fix: memory layout for conv2d_f Python API

The current binding implementation assumes that the inputs are all c_style contiguous arrays, which is not always true.
The bug can be reproduced with examples like
from python import _pyuTensor
import numpy as np
import tensorflow as tf
a_setting = (1, 5, 5, 3)
a = np.arange(np.prod(a_setting)).reshape(a_setting).astype(np.float32)
b_setting = (5, 5, 3, 5)
b = np.arange(np.prod(b_setting)).reshape(b_setting).astype(np.float32)

print(_pyuTensor.conv2d_f(a, b.transpose(3, 0, 1, 2), [0 for _ in range(b_setting[3])], [1, 2, 2, 1], "VALID"))
print(_pyuTensor.conv2d_f(a, b.transpose(3, 0, 1, 2).copy(), [0 for _ in range(b_setting[3])], [1, 2, 2, 1], "VALID"))
print(tf.nn.conv2d(a, b, strides=[1, 2, 2, 1], padding="VALID").numpy())

A temporary fix is just casting the arrays into c_style using pybind's constructor.

Can you attach a screen shot of the result of the script?
Just want to confirm the bug is fixed by the PR.

Sure!
Before:

After:

Looks good to me!


WriteGetObjectResponseRequest: Content-Type

Confirm by changing [ ] to [x] below to ensure that it's a bug:

[x] I've gone though Developer Guide and API reference
[x] I've searched for previous similar issues and didn't find any solution

Describe the bug
I'd like to be able to set the "Content-Type" on WriteGetObjectResponseRequest, as specified in the documentation:

https://docs.aws.amazon.com/AmazonS3/latest/API/API_WriteGetObjectResponse.html

Scroll to:
x-amz-fwd-header-Content-Type: ContentType
SDK version number
git/master
Platform/OS/Hardware/Device
Linux
To Reproduce (observed behavior)
Code does not set content-type at all:

https://github.com/aws/aws-sdk-cpp/blob/main/aws-cpp-sdk-s3/source/model/WriteGetObjectResponseRequest.cpp#L176-L188

Expected behavior
I'd like to set the content-type to be something other than just "text/plain"

I should also mentionned that the function SetContentType does not change the content-type of the output:
void Aws::AmazonStreamingWebServiceRequest::SetContentType	(	const Aws::String & 	contentType	)
ref:

http://sdk.amazonaws.com/cpp/api/LATEST/class_aws_1_1_amazon_streaming_web_service_request.html#ab3c257c251ecd26b096a0c3be7dfd69f

Hi @malaterre ,
Sorry for the response delay, I'm still trying to reproduce this but looking at the code it seems you are right, thanks for pointing it out to us.


Camera Error

any idea plz ?

I have this error too. about a month ago I used this package with react-native 0.54 and everything was good. but now I have this error and I can't find solution for it :/

Please note that React Native Camera underwent major changes, make sure after upgrading to the latest version of react-native-qrcode-scanner, you've followed the migration guide (from React Native Camera) : https://github.com/moaazsidat/react-native-qrcode-scanner/blob/master/migration.md

I found the solution here: (can't find the link. it's a capture of it :)


Exhaustive check fails to narrow type when union has only 1 type

TypeScript Version: nightly
Code
  type SkippedError = {
   tag: "Skipped"
 };

 const error: SkippedError = {} as any;

 switch (error.tag) {
   case "Skipped":
     const a1: SkippedError = error;
     return 1;
   default:
     const a2: never = error;
     return 0;
 }

Expected behavior:
Exhaustive check should narrow error type to never for a2.
Actual behavior:
Type 'SkippedError' is not assignable to type 'never'.(2322)
The type of error at the a2 is still SkippedError

I included happy-path variant of same code in playground link, where union for error has 2 types and code works correctly.
The problem reproduces not only for literal types, but also with classes.
Trying to narrow type with if has same result.
Only working solution in this case is checking classes with instanceof, but can not be applied for literals.

Playground Link: Playground Link

when union has only 1 type

What union? You don't have a union type.
Duplicate of #16976.


Getting "mismatched type to binary operator" for int64/int64 or float64/float/64 in 0.13

Working a script for checking memory usage. We pull total and used (in bytes) from InfluxDB, so I need to generate the percent myself. In 0.12, I was able to do something akin to this, an example of usage being over 50%:
.info(lambda: float("used") / float("total") > 0.50)

Unfortunatley, since 0.13, damn near any combination of the above (using float or int) results in an error like this:
E! error evaluating expression: mismatched type to binary operator. got int64 > int64. see bool(), int(), float()

Still learning this, so my gut say it's something silly I'm missing, but I can't seem to figure it out.
And FWIW, I've managed to get similar errors for float64 > float64, int64 / int64 and int64 * int64.

@mtslzr We significantly changed the implementation of lambda expressions in 0.13 so something like probably messed up somewhere. I'll take a look.

@mtslzr I can't reproduce this locally, could you share a more complete example where you are getting this error? Thanks.

@mtslzr I figured it out, and I am chasing down the bug now....

@nathanielc what's the bug? Sounds very weird bug..

@yosiat  I just posted the PR https://github.com/influxdata/kapacitor/pull/567
Besides some refactoring of the error messages, it boiled down to isDynamicNode not checking binary nodes. See https://github.com/influxdata/kapacitor/pull/567/files#diff-9de6492423b4172b3853bd681fd787ae


chmod operation not permitted using watch tag

Environment info:

KrakenD version: watch
System info: docker

Describe what are you trying to do:
i've try to use templates (like describe here https://www.krakend.io/docs/configuration/flexible-config/), but throw an error "chmod /etc/krakend/teste.json: operation not permitted
The config file has been created, but i dont know why the container says "chmod /etc/krakend/teste.json: operation not permitted"
Commands used:
# my docker compose
  api_gateway:
    image: devopsfaith/krakend:watch
    ports:
      - 1000:8080
    environment:
      - FC_ENABLE=0
      - FC_OUT=/etc/krakend/teste.json
      - FC_SETTINGS=/etc/krakend/configuracoes
      - FC_TEMPLATES=/etc/krakend/templates
    volumes:
      - ./configuracoes/krakend:/etc/krakend/
    command: ["run", "-dc", "krakend.tmpl"]

Logs:
✔ Container backend-api_gateway-1  Recreated                                                                              1.5s
Attaching to backend-api_gateway-1
backend-api_gateway-1  | Watching changes on files /etc/krakend/
backend-api_gateway-1  | Ignoring saves to file teste.json
backend-api_gateway-1  | [00] Starting service
backend-api_gateway-1  | [00] ERROR parsing the configuration file: failed to copy source file /tmp/KrakenD_parsed_config_template_1190468719 to /etc/krakend/teste.json: chmod /etc/krakend/teste.json: operation not permitted
backend-api_gateway-1  | [00] (error exit: exit status 255)
Additional comments:

The user that generates the file is uid 1000, so looks like the mounted directory has no permission to write there as uid 1000. Try changing the path of FC_OUT to /tmp or fix the directory permissions.

The user that generates the file is uid 1000, so looks like the mounted directory has no permission to write there as uid 1000. Try changing the path of FC_OUT to /tmp or fix the directory permissions.

Did not work.  Throws another error
LOG:
backend-api_gateway-1  | Watching changes on files /etc/krakend/
backend-api_gateway-1  | Ignoring saves to file tmp
backend-api_gateway-1  | [00] Starting service
backend-api_gateway-1  | [00] ERROR parsing the configuration file: failed to copy source file /tmp/KrakenD_parsed_config_template_3239884491 to /tmp: open /tmp: is a directory
backend-api_gateway-1  | [00] (error exit: exit status 255)

compose.yaml
api_gateway:
    image: devopsfaith/krakend:watch
    ports:
      - 1000:8080
    environment:
      - FC_ENABLE=0
      - FC_OUT=/tmp
      - FC_SETTINGS=/etc/krakend/configuracoes
      - FC_TEMPLATES=/etc/krakend/templates
    volumes:
      - ./configuracoes/krakend:/etc/krakend/
    command: ["run", "-dc", "krakend.tmpl"]

I meant to write the file inside /tmp:
- FC_OUT=/tmp/out.json

I meant to write the file inside /tmp:
- FC_OUT=/tmp/out.json

it works, but i have to add "disable_host_sanitize": true
¯_(ツ)_/¯

I meant to write the file inside /tmp:
- FC_OUT=/tmp/out.json

it works. Thanks


Data tab

In addition to having a map, we should have another tab that just shows the text results. Should allow you to filter through the data that would otherwise be shown on the map. Similar to a SQL query. Again, similar to the way Burgh's Eye View does it: http://analytics.pittsburghpa.gov/BurghsEyeView/, but ours would be structured slightly differently.

@coltonb What do you think about this one? I'm beginning to this this should be it's own data.html page, but on the other hand it would be great if we didn't lose all the data we already loaded into the browser. One idea would be to add all the HTML for the Data tab in the same HTML as the map, but to have Map and Data tabs (http://www.w3schools.com/bootstrap/bootstrap_tabs_pills.asp) that could be buttons that hide one section of the HTML and show the other. As it stands now, the Date and Legend filters you made could be used in this Data tab, too, to filter the data there. Essentially, we'd just want a non-map list of all the data results, and a searchbar to filter it all out.
This is a pretty big Issue, so it's going to take a lot of time and back-and-forth

@vonbearshark I can take this issue

@vonbearshark So I'm not sure exactly how you'd like to implement the data table. Currently I'm trying out DataTables, but it uses jQuery. Also I'm not sure what columns we'd like to include. I decided to populate the tables using the markers array, which allows us to use the existing filters and filter functions. Now the issue with this is that not all markers have the same data (e.g. non-traffic violations don't even have a date field). So I'm a little stuck in that regard lol but still working on it and I wanted to get your thoughts

First, that looks great. It's similar to what I had in mind.

Currently I'm trying out DataTables, but it uses jQuery

It would be better if we didn't use jQuery. We don't need the pagination that this library offers. We could just have a really big page. I don't know if the search functionality comes with it, but I think we can work around that, too.

. Also I'm not sure what columns we'd like to include.

If you take a look at the most recent changes we should have some kind of .table function for each dataset. It's not your responsibility to determine what properties are being displayed or even how they are displayed, you're just calling that table function on each marker item and putting the resulting returned HTML into the table. Like you said, this lets us use one array and one collection of filtering functions to display everything.
How are you switching between the map and this table page right now? This looks like some kind of overlay. It looks great. Does it navigate back and forth okay?

Yeah I figured as much but wanted to check with you to make sure! Yes the search comes with it as well. So you're saying there already is a .table function for each dataset, or we should eventually do that? I don't see that in the current code.
Currently I have two buttons, Map and Data, on the top of the sidebar. Basically when the user clicks Data, the sidebar extends over the map as an overlay, the sidebar toggle arrow disappears and the table is generated (if it's the first time). Then to get back to the map the user clicks Map, and the sidebar returns to its normal functionality. I'm having a slight issue switching from Data to Map mode but that's only because of the DataTables library I'm using, which we're getting rid of anyways.

If it's not there now, it will be, yeah. Just like there is a popup function that returns HTML for the map popup, we want a function that will return HTML for a table entry, given a typical data point in a data set.
That's a great solution--I didn't even consider that!

Oh sweet okay that makes a lot of sense. Thanks! Yeah still working out the kinks but we're making progress haha

How's this coming along, @seifriedc?

I have the table generating and displaying correctly, but I'm having trouble figuring out how to hook it in with our filters. I'll post some code I have and a screenshot in a few minutes when I'm on my laptop.
Currently I have it just appending rows to the table when we fetch the data, and hiding the table at first obviously. But I'm trying to figure out how to have each row mapped to it's corresponding record

Yeah that's a good question. I'm wondering if the filter function should throw out the whole table when it's called, then re-generate it every time.

That might be what we have to do. Ideally I'd like to just display: none each filtered table row.
So in data.js I have a table field for each dataset like this (just 3 columns as an example):
table: (record) => `Non-Traffic Violation
                   ${record["OFFENSES"]}
                   ${record.incidentMonth}/${record.incidentDay}/${record.incidentYear}
                   ${record["NEIGHBORHOOD"]}	`

Then in main.js, at the end of our fetch():
if (dataSource.table) {
  var tr = document.createElement("tr");
  tr.innerHTML = dataSource.table(record);
  document.getElementById("dataTable").appendChild(tr);
}

Is there a way we can save a reference to the row in each marker/record? Or maybe we can give each row an id (doesn't each record have one anyways?) and reference the row that way?
This is what it looks like currently:

That's a great system.
I don't think we can assume that each record will get a unique ID. We could probably make one using the datasource name + the index of the record.
Really, it might be cleanest if we just have filter totally toss the table (with something like innerHTML = "") and totally re-write everything. I think you'll find that's actually pretty quick, and it simplifies everything

Ok I have to adjust my code for the recent PR that was accepted. Do we want Pitt data in the table? Like laundry/labs?

No, it's fine without it for now

Sounds good - think I got it working so I'll submit a PR here soon

Also I'm working on the data view for mobile currently (not sure if I should make a new issue or not)

You can just sub a PR whenever you're ready


M302: Size of Register must be 8, 16 or 32 Bits

From CMSIS SVDConv docs

Message Number
Type
Message Text

M302
ERROR
Size of Register 'NAME':'NUM' must be 8, 16 or 32 Bits

This limitation is not obvious to me. For example, it is easier to create a 12-bit register for a 12-bit ADC than a 32-bit register with only one field.
svd2rust handles this by creating an unsafe method for working with such registers. But CMSIS prohibits such svd files. Should utilities handle this error?

This limitation is not obvious to me. For example, it is easier to create a 12-bit register for a 12-bit ADC than a 32-bit register with only one field.

Register is how CPU reads/writes memory. It does it by u8,u16,u32, u64 depending on architecture. Even if it physically only 12 lines are connected to register mapped on memory.
Field is abstraction which you can push/pop from register.

Do you mean that there are modes in which the processor can read or write registers differently? I thought that the data bus always has a fixed width. For example, a 32bit processor will always read/write all 32bits.

Do you mean that there are modes in which the processor can read or write registers differently?

Yes, depending on asm command you are using to read/write register microprocessor can react differently.
Usually this is allowed for data registers, for example SPI DR, TIM ARR, etc. This is what <size> tag about.

Thank you, it's clear now. So should introduce this limitation into the svdtool or into the svd?


Make fly-in animation for editor sort bar

I need to add a fly-in animation for the sort bar in the editor, to match the fly-in animation of the crew tab.

Done!


Budget app

Ruby on Rails capstone project - Budget app :dollar:
In this project I have created a Ruby on Rails application that allows the user to:

register and log in, so that the data is private to them.
introduce new transactions associated with a category.
see the money spent on each category.

I also created unit and integration tests for all the most important components of  RoR application.
Cheers :clinking_glasses: and happy reviewing :+1:

Thank you @emmanuellekamwa for approving my project :pray:


Add more Cloud8 jenkins jobs

The list of Cloud8 jenkins jobs is not the same as the Cloud7 jenkins
jobs. This commit adds some more jobs to reduce the gap.

 #cloud8 Make crowbar_register work for SP3


Add: a/c performance model + notebook + plots

Addition of Poll-Schumann aircraft performance model.
Modification of the notebook 15_flightplanning_tuto.ipynb to include changes.
Addition of unit tests for a/c models and Poll-Schumann implementation.

Note: data is included

yep I was talking about the .so ;-)
On Fri, 29 Mar 2024 at 14:51, nhuet @.***> wrote:

@.**** commented on this pull request.
On notebooks/15_flightplanning_tuto.ipynb
https://github.com/airbus/scikit-decide/pull/316#discussion_r1544540588:
About the outputs of the notebooks that should not be included as
explained by @galleon https://github.com/galleon, the github workflow
would have pointed this out but it is for now pending since it is your
first contribution to the repo. Someone with the maintainer rights (
@fteicht https://github.com/fteicht, @g-poveda
https://github.com/g-poveda, @neo-alex https://github.com/neo-alex ?)
must approve the workflow launch.
Actually if you install pre-commit and run it on your commits you will see
that it will complain about that :)
See
https://airbus.github.io/scikit-decide/contribute.html#coding-style-and-code-linting
for further explanation.
On __skdecide_hub_cpp.cpython-38-darwin.so
https://github.com/airbus/scikit-decide/pull/316#discussion_r1544537550:
Hi. I think he is talking about the file __
skdecide_hub_cpp.cpython-38-darwin.so (the comment is added to this file
if you notice).
And it is probably due to a git add a bit too large :)
We could add to .gitignore the pattern *.so to avoid this in the future.
—
Reply to this email directly, view it on GitHub
https://github.com/airbus/scikit-decide/pull/316#pullrequestreview-1968587540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB7NPGNRYDXSRNZBH4UWYLY2VPVTAVCNFSM6AAAAABFNHEBQ2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNRYGU4DONJUGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Guillaume


New release date

I noticed that the last release of this project was almost one year ago:
https://github.com/xenolf/lego/releases/tag/v0.3.1
Are there plans to create a new development release soon? Also, will the development release include all DNS providers?

Hello! Yes I've been slacking on releases "a bit". I'd like to get the error 429 issue over with and then have a new release. The release will include all DNS providers by default.

+1
new release tag would be great, so I can update my docker image accordingly

any progress here?

I spoke with Sebastian today; he's super busy with some important things in life, but expects to get back to this soon!

Release is out.


#5655 pr review

PR feedback for https://github.com/sigp/lighthouse/pull/5655

Thank you for the suggestions!


1016 - Add Scenario ACL sub-module

Scenario ACL Sub-module foundations added to ACL module
Overview

Adds new scenario access control submodule and service.
Adds unit tests for scenario-acl service.
Modifies unit tests and methods included in project-acl service as per latests ACL permissions matrix (found here

conflicts, needs a rebase

tests are failling


[4.1] Fix "variable unused" warning in Metadata.cpp in non-assert build

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 979fd613f9e67a7ba1c0625fc2d2dd065c7c67c2

Build failed
Swift Test OS X Platform
Git Sha - 979fd613f9e67a7ba1c0625fc2d2dd065c7c67c2

ping


Implement the third example of the POST multipart encoded file of the documentation

If i'm not mistaken, the third example given in the documentation (here)[http://aiohttp.readthedocs.org/en/v0.14.4/client.html#post-a-multipart-encoded-file] is not implemented:
>>> url = 'http://httpbin.org/post'
>>> files = {'file': ('report.csv',
...                   'some,data,to,send\nanother,row,to,send\n')
... }

>>> yield from aiohttp.request('post', url, data=files)

would post nothing

-1
The code for example was intentionally removed. I guess to remove example from docs also.
New multipart support should be build on top of #273

Ok, updated the commit to edit the docs instead. It's kind of misleading as-is…

Please fix pep8 errors (run "$ make test" command before pushing changes)

The pep8 error is unrelated to this commit ; I've fixed it none the less

Thanks!


Port Constraint samplers

I've been trying to fix the tests, but these tests depends on the df2_kdl.h,  I was having the following errors https://gist.github.com/anasarrak/9597153fe76ab6bb24d4002a77d09e6a, this might take us a bit to solve.. I prefer to continue porting the rest of the modules.

Fixed CI, let's see results @anasarrak

I'm a bit troubled. CI in OS X seems to be fine however in my local installation I get the following error:
In file included from /Users/victor/ros2_moveit_ws/src/moveit2/moveit_core/constraint_samplers/src/constraint_sampler.cpp:37:
In file included from /Users/victor/ros2_moveit_ws/src/moveit2/moveit_core/constraint_samplers/include/moveit/constraint_samplers/constraint_sampler.h:40:
In file included from /Users/victor/ros2_moveit_ws/src/moveit2/moveit_core/planning_scene/include/moveit/planning_scene/planning_scene.h:40:
In file included from /Users/victor/ros2_moveit_ws/src/moveit2/moveit_core/robot_model/include/moveit/robot_model/robot_model.h:43:
/Users/victor/ros2_moveit_ws/install/include/srdfdom/model.h:44:10: fatal error: 'urdf/model.h' file not found
#include <urdf/model.h>
         ^~~~~~~~~~~~~~
2 warnings generated.
1 error generated.
make[2]: *** [constraint_samplers/CMakeFiles/moveit_constraint_samplers.dir/src/constraint_sampler.cpp.o] Error 1
make[1]: *** [constraint_samplers/CMakeFiles/moveit_constraint_samplers.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
2 warnings generated.

Not the first time this happens. @LanderU, any thoughts on this? I'm using OS X 10.14.3.

@anasarrak please note https://github.com/AcutronicRobotics/moveit2/pull/14/commits/3c40d36c41088740b08a502a8b943c24c48b38bb.


Entity collision filtering

Preface: I'm very ignorant with Sponge, so please excuse any mistakes made in this ticket.
Introduction
As of the time of writing, I believe SpongeAPI provides a few events that involve entity collision. Regardless, I am writing this issue to provide a different view of how entity collision might be better handled.
Currently, listening to CollideEvent/CollideEntityEvent should provide the functionality for what is proposed here.
If one is to want to disable all collision for one Entity, they must do something along the following lines:
private Entity entity;

public void setEntity(Entity entity) {
    this.entity = entity;
}

@Listener
public void onCollide(CollideEntityEvent event) {
    Entity eventEntity = event.getOriginalEntities().get(0); // This assumes get(0) is our "pushing" entity.
    if (eventEntity.equals(this.entity)) {
        event.setCancelled(true);
    }
}

Functionality wise, it might be reliable, though I'm not sure about get(0).
However, it could be made more efficient in writing where each entity holds collision filter data.
Proposal
Filters. I'm not sure if there's some sort of filter API in Sponge, but the key behind this whole idea is filters for entities. The Entity would hold a data object that stores a collection of filters which determines whether an entity meets the requirements for collision or not (represented via boolean).
The collection would only need to meet one true value when invoking filters in order for it proceed with collision code.
The following block of code is the EntityFilter type which is simply a boolean-returning function. This is what the CollisionFilters data class would be utilizing:
public interface EntityFilter {
    /**
      * Return true to collide, false to terminate collision code.
      */
    boolean test(Entity pusher, List<Entity> entities);
}

The following code creates an EntityFilter which always returns false, meaning the pushing entity will never collide with any other given entity.
public void setEntity(Entity entity) {
    // ...
    this.entity.getOrCreate(CollisionFilters.class).get().add( (e, el) -> false )); // always disable
}

The following would disable collision with pigs:
public void setEntity(Entity entity) {
    this.entity.getOrCreate(CollisionFilters.class).get().add((e, el) -> {
        return !el.stream().anyMatch(e2 -> e2 instanceof Pig);
    });
}

When a collision is about to occur, the EntityFilters are called first prior to any collision processing, if all entity filters fail to return true, then all collision code is terminated. However, if one returns true, then the code proceeds onto event triggering, and so on.
Needless to say, this opens up possibilities to create a mini library of EntityFilters that makes it easier to combine certain filters together.
Disclaimer: As mentioned before, limitation wise, there's no discussion. However this makes writing collision handlers more focused on entities themselves rather than listeners then the entities themselves.

I don't really see any advantage over just listening and cancelling event appropriately.
Also, the entities in CollideEntityEvent are who you are colliding with, the source doing the colliding is in the Cause. There is a filter method on the event that would do what you want.


Connection reuse between MixnetNodes

Currently, each MixnetNode establish a TCP connection with the other MixnetNode every time they need to communicate each other: https://github.com/logos-co/nomos-node/blob/84dfb0fe6b63b1764614086f70857d2e41460889/mixnet/node/src/lib.rs#L50-L50.
I've used this way for a quick PoC, but it's not a good idea in terms of the overhead of connection establishment.
Assuming that the mixnet topology is not changed often, it would be great if each MixnetNode hold TCP connections and reuse them while the connections are alive.
(Currently, we don't have any design about how often we'll update the mixnet topology.)
We also need to handle disconnected connections, but it would be simpler to do that as a next step unless there is any simple way (library) that does all.
(FYI, I chose TCP without any serious consideration. If you have any other transport better, please feel free to suggest it).

This issue is important because I got the following errors when trying to send mulitple messages to the mixnet in parallel.
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)
failed to handle conn: Too many open files (os error 24)

What is your OS file descriptor limit? It would be nice to being able to retrieve that limit, as something below that would be the total number of connections we can handle.

It's 256 in my macOS. Yes. We should be able to retrieve this limit and use it for managing connections.
Currently, this error occurs because we always try to establish new connections every time we want to forward a single Sphinx packet to the next mixnode.

What is your OS file descriptor limit? It would be nice to being able to retrieve that limit, as something below that would be the total number of connections we can handle.

It's 256 in my macOS. Yes. We should be able to retrieve this limit and use it for managing connections. Currently, this error occurs because we always try to establish new connections every time we want to forward a single Sphinx packet to the next mixnode.

Do we actively close the connections? I'm not sure if they should be close when droped 🤔

See my comment in https://github.com/logos-co/nomos-node/pull/343#issuecomment-1698895118


Deny duplicates in proxy registration

What does this PR do?
With this PR, the POST-handler for apis checks if the proxy you are about to register already exists in the database. It denies it if it does and prevents the database from faulty state.

👍


LoraRF - SX126X - set device EUI

Hi,
I'm currently working on a project that involves sending data from SX126x (connected to raspberry pi 4b) to a Mikrotik Lora Gateway and then to ChirpStack. Browsing through multiple docs and projects I found that I need a Device EUI set on my SX126x in order to connect it to the server in order to receive data from it. On this repo I did not find any referece to Device EUI. Can you please help me with my project?
Regards,

I am also attempting this exact thing -- a MiroTik R11e-LR9 that is connected to ChirpStack, but also The Things Stack.
Looks like examples/network is close to what we're looking for: it uses a struct to assemble data fields into a binary packet.  But it's not exactly LoRaWAN, since it doesn't include OTAA or ABP functions (which require an App ID and Key, along with the Device EUI as you pointed out).
I think what we're going to have to do is learn how to implement OTAA or ABP ourselves so that we can join the MikroTik gateway, and then learn how to construct a raw LoRaWAN packet.
In other words, LoRaRF doesn't directly support LoRaWAN, but I think we can implement LoRaWAN ourselves with the help of some of this library's functions.
I've used LoRaWAN on Arduino, so my approach to this will be to see how one of those libraries work so I can replicate that in Python with this library.

Did you get this working? I'm trying to join a chirpstack gateway but can't find a propper python example.


Kpt fn creating duplicate resources

Example (recreate issue):

Create a kpt pipeline:
a)gcr.io/kustomize-functions/example-nginx
b) gcr.io/kpt-functions/set-namespace
Run kpt fn run . in the directory with the kpt pipeline twice.
View contents of either deployment_.yaml or service_.yaml.

You will see that the two configurations are really similar. The difference is in the ordering of the metadata section. One of the duplicates will have the metadata ordered as [name, namespace, labels] and the other will have [name,labels,namespace].
The problem here is that the deployment and service yamls will store what is essentially the same configuration twice.

/assign

@felixtran39 So if I understand what you're saying: the set-namespace function is not idempotent. Is that correct?

@seans3 Not exactly. Based on my understanding, this is what I think is happening.
The first time the pipeline runs: deployment/service resources are generated and then transformed with set-namespace. However, since the generated configs don't specify the namespace, set-namespace will specify a namespace at the bottom of the metadata section even if this does not agree with the ordering that kpt uses.
The second time the pipeline runs: the deployment/service resources generated from the first run are out of order, so the ordering of namespace and labels is corrected. The actions from the pipeline are then repeated as in the first run. However, the output of the pipeline has "incorrect ordering" so it ends up concatenating this configuration to the deployment/service resources, causing duplicates.

@felixtran39 What is your expectation for how kpt should behave on duplicate resources?
The example-nginx function is a generator function, so it creates new resources from templates each time it is run. If you create a kpt pipeline with only gcr.io/kustomize-functions/example-nginx and run it twice, do you expect to get duplicate resources or do you expect kpt to have de-duplicated the repeated resources?

@prachirp I expect kpt to de-duplicate the repeated resources.

@droot interesting case to consider here

@mikebz yes, this is going to be common occurrence if one is using generator function so de-duplication will address most of the cases we care about. Marking it 1.0 for now.

@mikebz yes, this is going to be common occurrence if one is using generator function so de-duplication will address most of the cases we care about. Marking it 1.0 for now.

Though this was reported for kpt alpha, we need to ensure behavior is clear for kpt v1.0.

This is a duplicate of https://github.com/GoogleContainerTools/kpt/issues/2466 so closing this.


fix(extras): migrate away from deprecated get_target_window() in mini-files

Description
This method is deprecated in the most recent version of mini-files: https://github.com/echasnovski/mini.files/blob/a3a9cce82115a69dba161ac45bda16f4e606f73b/lua/mini/files.lua#L1060-L1068, which causes a warning message to be displayed every time the <leader>wv and similar commands are used from the mini file browser.
I've tested this change locally and it gets rid of the warning message.
Screenshots
Warning message without this fix:

Checklist

[x] I've read the CONTRIBUTING guidelines.

ty!


Docker containers in CI

In terms of CI run efficiency it is better to use docker container with all installed dependencies.
Tasks:

To create docker image with installed dependencies(only boost for now) for all systems(macos, linux, windows).
To make it possible to buid our project inside docker container.
Edit github workflows so that it can build project inside this docker container.

@temikfart @cadnev WDYT?

@c71n93 notice:

docker image with installed dependecies must be uploaded on the Docker Hub on our organization account
build project with Dockerfiles for each OS and save these Dockerfiles in the docker folder
before editing CI yaml files, we need to make sure, that docker is installed on CI host machines.

Also, I think, that we can use only one runner, for example, ubuntu. Because we use docker.


Data Portal - theme links review

Mehdi asked to link the Biodiversity theme to an advice tag.
https://vocab.ices.dk/?codetypeguid=c1887911-3ea7-4ad9-a844-ba989e6b5b9c
Currently it is linked to Sea birds and Marine mammals in State, but it could need more links.
https://vocab.ices.dk/?codetypeguid=6410af4e-cbf3-4770-b229-939fb763edb2
https://vocab.ices.dk/?codetypeguid=ed99774f-e15f-46d2-8220-2fe1bb30c4e9
https://vocab.ices.dk/?codetypeguid=26971431-ad56-4962-ae59-59bfb5902328

@pcrjoana to link BD to research and tourism in humanactivity, mil, fisher, shp, power2, power1, telecom .
Link noise to  Power1, power2, SHP, aggex, mil, telecom, research.
Link all themes to research.

Add new links as follows:

Theme
Human   Activity
Pressures
State

AS

Food, PHab

BD

BIO

NOE
Food

BM
RES

Fish,Phab,Bhab

CAT
RECR, consider COLL

all

CON

Fish

FTS

Food,Phab

ICS

SES
Food

ML
RES

BHAB

NOISE

OCEAN

NOE
BHAB

SA
RES

Fish

SD
Fisher

VME

done collectively by the RMG


The Plex Music Provider marks every artist, album, and song as favorite.

What version of Music Assistant has the issue?
2.0.4
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Every single Artist, Track, and Album from the Plex provider is marked as favorited.
I can manually unfavor them, but they should not be favored by default.
How to reproduce
Add the Plex provider, and add your Plex music library
Music Providers
Plex
Player Providers
Not Applicable
Full log output
No relevant log output found.
Additional information
Creating issue on the request of @marcelveldt re this discord thread https://discord.com/channels/753947050995089438/1098863624073785364/1240278225423175801
What version of Home Assistant Core are your running
2024.5.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

@cnf please confirm this is fixed in 2.1.0b4

Was having the same issue as @cnf and I can confirm that it appears to be fixed in 2.1.0b4

Sorry for the late response, I have been dealing with some personal things.
Can confirm Plex no longer favorites everything in 2.2.5
It also won't favorite anything starred, is this expected/intended?
Thanks!

I don't use Plex so is a STAR the same as a favourite or can you give a item multiple stars (like a rating)?

Yep, it is a multiple star rating system, so no direct match to "favorite"
A setting to match "more than X stars maps to favorite in MA" would be nice, but that's a feature request, not a bug
The report issue is no longer showing up in my current version, 2.2.5.
Thanks!


adapt a model

How can I adapt a model from spandrel to dgenerate? Which lines, which files should I delete and add? Can you tell me or help me?
for example:
How do I add this model to dgenerate?
https://github.com/chaiNNer-org/spandrel/tree/main/libs/spandrel/spandrel/architectures/SAFMNBCIE

If they have not made a release with this architecture present, you can try installing spandrel directly from their main branch into the environment that dgenerate is using, this might work if you have installed dgenerate manually into a virtual environment.  It will not work for the windows installer.
You might need to install Git For Windows to use this pip feature
pip install git+https://github.com/chaiNNer-org/spandrel

I did pip install git+https://github.com/chaiNNer-org/spandrel
Then I manually moved the files from https://github.com/chaiNNer-org/ or C:\Program Files\Python311\Lib\site-packages\libs\spandrel spandrel to C:\Program Files\dgenerate345lite_internal\spandrel folder.
I could not run the SAFMNBCIE model in dgenerate.

It will not work for the windows installer environment, for your usage I would suggest not using the installer and instead installing dgenerate into a virtual environment
for instance
# create environment

python -m venv dgenerate_env

# activate environment, must be done every time before running dgenerate

dgenerate_env\Scripts\activate

pip install dgenerate --extra-index-url https://download.pytorch.org/whl/cu121/
pip install git+https://github.com/chaiNNer-org/spandrel

I could not get the newly added https://github.com/chaiNNer-org/spandrel/tree/main/libs/spandrel/spandrel/architectures/SAFMNBCIE model to work at all.
It is best to make a command or .bat file that will add and run the new auto detect model to the dgenerate.

spandrel updated. 0.3.2 version


Update webp coder

This improves RAM usage for static webp assets (see https://github.com/SDWebImage/SDWebImageWebPCoder/pull/30)
Potentially related to https://github.com/DylanVann/react-native-fast-image/issues/622

Any updates on this?


Is it possible to use router-links with vue-markdown?

Obviously the following code
[contact us](/contact)

gets translated into
contact us

Without vue-markdown I could write
<router-link to="/contact">contact us</router-link>

which will use client side routing instead of reloading the page. Is this possible to use this concept with vue-markdown?

I couldn't find an easy or efficient way to do this.  But I did find a way.
What I did is defined an alternate markdown syntax for internal links as [text]{my-internal-link}.  Notice the curly braces instead of parentheses.  Then I defined a vue-markdown prerender method to intercept the markdown, extract the link information and ultimately substitute the content with a router-link.
I had to jump through a few hoops to make this happen, but I was able to get it to work in the end.

I couldn't find an easy or efficient way to do this. But I did find a way.
What I did is defined an alternate markdown syntax for internal links as [text]{my-internal-link}. Notice the curly braces instead of parentheses. Then I defined a vue-markdown prerender method to intercept the markdown, extract the link information and ultimately substitute the content with a router-link.
I had to jump through a few hoops to make this happen, but I was able to get it to work in the end.

Could you detail your method with some code sample ?

Sure.  Like I said, I had to jump through some hoops, so open to simplifying things.  Ideally these features could be incorporated into vue-markdown.
Markdown
This is my md file.

[normal link](http://www.google.com)

[vue link]{/my/app/route}

Component Usage

<markdown :source="source">
  <template v-slot:image>

  </template>
</markdown>

Component Code
<template>

    <vue-markdown
      v-bind="$attrs"
      :toc="true"
      :breaks="true"
      :linkify="false"
      :toc-anchor-link-space="true"
      toc-anchor-link-symbol="¶"
      :prerender="prerender"
    >
      <slot></slot>
    </vue-markdown>

      <slot :name="s" v-if="slotsInserted">
      </slot>

    <router-link v-for="(link) in internalLinks"
      :to="link.to"
      :key="link.id"
      :ref="link.id">{{ link.text }}</router-link>

</template>

This feature should be prioritized.  vue-markdown was so awesome, until I hit this issue. All my pages are FULL reloading.  I use vue-markdown for localizing many of my pages.

Another (hacky) way I'm not proud of, but hey... it works!
document.querySelectorAll('#yourElement a').forEach(a => {
  a.addEventListener('click', async (e) => {
    e.preventDefault()
    this.$router.push({ path: a.attributes.href.value })
  })
})

not having support for this is outrageous

Was this ever addressed?

For anyone else who happens to stumble upon this facing the same issue, here is the hacky approach I took which combines a bit of methodologies from other commentors.
<template>
  <vue-markdown v-bind="$attrs" :linkify="false" ref="VMD">
    <slot></slot>
  </vue-markdown>
</template>

A simple wrapper that will replace a tags within the instance of that component with router-links after it's mounted. For my case, we're using markdown for the content of a Notifications list which also loads more Notifications as you scroll down, so I needed it to continually replace a tags as they were rendered.


[Roll for Shoes] Clean up dice display, add applied modifier number

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Changed: Visual improvements to the individual dice display
Added: Number for effective (non-zero) modifier number from roll prompt

Character Sheet Info Roll20 Internal Use only.


Removed public APIs from FeedOptions for page size and continuation token

Reduced scope of public APIs for page size and continuation token to package-private.
Updated live tests to use CosmosPagedFlux byPage(continuationToken, pageSize) APIs

/azp run java - cosmos - tests

/azp run java - cosmos - tests


refactor(google fonts): self host google fonts (DEV-471)

resolves DEV-471

I assume you have downloaded the latest version (especially the one for the icons)?

yeah, specifically the roboto fonts (from v19 -> v29)


There is no way to add alarms for the event

please implement alarms to the add event so that alarm can be triggered if it needs to be for specific events. for example to trigger alarm 10mins, 15 mins, 30mins, before the event.

@akshay030123 Creating calendar events with options is in the backlog to be implemented. But since finding and deleting events are not yet implemented, they have priority over this. Once those are implemented, this will be next.

@akshay030123 Creating calendar events with options is in the backlog to be implemented. But since finding and deleting events are not yet implemented, they have priority over this. Once those are implemented, this will be next.

I already implemented alarms, will submit a PR. Do you have an ETA on deleting events, I need that. What about updating an event?

@akshay030123 Creating calendar events with options is in the backlog to be implemented. But since finding and deleting events are not yet implemented, they have priority over this. Once those are implemented, this will be next.

I already implemented alarms, will submit a PR. Do you have an ETA on deleting events, I need that. What about updating an event?

I would expect to roll out deleteEvents by EOW. But for your PR, if possible i would ask you to wait at least until I have released version 1.0.0 as I have not yet fully defined the project requirements for myself. For updating events I dont have a defined timeline yet. My idea was to first implement create, read and delete so that i know how exactly a calendar event is defined so that i know what could be updated. Therefore the updating events will probably be picked up after delete events is functional.

Hey @ebarooni, the code to add alarms when creating an event is here:
https://github.com/aparajita/capacitor-calendar/commit/695809ce03f3a7977be9137b307b53742f9457e9


Update UserGuide.md

Codecov Report
Base: 70.59% // Head: 70.56% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (b0d5a38) compared to base (dd98861).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #179      +/-   ##
============================================
- Coverage     70.59%   70.56%   -0.03%
- Complexity      617      618       +1
============================================
  Files            99       99
  Lines          2200     2208       +8
  Branches        234      237       +3
============================================
+ Hits           1553     1558       +5
- Misses          585      586       +1
- Partials         62       64       +2

Impacted Files
Coverage Δ

.../address/logic/commands/AddAssignmentsCommand.java
86.20% <0.00%> (-3.80%)
:arrow_down:

...java/seedu/address/logic/commands/EditCommand.java
95.94% <0.00%> (ø)

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

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


Calendar Section

Is your feature request related to a problem? Please describe
A calendar component showing future events
Describe the solution you'd like
Get a calendar component and fill in the Google API Key as well as the calendar ID. Change the styles as needed.
Additional context

https://github.com/ericz1803/react-google-calendar

We do not plan to work this issue this semester, will be left to the next web dev team.


Shortening Race Condition window for Queued to Sent Batch Failure Condition

Description
Shortens window during which a process restart would require manual intervention to fix rollup batches.
Questions

Is there an easy way to get the tx hash ahead of sending so that we can eliminate this race condition? Note: my main problem is with estimating and setting gasLimit and gasPrice -- I know it's doable by doing a lot more of what ethers wraps manually, but this seems like the right ratio of improvement : time.
Does it make sense to just add wallet nonce to the batch submission table and update that ahead of sending a batch? That way, we can always check to see that the pending tx count is as expected. But if a nonce is stored, can't find the tx associated with it so 🤷

Metadata
Fixes

Fixes # YAS-647

Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

You should be able to get the tx hash by hashing the output of: https://github.com/ethers-io/ethers.js/issues/535#issuecomment-498401610

BatchSubmissionStatus.QUEUED is when its in the mempool and BatchSubmissionStatus.SUBMITTING is when its being sent to the mempool but before its confirmed to enter?


Changing default styleExt to scss gives me Value cannot be found

I've made new project using<EMAIL_ADDRESS>I want to change defalt style file to scss but command ng config defaults.styleExt=scss gives me  Value cannot be found.
How to change default style extention?

Heya, you can see here how to change the default for component generation: https://github.com/angular/angular-cli/wiki/stories-css-preprocessors

When I run that command ng config schematics.@schematics/angular:component.styleext scss I get error
Invalid JSON character: "s" at 0:0.
Error: Invalid JSON character: "s" at 0:0.
    at _readValue (C:\strony\www\infirma\admin\node_modules\@angular-devkit\core\src\json\parser.js:567:19)
    at parseJsonAst (C:\strony\www\infirma\admin\node_modules\@angular-devkit\core\src\json\parser.js:605:17)
    at Object.parseJson (C:\strony\www\infirma\admin\node_modules\@angular-devkit\core\src\json\parser.js:631:12)
    at normalizeValue (C:\strony\www\infirma\admin\node_modules\@angular\cli\commands\config.js:128:19)
    at ConfigCommand.set (C:\strony\www\infirma\admin\node_modules\@angular\cli\commands\config.js:195:23)
    at ConfigCommand.run (C:\strony\www\infirma\admin\node_modules\@angular\cli\commands\config.js:168:18)
    at C:\strony\www\infirma\admin\node_modules\@angular\cli\models\command-runner.js:278:30
    at Generator.next (<anonymous>)
    at fulfilled (C:\strony\www\infirma\admin\node_modules\@angular\cli\models\command-runner.js:4:58)
    at <anonymous>

Looks like our string processing is broken but we're taking a look at it in https://github.com/angular/angular-cli/pull/10956, sorry for that!

No problem

@filipesilva i'm getting that same error when running that command on v6.0.5. The merged PR didnt quite work i guess. Can you confirm?

@n-sikka the merge is in the v6.0.7. I had this problem and with that version it is solved.

The same problem with v.6.0.8

Why this was closed? I have this problem with v6.0.8.

@piernik When you say put
"schematics": {
        "@schematics/angular:component": {
          "styleext": "scss"
        }
      },

in every project, where exactly do you put it?  Do you have to put it in all of the projects?
Could you show some context?

You have to add in in every project:
{
  "$schema"<EMAIL_ADDRESS>...
  "projects": {
    "frontend": {
      ...
      "schematics": {
        "@schematics/angular:component": {
          "styleext": "scss",
        }
      }
}

@sannminwin you need to rename the *.css to *.scss files

I found the command, in the stackoverflow site:  ng config schematics.@schematics/angular:component.styleext scss. This command will automatically add the "schematics" section halfer mentioned to the angular.json file e generate the next components with scss instead of css

I found the command, in the stackoverflow site: ng config schematics.@schematics/angular:component.styleext scss. This command will automatically add the "schematics" section halfer mentioned to the angular.json file e generate the next components with scss instead of css

Didn't work for me with:
Angular CLI: 7.3.6
Node: 8.9.4
OS: darwin x64
Angular: 7.2.9


[Builtin][IO] Add close channel for byte-io, char-io, record-io samples

Currently the samples elaborated does not elaborate closing of I/O channel. Need to add that

The issue is fixed since 0.970.0-alpha


version?

Maybe I am blind, but I am not seeing a revision anywhere so that I know when a new one is available

nope but you can always commit history to see something got add or delete in code

Does every new commit mean a new version?  Or does it mean something you decided to make a commitment to work on?

Every new commit means a new extra to the existing code.

Just out of curiosity, why are the zip files from the main code page different from the history page?
$ ls -al Windows10Debloater-*
-rwxrw-rw-. 1 todd users 37915 Mar 21 21:12 Windows10Debloater-2019.02.16.zip
-rw-r--r--. 1 todd users 35816 Mar 21 21:13 Windows10Debloater-b202151316d72c65ddfa08e928a465d1d0de2916.zip

History does not show the latest commit:
https://github.com/Sycnex/Windows10Debloater/commits/master/Individual Scripts
Commits on Feb 16, 2019
And the main page does not say what the latest commit is:
https://github.com/Sycnex/Windows10Debloater
"Latest commit 11d50e8 7 days ago"
I am trying to check what the latest version is with a program.   If you put what the latest commit somewhere on the main page, I can carve it out with regex's

Found the latest commit on the main page, not the history page:
https://github.com/Sycnex/Windows10Debloater
span itemprop="dateModified">Mar 15, 2020
You could not make this any harder to find...

History does not show the latest commit:
https://github.com/Sycnex/Windows10Debloater/commits/master/Individual Scripts
Commits on Feb 16, 2019
And the main page does not say what the latest commit is:
https://github.com/Sycnex/Windows10Debloater
"Latest commit 11d50e8 7 days ago"
I am trying to check what the latest version is with a program. If you put what the latest commit somewhere on the main page, I can carve it out with regex's

because that history of commit of Individual Scripts folder not all whole project

If you dig into the source code of the web page, you can find it.  It would be nice if it showed on a browser though


Revert "Initialize Decimal with text value, fix #1035"

Reverts django-import-export/django-import-export#1039 as it is now breaking on master.

Test passes if force_str() is used instead. (force_text() is deprecated in 3.0)

Thanks @matthewhegarty Fixed on master!


Update Composer dependencies (2020-05-29-00-04)

Visual regression test passed!
View the visual regression test report


ResolveAssemblyReferences propagates TargetPath metadata to related files

contributes to internal bug devdiv#980195
RAR copies metadata from primary source items to outputs, including the TargetPath metadatum, and including to "related" non-primary output files. This means that a C# project that references a C++ WinMD project can get an item that looks like this:
C:\source\repos\WapProjTemplate36\Debug\WindowsRuntimeComponent1\WindowsRuntimeComponent1.pdb
    TargetPath = WindowsRuntimeComponent1.winmd

And that TargetPath may overlap with another real file.
There's already
https://github.com/microsoft/msbuild/blob/ed928d8d06ef21eb5795f6a7cc767d37d31e57bc/src/Tasks/AssemblyDependency/ReferenceTable.cs#L2918-L2926
Possibly just add TargetPath to that would solve the problem. But we need to think through the consequences of that.

Team triage: unknown regression risk from removing TargetPath from RAR outputs, and there's no known impact of the confusing metadata, so resolving won't-fix. If you see concrete impact, please comment and we can reevaluate.

@sbanni discovered a more important problem case for this:

Create new C# UWP Project
Create new C++ WinRT Component Library Project
Reference C++ project from the C# project
Build

Observed: ResolveAssemblyReference adds items to ReferenceCopyLocalPaths and the .pri file from the C++ library has a TargetPath ending with .winmd which will overwrite the file

For 16.6, let's look really hard at options in the order:

Don't propagate TargetPath.
Update TargetPath to have the right extension for the extra files.
Add a RAR input bool to control behavior
Figure out a target to fix this up after the fact.

Based on compat impact.


"should_update_effects_offscreen" does not accept booleans

Describe the bug
"should_update_effects_offscreen" in the scripts section in client entity files does not accept booleans.
To Reproduce
Steps to reproduce the behavior:

Set scripts/should_update_effects_offscreen to true
true is marked as invalid because it is not a string

Expected behavior
"should_update_effects_offscreen" and "should_update_bones_and_effects_offscreen" should accept booleans.

The only time I have found it was in minecraft:minecart and there it was used as a string,
Did you test it? then ill happly add it :)

Jannis confirmed that it works :)


Improve the dashboard

Lots of things to happen to the dashboard, some interesting statistics, personalised to the user.

Suggestion of graph based upon issue priority, how many are high medium low. Bar chart perhaps.

make this graph - http://puu.sh/fpPbj/a6f4e9ef65.png clickable to see what the issues were?

Jason had same thoughts, not supported with charts.js but will look into doing something like this. Might look around for different charting libraries.

Expect that this won't be completed in time for beta, have moved to 1.0.0 milestone. Still taking suggestions for improvements.


Add VeChain platform [WIP]

Closes https://github.com/TrustWallet/blockatlas/issues/52
TODO:

[x] Add vthor transactions

I decided use VeChain explorer API instead of thor RPC for couple of reasons

Consistency and accuracy
Speed

Very nice!


Bump ember-cli-babel to ^7.0.0

This PR:

bumped ember-cli-babel to ^7.0.0
bumps node to latest maintained version
bumps ember-cli-addon-docs due to node support version
remove testing ancient LTS versions

Ugh, gotta figure out why the tests are still failing 😫

@lupestro when you have a sec, mind reviewing?

It looks good to me. No issues.

It looks good to me. No issues.

Are you able to merge?


Update Common.cs

Break SendLoop instead of returning when SendMessagesBlocking returns false after an exception so we always close stream

Codecov Report

Merging #61 into master will increase coverage by 0.25%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
+ Coverage   88.01%   88.26%   +0.25%
==========================================
  Files           8        8
  Lines         392      392
  Branches       35       35
==========================================
+ Hits          345      346       +1
+ Misses         47       46       -1

Impacted Files
Coverage Δ

Telepathy/Common.cs
82.52% <0%> (+0.97%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7fbbbe...e0e667d. Read the comment docs.

Codecov Report

Merging #61 into master will increase coverage by 0.25%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
+ Coverage   88.01%   88.26%   +0.25%
==========================================
  Files           8        8
  Lines         392      392
  Branches       35       35
==========================================
+ Hits          345      346       +1
+ Misses         47       46       -1

Impacted Files
Coverage Δ

Telepathy/Common.cs
82.52% <0%> (+0.97%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7fbbbe...e0e667d. Read the comment docs.

Maybe a better PR name?

Sorry, first PR

Brief chats with @vis2k to try and resolve issues with unclosed streams on Socket Exceptions lead to https://github.com/vis2k/Telepathy/commit/7e64f2de92b5ca3cb992f66b3383a140cb189a27 - that didn't resolve my issue, this should

I'd also use try-catch-finally here, iirc finally always executes, no matter what

How about we put the close statements inside a finally block instead?

I've taken the lead from ReceiveLoop:
if (!ReadMessageBlocking(stream, MaxMessageSize, out content))
    break;`

So maybe we're looking at putting lines 212 to 220 inside a finally a well


clang toolchain compiling c++ with 'clang' instead of 'clang++'

Xmake Version
v2.9.6+20241030
Operating System Version and Architecture
Linux 6.11.7 NixOS (Unstable) x86_64
Describe Bug
I've been trying to use xmake lately to build projects, however when I try to compile using a clang-based toolchain, I am greeted by a compile error as the compiler is not finding the c++ standard headers due to it using the c compiler instead. I tried to search any duplicates to this issue but haven't found anything.
Here's the MRP:

flake.nix:

{
  description = "MRP";
  inputs.nixpkgs.url = "github:nixos/nixpkgs/nixos-unstable";
  outputs = { self, nixpkgs }:
    let
        pkgs = import nixpkgs { system = "x86_64-linux"; };
    in
    {
      devShells."x86_64-linux".default = pkgs.mkShell.override { stdenv = pkgs.clang19Stdenv; } {
            packages = with pkgs; [ xmake ];
      };
    };
}

test.cpp

#include <iostream>

int main() {
    std::cout << "Hello World?" << std::endl;
    return 0;
}

Expected Behavior
The code compiles correctly.
Project Configuration
---@diagnostic disable: undefined-global

target("test")
    set_kind("binary")
    add_files("test.cpp")
    set_languages("c++20")

    set_toolchains("clang")

Additional Information and Error Logs
checking for platform ... linux
checking for architecture ... x86_64
checking for clang ... /nix/store/l4bagvaywzkq3cbjd98rjkqwc0vkpxxw-clang-wrapper-19.1.3/bin/clang
checking for the c++ compiler (cxx) ... clang # (shouldn't it be clang++?)
checking for /nix/store/l4bagvaywzkq3cbjd98rjkqwc0vkpxxw-clang-wrapper-19.1.3/bin/clang ... ok
checking for flags (-fPIC) ... ok
> clang "-fPIC" "-Qunused-arguments" "-m64"
checking for flags (-std=c++20) ... ok
> clang "-std=c++20" "-Qunused-arguments" "-m64"
[ 50%]: cache compiling.release test.cpp
/nix/store/l4bagvaywzkq3cbjd98rjkqwc0vkpxxw-clang-wrapper-19.1.3/bin/clang -c -Qunused-arguments -m64 -std=c++20 -o build/.objs/test/linux/x86_64/release/test.cpp.o test.cpp
checking for flags (-MMD -MF) ... ok
> clang "-MMD" "-MF" "/dev/null" "-Qunused-arguments" "-m64"
checking for flags (-fdiagnostics-color=always) ... ok
> clang "-fdiagnostics-color=always" "-Qunused-arguments" "-m64"
error: @programdir/core/main.lua:329: @programdir/actions/build/main.lua:148: @programdir/modules/async/runjobs.lua:325: @programdir/modules/private/action/build/object.lua:91: @programdir/modules/core/tools/gcc.lua:938: test.cpp:1:10: fatal error: 'iostream' file not found
    1 | #include <iostream>
      |          ^~~~~~~~~~
1 error generated.
stack traceback:
    [C]: in function 'error'
    [@programdir/core/base/os.lua:1004]:
    [@programdir/modules/core/tools/gcc.lua:938]: in function 'catch'
    [@programdir/core/sandbox/modules/try.lua:123]: in function 'try'
    [@programdir/modules/core/tools/gcc.lua:879]:
    [C]: in function 'xpcall'
    [@programdir/core/base/utils.lua:244]:
    [@programdir/core/tool/compiler.lua:288]: in function 'compile'
    [@programdir/modules/private/action/build/object.lua:91]: in function 'script'
    [@programdir/modules/private/action/build/object.lua:122]: in function 'build_object'
    [@programdir/modules/private/action/build/object.lua:147]: in function 'jobfunc'
    [@programdir/modules/async/runjobs.lua:241]:
    [C]: in function 'xpcall'
    [@programdir/core/base/utils.lua:244]: in function 'trycall'
    [@programdir/core/sandbox/modules/try.lua:117]: in function 'try'
    [@programdir/modules/async/runjobs.lua:223]: in function 'cotask'
    [@programdir/core/base/scheduler.lua:406]:

stack traceback:
	[C]: in function 'error'
	@programdir/core/base/os.lua:1004: in function 'base/os.raiselevel'
	(...tail calls...)
	@programdir/core/main.lua:329: in upvalue 'cotask'
	@programdir/core/base/scheduler.lua:406: in function <@programdir/core/base/scheduler.lua:399>

The issue is most likely not from xmake using clang instead of clang++, clangs set up as a C++ compiler when a .cpp file is passed, this is probably coming from a missing libstdc++
I don't know nix though.

The issue is most likely not from xmake using clang instead of clang++, clangs set up as a C++ compiler when a .cpp file is passed, this is probably coming from a missing libstdc++ I don't know nix though.

@SirLynix You're right, After looking through clang verbose output, I saw that it's not xmake's fault. I'll see what I can do. Thanks!


Local development improvement

Summary
A single node docker based development environment that is developer friendly.
Problem Definition
Right now it's very easy to integrate with make localnet-start. The reasons for this is because it starts up provenanced with docker and provides some developer friendly utilities such as having a large pool of nhash in the genesis. P8e for instance can launch this and then easily import a key and transfer nhash to it.
The problem with this is that localnet is not ideal for local development of dependent services. It is heavy weight for a laptop and in heavy load scenarios the block cut time quickly starts to increase. I'm hopeful this is mostly due to running the full setup on a laptop.
Using the single, native provenance setup is not ideal for someone developing dependent services that already make use of docker because you then have to manage the stopping and starting of the native binary in scripts. We were doing this previously, but it's more error prone.
Proposal
A make target that spins up a single node provenance environment making use of a docker image with a sensical genesis file (loaded with nhash).
Taking this a step further it might even be valuable to have a few documented hardcoded mnemonics associated with addresses that have nhash as part of the genesis as well. This would remove the need for dependent services to launch the development environment and then have to create/import keys and transfer nhash as part of a one time setup script. I assume this step is mimicked in most dependent systems that start a local development provenance environment.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

As far as the original ask, this gets closed by https://github.com/provenance-io/p8e-scope-sdk/pull/4 for the metadata module use case. I'm not sure if there's value in duplicating some of that in a standalone environment within this repo or not.


Fix Apostroph

Fix: Grammar: Plural words in Germany don't take an apostrophe.
I also removed obsolete empty lines - and added when recommended by markdown lint's rules

Danke, euch fällt auch echt alles auf! 😄


"ReferenceError: self is not defined" on "embr fastboot"

Here's what I get when I try to run ember fastboot. To the best of my knowledge, everything is installed correctly, but apparently not.
elp2:example_project user$ ember fastboot
version: 0.2.2
Built project successfully. Stored in "dist".
self is not defined
ReferenceError: self is not defined
    at <anonymous>:63785:35
    at <anonymous>:75317:4

This is in a completely clean project that was created using ember new.
Any ideas?

What version of Ember?

Well, I get this for the Ember CLI:
elp2:example_project user$ ember --version
version: 0.2.2
node: 0.12.1
npm: 2.7.4
elp2:example_project user$

And this is the version of Ember I'm using:
/*!
..
 * @version   1.13.0-beta.1+canary.57f1fa1b
...
*/

Hope that helps.

@LukeLaupheimer Is this the same error as #11? If so, looks like an issue with Ember Data. I'll investigate.

@tomdale @LukeLaupheimer I have a PR to ember data...

Fixed upstream.  Ember Data canary builds should be working properly now.


Add pause skill timing feature to the XP Tracker

Add the ability to manually pause a skill, automatically pause a skill after logout, or after a timeout of no XP changes. Similarly, skills will be auto unpaused when XP is gained or when (if paused due to logout) logged back in.
Motivation
Some players would like to pause their expectations on skill ETAs when having lunch, going on a break, or sleeping for the night. By only counting effective time in game, players don't need to account for time outside of game or between skills when training.
Configuration changes

Added a setting for pausing skills without any XP changes, default value: 0 which behaves as no auto pausing by timeout
Added a setting for auto pausing skills in relation to logout, default value: Never
Changes to default behavior
No changes to default behavior
UI Changes
ProgressBar now extends a new Component class which comes with dimming of the foreground and background color.
ProgressBar will also dim the left and right label, with the expectation that the center label be used for an alternate display text.

Within the XP Tracker panel, right clicking on a skill will bring up a "Pause" menu item.

When a skill is paused, it will be labeled as "Unpause"
Example

Resolves #3598
Resolves #3113

Changes addressed for @deathbeam

Style changes made.
Can squash when everything looks good.

Changes addressed for @deathbeam

Rebased, altered again towards @deathbeam 's wishes.

Squashed into two commits, one for sharable UI changes, one for the pausing feature.
@Adam- @deathbeam

If you enable pause on logout while logged out it doesn't pause.


add pcode to wide table labels

email from gkeith:
Can you output the parameter code # with the wide data table? I just see a short description one for example says pH (not lab, not field, not filtered or unfiltered). I'm not sure how easy this would be but if the pcode and NWIS description was there, it would be easy to spot data having been uploaded into the wrong parameter.

closed by PR #165


add some libraries from mono

@plantain-00, since you're introducing the "Compression" section, it's reasonable to migrate "DotNetZip" and "SharpZipLib" from Misc.

The "XSP" project can be placed under already existing "Web Servers" section, which reflects the nature of the "XSP" project more precisely (from my perspective).

@vladen , Thank you, I am glad that someone reviews it.

My excuses for delay with reviewing ;)


Fix CppGenerator: handling of fixed/variable list types

fixes #13
fixes #15

@phoppermann thank you for this. I overlooked it until now.
Could you investigate the merge conflict and update the PR with a commit that address the conflict?

@leif81 I just had a look at the conflicts. Unfortunately, I'm not completely aware of the changes on master in the meantime. Would you mind trying the merge to avoid that I break something?

I have a question regarding this auto-indent: my idea was that I could simply run the auto-indentation on my branch to better see the real differences. However, I do not quite understand currently how this auto-indentation is/has been done. I was expecting something like formatter-maven-plugin.

I did it with Netbeans using the default formatting settings for Java.
I wasn't aware of the formatter plugin, I'll try that in the future.

See #23.

The merge is finished. It should be fine but nevertheless you should have a detailed look at it and try it out because the merge wasn't quite straight forward as you know.
General remark: I suggest to squash the commits when merging such that you do not get all tiny commits from the branches in master.

Thank-you @phoppermann


Does fail on errors

If I don't have security configured correctly for build service account, I can see in the logs that the error occurred, but the task completes with a success.

I will look into this and make sure it fails when it should :) - thanks
Sent from my iPhone

On 13 Jul 2017, at 21.12, GPetrites<EMAIL_ADDRESS>wrote:
If I don't have security configured correctly for build service account, I can see in the logs that the error occurred, but the task completes with a success.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

unfortunately, no exception is thrown - so a "simple" try catch will not alleviate this problem.
this would also cause the task to fail anyway.
I will get in touch with microsoft to see if this can be resolved in their end :)
Seems like an implementation issue if the command does not throw an exception, but gives an error.

Hi
Do you have any workaround for this issue? I have the same behavior.
log output:
2017-07-19T11:21:04.1349417Z ##[command] git checkout buildstorage
2017-07-19T11:21:04.1969479Z error: pathspec 'buildstorage' did not match any file(s) known to git.
2017-07-19T11:21:04.2049487Z ##[command] git tag  Release/identity/1.0.0-alpha9
2017-07-19T11:21:04.2669549Z fatal: tag 'Release/identity/1.0.0-alpha9' already exists
2017-07-19T11:21:04.2729555Z ##[command] git push origin Release/identity/1.0.0-alpha9
2017-07-19T11:21:05.0630345Z Everything up-to-date

@bqstony.
You issue doesnt seem to be security. The log will say something about missing permissions.
You seem to have not checked the "force" checkbox.
As the tag already exists.
But does the task not give an error? - I am still looking into the error handling though.

I might have a fix for the issue. Will try to push changes later today.

Fixed in 6bfd4b1c10077035ba1d3d5ff0710e67cf60f851