Add support for Stream construction from Iterators.

As defined here, the Iterator protocol is met by any object that implements a next method, with the expectation that such a function returns a hash of { value, done } for each subsequent call.
This PR implements support for creating streams from such Iterators, and tests three independent implementations (including one from ES6, via Babel).

Related to #37.

It's also interesting to note that the internal Iterator model differs from the object returned from Stream().iterator() in that it doesn't actually conform to the Iterator protocol.
I'd be happy to take a look at making that change as well (since it would remove the need for some of the noise like the IteratorIterator type), but I figured I'd send this out before I made any move to further disrupt the codebase.

Thanks for your contribution. Looks good at first glance. I'll look into the details tomorrow.
The reason that I've decided not to stick to the Iterator protocol for internal processing is memory consumption. I'm concerned that wrapping each object into an object hash {value, done} would unnecessarily pollute the heap when processing large amounts of data.
Though the project is lacking any serious perf test so this is all blank theory.

I can definitely see how memory consumption would be a concern, but it does seem like a bit of a premature optimization from my perspective.  Let me see what I can draw up quickly as a performance test – the IteratorIterator from this PR should actually give us some useful data in that regard.

Here are the preliminary results from my experiments.
Tests were run 20 times with a 100,000,000 element (lazy) source.
Test case: Direct Execution
Timings:

Min: 268
Max: 303
Avg: 280.85

Memory Profile:
Notes: This test was run to determine the lower bounds on time and memory requirements for the operations being performed.
Test case: Stream.range
Timings:

Min: 3258
Max: 5301
Avg: 4841.75

Memory Profile:
Notes: The slow steady growth is nothing to be particularly concerned about – it steps once per run of the test, which implies that the step up is just storing the run's statistics.
Test case: Stream.range
Timings:

Min: 3587
Max: 5534
Avg: 4759.1

Memory Profile:
Notes: Remarkably similar memory profile.  Overall not particularly surprising, considering we're discussing the creation of a single, short-lived object per-step; it's entirely possible that the JIT is optimizing the object away internally, since it's effectively being used as a tuple to pass two values to the IteratorOp#advance method.
Overall, I'd say that conforming to the Iterator protocol internally is likely to have less of a negative impact on either memory usage or performance than initially feared.

Thanks for your effort. So it seems we're good to go to just stick with the iterator protocol. I'll create another issue for that.

I've added the new constructor variant to the APIDOC.


About text a bit misleading regarding postal code amount for Germany

About text of this repo says:

Collection of postal codes in different formats, ready for importing. Germany (16481 total) [...]

But after reading CSV file into a DB which has a table for each type of entity (e.g. city, postal_code, community, state, etc.) I come to the following values for Germany:

14.584 cities
7.150 postal codes
411 rural districts (communities?)

Some postal codes are used multiple times, like 04720 for Döbeln and 5 other cities (including Großweitzschen).
I'd say 16481 postal codes is a bit misleading and may confuse people who wanna correctness of their numbers.
What do you think?

Hey, yes some codes are multiple times in the list. There are some villages with same postal code and I think that is ok. Postalcode are not unique.
I have updated the data from geonames and updated the numbers in the brackets.


Cleanup rot

As described in labscript-suite/labscript-suite#75, workflows have accumulated a bit of rot (mostly in the Release stage). Most of the changes are pretty simple things that should be fine.
However, it appears the official github actions for create-release and upload-release-asset have been abandoned. There are a few options suggested to replace them and it appears any of them can do both actions. I ended up choosing GH Release because it has the most stars (maybe not a great strategy). I'm less confident I've done that conversion correctly, and I can't remember how to actually test beyond making a PR to the main repo.

I needed to push out a fix for zprocess and hit all this bit rot. Thanks for addressing so much of it. I've added one more that was discovered here: https://github.com/chrisjbillington/desktop-app/pull/23.
With all these fixes, zprocess is building and releasing, so that's a good sign!
Let's merge this, and then confirm later if there are remaining issues for packages with C extensions, which zprocess isn't, so that's not tested yet.


Update externalId on login

With this PR the field externalId of the user is added to the attributes that are updated on user login.
Background:
When creating a shadow user manually the externalId field is typically not set. When a such a user now does a login, the field will not be updated and the field will stay empty. However if the shadow user is created during the first login of a user, the field will always be set. So the two types of users look different, which should not be the case.
With the proposed change the field will be updated when a user does a login and afterwards the field will always be set - regardless of how the shadow user was initially created.
This also updates the externalId if the value changes, e.g. if the IdP is changing the format (say from using an id to using the email)

Hi @tack-sap, thanks for the PR!
Could you help me understand the use case driving this change?

What is motivating the manual creation of shadow users?
What is the impact of the external id not being updated?

What is motivating the manual creation of shadow users?
The motivation to create them before their first login is, that some authorizations shall be assigned to the user already before he logs on to the UAA for the first time. So if I want to onboard a new user I can do it all upfront instead of waiting for him to login to the UAA before I can onboard him. (this is not necessarily a manual step, but the shadow users could also be created automatically by an external instance)

What is the impact of the external id not being updated?
As mentioned above: If this value is never updated, it will be empty for some users and filled for other users - so you cannot rely on it. So any usage of this attribute will be affected, in our special case we have customers that want to use this value to map the user of the IdP accross different environments.

@shamus Any update on this PR? It looks quite straightforward to me. Any objections?


OVS Dpdk merge

Hi,
request to merge the latest patch set specific to OVS DPDK. This patchset specifically deals with fixing a bug related to mempool management for DPDK and vhost ports.
Patchset has been reviewed and Acked by a number of trusted contributors.
Patchset has been validated follows:
Clang/Sparse compilation and analysis.
OVS Unit tests
Vsperf Performance tests for OVS DPDK for p2p, pvp, pvvp,pvpv
Vpserf integration tests for OVS DPDK for p2p, pvp, pvvp.
MTU tests for both physical and vhost DPDK port types.
Multi queue for DPDK and vhost port types with both kernel virtio and DPDK virtio interfaces in the guest.
Thanks
Ian

I merged this manually, from the command line.  Thank you!


[Feature Request] Support for new validation features introduced in ARC spec v2

As specified one https://github.com/nfdi4plants/ARC-specification/pull/95 is merged, there are 2 new quality control mechanisms defined for ARCs:
Validation packages
Validation packages are any kind of executable set of validation cases an ARC must pass. They are uniquely identified by a name and version
Continuous Quality Control (CQC)
CQC defines a branch structure for a special cqc branch that aggregates validation results, and a validation_packages.yml file that lists the name of validation packages that an ARC should be continuously validated against.
Now here is the actual question: On what level should these features and concepts be introduced? My hunch is that we should try to get as much into ARCtrl as possible, but i really do not want to make all the validation libs i am working on fable compatible (i also think for some it might just not be possible). We have the contracts mechanism though, so i think we should discuss a little here.
My ideas:

Incorporating the cqc branch should be easy. Just create an orphan branch with git checkout cqc --orphan. How is git functionality modelled at the moment?
the basics of just creating the validation_packages.yml file are also easy, however I do not know if there is a fable compatible yaml read/write library. Since this is IO, we could work with contracts here?
Getting the names of installable validation packages is non-trivial. They are queried via web API calls to a package registry. We have a F# implementation for this, however that is not transpilable. Also use contracts for this?

It might be that these things do really fit into ARCtrl though, so if that is the case ill be happy to move the issue to arccommander and/or arcitect

So we talked about this and come up with the following plan:

branch creation is a contract
yml file creation is a contract
validation packages types should be a part of our core domain model
API client for https://github.com/nfdi4plants/arc-validate-package-registry should be part of ARCtrl

So @Freymaurer @HLWeil could you please add some guidance on

where to add the core validation package types?
where and how to add the API client?

publishing these type models also would help me fixing https://github.com/nfdi4plants/arc-validate-package-registry/issues/7

Okay!

branch creation is a contract

Currently we have a file with git contracts on the top level of the library. In my opinion this should be lower level, so that multiple packages are able to reuse the git logic.
Maybe put all of these contracts into the Contract project.

yml file creation is a contract

yml can be written as string so it should be very easy to model this at any level you prefer.

validation packages types should be a part of our core domain model

where to add the core validation package types?

Yes, please add an additional project at /src/<your_project_name>!

API client for https://github.com/nfdi4plants/arc-validate-package-registry should be part of ARCtrl

where and how to add the API client?

In my opinion this is the most difficult step. At the moment we have GET logic for js and dotnet here. But i am not very happy with it. As it is only GET and not python comaptible AND contradicts contract logic.
This should be further discussed before implementation.

Feel free to contact me at any time if you need more guidance on how to work with ARCtrl

@kMutagene is this done?

No, but i think we decided on not implementing any API requests in arctrl so i think it can be closed


orderByValue() on NormalizedCollection does not sort dataset

When applying the orderByValue() query on the main ref of a NormalizedCollection, the data does not sort by value.  http://plnkr.co/edit/Zy1vax?p=preview

I guess the constructor takes Firebase References as parameters, not Firebase Queries. Although it does not throw an error, it simply ignores the Query part. I suppose it uses .toString() to get only urls of the references.
You can use orderByValue() in the normalized reference, but I think you will have the records ordered by the client side instead of by the Firebase server.
This would also probably apply to the issue [#61] (https://github.com/firebase/firebase-util/issues/61)

As per @AaronJPA response to #61,  querying the normalised reference works fine.

Let's leave this opened. May be a bug here.

I also encountered this bug. I'm using firebase-util with reactfire. From what I've seen, rectfire onChildAdded doesn't receive 'prevChildKey' parameter.


signaling issue for anastasiia-zolochevska/signaling-server (redirect)

when i click Deploy to Azure button i just see like below.but nothing happen

please help me solve the issue.

Hello @parandham03, it looks like there is currently an issue with the azure deploy site - as it isn't a microsoft service that has an SLA (it's provided as-is as a developer tool) there isn't much we can do. ☹

I would advise following the instructions here to deploy using a different method - In my experience, forking the repository and choosing to deploy from Github has worked well. 🚀
Hope that helps! I'm going to close this issue here, and advise you to please open issues against https://github.com/anastasiia-zolochevska/signaling-server/issues in the future, as that is the project that we'd want to track these sorts of issues on. Thanks! 👍


Group and apply rewrite rules according to their priority

Fixes https://github.com/kframework/kore/issues/900

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock

Credit goes to @virgil-serbanuta for the idea of doing it in the strategy building phase.


Formulate user stories for prototype

Focus on 2 primary use cases:

integrating look-and-feel / content
providing search-driven access to PDS data, content, tools, and services

