[RFC] Identify, Discuss, and Generate a List of Useful Guides

Problem

These Guides are meant to serve as an expansion to existing documentation for beginners of this project, that means Guides should be directly and uniquely related to this project. Generalized beginner level material about broader topics like Github, Git, Command-line basics, etc. may be better served in the Open Source documentation as a part of our new overall general beginner guides.

Description

Please read the guides README.md which outlines what a new Guide should include, and the conventions it should follow.
Please comment below with any suggestions for Guides we might benefit from creating that are not already there.
This Issue can be referenced in sub-issues that spin out from it.

Additional context

#110
#112
#141

Implementation

[ ] I would be interested in implementing this feature.

I'm currently on it now!

It's a discussion... Meaning you drop down your ideas here....

I'm currently on it now!

It's a discussion... Meaning you drop down your ideas here....

Thanks for that @ImaJin14 !
I said I'm on it, meaning going to look for ideas for this now

Talking about useful guides, I decided to contribute on CC Meta Search today, reading the internship task on Outreachy website it states interns are expected to

Rewrite the application, maintaining its original function, as a static site on GitHub pages, using semantic, standards-compliant, and accessible HTML, CSS, and JavaScript. Update the visual aesthetic and overall appearance to better correspond with other current CC web entities.

To achieve this I assume it means to rewrite the application to a JavaScript application, that can be hosted on Github pages while maintaining its current functions. This leads to the question if the JavaScript guide in the other RFC should be adhered to when refactoring the codebase? If yes, then can it be refactored using Vue as the preferred JavaScript framework?

@onyijne I think the Javascript guide on the open source site needs a little clarification. The meaning there is more that if one is going to use a front-end component framework, with JS the standards at the moment, they should go with Vue (rather than React, Angular, Preact, etc.)
But that doesn't mean a project has to go that far in abstraction. A project can have Javascript that's functional and behavioral, and leaves the component rendering to mostly HTML and CSS without the need for introducing a f/e JS library.
So, If the need calls for a f/e JS library, Vue is the route, but all projects utilizing JS don't necessarily need a f/e JS library to operate effectively.

I think putting up a guide which contains proper links for setting up docker on various os  and links that help with various github task such as opening up a PR... Cloning a repository and syncing it with the remote branch
Those where the most noticed beginner issues I noticed and had to aid some people with

Adding a detailed guide on setting up Docker will be very helpful. We could also add the possible errors that could spring up and include links that could help resolve those errors

A step by step guide for setting up docker most especially for windows with images will be very helpful as it will be easier to follow and better to understand.

A link to docker installation is already present in the readme.md…what we can really work on is stating out the possible errors, beginners might encounter while trying to set up the projects and proffer solutions to them. I would suggest this to be in a different folder

I think setting up a guide which explains how one can go about searching for issues to solve in the codebase and on the site will be of great help to new contributors and beginners.
Going through the contribution file, it explains how one can find issues from the issues list on the repo and how to create new ones but doesn't explain how you can search for them before creating. But in a case where all issues are taken or no issues available, it becomes abit confusing when you can't find one.
Having this guide will be of great help.

Suggestions on this discussion are awesome and would be beneficial to newcomers.
I think a guide that has a format for asking questions related to this project can be written as well or can be included in any of the above-mentioned guides as a postscript.

I think setting up a guide which explains how one can go about searching for issues to solve in the codebase and on the site will be of great help to new contributors and beginners. Going through the contribution file, it explains how one can find issues from the issues list on the repo and how to create new ones but doesn't explain how you can search for them before creating. But in a case where all issues are taken or no issues available, it becomes abit confusing when you can't find one. Having this guide will be of great help.

This is more a general guide that already exists in some part on the Contribution Guidelines. If anything there might be a need for that site to be modified or updated to address some specific concerns. But Guides here should be specific to this project, and while the above suggestion is useful, it's more general in application.

I agree that a Docker specific installation guide for Windows is likely useful, specifically for this project.

A Frequently Asked Question (FAQ) can also be considered a guide. I am not sure about Slack but I have seen questions being asked on a Discord community being turned into a FAQ channel where people can go to search for questions or ask questions related to the project if they do not find the answers they are looking for. Like a way of documenting relevant questions so that there isn't a repetition of messages. Almost like a mini stack overflow, but for CC search 😁

Choose a project:You can find open source projects on websites such as GitHub, GitLab, orBitbucket.
Familiarize yourself with the project:You can read the documentation and code base directly on the project'swebsite.
Find an issue to work on:You can find open issues by visiting the project's issue tracker.
Create a fork:Navigate to the project's repository and click the "Fork" button.
Make changes:Clone the forked repository to your local machine:
git clone https://github.com//.git
Change to the cloned directory:
bash
cd
Create a new branch to make your changes:
css
git checkout -b
Make changes to the code.

Fill out the pull request form and submit it.
Collaborate with the community:You can participate in discussions and help others through the project's issuetracker or community forums.

Note: The exact commands may vary depending on the version control system used bythe project (e.g. Git, Mercurial, etc.), the website hosting the project (e.g. GitHub, GitLab,Bitbucket, etc.), and the operating system you are using.6.


⚠ Bofrost.at has degraded performance

In 7259594, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 866 ms

Resolved: Bofrost.at performance has improved in 42c802b.


cos functions for #4555

For  issue #4555:

[x] Update Expression.js - Javascript and GLSL interpreters.

updated: Node.prototype.getShaderExpression
updated: function parseCall(expression, ast)
updated: function setEvaluateFunction(node)
added: Node.prototype._evaluateCosine

[x] Add tests to ExpressionSpec.js
[x]  Add a style to 3D Tiles Point Cloud Styling.html that uses the cos function to help visualize that things work.
[ ] screenshot taken / tested
[ ] updated 3d-tiles repo specs

Had to remove a secondary test case, but could add  more again once Math.PI supported in styling.

Here's a screenshot of the cos styling being used, just subtracted temperature by 90

Update: tests pass, it works. Thank god.

Heh so to explain https://github.com/AnalyticalGraphicsInc/cesium/pull/4578/commits/e1ba8d2fe55259b726c8be4352ceac1ba0b5fd6e...
GLSL shaders in WebGL do not support implict type conversions from integers to floats so our shader generation code always converts integers to floats, which is why the resulting shader expression is cos(0.0).
By the way the line var expression = new Expression('cos(0)'); would also pass the test.
We do the int->float conversion here: https://github.com/AnalyticalGraphicsInc/cesium/blob/3d-tiles/Source/Scene/Expression.js#L1025

Looks good @Dylan-Brown.

Thanks again, @Dylan-Brown!  Your twitter fame awaits: https://twitter.com/CesiumJS/status/794542226185535488


Authentication process public or private

I try to handle OAuth2 for private and public.
Into the doc I see that the public method is recommended for the app or for browser using javascript. This is ok for me. I see only a little difference when I auth without secret I got only the access token instead, when I use the secret into the header I got also the refresh token.
Now how I handle the refresh token for a public users?

You cant have a refresh token for public clients. This is potentially unsafe ans can be harmful to your users. Imagine the token is stolen, then without client secret any malicious actor could re-generate tokens indefinitely.
If you want to refresh the token, you need to implement authentication server side.


[UI] Review Summary - Animated Percentage Bar Headings

Perhaps something like [https://tympanus.net/Development/BlockRevealers/](Block Revealers)
Otherwise just the inner bar width animating.

meh


fixed icons

now it is working

Resolves #20
I fixed the Icons on the Server. I had to change some things in /etc/apache2/mods-available/alias.conf


storagetype switching

Discovered the problem with setStorageType, and think this will fix it.
Atleast my local js file is now switching properly using the local auth methods.
Building with ts became a hassle, so ive just pushed the ts file so you can do a build, if pull is accepted.
I've changed the "remove" method aswell to always delete the key in local aswell as session storage, and not just the currently selected.
I have not commited that though, please tell me if its a behaviour you want and ill to another pull request for it.

Thanks for the fix. I will build it on my machine and deploy a new version on NPM.
Could you provide a scenario for why one would want to remove a token from both local and session storage?

Great!
Nevermind the local/session removal, if my angular controller is behaving as expected i should never have a key in more than one place at any time.


Command+click and alt+click shortcuts are switched

Normally in my workspace:

cmd-click: goes to declaration
alt-click: creates a new cursor
For some reason, these two commands got switched yesterday (2019/08/01).
All the other shortcuts with commands and alts still work as expected.
Yes, I've made sure I'm pressing the right button 😁

VSCode Version: 1.36.1
OS Version: Mac OS 10.14.5

Steps to Reproduce:

Pressing cmd and hovering over a method does nothing (should make it look like a hyperlink and show a preview of the declaration)
Pressing cmd+click creates a new cursor
Pressing alt and hover over a method changes the method name style to a hyperlink and shows a preview of the declaration (should not do anything)
Pressing alt+click goes to declaration.

Does this issue occur when all extensions are disabled?: Yes

I can't reproduce this. If you open Settings and search for multi cursor modifier, what's that setting's value?


Readme/update linux part

Explain your changes:
update the readme-dev.md for linux dev environment setup
Explain how you tested your changes:
*
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #13180

!ci-build-me

!approved-for-mainnet

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet


fix: copy the unsafe row otherwise the queue will have all the same rows

The unsafe row use the same referrence of rows.
When we offer rows to queue, all the rows in queu will be same.

Copy the internal row to avoid this issue.

As discussed offline, the java/spark/src/main/java/com/lancedb/lance/spark/write/LanceArrowWriter.java is for zero copy write to Lance.
Thanks for the fix, could we convert from Spark Type to Arrow type and store the Arrow Type into the Queue directly

OK, I will refactor the queue later. But I think it's a fast fix for use.

Close this PR since this ISSUE will be fixed in #3110


Bug with mutations

Mutations are causing the Genome::_set_quick_access_arrays to mislabel some genes. Tests aren't catching this.
This is bad.

This was a doozy but was fixed in 6632c77.
Turns out inheriting the Gene object itself was a bad idea. Any mutation done to it affected every being that inherited that gene. Thus, prior to mutation it would be labeled appropriately for a spawned being but then, after the mutation, this would change the behavior of that gene but not the labeling done by the beings.
Changed the inheritance system to pass the gene_strings and reinstantiate genes on Being creation.


Uninteractive

This is a relatively simple, one might even say hacky, implementation of a non-interactive version of the schedule using a config value and a slimmed-down schedule filtering component.  Requested by people at Boskone who are not me.

I have one up at schedule.boskone.org/still/.  (The "Invalid Date" at the bottom is caused by bad data still in Grenadine, not my changes.)

Yes.  The regular site is still at schedule.boskone.org.  The PR doesn't permit serving both interactive and non-interactive at once, though presumably coding it that way would be very similar to what I did.
For one thing, I didn't want to be loading the schedule twice.  Lazy loading would make that less of a concern, but the audience for this is disjoint from the app-appreciative audience so there's isn't much to gain running both at once.  They do both read the same data files (refreshed sometimes from a cron job now, which is why I'm stuck with whatever bum data is in Grenadine at the moment).
At some point someone may copy the HTML from the static version into our main website and stop using the React version.  We shall see.

