BPDM: Adjust to Rights and Role Concept of 24.08. Release

Description

At the moment the rights and roles expected in the default BPDM configuration do not match entirely the configuration of the Central-IDP.

There is no client and therefore no roles for the BPDM Orchestrator component as specified here.
Like mention in  #146 there are no dedicated technical users seeded for establishing an authenticated golden record process
There is no technical user for the BPDM provider EDC to create an offer allowing access to the Pool member data

Aligning BPDM and Central-IDP reduces the initialization and configuration overhead. Therefore, I propose to fill the gap between the two systems to enhance the experience of the operators.
Acceptance Criteria

[ ] Add a client for the BPDM Orchestrator component containing the client roles as described in the permissions here.
[ ] Add Orchestrator roles to the technical_roles_management client according to the specification above
[ ] Add a service account for the Portal Gate to access the Orchestrator having the Orchestrator role 'Task Creator'
[ ] Add a service account for the Portal Gate to access the Pool having the Pool role 'Consumer'
[ ] Add a service account for the Cleaning Service Dummy to access the Orchestrator having the Orchestrator roles 'Clean Task Processor' and 'CleanAndSync Task Processor'
[ ] Add a service account for the Pool to access the Orchestrator having the Orchestrator role 'PoolSync Task Processor'
[ ] Add a service account for the BPDM provider EDC having the Pool role 'Member Consumer'

Additional Information

Hi @nicoprow very good that you make this inconsistency transparent!
Could you please set the milestone to the 24.12 release?
As we're at the end of E2E Testing for 24.08 this change can't be part of 24.08 anymore.
IMO, it should be documented as a known known for BPDM for the 24.08. release that its right and roles concept is not completely reflected in the CX-Central realm config and the workaround for it (so what configuration BPDM did in order to get through the E2E Testing) should be mentioned.
The IAM version for the 24.08. release is the 3.0.1 version and https://github.com/eclipse-tractusx/portal-iam/issues/146 will be part of it, as we added that technical user before E2E Testing started.
Could you please raise an issue for this topic in sig-release so that we can plan it accordingly for 24.12 release and also discuss the testing process for this change in the open planning?
Relates to https://github.com/eclipse-tractusx/sig-release/issues/578 more specifically https://github.com/eclipse-tractusx/portal-iam/issues/86
cc: @MaximilianHauer @jjeroch

Probably, this issue should be solved for 24.12 under the feature https://github.com/eclipse-tractusx/sig-release/issues/751

@nicoprow @Sebastian-Wurm does this issue also cover https://github.com/eclipse-tractusx/portal-iam/issues/132#issuecomment-2247221206?

@nicoprow @Sebastian-Wurm does this issue also cover #132 (comment)?

Yes, this should be included. Additionally to the missing technical users / service accounts for intra-service communication (BPDM internally), I also added a feature as seen by the company admin, that subscribes to BPDM: https://github.com/eclipse-tractusx/portal-iam/issues/168. Also this one should be handled in https://github.com/eclipse-tractusx/sig-release/issues/751

@nicoprow @Sebastian-Wurm does this issue also cover #132 (comment)?

Yes, this should be included. Additionally to the missing technical users / service accounts for intra-service communication (BPDM internally), I also added a feature as seen by the company admin, that subscribes to BPDM: #168. Also this one should be handled in eclipse-tractusx/sig-release#751

@Sebastian-Wurm ok regarding https://github.com/eclipse-tractusx/portal-iam/issues/132#issuecomment-2247221206, could you please take that over as part of https://github.com/eclipse-tractusx/portal-iam/pull/155#pullrequestreview-2224754775?
Regarding missing technical users / service accounts, I already commented on here https://github.com/eclipse-tractusx/portal-iam/issues/168#issuecomment-2273524074, it's not related to https://github.com/eclipse-tractusx/sig-release/issues/751 because in https://github.com/eclipse-tractusx/sig-release/issues/751 I'm referring to clients, service accounts, etc... that need to be seeded to be available at startup and with https://github.com/eclipse-tractusx/portal-iam/issues/168 you're are referring to service accounts that are created during runtime.

@nicoprow @Sebastian-Wurm does this issue also cover #132 (comment)?

Yes, this should be included. Additionally to the missing technical users / service accounts for intra-service communication (BPDM internally), I also added a feature as seen by the company admin, that subscribes to BPDM: #168. Also this one should be handled in eclipse-tractusx/sig-release#751

@Sebastian-Wurm ok regarding #132 (comment), @nicoprow could you please take that over as part of #155 (review)?
Regarding missing technical users / service accounts, I already commented on here #168 (comment), it's not related to eclipse-tractusx/sig-release#751 because in eclipse-tractusx/sig-release#751 I'm referring to clients, service accounts, etc... that need to be seeded to be available at startup and with #168 you're are referring to service accounts that are created during runtime.

OK, understood that you make the difference between Portal runtime and Portal startup time, where you only refer to Portal startup time in eclipse-tractusx/sig-release#751.
Do we get #168 implemented for 24.12, even if it's not so much related to eclipse-tractusx/sig-release#751.
From my point of view it's a left-over of 24.08 and also a security issue, as credentials are given to the Sharing Member, which secure the negotiated EDC assets of the BPDM Gate.

Do we get https://github.com/eclipse-tractusx/portal-iam/issues/168 implemented for 24.12, even if it's not so much related to https://github.com/eclipse-tractusx/sig-release/issues/751?

this conversation is slowly getting confusing.
we aligned to split up https://github.com/eclipse-tractusx/portal-iam/issues/168 first.
i assume the newly created story https://github.com/eclipse-tractusx/portal-iam/issues/175 is a part of this ? and therefor your question is referring if https://github.com/eclipse-tractusx/portal-iam/issues/175 could be part of 24.12 ?
anyway we are currently not able to provide a clear feedback on this as this is dependent on how quick we can deliver the topics steered and aligned already for 24.12

@MaximilianHauer Fair enough. #168 is now split up in #168 and #175. For both points there seems to be a workaround from Cofinity-X side / in their implementation. Still both requirements are valid. Let's see if someone from Cofinity-X wants to contribute.


Better intermediate format?

As @jmwright mentioned:

@Peque Here is a list of the supported export formats. Please note that there is direct three.js export support (TJS). That's actually what I use for cadquery-gui.

Maybe GLTF too, as @fragmuffin suggested.

Implemented in 1.2.0. Using CadQuery's TJS format (three.js JSON).


Video Content Not Resizing Properly

No matter how I change my settings the actual content displayed in a non-fullscreen video will not change it's dimensions properly and most videos will be off-center and have black space around them while others will resize beyond the boundaries of the player itself resulting in large portions of the video being cut-off.

I am using the standard userscript version of YouTube Center, v2.1.7, and my browser is Google Chrome 64bit version 42.0.2311.90, though I have had this problem for several months now while the resize worked fine in the past

Download the developer version instead, the 2.1.7 is outdated. You can download the dev version from here: https://github.com/YePpHa/YouTubeCenter/wiki/Developer-Version

Thank you very much, my problem has been fixed.


Make torch-mlir-dialects optional

Added IREE_BUILD_TORCH_MLIR_DIALECTS cmake option to opt-out building
torch-mlir-dialects. The IREE_HAVE_TORCH_MLIR_DIALECTS preprocessor
variable would be defined when the cmake option is turned on. Bazel build is ignored for now.

The failing test might not be unrelated. I think somewhere the iree-input-type=mhlo flag is getting dropped on that bot, and thats causing issues. This touches the build files so it might be related.
Not going to block it, but I really dislike having #ifdef in files. It is very easy to mess up.... It would be better if this could be refactored into files that either get compiled and linked or not. Might be hard to do in the present state though.

Nice! I think this may be missing documenting this config (docs/developers/get_started/cmake_options_and_variables.md). Seems all passing here except for the swiftshader one and that one's failures seem unrelated to me, could you double check if it is?

Looking at the failed test.


Patchoil book problems

Mystical World don't have some mentions on how to tame mob:

Roots:
The new ritual don't have discription in book(I'm talking about Gathering):

Thanks for the issue! I knew about this, I just haven't had time to rectify hte problem, but having an issue filed for it means I won't forget (which is what I did do when I released without it)!, so thank you so much.


New Syntax for CLI?

I've been trying to use the cli version of comictagger and I keep getting this error when trying to use this command.
comictagger -s -t cr -m "publisher: 'Quality Comics'" *.cbz
comictagger: error: argument -m/--metadata: invalid parse_metadata_from_string value: "publisher: 'Quality Comics'"
I thought I must have been doing something wrong, but I pulled this example directly from the cli guide thinking that it would work, however it does not. I am on the latest build directly for github. Was something changed and not added to the guide. The GUI works, but cli does not.

If you are comfortable pip installing an application, I recommend using https://github.com/ajslater/comicbox it has a much more robust cli for manual updates. I'll probably figure out why it isn't working later today

I think you might be on an old version, I can't re-produce this on the latest git commit. Can you provide the log? It would be located at.
%LOCALAPPDATA%\ComicTagger\logs on windows
~/Library/Logs/ComicTagger on macOS
~/.cache/ComicTagger/log on Linux

Here is the log for when i used cli. It doesn't have much in it. Is there some way to make it more verbose?
ComicTagger.log

Can you try running the appimage from the release page? I can't get it to fail in the same way

my version of glibc is too old for the appimage. It is part of the reason that I encode comictagger from source. I am on 2.31 but the appimage wants 2.35.

ok, lets try a virtual environment. Install pipx and run
pipx install git+https://github.com/comictagger/comictagger
it will install comictagger in ~/.local/bin
This will install the latest development code. I pushed a commit to hopefully catch the error that happens properly

If you need some python packages (pyicu,qt5) installed by your package manager (apt,dnf,apk) you can use
pipx install --system-site-packages git+https://github.com/comictagger/comictagger

I'll try that and report back. Thanks for the help.


Prepare schema 4.2 spatial datasets for UXR

Context:

For UXR, we need spatial datasets that are relevant to our participants' interests

DoD:

Have 3 to 5 datasets for UXR participants to use