July schedule status: Delayed due to increase in number of user interviews. No impact on future tasks as this can be completed in parallel with others on schedule.

August Schedule Status: Delayed due to resources diverted early to #7. This delivery is a prototype so schedule re-organization is expected, and not linked to system I&T.

September Schedule Status: Deferred from Build 11.0 Build Schedule. No impact on build since this is part of a larger multi-build effort

@jordanpadams , @edwardbarraza needs clarification on item 1.
@edwardbarraza can start with item 2.

@edwardbarraza i just threw something down for a description. feel free to modify the ticket description to make more sense to someone testing or trying to get a better understanding of what this pertains to. This applies to other future tickets as well since "Formulate user stories for prototype" means practically nothing to someone not on our tag-ups 😄

October Schedule Status ⏳:  On track

Completed demo at MC Virtual F2F. See demo here


Clean RUBYLIB as well in with_clean_env

Fixes #3982.

@indirect The original branch 3982-clean-rubylib was based on 1-10-stable. However, since the issue is also present on master also, this PR targets master (this required fixing some rubocop offenses too).
I didn't yet squash for clarity (I'll squash after the review).

Looks good! 👍🏻

Squashed.

@homu r+

:pushpin: Commit 42b5072 has been approved by indirect

:hourglass: Testing commit 42b5072 with merge a136afe...

:sunny: Test successful - status

Please take a look @ bundler/issues#4165, looks like this commit has broken rails's spring.


re-added inmanta.module.INSTALL_OPTS and added documentation (#2840)

Pull request opened by the merge tool on behalf of #2840

Merged into branches iso4 in b1bd2f96820297c2a98ac333f9885526825c1309


Add comment to PR if a changeset error is resolved

Closing this issue since we are not implementing this feature.


websockets infrastructure

Requirements
Initial implementation of a websocket handler, manager and server.
Description of the Change
This PR allows for a websocket handler to be created within the application config block.  A user can also define their own Socket and Channel functionality (authentication, message handling, etc).  Example code below.
Why Should This Be In Core?

Allows for websocket communication to be easily added to any application.

Features still TODO

Heartbeat - server: to kill dead connections, client: to ensure connection to server is still active
PubSub implementation through redis / postgres - for application scalability
Allow for different decoding methods - allow the user to use something other that JSON if they want.
rebroadcast join and leave automatically (see how other frameworks do this)?
Message buffering?
Long polling fallback?

Example Code

Create a socket and channel:

struct UserSocket < Amber::WebSockets::ClientSocket
  channel "chat_room:*", ChatChannel

  def on_connect
    # This runs when the socket connects.  Authentication should happen here.
    # If `on_connect` returns false, the socket will be closed.
    true
  end
end

class ChatChannel < Amber::WebSockets::Channel
  def handle_joined
    # Called when a user joins a channel
  end

  def handle_message(msg)
    # Called when a message is sent to the channel
    # use rebroadcast! to broadcast to the message to all users subscribed to the channel
    rebroadcast!(msg)
  end
end

In the application config block, define the handler endpoint

MyApp.config do
  ...
  socket_endpoint "/chat", UserSocket
 ...
end

Javascript library usage:

let socket = new Amber.Socket("/chat");
socket.connect();

let channel = socket.channel('chat_room:123')
channel.join()

channel.on('msg:new', (msg) => { console.log('new message : ', msg) } )

channel.push('msg:new', {message: 'hey guys'})

It would be good to create issues for any TODO lefts for this to be feature complete. What do you think?

@eliasjpr Yes definitely, I will do that.


Cant send file or image (%!d(float64=404))

Sending suddenly stopped work and return such error
upload responsed with %!d(float64=404)

problem on multiple whatsapp versions

#317


Play sound when main menu or WiFi is ready

Added

Play sound when completing startup and reaching main menu.
Play sound when WiFi/MQTT is ready to receive DigiROMs from the server.

Is this sound appropriate?

Not sure the main menu one is needed, maybe an option later for extra sounds?  It's nice in use, the WiFi/MQTT sound is quite useful as that can sometimes take a little bit for some connections.
I'm okay with this being pulled into the repo


Podfile on iOS

There's no podfile in my project.
How can I follow first step that add a line to podfile?

Hi :) In react-native you either link the native iOS modules via react-native link or manage your native dependencies via Cocoapods. Please do not mix up those 2 options. To start with cocoapods simply run pod init in your ios directory.


eth JSON-RPC debug method

Can be closed in favor of https://github.com/zeta-chain/node/issues/1219 right?


[Feature Request] Mysql and MongoDB Side by Side Active in same model

can anyone guide or suggest how i can used two database concurrently like mysql  and mongodb both.

I do not understand why you need to do that. Can you explain for us?

I want to twice entry one in mongodb cloud, and by default store in mysql cpanel. i handle the situation by initiate two db connection. so, no need now.


Use inline namespaces

We need to use inline namespaces to maintain ABI compatibility. On a project this size that is probably important.
We don't care much about ABI right now, since API keeps changing, but we should already start using the inline namespace in preparation.
I wonder what to name that inline namespace. inline namespace tt_v1?

Can you provide some resources for why ABI compatibility is important because all that google and Wikipedia came up with seemed rather dated.

ABI compatibility is less problematic on windows, since most applications are bundled together with their libraries.
For Linux this is more important, since libraries are installed on the system, Linux is currently the most important reason for C++ to keep having to have ABI compatibility.
What is happening is that during compilation headers are included, which contains inline code, which much match the code that was compiled when the library was build.
Imagine a class where in version 2 the order of member-variables has changes, if the application was build with version 1 the inline methods still point to the members in the original order.
The inline-namespace can solve this partly. There would be a v1 and a v2 namespace, both would contain that same class, the second one with the order reversed. One-and-only-one of those two namespaces would be inlined. When a namespace is inlined all the names in that namespace becomes visible in the containing namespace, but when the name is found it still refers to the original namespace.
When happens when you compile against such a library is that although you don't explicitly program against a version, the executable that is generated does reference the version explicitly, so when the library is changed it still points to the original class.
But not only ABI, but also API stability exists, if you want to compile an old application against the library, you can include the headers with an #define of a specific version, or by directly using the namespace by name.

Extra question, what happens with Doxygen with inline namespaces.

I wonder what to name that inline namespace

Maybe just the major version prefixed with v. It changes only, when you make incompatible API changes.
namespace library
{
    namespace v1
    {
        class car {
        };
    }

    inline namespace v2
    {
        class car {
            // new member(s)
        };
    }
}

what happens with Doxygen with inline namespaces.

Doxygen has partial support for inline namespace. The intermediate XML contains data for this item, since Release_1_8_19.
But i'm not sure, if output formats are ready, yet. The upstream issue is still open, ref.: https://github.com/doxygen/doxygen/issues/6741


before/after TotalMonthlyEstimatedCost  ?