A more likely use case would be to put up a less interactive version of the schedule before turning on full interaction, in case your schedule is preliminary or incomplete in some way.  Some more fine-grained settings for non-interactivity might be useful, too.


TCG-2460 - Prevent cache stampeding

What does this PR do?
TCG-2460
Checklist

[  ] Pull request contains a clear definition of changes
[  ] Tests (either unit, integration, or acceptance) written and passing
[  ] Relevant documentation produced and/or updated

In my opinion the functionality you are adding should not be in the Predis class, but instead be part of a custom ClientInterface instance.

Coverage remained the same at 100.0% when pulling f7a7a216d385f44b03f5ee42d16a5623e8e048e1 on guywithnose:master into 0e8a29e372bb30255b93a09d704228741581a930 on traderinteractive:master.

Coverage remained the same at 100.0% when pulling f7a7a216d385f44b03f5ee42d16a5623e8e048e1 on guywithnose:master into 0e8a29e372bb30255b93a09d704228741581a930 on traderinteractive:master.

I pulled out the locking part, because as @chadicus pointed out that may work better at a different abstraction level.  The part that measures runtime still needs to be here, so I left that.

@chadicus Here is where I moved the locking part: https://github.com/traderinteractive/lib-predis-anti-stampede-php/pull/1

I set refreshPercent to default to 0.  That should make it fully BC


Ensure CF roles are stored for users with over 100 roles

For each cf we get the connected user's roles (for example space developer)
Sometimes the number of roles can exceed a single page (for example over 100 space developer roles)
We were not persisting anything over the first page

fixes https://github.com/SUSE/stratos/issues/549

All e2e tests pass locally except three, all of which I think are env based.

Two relating to number of buttons shown in app autoscaler tab (expected 2, got 3). I think this is due to a newer autoscaler exposing the credential management feature that was not in the version of autoscaler deployed to eng cloud
One relating to a missmatch in a deployed app's stack. Think this is also fine due to a different default stack in my cf than to what the e2e test is expecting.

Approved by Neil, merging


fixing some NANs in hair

eta_p can become infinite if cos_theta_o is zero (ie div-by-zero)
an alternate fix could be to absorb+return like it does elsewhere in the code
eg
if (!math::isfinite(eta_p))
{
absorb(data);
return;
}

Hi, thanks for your pull request.
We have made a fix for the issue you reported (and a similar one), which will be included in the next release of the MDL-SDK.


Is it possible to use dynamic keyspace?

Hi,
I've a multi-tenant application. I want to read messages from a single Kafka topic and split messages across different scylladb keyspaces based on a field inside the Kafka message.
Does the connector support this? If not, would you consider supporting it?

Currently, it is not supported.
As a workaround, assuming the number of destination keyspaces is small and fixed, you could launch N instances of Sink Connector (each one writing to a specific keyspace) and use a Filter (Confluent version) SMT transformation on each connector instance to filter the messages to a correct keyspace.
We will consider adding this feature in the future.


Why WAPI.sendMessageToID always return false?

I tried to send message to<EMAIL_ADDRESS>using WAPI.sendMessageToID and it was sent. Surprisingly, it return false even when the message was successfully sent.

I tried to send message to<EMAIL_ADDRESS>using WAPI.sendMessageToID and it was sent. Surprisingly, it return false even when the message was successfully sent.

Try to replace wapi.js for the new in my recent pull request.

It's the same. Still return false

same issue I am facing after replace wapi.js file

any have this solved?

nope

see #671


GH Pages: Style / Blank Table Cells

From @jesseilev on January 5, 2016 22:55
The tables don't look right.
Possibly other little things?
Copied from original issue: Launch-Code/cs50x-live-2016#123

From @digshake on January 6, 2016 22:41
Tables and bullets work, but blank table cells aren't displayed so I'm going to leave this issue open as a reminder to fix them.

solution: fork the couscous template and modify the css


Request SettingsButton

Hello, thank you for the greats work.
May I know are you planning to add SettingsButton which introduced by Bot Api 7.0 (https://core.telegram.org/bots/webapps#december-29-2023)

Hi @clement128 ! Thank you for the issue! Yes, I am going to add this

I'm waiting for this feature too. Thank you for the project!


NETOBSERV-1111: Add support for RoundTripTime in FLP.

Related PR - https://github.com/netobserv/netobserv-ebpf-agent/pull/117
Since the above pull request makes changes in the ebpf agent proto buf which it exports, this PR is marked as breaking changes as we need same version of agent and flp to work together.

/ok-to-test

@KalmanMeth @eranra  do we need to hold these changes till the PR:https://github.com/netobserv/netobserv-ebpf-agent/pull/117 gets merged?

@KalmanMeth @eranra do we need to hold these changes till the PR:netobserv/netobserv-ebpf-agent#117 gets merged?

@dushyantbehl Yes, I think we need to first merge the ebpf PR and then the FLP PR.

/ok-to-test

Rebased. Please take a look @eranra @jpinsonneau @KalmanMeth

Thanks @jpinsonneau!!

@jpinsonneau @eranra @KalmanMeth
Can one of you please approve these changes as the PR https://github.com/netobserv/netobserv-ebpf-agent/pull/117 has been merged 😄

@jpinsonneau @eranra @KalmanMeth Can one of you please approve these changes as the PR netobserv/netobserv-ebpf-agent#117 has been merged 😄

@dushyantbehl before approving this you need to update your ebpf agent dependency in go.mod

/ok-to-test

thanks @dushyantbehl !
/lgtm
/approve


Implementation of ArrayList in Java

Basic Implementation of ArrayList in Java.

I want to contribute to this issue

Ok, sure


Update services-overview.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6563e8a21fd77757ae12aa4f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6563e9c0fdb1c355d57819aa--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6563e9ca09319d562140aaf4--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6563ea06d6402959d581ab67--harness-developer.netlify.app


Issue75 solution

Replaced Find-MissingRecommendations with Export-MissingRecommendations
Updated files names for the error exports where VScode found them
Solution for https://github.com/techservicesillinois/SecOps-Powershell-CISDSC/issues/75

comment

added


feat: Add RHEL9 support

Add RHEL9 kickstart for bastion host (#160).
Support for openvpn must be removed in an additional PR, because the robertdebock.openvpn packages can not be installed on RHEL9.

@mohammedzee1000 @AmadeusPodvratnik
Here is a patch with RHEL8 and RHEL9, if you like it more. Use it.

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


Add Kodiaq 2019 fingerprint

Add Kodiaq 2019 fingerprint

Thank you for the PR! However, you actually don't need to send this to both me and comma, sending directly to comma is sufficient since that's where the change ultimately needs to happen. I'm watching your comma upstream PR and I'll help get it merged there if help is needed.


Format alternate nested arrays without dash

Will improve our xerrors formatting a lot.
Though tbh it’d seem really bizarre to not have fields for loc and whatnot cause the number of fields is fixed and it’s semantically a map.

So
  -
    - a
    - b
    - c
  -
    - a
    - b
    - c

would become
  - a
    b
    c
  - a
    b
    c

TBH that's a lot harder to read than keeping them on separate lines.

I think the following is best:
  - - a
    - b
    - c
  - - a
    - b
    - c

Nvm, that's horrendous too. I'm gonna leave it as is for now.


tour: add a link to issue tracker

I propose that we add a link to golang.org/issue somewhere on the
Go tour,  so that each translated version can customize that link and
make the issue reports go to the correct place.

Created change https://go-review.googlesource.com/8312, please review

If I'm reading this right, then https://go-review.googlesource.com/#/c/8312/ was replaced with https://go-review.googlesource.com/#/c/8904/ -- which was merged... if that's correct, then this can be closed.

Correct, thanks.


Use of undeclared identifier AFNetworkReachabilityManager with additional Watch OS target in project

Hi, I'm importing <AFNetworking/AFNetworking.h> to my class and trying to use [AFNetworkReachabilityManager sharedManager]. But I always getting "Use of undeclared identifier" error. I think, that it is because my iOS project contains additional WatchOS target. In AFNetworking.h I've found this:
#if !TARGET_OS_WATCH
    #import "AFNetworkReachabilityManager.h"
#endif

My podfile looks like:
use_frameworks!
inhibit_all_warnings!

def shared_pods
      pod 'AFNetworking'
end

target 'Find My Kids' do
     platform :ios, '9.0'
     shared_pods
end

target 'Find My Kids Watch Extention' do
     platform :watchos, '2.0'
     shared_pods
end

If I'm right, how can I organize my podfile or project in order to get access to ReachabilityManager in iOS target? Thank you!

Are you trying to use the reachability manager from your watchOS code? It’s not available on watchOS, but that configuration should work on iOS.

Thank you for replying! The problem was exactly there. I was trying to use the reachability manager from code, that compiles for watchOS also.


Clean up Gaussian noise

In GitLab by @gregory.ashton on Mar 29, 2018, 01:14

In GitLab by @gregory.ashton on Mar 29, 2018, 23:41
@nikhil.sarin You did this yesterday right? Can you close this?

In GitLab by @nikhil.sarin on Mar 30, 2018, 00:32
closed


Privacy policy and term of use can have different pages

I can create privacy and term of use page  please assign to me.

@devnullone can we discuss about content for this two pages


admin token is added to public runtime config

Hi. Checking code, apiKey is directly added to public runtime config (https://github.com/Baroshem/nuxt-medusa/blob/ab120ac4ad45f1af35792a4bfd28fa76e727c24c/src/module.ts#L25) which i assume is an admin token. While client js supports it, isn't it with editing permissions?

Hey Pooya, thanks for noticing that!
I will take a look at it and probably add it to the private config instead :)

I've created a PR for this issue.
Please, check out my code: #10

Should we use privateRuntimeConfig to solve this issue?

Hey,
You wiuld have to check because private runtime config is not accessible from the frontend usually. Maybe we would need to inform users that if they want to use apiKey they have to use the server utility.
But medusa client normally accepts apiKey so it would be against it. I have to think about it.
I would say that for now it is not a security problem but I will take a look at it in a few days for the next version.
Let me know about your opinion on this :)

Hey,
You wiuld have to check because private runtime config is not accessible from the frontend usually. Maybe we would need to inform users that if they want to use apiKey they have to use the server utility.
But medusa client normally accepts apiKey so it would be against it. I have to think about it.
I would say that for now it is not a security problem but I will take a look at it in a few days for the next version.
Let me know about your opinion on this :)