@niknak33 should I assign this ticket to @sidneymbell ? Thank you!


Update cxx_config_cc.py to use is port.is_source

Fix for issue #181 Update the port description generation to use the port.is_source attribute.

Thanks for the contribution!

Hi, this test is failing the Change-ID check.
In order for the Change-ID test to pass, you must ensure you commit message has a Change-ID at the bottom. To amend your commit, run the following commands:
f=git rev-parse --git-dir/hooks/commit-msg ; mkdir -p $(dirname $f) ; curl -Lo $f https://gerrit-review.googlesource.com/tools/hooks/commit-msg ; chmod +x $f then run git commit --amend --no-edit
To learn more about Change-Ids, go here: https://www.gem5.org/documentation/general_docs/common-errors/#change-id
In addition, the commit message does not adhere to our current standard, and should include a tag within the header to categorize your changes.  You can read more about how to format your commit message in the committing section here (https://www.gem5.org/contributing), and see a list of accepted tags here (https://github.com/gem5/gem5/blob/stable/MAINTAINERS.yaml)
Thanks!

Updated the commit with change id and categories. Sorry for not checking the contributing guidelines


Make sure that list of plugins is sorted and unique

Description
Fixes #5396, #5402, #528
The sorting is performed by the file_name instead of the full text to be printed or full path
Checklist

[x] I have linked to any relevant issues.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

When running locally I now get this.
~ % forc plugins --paths
Installed Plugins:
/Users/joshuabatty/.cargo/bin/forc-doc
/Users/joshuabatty/.cargo/bin/forc-lsp

I would expect it should still show all of the forc tools? i.e forc-tx forc-fmt` etc...


[Question]: How to set a default prompt tempalte for a model?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I have this setup to use a remote llama-3 model:
import os

from llama_index.core import PromptTemplate
from llama_index.llms.openai_like import OpenAILike

SERVER_LLAMACPP = os.getenv("SERVER_LLAMACPP", "http://localhost:8181")
os.environ["OPENAI_API_KEY"] = "sk-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
os.environ["OPENAI_API_BASE"] = f"{SERVER_LLAMACPP}/v1"
os.environ["OPENAI_API_HOST"] = f"{SERVER_LLAMACPP}"
model = OpenAILike()

p = """<|begin_of_text|><|start_header_id|>system<|end_header_id|>

{system_prompt}<|eot_id|><|start_header_id|>user<|end_header_id|>

{prompt}<|eot_id|><|start_header_id|>assistant<|end_header_id|>"""
print(
    model.complete(
        prompt=p.format(
            system_prompt="You reply accurately to the user",
            prompt="What is the capital of France?",
        )
    )
)

There is a query_wrapper_prompt parameter I could assign in OpenAILike, however this doesn't seem to make any difference if later I call this (the output is gibberish):
model.complete(prompt="What is the capital of France?")

An option could be to sub-class OpenAILike to then format my content with the llama-3 prompt template, but this seems like overkill. I can't find information about default prompt templates in the documentation or issues, my questions are:

How do I set that PromptTemplate as the default wrapper for the model?
Is this the right way to work with llama-index?

@egeres you can set some function hooks in OpenAILike -- completion_to_prompt() and messages_to_prompt()
def completion_to_prompt(completion):
  system_prompt = "..."
  return f"""<|begin_of_text|><|start_header_id|>system<|end_header_id|>

{system_prompt}<|eot_id|><|start_header_id|>user<|end_header_id|>

{completion}<|eot_id|><|start_header_id|>assistant<|end_header_id|>"""

# this is very naive, I know llama3 has a chat format, I'm not typing out the logic
# normally you'd use message.role and message.content to build a prompt
def messages_to_prompt(messages):
  prompt_str = "\n".join([str(x) for x in messages])
  return completion_to_prompt(prompt_str)

llm = OpenAILike(..., messages_to_prompt=messages_to_prompt, completion_to_prompt=completion_to_prompt)


Repository

My Introduction

Name
Richay Bikhari
Class
Tech 3
About me
I really like doing graphic design and am fairly good at it. Aside from that, coding can also be fun once you get into it. I enjoy the problem-solving challenges that come with it and I like creating in general (systems/programs).
Goals
My goals for this class are to take my JavaScript skills to the next level and become a real wizard.
Skill level
I would describe myself as a intermediate JavaScript programmer.
GitHub username
https://github.com/richayb
GitHub repository
https://github.com/richayb/javascript-bootcamp

👋 Hi Richay, the link to your repo has been added to the admin sheet. Thanks! 🎉


How to prevent to return at pos on partial discounted/Promotion/Quantity break?

For example,
Case 1:
Customer can buy product X at 5$, Buy 5 quantity of x product at 3$, Incase customer bought 5 quantity at each 3$ by quantity break, so when customer wants to return 4 quantity, POS should not allow to return the 4 quantity, instead system should allow to return 5 quantity only.
Case 2:
If customer bought buy 2 get 1 free on promotion, POS should not allow to return that 2 product which customer paid early and walk away with free goods.
Any promotion/discounted/quantity break must not allowed to return partially, either allowed to return complete promotion/discounted/quantity break or NOT ALLOWED to return LOGICALLY
How can to prevent those cases in Dynamics 365 commerce?.

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

ID: 446bde34-336d-f791-b24f-c5b13f0ed685
Version Independent ID: 2f358f3a-acdd-e710-beef-24e19d154617
Content: Retail discounts - Commerce | Dynamics 365
Content Source: articles/commerce/retail-discounts-overview.md
Service: dynamics-365-commerce
Product: ****
Technology: ****
GitHub Login: @ShalabhjainMSFT
Microsoft Alias: shajain

Bug 756904 has been logged to address this issue.

This would need to be achieved via customization, as majority of users prefer not to have such constraints. Each CartLine has DiscountLines object which contains info of discounts applied to the line. One possible approach is for user to customize RecallSalesInvoiceRequest to prevent partial recalling.


Output the limited support reason that couldn't be removed

What?
Add the limited support reason to the log message of Found no limited support reason to remove, as this could lead to confusion when we remove several reasons.
Why?
Reduce confusion when going through CAD logs

/lgtm


Too much memory committed at static initialization

I am using OpenBLAS via OpenCV, compiled on SuSE 15SP5. The key point is that I used vm.overcommit_memory=2 for the whole system, to avoid OOM.
It works really fine until, we had systems with dual 20-cores with HT (so 80 virtual cores) systems. My program starts allocating a lot of memory, and in either free or htop, a lot of it is available, because it wasn't written, but the kernel see this as committed memory, and if I launch a second instance, it terminates. Worse, either program can't be launched anymore because of this committed memory.
So far, I've seen those solutions:

Use vm.overcommit_memory=0, but what if OpenBLAS really starts to use the allocated memory?
Use vm_overcommit_ratio higher than 100 and hope for the best
Use a low number of OPENBLAS_NUM_THREADS so that allocated memory is constrained, but how about performance?

Is there any preferred path? Is there any other runtime tunable? For maintenance and security, I can't recompile those library.
I hoped OpenBLAS provides something like "I want not too much memory", but it seems it's not possible.

OpenBLAS uses a statically allocated buffer for splitting/sharing matrix data between threads, its size is only configurable at compile time (BUFFERSIZE in Makefile.rule). So OPENBLAS_NUM_THREADS is your only option if memory is constrained. Do you happen to know how the library was compiled (even if you cannot change that right now) ? I've seen occasional cases where someone inadvertently built with NO_WARMUP=0 which causes a lot of memory trashing on startup as the library tries to find the fastest contiguous memory region to put the buffer.

The generated pkg-config files says: openblas_config= USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS= NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP=0 CORE2 MAX_THREADS=16
So I guess there's no BUFFERSIZE defined, and the default value of 32 << 22 is used.

Yes, it's the stock SuSE Enterprise package. But it's hard to find the spec file for that. It's available for openSuse, though.
My use case is "basic" computer vision, so yes, I guess that buffer is oversized. As a hint, htop report "only" 5 GB, while Committed_AS from /proc/meminfo gave ~36GB.

OpenBLAS' ancestor GotoBLAS was basically "build on your computer, with options according to your needs" - nobody thought of distributions packaging prebuilt libraries that are expected to cover all use cases. Maybe the /etc/alternatives mechanism could be leveraged to offer "small", "medium", "large" builds or some such, or OpenCV could bundle its own build (I guess nobody is going to love this idea...)

Thanks for the support, history and ideas. I'd go for OPENBLAS_NUM_THREADS, that I'll try to guess as a "best" according to my need and the combination of CPU count and RAM.

Chances are that the hyperthreading nodes are not offering that much more in terms of compute power, so something in between physical core count and full HT count may give best performance - and when your problem size is very small, most if not all BLAS functions in OpenBLAS should switch to single-threading anyway. (though I guess SuSE may carry a fairly old version of OpenBLAS where that may not be a given)

if I had fixed the bugs that caused gemv and gesv to use all-core multithreading even for small problem sizes back then already...


feat: deprecate deep requires

[x] Add test that covers the fact that deep requiring still works.

@broofa I have thought about this one again and I believe now is not the time to introduce a breaking change in how this library can be imported/required.
Instead, I decided to remain backwards-compatible with deep requiring the algorithm versions but emit a deprecation warning when people do that.
It looks something like this:
> const uuidv4 = require('./v4');
undefined
> uuidv4();
'5b7cced3-d703-49ff-b05e-a3c727f6e3e2'
> (node:77419) DeprecationWarning: Deep requiring like `const uuidv4 = require('uuid/v4');` is deprecated as of<EMAIL_ADDRESS>Please require the top-level module when using the Node.js CommonJS module or use ECMAScript Modules when bundling for the browser. See https://github.com/uuidjs/uuid/blob/master/README.md#upgrading-from-v3x-of-this-module for more informat

The warning is currently emitted each time the deep-required function is called. That could potentially lead to a lot of noise in applications, that generate a lot of UUIDs.
How do you feel about this? Do you think the risk of people flooding their logs is too big? Or should this be annoying enough for people to change their implementation?
An alternative would be to emit such a warning only once when the module is required.

@broofa I have now added a test case which asserts that deep imports now produce a DeprecationWarning.

No strong opinions, just a couple observations:

We're talking about a major release here... breaking changes are, by definition, expected. 🤷♂
What API would a "first principles"  dictate?  I.e. Ignoring past history, and given the current JS landscape, what is the "right" API here? Or, to put it differently, what API are we heading for with all this?

On that last item, one approach we haven't really discussed is breaking this up into separate modules, ala lodash's "lodash/array", "lodash/core", etc.  E.g. npm install uuid/v4, then require('uuid/v4').  This would address the current "deep" dependency issue that tends to break packagers.  I'm not necessarily advocating this, but it's worth considering.

We're talking about a major release here... breaking changes are, by definition, expected. 🤷♂

When I initially picked up the modernization of this library it was because more and more people were specifically asking for ESModule support.
So my main intention was to put out a new major release that makes the requested switch to ESModules so that Typescript and Webpack/rollup users can benefit from that.
Dropping support for default insecure RNGs seems like a long-overdue leftover that, given the widespread support for crypto, is likely not a breaking change for >99% of the users of this library.
I think we can provide what has been asked for by many users (ESModules) without having to break any existing usage of this library.
After that we can still think of more radical approaches to evolve the API of this library, but I would prefer to deliver ESModules without any significant other changes to the API surface.
Does that sound reasonable?

What API would a "first principles" approach  dictate?  I.e. Ignoring past history, and given the current JS landscape, what is the right API? ("What API are we heading for?")

On that last item, one approach we haven't really discussed is breaking this up into separate modules, ala lodash's "lodash/array", "lodash/core", etc. E.g. npm install uuid/v4, then require('uuid/v4'). This would address the current "deep" dependency issue that tends to break packagers. I'm not necessarily advocating this, but it's worth considering.

I think these days ESModules and named exports are the way to go.
When you compare to lodash the per-method packages like lodash.get are now deprecated and their docs mention that deep requiring is no longer necessary when using modern bundlers (https://lodash.com/per-method-packages):

Furthermore, modern tree-shaking bundlers like webpack and rollup can avoid bundling code you don't need even if you don't use direct imports or the babel plugin.

So I'm confident that promoting the modern
import { v4 as uuidv4 } from 'uuid';

style syntax is as good as we can get with JS in 2020.
The only thing that would come to my mind is renaming the imports, e.g.
import { uuidv4 } from 'uuid';

However then we should probably go into the discussion of export names again. Shouldn't it then maybe rather be
import { randomUUID, timeUUID, nameUUID3, nameUUID5 } from 'uuid';

?
I would prefer to leave that discussion for a later time.

I'm considering to rewrite the entire README to use import instead of require syntax. However it looks like it would be a major effort to get this to work with runmd. Any thoughts on this one @broofa?

That all sounds good to me.
Re: RunMD and ES6 imports, RunMD was a weekend project of mine (inspired by the need to keep uuid's docs up to date, actually) that's proven useful, but it's also... well... "quirky". Supporting ES6 is doable, but not something I have time to dive into at the moemnt.  Thus, if that gets in the way of a README rewrite I'm fine with ditching it.  If when someone (me, probably) gets RunMD working, we can decide whether to pull it back into the toolchain here.


Add a method to osmoutils for having a function write to state if no err

@antstalepresh made a neat method for doing this in a few places in the
superfluid epoch logic. This just refactors it to a standalone method in
osmoutils.
Furthermore, this PR also renames osmotestutils -> osmoutils, which
is a change I've wanted to make for a while.

For contributor use:

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #872 (0e268a8) into main (9b977f3) will decrease coverage by 0.00%.
The diff coverage is 69.23%.

@@            Coverage Diff             @@
##             main     #872      +/-   ##
==========================================
- Coverage   19.89%   19.88%   -0.01%
==========================================
  Files         191      191
  Lines       24777    24772       -5
==========================================
- Hits         4929     4927       -2
+ Misses      18984    18983       -1
+ Partials      864      862       -2

Impacted Files
Coverage Δ

x/pool-incentives/client/cli/tx.go
0.00% <0.00%> (ø)

x/superfluid/keeper/distribution.go
85.71% <100.00%> (+12.03%)
: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 9b977f3...0e268a8. Read the comment docs.


Custom config file input support

As of now, the default config is used to read inputs, with the support of the config flag one can provide custom config as well.

Blocked by goflags pending PRs


Haxe 3.2

Can we push the try haxe compiler along with the new 3.2 release? (Planned tomorrow)

Yes I'll merge all PRs and upgrade to Haxe 3.2 tomorrow.
Thanks :)

It would be also nice to have an option for this new static analyzer.

?

from the release notes: "Another new feature is the static analyzer which can be activated by compiling with -D analyzer. It applies various optimizations such as constant propagation and expression-level DCE which improves the quality of the generated code on targets such as Javascript. "

:+1: to upgrading to Haxe 3.2.0-rc.1

I've just upgraded to 3.2.0 rc2
Thanks !

Please mention the new version in the footer


No bluetooth devices listed on new device

Describe the bug
On Android 12, Devices in the surrounding are only shown if Bluetooth and GPS is activated
To Reproduce
Steps to reproduce the behavior:

Go to Devies Page
Click on 'Scan for available devices'
No devices are shown

Expected behavior
All available bluetooth devies should be displayed in the list
Smartphone (please complete the following information):

Device: Galaxy S20
OS: Android 12
Version Sprint4-Release

Additional context:
This is not really a bug, but it is suprising for the user that Bluetooth Search needs GPS to be activated.
Android 12 changed the way permissions work, so we have to check on how to implement it to work there without GPS.

Thought from Industry Partner (27.05.22): Maybe we should check to have all permissions or we write code to get all permissions?
No agreement so far. POs discuss it in the next team meeting if we should add this feature.

@Wandinad: You already edited the text, right? If so, please just delete the upper part of the body of the issue.

@Wandinad: As discussed today, please just implement a simple popup that appears when the user has bluetooth not switched on. The popup should inform the user that he needs to switch on Bluetooth for the features of our app to work. No more functionality needed.

Here is what the Pop-up looks like on Android, depending on which Services are implemented:


Send headers, status and response in one single GIL call

Instead of calling multiple times C, call just one time when writing or end to use less GIL

closed in favor of https://github.com/cirospaciari/socketify.py/issues/32


Place redirect from get to path param

Add redirects from /place?dcid=foo&topic to /place/foo?topic

finally fixed merge + tests, ptal :)


Configure Object Pages Using SAP Fiori Tools

Tutorials: https://developers.sap.com/tutorials/fiori-tools-configure-object-pages.html
I am unable to see PageMap and it's empty.

Write here how you think we can improve the tutorial ...

Hi Vinay,
Sorry to see that you are having this issue. When you preview the app, are you able to navigate to the object page?
Kind regards,
Conrad

Yes, I am able to navigate.

Thanks, Vinay. In that case, I will reach out to a colleague on the SAP Fiori tools team who may be able to better assist you. Thank you for your patience.

Thanks, Conrad. I will wait for assistance. As I am unable to proceed further.

Hi Vinay, Conrad forwarded me your case. For deeper analysis we would need to have a look at your project. Can you please delete the "node_modules" folder in your project, zip the project folder with remaining contents, and attach it here? Thanks!
Best regards,
Christoph

Thank you for your feedback. We haven't heard from you in a while, so we are closing the issue.
If you still have questions, feel free to reopen the issue.

Hi Vinay, Conrad forwarded me your case. For deeper analysis we would need to have a look at your project. Can you please delete the "node_modules" folder in your project, zip the project folder with remaining contents, and attach it here? Thanks!
Edit: in SAP Business application studio there is a download option form the context menu of the project folder, or from File menu.

lscreation.zip
here are my files, please have a look

Best regards, Christoph

Thank you Christoph

Hi @vinay31 please apologize the late response. With the latest version of ux-specification this problem shall be fixed.
Please upgrade to the latest Fiori tools release and open the Application info page for the project (palette command "Fiori: Open Application Info"), there you should find a link under Application status, which gives instructions how to upgrade the node module. After this please execute the command "Reload Window" in VSCode or refresh the browser window with BAS.


The XAML snippet will not work by itself. Needs c# with constructor and "InitializeComponent".

The XAML snippet will not work by itself.
As seen in my StackOverflow answer, you also have to have a c# code behind file, with the following contents:
public partial class SimplePopup: CommunityToolkit.Maui.Views.Popup
{
	public SimplePopup()
	{
		InitializeComponent();
	}
}

Without this, you'll get error message:

CommunityToolkit.Maui.Core.Handlers.PopupHandler found for TestApp.ProfilePopup is incompatible

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

ID: c6126a74-78aa-2c4f-ce92-2cda527ca9fd
Version Independent ID: c6126a74-78aa-2c4f-ce92-2cda527ca9fd
Content: Popup - .NET MAUI Community Toolkit - .NET Community Toolkit
Content Source: docs/maui/views/popup.md
Product: dotnet-communitytoolkit
GitHub Login: @bijington
Microsoft Alias: bramin

Fixed by #119


Submitting SlickCmd version 0.1.2

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

@wingetbot run

I tested the installation locally using the following command:
winget install --silent --manifest D:\github\johnlng\winget-pkgs\manifests\j\johnlng\slickcmd\0.1.2
The installation completed successfully on my machine.
Is it possible to get more details on what specific issue the package manager bot encountered, or any error logs?

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-10-15 05:34:24.434 [FAIL] Installer failed security check. Url: https://github.com/johnlng/slickcmd/releases/download/v0.1.2/slickcmd-0.1.2-x86_64-pc-windows-msvc.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 896.)

@wingetbot run

It's a known issue that Windows Defender flags slickcmd.exe as potentially malicious. I've already submitted the files for review at https://www.microsoft.com/wdsi/filesubmission, but unfortunately, the status has remained 'In progress' for some time.

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-10-16 08:07:04.277 [FAIL] Installer failed security check. Url: https://github.com/johnlng/slickcmd/releases/download/v0.1.2/slickcmd-0.1.2-x86_64-pc-windows-msvc.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 897.)

Hi @johnlng,
The package has to pass a Defender scan. Unfortuntely, I don't know of other ways to expedite.

@wingetbot run

Verify:

This is a tray-based app.
Hi @johnlng,

Thie application displays 1.0, while the 0.1.2 is written to the Registry. Should a DisplayVersion be used here?


TKWAR-4 Done

GFX Backend Implemented

Apply standard formatting. Load my settings (new ones are coming) and use Ctrl+Alt+Shift+L

Proper comments and documentation
Remove "Created by..."
On TankRenderer.scala, line 36: the renderer must know nothing about a Tank's status. Find alternative solution (the GFXManager will probably not call the renderer of dead tanks)
In UIManager.scala, "object Flags" is not good. It uses mutables. I understand they should be global flags. There's no "global" in the project, everything has a context. Probably the flags are associated with a manager. Create an immutable class that doesn't use var nor mutables. I like to suggest a simple "var flags: Map" inside a manager.
It doesn't seem ButtonControl.scala is used. Why label it deprecated? Can you remove it?
The try/catch GFXManager.scala, lne 46 looks pointless.
CircleRenderer looks redundant. It is a subcase of FilledCircleRenderer. Just use the name CircleRenderer and an option.
Rename from Color3d to Color3D

Rejected as we agreed few changes for the next one


Mmr unpack with max peaks bound

I wonder if we should introduce a variant of mmr.unpack that bounds the maximum number of peaks. This will allow the user to have an understanding of the memory requirements when allocating the mmr_ptr.  As it stands we cant guarantee that the peaks don't overwrite other data in memory.  This variant called be called mmr.checked_unpack, mmr.bounded_unpack or something along these lines.  Wdyt @hackaugusto / @bobbinth ?

The current implementation is limited by how many peaks can be represented in num_leaves. We use a u64 for that, so the maximum size is 65 words in total (1 for the num_leaves, plus the 64 possible peaks).
I think this should be made into a constant.

The current implementation is limited by how many peaks can be represented in num_leaves. We use a u64 for that, so the maximum size is 65 words in total (1 for the num_leaves, plus the 64 possible peaks).
I think this should be made into a constant.

I see, that's good to know - thanks for explaining.
As we are using Felt's to represent the num_leaves would it be 63 possible peaks as a Felt < 64 bits?

As we are using Felt's to represent the num_leaves would it be 63 possible peaks as a Felt < 64 bits?

Yep, I think that is correct.


Dorado models

Hello - are there any plan to release a new version with dorado models?

Need to switch from using Clair-Ensemble to using Clair3. Are you using ClusterV for HIV or some other species?

Thanks! yes HIV!


Add external command callbacks

While working on resolving issue #1661 I noticed that pragma unknown-command = disallow mostly does what I needed and made it easy to identify most (all?) of the external commands used by the epm builtin. I say most because it does not warn about explicit external commands; e.g., e:cat. The epm builtin currently does not use such explicit external commands (AFAICT), but if it did the unknown-command pragma would not be useful for identifying all of the external commands it uses. A callback that is run before any external command is executed would solve this need and potentially have other uses. The callback should, obviously, be invoked with the external command path and any arguments. It's not clear whether the value stream output of the callback (interpreted as a bool value) should affect whether the command is actually run. If that feature were implemented it would presumably mean treating the external command as if it exited with a non-zero status if the callback indicated the external command should not be run. I don't see a good use-case for that behavior and am therefore not inclined to implement it but I think it's an open question.

The callbacks should include additional info to make it easier to know the context in which the external command is being run; e.g., which module, script, or interactive context invoking the external command. This came to light in response to a change proposed by @xiaq to my PR #1680 to replace some external commands in the epm module with builtins.


Added downloads page and updates links to download page

Due to an error updating the main repository this pull request was inadvertently closed and will need to be resubmitted.


unable to install on linux

i cannot install this on manjaro (arch linux based distros) it tells me that the cache folder is containing root-owned files, but it can't because i deleted all .npm folders multiple times after trying the suggested solution of sudo chown -R 65534:1000 "/root/.npm". it throws an error:
node:internal/modules/cjs/loader:928
  throw err;
  ^

Error: Cannot find module 'node-linux-x64/package.json'
Require stack:
- /usr/lib/node_modules/mineflayer-statemachine/node_modules/node/installArchSpecificPackage.js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:925:15)
    at Function.resolve (node:internal/modules/cjs/helpers:98:19)
    at ChildProcess.<anonymous> (/usr/lib/node_modules/mineflayer-statemachine/node_modules/node-bin-setup/index.js:18:27)
    at ChildProcess.emit (node:events:376:20)
    at maybeClose (node:internal/child_process:1063:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:295:5) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/usr/lib/node_modules/mineflayer-statemachine/node_modules/node/installArchSpecificPackage.js'
  ]
}

this error seems to be specific to mineflayer-statemachine or a dependency
2021-01-10T00_13_00_096Z-debug.log

fixed by changing default npm directory for symlinks, permissions made problems. adding the ability to use sudo for installation should be considered. else an error on my part

fixed by changing default npm directory for symlinks, permissions made problems. adding the ability to use sudo for installation should be considered. else an error on my part


Add multi-head template for migration

Todo before merge:

waiting on merge: https://github.com/dotnet/maui/pull/20365

Retrieve Net 8  version that this will be released on to add as a note to the Multi Template section

add more information on app entry points?

Internal previews

📄 File
🔗 Preview link

docs/migration/multi-project-to-multi-project.md
Manually upgrade a Xamarin.Forms app to a multi-project .NET MAUI app

Superseded by #2288


mxnet.ndarray.sparse.CSRNDArray.astype docs error

Description
Sphinx throws an error when generating the docs for this function.
Error
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/sparse.py:docstring of mxnet.ndarray.sparse.CSRNDArray.astype:5: ERROR: Unexpected indentation.
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/sparse.py:docstring of mxnet.ndarray.sparse.CSRNDArray.astype:8: WARNING: Block quote ends without a blank line; unexpected unindent.

@mxnet-label-bot [Doc]

PR merged and issue fixed, issue good to close


Add more langs

This adds support for more langs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi! Just saw this, can we remerge this, since I have now signed the CLA?


A stab class for the ha-checker agent

This class is supposed to implement ha-checker agents in the future.

@MalloZup could you please give your feedback?


Could not find service BluePic-Insights-for-Weather to bind to .....

Using the "easy button" within the readme, the deployment fails with "Could not find service BluePic-Insights-for-Weather to bind to ....."

Moving this issue to the BluePic repo

Issue moved to IBM-Swift/BluePic #292 via ZenHub


Resolver Service has a problematic static reference to the UDDI library

If the UDDI library is not installed, the Resolver Service cannot run due to a static reference on the UDDI library.

We can anticipate that the UDDI library will often not be required.  The code automatically asserts an instance of its rule helper class (InquiryService) using a static reference.  I have removed this and replaced it with a dynamic reference using the System.Activator class.


"HTTP Error 404: Not Found" when running installer

(.venv) jannetta@Floki:/media/jannetta/WORKDRIVE/DATA/GitHub_Repositories/Carpentries/screenscrape$ offlinedatasci install all .

Destination:  R/R-4.3.1-win.exe
Download path:  https://cran.r-project.org/bin/windows/base/R-4.3.1-win.exe
****Downloading file:  R/R-4.3.1-win.exe

Destination:  R/R-4.3.1.pkg
Download path:  https://cran.r-project.org/bin/macosx/base/R-4.3.1.pkg
****Downloading file:  R/R-4.3.1.pkg
Traceback (most recent call last):
  File "/media/jannetta/WORKDRIVE/DATA/GitHub_Repositories/Carpentries/screenscrape/.venv/bin/offlinedatasci", line 8, in <module>
    sys.exit(cli.main())
  File "/media/jannetta/WORKDRIVE/DATA/GitHub_Repositories/Carpentries/screenscrape/.venv/lib/python3.10/site-packages/offlinedatasci/cli.py", line 53, in main
    get_installer_functions(i, ods_dir)
  File "/media/jannetta/WORKDRIVE/DATA/GitHub_Repositories/Carpentries/screenscrape/.venv/lib/python3.10/site-packages/offlinedatasci/cli.py", line 8, in get_installer_functions
    download_r(ods_dir)
  File "/media/jannetta/WORKDRIVE/DATA/GitHub_Repositories/Carpentries/screenscrape/.venv/lib/python3.10/site-packages/offlinedatasci/main.py", line 39, in download_r
    download_r_most_current_ver(latest_version_url_mac, destination_path)
  File "/media/jannetta/WORKDRIVE/DATA/GitHub_Repositories/Carpentries/screenscrape/.venv/lib/python3.10/site-packages/offlinedatasci/main.py", line 156, in download_r_most_current_ver
    urllib.request.urlretrieve(download_path, destination_path2)
  File "/usr/lib/python3.10/urllib/request.py", line 241, in urlretrieve
    with contextlib.closing(urlopen(url, data)) as fp:
  File "/usr/lib/python3.10/urllib/request.py", line 216, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.10/urllib/request.py", line 525, in open
    response = meth(req, response)
  File "/usr/lib/python3.10/urllib/request.py", line 634, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.10/urllib/request.py", line 563, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.10/urllib/request.py", line 496, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.10/urllib/request.py", line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 404: Not Found

Notes on investigation into this:

The base R source code for installing mac binaries uses contriburl. I can't get the construction there to work, but it can be set manually to either:

"https://cloud.r-project.org/bin/macosx/big-sur-x86_64/contrib/4.3"
"https://cloud.r-project.org/bin/macosx/big-sur-arm64/contrib/4.3"

This works in download.packages() and available.packages(), so this allows the core behavior in miniCRAN to work in base R
However, miniCRAN's functions pkgAvail(), updatePackages(), and makeRepo() don't take contriburl as an optional argument and therefore can't pass it on to available.packages()


Check case insensitive for DDOS-GUARD

Fixes #15

This is a clever way of doing it. Since String.Contains() itself can’t be done case insensitively on .NET Standard < 2.0 I just defaulted to the naive way. Either way I hope Flaresolverr or whoever accepts the commit and pushes it out, since it’s been over a month now since the two of us have submitted fixes.

Thank you both!


How to set ENV["JUPYTER"] to the path of the jupyter program?

I install jupyter by pip. But how to set ENV["JUPYTER"] to the path of the jupyter program? I didn't know where is the path of jupyter on windows 10?

where jupyter in the Windows command line should tell you where it is.
Please don't file multiple issues (#566) when you have problems.


support pureproxies for multisig creation/mgmt

currently multix requires at least one "local" account as participant in multisig and wont allow proxy controls

Right, to create a multisig you need to be a signatory. You can watch any multisig/proxy though using the Settings>Watch account.
I'm not sure I understand the second part of your question.
Multix is a tool for multisigs, it's not aimed at being a general purpose wallet. So we're trying to support anything related to multisigs and proxy controlled by a multisig, but not more.

oh I understand maybe. You mean that you can't have a

account -> pure proxy -> signatory of a multisig. Is that it?

In that case, yes this is a limitation because I believe that such constructs are a super niche. The construct with a simple multisig controlling a pure is more transparent to all users, since a key change in the signatories needs a re-approval by the threshold of signatories. I do undertsand it may not be sufficient in very specific cases, but making the application more complex, and hence less maintainable for this reason. It's a trade off we have to take :(

yes exactly that. it was possible to do via polkadotjs, but quite a footgun.
spending countless hours today setting up IdentityJudgement right proxy on people-paseo to multisig without success we end up dropping the initial multisig draft and just use direct signer to control multisig and rotate new multisig to control final pureproxy if changes required.


npx browserslist@latest --update-db

Current dependencies on/for this PR:

master

PR #77

PR #78   👈

This comment was auto-generated by Graphite.


fix(firestore-send-emial): add SMTP secret password

fixes: #648
fixes: #692
Problems

password isn't hidden and could be compromised
password containing some special characters can make an address decode incorrectly
possible problems with regex validation

Solution
Break the URI address into sections. Create a new optional password field for setting up an SMTP server with a hidden secret password.
Checklist

[x] add optional field in extensions.yaml. When SMTP_PASSWORD is provided the SMTP_CONNECTION_URI password won't be used. (this should provide a smooth update process)
[x] add new ENV variables in config and jest-config, emulator-param files
[x] add a setSmtpCredentials helper function for selecting the right mail config
[x] add unit tests for setSmtpCredentials helper function
[x] fix URI regex validation

Tests
Update Engine

Installation and configuration of extension

Successful results (both setups)

Extra tests
Old setup

New setup

Codecov Report

Merging #840 (c1f897f) into next (3b1cf99) will increase coverage by 0.45%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             next     #840      +/-   ##
==========================================
+ Coverage   50.19%   50.64%   +0.45%
==========================================
  Files          30       31       +1
  Lines        1329     1343      +14
  Branches      197      201       +4
==========================================
+ Hits          667      680      +13
- Misses        661      662       +1
  Partials        1        1

Impacted Files
Coverage Δ

firestore-send-email/functions/src/index.ts
5.75% <50.00%> (+0.55%)
:arrow_up:

firestore-send-email/functions/src/helpers.ts
91.67% <91.67%> (ø)

firestore-send-email/functions/src/config.ts
100.00% <100.00%> (ø)


manylinux packages for Python

This adds three lists with manylinux dependencies as well as three
packages that include all the manylinux dependencies.

Motivation for this change
Thanks to @thomasjm for their work in https://github.com/NixOS/nixpkgs/pull/55812.
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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-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
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @jonringer @adisbladis @thomasjm

I think you need to also edit the interpreters so that they say they are "manylinux" compliant
  echo "manylinux1_compatible=True" >> $out/lib/${libPrefix}/_manylinux.py

yes, that's a separate step. I'm still thinking about the interface.

should we mark this as WIP then?

No, I make that change separate as it would be a mass-rebuild. This is usable without anyway.

Yea, that's fair

@FRidh I have renamed manylinuxPackages to pythonManylinuxPackages (as suggested on IRC) and rebased.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/mach-nix-create-python-environments-quick-and-easy/6858/17


Problems with cells containing unicode characters

I am trying to extract the first row of an excel sheet to pull the column headers. However, I noticed that for a header called DÍA it is completely ignored. Here is the code.
var _ = require('lodash');
var XLSX = require('xlsx');

var workbook = XLSX.readFile('test.xlsx');
_.each(workbook.Sheets['SEPTIEMBRE 2015'], function(cellValue, cell) {
    var column = cell.replace(/[0-9]/g, '');
    var row = cell.replace(/[A-Z]/g, '');

    if (row == "1") {
        console.log(column, "...",cellValue.v);
    }
});

Here is the result. In bold is the data ignored by js-xsls.
A ... DÍA
B ... MES
C ... ANYO
D ... FECHA DE TOMA
E ... RESPONSABLE
F ... PROCEDENCIA
G ... NOMBRE
H ... APELLIDO
I ... CEDULA
J ... # DE LA UNIDAD
K ... RESULTADO
L ... RESULTADO
M ... RESULTADO
N ... HBSAG
O ... xxx
P ... xxx
Q ... CORE
R ... PLACA DE ARCHIVO
The weird thing is, this just happend when the word DÍA is processed. If I put a tilde in any other string it does not ignore it. So, for example, if the cell contained the word PASTELERíA it will ignore it.
Please help.

Can you share an offending file?  I tried to manually generate a file with the text you provided in Excel 2016 and I see the correct results.  The XML shared string table shows:
<sst xmlns="http://schemas.openxmlformats.org/spreadsheetml/2006/main" count="2" uniqueCount="2"><si><t>DÍA</t></si><si><t>PASTELERíA</t></si></sst>

and reading with XLSX shows the same result:
> require('xlsx').readFile('DIA.xlsx').Sheets.Sheet1
{ '!ref': 'A1:A2',
  A1: { t: 's', v: 'DÍA', r: '<t>DÍA</t>', h: 'DÍA', w: 'DÍA' },
  A2:
   { t: 's',
     v: 'PASTELERíA',
     r: '<t>PASTELERíA</t>',
     h: 'PASTELERíA',
     w: 'PASTELERíA' } }

Without seeing the test.xlsx file it's hard to say what's going on.  Please update to the latest version and test again.  If the problem persists, reopen when you get a chance and attach a sample file so we can take a look


Feature Request: Multiview - pan for cells

When viewing multiple streams using multiview, can I make it so that the audio for each cell can be panned to the left or right?
It's very difficult for me to distinguish between 4 or more chat-streams voices that are not separated.

Rejected for same reason as #502 . May be considered as a browser extension feature.


[Non-Issue] Only works with sudo. setcap CAP_SYS_PTRACE=+eip sekirofpsunlock, solved it

Hi,
I had to set capability, for it to make it work without sudo.
just it posting it here for reference.
sudo setcap CAP_SYS_PTRACE=+eip sekirofpsunlock

Interesting. Either you're running the game as a different user or using some restrictive security policy. I'd be glad if you could answer the following questions:

Which distribution are you on?
Are you running a custom kernel?
Does the file /proc/sys/kernel/yama/ptrace_scope exist?

yes, I was able to read it with cat, sudo cat was not required.
value is 1,
I am on Arch with xanmod kernel - 5.17.12-xanmod1-1


Add OpenEBS k8s Provisioner

OpenEBS Provisioner (openebs-provisioner) implements the kubernetes dynamic storage provisioner.
Create and Delete methods interact with the maya-apiserver (a container running in K8s) to create and delete OpenEBS volumes respectively.

@satyamz Please include a README in this PR.

/assign @humblec

@satyamz @kmova something is really wrong here, may be while you pulled GetAccessMode() and subjected patch from gluster-block , it also pulled entire gluster/block/cmd/glusterblock-provisioner/glusterblock-provisioner.go or some other commit. Please remove those :(

@humblec I agree that there's something wrong. You added lib/util in the latest commits to kubernetes-incubator/external-storage which we needed for the validation of AccessModes. However, when I pulled (fetch+merge) kubernetes-incubator/external-storage as a remote on openebs/external-storage mirror, it somehow got previous commits (and committed by me) too and that's terrible. I tried to rebase it and then pulled it but again the result was same. Hence, we've decided to rebase everything on the openebs/external-storage mirror and pull the remote kubernetes-incubator/external-storage again, and will raise new PR.
Regarding CLA: I signed CLA as 2 days ago but it seems not working too. I hope this will also get fixed in new PR :slightly_smiling_face:
Many Thanks,
Satyam
/cc @kmova

Closing this PR as new PR is raised #315 with same changes and appropriate email id in commits for CLA.

@satyamz I doubt this is the correct way to do. That said, just for sake of fixing a commit conflict opening a new PR is not recommended atleast IMO. This PR has background of comments and reviews it will be kind of side tracked by the new PR and if we follow the same method, if there is a new conflict in next PR, it will land to endless PR spawning.

@humblec  I agree with you that this PR has important comments and reviews. I am not sure how to get rid of last merge commit. I just didn't want to spend more time on these git commits issue as I lost the track of git operations I tried! Another reason for opening new PR was CLA, today we solved it. In the PR https://github.com/kubernetes-incubator/external-storage/pull/315,  I used my official email to commit the changes (which is necessary for the CLA) and there's no conflict, merge commit, etc. Hence, we would like to get those changes/commits in than this.  I have referenced this PR in new PR so that we won't lose the track! It would be highly appreciable if you could just close this PR.

@satyamz actually getting the solution for merge conflict would have been easy. Any way, I will look into new PR, however I need to kind of start from scratch on new PR. Here I knew the checkpoints/commits to resume.. Let me take a look at the new PR and will do the needful accordingly.

@humblec thanks!

I am closing this PR in favor of https://github.com/kubernetes-incubator/external-storage/pull/315


Display error message

When for example the user does not have enough pokedollars, it now display a 400, but could display "user does not have enough pokedollars"

Not quite fixed, apparently


modernize recipe ad add maintainer

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

waiting for pythonhosted 504 errors to be fixed

@conda-forge/core can someone please merge to get this feedstock going? @pmlandwehr is inactive since quite some time here con CF, so I will gladly help maintain the package. @pmlandwehr feel free to comment!
not sure about what to do with the 504 errors on pythonhosted, should we want and then merge?


Wrong artist in case of non-produced track

If someone post a track who doesn't belong to him and put Artist - Title as a title, we have to take into account that Artist is the artist and not the user who posted the track.
While calculating the title during the ID3 identification, why not check if a dash - exists? If it's the case, the first substring is the Artist and the other the Title
Example : Fouk - Orchard (STW Premiere)

Yeah, this is a good idea but it doesn't work in practice, I'm sorry. There are so many tracks that use different formats, there are songs that have '-' in their titles, some art Artist - Track, some are Track - Artist, etc. It's a wild goose chase. :[

You are right, no problem. Good job anyway for your excellent script.

Thanks! Please report any other issues you find!


chore: Add readiness check on cache sync

Fixes #N/A
Description
This check was dropped ahead of v1 in https://github.com/kubernetes-sigs/karpenter/pull/1460
Adding it back.
How was this change tested?
Manual validation
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 77.864%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/operator/operator.go
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/scheduling/requirements.go
2
98.01%

Totals

Change from base Build<PHONE_NUMBER>2:
0.01%

Covered Lines:
8903

Relevant Lines:
11434

💛  - Coveralls


Popover: isDismissable support

Support for isDismissable on Portal as users might need to do several actions inside or using Autocomplete without closing Popover.
This can also be done with ignoring onClose on Popover and just have a button setting open state to false.
Decide which approach feels best

When onClose is ignored on Popover, it won't be dismissed by clicking outside of the Popover.


Update ms.topic

We're using landing-page for a different kind of content. Please use "overview" or "article" here. Feel free to email sabarret with any questions.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Rewrite: rewrite for simplicity and easier testing

Summary of Issue
the process of writing tests for smolcart.js as well as looking over best practices for this type of library from an API perspective has inspired me to do a bit of a rewrite.
The goal would be to:

change the internal action API as well as the Smolcart object API for easier testing.
possibly make the wrapper even smaller

#8 closes this issue


[BUG] Anchor acting strange

Checklist before submitting an issue

[X] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[X] I have specifically verified that this bug is not a common user error
[X] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug
I was trying to make a vertical bar.
I started out with this:
(defwindow bar
:monitor 0
:geometry (geometry :x "0%" :y "0%" :width "2%" :height "100%" :anchor "bottom left")
:stacking "fg"
:exclusive true
"a"
)
but then i found out, (#296) that i couldn't use exclusive with a bottom left anchor, it had to be center for at least one.
so, i changed my geometry anchor to bottom center, intending to just have -50% x to put it on the left side of the screen.
u cant change the x position when its bottom center. and my window manager(wayfire) starts acting very strange with window management now. (it also breaks at top center)
when i set it to just center, it was the same as top center and bottom center, but as well as breaking my window manager and not being able to change the x, it wasnt even 100% of the screen now, it was slightly downwards and changing y in the config didn't help either.
at left center, exclusive works, but it has the same y position issue as when i just do center.
Reproducing the issue
in a window on wayland, put exclusive true, and mess around with the anchor.
Expected behaviour
Well, it would be great if i could change the x and y position even when something has a center anchor.
Additional context
First one (left bottom)

Next one (-50% x, bottom center)

Last one (left center)

oh, actually seemed to be caused by the default wayfire bar, when i turned that off, the left center one seemed to work.


exception in GridDistortion

version: albumentations-0.1.7
call:
    distortion = albumentations.GridDistortion(num_steps=32, distort_limit=0.7, border_mode=cv2.BORDER_CONSTANT, p=0.3)
    sample = distortion(image=sample)['image']

image is 256x256x3
Thrown exception:

File "C:\Users\owner\AppData\Local\conda\conda\envs\allegro_env\lib\site-packages\albumentations\augmentations\functional.py", line 412, in grid_distortion
cur = prev + x_step * xsteps[idx]
IndexError: list index out of range

This seems not to be an issue on<EMAIL_ADDRESS>Therefore I'm closing it. If the problem persist feel free to re-open it and attach a test case for reproducing.

Hi, i receive this error too. Code below to reproduce it.
from albumentations import GridDistortion
from albumentations import Compose
import numpy as np
import cv2

augmentation = Compose([GridDistortion(num_steps=5, distort_limit=0.3, interpolation=cv2.INTER_LINEAR, border_mode=cv2.BORDER_CONSTANT, p=1)], p=1)

err_height = []
for i in range(200):
    try:
        image = np.random.random((i, 100))
        image = augmentation(image=image)['image']

    except Exception:
        err_height.append(image.shape)

print(err_height)

Output:
[0, 1, 2, 3, 4, 7, 8, 9, 14]
Seems like there are some special numbers. By increasing the num_steps, the number of special numbers increase too.
At num_steps = 10 we have
Output:
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 12, 13, 14, 15, 16, 17, 18, 19, 24, 25, 26, 27, 28, 29, 36, 37, 38, 39, 48, 49]


Descriptions related to Negin's analysis is ready to review

Hello Negin!
I appreciate the efforts you made to analyze the data. Definitely, the result of your own work was very organized additionally with explanations and notes that make it easy to read and understand this analysis.
But here are a few things that I have realized from the results of the work of all my friends. The first is that, in the presentation of data in the form of a graph, less attention has been paid to details such as numerical values and the final goal of analyzing this data. For this reason, we have a large number of graphs generated from raw and unclassified data. Displaying raw data as a graph cannot provide us with useful information; However, it may visually present a general picture and we may notice some relationships and trends. For example, In charts (Analyze preference variation based on different factors: country), countries like Afghanistan and Germany cannot be compared, at least here. In this case, we probably need to convert the data into many other indicators and provide a correct understanding and description with their help.
The next problem is the lack of measured values (numerical values) in the graphs. As it has been said, visually it is possible to make guesses about the amount and the upward or downward trend of various phenomena by means of a diagram. Because these graphs (especially when they are extracted directly from raw data) can provide a good general picture, but to describe and present the findings of this analysis, we need numerical values to be able to explain the details to the reader. In this case, I can refer to the graph related to (Distribution of Career Paths by Educational Background). In this diagram, we have many elements of Career Paths, but unfortunately no data of the value of each of these is provided. We need numerical values to analyze and describe this diagram and the impact of each of these elements.
Final issue: For some charts, I think the explanations you wrote yourself are enough. Because I could not add more description to it.
I must mention that almost these problems are exist in most of the analyzes presented by you, Mustafa and Joshua (but less in your work). I even tried to analyze the data a second time to get numerical values. But in the remaining cases, the complexity of the matter and lack of time prevented it from being done. If you think it is possible to modify these for yourself, I will definitely do the description part.
The job of describing the analyzes related to Mustafa and Joshua has also been completed and Zia has taken the responsibility push them. I hope you will read the draft of these descriptions as soon as possible and provide your comments to complete it.
Thank
Murtaza


fix for issue 89

I recall we discussed a unit test for this, which I forgot about until after I made the pull request.
This failed creating a bosco submission file, it would also have been caught if we had a bosco CE to test submissions to.
I suppose we need to unit-test all supported entry types, the ones I remember off the top of my head are  bosco, htCondor, aws, gce.    I think slurm and pbs now sit behind htCondor job routers but I would have to dig.


Fix Docker build CI

https://github.com/FedCampus/dyn_flower_android_drf/actions/runs/5676296947/job/15382814794
It seems that the runner does not have requests for its Python. @Johnnybyzhang

Fixed as of #34


Wrong error message for function call

When following the readme in near-bindgen cross contract example https://github.com/nearprotocol/near-bindgen/tree/master/examples/cross-contract-high-level, at this step
near call cross_contract simple_call "{\"account_id\": \"status_message\", \"message\":\"bonjour\"}" --accountId=test_near --gas<PHONE_NUMBER>000000

near shell gives
Error:  ActionError: The used access key requires exactly one FunctionCall action
    at Object.parseRpcError (/usr/local/lib/node_modules/near-shell/node_modules/nearlib/lib/utils/rpc_errors.js:25:19)
    at Account.signAndSendTransaction (/usr/local/lib/node_modules/near-shell/node_modules/nearlib/lib/account.js:94:36)
    at processTicksAndRejections (internal/process/task_queues.js:85:5)
    at async scheduleFunctionCall (/usr/local/lib/node_modules/near-shell/commands/call.js:29:34)
    at async Object.handler (/usr/local/lib/node_modules/near-shell/utils/exit-on-error.js:4:9) {
  type: 'ActionError',
  index: 0
}

but the actual error I see from printing results in nearcore is Failure(Action #0: Exceeded the prepaid gas.).
It could be a bug in nearcore or nearlib and we should transfer this issue when we have more insights into the issue.

Verified that this is fixed.
Note that the original steps to reproduce are no longer valid (create account commands and gas values are out of date - filed an issue https://github.com/near/near-sdk-rs/issues/152)


VS Code tab not working

Hello!
I've got my WSL2 installed on my W10, but the VS Code tab not working on devtools.

  System:
    OS: Linux 5.10 Ubuntu 22.04.1 LTS 22.04.1 LTS (Jammy Jellyfish)
    CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
    Memory: 23.56 GB / 25.02 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.12.1/bin/yarn
    npm: 8.19.2 - ~/.nvm/versions/node/v18.12.1/bin/npm

Tested directly with the debugger of Code v1.76.2 and with Webstorm v2022.3.4.

I cannot help as the given information is not sufficient. Please make sure you have code-server cli available in your environment.


Bump to latest versions (along with the wasi-sdk-20 builds)

Add latest versions for libxml2, ruby, php, python
Made dependencies declarative via wlr-info.json, which shall also contain the rest of the information with time (currently spread in wlr-*.sh scripts)
Added basic smoke test with sqlite and python.
Slim version of ruby and php will be added subsequently and libraries will be published subsequently.

@assambar, the PR LGTM, great work!
I would double-check new builds with our current demos.
For instance, we could check PHP 8.2.6 with the Drupal demo by updating the Dockerfile and performing manual inspection (ie: setup, a new page, attaching pictures, etc.).


More of a concern than an issue

Recently the server has switched to Togglesnipe 2 with 1.5 second timer - the settings for the channel has been updated respectively as shown in the attached images.

However, after every roll it will show valueError and a bunch of other messages relating to get_snipe_time and snipe_delay and things like that as shown in the first image.
As of right now, no wishes have dropped, and it still claims kakera as it should.
Is there anything in those messages that should raise concern?

Just an update, this is what it looks like when I roll myself:

The first three were me using the $mk command, the fourth was me using $ma

Update: A wish of mine appeared and it was not able to claim or even attempted to claim

I wasn't there for this so I didn't see exactly what happened, but before the togglesnipe came into play this would've claimed no problem

Update: the additional error type messages that show up after every roll seem to only coming up when I have claim - when I no longer have claim it shows someone else rolled in the channel as normal

The first few rolls at the start of the screenshot are when I don't have claim, the last one is after my claim time has reset.
When a wish shows up, the script isn't able to pick it up likely due to the message it shows when I do have claim.
Any advice?

Basically the claim delay is getting to many variables and not getting proper settings let me test when I change mine to a value with a decimal it might be an easy fix we shall see another work around is to set ur 1.5 > 2 so that way it can still snipe
Sent from my iPhone

On Aug 17, 2021, at 11:04 AM, Kaizuk-O @.***> wrote:

Update: the additional error type messages that show up after every roll seem to only coming up when I have claim - when I no longer have claim it shows someone else rolled in the channel as normal
The first few rolls at the start of the screenshot are when I don't have claim, the last one is after my claim time has reset.
When a wish shows up, the script isn't able to pick it up likely due to the message it shows when I do have claim.
Any advice?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Yep, I just tested it in a server with 3 sec delay instead of a number with a decimal number and it seems to be fine.
For a server using 2 sec, 3sec, 4sec. etc. delay; what would you say is the best claim delay to use in the Settings_mudae.json file to ensure it still claims as instantly as it can?

alright I've push code changes the new revision to support decimal numbers in $settings if found

2cff11b930cd742f9ca4b7a7e0e1bd3cd9a679db


Document all public types

Kinda opinionated, I feel.
Also, a generic crate level README.
I'm a pretty big fan of these Clippy lints:
“missing-docs”,
// less useful for plugins/libraries
// not enabled for this PR
“clippy::missing_docs_in_private_items”
Not intended to be merged just yet, if at all. This is probably the one that's going to need the most refinement.

is there is a way to disable doc test for examples?
honestly i dont know if the example at the top is worth the effort.
also sorta feel the same way about documenting everything but it really does come down too "feel"

is there is a way to disable doc test for examples? honestly i dont know if the example at the top is worth the effort.

There is
```ignore

But I don't think we should use it unless we absolutely have to. I think tested or at least compiled doc comments are great.

does this need too be rebased on the most recent commits?
i noticed that the gamepad.rs on this branch does not have the updated TOUCH_GAMEPAD_ID and related comment?
will git solve this on its own when merged?

I think this is ready to merge.
I ran clippy::pedantic and noticed some code that didn't really match the rest of the style, so I “fixed” it.
Furthermore, I did not change the doc comment for TOUCH_GAMEPAD_ID, but I did back-tick the const being used

docs for TOUCH_GAMEPAD_ID shouldnt include why its u32
if this does stay, it should be a normal comment.

Since this is internal API (not pub), I think it's good to have it as a doc comment.


doc: Add README for shared, and link READMEs in the crates

This should fix the big looming "appears to have no README.md file" texts on crates.io. I don't think we can really test this other than wait what it looks like after the next release, but the absense of "unused manifest key" warnings in a cargo check is a good sign.
The main crate just uses the top-level README; all others use a shared one that points to the main crate.

Thank you for this PR, it looks good. Left one minor comment.

why it points to the README of the shared crate?

Because it fast-forwards the user to the relevant main crate with the
overview documentation. Several crates with all the same text (and the
user won't know unless they diff, maybe the detail is somewhere down)
are confusing to users. The shared README is ... well, shared among all
other crates :-) and is just that: a pointer that says it's part of the
big thing.

I see, then it can just be overridden when crate-specific READMEs are added. Cool!


Need help with hosting the Electron app under an existing IIS website as an application

I am a newbie to Electron and trying to figure out how to host the application built using Electron.NET using .Net5 framework. Any kind of documents or youtube link or steps would help out in this regard.

CLosed it bymistakely. Looking forward for any possible solutions in this regard.

🎉🚀 New Electron.NET version 23.6.1 released 🚀🎉
With native Electron 23 and .NET 6 support. Your problem should be fixed here. If you continue to have the problem, please let us know. Please note the correct updating of your API & CLI. Info in the README. Have fun!


Implement auth

To see what lunches you ordered!

https://github.com/filiptronicek/strava.cz-api/commit/0d88791688b141cd63f84464c01d792495365d09


[PR #1300 @RexSong] #110 Convert rlexe.xml tags to use new build architecture and type terms

Includes PR #1300 by @RexSong
Fixes #110

@RexSong Feel free to update in #1300. This PR was just meant to run CI when runtests.py has the execute bit set. I may also use this PR to add any additional changes that I would need to make after your PR is updated.


Hide without camera

Hi
I want to hide those whose camera is turned off from the screen. How can I do that.
thanks.
@misi

@Astagor Great work! This is needed! Thanks!

@Astagor Great work! This is needed! Thanks!

thank you. good work.

thank you. good work.


add copyright and author to .vcc file in vmod

Hi Daniel, I modified your .vcc comment, please check when you have time. XD

Merged. Thank you.


pull request for e2e test at 01/19/2017 01:45:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Create keystore without cert

EnvKeyStore should be able to create a keystore file without a certificate. This would be analogous to a command like:
keytool -genkeypair -alias pac4j-demo -keypass pac4j-demo-passwd -keystore samlKeystore.jks -storepass pac4j-demo-passwd -keyalg RSA -keysize 2048 -validity 3650

This is derived from the example in the pac4j documentation.

Closing since this is an old issue without any traction currently. If a user of this library needs this functionality, please comment here or open a new issue.


Docker Windows mysql hostname

This is  a bug.

Prerequisites

[X] Are you running the latest version?
[X] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[X] Did you check the documentation?

Description
artisan works using <IP_ADDRESS> or localhost but application requires setting hostname to enso-mysql (the containers name)
Steps to Reproduce

Expected behavior

Actual behavior

Setting DB_HOST=enso-mysql in your .env file should be all you need to change for the docker-based set-up.
Can you give reproducible steps please?

It does work when setting to enso-mysql but I didn't see that in the documentation.

Docker support was a recent contribution by a member, however the steps to use under Docker do state

Rename .env.example to .env and fill out the appropriate variables.

As the current (localhost) .env.example still makes sense for most I believe this is correct.
Feel free to contribute to the documentation to highlight this.
p.s.: note that even the default installation steps point this out

edit the database configuration information


Deep Dive edits: 1

I'm doing  a series of deep dives, making sure that the docs are where they should be.
This will be an ongoing process; please flag any issues you see, but I think the changes I'm making are generally positive.
One thing to note: this commit, there was a random on line 68 of this contract. I deleted it. was supposed to read "there was a random x on line 68 of this contract. I deleted it.", but for some reason bash interpreted my x as a command, even in the context of double quotes, and removed it ?!?

@cedrabner, I'd appreciate a review and merge, if possible.

So... I can't tell if the links are actually broken, or not. :shrug:

I've checked some of the Docusaurus links; they're false negatives. Others are API endpoints. Merging.


Add mobile device permission request detection

Add support for mobile device permissions request detection.  Note there a LOT of them potentially.  TBD to decide which to report on or granularity.
gfs commented 2 days ago:

For iOS how this works might be interesting - https://github.com/delba/Permission
For BSD you could look at pledges: https://man.openbsd.org/pledge.2
UWP https://docs.microsoft.com/en-us/windows/uwp/packaging/app-capability-declarations

We will continue this topic within #260 instead and close this.


Update TanStack Start adapter

The Start demo is currently broken, we should update @tanstack/react-form/start with the new createServerFn APIs

Also mentioning #1006 as it might be included here

Hi - would there an update here since Start is now in beta?

Any update on this? It seems not to be working.
In this example:

export const handleForm = createServerFn(
  'POST',
  async (formData: FormData, ctx) => {
    try {
      await serverValidate(ctx, formData)
    } catch (e) {
      if (e instanceof ServerValidateError) {
        return e.response
      }

      // Some other error occurred when parsing the form
      console.error(e)
      return new Response('There was an internal error', {
        status: 500,
      })
    }

    return new Response('Form has validated successfully', {
      status: 200,
    })
  },
)

It no longer works and I try to refactor but I don't know what to pass as the ctx, also should we use the validator to run the serverValidate function or we run it in our handler? And the response return no longer seems to work in the handler as the handler does not expect a response to be returned.


Wire up FSDataInputStream.setDropBehind, set to true in FileCompactor

Closes #3078

Be sure to look at commit msg for explanation of changes


Data layers visible when user switches styles while editing

Steps to reproduce:

Draw a polygon
Enter edit mode
Move the polygon
Switch styles

The layer that displays the polygon is hidden when entering edit mode.  When changing styles, the sources and layers for the current data are added on style.load.
To fix:

on style.load check if user is editing. If so, set visibility to none in each addLayer()
similarly, pass isEditing into addMarkers, and set each marker's element style to display: none before adding to the map.
when the user exits edit mode, existing code will toggle the visibility of these layers and markers back to visible

A simpler fix for this may just be to disable/hide the layer-switching UI when in editing mode.


Use logrus to log info to the user

logrus provides more information than fmt functions do,
and this helps standardize the way we present information
from lokomotive.
Related to https://github.com/kinvolk/lokomotive/issues/320

Thanks @BrainBlasted! I like the initiative and am happy you've picked this up :+1:
While I do think it makes sense to "unify" the output we show to the user, I'm wondering about the following:
Should we separate the lokoctl output to two categories - user output and logs? By "user output" I mean text which constructs the UI, in this case a text-based UI (as opposed to a GUI). By "logs" I mean messages which are more diagnostic in nature and aren't a part of the UI.
To make this distinction clearer, imagine lokoctl was a graphical UI rather than a text-based UI. Which text would you build into the GUI and which text would you, say, write to a log file?
To make this even more concrete, in the current state the following looks weird to me:
INFO[0694]
Your configurations are stored in ./assets   args="[]" command="lokoctl cluster apply"

Now checking health and readiness of the cluster nodes ...

Node                             Ready    Reason          Message

johannes-test-pool-1-worker-1    True     KubeletReady    kubelet is posting ready status
johannes-test-controller-0       True     KubeletReady    kubelet is posting ready status
johannes-test-pool-1-worker-0    True     KubeletReady    kubelet is posting ready status

Success - cluster is healthy and nodes are ready!

Having INFO[xxxx] on lines which are clearly user output is weird IMO. Same goes for having things like args="[]" command="lokoctl cluster apply" mixed with the user output. I'm wondering if we even need these pieces of information at all (because if we hadn't, the question above would be less relevant).
Happy to hear any thoughts on this.

Having INFO[xxxx] on lines which are clearly user output is weird IMO.

You're right. Though some lines are a bit confusing as to whether they're strictly user output, or would be useful for diagnostics. Should all lines visible to the user running lokoctl cluster apply, for example, be considered user output?

Happy to hear any thoughts on this.

I think we need to put a bit more though in how do we handle the user output. In my opinion using a logger for user output everywhere is a good thing, however we should do more in terms of formatting the output. I have a following idea: when logging on INFO level, we should skip INFO[xxxx] and other standard pieces and just print a text, like we would do with fmt.Println. And for DEBUG logs, it should be fine to mix DEBUG[xxxx] output with regular lines printed on INFO level. Or we could format them the same way as DEBUG logs then, for consistency, as while debugging, the UX is not so important.
Another thing is, that I have some doubts regarding the DEBUG logs, as it is difficult to predict in which places to put them and with what information to make them useful, which may lead to some abuse of those in the code. Maybe we could even skip them entirely? Logging seems more useful to me for event-driven applications like web-server and not necessarily for the CLI tools.
Last though for the further work is, that I think we should treat logging as a dependency of our objects and code, so it is easier to control the behavior of logs (to be able to suppress them or redirect to file programatically, if needed).

Having INFO[xxxx] on lines which are clearly user output is weird IMO.

You're right. Though some lines are a bit confusing as to whether they're strictly user output, or would be useful for diagnostics. Should all lines visible to the user running lokoctl cluster apply, for example, be considered user output?

I don't have a concrete concept in mind regarding what should be user-facing output and what shouldn't. I mainly wanted to raise the question because I sensed it was glossed over.

I think we need to put a bit more though in how do we handle the user output. In my opinion using a logger for user output everywhere is a good thing, however we should do more in terms of formatting the output. I have a following idea: when logging on INFO level, we should skip INFO[xxxx] and other standard pieces and just print a text, like we would do with fmt.Println. And for DEBUG logs, it should be fine to mix DEBUG[xxxx] output with regular lines printed on INFO level. Or we could format them the same way as DEBUG logs then, for consistency, as while debugging, the UX is not so important.

I agree.

Another thing is, that I have some doubts regarding the DEBUG logs, as it is difficult to predict in which places to put them and with what information to make them useful, which may lead to some abuse of those in the code.

The beauty about logging is that we don't need to predict where we may want to log. It's the same as tracing or metrics: all we have to do is initialize logging centrally and use a tried-and-true method for making the logger available in arbitrary parts of the code. I'm sure there are Go best practices for that, and likely a bunch of community practices as well. Happy to do a bit of research about this if you guys need my help there.

Maybe we could even skip them entirely? Logging seems more useful to me for event-driven applications like web-server and not necessarily for the CLI tools.

I disagree. I think complex CLI tools definitely need logging. Take Terraform for example. How many times were you able to solve a problem by running the tool with TF_LOG=debug? lokoctl is at least as complex as Terraform because it uses Terraform...

The beauty about logging is that we don't need to predict where we may want to log. It's the same as tracing or metrics: all we have to do is initialize logging centrally and use a tried-and-true method for making the logger available in arbitrary parts of the code.

I agree that it's important to keep it around so it's available, but as I said, we need to be careful what we log to avoid cluttering the output.

I'm sure there are Go best practices for that, and likely a bunch of community practices as well. Happy to do a bit of research about this if you guys need my help there.

Still on my TODO list, if someone has some proven expertise, I'm happy to learn :smile:

I disagree. I think complex CLI tools definitely need logging. Take Terraform for example. How many times were you able to solve a problem by running the tool with TF_LOG=debug? lokoctl is at least as complex as Terraform because it uses Terraform...

Well, TF_LOG=debug is only helpful in particular cases, mainly debugging 3rd party providers I'd say, but yeah, as we discussed above, keeping it a round definitely make sense. Thanks @johananl.

Hm, how should we proceed with this PR? @BrainBlasted would you like to continue working on it, taking our feedback into account? Or should we make a summary of findings in the issue and close it, so we can make another attempt another time?

Any updates on this PR?

I'll close this PR and we can come back to it later.


Feature Request: Only output the identified language with -id and -b

Right now I do this :
root@z:~# ./trans -b --id --input /home/x/Desktop/sample.vtt
Malti
Name                  Maltese
Family                Afro-Asiatic
Writing system        Latin
Code                  mt
ISO 639-3             mlt
SIL                   http://www-01.sil.org/iso639-3/documentation.asp?id=mlt
Glottolog             http://glottolog.org/resource/languoid/id/malt1254
Wikipedia             http://en.wikipedia.org/wiki/Maltese_language

[...]

Português
Name                  Portuguese
Family                Indo-European
Writing system        Latin
Code                  pt
ISO 639-3             por
SIL                   http://www-01.sil.org/iso639-3/documentation.asp?id=por
Glottolog             http://glottolog.org/resource/languoid/id/port1283
Wikipedia             http://en.wikipedia.org/wiki/Portuguese_language

I suppose this is the guesses it does, until it reaches a final decision on the last output right? But that's way too much information, and not friendly with scripts.
Maybe I'm missing something? I looked at the --help section, tried all kind of combinations of commands to suppress this output, and it seems to be breaking grep too:

It would be nice if we had a option to suppress even more output so we can get right to what we need in a non-interactive way, like ISO codes, name, etc.
Thanks ❤

Hey I see you closed this, but I'm still getting the same output? Is it a different combination I have to use?


Modelica.Icons.InternalPackage

Reported by stefanv on 26 Feb 2016 16:43 UTC
There are a number of packages in the MSL and other libraries that extend from Modelica.Icons.InternalPackage.
When generating help for a library, should the contents of such a package be visible? The description invariably ends up being,
This icon shall be used for a package that contains internal classes not to be directly utilized by a user.
because the extending package doesn't add its own documentation. However, there is no annotation suggesting that the package should not be visible. Is there a convention that says it shouldn't be (because it extends from Modelica.Icons.InternalPackage)? What do other tools do with such packages?

Migrated-From: https://trac.modelica.org/Modelica/ticket/1927

Modified by dietmarw on 26 Feb 2016 17:04 UTC

Comment by hansolsson on 8 Mar 2016 13:46 UTC
Dymola doesn't do anything special for this for the help-pages. I think this is acceptable (tried to recreate it as good as possible - except the links), and think the text is useful.
--
[[span(style=color: #007F00, Information )]]
Extends from Modelica.Icons.InternalPackage (Icon for an internal package (indicating that the package should not be directly utilized by user)).
[[span(style=color: #007F00, Package Content )]]

Name
Description

Fet
Record for Fet parameters

FetModelLine
Record for Fet model line parameters

CurrrentsCapacitances
Currents and Capacities

fetRenameParametersDev
Device Parameter renaming to internal names

--

Modified by otter on 5 Aug 2016 05:58 UTC

@svorkoetter Closing with worksforme.

@HansOlsson @svorkoetter We could set the annotation(Protection(access=Access.icon)) for these Internal packages to explicitely exclude them from HTML generation. What do you think?

To me it is still not clear that there is a problem with having documentation for the classes in the internal package.
As an example consider
Modelica.Blocks.Continuous.Internal.Filter.Utilities.BesselBaseCoefficients used in
Modelica.Blocks.Continuous.Internal.Filter.base.Bessel used in
Modelica.Blocks.Continuous.Filter
to me it makes sense that one can dig down to BesselBaseCoefficients to understand the underlying calculations; without having all that documentation in the Filter-model.
And I don't see a good way to document the models in Modelica.Blocks.Continuous.Internal without having some documentation for that package.

As an example

Enough reason to include the generated HTML documentation of Internal classes as is.


Escaping filenames sent to shell command, protecting against 'dangerous'...

... characters.

Thanks for both PR's! I'm currently on the road, I'll take a look next weekend!

Hello, thank you!  It is my first time contributing, let me know if I should do anything differently in the future!


Fix metadata service queries treating 404s as OK

Fixes https://github.com/aws/eks-anywhere/issues/5564
When the Hegel metadata service doesn't have a Hardware record it returns 404. Our writefile Tinkerbell action treats the 404 as OK resulting in invalid metadata being written to disk and ultimately consumed by kubelet on launch, causing it to fail.
This change treats non-200 responses as errors ensuring we move to query any additional metadata endpoints.

/approve


Display the cable color in the Interfaces and the Front/Back Ports table

Environment

Python version: 3.9.2
Nautobot version: 1.2.5

Proposed Functionality
Display the cable color (if configured) in the Interfaces and the Front/Back Ports table, for example next to the cable ID.
Use Case
If a color coding scheme for cable patching is employed at the site, displaying the configured cable color helps users quickly identify the cable's use.
Database Changes
None. Colors are already a part of the Cable model.
External Dependencies
None. See above.

Thanks for the feature request!