would it be possible to log both a before and after TotalMonthlyEstimatedCost?  From what I can tell, the TotalMonthlyEstimatedCost that's logged currently appears to be "current state"
could we also assemble a projected state (since we can't query it post update since updates dont apply immediately) and estimate a cost off of that?
Seems like it would be useful to see the projected cost before/after any particular update.

for that matter, a before/after TotalProvisionedThroughput would be a nice touch too..

This is more like analytics which is much easier to integrate using a proper oss analytics package using all the raw data which this package outputs.  The projected cost is extremely rough as it literally only knows about that particular point in time.  For that matter aws charts based upon the actual dynamodb stats would be far better.


Draft benchmark

Resolves #13

btw, where do we publish it?

Good job!
You can ignore my comments, it's mostly about Rust, we don't really care about it here.

I applied almost all of them !
Thanks.


Should if statement add result == UNCANCELLABLE of DefaultPromise#getNow()

Should if statement change to below:

and if result==null,  setUncancellable() return true?

@zhaojigang good catch.. See https://github.com/netty/netty/pull/8154.


[CH] Add GlutenJsonExpressionsSuite

What changes were proposed in this pull request?
How was this patch tested?
PASS CI

LGTM

looks like we missed this when enabling Spark 33/34/35
CC @yma11

looks like we missed this when enabling Spark 33/34/35 CC @yma11

I think so. Let me pull a update for this.


Build error handler and notification system

I'd like to start finding and standardizing potential errors that a user could run into (i.e. "incorrect file type", or "cannot union these two geometries"). I think logging errors is fine right away, but I'd like to connect it to a notification system within the application that is used for alerts/successes/etc.
There are some libraries that do this, such as notify.js. Honestly, I think I might roll our own because this seems like a fantastic UI exercise.

I believe that #33 solved this (used to demonstrate errors in #97).


[C#] Observer Design Pattern revamp

Continuing with the revamps in #18, this is the implementation of Observer Design Pattern.
Changes implemented

Created a dedicated Observer project, with an example in Program.cs to show demo code.
Sealed all concrete classes.
Use File-Scoped namespaces, resulting in one less indentation level.
Boosted looping on NotifySubscribers by using CollectionsMarshal.AsSpan(). [Reference]
Avoided dictionary double checkup by replacing ContainsKey() with TryGetValue() as per CA1854
XML Documentation on main classes and methods

@mahyoussef Looking forward to hearing your feedback.

@mahyoussef  Just fixed a merge conflict with main. Kindly review and proceed with the merge.


Adding some missing docs for choreo connect

Purpose
This Pr will remove some incompatible documentations from Choreo connect and add/update docs for the following features. (Some were already added with https://github.com/wso2/docs-apim/pull/3506 )

Go consumer to retrieve events  and failover handle for amqp connection related information (Concepts - Event-hub),
3rd party KeyManagers (Concepts - Choreo Connect with API Manager as Control Plane),
Prototype Endpoint (Endpoints- Endpoint Types),
Internal Key support (Security - Authentication),
Overriding Endpoints for deployed APIs (Endpoint -Overriding Endpoints for deployed APIs)
Backend Security Basic auth (Endpoints - Defining a Backend Security Scheme)
Remove auth header based on config (Security configurations in config.toml)
Custom auth header (Security - Use a Custom Authorization Header)

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

@HiranyaKavishani , please address the review comments.


Take a screenshot when test fails and send it via my "slack" reporter

Hey
Im using a slack reporter that sends text message when one or more of my tests fail. my question is there a way to take a screenshot of the page only when the test failed?

hi @galtubi91
its have some difficulty about this case, see https://github.com/smooth-code/jest-puppeteer/issues/131

Follow up in #131


Update dis module from Python 3.12

This PR add functions and method from Python 3.12 for dis module and also updates test_dis.py.
test_dis.py mostly gets skipped due to lack of _testinternalcapi (test_dis skipped -- _testinternalcapi required) but that would be separate work (to make _testinternalcapi functional).
For this change, I have removed one OPCODE RETURN_CONST and commented out some test cases with TODO: RustPython  as they required features which are not yet available.
This also addresses https://github.com/RustPython/RustPython/issues/3846
EDIT 1:
Failures: extra_tests/snippets/stdlib_dis.py
import dis

dis.dis(compile("5 + x + 5 or 2", "", "eval"))

fails with error
File "/home/ankit/RustPython/pylib/Lib/dis.py", line 113, in dis
type(x).name)
TypeError: don't know how to disassemble code objects
on cpython, compile("5 + x + 5 or 2", "", "eval") object has co_code attribute which leads to it to branch to co_code handler.
    elif hasattr(x, 'co_code'): # Code object
        _disassemble_recursive(x, file=file, depth=depth, show_caches=show_caches, adaptive=adaptive)

On RustPython, co_code attribute is not there and hence none of if..else branch in dis function gets matched leading to TypeError fail getting triggered (as mentioned above).
Looking into how it used to work before and how can this be fixed.
EDIT 2:
probably need co_code

Closing this, will revisit again with better approach.


Updated helpline numbers

Related Issue
Complete the helpline numbers
Fixes: #169
Description
Appended the table of file helpline_numbers.html
Checklist:
My code follows the style guidelines of this project.
I have performed a self-review of my own code.
My changes generate no new warnings.

@sakshikhachane Please review it.
Thanks

@sakshikhachane Done ma'am.

@swapnilsparsh bt sir , links r blue in color so it seems they r active . U want black color of links?

@swapnilsparsh bt sir , links r blue in color so it seems they r active .

Remove the website links also.

@swapnilsparsh @sakshikhachane Please check it now .
This has completed.
Thanks.

Resolve the conflicts.

sry sir bt i can't figure out the conflict in this .
Can u tell me please?

@mansijain980 Any updates?

Actually Maam my xams r going on ...sorry for the delay but day after i will change it . Or if in case i will find sometime today or tomorrow i will do it then .
Thanks.

Ma'am can u please tell me why u closed this?
On Sat, Apr 10, 2021, 09:27 Sakshi Khachane @.***>
wrote:

Closed #192 https://github.com/swapnilsparsh/Rescue/pull/192.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/swapnilsparsh/Rescue/pull/192#event-4577057221, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APCMC7Y3WFOQJUV2SERJH23TH7EDNANCNFSM4ZTJNH4Q
.

@sakshikhachane ma'am can u please tell me why u closed this pull request ? as all work had been done .Only some minor changes r left.


[BUG] Test case test_node_metrics fails in Talos due to missing cpu/memory usage metrics

Describe the bug
Setting up Talos v1.8.3 by the refined terraform script and installing Longhorn. Test case test_node_metrics fails because the following metrics are not found:
longhorn_node_cpu_usage_millicpu
longhorn_node_memory_usage_bytes

https://ci.longhorn.io/job/public/job/master/job/talos/job/amd64/job/longhorn-tests-talos-amd64/28/testReport/junit/
To Reproduce
Run test case test_node_metrics.
Expected behavior

Support bundle for troubleshooting

Environment

Longhorn version: master-head
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: v1.31.1

Number of control plane nodes in the cluster: 1
Number of worker nodes in the cluster: 3

Node config

OS type and version: talos v1.8.3
Kernel version:
CPU per node:
Memory per node:
Disk type (e.g. SSD/NVMe/HDD):
Network bandwidth between the nodes (Gbps):

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context

Workaround and Mitigation

2024-12-02T08:17:07.596184133Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*InstanceManagerCollector).collectActualUsage" file="instance_manager_collector.go:147" collector=instance_manager error="the server could not find the requested resource (get pods.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.596844557Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*NodeCollector).collectNodeActualCPUMemoryUsage" file="node_collector.go:249" collector=node error="the server could not find the requested resource (get nodes.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.596858092Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*ManagerCollector).Collect" file="manager_collector.go:85" collector=manager error="the server could not find the requested resource (get pods.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.633247524Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*ManagerCollector).Collect" file="manager_collector.go:85" collector=manager error="the server could not find the requested resource (get pods.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.633366345Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*NodeCollector).collectNodeActualCPUMemoryUsage" file="node_collector.go:249" collector=node error="the server could not find the requested resource (get nodes.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.633753061Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*InstanceManagerCollector).collectActualUsage" file="instance_manager_collector.go:147" collector=instance_manager error="the server could not find the requested resource (get pods.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.666223431Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*NodeCollector).collectNodeActualCPUMemoryUsage" file="node_collector.go:249" collector=node error="the server could not find the requested resource (get nodes.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.666411118Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*ManagerCollector).Collect" file="manager_collector.go:85" collector=manager error="the server could not find the requested resource (get pods.metrics.k8s.io)" node=ip-10-0-1-219
2024-12-02T08:17:07.666785480Z time="2024-12-02T08:17:07Z" level=warning msg="Error during scrape" func="metrics_collector.(*InstanceManagerCollector).collectActualUsage" file="instance_manager_collector.go:147" collector=instance_manager error="the server could not find the requested resource (get pods.metrics.k8s.io)" node=ip-10-0-1-219

Cause: Missing metric server in the Talos Linux cluster.
Fix: Install the metric server. See: https://www.talos.dev/v1.8/kubernetes-guides/configuration/deploy-metrics-server/
Closing this issue as it is not a bug.
cc @derekbit @yangchiu

Additional issue ref: https://github.com/longhorn/longhorn/issues/9764


Fix subdirectory argument scan bug

Context
Fix directory scan bug, overwritting the currentDir introduced a bug and broke the values file history view in the changelog
Checklist

[x] Change meets or does not compromise the Baseline Security Requirements

Enriched changelog with PR description. IT assumed the gh CLI tool is installed and configured, which I think it is by default in GH runners

/emergency-merge First PR, setting up CODEOWNERS.

/wise-bot emergency-merge First PR to repo, setting up CODEOWNERS

/wise-bot emergency-merge First PR to repo, setting up CODEOWNERS

/wise-bot approve


New Linter: SublimeLinter-contrib-rstylelint

I've made a linter that lints RSL language (.mac) https://github.com/mom1/SublimeRStyle
Here's the repository:
https://github.com/mom1/SublimeLinter-contrib-rstylelint

Please make a release and it will be available in SublimeText.
Thanks!

I did

https://packagecontrol.io/packages/SublimeLinter-contrib-rstylelint


pollTunnelStatus [as _onTimeout]

When I run 'npm run dev -- --reset', I encounter this error：
To investigate the issue, examine this stack trace:                                                     │
│    at pollTunnelStatus [as _onTimeout] (@shopify/app/src/cli/services/dev/urls.ts:106)                   │
│      if (result.status === 'error') return reject(new BugError(result.message))                          │
│    at listOnTimeout (node:internal/timers:569)                                                           │
│    at processTimers (node:internal/timers:512)

Hi, this seems to be a shopify cli issue as reported here #2170
Here are somethings you can try
Method 1 (The best method)
I would recommend you use ngrok to start a tunnel and then run the app through that, This will also allow you to easily test webhooks on your device.
Here's how to start the server using ngrok

download ngrok  and add it to your system environment variables
run npm run dev -- --tunnel ngrok

Hi, this seems to be a shopify cli issue as reported here #2170
Here are somethings you can try
Method 1 (The best method) I would recommend you use ngrok to start a tunnel and then run the app through that, This will also allow you to easily test webhooks on your device.
Here's how to start the server using ngrok

download ngrok  and add it to your system environment variables
run npm run dev -- --tunnel ngrok

It works, thank you


vscode not able to show poetry environments when trying to select interpreters

Type: Bug
I am using poetry  to manage my python application. I am letting poetry create the virtualenv in the default location. However when I am trying to select the desired python
interpreter from within vscode, its not listing any of my poetry envs. I can see my global python installations and pyenv installations as well but not poetry. I event tried navigating manually
but that did not help.
Steps to reproduce

pyenv local (to set python version to the one mentioned in .python-version file)
pyenv install (to install the py version)
poetry env use ~/.pyenv/versions/3.10.8/bin/python
poetry install

Expected behaviour

Within vscode the py interpreter from the virtual env created by python should be available to select.
Once selected all the imports should resolve.

Actual behaviour

The desired py env is not listed in the select interpretor dropdown list.

Output of poetry env info

Virtualenv
Python:         3.10.8
Implementation: CPython
Path:           /Users/dmondal/Library/Caches/pypoetry/virtualenvs/hot-escalation-K4Tshkpo-py3.10
Valid:          True

System
Platform: darwin
OS:       posix
Python:   /Users/dmondal/.pyenv/versions/3.10.8

Out of poetry config --list
cache-dir = "/Users/dmondal/Library/Caches/pypoetry"
experimental.new-installer = true
installer.parallel = true
repositories.artifactory.url = "https://packages.atlassian.com/pypi/pypi/simple/"
virtualenvs.create = true
virtualenvs.in-project = null
virtualenvs.path = "{cache-dir}/virtualenvs"  # /Users/dmondal/Library/Caches/pypoetry/virtualenvs

Environment Data

VS Code version: 1.73.1
Extension version (available under the Extensions sidebar): v2022.18.2
OS and version: macOS 10.15.7
Python version (& distribution if applicable, e.g. Anaconda): 3.10.8 (pyenv)
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): Poetry
Relevant/affected Python packages and their versions: Poetry 1.1.13

I have tried keeping the venv inside project root using virtualenvs.in-project = true, that works fine. Issue seems to be with envs in default poetry location.
I have tried the solution presented here: https://github.com/microsoft/vscode-python/issues/16232
I tried adding the setting "python.experiments.optInto": ["All"]  but that did not help.
Extension version: 2022.18.2
VS Code version: Code 1.73.1<PHONE_NUMBER>f055b99068d3688932416f2346dd3b, 2022-11-09T02:08:38.961Z)
OS version: Darwin x64 19.6.0
Modes:
Sandboxed: No

Fix should be out in the pre-release version of the extension, use the following to try it out:

Closing as dup of https://github.com/microsoft/vscode-python/issues/20073#issuecomment-1307183368.

No but it should be in the pre-release.

Feel free to open a new issue with details in that case. New release is expected to go out Dec 7th.


Add links to Jupyter notebooks and science notebooks

As advertised. Also added a bullet to the conference tips.

@hjwp Press the green button ;-)

thanks Carol!


Update RtcPeerConnection with connection_state