Thank you for more knowledge. I got more points from this.
I can't figure out another way without storing the API key in the server, using their utility, if you don't want it exposed.
If you have a good solution, please let me know, really want to learn from your project. 😂
Thank you so much. 😁

@runyasak
Sorry for no contact on my side. I was quite busy with other tasks this week.
Coming back to the topic. I would recommend to implement an approach where apiKey is only to runtime config private and accessible by the server utility or plugin (server only).
This might be different approach from what Medusa client is delivering but I would prefer to deliver users a safer approach (while apiKey will still be accessible in the server side hidden from the frontend code)

Hey do you mean this?
https://nuxt-medusa-tailwind.vercel.app/

I also have the module docs deployed here -> https://nuxt-medusa.vercel.app/. Not sure what do you mean by that @pi0

Ah, so I dont have that.
I think we can actually remove it from the runtime config and just add it to private for usage in server utility where this will be hidden.
What do you think @pi0 ?

That would be nice to be done (this seems an important issue not exposing adding token!)

Any update on this?

I was hoping for some community contribution by @runyasak 😉
If you wont answer in the upcoming days, I will develop this feature
@pi0

@Baroshem  Sorry, for my late response. And thank you for waiting me.
As far as I know, private runtime config would be a good choice for this issue.
I will contribute soon by today.
Thank you so much again. 😁

@Baroshem I've submitted a new PR for private config.
I'm not sure that it would be your expectation or not. Please check out and review my new PR: #17 😁

Released with 0.4.0 :)
Thanks for the contribution!


Add XarrayDataArrayEqualityOperator

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (a5f21ed) 100.00% compared to head (934bfe0) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #107   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          391       403   +12
  Branches        97       100    +3
=========================================
+ Hits           391       403   +12

Impacted Files
Coverage Δ

src/coola/xr.py
100.00% <100.00%> (ø)

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


Increased snapshot verification accuracy

Discovered that Badge test still passes if I change text from 12 to 1ą or even 11ą.
It only fails if I change it to 111ą.
Played a bit with snapshot accuracy and from 0.996 and up the Button test starts to fail because of minor text shifts in multiple previews. With 0.995 Button test starts to pass and Badge test fails if text is changed to 11ą (still passes if changed to 1ą).
So keeping the 0.995 setting and this means tests may miss a change of one character.

@justasm yeah, we did! Just pushed parameterized version, unfortunately Badge also needed precision <1, but was OK with stricter precision than Button. It seems we will need to see with time how different precision we'll need for different components.


Switch from browser to rendering engines detection [was: Opera Blink is recognized as Chrome]

Follow-up the discussion about the future of Opera here:
https://groups.google.com/forum/#!searchin/mathjax-dev/future/mathjax-dev/pvJTgrhwB-M/UJNjdHQG0FAJ
Opera will use Blink in future versions, so all the Opera hacks should be replaced by Chrome hacks. The good news is that the MathJax sniffing detects Opera Blink as Chrome, so this is done automatically.
However, this means that things like MathJax.Hub.Browser or the about box now give the wrong information.
As mentioned in the Gecko sniffing issue, it could be better to have a rendering engine detection (Gecko, Blink, WebKit, Trident) that would be used for the rendering hacks. Probably the MathJax.Hub and about box should keep the browser detection.

This is no longer relevant in v3, which doesn't do browser detection of any kind.


Uses latest os version when finding device udid for some commands ...

run prior to snapshotting
Additionally adds error message if simulator can not be found.
Fixes #6097

Hi @WolfgangD - thanks for the report 👍
We also received #6058 at around the same time, and we have produced #6204 to address it. I believe that PR addresses your situation as well, but it does it in the opposite sort of way (printing a warning, but being more permissive - rather than immediately halting).
This additional "smarts" and convenience felt like the fastlane way to handle it. If this doesn't help you out, can you let us know?
Thanks!


Saving with title as name overwrites files when multiple images have same title

Implementing a simple check whether such a file already exists and appending (1), (2), or whatever number necessary would avoid this, allowing all images to be downloaded instead of some being overwritten.

In my fork I added a fix appending a sequential number if the image already exists.

(close the issue)


Disable ansys corba warning

Need to disable the user warning and raise an exception when trying to create an interactive APDL interface on Windows.  This will make Mac OS support cleaner.
https://github.com/akaszynski/ansys_corba/issues/1

Done.  Will now raise an exception when APDL connection with corba is attempted without a compatible OS.


Request to provision Microsoft 365 Learning Pathways template failed

The email told me to report the issue here, correclation ID 8962394f-098c-492f-a417-65a69c2b6745
Please help!
Lorene

Hi, Thanks for your feedback. I'm closing this item because it is now outdated.


Add typescript definitions on DefinitelyTyped

Hi @mpetroff. I have added pannellum types on DefinitelyTyped and would like you to review them. If any changes are needed, I'd be happy to patch things up.

I don't use TypeScript, but I'm not sure you properly captured the panorama option when the dynamic option is set to true. In this case, the panorama option accepts anything that the pixels argument to texImage2D  accepts. It looks to me that you're forcing the panorama option to be a string.


The printer returned an unexpected response: 'Error:File is not existed or empty, file "XYZ"

So I'm not sure what causes this, but I've ran into it 3 times now, where I upload a file, I can open the file in preview panel it renders, etc., but then I select to print it and I get an exception.
Traceback (most recent call last):
File "/opt/venvs/mariner3d/lib/python3.7/site-packages/flask/app.py", line 1950, in full_dispatch_request
rv = self.dispatch_request()
File "/opt/venvs/mariner3d/lib/python3.7/site-packages/flask/app.py", line 1936, in dispatch_request
return self.view_functionsrule.endpoint
File "/opt/venvs/mariner3d/lib/python3.7/site-packages/mariner/server/api.py", line 208, in printer_command
elegoo_mars.start_printing(filename)
File "/opt/venvs/mariner3d/lib/python3.7/site-packages/mariner/mars.py", line 140, in start_printing
raise UnexpectedPrinterResponse(response)
mariner.exceptions.UnexpectedPrinterResponse: Error:File is not existed or empty, file /crossbows2.ctb
It seems like if I reload the Mariner web interface it starts to work, but until than I start to continually get this error for any files uploaded and attempts to issue a print command.

It's probably not reloading that fixed the issue for you, but waiting until the printer "sees" the new file.
Which printer are you using?

Sonic Mini 4k and Sonic Mighty 4k

The solution we're discussing on #311 will help with this, so going to close this.


Convert stacks to string

Implement proc `$`(stack: MultiStack[T]): string, which is helpful for debugging.

Stacks should not export pushed values except its top, IMO.


[Challenge descriptions] - Move punctuation inside code blocks/tags?

At certain widths, punctuation next to a code block gets pushed to its own line because it treats it like there's a space there, or like it's it's own element or something....

A few more examples...

I'd say that's because to HTML, those two are different things. I don't really think that there's a workaround other than wrapping them up in a wrapper and making its width fit-content.

I was thinking to change it to something like 1fr. instead of 1fr. in the descriptions.

Hi @moT01, Let me know whether It needs to be changed or not. I would like to work on this.

Keep your eye on the conversation here. This would pretty much touch every challenge file across all languages, and might be more of a hassle than it's worth.

That would make it look like 1fr. and cause widespread confusion, not really a good idea IMO

Good point @thecodingaviator, that's probably not the way to go - something like you suggested then is probably best. Maybe just wrapping them in a span tag - 1fr.

We'll also need to make the span have width: fit-content so it does not snap to the next line

I think this is primarily a css issue.  The code elements are styled with
code:after {
    letter-spacing: -.2em;
    content: " ";
}

This puts a breaking space after the code block allowing punctuation to dangle.  Replacing the content with a non-breaking space stops that.  The following should solve the problem:
code:after {
    letter-spacing: -.2em;
    content: "\00a0";
}


Add a Heroku Button

Cool... but I think this should also accept some configuration options for appId, masterKey, etc.

@gfosco Should we just support all the config?

@gfosco I've added support for APP_ID, MASTER_KEY, and REST_API_KEY .. any others worth dropping in?

great news guys that you are on it.  I would give an example after its all set up how to make a test call and see example data returned or some sample test scripts..
Another thing would be to run benchmarks on the various server sizes and maybe use Jmeter/Blazemeter to define a request per second sort of baseline.  A lot of the people are used to how parse did their billing with the slider and would want to know if I have this server this is what I can expect the throughput to be since many people never had to build their own backend.
big missing link though is push notifications..