I wanted access to the onconnectionstatechange event, so I tried to follow the readme in the web-sys folder and update the idl for RtcPeerConnection and related definitions as needed.
However, everything is building now, but when trying it out in practice, I don't actually see the event firing... I'm, not really sure why though, (if the browser is just not sending the event or it's some other reason).
Still, perhaps this might be of use if someone needs more up-to-date definitions for RtcPeerConnection for whatever reason...

I would like this feature as well.

@johanhelsing apologies, I made a mistake closing this. #3380 doesn't replace this PR, it only covered a small portion of the API added here.
If you don't mind rebasing this I will review it ASAP.

Hey, sorry, I don't have the capacity to continue working on this. Feel free to adopt!

No problem!


FAIL: TestClientServer_infoLocals

Running tests @ https://github.com/derekparker/delve/commit/04c4b019f7557344910ade621f68e2695eabe2e5 on Windows 10 with Go 1.8.3 / Go 1.9 beta 1 gives this:
--- FAIL: TestClientServer_infoLocals (0.19s)
        support.go:129: enabling recording for github.com/derekparker/delve/service/test_test.TestClientServer_infoLocals
        integration2_test.go:478: Expected 3 locals, got 2 []api.Variable{api.Variable{Name:"j", Addr:0xc04206bea0, OnlyAddr:false, Type:"int", RealType:"int", Kind:0x2, Value:"1", Len:0, Cap:0, Children:[]api.Variable{}, Unreadable:""}, api.Variable{Name:"f", Add
r:0xc04206beb0, OnlyAddr:false, Type:"int", RealType:"int", Kind:0x2, Value:"2", Len:0, Cap:0, Children:[]api.Variable{}, Unreadable:""}}

Well seems my environment is / was a bit strange and it ran with Go 1.9. Sorry for noise.


Revert "Fix: use gvk instead of group resource in resource indentifier"

Reverts kubevela/pkg#63
We shouldn't expose the version changes to users just for the programming convenient

ping @FogDong , please create a revert one and keep the considerations


[20.09] openvswitch-lts: 2.5.9 -> 2.5.12, addressing CVE-2020-27827 & CVE-2020-35498

Motivation for this change
https://nvd.nist.gov/vuln/detail/CVE-2020-27827
https://nvd.nist.gov/vuln/detail/CVE-2020-35498
Backport of #118485
Things done

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

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

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

@ofborg eval


Monitor clusteroperator status

In addition to monitoring nodes and pod health, clusteroperator status is another key indicator of cluster health.
Suggestion for cerberus enhancement:  add a config option to monitor clusteroperator status and watch for degraded operators.    See:  oc get clusteroperators -o yaml

/cc: @chaitanyaenr @yashashreesuresh  : Assigning this to @paigerube14 .   Please comment if you have suggestions/preferences on the approach.

Some thoughts on getting started. Please feel free to comment.
start_cerberus.py: add an optional "watch_operators" key with possible values of True or False to the yaml file
if the watch_operators value is true, use the invoke function to "oc get clusteroperators -o yaml"
I couldn't find any kubernetes client functions for the cluster operators area so I think the invoke is the only way to do this. Please point me to it these functions, if they do exist.
Then loop through the output returned from the invoke function to look at the "degraded" column
Could add the loop in to the client.py file to log which operators are degraded.
Do we want to be able to know which iteration the certain operator started to degrade, similar to the monitor_component function?

@paigerube14 The approach sounds good to me.
https://github.com/kubernetes-client/python/blob/master/kubernetes/docs/AppsV1Api.md doesn't seem to have a way to get the list of cluster operators I guess, so using the invoke function to run the command is the way to go like you mentioned.
We can display the failure along with name of the cluster operator in degraded state similar to how the components in namespaces are being monitored. Any logging.info statement gets automatically included in the report file generated in the run directory. This way, we can take a look at what cluster operator was failing at what time/iteration and whether it recovered in the next iterations in addition to perf/scale and chaos test automation/tooling using the no-go signal ( False ) to act on the failure.

Fixed by https://github.com/openshift-scale/cerberus/pull/46. Thanks @paigerube14.


Support @computedFrom on functions

Currently functions do not support computedFrom, but accessors do. In the case where a VM is exposing an internally calculated value, there is currently no proper way to ask Aurelia to include the method in a dirty check, or to indicate which dependencies need monitoring for change tracking.
Accessors are methods, so when we have a method such as FullName we can decorate it because FullName takes no parameters and returns data dependent on two internal variables that the view never sees, so we are hiding the implementation.
However, if we decide to have arrays of firstNames and lastNames, fullName requires a parameter, typically $index from a repeater. There is now no way of using a get accessor, so we have to 'bodge' the system and include firstName[$index] and lastName[$index] in a method call to simulate 'computedFrom' and ensure the method is called if either of those instances is updated, but this also exposes the implementation to the view.
My suggestions is that we support @computedFrom(...vars) on methods, as this retains knowledge of the dependencies in the VM.
What happens when a method is method is non-deterministic? We don't have a means of specifying dirty-checking, so perhaps we need to support a parameterless @computedFrom(), or allow @binding on a method.

linking to #147 and closing.  Will update #147's title to make it easier to find


Portable BCUninstaller 5.4 making error on start - "Failed to start BCUninstaller - %1 не является приложением Win32"

Windows 7 x64. Portable BCUninstaller 5.4 (and 5.2) making error on start - "Failed to start BCUninstaller - %1 не является приложением Win32"

Run windows update and install all system and .NET updates offered.

The second method works. Thanks !

checked version 5.5 Portable. Nothing changed, still have error with
main installer.
------ Исходное сообщение ------
От: "Marcin Szeniak" @.>
Кому: "Klocman/Bulk-Crap-Uninstaller"
@.>
Копия: "ded-sergei" @.>; "Author"
@.>
Отправлено: 03-Mar-23 20:08:51
Тема: Re: [Klocman/Bulk-Crap-Uninstaller] Portable BCUninstaller 5.4
making error on start - "Failed to start BCUninstaller - %1 не является
приложением Win32" (Issue #450)

Can you test if this gets fixed in v5.5? I'll release it within a day
or so. If the bug still happens, please reopen this issue and send me
the full path to the launcher.
—
Reply to this email directly, view it on GitHub
https://github.com/Klocman/Bulk-Crap-Uninstaller/issues/450#issuecomment-1453907314,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6BKBJLANZPCJAPFW6UE3RLW2IXTHANCNFSM6AAAAAAVDIOYEU.
You are receiving this because you authored the thread.Message ID:
@.***>

installed the not portable version in D:\Program Files\BCUninstaller and faced the same error. uninstalled and reinstalled in D:\BCUninstaller and the error was gone

@ded-sergei @LamaLello Can you check if this build fixes the issue?
launcher debug.zip

Debug version works OK.
------ Исходное сообщение ------
От "Marcin Szeniak" @.>
Кому "Klocman/Bulk-Crap-Uninstaller"
@.>
Копия "ded-sergei" @.>; "Mention"
@.>
Дата 27.09.2023 23:04:33
Тема Re: [Klocman/Bulk-Crap-Uninstaller] Portable BCUninstaller 5.4
making error on start - "Failed to start BCUninstaller - %1 не является
приложением Win32" (Issue #450)

@ded-sergei @.***
https://github.com/LamaLello Can you check if this build fixes the
issue?
launcher debug.zip
https://github.com/Klocman/Bulk-Crap-Uninstaller/files/12742882/launcher.debug.zip
—
Reply to this email directly, view it on GitHub
https://github.com/Klocman/Bulk-Crap-Uninstaller/issues/450#issuecomment-1738002983,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6BKBJOIWLWQD5H7CLTASODX4SBFDANCNFSM6AAAAAAVDIOYEU.
You are receiving this because you were mentioned.Message ID:
@.***>


Feature request: Add "return null for missing leaf" option in SelectTokens

Feature request
Add an option to allow the return of null for missing leafs when using SelectTokens. Maybe an option in https://github.com/JamesNK/Newtonsoft.Json/blob/master/Src/Newtonsoft.Json/Linq/JsonSelectSettings.cs
Example in another lib: https://github.com/json-path/JsonPath#options > DEFAULT_PATH_LEAF_TO_NULL
Fiddle
Related SO question
Example of current output using Newtonsoft
var json = @"{
  ""Values"": [
    {
      ""Name"": ""Bob"",
      ""Age"": ""25"",
      ""Address"": ""1234 Easy St.""
    },
    {
      ""Age"": ""28"",
      ""Address"": ""1600 Pennsylvania Ave.""
    },
    {
      ""Name"": ""Joe"",
      ""Age"": ""31"",
      ""Address"": ""653 28th St NW""
    }
  ]
}";

var data = JToken.Parse(json);

var query = data.SelectTokens("Values[*].Name");

Console.WriteLine(JsonConvert.SerializeObject(query, Formatting.Indented));

Outputs
[
  "Bob",
  "Joe"
]

With the new feature, the method would return
[
  "Bob",
  null,
  "Joe"
]

I agree. We are using json path syntax in Azure Data Factory (Copy Activity > Mapping) to select nested object values and flatten them into columns, and have the same root problem.

Have the same problem. This should definitely be an option.

@smddzcy +++++++++

Running into the same problem which flattening JSON Array. Without NULLs, the ordering of the fields is lost.


[BUG] img height error

Describe the bug 描述你遇到的错误
img height error
https://hugodoit.pages.dev/basic-markdown-syntax/#images
The Stormtroopocat image height error.
Some markdown images may not be fully displayed
Expected behavior 期待的行为
No response
Screenshots 屏幕截图

Build Environment 构建环境
No response
Preview Environment 预览环境
No response
Additional Information 补充信息
No response

Yep, the good old sizes=auto issue. Should have never done that.


Add Github topics to the repository

Hi, can you add the following topics to this repository?

https://github.com/topics/fints
https://github.com/topics/hbci
https://github.com/topics/php
https://github.com/topics/banking

Thanks!

:heavy_check_mark:

👍

Thanks 👍


Add expectContentsFile parameter to GlyphSet

For use when reading existing UFOs, to comply with the specification stating that a contents.plist file must exist in a glyph set.
Avoid unconditionally checking for the file because UFO(Reader|Writer).getGlyphSet and therefore indirectly GlyphSet are used to create new fresh glyph sets.
Closes https://github.com/fonttools/fonttools/issues/2111.

The test suites for defcon, fontTools and ufoLib2 pass with this change :)


GPIO accessible via gpio group

can you put the arduino interface DI, DO, user leds , ..  in a gpio group.
i managed to put the led via the udev rules into the gpio group but failed with the DOs of the arduino shield.
cat >/etc/udev/rules.d/98-comLed.rules  << EOF1
SUBSYSTEM=="gpio*", PROGRAM="/bin/sh -c 'chown -R root:gpio /sys/class/leds && chmod -R 770 /sys/class/leds && chown -R root:gpio /sys/devices/platform/leds/leds && chmod -R 770 /sys/devices/platform/leds/leds'"
EOF1
can you set this per default?

What is the requirement behind this request? Is it about access control management?
The example image is a reference for creating own customization on top. Adding own users, managing their permissions would be one of the possible customizations. If a pattern is generic enough, we are happy to consider including it also in the example image. The best approach to propose that is by submitting a pull request with the desired changes. If that is too tricky, we need at least some more details on the desired use case.

if you using the example image. you are forced to use the root user for node-red.
otherwise you cannot use the gpios.
So in my view it is generic to put the leds, gpios and user buttons into a special "gpio" group.
Therefore someone can put own customization on top, adding new user, and give them easily access to the hardware via the gpio group.

Ok, so the actual requirement is "run Node Red as non-root user". Permitting non-root access to GPIOs is a building block for that, but it is not reasonable to only do that without a use case, thus also a test case.

I just learned that there are limits so far in de-privileging anything that needs mraa to access IOs - and that included node-red. The reason is the way the pin mux controller is accessed from mraa: as directly mapped MMIO via /dev/mem, and that is naturally a highly privileged access. Currently, we are missing an alternative. And nothing would be won by simply adding /dev/mem to such a GPIO group as well. Any user in that group would still have effective root rights.

I just learned that there are limits so far in de-privileging anything that needs mraa to access IOs - and that included node-red. The reason is the way the pin mux controller is accessed from mraa: as directly mapped MMIO via /dev/mem, and that is naturally a highly privileged access. Currently, we are missing an alternative. And nothing would be won by simply adding /dev/mem to such a GPIO group as well. Any user in that group would still have effective root rights.

ok, i understand. That makes sense.
Thank you for your researching.

ok, i understand. That makes sense.
Thank you for your researching.

We can reopen this topic when the dependency on /dev/mem is resolved. But we have no good idea for that yet, though it's clearly desirable.

We can reopen this topic when the dependency on /dev/mem is resolved. But we have no good idea for that yet, though it's clearly desirable.

@jan-kiszka what about creating a second chardev which provides access to the pin mux controller's memory only (i.e. /dev/gpiomem) and adding this one to a GPIO group?
It appears that the raspberry-pi guys do it like this as well:
https://github.com/eq-3/RaspberryMatic/blob/master/linux-4.1/drivers/char/broadcom/bcm2835-gpiomem.c

We are in fact looking into overcoming /dev/mem for pinmux'ing right now, for the purpose of enable unprivileged mraa usage.
@mwesenjak thanks for the pointer. However, it's a not a good example as it is an out-of-tree approach. We are particularly looking for a solution that permits upstreaming it, see also https://marc.info/?l=linux-gpio&m=166850640920120&w=1.

A root-free mraa concept is merged into master by now, testers welcome. Reopen if there is anything remaining in this context, or create a new issue.

@jan-kiszka I've been trying to get this to work but I'm not quite sure what permission a user/group needs to be able to access pins with for example mraa-gpio get 20. Any info of where to start with this would be really appreciated!

Documentation really would be great 😅. What I've tried so far is adding the following udev rules:
SUBSYSTEM=="gpio*", PROGRAM="/bin/sh -c '\
        chown -R root:gpiouser /sys/class/gpio && chmod -R 770 /sys/class/gpio;\
        chown -R root:gpiouser /sys/devices/virtual/gpio && chmod -R 770 /sys/devices/virtual/gpio;\
        chown -R root:gpiouser /sys$devpath && chmod -R 770 /sys$devpath;\
        chown -R root:gpiouser /sys/kernel/debug && chmod -R 770 /sys/kernel/debug;\
'"

KERNEL=="gpiochip*", GROUP="gpiouser", MODE="0660"

Access to /sys/kernel/debug allows my user that is part of the gpiouser group to run:
$ mraa-gpio get 1
Pin 1 = -1
$ mraa-gpio get 1
Pin 1 = 0

The problem here is that the value of the pin is only given the second time i run the command. This is somehow also related to NodeRed because i can only access the value of pins that I've run the command for (in this case D1).

Another obervation I've made is that i can only set the value of a pin via NodeRed after setting it before with mraa-gpio set 1 1.

It seems, that mraa is currently not completely root-free.
The problem here is that mraa is still relying on sysfs for controlling GPIOs.
The sysfs creates new GPIO files on-the-fly when exporting/initializing a GPIO pin. Unfortunately the new files are created with default root permissions.
I think you controlled the pin as root somewhere in between:
$ mraa-gpio get 1
Pin 1 = -1

# somewhere here you ran mraa as root ...

$ mraa-gpio get 1
Pin 1 = 0

.... am I right here?
MRAA currently supports general gpio-chardev access but I have to adapt the iot2050 layer to support it.
I think the step towards using the chardev interface is the way to go.

But the export only happens on first access, and there is no unexport, right? Couldn't we do those upfront, set the rights accordingly, and then only use the existing ones in mraa?

Unfortunately I was definitely wrong with the assumption that the sysfs files for GPIO are created with default root permissions, in case the above udev rules from @Int3rC3pt0r were used.
After adding a udev rules file that adjusts permissions I see:
ls -l /sys/class/gpio/gpio490/
total 0
-rwxrwx--- 1 root gpio 4096 Apr 12 13:11 active_low
lrwxrwxrwx 1 root root    0 Apr 12 13:11 device -> ../../../gpiochip1
-rwxrwx--- 1 root gpio 4096 Apr 12 13:11 direction
drwxrwx--- 2 root gpio    0 Apr 12 13:11 power
lrwxrwxrwx 1 root gpio    0 Apr 12 13:11 subsystem -> ../../../../../../../../../../class/gpio
-rwxrwx--- 1 root gpio 4096 Apr 12 13:11 uevent
-rwxrwx--- 1 root gpio 4096 Apr 12 13:11 value

The actual problem is that the udev commands are executed asynchronously to MRAA. The required user permissions are not completely setup while MRAA accesses those files, hence the errors.
Even though the chardev implementation may avoid these problems, the PWM implementation still relies on almost the same sysfs export mechanism as GPIO does.


Set Fixed Dates

Hello,
How can I set the timeline to show yesterday, today and tomorrow only?
Thanks

vis offers a way to use buttons http://visjs.org/examples/timeline/interaction/animateWindow.html but im not sure how to get them to interact with the react based timeline. any suggestions?

looks like its answered here https://github.com/Lighthouse-io/react-visjs-timeline/issues/33#issuecomment-330092443


missing k8s-bigip-ctrl deployment note  in  "important note" in clouddocs Add BIG-IP device to flannel VXLAN

// Title: Provide a short but descriptive summary of the issue.
Important note in http://clouddocs.f5.com/containers/v2/kubernetes/kctlr-use-bigip-k8s.html#use-bigip-k8s-flannel mis-lead customer to think BIGIP can communicate to k8s pod over overlay network before k8s-bigip-ctrl deployment
// Do not include requests for development or report issues with code or products here.
This repository contains documentation only.
Bugs
Describe the bug in detail:

What doc are you reporting an issue with (provide URL)?
http://clouddocs.f5.com/containers/v2/kubernetes/kctlr-use-bigip-k8s.html#use-bigip-k8s-flannel

What information is inaccurate?
in Important note:

"Once you’ve set up the BIG-IP tunnel and added the BIG-IP “dummy” Node, you should be able to send traffic through the BIG-IP system to and from endpoints within your Kubernetes Cluster"
this give customer impression that before customer deploy k8s-bigip-ctrl, BIGIP can communicate to k8s pod through overlay network, the correct notes should be something like:
"Once you’ve set up the BIG-IP tunnel, added the BIG-IP “dummy” Node and deployed the k8s-bigip-ctrl, you should be able to send traffic through the BIG-IP system to and from endpoints within your Kubernetes Cluster"

Is information missing?
yes
Is a code sample incorrect?
no

Hi @vincentmli! I've added a fix for this issue to PR #401, at this commit: 9a4969c8c4c857056d7e21c98a99d43b16b5241c


Fix no-jobs case in Test Report page

Fix no-jobs case in Test Report page and avoid the infinite loading icon

/lgtm


Add more options to chart

Codecov Report

Merging #113 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
+ Coverage   98.78%   98.78%   +<.01%
==========================================
  Files          19       19
  Lines         246      247       +1
==========================================
+ Hits          243      244       +1
  Misses          3        3

Impacted Files
Coverage Δ

src/yaml-format/parser.js
100% <100%> (ø)
:arrow_up:

src/components/chart/Chart.js
100% <100%> (ø)
: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 bec1467...151abd5. Read the comment docs.


Support installing LXD from snap

While not the most popular option a lot of people use it to get LXD up and running fast. Changes made:

Added variable to control how lxd is expected to be installed. Defaults to deb. If support for non-Debian family OS's is expected to be handled by the module, it might make sense to rename the deb option to something else. As per the project description, it doesn't seem to be any such plans.
Added logic to determine how to install the package (package resource or snap exec). Atm it is expected that snapd will be managed by the module, but I can easily switch it to be a configurable option that defaults to true.
Included tests for this logic

Closing this one as I messed up branches & included untested commits.
The PR was re-opened via https://github.com/ovh/lxd-puppet-module/pull/3