Oh, another thing is how are files stored, I believe parse saved their files in s3 buckets and linked them in the db.. Looking at their migration docs they have this in

fileKey: A key that specifies a prefix used for file storage. (only necessary for old files in s3 clarify this)

Sorry and lastly we prob need more keys to set up see their example listed here.
var api = new ParseServer({   databaseURI: 'mongodb://your.mongo.uri',   cloud: './cloud/main.js',   appId: 'myAppId',   fileKey: 'myFileKey',   masterKey: 'mySecretMasterKey',   clientKey: 'myClientKey',   restAPIKey: 'myRESTAPIKey',   javascriptKey: 'myJavascriptKey',   dotNetKey: 'myDotNetKey', });

databaseURI: Connection string URI for your MongoDB.
cloud: Path to your app’s Cloud Code.
appId: A unique identifier for your app.
fileKey: A key that specifies a prefix used for file storage. (only necessary for old files in s3 clarify this)
masterKey: A key that overrides all permissions. Keep this secret.
clientKey: The client key for your app.
restAPIKey: The REST API key for your app.
javascriptKey: The JavaScript key for your app.
dotNetKey: The .NET key for your app.

so it would be good to be able to fill these out upon provisioning and be able to change them, esp the master key because if someone hacks an app they can do a lot of damage and you would want to be able to easily change this value.

@gateway Thanks for your feedback! I've added what I believe are the core config vars -- happy to add more. You make good points re. Push and S3, I could also use some clarity there, perhaps open an issue?
Also - The express-server-example does include a sample request, you'd just need to update the endpoint to reflect *.herokuapp.com.

@gateway oh and, updating any of the Config Vars would be trivial -- just visit your app on Heroku.. settings > config vars .. edit away -- the nice thing about that being that it will generate a new release for you application, so if you make a mistake you can quickly roll-back.

Looks like they use gridstore for saving images to mongo.. I could of swore I saw aws links in our images but never the less here is the code
https://github.com/ParsePlatform/parse-server/blob/master/GridStoreAdapter.js
filesAdapter - The default behavior (GridStore) can be changed by creating an adapter class (see FilesAdapter.js)
databaseAdapter (unfinished) - The backing store can be changed by creating an adapter class (see DatabaseAdapter.js)

Oh, another point is which has become much more difficult now is to deal with class level permissions.
Parse server does not include a web-based dashboard, which is where class-level permissions have always been configured. If you migrate an app from Parse, you'll see the format for CLPs in the SCHEMA collection. There is also a setPermissions method on the Schema class, which you can see used in the unit-tests in Schema.spec.js You can also set up an app on Parse, providing the connection string for your mongo database, and continue to use the dashboard on Parse.com.
Here is how they have it in their interface.  I'm not suggesting you guys build this out fully but a lot of questions will crop up on how to do this.
http://blog.parse.com/learn/secure-your-app-one-class-at-a-time/
This is pretty important because you dont want some roles/users to be able to change values and such..

As far as push notifications I think this company has been around the longest.. https://www.urbanairship.com/products/engage I personally never used it and we rolled our own on our last app.
Amazon has SNS but it just takes the device tokens that are imported into their system and then allows you to connect via an api to push our messages to users.. https://aws.amazon.com/sns/  this is the part to push whats missing is the server/service that triggers a push and builds custom or tokenized messages to users.
Here is an open source one running node.js I have never tested it.. https://aerogear.org/docs/specs/aerogear-unifiedpush-nodejs-client/
Also note at least for iOS in the Parse iOS SDK you can enable push and if the user accepts that, it saves the apple device token to the installation class in the mongo database.  Ideally these could be migrated into SNS or some other service how ever triggering what types of pushes you want is more difficult.  Its probably worth just logging into parse and seeing how they set up their system but I believe right now this is out of the scope of dev and would require quite a bit of code along with api endpoint and the service to connect to apple's or googles server to push out notifications.

@gfosco, @gateway I've dropped in lodash, and made all of the configuration (I think) available via. config vars.. but not via the Heroku Button deploy -- I've kept that simple, and quick.. It's so easy to add config vars after it's been deployed I don't think it's worth making the form a mile long.

The permission per class is pretty important, maybe someone will build a front end UI for this..
Any chance I could test it and provide feedback?
username<EMAIL_ADDRESS>

@mattcreager I get the intent behind adding Lodash and using the defaults, but I think it really bloats the sample and makes it harder to understand at a glance... Can you remove that and continue the initialization method originally used?

@gfosco sorry, got carried away :)

i just used the FANTASTIC Heroku button and I now have a working parse server. @mattcreager unreal work. I noticed that the server that was deployed was several commits behind. I'm a little confused how the migration works. Should I wait until you guys get all the kinks worked out and deploy this button on the main branch. Or can i keep the deployment I have now and just update the code later?  (sorry, for the foggy question.)

Can we test deploy this yet?

Woo! Thanks @mattcreager ! :+1:

Just fyi, they pushed a lot of changes today, exp being able to save your pffiles to s3 bucket which I prefer to have over keeping them in a database.. Im gonna test out the latest version .. cheers

Hey Guys I did some simple benchmarking today with a Jmeter script , just does a POST and a GET of the object created and I also used apache benchmark... From the looks of the data I'm seeing pretty high response times from the results.. I'm not sure if its the connection between you guys and MongoLab or the small free dyno im running.. here are some of my findings.. Any thoughts?  https://groups.google.com/forum/#!topic/parse-developers/puldzNyFEGs


Unable to cast vote on Sovereign testnet

This is a Bug Report
:tophat: Description
For bug reports:

What went wrong? When I cast a vote, MetaMask sends a message that transaction is complete, but the votes are not showing up where I am staking them, the process gets arrested, I just see a spinning 'wait' icon. I am able to make a post no problem, just cannot vote.
What did you expect should have happened? votes I cast in my MetaMask wallet to show up staked on the proposal that I staked them on.
What was the config you used? I tried to vote from both Chrome and Brave. I restarted the browsers and the app, and my MacBook Air.
What stacktrace or error message from your provider did you see?

For feature proposals:

What is the use case that should be solved. The more detail you describe this in the easier it is to understand for us.
If there is additional config how would it look

:pushpin: Related issues

#010101

:clipboard: Additional Data

Sovereign deployment where you found the issue: https://votest.democracy.earth/
Browser & version:  Chrome Version 72.0.3626.109,  Brave Version 0.59.34 Chromium: 72.0.3626.81 (Official Build) (64-bit). On my
Screenshot:

Error messages:
URL to reproduce the error: https://votest.democracy.earth/

per conversation on Signal!

@SFSandra this should be fixed now, seems it was an issue with Infura:
https://github.com/ethereum/web3.js/issues/1679#issuecomment-413234902
Can you confirm from your end and close this issue if so?