[#182537980] Parameterize tables registry schema test !!WIP¡¡

Ticket
Description
Removed the expected data model json files from the project and parameterized the e2e test to load the json data model from env variables instead
Test run
Pass in the table name & expected json data model as env values -

Ran against data-connect-trino-public in staging:

just add the table names & json data models as secrets manually, and update before_test to just export them
update before_test to write the table names & json data models as secrets

The first option seems the easiest, but if the secrets get accidentally deleted (or isn't transferred with our eventual move to publisher.dnastack.com from viral.ai) then the test stops running

I'd say we don't need secrets for this. You could put the literal expected data models in the before-test script. This will be a difference we have to deal with in the deployment packaging epic, but hopefully by then we will have developed tools and techniques to deal with this kind of thing.

I'd say we don't need secrets for this. You could put the literal expected data models in the before-test script. This will be a difference we have to deal with in the deployment packaging epic, but hopefully by then we will have developed tools and techniques to deal with this kind of thing.

@jfuerth Sounds good, I've added them in this commit


Transferred a partial file downloading google protobuf.

We are having this "partial file download" issue with https://github.com/google/protobuf/archive/v3.6.1.tar.gz
This is a repeatable issue and we can't manually download the file because it's on a CI build machine we don't have access to.  We've seen it 4-5 times in a row.  There is no proxy, and apparently it works for the x86-windows triplet.  The development environment is the Visual Studio 2017 AppVeyor image, note however we are cloning vcpkg from scratch into c:\vcpkg rather than using the installed one (AppVeyor states that it comes with vcpkg 0.0.113 pre-installed, I'm not sure how to make sense of that version number.
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:151 (message):

      Failed to download file.
      If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
      variables to "https://user:password@your-proxy-ip-address:port/".
      Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues
Call Stack (most recent call first):
  scripts/cmake/vcpkg_from_github.cmake:108 (vcpkg_download_distfile)
  ports/protobuf/portfile.cmake:3 (vcpkg_from_github)
  scripts/ports.cmake:71 (include)
Error: Building package protobuf:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: protobuf:x64-windows
  Vcpkg version: 0.0.113-nohash```

https://github.com/google/protobuf/releases

Care to explain what you mean by those 3 links?
https://github.com/google/protobuf/archive/v3.6.1.tar.gz is the file that vcpkg is trying to download.

Hi @jjpepper,
Could you try these two things and let us know if it helps?

First, pass --x-use-aria2 to the vcpkg install command. This will use the aria2 download client internally instead of cmake which may work better in your scenario.
Second, if the above doesn't work, could you try explicitly downloading the file via appveyor's instructions[1] as downloads\google-protobuf-v3.6.1.tar.gz, and then run the install command?

Knowing which of those works will really help figure out a long term solution!
[1] https://www.appveyor.com/docs/how-to/download-file/

Hi Robert,
This seems to have stopped happening.  It certainly seemed repeatable at
the time.  If it happens again we will make sure it's repeatable and then
try the two suggestions you have and let you know the results.  Perhaps it
was just an intermittent/temporary fault?
On Tue, Aug 21, 2018 at 2:07 AM Robert Schumacher<EMAIL_ADDRESS>wrote:

Hi @jjpepper https://github.com/jjpepper,
Could you try these two things and let us know if it helps?

First, pass --x-use-aria2 to the vcpkg install command. This will
use the aria2 download client internally instead of cmake which may
work better in your scenario.
Second, if the above doesn't work, could you try explicitly
downloading the file via appveyor's instructions[1] as
downloads\google-protobuf-v3.6.1.tar.gz, and then run the install
command?

Knowing which of those works will really help figure out a long term
solution!
[1] https://www.appveyor.com/docs/how-to/download-file/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vcpkg/issues/4101#issuecomment-414409972,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKDoYyYXryHAA54QbBhQavu9sQ1jHQBYks5uSvrHgaJpZM4WA8bH
.


Spamassassin infinitely eats server disk space (in overlay2)

Hi,
Using docker-postal in a VPS I have encountered a "no space left on device". After having investigated, I figured out that the issue comes from the spamassasin container.
I turns out that there is an infinitely number of empty folders created within postal-spamassasin-app container, inside the /tmp folder, here's an example (I verifierd that the sise in /var/lib/docker/overlay2 growth as well because of those folders) :
bash-5.0# ls -A  /tmp | head |  xargs du -sh
8.0K    .spamassassin10001DGbDLFtmp
8.0K    .spamassassin10001UD2WI8tmp
8.0K    .spamassassin1000243lK31tmp
8.0K    .spamassassin100026eeizUtmp
8.0K    .spamassassin10002JhfDx4tmp
8.0K    .spamassassin10002s1rxFetmp
8.0K    .spamassassin10002xi6f1Ntmp
8.0K    .spamassassin10004VepMcItmp
8.0K    .spamassassin10005NLVJRhtmp
8.0K    .spamassassin10005TAVulwtmp

After running the container for ~1days, I got more than 70k folder here
bash-5.0# ls -A  /tmp | wc -l
71776

Finally here is a sample of this container logs which may help understanding why all those folder are created
$ docker logs postal-spamassassin-app 2>&1 | tail -n20
config: no rules were found!  Do you need to run 'sa-update'?
[INFO] ** [spamassassin] Starting spamassassin
Jan 30 15:20:25.815 [9416] error: config: no rules were found! Do you need to run 'sa-update'?
config: no rules were found!  Do you need to run 'sa-update'?
[INFO] ** [spamassassin] Starting spamassassin
Jan 30 15:20:26.876 [9440] error: config: no rules were found! Do you need to run 'sa-update'?
config: no rules were found!  Do you need to run 'sa-update'?
[INFO] ** [spamassassin] Starting spamassassin
Jan 30 15:20:27.918 [9464] error: config: no rules were found! Do you need to run 'sa-update'?
config: no rules were found!  Do you need to run 'sa-update'?
[INFO] ** [spamassassin] Starting spamassassin
[and so on...]

docker logs postal-spamassassin-app 2>&1 | wc -l
215939

Any idea how we can solve this issue ?

A precise that running sa-update inside the postal-spamassassin-app container gives me the following error:
bash-5.0# sa-update
config: no configuration text or files found! do you need to run 'sa-update'?
Timeout::_run: check: no loaded plugin implements 'check_main': cannot scan!
Check that the necessary '.pre' files are in the config directory.
At a minimum, v320.pre loads the Check plugin which is required.

A precise that running sa-update inside the postal-spamassassin-app container gives me the following error:
bash-5.0# sa-update
config: no configuration text or files found! do you need to run 'sa-update'?
Timeout::_run: check: no loaded plugin implements 'check_main': cannot scan!
Check that the necessary '.pre' files are in the config directory.
At a minimum, v320.pre loads the Check plugin which is required.


Immutable.fromJS() not returning Immutable object in custom REPL

[tfalgout@beyondinto-lm foo]$ cat repl
!/usr/bin/env node
var repl = require("repl");
var replServer = repl.start({
prompt: "necs> ",
});
replServer.context.Immutable = require('immutable');
[tfalgout@beyondinto-lm foo]$ ./repl
necs> Immutable.fromJS({1:2});
{ '1': 2 }
necs> .exit
[tfalgout@beyondinto-lm foo]$ node

var Immutable = require('immutable');
undefined
Immutable.fromJS({1:2});
Map { "1": 2 }
.exit
[tfalgout@beyondinto-lm foo]$ node -v
v5.2.0
[tfalgout@beyondinto-lm foo]$ npm list
/private/tmp/foo
└──<EMAIL_ADDRESS>

This would be more a problem with the repl module which doesn't print the type no ?

Bumping this since I would really like to resolve this issue so I can use ImmutableJS:
I don't believe that it's an issue with the repl module since I see this issue even if I console.log() the output.  I'm unable to get ImmutableJS to return a Map in any runtime environment of Node outside of running the CLI:
Doesn't work:
[tfalgout@beyondinto-lm necs]$ cat bin/repl2
#!/usr/bin/env node
"use strict";
var repl = require("repl");
var replServer = repl.start({});
[tfalgout@beyondinto-lm necs]$ bin/repl2
>  var Immutable = require('immutable');
undefined
>  Immutable.fromJS({a:{b:{c:[3,4,5]}}});
{ a: { b: { c: [Object] } } }
> console.log( Immutable.fromJS({a:{b:{c:[3,4,5]}}}));
{ a: { b: { c: [Object] } } }

Works:
[tfalgout@beyondinto-lm necs]$ node
> var Immutable = require('immutable');
undefined
> console.log( Immutable.fromJS({a:{b:{c:[3,4,5]}}}));
Map { "a": Map { "b": Map { "c": List [ 3, 4, 5 ] } } }
undefined
> .exit

I'm sure there's something with my environment that is causing problems; however, I'm not sure where to look.  I have the same issue if I run via mocha test too.  (i.e. Immutable returns an object, not a Map)

I have tested this issue on v.0.10.41 and v4.3.0

Merging into #1039


add modem function TEST cases and more error handling

test reset, powerup, powerDown and modem connect functions

i have made the PR fixes, and
added IMEI get function, where in during modem-connect the device get the imei and stores it, no need to send IMEI at command every time we need the imei


No error when interface declares a function and property with same name and return type

TypeScript Version:  2.3.4 (same in 2.4.0-dev.20170601)
Code
interface Oops {
  foo(x: string): number;
  foo: number;
}

Expected behavior:
error TS2300: Duplicate identifier 'foo'.
Actual behavior:
No error.
I bisected using the NPM nightly builds (I ran into build problems trying to bisect from source), and this broke between 1.9.0-dev.20160524-1.0 and 1.9.0-dev.20160525-1.0.  I suspect #8696.

I would like to check on this one as a first shot. Could anybody please point me in the right direction? Thank you

Duplicate of #20826.


Add Dana option to built-in pump simulator

This adds Dana as an option for the pump simulator for the Trio app.
Problem with this PR
This may be an existing Trio issue that is revealed by this PR.
I set Max Bolus and Max Basal to the allowed values for Dana (80U and 35 U/hr) and that was accepted when the in-app simulator was accepted.
Omnipod Accepted same limits
When I deleted the simulator and added Omnipod DASH, the Max Bolus/Basal values remained too big for Omnipod.
There should be some range checking when Omnipod is paired - the values are not saved to the pods because pods have no such concept, but the limits should be respected.
Medtronic works as expected
I then attempted to pair a Medtronic and got - max settings exceeded.
After changing the max values to 2 and 2 and pairing pump

attempted maxBolus of 35 (this pump allows up to 25).
No error message is presented to the user, but the value reverted to the original 2U that was there before.
Seems like the error shown in the Xcode log should get picked up and reported:

2024-06-20 14:06:17.349292-0700 FreeAPS[14404:7137695] [MinimedPumpManager] Save delivery limit settings failed: command(MinimedKit.PumpOpsError.pumpError(Max setting exceeded))

same comment for maxBasal - I attempted 35, which was accepted by this pump and then 70. The attempt at 70 was restored on the display to 35 (no error was reported - same message as above).

Tagging @avouspierre and @bastiaanv for review.

Status
Confirmed Omnipods have appropriate behavior
Change from draft to final, ready for approvals.

Note - with this change the maxBolus and maxBasal can exceed Pod capabilities, but pumpManager limits requests to allowed values.
It would be better to limit these values to the pump values but that is outside the scope of this PR

Details
I returned to test this with an Omnipod rPi DASH pod.
Set pump limits to DanaKit max:

maxBasal = 35 U/hr
maxBolus = 80 U

Attempt to set Omnipod basal rate to 35 U/hr.

pump manager only goes up to 30 U/hr

Attempt to bolus 35 U:

It silently limited the bolus to 30 U.
Silent because the maxBolus was set higher than pumpManager allows

From rPi:
command; 0x1a; PROGRAM_INSULIN; HEX, 170e9b71101f1a0e494e532e02015a01258002580258170dc0177000030d400000000000008092
response 0x1d; HEX, 170e9b71140a1d6802092258002f17ff0000

Parsed HEX command:
  msgType =   0x1a17
  msgMeaning =   SetBolus
  autoBolus =   False
  prompt_bolus_u =   30.0
  nonce =   0x494e532e
  TableNum =   2
  chsum =   346
  hhEntries =   1
  secsX8Left =   9600
  pulsesPerHhr =   600
  pulse0 =   600
  xtype =   23
  xlen =   13
  reminders =   192
  promptTenthPulses =   6000
  promptDelay =   200000
  extendedTenthPulses =   0
  extendedDelay =   0
  mlen =   14
  msg_body =   1a0e494e532e02015a01258002580258170dc0177000030d40000000000000
  seqNum =   4
  rawHex =   170e9b71101f1a0e494e532e02015a01258002580258170dc0177000030d400000000000008092
  CRC =   0x8092

What’s the status on this PR?

It is ready to be merged

LGTM


Added campus leader section

Added the section for campus leaders on index.html. FIxes #160

Please change the background image

This is not mobile responsive please fix it ASAP

Yes please


Use react suspense to load the element when it's ready, instead of JavaScript

We are using JavaScript to wait for API to be ready, then render some components, this isn't the React way to do things.
Example:
     if (showLoading) {
        return <Loader fullScreen={true} />;
      }
      return <FourOhFour />;

https://github.com/Sboonny/freeCodeCamp/blob/240655ecd4c4ead1906c3c07ba10e4ad28af4622/client/src/client-only-routes/show-profile-or-four-oh-four.tsx#L74-L84?plain=1
  if (showLoading) {
    return <Loader fullScreen={true} />;
  }

https://github.com/Sboonny/freeCodeCamp/blob/240655ecd4c4ead1906c3c07ba10e4ad28af4622/client/src/client-only-routes/show-settings.tsx#L149-L151?plain=1
React provides a better way of doing things, https://beta.reactjs.org/reference/react/Suspense#caveats. we can use this instead to provide a better experience for the campers.
For example, there is two searchbars, one optimized and the other isn't. we can use suspense to display the optimized one, while the other one is loading.
        <Media minWidth={MAX_MOBILE_WIDTH + 1}>
          <Suspense fallback={<SearchBarOptimized />}>
            <SearchBar />
          </Suspense>
        </Media>

Here is branch, if you want to mess with the searchBar https://github.com/Sboonny/freeCodeCamp/tree/lazyload/searchbar
This will result in the searchbar being displayed in homepage, and if it take too long to load it display the optimize one.

If we do this we can add Errorbondary later one https://github.com/bvaughn/react-error-boundary, to have fault tolerance in our codebase, which will allow us to display 3rd component if the API fails, may display a button for camper that refresh the page or a function that ask the API to rerun.

Does this issue need help?

Nope, only issues with the https://github.com/freeCodeCamp/freeCodeCamp/labels/help wanted or https://github.com/freeCodeCamp/freeCodeCamp/labels/first timers only label is open for contribution.
If you would like to contribute, and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum

Okay Thanks!

So if we would want to replace
This :
     if (showLoading) {
        return <Loader fullScreen={true} />;
      }
      return

We can use this :
<Suspense fallback={<Loader fullScreen={true} />}>
  <FourOhFour />
</Suspense>

Right?

And lazy load the component https://beta.reactjs.org/reference/react/lazy#returns

@Sboonny I can work on this feature easily as I have already used Suspense and know how to lazy load components in suspense. Please assign it to me.

And lazy load the component https://beta.reactjs.org/reference/react/lazy#returns

Noted!!

@mud1tx, you can open PR after reading our contribution guide, I remember @Vansh-Baghel wanting to work on this, if you want to collab.

If you would like to fix this issue, please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or the forum if you need help contributing, our prolific contributors will guide you through this.

@Sboonny Thanx Sboonny, I will read the guidelines before contributing.

@Sboonny Hey I see this code in /show-settings.tsx, so may I wrap the full return with Suspense or let it be the way it is and only change where only one component is rendered and not the whole?


CI - rename the nix_installer_and_cache composite action to cache_nix

Rename the nix_installer_and_cache composite action to cache_nix

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

Hi @metaclips, How are you doing?
I'm Suraj can pick up this issue if yes please assign this to me and can I know more about this issue in order to understand it better and solve this issue?

All yours @Suraj-kumar00

So I just have to replace the nix_installer_and_cache composite action with cache_nix ?

Hello @Suraj-kumar00 you need to rename the folder nix_installer_and_cache  to cache_nix

Hi @metaclips, How are you doing?
I changed the folder name and created a pull request, please review...

Thanks @Suraj-kumar00 for getting this closed, we also have a discord community where we discuss Ockam stuffs, happy to hangout there.


Refactored code gen out of ExecuteJob. Fixes #1014

I need to double check that the cifar10 example is working and maybe add more tests that actually test a whole pipeline


Fail to run: cyber_recorder play -f demo_2.5.bag -l

Hi,
I comipled the apollo r5.5.0 and try to play the demo bag, but failed. See following for more info:
$ cyber_recorder play -f demo_2.5.bag -l
E0829 12:43:09.813866 18690 record_file_reader.cc:67] [cyber_recorder]Check section type failed, expect: 0, actual:<PHONE_NUMBER>
E0829 12:43:09.813889 18690 record_file_reader.cc:42] [cyber_recorder]Read header section fail, file: demo_2.5_2.bag
E0829 12:43:09.813899 18690 record_reader.cc:35] [cyber_recorder]Failed to open record file: demo_2.5_2.bag
earliest_begin_time:<PHONE_NUMBER>3709551615, latest_end_time: 0, total_msg_num: 0
E0829 12:43:09.813922 18690 play_task_producer.cc:176] [cyber_recorder]begin time are equal or larger than end time, begin_time_ns=18446744073709551615, end_time_ns=0

Any hints for this problem?
Regards,
HY

please try to run rosbag play 2.5.bag -l

Don't have that command at all, see following for more info:
$ rosbag -h
bash: rosbag: command not found

in version apollo3.0 or below can do this

Starting from Apollo 3.5, rosbags are no longer supported. Please refer to earlier version of ROS based Apollo. This issue is being closed. Please feel free to raise new issues should you have any. Thanks for using Apollo.


Add action to run codeql

Closes #40

@sleberknight I wonder, does your delombok script not work with multi-module maven projects?

@chrisrohr No the delombok does not work "out of the box" like that. You would need to define it once for each module that you want to delombok, passing in the appropriate source path. I also don't know if CodeQL itself handles multi-module Maven projects since none of the kiwi projects uses multi-module. It probably does, but I'd need to check. When I get some time, I'll look into this. In the meantime, we can either convert this PR to Draft, or just close it, since it might be "a while" until I can take a look at all that.

Without changing the existing delombok-action, we could actually change this workflow to run once for each module that contains Java code. I think that would work, even if a bit inelegant and inefficient.

Actually I'm wrong on that last comment about being able to run for each module, because there are currently some things in the shell script that will not work in a multi-module project. While you can specify the source directory, the way it builds the class path (needed for @Delegate to work properly) won't work. Though, it might still work if there is no usage of @Delegate.

And, I think I would also need to allow callers to specify not only the source directory (which you can do right now), but also the delombok source directory, which now is always src-delombok at the root level. So each module would delombok into the same directory, which would cause problems when copying the delomboked code back into the source directory. But I think that is an easy change. It might also be easy to change the Maven command to get the class path correctly.
We could maybe keep the source directory argument as-is, but add a new one which specifies the module subdirectory, e.g. so the delombok-action could be defined like:
    # Delombok before building code
    - name: Delombok elucidation-bundle
      uses<EMAIL_ADDRESS>      sourcePath: src
      modulePath: elucidation-bundle

If modulePath is specified, then it would look for the Java files to delombok in elucidation-bundle/src (combining the module path and source path), and it could then include the module path as part of the delombok source directory, e.g. elucidation-bundle-src-delombok.
And these would be pretty easy changes I think. But, the above would need to be done for each module. Which is somewhat annoying.
If we wanted to get more fancy, we could let the input specify all the modules to delombok, and make the script handle that. Of course, this makes things more specific to Maven. So maybe instead of the above we just do something like:
    # Delombok before building code
    - name: Delombok elucidation-bundle
      uses<EMAIL_ADDRESS>      sourcePath: elucidation-bundle/src

And then maybe we create the delombok source directory based on the source path, e.e. if the source directory is just src (the default) then the delombok source directory is just src-delombok. But if the source directory is elucidation-bundle/src, then the delombok source directory becomes elucidation-bundle/src-delombok. That way it can all stay separate, and yet still generic enough.
The issue with the class path still remains, and that is currently Maven-specific. Though, I just tried running the exact command the delombok-action uses and it actually put a classpath.txt file in each module directory, which is exactly what we want. Then, we would just need to make sure to read the correct class path file in for each module.


add support for filtering synched groups

not all groups in  a given IDP must always be synched.
add support for filtering groups.
this is probably an IDP-dependent implementation.

Implemented in 47d32c6620dd027ab0a9cc0b614eaf14c878e2ad


substates do not record parent state info

When entering character instances, if you chose a substate, the parent state is not recorded. This info is very important to have recorded, can this be automated?
For example:
character: Margin type
State untoothed
State toothed
substate dentate
substate serrate
substate crenate.
If I choose dentate, the fact that it is "toothed" is not recorded. Currently I would have to make two character instances to record the margin type being toothed:dentate.
We need to have parent states recorded for search and matching purposes, given that for some fossils you may know the parent state but not the substate.

Will this always be the case - i.e., will we ALWAYS want the state to substate relationship in ALL schemas?

Yes, Botany team has conferred, and yes definitely we will ALWAYS want the parent states recorded too.


x/crypto/ssh: allow returning arbitrary data from the server authorization callbacks

By server authorization callbacks, I mean the various callbacks listed here on godoc that return the Permissions structure.
I'm writing a SSH server that will check against a database to authorize the user's public key and then it will allow the user various abilities based on their permissions as listed in the database. The permissions are returned with the user. They are a integer type.
The problem is that I cannot pass those permissions from the public key callback straight to my implementation code. Instead, right now, I'm encoding the permissions as a string, putting them into the Extensions field of the Permissions structure and then decoding them in my implementation code. This is janky and will not scale well if I have to add more data that needs to be passed to my implementation code. I could query for the user in the database again in my implementation code, but that would be 2 queries for the exact same thing which is inefficient. Furthermore, I could use a shared map (that has its own set of issues) but I think there is a better solution.
I propose we add a new field to the Permissions structure that will carry arbitrary data. Presently, there exists the Extensions field but that is only a map from a string to a string, but I think we should add a new field that is a interface{} and will carry implementation specific data. I'm unsure of the name but that could be figured out later.

I understand your problem, but marshalling an int to string seems straightforward enough, so your case by itself doesn't warrant an API change.
I'd love to see some more reports of what people want to pass around.
(Maybe we should this be fold this together with a context-ization of the SSH package?)

closing this for now.


v2.0.0-RC8 release?

It would be nice to release the recent commit to support latest zio 2 rc.

I can help with the backport of #1259 if needed. I just can't find the scala 3 version of it. If there is some work already open I'll happily join.

@gurghet the branch is called 'zio-series/2.x'

Dropped #1279

I've added a port of netty changes from master to zio2 branch - https://github.com/dream11/zio-http/pull/1278

Fixed in https://github.com/dream11/zio-http/pull/1280

@amitksingh1490 thanks for publishing zhttp. It looks like zhttp now relies on zhttp-logging and that does not appear to be released.
https://repo1.maven.org/maven2/io/d11/zhttp_3/2.0.0-RC8/zhttp_3-2.0.0-RC8.pom
<dependency>
<groupId>io.d11</groupId>
<artifactId>zhttp-logging_3</artifactId>
<version>2.0.0-RC8</version>
</dependency>

There is no zhttp-logging_3 directory at https://repo1.maven.org/maven2/io/d11/


long email address touching the submit button

When entering a long email address to subscribe it was touching the submit button like below

By change the padding we can fix the problem like this.

I want to fix the problem as the part of Gssoc-2023

I am also interested in this issue.


./Build test ethereum.md fails with a parse error

On version dff137889e18b8a73b91e55bbb88dd8ee7702505, I see the following error when I try ./Build test ethereum.md.
$  ./Build test ethereum.md
== Using uiuck
== kompile: .build/uiuck/ethereum-kompiled/extras/timestamp
==  git submodule: cloning upstreams test repository
Submodule '.build/secp256k1' (https://github.com/bitcoin-core/secp256k1) registered for path '.build/secp256k1'
Submodule 'tests/ci/rv-k' (https://github.com/runtimeverification/k) registered for path 'tests/ci/rv-k'
Submodule 'tests/ci/uiuc-k' (https://github.com/kframework/k/) registered for path 'tests/ci/uiuc-k'
Submodule 'tests/ethereum-tests' (https://github.com/ethereum/tests.git) registered for path 'tests/ethereum-tests'
Cloning into '/home/yh/src/evm-semantics/.build/secp256k1'...
Cloning into '/home/yh/src/evm-semantics/tests/ci/rv-k'...
Cloning into '/home/yh/src/evm-semantics/tests/ci/uiuc-k'...
Cloning into '/home/yh/src/evm-semantics/tests/ethereum-tests'...
 I think you are looking for ./Build test-all that runs all tests or ./Build partest-all that use gnu parallel to run them in parallel. Besides all, there are other test lists you can use too. See ./Build help for details.

Then I was running a wrong command. Thank you!


Allow variable predicates in queries

Handling arbitrary chains of predicates is tricky, so focus on one predicate var per term for now (possibly including +,?,* support if it turns out to not be too tricky)

[ ] ?s ?p ?o (7461d2eba84954c58c1c2f3b1fc0fa0c41eca233, e6f234682420fd5bd1ef5a04cd62d04d736d6d2d, and ef66374c5a29ace6ea2d7667bde300be35106b63 so far)
[x] ?s ?p o (3acf8cd9bf83c11f8dbb1ba51721b6d4f1f71e92)
[x] s ?p ?o (3acf8cd9bf83c11f8dbb1ba51721b6d4f1f71e92)
[x] s ?p o (007a64b495f651a3c3a3a2833a041a89e06d6162)

We can move ahead with implementation of #4, even though the ?s ?p ?o pattern is still not implemented.
This last pattern is a little trickier because it involves changing the query planner to be aware of terms with 3 variables, which wasn't In the original design.

Turns out we did need the ?s ?p ?o pattern, but it was simple to do provided we had constrained at least one of the variables. The last part to do is the simple dump of the triples in the graph, which requires writing some new traversal operators. These should be easy because its just iterating over the underlying LevelDB, but we need to give some thought on how to iterate:

do we list the triples as they were inserted?
do we only want to list triples that are in the building?
do we want to ignore the 'inverse' triples?

We don't need this functionality just yet so I'm punting on implementing it.

Migrating this last issue to #12


Localizations added to View files #21

All view files have been update.

sample faker german language pack added for proof.

:clap: :100:
Wow, you've been busy. Incredible. Kordy is going to be overwhelmed by our PRs lol

@fishmad  Why closed?

Resubmitted with Controllers localized


Upgrade ORT ver to 1.11.0

Upgrade ORT ver to 1.11.0

@pranavsharma We see a compilation failure on windows. Did you test that build?
12>C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\onnx_utils.hpp(155,104): error C2660: 'google::protobuf::io::CodedInputStream::SetTotalBytesLimit': function does not take 2 arguments (compiling source file C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\ModelImporter.cpp) [C:\workspace\build\Release\external\onnx-tensorrt\nvonnxparser_static.vcxproj] [C:\tmp\tritonbuild\onnxruntime\build\ort_target.vcxproj]
12>C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\ModelImporter.cpp(362,108): error C2660: 'google::protobuf::io::CodedInputStream::SetTotalBytesLimit': function does not take 2 arguments [C:\workspace\build\Release\external\onnx-tensorrt\nvonnxparser_static.vcxproj] [C:\tmp\tritonbuild\onnxruntime\build\ort_target.vcxproj]
12>C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\ModelImporter.cpp(379,108): error C2660: 'google::protobuf::io::CodedInputStream::SetTotalBytesLimit': function does not take 2 arguments [C:\workspace\build\Release\external\onnx-tensorrt\nvonnxparser_static.vcxproj] [C:\tmp\tritonbuild\onnxruntime\build\ort_target.vcxproj]

@pranavsharma We see a compilation failure on windows. Did you test that build?
12>C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\onnx_utils.hpp(155,104): error C2660: 'google::protobuf::io::CodedInputStream::SetTotalBytesLimit': function does not take 2 arguments (compiling source file C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\ModelImporter.cpp) [C:\workspace\build\Release\external\onnx-tensorrt\nvonnxparser_static.vcxproj] [C:\tmp\tritonbuild\onnxruntime\build\ort_target.vcxproj]
12>C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\ModelImporter.cpp(362,108): error C2660: 'google::protobuf::io::CodedInputStream::SetTotalBytesLimit': function does not take 2 arguments [C:\workspace\build\Release\external\onnx-tensorrt\nvonnxparser_static.vcxproj] [C:\tmp\tritonbuild\onnxruntime\build\ort_target.vcxproj]
12>C:\workspace\onnxruntime\cmake\external\onnx-tensorrt\ModelImporter.cpp(379,108): error C2660: 'google::protobuf::io::CodedInputStream::SetTotalBytesLimit': function does not take 2 arguments [C:\workspace\build\Release\external\onnx-tensorrt\nvonnxparser_static.vcxproj] [C:\tmp\tritonbuild\onnxruntime\build\ort_target.vcxproj]

Our windows build uses an older version of TensorRT, so perhaps 1.11.0 requires a more recent TensorRT?

We also see many of these warnings on Ubuntu build:
/workspace/onnxruntime/onnxruntime/gsl/gsl-lite.hpp(1959): warning #20014-D: calling a host function from a host device function is not allowed
/workspace/onnxruntime/onnxruntime/gsl/gsl-lite.hpp(1873): warning #20014-D: calling a host function from a host device function is not allowed
And more problematically we see a couple of errors like the following on Ubuntu:
In file included from /workspace/onnxruntime/cmake/external/onnx-tensorrt/ModelImporter.cpp:8:
/workspace/onnxruntime/cmake/external/onnx-tensorrt/onnx_utils.hpp: In function ‘bool ParseFromFile_WAR(google::protobuf::Message*, const char*)’:
/workspace/onnxruntime/cmake/external/onnx-tensorrt/onnx_utils.hpp:155:104: error: no matching function for call to ‘google::protobuf::io::CodedInputStream::SetTotalBytesLimit(int, int)’
155 |     coded_input.SetTotalBytesLimit(std::numeric_limits::max(), std::numeric_limits::max() / 4);
|                                                                                ^
In file included from /workspace/build/Release/external/onnx/onnx/onnx-ml.pb.h:23,
from /workspace/onnxruntime/onnxruntime/../cmake/external/onnx/onnx/onnx_pb.h:52,
from /workspace/onnxruntime/cmake/external/onnx-tensorrt/ShapedWeights.hpp:8,
from /workspace/onnxruntime/cmake/external/onnx-tensorrt/onnx2trt.hpp:8,
from /workspace/onnxruntime/cmake/external/onnx-tensorrt/ImporterContext.hpp:7,
from /workspace/onnxruntime/cmake/external/onnx-tensorrt/ModelImporter.hpp:7,
from /workspace/onnxruntime/cmake/external/onnx-tensorrt/ModelImporter.cpp:5:
/workspace/onnxruntime/cmake/external/protobuf/src/google/protobuf/io/coded_stream.h:401:8: note: candidate: ‘void google::protobuf::io::CodedInputStream::SetTotalBytesLimit(int)’
401 |   void SetTotalBytesLimit(int total_bytes_limit);
|        ^~~~~~~~~~~~~~~~~~
/workspace/onnxruntime/cmake/external/protobuf/src/google/protobuf/io/coded_stream.h:401:8: note:   candidate expects 1 argument, 2 provided

Ok, let me follow up.

It looks like onnx-tensorrt parser was upgraded to accommodate the protobuf upgrade in ORT in 1.11. I'm testing the fix right now.

@deadeyegoodwin it's longer than expected for me to test due to some issue with my env. I'll try to test tonight, but if you get a chance before that it'll be great. All the errors you've seen so far was because of the tensorrt parser. Thanks!

It looks like you just reverted this change. I will start a CI with only the server repo change.

Yeah, because of this line https://github.com/triton-inference-server/onnxruntime_backend/blob/main/tools/gen_ort_dockerfile.py#L560. No special config is required; it should just use all the ORT defaults for 1.11.

@pranavsharma can this PR be closed if it is no longer needed

I tested this just now with ORT 1.11 and it works just fine. No PR needed.