@santisiri we should think about what can we do to have Sovereign handle a null from getTransaction` more gracefully


Support aggregators: min(), max(), avg(), sum()

https://www.sqlite.org/lang_aggfunc.html

Closed by https://github.com/gfx/Android-Orma/pull/323


Need to understand why ACIS is rounding PRISM fahrenheit values

I am puzzled why they aren't vending the temperature grids to the tenths of a degree F.

I figured it out - need to specify 'prec':1 in elem


Curse tactic

Hi again!
Did you think anything about tactic with cursed tile?
Maybe I can help with something?

I looked a bit at adding this here,
https://github.com/AxelBjork/Rush-Royale-Bot/tree/Curse_dodge
However I'm not sure how well it will work.

I think better, than before, thanks


The mobile nav items are different than the normal nav items

Description
There's some strange behavior where the sidebar contents are rendered for the navigation in mobile view. This is not expected behavior and it should instead render the same menu defined for the site.
Steps To Reproduce
Just create a normal page with a title, and the title will be used for the mobile name, while the nav item name is used for the desktop nav.
Expected Behavior
The mobile nav should display the nav .Name as defined in the hugo.yaml file, not the page's title.
Actual Behavior
It shows the title of the page in mobile view, and .Name in desktop view. This makes it impractical to assign a title to a page as it has to be the same thing as the .Name in order to avoid confusing behavior.

see also #449


1.6.7 can't be run on Ubuntu LTS 20.04 (GLIBC_2.34 not found)

Description
Hey guys,
I've been unable to run the latest containerd (1.6.7) on my Ubuntu 20.04 fleet, but 1.6.6 works fine. The following illustrates the problem:
1.6.6:
root@mn1:/tmp# ./1.6.6/bin/containerd --version
containerd github.com/containerd/containerd v1.6.6 10c12954828e7c7c9b6e0ea9b0c02b01407d3ae1
root@mn1:/tmp#

1.6.7:
./1.6.7/bin/containerd --version
./1.6.7/bin/containerd: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by ./1.6.7/bin/containerd)
./1.6.7/bin/containerd: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by ./1.6.7/bin/containerd)
root@mn1:/tmp#

Steps to reproduce the issue

Download and uncompress the latest 1.6.7 from https://github.com/containerd/containerd/releases/download/v1.6.7/containerd-1.6.7-linux-amd64.tar.gz
Attempt to run ./bin/containerd --version on Ubuntu 20.04 environment
Observe error

Describe the results you received and expected
I should be able to upgrade from 1.6.6 to 1.6.7 without issue
What version of containerd are you using?
1.6.7
Any other relevant information
root@mn1:/tmp# uname -a
Linux cn1 5.13.11-051311-generic #202108151332 SMP Sun Aug 15 13:57:01 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Show configuration if it is related to CRI plugin.
No response

Observing the same issue on Ubuntu 20.04

Those binaries were probably built on a newer version of ubuntu, and dynamically linked. As a workaround, you can probably use the binary that's included in the docker.io packages (they've not yet been published to download.docker.com, but are on the staging server currently; https://download-stage.docker.com/linux/ubuntu/dists/focal/pool/stable/amd64/)

It seems it also affects users of https://github.com/crazy-max/ghaction-setup-containerd: https://github.com/flexkube/libflexkube/runs/7687197619?check_suite_focus=true

/cc @crazy-max FYI ^^

@invidian Enforce 1.6.6 in the meantime:
      -
        name: Set up containerd
        uses: crazy-max/ghaction-setup-containerd@v2
        with:
          version: v1.6.6

Or switch to ubuntu-2204 in your workflow.

I have the same problem on CentOS 7

I have the same problem on CentOS 7

I have the same problem on CentOS 7

-- Unit containerd.service has begun starting up.
8月 05 21:45:13 localhost.localdomain containerd[47761]: /usr/local/bin/containerd: /lib64/libc.so.6: version `GLIBC_2.34' not found (required by /usr/local/bin/containerd)
8月 05 21:45:13 localhost.localdomain containerd[47761]: /usr/local/bin/containerd: /lib64/libc.so.6: version `GLIBC_2.32' not found (required by /usr/local/bin/containerd)
8月 05 21:45:13 localhost.localdomain systemd[1]: containerd.service: main process exited, code=exited, status=1/FAILURE
8月 05 21:45:13 localhost.localdomain systemd[1]: Failed to start containerd container runtime.
-- Subject: Unit containerd.service has failed
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit containerd.service has failed.
--
-- The result is failed.

[root@localhost ~]# uname -a
Linux localhost.localdomain 3.10.0-1160.el7.x86_64 #1 SMP Mon Oct 19 16:18:59 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Just want to confirm if this considered a bug or breaking change that not gonna be fixed? If it will be fixed, would that be repushing all the assets in 1.6.7 release or 1.6.8? Thx.

I don't think this is a bug in the source, only an issue with the binaries that were built.
This is a limitation of having a single binary to target all possible distros, instead of builds specific for each distribution.

Regression in https://github.com/containerd/containerd/commit/0e56e4f9ff2925d0755908f227f2b5fc9a1c4c78 🙇
I guess we have to rollback Ubuntu to 18.04 except for RISC-V 64 (because Ubuntu 18.04 lacks RISC-V 64 support)

Keeping open until releasing v1.6.8

v1.6.8 is now available


v0.1.10 not updated in changelog

A new version was recently pushed and not updated in the changelog or reflected in the README. Can a badge or other indicator be added to the README for the current version?

I noticed this too. I'll try to get it updated.


Tokenizer used for NER in Scipacy model : "en_ner_bionlp13cg_md"

Helllo,
I'm trying to cluster the embeddings of entities generated by my model. To train an embedding model I'll need the same tokenizer that my scispacy model uses. I tried using scispacy custom tokenizer but it does not give me the exact tokens. Could you please help me identify the tokenizer that my NER model uses to identify tokens?

Could you please provide the versions of scispacy and spacy in your environment, and a concrete example of what you are doing and where the tokenization differs? The tokenizer on all the scispacy models should be the same as the custom tokenizer in the repo.

scispacy==0.3.0
spacy.version == 2.3.5
model = en_ner_bionlp13cg_md-0.3.0
Code :
nlp = en_ner_bionlp13cg_md.load()
doc = nlp(' '.join(text))
Entities extracted:

Tokens:

Some of the 4-word entities aren't identified as tokens. Is it possible to get all the entities as tokens?

A couple things

a "token" is approximately a "word". Entities can be multiple words in general. The entity boundaries should line up with tokens though. So, "New York City" might be an entity, but composed of three tokens, "New", "York", and "City". If you want a vector for "New York City", one way to do this is to average the individual token vectors. I do not know if this would work well for your use case, but it is a common thing to do.
From the "Tokens" output, it looks like you are passing a long list of strings to the model. The models were trained on proper sentences, and may not work as well when applied to ungrammatical text.

Thanks for the informatiom.


Get average decibel level of an audio recording

How would I to find the average decibel level of a recording?

Really need this feature, anyone has a idea ?

sorry i have missed the doc :)
MeteringEnabled: true

But only for IOS, is it hard to make it available to Android ?

@richmonkey add the missing code for Android :)
forked

@stephanedemotte This helped me, thank you very much. :smile:

I have a similar request, except that I need the average decibal during onProgress - https://github.com/jsierles/react-native-audio/issues/179

Oops never mind, my request is exactly this. I am on Android. Is there any chance this merges to master? It would be sooo awesome! :)

I think it's a good feature for android, at least most of people got android phone in this world.

@stephanedemotte
react-native : 0.57.0
react :  16.5.0
using react-native-sound ? Yes
"react-native-audio": "^4.2.1",
Can not get the "currentMetering" from data in "onProgress" method for the android?
Is there a version issue?
Can you pls let me know about this?

Any update on this feature for Android?

Hi, when I tried to install to forked repository of @richmonkey i got this error i compilation :
react-native-audio\android\src\main\java\com\rnim\rn\audio\ReactNativeAudioPackage.java:35: error: method does not override or implement a method from a supertype     @Override
Any news about an officiel update of the package to support this feature ?

@AbdelZinir older react native version


[ansible] flag to disable logs

Hello,
Similar to the helm chart https://github.com/signalfx/splunk-otel-collector-chart/blob/01d60f316bc003c473b55f33dcfe19742a2a85c6/helm-charts/splunk-otel-collector/values.yaml#L58
we need a way to install the otel collector without enabling logging.
For now there is an option to disable fluentd installation (which does not uninstall it when we change it to false for your information).
But the splunk hec exporter is still configured which will lead to a fail:
Jun 10 13:04:17 lveulnx4053q otelcol[385619]: 2021-06-10T13:04:17.241Z#011info#011exporterhelper/queued_retry.go:314#011Exporting failed. Will retry the request after interval.#011{"kind": "exporter", "name": "splunk_hec", "error": "HTTP 503 \"Service Unavailable\"", "interval": "7.032856404s"}

we can use a custom configuration for otel collector but it seems not straightforward enough.
thanks

Does the error message shown still happen after you uninstalled fluentd? Don't think anything should be sending over logs otherwise.

the error is still shown after restart yes.

Can you figure out what is sending logs? You can enable the logging exporter in the logs pipeline to find out.

I use the default configuration: https://github.com/signalfx/splunk-otel-collector/blob/fcddf92080a818751913f412bd192fd11df8e9fa/cmd/otelcol/config/collector/agent_config.yaml#L149 deployed by ansible but I don't have more information with loglevel: debug.
In fact I got confused by using this role with different configurations for testing purpose. As the role does not handle fluentd except its installation, if you do not use it on a fresh installation you can misunderstand the behavior.
For example, if I set install_fluentd to false after already run the playbook with its value to true so it will do nothing which is correct : it does not install it (and and not uninstall it either).
So in true, when otel collector is started after fluentd is stopped I confirm the error is not here. if I just stop fluentd and let otel collector the error persists (probably until the buffer is empty).
so my bad everything is technically correct.
I would like to suggest to add feature to handle the state of the collector(s) in addition to the installation, for example enable_fluentd (or merging to a more global one).
Obviously, we should ideally use the role only on fresh installation to build an image and deploy it but adding capability to ensure of the final state of both collectors from ansible seems to be a good thing.
this will prevent similar confusion (here ansible will stop fluentd even on existing installation) and it could be related to the need to not start the agent after installation : https://github.com/signalfx/splunk-otel-collector/issues/460.
does it make sens ?

It can be hard to know whether we should remove fluentd since it's a 3rd party package that we may not have installed ourselves (ie was already on system that user installed so should we uninstall it?). Long term fluentd will be replaced with native OpenTelemetry logging. @dmitryax thoughts?

@jrcamp my suggestion is not about uninstalling fluentd but ensuring the splunk related configuration is not run (if enable_fluentd:false).

#298 will cover or at least enable that case.

Closing this issue for inactivity. Please reopen if further work is required.


Missing characters in output for 'Sentence_found' (part of engine.m_traces in genTrace.py)

Although this issue is about a Python script, it actually concerns something that needs to be changed in the engine.
Where PreprocessToken introduces a space within a token (thus creating 2 tokens), the last character of the first part is replaced by space in Sentence_found, e.g. PreprocessToken "It's" -> "It 's", Sentence_found "I 's".

This problem also appeared in the IRIS version, fixed now, see Issue#28


Support x-pci-vendor-id and x-pci-device-id pass to qemu

Close #97
As the issue says, in our environment, sometimes need to specified vendor-id and device-id.

@Ace-Tang two comments:

please update the unit test
you mean fixes and not close in your commit message, right?

Thanks for the commonts,

you mean fixes and not close in your commit message, right?

I am not sure the different between fix and close, if you like, I can change close to fix, I am not feel different :confused:

@Ace-Tang I think you are right https://help.github.com/en/articles/closing-issues-using-keywords, close is fine

Coverage increased (+0.01%) to 80.479% when pulling 514573c22f96f37373ef3b991fa1593ea22080d3 on Ace-Tang:add-pci-param into 9f389cb319af066f210b28dc8846d679878fe9f2 on intel:master.

@sboeuf, @amshinde Does this look okay to you?  You seem to be have added/last modified this code.

@sboeuf , I have updated the commit
since some vendor id like 1ded can not be identified by virtio-pci
driver, so upper level need to pass a specified vendor id to qemu.

the upper level will change unavailable id and pass it to qemu.


Harbor Disaster Recovery solutions

InCluster POSTGRESQL or SYNC_REGISTRY for S3?
For more information please read this issue https://github.com/goharbor/harbor/issues/11773

We go for backups of postgres for now
Should work through configuration of harbor-operator-app


Want to add a new program

Description
Want to add DEVELOP for GOOD progam in the list
Example  Fill x { [x] }  into  bracket

[ x] I have checked the existing issues *
[ x] I have read the Contributing Guidelines *
[ x] I have starred the repository *
[ x] Working under Gssoc24*

Could please explain bit more about the requirement and what needs to be done.

@anshgautam-github  can you explain me

@HimanshuNarware  Develop for Good powers social good through students from all colleges and backgrounds. They prioritize making its valuable, hands-on project experiences accessible to underserved and underrepresented students in tech to accelerate their careers. It strives to empower the next generation of diverse tech leaders to learn by doing good, all while furthering the digital transformation of the nonprofit sector. At its core, Develop for Good hopes to create a world where students, nonprofits, and corporations are dedicated to furthering technical education, advancing digital equity, and helping one another unlock their fullest potential.


improve docs of local Git repo configuration

Hello 👋🏻  I'm very excited about sveltia-cms 😄 I tested it a couple months ago with the local Git repository option and it is all looking wonderfully well done.
However, my deployment still required decap-cms and I reconfigured my dev environment. Since then I cannot get serving locally working again. I get this error:

My config.yml file works just fine with the decap npx netlify-cms-proxy-server so I am sure i'm just misunderstanding something.
Perhaps putting an example config or documenting where sveltia-cms differs from decap-cms would be helpful to me and others.
Anyway, thanks for your hard and really polished work here! Looking forward to future releases 😀

Thanks for checking Sveltia CMS! Well, your Decap CMS config file should be able to be reused without any modification. Can you please open Chrome DevTolls and take a screenshot of the Console tab? There would be some errors displayed there.

I just deployed version 0.3.3 to make sure errors are always logged in the console. Can you take a screenshot and post it here? Thanks! 🙏🏻

Thanks for enabling logging. Previously my console showed no error message. Now I see this:

Thanks for the screenshot! I think now understand the cause. Will test and deploy the fix.

Tested. This seems it was exactly the bug and makes sense given fresh state of my dev environment where I had successfully used the local feature. Thank you for the quick investigation + bug fix. Works great again 👌🏻 🥳


Warning when pod install when add lib .Can't merge user_target_xcconfig for pod targets. Singular build setting EXCLUDED_ARCHS[sdk=iphonesimulator*] has different values.

Can't merge user_target_xcconfig for pod targets: ["FaceDetection", "MLKitCore", "MLKitCommon", "MLKitFaceDetection", "MLKitVision", "Map4dMapUtils"]. Singular build setting EXCLUDED_ARCHS[sdk=iphonesimulator*] has different values.

Fixed on latest version

Fixed on latest version
??
What latest version?


buildGoModule: symlink vendor instead of copying

Go doesn't need to modify this and not having to copy it, increase build time quite a bit
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

still testing this on some packages.

@ofborg build k3s

Probably also should go through staging.

Telegraf is meaningless because it uses proxyVendor

@ofborg build coredns

@ofborg build ncdns


Add node discovery page to Elasticsearch section

We currently have no content on node discovery and cluster formation. What are they? Why are they so important to Elasticsearch? What settings govern them, and what options do they have? How does the cluster determine master-eligible nodes and elect a master? What are the other node types, and what do they do? How do you designate a node as being of a particular type?

This is partially addressed here, but we still want far more reference information on node types and settings.

I'd call this issue essentially addressed at this point.


Fix outstanding issues to color text correctly.

This should fix most of the issues that are out there. Javascript should be correctly targeted now.
A couple known issues that I am working on.
Need to get keyboard shortcuts working again. Especially for commenting out code.
If there is a /* comment block on the same line as an opening tag with no closing tag on the line. It will default to the entire block being commented. This is my way of making things work for now.
Works correctly
<% /* some commented out text */ %>

Does not work (Work around is place /* on next line.
<% /* Some commented out text

more commented out text

*/

foo();
%>

Note: Issues still exist within

Keyboard shortcut will work now if the begin or end tag is not on the line. Still working out getting the correct comment for single ejs lines.


feat: new product tour component

Thanks for the pull request, @ciduarte! I've created OSPR-6453 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.
:warning: We can't start reviewing your pull request until you've submitted a     signed contributor agreement     or indicated your institutional affiliation. Please see the CONTRIBUTING     file for more information.     If you've signed an agreement in the past, you may need to re-sign.  See     The New Home of the Open edX Codebase     for details.

OK, I think I've address all your comments (removed utility classes, renamed the id, and used global (shouldn't we be using globalThis instead?).
But I've also made a new PR (https://github.com/openedx/paragon/pull/1087) to work around the CLA bot, which doesn't know that Carla was an edX employee when she started this - so I've moved it to a PR I filed.

@ciduarte Even though your pull request wasn’t merged, please take a moment  to answer a two question survey so we can improve your experience in the future.


TeamsInfo.get_member returns  Operation returned an invalid status code 'Not Found'

Sample type: BotBuilder-Samples/samples/python/03.welcome-user/
Sample language: python
Sample name: welcome-user

Describe the bug
I wanted to get the user identity so, I am using the TeamsInfo class by following this documentation,
https://docs.microsoft.com/en-us/microsoftteams/platform/bots/how-to/get-teams-context?tabs=python#get-single-member-details
But it returns, Operation returned an invalid status code 'Not Found'
To Reproduce
Steps to reproduce the behavior:

Clone BotBuilder-Samples/samples/python/03.welcome-user/
Open welcome_user_bot.py
Add the below line under on_mesage_activity handler,
member = await TeamsInfo.get_member(turn_context, turn_context.activity.from_property.id)
print(member.name, member.id)
Run the app, open Bot Framework emulator and send a message

Expected behavior
Print the member details like id, name, email, AAD id, etc.
[bug]
clientActivityID is not a known attribute of class <class 'botbuilder.schema.teams._models_py3.TeamsChannelData'> and will be ignored
clientTimestamp is not a known attribute of class <class 'botbuilder.schema.teams._models_py3.TeamsChannelData'> and will be ignored

 [on_turn_error] unhandled error: Operation returned an invalid status code 'Not Found'
Traceback (most recent call last):
  File "/Users/sample/.virtualenvs/welcome-bot/lib/python3.8/site-packages/botbuilder/core/bot_adapter.py", line 109, in run_pipeline
    return await self._middleware.receive_activity_with_status(
  File "/Users/sample/.virtualenvs/welcome-bot/lib/python3.8/site-packages/botbuilder/core/middleware_set.py", line 69, in receive_activity_with_status
    return await self.receive_activity_internal(context, callback)
  File "/Users/sample/.virtualenvs/welcome-bot/lib/python3.8/site-packages/botbuilder/core/middleware_set.py", line 79, in receive_activity_internal
    return await callback(context)
  File "/Users/sample/work-python/welcome-bot/bots/welcome_user_bot.py", line 41, in on_turn
    await super().on_turn(turn_context)
  File "/Users/sample/.virtualenvs/welcome-bot/lib/python3.8/site-packages/botbuilder/core/activity_handler.py", line 64, in on_turn
    await self.on_message_activity(turn_context)
  File "/Users/sample/work-python/welcome-bot/bots/welcome_user_bot.py", line 86, in on_message_activity
    member = await TeamsInfo.get_member(turn_context, turn_context.activity.from_property.id)
  File "/Users/sample/.virtualenvs/welcome-bot/lib/python3.8/site-packages/botbuilder/core/teams/teams_info.py", line 175, in get_member
    return await TeamsInfo._get_member(
  File "/Users/sample/.virtualenvs/welcome-bot/lib/python3.8/site-packages/botbuilder/core/teams/teams_info.py", line 264, in _get_member
    member: TeamsChannelAccount = await connector_client.conversations.get_conversation_member(
  File "/Users/sample/.virtualenvs/welcome-bot/lib/python3.8/site-packages/botframework/connector/aio/operations_async/_conversations_operations_async.py", line 735, in get_conversation_member
    raise models.ErrorResponseException(self._deserialize, response)
botbuilder.schema._models_py3.ErrorResponseException: Operation returned an invalid status code 'Not Found'
Datetime with no tzinfo will be considered UTC.

Hi @aravindtrue,
This is error is somewhat of a red herring. The TeamsInfo class is designed to facilitate interactions for bots within Microsoft Teams. It's not intended for the emulator. If you want to get the information of the person who just joined you need to look at the on_members_added_activity. Each member object has the name, id, and other properties you'd be interested in.
I'll make a task to look into a way of cleaning up the error so it's better.

Hi @Virtual-Josh, Thank you so much for the quick response! Is there any other way to store user information other than id in the emulator?

There are many ways to store information with the bot framework. If you want to see how an list can be used you can view the bot here.
If you want deeper reading on the topic you can look here.


Cilium health probes are sent in plain-text when IPSec is enabled

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I've setup cilium 1.13.0 with IPSec security on a Kind cluster, following the Isovalent demo.  The cluster has 2 worker nodes and one control plane nod.
I used Wireshark to capture the traffic on the Kind bridge network.  Most of the traffic appears to be properly tunnelled, but the HTTP health probes exchanges between cilium agents are in plain-text.
Here are the cilium pods, their IP addresses and the nodes on which they are running:

And here is wireshark output showing plain-text HTTP request between these IP addresses:

And another wireshark screenshot showing the plain-text HTTP requests interleaved with the IPSec UDP traffic:

Cilium Version
v1.13.0
Kernel Version
Linux cilium-ipsec-worker 5.15.0-1027-oracle #33-Ubuntu SMP Fri Jan 6 16:30:21 UTC 2023 x86_64 GNU/Linux
Kubernetes Version
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.3", GitCommit:"434bfd82814af038ad94d62ebe59b133fcb50506", GitTreeState:"clean", BuildDate:"2022-10-25T19:35:11Z", GoVersion:"go1.19.2", Compiler:"gc", Platform:"linux/amd64"}
Sysdump
cilium-sysdump-20230215-153737.zip
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

That looks expected. Cilium agent pods are running in the host-network namespace. With IPsec, only traffic between Cilium-managed pods (i.e., pods running in their own network namespaces) is encrypted.

I am trying to ensure all traffic between kubernetes nodes is encrypted.  Is there a way to encrypt the cilium agent heartbeats?  At this point it appears to be the only plain-text traffic in my cluster.

No, with IPsec only pod-to-pod traffic is encrypted. In v1.14, WireGuard will support node encryption in addition to the usual pod-to-pod encryption.


Compile issue with boost in Ubuntu 18.04 and 16.04....

Hello Justin and Marpe, I should have been a little more clear on the previous post, issue #1040. Working nights and all.
So I have boost version 1.67 - 1.73 installed on both machines and i direct the "./configure" to the appropriate boost version, as well as openssl-1.1.1g.
I get the same compile error complaining about "boost.......deprecated..." on both machines of different OSes(18.04 and 16.04).
What boost version are you using to compile? I'll just install that one to see if I can get it working.
I may not respond right away cause of the night shift work. So let me know of what possible solutions you have for the boost problem even on the "Ubuntu 18.04".
Thanks and keep the good work going.

Could you maybe add the specific commands you used and what the log showed you. Otherwise I would have to blindly guess what's wrong there.

Hello Marpme, my config setting are below:
./configure LDFLAGS="-L${BDB_PREFIX}/lib/" CPPFLAGS="-I${BDB_PREFIX}/include/" --enable-hardening --with-boost-libdir=/home/joe/boost_1_73_0/stage/lib --with-boost=/home/joe/boost_1_73_0/ LDFLAGS="-L/home/joe/openssl-1.1.1g/" CPPFLAGS="-I/home/joe/openssl-1.1.1g/include/openssl/"

Could you try to use your local openssl version that is coming with your current OS since the boost linkage looks fine and I remember there are incompatibilities with latest openssl

I have tried these versions of openssl-1.1.1, 1.1.d, 1.1.1g. That is why i'm stumped. I checked to see what was the original openssl version for 18.04, I found a reference to the openssl version being v1.1.1. I had tried that and the make fails.
If it is different version, let me know the openssl version and i'll give it a try.  Better yet, what does the version you use say?
I'll try that.
Thanks.

I have tried the following './configure' below removing the OpenSSL, the error I get complains about the SSL.
./configure LDFLAGS="-L${BDB_PREFIX}/lib/" CPPFLAGS="-I${BDB_PREFIX}/include/" --enable-hardening --with-boost-libdir=/home/lazarus/boost_1_73_0/stage/lib --with-boost=/home/lazarus/boost_1_73_0/
In file included from src/lib/tls/tortls_openssl.c:61:0:
./src/lib/tls/tortls_internal.h:55:8: error: conflicting types for ‘SSL_SESSION_get_master_key’
size_t SSL_SESSION_get_master_key(struct ssl_session_st *s,
^
In file included from src/lib/tls/tortls_openssl.c:48:0:
/usr/local/include/openssl/ssl.h:2007:15: note: previous declaration of ‘SSL_SESSION_get_master_key’ was here
__owur size_t SSL_SESSION_get_master_key(const SSL_SESSION *sess,
^
src/lib/tls/tortls_openssl.c: In function ‘tor_tls_client_is_using_v2_ciphers’:
src/lib/tls/tortls_openssl.c:931:20: error: dereferencing pointer to incomplete type ‘SSL_SESSION {aka struct ssl_session_st}’
ciphers = session->ciphers;
^
src/lib/tls/tortls_openssl.c: At top level:
src/lib/tls/tortls_openssl.c:1579:1: error: conflicting types for ‘SSL_get_client_random’
SSL_get_client_random(SSL *s, uint8_t *out, size_t len)
^
In file included from src/lib/tls/tortls_openssl.c:48:0:
/usr/local/include/openssl/ssl.h:2003:15: note: previous declaration of ‘SSL_get_client_random’ was here
__owur size_t SSL_get_client_random(const SSL *ssl, unsigned char *out,
^
In file included from ./src/lib/crypt_ops/crypto_rand.h:16:0,
from src/lib/tls/tortls_openssl.c:31:
src/lib/tls/tortls_openssl.c: In function ‘SSL_get_client_random’:
src/lib/tls/tortls_openssl.c:1584:15: error: dereferencing pointer to incomplete type ‘SSL {aka struct ssl_st}’
tor_assert(s->s3);
^
./src/lib/cc/compat_compiler.h:145:49: note: in definition of macro ‘PREDICT_LIKELY’
#define PREDICT_LIKELY(exp) builtin_expect(!!(exp), 1)
^
./src/lib/log/util_bug.h:105:7: note: in expansion of macro ‘ASSERT_PREDICT_LIKELY’
if (ASSERT_PREDICT_LIKELY(expr)) {
^
./src/lib/log/util_bug.h:102:26: note: in expansion of macro ‘tor_assertf’
#define tor_assert(expr) tor_assertf(expr, NULL)
^
src/lib/tls/tortls_openssl.c:1584:3: note: in expansion of macro ‘tor_assert’
tor_assert(s->s3);
^
src/lib/tls/tortls_openssl.c: At top level:
src/lib/tls/tortls_openssl.c:1592:1: error: conflicting types for ‘SSL_get_server_random’
SSL_get_server_random(SSL *s, uint8_t *out, size_t len)
^
In file included from src/lib/tls/tortls_openssl.c:48:0:
/usr/local/include/openssl/ssl.h:2005:15: note: previous declaration of ‘SSL_get_server_random’ was here
__owur size_t SSL_get_server_random(const SSL *ssl, unsigned char *out,
^
src/lib/tls/tortls_openssl.c:1605:1: error: conflicting types for ‘SSL_SESSION_get_master_key’
SSL_SESSION_get_master_key(SSL_SESSION *s, uint8_t *out, size_t len)
^
In file included from src/lib/tls/tortls_openssl.c:48:0:
/usr/local/include/openssl/ssl.h:2007:15: note: previous declaration of ‘SSL_SESSION_get_master_key’ was here
__owur size_t SSL_SESSION_get_master_key(const SSL_SESSION *sess,
^
Makefile:16641: recipe for target 'src/lib/tls/src_lib_libtor_tls_a-tortls_openssl.o' failed
make[2]: *** [src/lib/tls/src_lib_libtor_tls_a-tortls_openssl.o] Error 1
make[2]: Leaving directory '/home/joe/coinz/verge-6.0.2Stealth_4.3.5/src/tor'
Makefile:6091: recipe for target 'all' failed
make[1]: *** [all] Error 2
make[1]: Leaving directory '/home/joe/coinz/verge-6.0.2Stealth_4.3.5/src/tor'
Makefile:786: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1

The next step is for me to try and compile the code under "Virtualbox" only updating and Install the boost version(1.73) and see what the results are.
I'll do my best to get it done quickly to let you know if it passes or fails.
Thanks for patience.

on a fresh distribution, you will have no problems if you:
sudo apt-get update && sudo apt-get install software-properties-common && sudo add-apt-repository ppa:bitcoin/bitcoin

sudo apt-get update && sudo apt-get install libdb4.8-dev libdb4.8++-dev build-essential libtool autotools-dev automake pkg-config bsdmainutils python3 libssl-dev libevent-dev libboost-all-dev zlib1g-dev libseccomp-dev libcap-dev libncap-dev && git clone https://github.com/vergecurrency/VERGE.git && cd VERGE && ./autogen.sh && ./configure --disable-bench --disable-tests && make

Hello Justin and Marpme, the following was discovered during a recompile of updated Verge with Tor-4.3.5.
So the verge(with tor-4.3.5) compiled fine under VirtualBox with out of the compilation. This yielded Boost-1.65 and openssl-1.1.1.
I had tried to compile verge(with tor-4.3.5) under VirtualBox with Boost-1.73.0. It seems there is an issue with the Boost-1.73.0.
when I run the ./configure it complains about boost::sleep.
OpenSSL-1.1.1g works fine under compilation in VirtualBox.
I had tried to compile Boost-1.73.0 with the following various settings listed below, there are other ./configure settings I could try and will later. (Keep in mind I had also tried to compile boost with Python-3.8.3).
tried the following for _**boost-1.73.0:
./configure --with-libraries=all --with-python=/home/joe/Python-3.8.3/python
--->> Compiles fine some complaints about the "Deprecation", fails for Verge(tor-4.3.5).
./configure --with-libraries=all --with-python=python3 (System version 3.6)
--->> Compiles fine some complaints about the "Deprecation", fails for Verge(tor-4.3.5).
./configure --with-libraries=all
--->> Compiles fine(unsure if complaints about "Deprecation", fails for Verge(tor-4.3.5).**_
For the above I have tried to compile with and without OpenSSL-1.1.1g.
So concluding factor for VirtualBox compilation is Verge(tor-4.3.5) compiles fantastically with Ubuntu-18.04 Boost-1.65 and with OpenSSL-1.1.1g.
I don't know if VirtualBox is the issue for Boost-1.73.0.
Problem with my Ubuntu-18.04 bare-metal install is I had over written my Python3 with Python-3.8.3 and my Boost-1.65 with Boost-1.73.0.
I know it sounds weird. In any case, maybe this info could be of help on troubleshoot issues you or other may have.
I have the new Verge(tor-4.3.5) copied over to my bare-metal and seems to be running fine. I get peer connects and block updates.
Thanks for your input and help.


Rename "break" methods to "brake"

Ideally we don't want the robot to break, we want it to brake.

Why is this still open?


chore: move swappers into packages

Description
Move the swappers into @shapefshiftoss/swapper with the goal of maintaining a clear boundary between API abstractions and react.
What this includes

New package @shapeshiftoss/utils which contains many of the shared functions between web and swappers
Moving of files to @shapeshiftoss/swapper and @shapeshiftoss/utils
Moving of functions to /packages to @shapeshiftoss/swapper and @shapeshiftoss/utils
Updated imports in web to point to new locations
Refactoring of swappers and tests to include the dependency injection of deps from web

What this excludes
There are many opportunities to clean things up made much clearer in this PR, which are deliberately not actioned here due to the obscene surface area of the changes. We should follow up with some analysis to generate follow-up tasks once this PR is merged.
Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #5739
Risk

High Risk PRs Require 2 approvals

High risk.

What protocols, transaction types or contract interactions might be affected by this PR?

All protocols supported for swapping
All thorchain transaction types apart from sends, including swaps LP staking defi etc
rFOX arbitrum bridge

Testing
This PR will require some serious testing to ensure we dont introduce a production issue:

[ ] All trade types of LIFI

[ ] swaps
[ ] cross chain
[ ] multi-hop
[ ] affiliate fees
[ ] slippage

[ ] CoW swap trades

[ ] swaps
[ ] affiliate fees
[ ] slippage

[ ] 0x trades

[ ] swaps
[ ] affiliate fees
[ ] slippage

[ ] All trade types of THORChain

[ ] swaps on UTXO, Cosmos, EVM
[ ] streaming swaps on UTXO, Cosmos, EVM
[ ] affiliate fees
[ ] slippage

[ ] rFOX Arbitrum bridge
[ ] LP deposits
[ ] LP withdrawals
[ ] Lending
[ ] THORChain savers

Engineering
Pay particular attention to:

[ ] L1 gas estimation in lifi was migrated from ethers to viem
[ ] Cowswap react-app-rewired changes to silence a webpack warning which should be benign

Operations

Screenshots (if applicable)
THORChain swap

THORChain streaming swap

0x trade

LIFI trade

CoW swap trade

rFOX bridge

rFOX stake

LP withdrawal

LP consolidation and deposit

Runtime tests

[x] All trade types of LIFI

[x] swaps
[x] cross chain
[x] multi-hop
[x] affiliate fees
[x] slippage

[x] CoW swap trades

[x] swaps
[x] affiliate fees
[x] slippage

[x] 0x trades

[x] swaps
[x] affiliate fees
[x] slippage

[ ] All trade types of THORChain

[x] swaps on UTXO, Cosmos, EVM
[ ] streaming swaps on UTXO, Cosmos, EVM
[x] affiliate fees
[x] slippage
[x] longtail

[x] rFOX Arbitrum bridge
[x] LP deposits
[x] LP withdrawals
[ ] Lending (can't test)
[x] THORChain savers

Li.Fi Allbridge
ETH.ARB => AVAX, seems good and

Multihop

Thorchain longtail

Cowswap quote

0x

LP deposit

LP Withdraw

Thorchain savers

rFOX bridge

Thorchain AVAX => BTC

Thorchain AVAX => BCH

Inbound: https://avascan.info/blockchain/c/tx/0xA2AC2BE10AC3F3E1368975F7B20769D8BAA9E776E51E852F90F5D88E0AFCB3B8
Outbound: https://viewblock.io/thorchain/tx/A2AC2BE10AC3F3E1368975F7B20769D8BAA9E776E51E852F90F5D88E0AFCB3B8
I'll leave streaming swaps for the ops but memo looks fine!
LGTM at runtime!

@NeOMakinG

Weird that we didn't have this warning before, wait, did we have it? What made it out loud?

Its due to the different compiler options of packages vs web, where webpack is unable to interpret the import as an expression unlike the compiled output of web which uses different configuration


Printing Longest Common Subsequence

Technique to solve printing problems involving dp.

According to hacktoberfest 2022 rules you can only make one valid PR in any specific repository, so request you to please not make a lot of contribution in a single repository otherwise it will get marked as spam. Hope you can understand it and will keep in mind from ahead.


Icons are not working

Hi!
I am not an English speaker so my English is not too good, and, i am new at vusax and i was trying to use icons but i cannot use them, i have imported cdn from awesome fonts, google icons but nothing
this is was is suppose to show:

This is what my computer shows, in opera gx

this is my code:

I do not know what i am doing wrong, heeeelp pleaseeee 😞

From the Setting up documentation it is mentioned that you need to install by yourself since Material icons used by this framework are currently not included. I suggest you go check it out if you haven't.
npm install material-icons --save
then import 'material-icons/iconfont/material-icons.css';  in when setting up your project

Added import 'material-icons/iconfont/material-icons.css';  to my main.js file but it didn't work still.


Node Header: MAX_PREFIX_LEN & uint8_t num_children

Hello!
ATM I'm trying to get into the same paper you implemented here and noticed a problem:

You defined the node header with a contant size of 16 Byte like the paper.
The paper proposed to store a compressed path of 8 Bytes whilst your implementation defines it through MAX_PREFIX_LEN as 10 Bytes.
Due to the necessity to store a compressed path of 10 Bytes & it's length (4 Bytes), you only have 2 Bytes left to store the node type & the amount of non null children (num_children).

Problem:
I think num_children should be an uint16_t since at most we can store 256 children. Thus num_children should cover the integer range 0..256 (257 numbers) which don't fit into an uint8_t.

This is related with https://github.com/armon/libart/issues/34

House keeping: Related issue seems to be closed


[RFC] Translate-able Prefabs

The intention of this RFC is to define a way, how translation functionality can be integrated into prefabs.
rendered

I think this is nothing that needs an RFC. Would you be willing to create a PR for this @minecrawler?


Fixed a few typos in practical3

Fixed some minor typos in practical3.

This one's also causing merge conflicts...

Sorry about this Avishkar! I find collaborating on notebooks a bit difficult. If the changes aren't made on the raw metadata it is hard to see what changed and it causes these merge conflicts even if you worked on entirely different sections of the code. Do you maybe know of a better way I could do this? For now I will reset the branch to the current master and redo the changes then create another pull request. :)


Docs: Href tags not clickable in certain areas

In the Comparison to SharePoint and Microsoft 365 PowerShell docs, the redirection to the M365 CLI command is not always clickable.
Example
I can click anywhere on spo user get, but for spo user list only the letter s will redirect you to the correct page.
The part in red will not act as a hyperlink.

That's awkward and certainly not intentional. Let's see how we can fix it. Thank you for reporting this @Abderahman88 👏

I think that's the problem:

It seems that the table exceeds the body area and spills under the sidebar which blocks clicking.

One way to fix it would be to add overflow-x: auto to the body container so that it can be scrolled horizontally:

Doesn't look awesome, but definitely more usable.
@StfBauer, do you happen to have other ideas?

I will check once it is back online ;)

@StfBauer would you mind using:
https://pnp.github.io/cli-microsoft365/about/comparison-powershell.html that one should work :D

Noticed it too. Not sure what's wrong and if it's GitHub pages or us 😕

It's the trailing slash at the end of the URL that causes the issue

Still, it's odd, because it used to work just fine. I'm also seeing that all links point to .html rather than extension-less URLs, so there is something off

Locally, docs work just as intended so you'd almost think there is something wrong with GitHub pages

I think it's working again. Can't reproduce the issue

Probably it was a Github pages issue. They had some issues a couple of days ago.

@Abderahman88, still all your URLs are .html rather than the intended /

This is the issue you are having with the overflow:

there is a margin there which overlaps. Interesting enough the sidebar is empty.
@media only screen and (min-width: 76.25em) {
    .md-sidebar--secondary {
        margin-left: 122rem;
    }
}

So even if empty then it has a margin-left which then still overlaps the content. You could hide it with an inline style and set it to display: none and detect the element with a query like this
var isSideBarEmpty = document.querySelector(".md-nav.md-nav--secondary");

if(isSideBarEmpty && isSidebarEmpty.textContent.trim() === ""){
    var sideBar = isSideBarEmpty.closest(".md-sidebar.md-sidebar--secondary");
    sideBar.style.display = "none";
}

CSS Setup itself seems to be a bit ancient, no one would do something like this not even if you have to support IE11 still ;)

@waldekmastykarz I presume they (Github pages) were updating the platform, while we were testing. Everything is working again and all URL's are .html

@Abderahman88 by design URLs should have no extension, so the fact that they have the .html extension means that something's broken.
@StfBauer thanks for looking into it. If we're to change the sidebar behavior, wouldn't that affect other pages that do use it?

@waldekmastykarz Well. the idea is to hide what is not necessary. When the secondary sidebar navigation is empty, no need to have it visible on the page. In Jekyll, I would simply define a different template without a sidebar.
Haven't looked into this template for a while. Seems like the latest version handle of the theme handles it a bit different now.
So you could upgrade and test or use this simple JS.

Just confirming that this is still an issue and it wasn't caused by changes on the GitHub side.
I'll look into both the template and upgrade options.
I think it would be a good idea however to to investigate the impact of upgrading the version of MKDocs Material used for our docs, as we currently use 3.1.0 and the latest is 7.2.6, we can then include this in our monthly check of updating dependencies so we don't fall behind.

I think it would be a good idea however to to investigate the impact of upgrading the version of MKDocs Material used for our docs, as we currently use 3.1.0 and the latest is 7.2.6, we can then include this in our monthly check of updating dependencies so we don't fall behind.

I had a look at it a while back. One of the biggest changes are tabs and we'd need to rework all of them. I've started the work (#2498) but put in on-hold to focus on more urgent tasks needed for current releases.

Looks like this https://github.com/squidfunk/mkdocs-material/issues/922 is relevant to this issue and looks like we may need to add something custom to fix this specific table overflow problem, unless we can think of an alternative way of representing the information on the page.

I think this is fixed since we updated docs to a newer version of MkDocs


Feature/main page last edit

Made the changes from Inna's last review. Changed font-sizes from px to rems. Design of home page - finished.

Glad to hear! Thank you!

Great job!


Add auto-completion spec conformance to XMLNmespaceDeclaration statement

Description:
$title

Completion items and order of sorting in XMLNamespaceDeclarationStatements are inline with the spec. Hence closing the issue.


Fix npm package versions with package-lock.json

For reproduction, these are clean-room builds inside of a debian/stretch container with nodejs 8.7.0 installed. Which nodejs version are you using?

On my workstation
npm --version
5.4.0
node --version
v7.10.1

On Travis
$ node --version
v7.10.1
$ npm --version
4.2.0

And both work.

Your package-lock.json causes a failure earlier in the npm install process (chromedriver). master actually makes it further.
It seems the error occurs only with --only=dev (as recommended in the documentation), presumably because bluebird is not listed as a devDependency.

Hmmm, this is interesting since the --only=dev is supposed to be tested in Travis. Unfortunately it uses previously downloaded packages from npm install command. This is clearly a bug. I will fix it soon.

Thank you! Either way, I would recommend rolling back the package-lock.json change, as I suspect this will have very unexpected behavior.

Thank you for troubleshooting. I think package-lock.json should be in source code anyway. It did not impact any of the tests in Travis so I guess it's ok not to revert it. I prefer correcting unexpected behaviors if they appear.